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

Check QSortFilterProxyModel behavior

    XMLWordPrintable

    Details

    • Type: Task
    • Status: In Progress
    • Priority: P2: Important
    • Resolution: Unresolved
    • Affects Version/s: None
    • Fix Version/s: None
    • Component/s: Core: Item Models
    • Labels:
      None
    • Story Points:
      8
    • Sprint:
      Team One Foundation Sprint 52, Team A Foundation Sprint 53

      Description

      Assuming the following:

      • In QAbstractItemModel, only one change is allowed to be in flight at a time, i.e. rowsAboutToBeInserted/rowsInserted must not nest or overlap.
      • In one rowsAboutToBeInserted/rowsInserted change, only rows with a common parent can be inserted.
      • When changing filters in QSortFilterProxyModel, rows on multiple levels, with different parents, are inserted at once.
        If all of these three are correct, I cannot see how changing the filters can be correctly implemented in QSortFilterProxyModel.

      This item is about the following:

      • Check that the three assumptions above are correct. Make sure that the documentation clearly states whether these assumptions are correct.
      • Extend QAbstractItemModelTester to check consistency in number of rows/cols. I.e. rows are only added between rowsAboutToBeInserted and rowsInserted.
      • Apply QAbstractItemModelTester in all tests of QSortFilterProxyModel
      • Extend this to QFileSystemModel

        Attachments

          Issue Links

          No reviews matched the request. Check your Options in the drop-down menu of this sections header.

            Activity

              People

              Assignee:
              cnn Qt Core & Network
              Reporter:
              andreasbuhr Andreas Buhr
              PM Owner:
              Vladimir Minenko Vladimir Minenko
              RnD Owner:
              Alex Blasche Alex Blasche
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:

                  Gerrit Reviews

                  There are no open Gerrit changes