fix: apply proper query param encoding everywhere

Author: tngraf

sw360/packages.py

@@ -1,5 +1,5 @@
 # -------------------------------------------------------------------------------
-# Copyright (c) 2024 Siemens
+# Copyright (c) 2024-2025 Siemens
 # All Rights Reserved.
 # Authors: thomas.graf@siemens.com
 #
@@ -67,7 +67,7 @@ def get_all_packages(self, name: str = "", version: str = "", purl: str = "",
         """
         fullbase_url = self.url + "resource/api/packages"
         params = {}
-        
+
         if all_details:
             params["allDetails"] = "true"
 
@@ -79,7 +79,7 @@ def get_all_packages(self, name: str = "", version: str = "", purl: str = "",
 
         if purl:
             params["purl"] = purl
-            
+
         if page > -1:
             params["page"] = str(page)
             params["page_entries"] = str(page_size)
@@ -123,7 +123,8 @@ def get_packages_by_packagemanager(self, manager: str, page: int = -1,
         if sort:
             params["sort"] = sort
 
-        resp = self.api_get(fullbase_url)
+        full_url = self._add_params(fullbase_url, params)
+        resp = self.api_get(full_url)
 
         if page == -1 and resp and ("_embedded" in resp) and ("sw360:packages" in resp["_embedded"]):
             return resp["_embedded"]["sw360:packages"]

sw360/vendor.py

@@ -52,7 +52,7 @@ def create_new_vendor(self, vendor: Dict[str, Any]) -> Dict[str, Any]:
 
         API endpoint: POST /vendors
 
-        :param vendor: the new vedor data
+        :param vendor: the new vendor data
         :type vendor: JSON vendor object
         :raises SW360Error: if there is a negative HTTP response
         """

tests/test_sw360_packages.py

@@ -202,7 +202,7 @@ def test_get_packages_by_packagemanager_with_details(self) -> None:
             adding_headers={"Authorization": "Token " + self.MYTOKEN},
         )
 
-        packages = lib.get_packages_by_packagemanager("nuget", page=1, page_size=5, sort="name%2Cdesc")
+        packages = lib.get_packages_by_packagemanager("nuget", page=1, page_size=5, sort="name,desc")
         self.assertIsNotNone(packages)
         self.assertTrue(len(packages) > 0)
         pkgs = packages["_embedded"]["sw360:packages"]
@@ -410,4 +410,4 @@ def test_delete_package_failed(self) -> None:
 
 if __name__ == "__main__":
     APP = Sw360TestPackages()
-    APP.test_get_packages_by_packagemanager()
+    APP.test_get_packages_by_packagemanager_with_details()