Fix bug#172 - "editing a QListWidgetItem stored in a variable makes it impossible to delete"
Reviewer: Luciano Wolf <luciano.wolf@openbossa.org>
Marcelo Lira <marcelo.lira@openbossa.org>
This commit is contained in:
parent
ef35dc4cd4
commit
83f8447c95
2 changed files with 11 additions and 15 deletions
|
|
@ -2933,11 +2933,6 @@
|
||||||
<parent index="this" action="add"/>
|
<parent index="this" action="add"/>
|
||||||
</modify-argument>
|
</modify-argument>
|
||||||
</modify-function>
|
</modify-function>
|
||||||
<modify-function signature="takeTopLevelItem(int)">
|
|
||||||
<modify-argument index="return">
|
|
||||||
<parent index="this" action="add"/>
|
|
||||||
</modify-argument>
|
|
||||||
</modify-function>
|
|
||||||
<modify-function signature="addTopLevelItems(const QList<QTreeWidgetItem*> &)">
|
<modify-function signature="addTopLevelItems(const QList<QTreeWidgetItem*> &)">
|
||||||
<modify-argument index="1">
|
<modify-argument index="1">
|
||||||
<parent index="this" action="add"/>
|
<parent index="this" action="add"/>
|
||||||
|
|
@ -2965,11 +2960,7 @@
|
||||||
</modify-function>
|
</modify-function>
|
||||||
<modify-function signature="takeTopLevelItem(int)">
|
<modify-function signature="takeTopLevelItem(int)">
|
||||||
<modify-argument index="return">
|
<modify-argument index="return">
|
||||||
<parent index="this" action="remove"/>
|
<define-ownership owner="default"/>
|
||||||
</modify-argument>
|
|
||||||
</modify-function>
|
|
||||||
<modify-function signature="removeItemWidget(QTreeWidgetItem*,int)">
|
|
||||||
<modify-argument index="1">
|
|
||||||
<parent index="this" action="remove"/>
|
<parent index="this" action="remove"/>
|
||||||
</modify-argument>
|
</modify-argument>
|
||||||
</modify-function>
|
</modify-function>
|
||||||
|
|
@ -3198,11 +3189,6 @@
|
||||||
<parent index="this" action="add"/>
|
<parent index="this" action="add"/>
|
||||||
</modify-argument>
|
</modify-argument>
|
||||||
</modify-function>
|
</modify-function>
|
||||||
<modify-function signature="removeItemWidget(QListWidgetItem*)">
|
|
||||||
<modify-argument index="1">
|
|
||||||
<parent index="this" action="remove"/>
|
|
||||||
</modify-argument>
|
|
||||||
</modify-function>
|
|
||||||
<modify-function signature="setItemHidden(const QListWidgetItem*,bool)" remove="all"/>
|
<modify-function signature="setItemHidden(const QListWidgetItem*,bool)" remove="all"/>
|
||||||
<modify-function signature="isItemHidden(const QListWidgetItem*)const" remove="all"/>
|
<modify-function signature="isItemHidden(const QListWidgetItem*)const" remove="all"/>
|
||||||
<modify-function signature="setItemSelected(const QListWidgetItem*,bool)" remove="all"/>
|
<modify-function signature="setItemSelected(const QListWidgetItem*,bool)" remove="all"/>
|
||||||
|
|
@ -3210,6 +3196,7 @@
|
||||||
<modify-function signature="takeItem(int)">
|
<modify-function signature="takeItem(int)">
|
||||||
<modify-argument index="return">
|
<modify-argument index="return">
|
||||||
<define-ownership owner="default"/>
|
<define-ownership owner="default"/>
|
||||||
|
<parent index="this" action="remove"/>
|
||||||
</modify-argument>
|
</modify-argument>
|
||||||
</modify-function>
|
</modify-function>
|
||||||
<modify-function signature="mimeData(const QList<QListWidgetItem*>)const" remove="all"/>
|
<modify-function signature="mimeData(const QList<QListWidgetItem*>)const" remove="all"/>
|
||||||
|
|
|
||||||
|
|
@ -41,6 +41,15 @@ class QListWidgetTest(UsesQApplication):
|
||||||
self.assert_(sys.getrefcount(i), 2)
|
self.assert_(sys.getrefcount(i), 2)
|
||||||
del i
|
del i
|
||||||
|
|
||||||
|
def testIt(self):
|
||||||
|
lst = QtGui.QListWidget()
|
||||||
|
lst.show()
|
||||||
|
slot = lambda : lst.removeItemWidget(lst.currentItem())
|
||||||
|
lst.addItem(QtGui.QListWidgetItem("foo"))
|
||||||
|
QtCore.QTimer.singleShot(0, slot)
|
||||||
|
QtCore.QTimer.singleShot(0, lst.close)
|
||||||
|
self.app.exec_()
|
||||||
|
self.assertEqual(lst.count(), 1)
|
||||||
|
|
||||||
if __name__ == '__main__':
|
if __name__ == '__main__':
|
||||||
unittest.main()
|
unittest.main()
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue