Marcel
Marcel

Reputation: 21

Python - Deleting a selected row from a QSQLRelationalTableModel / QTableView

I have a QTableView class, and a QSQLRelationalTableModel class for displaying data from within an SQLITE database.

I have subclassed these myself as I need to overwrite various methods within the PyQt classes.

I am trying to implement a 'Delete Row' function which will then automatically redraw the TableView.

Unfortunately I'm at a loss. I have overwritten the data() method within the model, as I need to modify the data that's returned for some columns. When I delete a row, this data() method obviously then has nothing to return for that row, which throws type errors (It's expecting to return data for a missing row). For the moment I have it sat in a try/except block (I know that's bad practice), and to return "Deleted" if it doesn't work.

Here is my complete model:

class PayRateTableModel(QtSql.QSqlRelationalTableModel):
    def __init__(self,master):
        QtSql.QSqlRelationalTableModel.__init__(self,db=master.db)
        self.header_labels = ['ID','From','To','Day Rate','Night Rate', 'Description']
        self.editable_columns = [1,2,3,4,5]
        self.setTable('payrates')
        self.setEditStrategy(QtSql.QSqlRelationalTableModel.OnRowChange)
        self.select()

    def columnCount(self,master,parent=QtCore.QModelIndex()):
        return 6

    def flags(self, index):
        flags=super(self.__class__, self).flags(index)
        if index.column() not in self.editable_columns:
            flags ^= QtCore.Qt.ItemIsEditable
        return flags

    def headerData(self, p_int, Qt_Orientation, role=QtCore.Qt.DisplayRole):
        if role == QtCore.Qt.DisplayRole and Qt_Orientation == QtCore.Qt.Horizontal:
            return self.header_labels[p_int]
        return QtCore.QAbstractTableModel.headerData(self,p_int, Qt_Orientation,role)

    def data(self, index, role):
        row=index.row()
        col=index.column()
        data = super(PayRateTableModel, self).data(index, role)
        if role==QtCore.Qt.TextAlignmentRole:
            return QtCore.Qt.AlignCenter
        elif role==QtCore.Qt.FontRole:
            f=QtGui.QFont
            return f
        elif role==QtCore.Qt.ForegroundRole and index.column() not in self.editable_columns:
            return QtGui.QColor(150,150,150)
        try:
            if (col in [1,2]) and (role==QtCore.Qt.DisplayRole) and data:
                data=super(PayRateTableModel, self).data(index,role)
                year=data[:4]
                month=data[5:7]
                day=data[8:10]
                data=QtCore.QDate(int(year),int(month),int(day)).toString("dd/MM/yyyy")
                return data
            elif col in [3,4] and role ==QtCore.Qt.DisplayRole:
                data=super(PayRateTableModel, self).data(index,role)
                return '%.4f' % data
        except:
            return "Deleted"

        return super(PayRateTableModel, self).data(index,role)

Here is my TableView

class PayRateTableView(QtWidgets.QTableView):
    """ Display table of editable pay rates"""
    def __init__(self, master):
        QtWidgets.QTableView.__init__(self)
        self.hideColumn(0)
        self.setSortingEnabled(True)
        self.model = PayRateTableModel(master)
        self.sortmodel=DateSortProxyModel()
        self.sortmodel.setSourceModel(self.model)
        self.setModel(self.sortmodel)
        self.sortByColumn(1,QtCore.Qt.AscendingOrder)
        self.setSelectionBehavior(QtWidgets.QTableView.SelectRows)
        self.resizeRowsToContents()
        self.resizeColumnsToContents()
        self.setAlternatingRowColors(True)
        self.horizontalHeader().setSectionResizeMode(QtWidgets.QHeaderView.Stretch)
        self.setItemDelegateForColumn(1, CalendarDelegate(master))
        self.setItemDelegateForColumn(2, CalendarDelegate(master))
        self.setItemDelegateForColumn(3, PayRateDelegate(master))
        self.setItemDelegateForColumn(4, PayRateDelegate(master))
        self.vhead=self.verticalHeader()
        self.vhead.setVisible(False)
        self.vhead.setContextMenuPolicy(QtCore.Qt.CustomContextMenu)
        self.vhead.customContextMenuRequested.connect(self.contextMenuEvent)
    def sizeHintForRow(self, p_int):
        return 20

    def contextMenuEvent(self, event):
        index = self.currentIndex()
        menu=QtWidgets.QMenu(self)
        menu.popup(QtGui.QCursor.pos())
        deleteAction = menu.addAction("Delete Row")
        action = menu.exec_(self.mapToGlobal(event.pos()))
        if action == deleteAction:
            self.model.removeRow(index.row())
            self.model.submitAll()

This actually works but although it deletes the data, it leaves half a row at the top of the table like this (Row 3 has just been deleted):

Screenshot of QTableView with one row deleted

Any changes I make that seem to successfully delete the row, make the app quit with a "Processed finished with exit code 11" which I cannot find any detailed explanation (Platform specific, I'm using PyCharm on Mac OS and Python 3.6)

Thanks

Upvotes: 0

Views: 2548

Answers (1)

Marcel
Marcel

Reputation: 21

As it turns out, I seem to have figured it out. Kind of. Although it's not a direct answer per se, I changed the deletion method from a context menu to a toolbar icon. Select row, click toolbar icon, invoke the following Class method:

def deletePayRate(self):
    index = self.currentIndex()
    deleteconfirmation = QtWidgets.QMessageBox.critical(self.parent(), "Delete row", "Really delete the selected Pay Rate?", QtWidgets.QMessageBox.Yes, QtWidgets.QMessageBox.No)
    if deleteconfirmation == QtWidgets.QMessageBox.Yes:
        self.model.removeRow(index.row())
        self.model.submitAll()
        return
    else:
        return

Simple but it works.

Upvotes: 1

Related Questions