Skip to content

Use new hook Minz_HookType::ActionExecute for extending actions#436

Merged
Alkarex merged 2 commits intoFreshRSS:mainfrom
Inverle:action-execute-hook
Mar 14, 2026
Merged

Use new hook Minz_HookType::ActionExecute for extending actions#436
Alkarex merged 2 commits intoFreshRSS:mainfrom
Inverle:action-execute-hook

Conversation

@Inverle
Copy link
Member

@Inverle Inverle commented Mar 14, 2026

Solves the issue described in this TODO:

// TODO: Use another approach than class inheritance to allow multiple extensions to work with the same controller.
// phpcs:ignore Generic.Classes.DuplicateClassName.Found
class FreshExtension_auth_Controller extends FreshRSS_auth_Controller {

Depends on FreshRSS/FreshRSS#8599 and FreshRSS/FreshRSS#8600
now merged

@Inverle Inverle marked this pull request as ready for review March 14, 2026 21:54
@Inverle Inverle requested a review from Alkarex March 14, 2026 21:54
@Alkarex
Copy link
Member

Alkarex commented Mar 14, 2026

Looks like a nice improvement 👍🏻

@Alkarex Alkarex merged commit b42fc58 into FreshRSS:main Mar 14, 2026
1 check passed
@Inverle Inverle deleted the action-execute-hook branch March 14, 2026 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants