Feature No Code Attached Yet
avatar brianteeman
brianteeman
20 Jun 2025

Is your feature request related to a problem? Please describe.

The schedule task Image Size Check lets you select the image folders to process. It is hard coded to /images as the parent and you can select a subfolder.

Describe the solution you'd like

The plugin should not be hard coded to the images folder but should support the folder defined in the plugin Filessystem-Local

Additional context

path to images should always respect the settings in the plugin and not be hardcoded anywhere

avatar brianteeman brianteeman - open - 20 Jun 2025
avatar joomla-cms-bot joomla-cms-bot - change - 20 Jun 2025
Labels Added: No Code Attached Yet
avatar joomla-cms-bot joomla-cms-bot - labeled - 20 Jun 2025
avatar alikon alikon - change - 20 Jun 2025
Labels Added: Feature
avatar alikon alikon - labeled - 20 Jun 2025
avatar richard67
richard67 - comment - 21 Jun 2025

should support the folder defined in the plugin

Folder or folders (there might be multiple).

avatar Vineet7875
Vineet7875 - comment - 22 Jun 2025

Backward-Compatible Solution Proposal: Dynamic Filesystem Root Support for

To enhance flexibility without breaking existing functionality, I propose an update to the field that optionally supports dynamic filesystem roots.

Implementation:

  • Extend the directory attribute to accept a value like @filesystem:adapter, e.g. @filesystem:local
  • If the prefix @filesystem: is detected, resolve the adapter name and fetch the root_folder or repeatable directories param from the plugin
  • Loop through each configured directory (if multiple) and merge their subfolders into the options list
  • If no match or plugin is missing, fallback to the existing hardcoded path like images

What Stays the Same:

  • If directory="images" or any static folder is provided, existing logic remains unchanged
  • No modifications required in existing XMLs, templates, or plugin configs
  • Works exactly as before for all current use cases

Benefits:

  • Adds support for dynamic folder roots based on active Filesystem plugin
  • Completely non-breaking and opt-in
  • Useful for future use cases like using S3, external media storage, or dynamic local folders

I'm happy to contribute the updated getOptions() method or submit a patch. Let me know if this can be considered for core inclusion.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45630.
avatar Vineet7875
Vineet7875 - comment - 23 Jun 2025

@brianteeman review above solution. Happy to contribute!


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/45630.

Add a Comment

Login with GitHub to post a comment