codeigniter4 / settings

Settings Library for CodeIgniter 4
https://settings.codeigniter.com
MIT License
86 stars 24 forks source link

Bypass config($class) performance impact #131

Open pixobit opened 6 months ago

pixobit commented 6 months ago

Description Huge performance improvement. config($class) seems to be an expensive process, and we don't really need it before falling back to the config property. Note: Try creating a for loop of 1000 to get a settings value

kenjis commented 6 months ago

Thank you for sending this PR!

We expect the following in all Pull Requests (PRs).

[!IMPORTANT] We expect all code changes or bug-fixes to be accompanied by one or more tests added to our test suite to prove the code works.

If pull requests do not comply with the above, they will likely be closed. Since we are a team of volunteers, we don't have any more time to work on the framework than you do. Please make it as painless for your contributions to be included as possible.

See also the documentation in the main repository: https://github.com/codeigniter4/CodeIgniter4/blob/develop/contributing/pull_request.md

kenjis commented 6 months ago

Do you need any help completing this PR?