Custom sortable columns in QFileSystemModel

Jo Jo

@JonB I'm thinking about how to correctly implement comparison taking into account types but have some problems. Currently my code looks like this but not workiing properly. Maybe you can suggest solution?

class CustomSortFilterProxyModel : public QSortFilterProxyModel
{
    QList<int> m_dtCols{3,4};
public:
    explicit CustomSortFilterProxyModel(QObject *parent = nullptr) : QSortFilterProxyModel(parent) {}

protected:
    bool lessThan(const QModelIndex &left, const QModelIndex &right) const override
    {
        auto model = (QFileSystemModel*)sourceModel();
        bool isLeftDir = model->isDir(left);
        bool isRightDir = model->isDir(right);

        if ((isLeftDir && isRightDir) || (!isLeftDir && !isRightDir))
        {
            auto leftName  = model->data(left,  Qt::DisplayRole);
            auto rightName = model->data(right, Qt::DisplayRole);
            const int compare = m_dtCols.contains(sortColumn()) ?
                leftName.toDateTime() < rightName.toDateTime() :
                QString::localeAwareCompare(leftName.toString(), rightName.toString());

            if(compare != 0) {
                return compare < 0;
            }
        }
        else if (isLeftDir)
        {
            return true;
        }

        return false;
    }
};

JonB

I'm not sure what "not workiing properly" means? What does not work right? Have you put a break or qDebug() statements here to see/verify what it is comparing and returning?

Jo Jo

@JonB Sorry, sorting by "Date Modified" and "Date Created" columns does not work properly (yes, i use qDebug() for tests)

UPDATE:
I think it should be implemented like this

class CustomSortFilterProxyModel : public QSortFilterProxyModel
{
    QList<int> m_dtCols{3,4};
public:
    explicit CustomSortFilterProxyModel(QObject *parent = nullptr) : QSortFilterProxyModel(parent) {}

protected:
    bool lessThan(const QModelIndex &left, const QModelIndex &right) const override
    {
        auto model = (QFileSystemModel*)sourceModel();
        bool isLeftDir = model->isDir(left);
        bool isRightDir = model->isDir(right);

        if ((isLeftDir && isRightDir) || (!isLeftDir && !isRightDir))
        {
            auto leftName  = model->data(left,  Qt::DisplayRole);
            auto rightName = model->data(right, Qt::DisplayRole);

            if (m_dtCols.contains(sortColumn()))
            {
                return (leftName.toDateTime() < rightName.toDateTime());
            }

            const int compare = QString::localeAwareCompare(leftName.toString(), rightName.toString());

            if(compare != 0) {
                return compare < 0;
            }
        }
        else if (isLeftDir)
        {
            return true;
        }

        return false;
    }
};

But any improvement is appreciated

JonB

@Jo-Jo
Well, start by checking that goes through the toDateTime() path, what their values are and that is getting it right. I don't know what is wrong just by looking at it.

Btw, check whether the QString::localeAwareCompare(0 is case insensitive or case sensitive? Windows, but not Linux, sorts case insensitively, yours might not.

You have just updated your code: I'm not sure it is any different from what you first wrote, but if it is and works that's fine.

Jo Jo

@JonB Everything worked except for the "Date Modified" column. It turned out that Qt returns this date as a QString. I had to convert this string to QDateTime. Now the code looks like this:

class CustomSortFilterProxyModel : public QSortFilterProxyModel
{
    QList<int> m_dtCols{3,4};
public:
    explicit CustomSortFilterProxyModel(QObject *parent = nullptr) : QSortFilterProxyModel(parent) {}

protected:
    bool lessThan(const QModelIndex &left, const QModelIndex &right) const override
    {
        auto model = (QFileSystemModel*)sourceModel();
        bool isLeftDir = model->isDir(left);
        bool isRightDir = model->isDir(right);

        if ((isLeftDir && isRightDir) || (!isLeftDir && !isRightDir))
        {
            auto leftName  = model->data(left,  Qt::DisplayRole);
            auto rightName = model->data(right, Qt::DisplayRole);

            //return (leftName.toDateTime() < rightName.toDateTime()); // OK, works fine for dates

            if (m_dtCols.contains(sortColumn()))
            {
                if (sortColumn() == 3)
                {
                    QDateTime dt1 = QDateTime::fromString(leftName.toString(), "M/d/yyyy h:mm AP");
                    QDateTime dt2 = QDateTime::fromString(rightName.toString(), "M/d/yyyy h:mm AP");
                    return dt1 < dt2;
                }

                return (leftName.toDateTime() < rightName.toDateTime());
            }

            const int compare = QString::localeAwareCompare(leftName.toString(), rightName.toString());

            if(compare != 0) {
                return compare < 0;
            }
        }
        else if (isLeftDir)
        {
            return true;
        }

        return false;
    }
};

However, I'm not sure that I'm setting the date format correctly. It can change depending on the OS settings, or am I wrong?

Christian Ehrlicher

@Jo-Jo said in Custom sortable columns in QFileSystemModel:

I had to convert this string to QDateTime

You already converted the QVariant to a QDateTime above - so why do you use the way through QString now again?

Jo Jo

@Christian-Ehrlicher said in Custom sortable columns in QFileSystemModel:

You already converted the QVariant to a QDateTime above - so why do you use the way through QString now again?

Because "Date Modified" column is created by Qt and Qt using QString for this column. If I convert QVariant to QDateTime, in this case comparison does not work properly

Christian Ehrlicher

Then you should debug why ... e.g. with a qDebug() statement as @JonB already wrote some time ago....

JonB

@Jo-Jo said in Custom sortable columns in QFileSystemModel:

Everything worked except for the "Date Modified" column. It turned out that Qt returns this date as a QString.

I am surprised at this, but untested. You are now talking about a "Date Modified" column inbuilt into QFileSystemModel, and not the "creation date" column you added for your birthTime()? But that is QDateTime QFileSystemModel::lastModified(). So I don't understand where you are getting any string for a datetime which you need to convert.

Jo Jo

@JonB said in Custom sortable columns in QFileSystemModel:

You are now talking about a "Date Modified" column inbuilt into QFileSystemModel, and not the "creation date" column you added for your birthTime()?

Yes. This column value is QString, not QDateTime. You may check sources: https://codebrowser.dev/qt5/qtbase/src/widgets/dialogs/qfilesystemmodel.cpp.html#808

Christian Ehrlicher

Thast's why https://doc.qt.io/qt-6/qfilesystemmodel.html#lastModified is there.

JonB

@Jo-Jo
I now see that, and see from https://codebrowser.dev/qt5/qtbase/src/widgets/dialogs/qfilesystemmodel.cpp.html#737 that it is called in the implementation of QFileSystemModel::data() for column #3, which is what I needed to know.

I also see it is formatted as a string with QLocale::ShortFormat. I don't know what that looks like, I don't know how "precise" it is (e.g. down to, say, milliseconds or likely not?), I don't know what the locale-aware formatting does with it. Putting all these together it would not surprise me if it is unsuitable for sorting, and/or for converting back (accurately) to a datetime/comparing. Whereas the birthTime() you use for your extra column is indeed a datetime type and is suitable.

Hence, as both I and @Christian-Ehrlicher have said, I would use lastModified() just as you use birthDate() for this column in your own code for data() method.

Jo Jo

@Christian-Ehrlicher @JonB agree with you guys about lastModified, thanks!