[solved]QAbstractListModel - update data in table view [linux]

s_z_p

I checked and QWatcher works fine. Problem is related to function resetItems but I haven't found solution.. any idea?

SGaist

@AttributeError: ‘QString’ object has no attribute ‘beginResetModel’@

means that you are trying to call beginResetModel on a QString class, which is wrong.

stockdata and __stockdata don't seem to represent the same thing, you should use more meaningful names for your variables to make your code easier to understand

s_z_p

how then resetItems has to look like? in this particular case. thx

SGaist

It should be part of your subclass of QAbstractItemModel

s_z_p

what you have written means that this is wrong ?

@class StockListModel(QtCore.QAbstractListModel):
def init(self, stockdata = [], parent = None):
QtCore.QAbstractListModel.init(self, parent)
self.stockdata = stockdata

    def getItems(self):
       return stockdata

#resetItems:
def resetItems(self,stockdata):
self.beginResetModel()
self.stockdata = stockdata
self.endResetModel()

    def rowCount(self, parent):
        return len(self.stockdata)
        
    def data(self, index, role):        
        if role == QtCore.Qt.DisplayRole:
            row = index.row()
            value = self.stockdata[row]
            return value
            
    file_check = QtCore.QFileSystemWatcher(['/home/user/Desktop/plik.txt'])
    file_check.fileChanged.connect(resetItems)

if name == 'main':
app = QtGui.QApplication(sys.argv)
app.setStyle("plastique")

    tableView = QtGui.QTableView()      
    tableView.show()
 
    a = os.popen("cat /home/user/Desktop/plik.txt")
    a = a.read()
    time_variable = QtCore.QString("%s"%a)
 
    model = StockListModel([time_variable])
 
    tableView.setModel(model)
    sys.exit(app.exec_())@

and instead of QtCore.QAbstractListModel i should use QtCore.QAbstractItemModel ? or add aditional subclass?

SGaist

Shouldn't stockdata be a list ? If so, your connection doesn't make sense

s_z_p

Yes it is a list but I didn't paste the whole code. There is only one cell in this code. What do you mean "connection doesn't make sense", which part?

SGaist

fileChanged gives you the path of the file that changed, not it's content, so basically you are replacing your list by a string.

Also, why are you using the model/view approach if you only have one entry ?

s_z_p

Model view approach is needed for let's say full version of the script.

I think I get your point about string. Let me check that.

SGaist

The signal will just tell you that the file has changed, it will not reload the content for you. You'll have to parse the file again and update stockdata with what you just read from it

s_z_p

One more thing.
file_check = QtCore.QFileSystemWatcher(['/home/user/Desktop/plik.txt'])
file_check only checks if file was changed if was:

file_check.fileChanged.connect(resetItems)
it sends signal to 'resetItems' and 'resetItems' should reload data, true?
and I have to find a way how accept the string parameter emitted by fileChanged..

s_z_p

Is it better ? but I have got error like that:
AttributeError: 'StockListModel' object has no attribute 'beginResetModel'

@def init(self, stockdata = [], parent = None):
QtCore.QAbstractListModel.init(self, parent)
self.stockdata = stockdata
self.file_check = QtCore.QFileSystemWatcher(['/home/user/Desktop/file.txt'])
self.file_check.fileChanged.connect(self.resetItems)

    def getItems(self):
       return stockdata

    @QtCore.pyqtSlot(str)
    def resetItems(self, path):
       self.beginResetModel()
       ....

@

SGaist

You have to the data reloading yourself. Like in your main function.

As for beginResetModel I don't know why it's saying that

s_z_p

I cannot find a solution for this error.
@SGaist, writing "reloading data" do you mean file with data?

SGaist

Yes, that's what I mean. On you program startup you are reading that file and put it's content in the stockdata variable. When the QFileSystemWatcher triggers, you must do that parsing again.

s_z_p

Finally solved. 'beginResetModel', 'end...' and 'self.stockdata' in 'resetItems' are not needed. It can be done without it. Thank you for support.

SGaist

You're welcome !

Since you have it working now, please update the thread title prepending [solved] so that other forum users may know a solution has been found :)