kastatsfavoritesmodel.cpp: prefer .value() over operator[]

Akseli Lahtinenrequested to merge
work/akselmo/ensure-itementry-value-exists into master
  • In removeResult, use resource directly so we are not comparing to a possible nullptr which can cause crashing
  • Prefer .value over operator[] to avoid leaving behind nullptr entries when reading items

BUG:482887

Edited by Akseli Lahtinen

Merge request reports