Skip to content

Conversation

@dpanta94
Copy link
Member

@dpanta94 dpanta94 commented May 22, 2025

Adds a filter to allow external plugins to filter the group path being created

@dpanta94 dpanta94 requested review from bordoni and lucatume May 22, 2025 11:15
@dpanta94 dpanta94 self-assigned this May 22, 2025
@dpanta94 dpanta94 added the enhancement New feature or request label May 22, 2025
'prefix' => $is_using_asset_directory_prefix,
];
/**
* Allows for the group path to be filtered.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Maybe add a note here about how this handles custom wp-content folders better, so it doesn't get missed in the future?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mathetos I did add a comment explaining how to use the filter. It's a shame that the library currently at least is unable of handling such issues itself.

As you can see here when plugins (like TEC suite plugins) are using the library they have to utilize the filter to ensure symlinks are followed.

@Camwyn Camwyn requested a review from mathetos May 22, 2025 12:43
Copy link

@mathetos mathetos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks all for the quick work here!

@dpanta94 dpanta94 merged commit 0e66779 into main May 22, 2025
3 checks passed
@dpanta94 dpanta94 deleted the feat/allow-group-path-to-be-filtered branch May 22, 2025 12:59
* @param string $group_path_slug The slug of the group path.
*/
static::$group_paths[ $group_path_slug ] = apply_filters(
'stellarwp/assets/group_path',
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dpanta94 This filter is missing the hook prefix that is used by other filters in this library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants