tec: Clean empty directories when cleaning media

src/cli/Media.php

@@ -93,6 +93,14 @@ public function clean(): mixed
             if (!$file_names_to_delete) {
                 yield Response::text(200, "Nothing to delete under {$subdir_name}/.");
             }
+
+            yield Response::text(200, "Cleaning empty dirs under {$subdir_name}/...");
+
+            $this->cleanTreeEmptyDirectories("{$path_cards}/{$subdir_name}");
+            $this->cleanTreeEmptyDirectories("{$path_covers}/{$subdir_name}");
+            $this->cleanTreeEmptyDirectories("{$path_large}/{$subdir_name}");
+
+            yield Response::text(200, "Cleaned empty dirs under {$subdir_name}/.");
         }
 
         if ($count_deleted > 0) {
@@ -101,4 +109,37 @@ public function clean(): mixed
             yield Response::text(200, 'No files deleted.');
         }
     }
+
+    private function cleanTreeEmptyDirectories(string $directory): bool
+    {
+        if (!is_dir($directory)) {
+            return false;
+        }
+
+        $files = scandir($directory);
+
+        if ($files === false) {
+            return false;
+        }
+
+        $files = array_diff($files, ['.', '..']);
+
+        foreach ($files as $file) {
+            $filepath = "{$directory}/{$file}";
+
+            if (is_dir($filepath)) {
+                $can_delete_dir = $this->cleanTreeEmptyDirectories($filepath);
+            } else {
+                $can_delete_dir = false;
+            }
+
+            if (!$can_delete_dir) {
+                return false;
+            }
+        }
+
+        rmdir($directory);
+
+        return true;
+    }
 }

tests/cli/MediaTest.php

@@ -83,11 +83,16 @@ public function testCleanDeletesUnusedFiles(): void
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, "Deleted file {$image_filename}");
         $response->next();
+        $response->next();
+        $response->next();
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, '1 files deleted.');
         $this->assertFalse(file_exists($card_filepath));
         $this->assertFalse(file_exists($cover_filepath));
         $this->assertFalse(file_exists($large_filepath));
+        $this->assertFalse(file_exists(dirname($card_filepath)));
+        $this->assertFalse(file_exists(dirname($cover_filepath)));
+        $this->assertFalse(file_exists(dirname($large_filepath)));
     }
 
     public function testCleanKeepsFilesUsedByLinks(): void
@@ -127,6 +132,8 @@ public function testCleanKeepsFilesUsedByLinks(): void
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, "Nothing to delete under {$subdir_name}/.");
         $response->next();
+        $response->next();
+        $response->next();
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, 'No files deleted.');
         $this->assertTrue(file_exists($card_filepath));
@@ -171,6 +178,8 @@ public function testCleanKeepsFilesUsedByCollections(): void
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, "Nothing to delete under {$subdir_name}/.");
         $response->next();
+        $response->next();
+        $response->next();
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, 'No files deleted.');
         $this->assertTrue(file_exists($card_filepath));
@@ -215,6 +224,8 @@ public function testCleanKeepsFilesUsedByTopics(): void
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, "Nothing to delete under {$subdir_name}/.");
         $response->next();
+        $response->next();
+        $response->next();
         $this->assertResponseCode($response, 200);
         $this->assertResponseEquals($response, 'No files deleted.');
         $this->assertTrue(file_exists($card_filepath));