Skip to content

Conversation

@ADmad
Copy link
Member

@ADmad ADmad commented Nov 15, 2025

No description provided.

@ADmad ADmad added this to the 7.x milestone Nov 15, 2025
Copy link
Member

@dereuromark dereuromark left a comment

Choose a reason for hiding this comment

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

Sounds reasonable.
I do the same all the time for e.g. https://github.com/dereuromark/cakephp-captcha/blob/d9a75ef5d1e2ac6934f6b1302b2b59bde61403dc/src/Controller/Component/CaptchaComponent.php#L75

But personally, I like to use the beforeRender() callback as this is not needed an any non GET page load (e.g. P from PRG).

@ADmad
Copy link
Member Author

ADmad commented Nov 15, 2025

I like to use the beforeRender() callback

That's definitely where it should be added, don't know why I added it to startup() 😄

@ADmad ADmad force-pushed the autoload-helper branch 2 times, most recently from a24f6c2 to 76dd8f9 Compare November 15, 2025 17:00
@ADmad ADmad merged commit 8ebcd81 into master Nov 15, 2025
7 checks passed
@ADmad ADmad deleted the autoload-helper branch November 15, 2025 17:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants