Skip to content

trunk StatusCommand: implement a lazy filter - #7897

Closed
daniwe4 wants to merge 1 commit into
ILIAS-eLearning:trunkfrom
daniwe4:trunk_status_command_lazy_filter
Closed

daniwe4 wants to merge 1 commit into
ILIAS-eLearning:trunkfrom
daniwe4:trunk_status_command_lazy_filter

Conversation

@daniwe4

@daniwe4 daniwe4 commented Aug 6, 2024

Copy link
Copy Markdown
Contributor

Metric now uses a callable as the third parameter. This means that only the requested values are queried when filtering in the status command.

To use the filter in the status command, simply add the -f option. This is followed by the filter string separated by a period.
For example:

php setup/setup.php status -f config.common

@daniwe4
daniwe4 force-pushed the trunk_status_command_lazy_filter branch from 66aa331 to cb0d99d Compare August 6, 2024 11:31
@daniwe4 daniwe4 changed the title trunk StatusCommand: impement a lazy filter trunk StatusCommand: implement a lazy filter Aug 6, 2024
@daniwe4
daniwe4 force-pushed the trunk_status_command_lazy_filter branch 2 times, most recently from 31b6070 to 05d98ef Compare August 6, 2024 11:44
Metric now uses a callable as the third parameter. This means that only the requested values are queried when
filtering in the status command.

To use the filter in the status command, simply add the -f option. This is followed by the filter string separated
by a period.
For example:

php setup/setup.php status -f config.common
@daniwe4
daniwe4 force-pushed the trunk_status_command_lazy_filter branch from 05d98ef to e575c4e Compare August 6, 2024 11:47
@dsstrassner dsstrassner assigned thibsy and oliversamoila and unassigned klees Dec 29, 2025
@dsstrassner

Copy link
Copy Markdown
Contributor

Hi @oliversamoila, I think you should discuss this with @thibsy 🙈

@chfsx

chfsx commented Oct 1, 2026

Copy link
Copy Markdown
Member

@daniwe4, thanks for the proposal. I picked it up in #12155 with a different implementation that
keeps the metrics API unchanged. If that works for you, I would close this PR in favour of #12155.

@chfsx chfsx closed this Oct 8, 2026
chfsx added a commit to srsolutionsag/ILIAS that referenced this pull request Oct 27, 2026
chfsx added a commit that referenced this pull request Oct 27, 2026
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.

6 participants