[ArtificialOwl]

This is draft on fixing #63159

Summary

• returns to previous default behavior,
• allow to disable local and/or external shares using a scope,
• previous config is deprecated, converted via migration step

TODO

• Admin documentation for core/hide_disabled_user_shares_scope

Checklist

• Code is properly formatted
• Sign-off message is added to all commits
• Tests (unit, integration, api and/or acceptance) are included
• Screenshots before/after for front-end changes
• Documentation (manuals or wiki) has been updated or is not required
• Backports requested where applicable (ex: critical bugfixes)
• Labels added where applicable (ex: bug/enhancement, 3. to review, feature component)
• Milestone added for target branch/version (ex: 32.x for stable32)

AI (if applicable)

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

[ArtificialOwl]

/backport to stable35

[ArtificialOwl]

/backport to stable34

[ArtificialOwl]

/backport to stable33

[susnux]

seems a bit over-complicated, why not just add second option to allow a different behavior on internal shares?
Would be a ~10 lines change then.

returns to previous default behavior,

❌ No this is not possible, the change was on purpose and has to stay.

[ArtificialOwl]

seems a bit over-complicated, why not just add second option to allow a different behavior on internal shares? Would be a ~10 lines change then.

That was my previous work, but the logic were strange:

• one to hide all (default),
• one to hide external with admin needing to disable the first one,

therefor the use of scope seemed better.
Should I switch back to a 2 settings solution ?

returns to previous default behavior,

❌ No this is not possible, the change was on purpose and has to stay.

ok, wanted confirmation it was on purpose :)