Sort KitList by HW type

That way the grouping makes more sense.

Author: Vogtinator

kitmodel.cpp

@@ -54,11 +54,15 @@ QDataStream &operator>>(QDataStream &in, KitModel &kits)
     unsigned int version;
     in >> version;
 
+    kits.beginResetModel();
+
     if(version == 1)
         in >> kits.kits >> kits.nextID;
     else
         qWarning() << "Unknown KitModel serialization version " << version;
 
+    kits.endResetModel();
+
     return in;
 }
 

kitmodel.h

@@ -14,8 +14,8 @@ class KitModel : public QAbstractListModel
     Q_OBJECT
 public:
     enum Role {
+        NameRole = Qt::DisplayRole,
         IDRole = Qt::UserRole + 1,
-        NameRole,
         TypeRole,
         FlashRole,
         Boot1Role,
@@ -32,7 +32,7 @@ class KitModel : public QAbstractListModel
     Q_INVOKABLE virtual QVariant data(const QModelIndex &index, int role = Qt::DisplayRole) const override;
     Q_INVOKABLE bool setData(const QModelIndex &index, const QVariant &value, int role = Qt::EditRole) override;
     Q_INVOKABLE bool copy(const int row);
-	Q_INVOKABLE bool removeRows(int row, int count, const QModelIndex &) override;
+    Q_INVOKABLE bool removeRows(int row, int count, const QModelIndex &parent = QModelIndex()) override;
     Q_INVOKABLE void addKit(QString name, QString boot1, QString flash, QString snapshot_path);
     Q_INVOKABLE QModelIndex indexForID(const unsigned int id) const;
     Q_INVOKABLE bool allKitsEmpty() const;

qml/Firebird/UIComponents/KitList.qml

@@ -17,11 +17,6 @@ Rectangle {
         width: 1
     }
 
-    QSortFilterProxyModel {
-        id: sortedModel
-        sortRole: KitModel.TypeRole
-    }
-
     ScrollView {
         anchors.margins: parent.border.width
         anchors.fill: parent
@@ -37,6 +32,12 @@ Rectangle {
             highlightMoveDuration: 50
             highlightResizeDuration: 0
 
+            QSortFilterProxyModel {
+                id: sortedModel
+                sortRole: KitModel.TypeRole
+                Component.onCompleted: Emu.sortProxyModel(sortedModel, 0)
+            }
+
             model: sortedModel
 
             highlight: Rectangle {
@@ -45,11 +46,13 @@ Rectangle {
             }
 
             delegate: Item {
-                property variant myData: model
+                property var myData: model;
 
                 height: item.height + 10
                 width: listView.width - listView.anchors.margins
 
+                property int kitIndex: sortedModel.mapToSource(sortedModel.index(index, 0)).row
+
                 MouseArea {
                     anchors.fill: parent
                     onClicked: function() {
@@ -73,7 +76,6 @@ Rectangle {
                     id: item
                     width: parent.width - 15
                     anchors.centerIn: parent
-
                     kitName: name
                     flashFile: Emu.basename(flash)
                     stateFile: Emu.basename(snapshot)
@@ -90,7 +92,7 @@ Rectangle {
                     text: qsTr("Remove")
                     visible: parent.ListView.view.currentIndex === index && parent.ListView.view.count > 1
                     onClicked: {
-                        kitModel.removeRows(index, 1)
+                        kitModel.removeRows(kitIndex, 1)
                     }
                 }
 
@@ -107,7 +109,7 @@ Rectangle {
                     text: qsTr("Copy")
                     visible: parent.ListView.view.currentIndex === index
                     onClicked: {
-                        kitModel.copy(index)
+                        kitModel.copy(kitIndex)
                     }
                 }
             }

qmlbridge.cpp

@@ -483,7 +483,14 @@ bool QMLBridge::saveDialogSupported()
         return QVersionNumber::fromString(QString::fromUtf8(qVersion())) < QVersionNumber(5, 13);
     #else
         return true;
-    #endif
+#endif
+}
+
+void QMLBridge::sortProxyModel(QObject *model, int column)
+{
+    auto *sortfilterproxymodel = qobject_cast<QSortFilterProxyModel*>(model);
+    if(sortfilterproxymodel)
+        sortfilterproxymodel->sort(column);
 }
 
 void QMLBridge::speedChanged(double speed)

qmlbridge.h

@@ -96,6 +96,8 @@ class QMLBridge : public QObject
 
     Q_INVOKABLE bool saveDialogSupported();
 
+    Q_INVOKABLE void sortProxyModel(QObject *model, int column);
+
     void setActive(bool b);
 
     void notifyButtonStateChanged(int row, int col, bool state);