Skip to content

Conversation

@JulianAtkins
Copy link

Description
I installed the ci4 settings project and I tried to get all my options, then it hit me, the cold dread of a missing feature.. oh no, there is no getAll() function! So... behold a pull request with a getAll() function implemented.

What has been changed, well not much, the getAll() function eventually ends in the backend to a getAllStored() function that gathers all handlers and collects all values in an array with a key based on the handler name.

Please let me know if its something you'd like to merge or if I should drop it.

Thank youu!!

Checklist:

  • Securely signed commits
  • Component(s) with PHPDoc blocks, only if necessary or adds value
  • Unit testing, with >80% coverage
  • User guide updated
  • Conforms to style guide

@datamweb
Copy link
Contributor

datamweb commented Dec 8, 2025

Hi there,

Thank you for sending this PR!

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

@JulianAtkins
Copy link
Author

I just read the GPG-signed document... I need 2 hours to setup the whole thing, just to submit 10-12 lines of code, so not really worth doing. Closing.

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