Remove import of license information

biigle · Sep 30, 2024 · 3ae9d6b · 3ae9d6b
1 parent 53caf84
commit 3ae9d6b
Show file tree

Hide file tree

Showing 5 changed files with 0 additions and 644 deletions.
diff --git a/src/Coco.php b/src/Coco.php
@@ -9,7 +9,6 @@ class Coco
     public Info $info;
     public array $images;
     public array $annotations;
-    public ?array $licenses;
     public array $categories;
 
     public static function getCocoUser(): User
@@ -56,15 +55,7 @@ public static function create(array $data): self
             return Category::create($categoryData);
         }, $data['categories']);
 
-        // Create the License objects
-        if (array_key_exists('licenses', $data)) {
-            $instance->licenses = array_map(function ($licenseData) {
-                return License::create($licenseData);
-            }, $data['licenses']);
-        }
-
         // validate the data consistency
-        $instance->validateLicensesInData();
         $instance->validateCategoriesInData();
         $instance->validateImagesInData();
 
@@ -82,26 +73,6 @@ public static function validate(array $data): void
         }
     }
 
-    /**
-     * Validate that all image license IDs are valid.
-     */
-    public function validateLicensesInData(): void
-    {
-        if (!isset($this->licenses)) {
-            return;
-        }
-
-        $licenseIds = array_map(function ($license) {
-            return $license->id;
-        }, $this->licenses);
-
-        foreach ($this->images as $image) {
-            if (isset($image->license) && !in_array($image->license, $licenseIds)) {
-                throw new \Exception("Invalid license ID '{$image->license}' in image '{$image->id}'");
-            }
-        }
-    }
-
     public function validateCategoriesInData(): void
     {
         $categoryIds = array_map(function ($category) {

diff --git a/src/License.php b/src/License.php
diff --git a/tests/CocoParserTest.php b/tests/CocoParserTest.php
@@ -27,13 +27,6 @@ public function testRecognizesCorrectFileWithAdditionalData()
         $this->assertTrue($parser->recognizesFile());
     }
 
-    public function testRecognizesCorrectFileWithLicense()
-    {
-        $file   = new File(__DIR__ . "/files/license-coco-import-volume.json");
-        $parser = new CocoParser($file);
-        $this->assertTrue($parser->recognizesFile());
-    }
-
     public function testRecognizesMissingCategoryFile()
     {
         $file   = new File(__DIR__ . "/files/missing-category-coco-import-volume.json");
@@ -48,13 +41,6 @@ public function testRecognizesEmptyFile()
         $this->assertFalse($parser->recognizesFile());
     }
 
-    public function testRecognizesBrokenLicenseFile()
-    {
-        $file   = new File(__DIR__ . "/files/broken-license-coco-import-volume.json");
-        $parser = new CocoParser($file);
-        $this->assertFalse($parser->recognizesFile());
-    }
-
     public function testGetMetadata()
     {
         $file   = new File(__DIR__ . "/files/full-coco-import-volume.json");

diff --git a/tests/files/broken-license-coco-import-volume.json b/tests/files/broken-license-coco-import-volume.json