Store items directly in listModel when parsing them

No need to use a custom list in Python and update the ListModel from that. This is much simpler and more efficient.

Contributes to issue CURA-8556.

plugins/Marketplace/PackageList.py

@@ -31,10 +31,11 @@ class PackageList(ListModel):
     def __init__(self, parent: "QObject" = None):
         super().__init__(parent)
 
-        self._packages: List[PackageModel] = []
         self._is_loading = True
         self._scope = JsonDecoratorScope(UltimakerCloudScope(CuraApplication.getInstance()))
 
+        self.addRoleName(self.PackageRole, "package")
+
         self.requestFirst()
 
     def requestFirst(self) -> None:
@@ -82,8 +83,7 @@ class PackageList(ListModel):
 
         for package_data in response_data["data"]:
             package = PackageModel(package_data, parent = self)
-            self._packages.append(package)
+            self.appendItem({"package": package})  # Add it to this list model.
-        self._update()
 
     def _onError(self, reply: "QNetworkReply", error: Optional["QNetworkReply.NetworkError"]) -> None:
         """
@@ -92,7 +92,3 @@ class PackageList(ListModel):
         :param error: The error status of the request.
         """
         pass  # TODO: Handle errors.
-
-    def _update(self) -> None:
-        # TODO: Get list of packages from Marketplace class.
-        pass

plugins/Marketplace/resources/qml/Plugins.qml

@@ -5,7 +5,7 @@ import QtQuick 2.15
 import QtQuick.Controls 2.15
 import Cura 1.7 as Cura
 
-Item
+Column
 {
     Repeater
     {