Uploaded image for project: 'Qt for Python'
  1. Qt for Python
  2. PYSIDE-570

Base class overloads are not working properly

    XMLWordPrintable

Details

    • Bug
    • Resolution: Done
    • P2: Important
    • 5.11.1
    • 5.6
    • Shiboken
    • I'm bilding latest PySide2 against Qt 5.6.2 on Windows
    • 4d858949958e48fdffa136688f1eb82f85439404 (pyside-setup/5.11, 26.6.2018, 5.11.1)

    Description

      Probably after fixing bug PYSIDE-331 the QFileSystemModel is broken in Pyside.
      The index function get wrongly generated overloads decisor.

      // Overloaded function decisor
// 0: index(QString,int)const
// 1: index(int,int,QModelIndex)const
// 2: index(int,int,QModelIndex)const

      with overloads 1 and 2 being the same.

      It is then impossible to use QFileSystemModel, because it then throws an error of

      PyErr_SetString(PyExc_NotImplementedError, "pure virtual method 'QAbstractItemModel.index()' not implemented.");

      I'm attaching QFileSystemModelWrapper.cpp for reference

      Attachments

        1. py570.py
          0.7 kB
        2. pyside570_diag.diff
          5 kB
        3. qfilesystemmodel_wrapper.cpp
          181 kB

        Issue Links

          relates to

          Bug - A problem which impairs or prevents the functions of the product. PYSIDE-1653 shiboken generates too many member functions

          • Not Evaluated - Not yet evaluated/prioritized
          • Closed
          resulted from

          Task - A task that needs to be done. PYSIDE-331 Fix qApp

          • P2: Important - Urgent, should be fixed, but will not stop the release.
          • Closed
          For Gerrit Dashboard: PYSIDE-570
          # Subject Branch Project Status CR V
          233275,4 Fix index() method of QAbstractItemModel-derived classes 5.11 pyside/pyside-setup Status: MERGED -2 0

          Activity

            People

              kleint Friedemann Kleint
              zwadar Roman Zawada
              Votes:
              2 Vote for this issue
              Watchers:
              4 Start watching this issue

              Dates

                Created:
                Updated:
                Resolved:

                Gerrit Reviews

                  There are no open Gerrit changes