(Un)ChechkingAll from Tree View

Karoluss96

I change modelItem to takeItem, but stil didn't catch checkboxes form treeView

JonB

@Karoluss96 said in (Un)ChechkingAll from Tree View:

I change modelItem to takeItem

Why, and what do you expect this to achieve? @SGaist said you should call setCheckState() on each QStandardItem, what about that?

Karoluss96

Now i have in function checkall that:

checkBoxList= QStandardItem() 
 checkBoxList.setCheckState(Qt.Checked)

no error, but when I unchecked some boxes in TreeView they don't chacked after cliking this button

JonB

@Karoluss96
Why would you name a single QStandardItem() as checkBoxList? Won't that just confuse you and me?

You seem to be creating a QStandardItem into a local variable. I really don't know what you are doing with that item, if anything. You are supposed to be calling setCheckState() on each item in your model, which is what the view is displaying.

While I notice it, you showed originally:

        for checks in checkBoxList:
            if  checkBoxList.CheckState():
                checkBoxList.setCheckState(Qt.Checked)
                print(checkBoxList)

What is the point of a for checks in checkBoxList loop if you only access checkBoxList inside it?

Karoluss96

I need an access for data in tree View, which is make in a different function.
I'll show on attached photo:
is in polish so: "zaznacz wszystko"="check all"
odznacz wszystko ="un check all".
Right, I thought that loop will be better, but all times
but I have an error: " 'QStandardItem' object is not iterable"

JonB

@Karoluss96 said in (Un)ChechkingAll from Tree View:

but I have an error: " 'QStandardItem' object is not iterable"

Exactly, it is not. If a single QStandardItem, or a single checkbox, is named checkBoxList it's not helpful. There is no list and nothing to loop over.

I don't know what to say that @SGaist has not stated clearly in:

My suggestion is that you use that QStandardItemModel directly and go through all the items you want/need to modify their value.

calling setCheckState(Qt.Checked) on each item.

Karoluss96

yes, the variable name 'checkBoxList' is confusing, but it doesn't matter for that I need a loop for all Checkboxes as I shown in the print screen above

JonB

@Karoluss96 said in (Un)ChechkingAll from Tree View:

yes, the variable name 'checkBoxList' is confusing,

Then change it, and get rid of " " 'QStandardItem' object is not iterable"" error.

I need a loop for all Checkboxes as I shown in the print screen above

And I referred you to @SGaist's answer to that, which is what you need to do.

Karoluss96

@JonB said in (Un)ChechkingAll from Tree View:

@SGaist can you explain more what do you mean?

JonB

@Karoluss96

@SGaist said in (Un)ChechkingAll from Tree View:

When you want to change them all, go through all the items in your model and call setCheckState on them.

Do you know how to go through all the items/rows in your model? Suppose you wanted to print out (or change) the text of each item, like Kontrola atrybutowe, Kontrola 1, Kontrola 2, etc. How would you do that?
Once you have the loop for 1, presumably you know how to call setCheckState(Qt.Checked) on each item visited.

Karoluss96

If put print (QStandardItemModel()) I gain: <PyQt5.QtGui.QStandardItemModel object at 0x00000229EA8F7160>
all data in Tree View come from parsed xml (to variable tree) file (I can't publish them), by

root = tree.getroot()

and later

for grupaKontroli in root:
            parent = QStandardItem(grupaKontroli.get('name')) 
            QStandardItemModel.appendRow(parent)
            parent.setCheckable(True)
            setCheckState(Qt.Checked)

Answer for that is in function above

This function in the end of main post

JonB

@Karoluss96
No. I'm afraid you do not seem to have a grasp of models/what is going on. You might benefit from reading up on this in the documentation.

The answer to, for example, how do I set every checkbox in my model would be:

for i in range(model.rowCount()):
    model.item(i).setCheckState(Qt.Checked)

If you want to only check items in and below some one particular node you would need to recurse down its descendants. But you asked for check every item.

Karoluss96

Looks useful, pretty and short, but I still didn't see its results- It need to take data from treeView!

JonB

@Karoluss96 said in (Un)ChechkingAll from Tree View:

It need to take data from treeView!

A QTreeView does not store any data. It's a view onto a model. The model is where you store the data. @SGaist has said this, I have said this. I suggested you might like to read up on Qt model/view architecture.

SGaist

Please provide a minimal script a minimal script that creates your tree view and its content. We will then be able to reason on it properly to help you achieve your goal.

Karoluss96

@SGaist said in (Un)ChechkingAll from Tree View:

Please provide a minimal script a minimal script that creates your tree view and its content. We will then be able to reason on it properly to help you achieve your goal.

OK, so I attached it;

 def kontrolBDOTread(self): #BDOT is a name of database
        path = str(self.controlPath).replace('\\','/') + '/' + self.dlg.comboBox_4.currentText()
        
        tree = et.parse(path)#parsing XML
        root = tree.getroot()   
        wersjaSzablonuKontroli = root.get('version')
        self.dlg.label_8.setText('Wersja szablonu: '+str(wersjaSzablonuKontroli)
        model = QStandardItemModel()
        model.setHorizontalHeaderLabels(['Lista kontroli'])#Nagłówek
        self.dlg.treeView.setModel(model)
        self.dlg.treeView.setUniformRowHeights(True)
        
        i = 0
        for grupaKontroli in root:
            parent = QStandardItem(grupaKontroli.get('name')) 
            model.appendRow(parent)
            parent.setCheckable(True)
            parent.setCheckState(Qt.Checked)
            parent.setUserTristate(True)
            for kontrola in grupaKontroli:
                child = QStandardItem(kontrola.get('name'))
                child.setCheckable(True)
                child.setCheckState(Qt.Checked)
                parent.appendRow([child]) 
            self.dlg.treeView.setFirstColumnSpanned(i, self.dlg.treeView.rootIndex(), True)
            i = i + 1   
        self.dlg.treeView.show()

Karoluss96

And how its looks like

SGaist

By minimal script, I meant a script that we can run ourselves on our machines which is not possible with what you posted.

Karoluss96

Well, it's a UI plugin. It need to be a part of QGIS program (it's free on download by: https://qgis.org/pl/site/forusers/download.html)

SGaist

The fact that you need to write a plugin for QGIS is currently unrelated to your issue hence why I am requesting a minimal script that creates a QTreeView with the model and the buttons you want. With that we can fix the current issue you have and you can then update your plugin code with that.