Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Dynamically call the handler instance #1

Closed
wants to merge 1 commit into from
Closed

Dynamically call the handler instance #1

wants to merge 1 commit into from

Conversation

agungsugiarto
Copy link

No description provided.

@MGatner
Copy link
Member

MGatner commented Jan 31, 2021

@agungsugiarto I liked this solution, maybe with a guard for actual callable methods. Is there a reason you closed it?

@agungsugiarto
Copy link
Author

@MGatner close because already implement. https://github.com/lonnieezell/codigniter-shield/blob/3acb1761ceb9b385408ec3265c084c8c2049b3bd/src/Auth.php#L137-L146

@MGatner
Copy link
Member

MGatner commented Jan 31, 2021

Oh! Ha I missed that, sorry. Glad to see it made it.

@agungsugiarto
Copy link
Author

Take a look my adaption @MGatner, im just playing with authentication.
https://github.com/agungsugiarto/codeigniter4-authentication.git

@MGatner
Copy link
Member

MGatner commented Jan 31, 2021

I will check it out later this week, thanks for sharing!

lonnieezell pushed a commit that referenced this pull request Apr 30, 2022
kenjis referenced this pull request in kenjis/codeigniter-shield Jun 12, 2022
ErrorException: strtolower(): Passing null to parameter #1 ($string) of type string is deprecated
@ValdisM ValdisM mentioned this pull request Jan 11, 2023
davidnsai pushed a commit to davidnsai/shield that referenced this pull request Feb 21, 2023
ErrorException: strtolower(): Passing null to parameter codeigniter4#1 ($string) of type string is deprecated
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.

2 participants