Skip to content

feat(config): make occ config:list output stable and diff-friendly#61952

Draft
joshtrichards wants to merge 1 commit into
masterfrom
jtr/feat-occ-configList-ksort
Draft

feat(config): make occ config:list output stable and diff-friendly#61952
joshtrichards wants to merge 1 commit into
masterfrom
jtr/feat-occ-configList-ksort

Conversation

@joshtrichards

Copy link
Copy Markdown
Member
  • Resolves: #

Summary

  • Makes occ config:list output stable, easier to scan, and diff-friendly
  • Only changes presentation order

Before:

{
    "system": {
        "htaccess.RewriteBase": "\/",
        "memcache.local": "\\OC\\Memcache\\APCu",
        "apps_paths": [
            {
                "path": "\/var\/www\/html\/apps",
                "url": "\/apps",
                "writable": false
            },
            {
                "path": "\/var\/www\/html\/custom_apps",
                "url": "\/custom_apps",
                "writable": true
            }
        ],
        "upgrade.disable-web": true,
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
            "localhost"
        ],
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "sqlite3",
        "version": "33.0.3.2",
        "overwrite.cli.url": "http:\/\/localhost",
        "installed": true,
        "maintenance": false,
        "dbpassword": "***REMOVED SENSITIVE VALUE***"
    }
}

After:

{
    "system": {
        "apps_paths": [
            {
                "path": "\/var\/www\/html\/apps",
                "url": "\/apps",
                "writable": false
            },
            {
                "path": "\/var\/www\/html\/custom_apps",
                "url": "\/custom_apps",
                "writable": true
            }
        ],
        "datadirectory": "***REMOVED SENSITIVE VALUE***",
        "dbpassword": "***REMOVED SENSITIVE VALUE***",
        "dbtype": "sqlite3",
        "htaccess.RewriteBase": "\/",
        "installed": true,
        "instanceid": "***REMOVED SENSITIVE VALUE***",
        "maintenance": false,
        "memcache.local": "\\OC\\Memcache\\APCu",
        "overwrite.cli.url": "http:\/\/localhost",
        "passwordsalt": "***REMOVED SENSITIVE VALUE***",
        "secret": "***REMOVED SENSITIVE VALUE***",
        "trusted_domains": [
            "localhost"
        ],
        "upgrade.disable-web": true,
        "version": "33.0.3.2"
    }
}

TODO

  • Possibly update tests

Checklist

AI (if applicable)

  • The content of this PR was partly or fully generated using AI

- Only changes presentation order
- Makes output stable and diff-friendly


Signed-off-by: Josh <josh.t.richards@gmail.com>
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.

1 participant