fix: disable upload_path and upload_url_path options

carlalexander · carlalexander · commit d98b2fe9cb62 · 2025-07-04T11:45:23.000-04:00
diff --git a/src/Subscriber/UploadsSubscriber.php b/src/Subscriber/UploadsSubscriber.php
@@ -89,12 +89,22 @@ public static function getSubscribedEvents(): array
     {
         return [
             'pre_wp_unique_filename_file_list' => ['getUniqueFilenameList', 10, 3],
+            'pre_option_upload_path' => 'disableOption',
+            'pre_option_upload_url_path' => 'disableOption',
             'upload_dir' => 'replaceUploadDirectories',
             'upload_size_limit' => 'overrideUploadSizeLimit',
             '_wp_relative_upload_path' => ['useFileManagerForRelativePath', 10, 2],
         ];
     }
 
+    /**
+     * Disable the option by returning an empty string.
+     */
+    public function disableOption(): string
+    {
+        return '';
+    }
+
     /**
      * Get the files used for wp_unique_filename() to prevent performance issues with scandir in large directories.
      */
diff --git a/tests/Unit/Subscriber/UploadsSubscriberTest.php b/tests/Unit/Subscriber/UploadsSubscriberTest.php
@@ -71,6 +71,8 @@ public function testGetSubscribedEvents()
 
         $subscribedEvents = [
             'pre_wp_unique_filename_file_list' => ['getUniqueFilenameList', 10, 3],
+            'pre_option_upload_path' => 'disableOption',
+            'pre_option_upload_url_path' => 'disableOption',
             'upload_dir' => 'replaceUploadDirectories',
             'upload_size_limit' => 'overrideUploadSizeLimit',
             '_wp_relative_upload_path' => ['useFileManagerForRelativePath', 10, 2],
