From 15f253a46aa45e2a9fa1055799fa2768ba49b9a3 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Thorbj=C3=B8rn=20Lund=20Martsum?= Date: Thu, 15 Dec 2011 16:48:51 +0100 Subject: Introduce QItemDelegate::destroyEditor virtual invoked at editor close This provides a stronger mechanism e.g when inheriting QItemDelegate. It makes some things much easier e.g avoid delete of an editor and maybe only delete depending on what the editor says itself. This introduces a new virtual function. Task-number: QTBUG-2299 Change-Id: I8410f8199775987dbacffd99e4c354fdadcdd21f Reviewed-by: Stephen Kelly --- doc/src/examples/spinboxdelegate.qdoc | 6 +- src/widgets/itemviews/qabstractitemdelegate.cpp | 15 +++- src/widgets/itemviews/qabstractitemdelegate.h | 2 + src/widgets/itemviews/qabstractitemview.cpp | 10 +-- src/widgets/itemviews/qabstractitemview_p.h | 9 ++- .../qabstractitemview/tst_qabstractitemview.cpp | 32 +++++++- .../manual/widgets/itemviews/delegate/example.cpp | 86 ++++++++++++++++++++++ .../manual/widgets/itemviews/delegate/example.pro | 2 + 8 files changed, 150 insertions(+), 12 deletions(-) create mode 100644 tests/manual/widgets/itemviews/delegate/example.cpp create mode 100644 tests/manual/widgets/itemviews/delegate/example.pro diff --git a/doc/src/examples/spinboxdelegate.qdoc b/doc/src/examples/spinboxdelegate.qdoc index c00f014afe..b36176b042 100644 --- a/doc/src/examples/spinboxdelegate.qdoc +++ b/doc/src/examples/spinboxdelegate.qdoc @@ -55,10 +55,14 @@ create an editor widget, display it at the correct location in a view, and communicate with a model. Custom delegates can also provide their own painting code by reimplementing the \c paintEvent() function. + Furthermore it is also possible to reuse (and avoid deleting) the editor + widget by reimplementing the \a destroyEditor() function. A reused widget + could be a mutable member created in the constructor and deleted in + the destructor. \section1 SpinBoxDelegate Class Implementation - Since the delegate is stateless, the constructor only needs to + Delegates are often stateless. The constructor only needs to call the base class's constructor with the parent QObject as its argument: diff --git a/src/widgets/itemviews/qabstractitemdelegate.cpp b/src/widgets/itemviews/qabstractitemdelegate.cpp index b34cfa377c..6184a5845a 100644 --- a/src/widgets/itemviews/qabstractitemdelegate.cpp +++ b/src/widgets/itemviews/qabstractitemdelegate.cpp @@ -232,7 +232,7 @@ QAbstractItemDelegate::~QAbstractItemDelegate() editor paints its own background (e.g., with \l{QWidget::}{setAutoFillBackground()}). - \sa setModelData() setEditorData() + \sa destroyEditor() setModelData() setEditorData() */ QWidget *QAbstractItemDelegate::createEditor(QWidget *, const QStyleOptionViewItem &, @@ -241,6 +241,19 @@ QWidget *QAbstractItemDelegate::createEditor(QWidget *, return 0; } + +/*! + A function called when the editor is no longer needed and should be + destroyed. The default behavior is a call to deleteLater on the editor. + It possible e.g. to avoid this delete by reimplementing this function. + + \sa createEditor() +*/ +void QAbstractItemDelegate::destroyEditor(QWidget *editor, const QModelIndex &) const +{ + editor->deleteLater(); +} + /*! Sets the contents of the given \a editor to the data for the item at the given \a index. Note that the index contains information diff --git a/src/widgets/itemviews/qabstractitemdelegate.h b/src/widgets/itemviews/qabstractitemdelegate.h index 2cb1175a22..2c51dfcac8 100644 --- a/src/widgets/itemviews/qabstractitemdelegate.h +++ b/src/widgets/itemviews/qabstractitemdelegate.h @@ -88,6 +88,8 @@ public: const QStyleOptionViewItem &option, const QModelIndex &index) const; + virtual void destroyEditor(QWidget *editor, const QModelIndex &index) const; + virtual void setEditorData(QWidget *editor, const QModelIndex &index) const; virtual void setModelData(QWidget *editor, diff --git a/src/widgets/itemviews/qabstractitemview.cpp b/src/widgets/itemviews/qabstractitemview.cpp index e6ba21e701..98a17a5178 100644 --- a/src/widgets/itemviews/qabstractitemview.cpp +++ b/src/widgets/itemviews/qabstractitemview.cpp @@ -1098,7 +1098,7 @@ void QAbstractItemView::reset() d->delayedReset.stop(); //make sure we stop the timer foreach (const QEditorInfo &info, d->indexEditorHash) { if (info.widget) - d->releaseEditor(info.widget.data()); + d->releaseEditor(info.widget.data(), d->indexForEditor(info.widget.data())); } d->editorIndexHash.clear(); d->indexEditorHash.clear(); @@ -2778,7 +2778,7 @@ void QAbstractItemView::closeEditor(QWidget *editor, QAbstractItemDelegate::EndE editor = ed; if (!isPersistent && editor) - d->releaseEditor(editor); + d->releaseEditor(editor, index); } // The EndEditHint part @@ -3102,7 +3102,7 @@ void QAbstractItemView::closePersistentEditor(const QModelIndex &index) closeEditor(editor, QAbstractItemDelegate::RevertModelCache); d->persistent.remove(editor); d->removeEditor(editor); - d->releaseEditor(editor); + d->releaseEditor(editor, index); } } @@ -3314,7 +3314,7 @@ void QAbstractItemView::rowsAboutToBeRemoved(const QModelIndex &parent, int star QEditorInfo info = d->indexEditorHash.take(index); i = d->editorIndexHash.erase(i); if (info.widget) - d->releaseEditor(editor); + d->releaseEditor(editor, index); } else { ++i; } @@ -3393,7 +3393,7 @@ void QAbstractItemViewPrivate::_q_columnsAboutToBeRemoved(const QModelIndex &par QEditorInfo info = indexEditorHash.take(it.value()); it = editorIndexHash.erase(it); if (info.widget) - releaseEditor(editor); + releaseEditor(editor, index); } else { ++it; } diff --git a/src/widgets/itemviews/qabstractitemview_p.h b/src/widgets/itemviews/qabstractitemview_p.h index ab2fc67ade..3826f3b4e9 100644 --- a/src/widgets/itemviews/qabstractitemview_p.h +++ b/src/widgets/itemviews/qabstractitemview_p.h @@ -200,13 +200,18 @@ public: // reimplemented in subclasses virtual void adjustViewOptionsForIndex(QStyleOptionViewItemV4*, const QModelIndex&) const {} - inline void releaseEditor(QWidget *editor) const { + inline void releaseEditor(QWidget *editor, const QModelIndex &index = QModelIndex()) const { if (editor) { QObject::disconnect(editor, SIGNAL(destroyed(QObject*)), q_func(), SLOT(editorDestroyed(QObject*))); editor->removeEventFilter(itemDelegate); editor->hide(); - editor->deleteLater(); + QAbstractItemDelegate *delegate = delegateForIndex(index); + + if (delegate) + delegate->destroyEditor(editor, index); + else + editor->deleteLater(); } } diff --git a/tests/auto/widgets/itemviews/qabstractitemview/tst_qabstractitemview.cpp b/tests/auto/widgets/itemviews/qabstractitemview/tst_qabstractitemview.cpp index 7b9d7d0a58..be7c58f20b 100644 --- a/tests/auto/widgets/itemviews/qabstractitemview/tst_qabstractitemview.cpp +++ b/tests/auto/widgets/itemviews/qabstractitemview/tst_qabstractitemview.cpp @@ -224,16 +224,29 @@ private slots: void ctrlRubberbandSelection(); void QTBUG6407_extendedSelection(); void QTBUG6753_selectOnSelection(); + void testDelegateDestroyEditor(); }; class MyAbstractItemDelegate : public QAbstractItemDelegate { public: - MyAbstractItemDelegate() : QAbstractItemDelegate() {}; + MyAbstractItemDelegate() : QAbstractItemDelegate() { calledVirtualDtor = false; } void paint(QPainter *, const QStyleOptionViewItem &, const QModelIndex &) const {} QSize sizeHint(const QStyleOptionViewItem &, const QModelIndex &) const { return QSize(); } - QWidget *createEditor(QWidget *parent, const QStyleOptionViewItem &, - const QModelIndex &) const { return new QWidget(parent); } + QWidget *createEditor(QWidget *parent, const QStyleOptionViewItem &, const QModelIndex &) const + { + openedEditor = new QWidget(parent); + return openedEditor; + } + void destroyEditor(QWidget *editor, const QModelIndex &index) const + { + calledVirtualDtor = true; + // QAbstractItemDelegate::destroyEditor(editor,index); + editor->deleteLater(); + } + + mutable bool calledVirtualDtor; + mutable QWidget *openedEditor; }; // Testing get/set functions @@ -1497,5 +1510,18 @@ void tst_QAbstractItemView::QTBUG6753_selectOnSelection() QCOMPARE(table.selectedItems().first(), table.item(item.row(), item.column())); } +void tst_QAbstractItemView::testDelegateDestroyEditor() +{ + QTableWidget table(5, 5); + MyAbstractItemDelegate delegate; + table.setItemDelegate(&delegate); + table.edit(table.model()->index(1, 1)); + TestView *tv = reinterpret_cast(&table); + QVERIFY(!delegate.calledVirtualDtor); + tv->tst_closeEditor(delegate.openedEditor, QAbstractItemDelegate::NoHint); + QVERIFY(delegate.calledVirtualDtor); +} + + QTEST_MAIN(tst_QAbstractItemView) #include "tst_qabstractitemview.moc" diff --git a/tests/manual/widgets/itemviews/delegate/example.cpp b/tests/manual/widgets/itemviews/delegate/example.cpp new file mode 100644 index 0000000000..20cff2663a --- /dev/null +++ b/tests/manual/widgets/itemviews/delegate/example.cpp @@ -0,0 +1,86 @@ +/**************************************************************************** +** +** Copyright (C) 2012 Thorbjørn Lund Martsum - tmartsum[at]gmail.com +** Contact: http://www.qt-project.org/ +** +** This file is part of the test suite of the Qt Toolkit. +** +** $QT_BEGIN_LICENSE:LGPL$ +** GNU Lesser General Public License Usage +** This file may be used under the terms of the GNU Lesser General Public +** License version 2.1 as published by the Free Software Foundation and +** appearing in the file LICENSE.LGPL included in the packaging of this +** file. Please review the following information to ensure the GNU Lesser +** General Public License version 2.1 requirements will be met: +** http://www.gnu.org/licenses/old-licenses/lgpl-2.1.html. +** +** In addition, as a special exception, Nokia gives you certain additional +** rights. These rights are described in the Nokia Qt LGPL Exception +** version 1.1, included in the file LGPL_EXCEPTION.txt in this package. +** +** GNU General Public License Usage +** Alternatively, this file may be used under the terms of the GNU General +** Public License version 3.0 as published by the Free Software Foundation +** and appearing in the file LICENSE.GPL included in the packaging of this +** file. Please review the following information to ensure the GNU General +** Public License version 3.0 requirements will be met: +** http://www.gnu.org/copyleft/gpl.html. +** +** Other Usage +** Alternatively, this file may be used in accordance with the terms and +** conditions contained in a signed written agreement between you and Nokia. +** +** +** +** +** +** +** $QT_END_LICENSE$ +** +****************************************************************************/ + +#include + +class ExampleEditor : public QLineEdit +{ +public: + ExampleEditor(QWidget *parent = 0):QLineEdit(parent) { qDebug() << "ctor"; } + ~ExampleEditor() { QApplication::instance()->quit(); } +}; + +class ExampleDelegate : public QItemDelegate +{ +public: + ExampleDelegate():QItemDelegate() { m_editor = new ExampleEditor(0); } +protected: + QWidget* createEditor(QWidget *p, const QStyleOptionViewItem &o, const QModelIndex &) const + { + m_editor->setParent(p); + m_editor->setGeometry(o.rect); + return m_editor; + } + void destroyEditor(QWidget *editor, const QModelIndex &) const + { + editor->setParent(0); + qDebug() << "intercepted destroy :)"; + } + + // Avoid setting data - and therefore show that the editor keeps its state. + void setEditorData(QWidget*, const QModelIndex &) const { } + + ~ExampleDelegate() { delete m_editor; } + mutable ExampleEditor *m_editor; +}; + +int main(int argc, char *argv[]) +{ + QApplication app(argc, argv); + QTableView tv; + QStandardItemModel m; + m.setRowCount(4); + m.setColumnCount(2); + tv.setModel(&m); + tv.show(); + tv.setItemDelegate(new ExampleDelegate()); + app.exec(); +} diff --git a/tests/manual/widgets/itemviews/delegate/example.pro b/tests/manual/widgets/itemviews/delegate/example.pro new file mode 100644 index 0000000000..e9f2189272 --- /dev/null +++ b/tests/manual/widgets/itemviews/delegate/example.pro @@ -0,0 +1,2 @@ +TEMPLATE = app +SOURCES=example.cpp -- cgit v1.2.3