Set qualifiers as a string when setting fields

pombredanne · pombredanne · commit c486db4b678b · 2018-12-21T14:41:14.000+01:00
Before this, we were saving a dict() in the qualifiers field of
PackageURLMixin resulting in damaged data (Django being smart enough
would serialize the dict() to a string. That string would not be usable
to recreate a qualifiers mapping afterwards.


Signed-off-by: Philippe Ombredanne &lt;pombredanne@nexb.com&gt;
diff --git a/src/packageurl/contrib/django_models.py b/src/packageurl/contrib/django_models.py
@@ -112,12 +112,12 @@ def package_url(self):
 
     def set_package_url(self, package_url):
         """
-        Set values for each related field of the provided `package_url` string.
-        Empty/Null values are normalized to `None` and are set as well
-        to replace any existing values.
-        This prevent mixing newly provided values with old ones.
+        Set each field values to the values of the provided `package_url` string
+        or PackageURL object. Existing values are overwritten including setting
+        values to None for provided empty values.
         """
-        purl = PackageURL.from_string(package_url)
+        if not isinstance(package_url, PackageURL):
+            package_url = PackageURL.from_string(package_url)
 
-        for field_name, value in purl.to_dict().items():
+        for field_name, value in package_url.to_dict(encode=True).items():
             setattr(self, field_name, value or None)
