manifestFiles could exist for direct but be none. fixed the check (#96)

Author: dacoburn

pyproject.toml

@@ -6,7 +6,7 @@ build-backend = "hatchling.build"
 
 [project]
 name = "socketsecurity"
-version = "2.1.16"
+version = "2.1.17"
 requires-python = ">= 3.10"
 license = {"file" = "LICENSE"}
 dependencies = [

socketsecurity/__init__.py

@@ -1,2 +1,2 @@
 __author__ = 'socket.dev'
-__version__ = '2.1.16'
+__version__ = '2.1.17'

socketsecurity/core/__init__.py

@@ -858,12 +858,11 @@ def get_source_data(package: Package, packages: dict) -> list:
         introduced_by = []
         if package.direct:
             manifests = ""
-            if not hasattr(package, "manifestFiles"):
+            if not hasattr(package, "manifestFiles") or package.manifestFiles is None:
                 return introduced_by
-            if hasattr(package, "manifestFiles"):
-                for manifest_data in package.manifestFiles:
-                    manifest_file = manifest_data.get("file")
-                    manifests += f"{manifest_file};"
+            for manifest_data in package.manifestFiles:
+                manifest_file = manifest_data.get("file")
+                manifests += f"{manifest_file};"
             manifests = manifests.rstrip(";")
             source = ("direct", manifests)
             introduced_by.append(source)