Merge pull request #160 from fosslight/max_lic

Check empty license
fosslight · Jun 11, 2024 · e07778c · e07778c
2 parents 467f89f + 1be9d52
commit e07778c
Showing 1 changed file with 9 additions and 8 deletions.
diff --git a/src/fosslight_source/_scan_item.py b/src/fosslight_source/_scan_item.py
@@ -71,15 +71,16 @@ def licenses(self, value):
         if value:
             max_length_exceed = False
             for new_lic in value:
-                if len(new_lic) > MAX_LICENSE_LENGTH:
-                    new_lic = new_lic[:MAX_LICENSE_LENGTH]
-                    max_length_exceed = True
-                if new_lic not in self._licenses:
-                    self._licenses.append(new_lic)
-                    if len(",".join(self._licenses)) > MAX_LICENSE_TOTAL_LENGTH:
-                        self._licenses.remove(new_lic)
+                if new_lic:
+                    if len(new_lic) > MAX_LICENSE_LENGTH:
+                        new_lic = new_lic[:MAX_LICENSE_LENGTH]
                         max_length_exceed = True
-                        break
+                    if new_lic not in self._licenses:
+                        self._licenses.append(new_lic)
+                        if len(",".join(self._licenses)) > MAX_LICENSE_TOTAL_LENGTH:
+                            self._licenses.remove(new_lic)
+                            max_length_exceed = True
+                            break
             if max_length_exceed and (SUBSTRING_LICENSE_COMMENT not in self.comment):
                 self.comment = f"{self.comment}/ {SUBSTRING_LICENSE_COMMENT}" if self.comment else SUBSTRING_LICENSE_COMMENT