Can't change the text display of a QTreeview item.

[johnny_sparx]

I have a model-view set up and so far, they appear synchronized. Now my problem appears when I am trying to change the text in one of the QTreeView leaves.

Given a QModelIndex, I can change the leaf's color using something like:

QAbstractItemModel *Model = const_cast<QAbstractItemModel*>(Current_Model_Index->model());
Model->setData(*Current_Model_Index,Qt::blue,Qt::TextColorRole);

QAbstractItemModel *Model = const_cast<QAbstractItemModel*>(Current_Model_Index->model());
Model->setData(*Current_Model_Index,Qt::blue,Qt::TextColorRole);

To copy to clipboard, switch view to plain text mode 

Similarly, I attempted to change the value of the text using this:

Model->setData(*Current_Model_Index,tr("XXXXXXX"),Qt::DisplayRole);

Model->setData(*Current_Model_Index,tr("XXXXXXX"),Qt::DisplayRole);

To copy to clipboard, switch view to plain text mode 

That did not work. So I checked: Model->data(*Current_Model_Index,Qt::ItemIsEditable).toB ool() and this returned false.

I tried to change this property to true but it would not hold (the values was not retained when checked). How can I achieve my goal of assigning a text value to a tree leaf given its QModelIndex?

[wysota]

Did you reimplement QAbstractItemModel::flags() to return proper flags for your items?

[johnny_sparx]

No, I did not actually.... I did not think I had to!

What is wrong with the default flags()? For that matter, shouldn't I be using something like setFlags?

[wysota]

What is wrong with the default flags()?

It depends which model you use. If you use a direct subclass of QAbstractItemModel, then it doesn't allow items to be modified. You should make sure it returns ItemIsEditable as part of the flags.

For that matter, shouldn't I be using something like setFlags?

No. The model itself should know whether a particular item is to be editable (or otherwise interactable) and should return proper flags for the item. If you want something like "setFlags", you should somehow set/change the internal state for an item and tell the model to act accordingly upon this. For example you might implement the model in such a way that it won't allow items to be editable if they contain children. It would look more or less like so:

Qt::ItemFlags MyModel::flags ( const QModelIndex & index ) const {
  if(hasChildren(index)){
    return Qt::ItemIsEnabled|Qt::ItemIsSelectable;
  } else {
    return Qt::ItemIsEnabled|Qt::ItemIsSelectable|Qt::ItemIsEditable;
  }
  return Qt::ItemIsEnabled;
}

Qt::ItemFlags MyModel::flags ( const QModelIndex & index ) const {
  if(hasChildren(index)){
    return Qt::ItemIsEnabled|Qt::ItemIsSelectable;
  } else {
    return Qt::ItemIsEnabled|Qt::ItemIsSelectable|Qt::ItemIsEditable;
  }
  return Qt::ItemIsEnabled;
}

To copy to clipboard, switch view to plain text mode 

Allowing a user to directly modify the state of any item could ruin the model.