ListView delegate isnt able to access model properties after inserting into QAbstractItemModel

[Christian Ehrlicher]

@Creaperdown said in ListView delegate isnt able to access model properties after inserting into QAbstractItemModel:

oes moving the functions into the model make a difference functionality wise? Using the signal-approach should be doing exactly the same, shouldn't it?

Yes, but the user of your model must know to call begin/endFoo() for no reason - can be done directly in addBookmark() and nothing can go wrong then like e.g.:

emit bookmarkInsertionStarted(book->getBookmarks().count());
book->addBookmark(bookmark);
book->addBookmark(bookmark);
emit bookmarkInsertionEnded();

or

emit bookmarkInsertionStarted(book->getBookmarks().count());
emit bookmarkInsertionEnded();

which can be easily avoided.

But that's not the your current issue, just a bad desgin (imo).

[Creaperdown]

@Christian-Ehrlicher thanks for the recommendation, I'll be looking into that but I'm still not sure what causes me actual issue

[Christian Ehrlicher]

@Creaperdown There I can't help - don't use QML

[Creaperdown]

@Christian-Ehrlicher So you think this is a qml related problem?

[Christian Ehrlicher]

I would create a single/atomar add function to really make sure the add is correct but the rest looks good.

[JonB]

@Creaperdown
Yes indeed it is, which is why I suggested this thread might be better moved to QML forum in the first place, though never mind now.

[Creaperdown]

@JonB I'll be closing this here then and moving it to the Qml section

[Christian Ehrlicher]

Why do you delete this thread? To start over the same discussion about the possible wrong usage of begin/endInsertRows again?

[JoeCFD]

@Creaperdown in your model, do the following:

    beginResetModel();
     add new row()
    endResetModel();

[SGaist]

@JoeCFD said in ListView delegate isnt able to access model properties after inserting into QAbstractItemModel:

@Creaperdown in your model, do the following:

    beginResetModel();
     add new row()
    endResetModel();

If you need to reset the whole model because you insert one row, there's likely a design issue.

[JoeCFD]

@SGaist It is an option. I am not sure if everything is redone.

[SGaist]

@JoeCFD basically, it tells the view that everything has changed so it should reload everything as a consequence.

[JoeCFD]

@SGaist True. kind of global update. It is better to use
beginInsertRows() and endInsertRows().

[Creaperdown]

The problem was a missing reference on the container. It should have been QList<Bookmark>&, thus it didnt update the container when I add a book.