Skip to content

Mk/dev/use config import#148

Draft
printminion-co wants to merge 7 commits into
mainfrom
mk/dev/use_config_import
Draft

Mk/dev/use config import#148
printminion-co wants to merge 7 commits into
mainfrom
mk/dev/use_config_import

Conversation

@printminion-co

Copy link
Copy Markdown
Contributor

No description provided.

Introduces import_app_config() which pipes a JSON string to
`occ config:import`, allowing multiple app/system config values
to be set in a single PHP process invocation instead of one
occ call per key.
Replace 5-6 individual config:app:set calls in configure_collabora_app()
with a single import_app_config call built via jq. Sensitive-free values
(wopi_url, enabled, cert verification, allowlist) are batched into one
JSON import. app:disable, app:enable and richdocuments:activate-config
remain as individual OCC calls.
Replace 5 individual config:app:set calls in configure_files_antivirus_app()
with a single import_app_config call built via jq.
Replace 2 non-sensitive config:app:set calls in configure_ionos_mailconfig_api()
with a single import_app_config call. ionos_mailconfig_api_auth_pass remains
as an individual --sensitive call.
…import

Replace 5 config:app:set calls and 2 set_app_config_typed calls (each with
an extra config:list read) in configure_ionos_ai_model_hub() with a single
import_app_config call. Also batches ai.taskprocessing_guests (settings app).
api_key remains as an individual --sensitive call.
Introduces enable_apps() which passes all arguments to a single
`occ app:enable` call, allowing multiple apps to be enabled in one
PHP process invocation.
Replace 17 individual enable_app() calls with 2 enable_apps() calls,
reducing OCC invocations from 17 to 2 for app activation.

Apps with surrounding conditional logic (richdocuments, notify_push,
files_antivirus, spreed) remain as individual enables in their own
configure_*() functions.
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.

1 participant