QStandardItemModel appendRow doesn't update QML TreeView

[BitteWenden]

Hey, adding an item into my subclassed QStandardItemModel works perfectly fine when using appendRow in the constructor of the model, however it works nowhere else. The items are to the model when added to the invisibleRootItem, but they are not shown in the tree. Do I need to update the TreeView manually somehow?

[BitteWenden]

No idea?

[SGaist]

Hi,

Please practice some patience, allow 24 hours before bumping your own thread. Not all people on this forum live in the same timezone as you.

Did you follow the example about [Inserting and removing rows](http://doc.qt.io/qt-5/model-view-programming.html ?

[BitteWenden]

I guess? Do I need to reimplement insertRows? If so, how does that work for my QStandardItems?

[SGaist]

What did you change in you subclass regarding QStandardItemModel ?

[BitteWenden]

Nothing apart from custom roles I use in childEntry->setData.

class ProjectTreeModel : public QStandardItemModel
{
    Q_OBJECT
public:
    explicit ProjectTreeModel(QObject *parent = 0);
    virtual ~ProjectTreeModel() = default;

    Q_INVOKABLE QString initializeModel(QString dirLoation);
    enum ProjectTreeModel_Roles
    {
        ProjectTreeModel_Role_Name = Qt::DisplayRole,
        ProjectTreeModel_Role_Type = Qt::WhatsThisRole,
        ProjectTreeModel_Role_Icon = Qt::DecorationRole
    };

    QHash<int, QByteArray> roleNames() const override;

private:
    void addChildEntry( const QString& name, const QString& type, const QString& icon, QStandardItem parent);
    void addRootEntry( const QString& name, const QString& type, const QString& icon);
    QHash<int, QByteArray> m_roleNameMapping;
};

This is my class, so as you can see there is nothing really done with it.

[SGaist]

How/where are you appending rows outside the constructor ?

[BitteWenden]

In the addRootEntry (and later in the addChildEntry where it should add a child to a given QStandardItem) in reaction to a button press.

auto rootEntry = new QStandardItem( name );

    rootEntry->setData( icon, ProjectTreeModel_Role_Icon );
    rootEntry->setData( type, ProjectTreeModel_Role_Type );
    rootEntry->setData( name, ProjectTreeModel_Role_Name );

    invisibleRootItem()->appendRow(rootEntry);

This is what it does.

[BitteWenden]

Do I need to tell my view somehow that the model got updated?

[SGaist]

I'd emit dataChanged with the new index passing your custom roles.

[BitteWenden]

I added emit dataChanged(QModelIndex(), QModelIndex(), roles); after my appendRow, which should update the whole view(?), but it still doesn't work.

[BitteWenden]

This is my whole class right now, in case that helps:

ProjectTreeModel::ProjectTreeModel(QObject *parent) :
    QStandardItemModel(parent)
{

    m_roleNameMapping[ProjectTreeModel_Role_Name] = "name_role";
    m_roleNameMapping[ProjectTreeModel_Role_Type] = "type_role";
    m_roleNameMapping[ProjectTreeModel_Role_Icon] = "icon_role";


}

void ProjectTreeModel::addRootEntry( const QString& name, const QString& type, const QString& icon)
{
    auto rootEntry = new QStandardItem( name );

    rootEntry->setData( icon, ProjectTreeModel_Role_Icon );
    rootEntry->setData( type, ProjectTreeModel_Role_Type );
    rootEntry->setData( name, ProjectTreeModel_Role_Name );

    invisibleRootItem()->appendRow(rootEntry);
    qDebug() << rootEntry;
    qDebug() << this->rowCount();

    QVector<int> vector(0);
    vector.append(ProjectTreeModel_Role_Icon);
    vector.append(ProjectTreeModel_Role_Type);
    vector.append(ProjectTreeModel_Role_Name);

    emit dataChanged(QModelIndex(), QModelIndex(), vector);
}


QHash<int, QByteArray> ProjectTreeModel::roleNames() const
{
    return m_roleNameMapping;
}

[SGaist]

AFAIK, no. You are passing two invalid indexes. If you want to trigger a refresh, give the top left and bottom right indexes of the part of the model that should get refreshed on the view.

[BitteWenden]

Okay, tried it with emit dataChanged(invisibleRootItem()->index(), rootEntry->index(), vector); which should work, but it doesn't.

[BitteWenden]

Or at least I think that should update everything from the root item to the newly created entry.

[SGaist]

How are you adding that new row outside the constructor ?

[BitteWenden]

With the appendRow() method (on the invisible root item) as mentioned earlier.

[SGaist]

I meant, where are you calling appendRow ?

[BitteWenden]

In a function that is called in response to a button click. (that function calls addRootItem multiple times)

[Orkblutt]

What about using beginInsertRows() / endInsertRow() ?