feat(files): add --use-user-id option to transfer-ownership command#56664
Conversation
Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
|
Thanks for your pull request 👍 I wonder if we should always use the sourceId or maybe both "Displayname (userid)". |
|
Thanks for the feedback @kesselb! Using both display name and user ID (e.g. Either way, implementing it with both by default or using only the user ID should be straightforward from an implementation perspective. Happy to adjust the implementation based on what works best. |
|
cs-fixer needs to be executed |
Signed-off-by: Cristian Scheid <cristianscheid@gmail.com>
Co-authored-by: Kate <26026535+provokateurin@users.noreply.github.com> Signed-off-by: Cristian Scheid <74515775+cristianscheid@users.noreply.github.com>
|
Congratulations @cristianscheid for the merge! |
|
Thank you @sorbaugh! It was great working on this. Thanks to everyone for the reviews and feedback! |
|
Hello there, We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process. Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6 Thank you for contributing to Nextcloud and we hope to hear from you soon! (If you believe you should not receive this message, you can add yourself to the blocklist.) |
Summary
Adds the
--use-user-idoption to theocc files:transfer-ownershipcommand, allowing administrators to use user IDs instead of display names when generating target folder names during ownership transfers.This avoids ambiguity in environments where multiple users share the same display name, ensuring clear and consistent identification.
Changes
--use-user-idoption to theocc files:transfer-ownershipcommandBehavior
Default (without flag):

Command:
occ files:transfer-ownership source_user dest_userWith

--use-user-idflag:Command:
occ files:transfer-ownership source_user dest_user --use-user-idChecklist
3. to review, feature component)stable32)