Uploaded image for project: 'Qt'
  1. Qt
  2. QTBUG-93301

QItemSelectionModel does not emit selectionChanged when selection is changed through QStandardItem::setFlags

    XMLWordPrintable

Details

    • Bug
    • Resolution: Unresolved
    • P3: Somewhat important
    • None
    • 5.15.3
    • Core: Item Models
    • None
    • 21
    • Qt6_Foundation_Sprint 37, Qt6_Foundation_Sprint 38, Team 1 Foundation_Sprint 45, Team 1 Foundation_Sprint 46, Team 1 Foundation_Sprint 47, Team 1 Foundation_Sprint 48, Team One Foundation Sprint 49, Team One Foundation Sprint 50

    Description

      See reproducer in https://codereview.qt-project.org/c/qt/qtbase/+/346010

      When a selection is cleared by calling
      model.item(j)->setFlags({ });
      i.e. by clearing the flags, a QItemSelectionModel connected the the model should emit selectionChanged because, well, the selection changed. However, this is not the case. The model emits the signal dataChanged() in that case. This signal is not connected to by QItemSelectionModel, see
      https://github.com/qt/qtbase/blob/582fdf06d179ee17ff68c67fb2f55ac10b7b6e4a/src/corelib/itemmodels/qitemselectionmodel.cpp#L606
      Proper handling is necessary here.

      Attachments

        Issue Links

          depends on

          Bug - A problem which impairs or prevents the functions of the product. QTBUG-93829 QItemSelectionModel::hasSelection is inconsistent

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Closed

          Task - A task that needs to be done. QTBUG-100857 Implement flagsChanged signals in QSqlTableModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported

          Task - A task that needs to be done. QTBUG-100858 Implement flagsChanged signals in QUrlModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported

          Task - A task that needs to be done. QTBUG-100859 Implement flagsChanged signals in QTableModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported

          Task - A task that needs to be done. QTBUG-100860 Implement flagsChanged signals in QCalendarModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Reported

          Task - A task that needs to be done. QTBUG-99519 Implement flagsChanged signals in QDirModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Open

          Task - A task that needs to be done. QTBUG-99520 Implement flagsChanged signals in QTreeModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Open

          Task - A task that needs to be done. QTBUG-99518 Implement flagsChanged signals in QFileSystemModel

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • In Progress
          is required for

          Task - A task that needs to be done. QTBUG-92991 Port QItemSelectionModel to bindable properties

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • In Progress
          relates to

          Task - A task that needs to be done. QTBUG-99521 Benchmark impact of new flagsChanged signal

          • P3: Somewhat important - Should be fixed, but doesn't affect the release in any way.
          • Open

          Task - A task that needs to be done. QTBUG-100951 Check QSortFilterProxyModel behavior

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • In Progress

          Activity

            People

              cnn Qt Core & Network
              andreasbuhr Andreas Buhr
              Vladimir Minenko Vladimir Minenko
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

                Created:
                Updated: