Skip to content

Conversation

@alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Jul 7, 2025

  • Tests written, or not not needed

Issues

  1. Since Nextcloud 30, the enabled key have been renamed to value and supports more than boolean. Thus, client needs to support both type.

Documentation

  1. When user change the permission from sharing details (quick permission bottom sheet ), share is not updating correctly.
  2. Choosing Can edit permission from share settings screen and choosing File requestfrom quick permission bottom sheet makes disable the bottom sheet actions.

Notes

File request can only be set if share is public or mail and share is folder

Download limit can be only set if share is not folder and share is public

If user creates a folder share and use just read custom permission type it also means View Only thus showing View Only doesn't mean wrong, check web behavior.

If user creates a folder share and use just create custom permission type it also means File request thus showing File request doesn't mean wrong, check web behavior.

If user creates a file share and use just read custom permission type it also means View Only thus showing View Only doesn't mean wrong, check web behavior.

If user creates a file share and use read and edit custom permission type it also means Can edit thus showing ``Can edit` doesn't mean wrong, check web behavior.

How to test?

Scenario 1: Updating an Existing Share

  1. Set up two instances: one with Nextcloud 28 and another with Nextcloud version 30 or higher.
  2. Create a share on both instances.
  3. Update the share settings to enable "Allow download and sync."
  4. Ensure the update is successfully completed.
  5. Repeat for other share permission types

Scenario 2: Creating a Share with Specific Permissions

  1. Set up two instances: one with Nextcloud 28 and another with Nextcloud version 30 or higher.
  2. Create a share with "Allow download and sync" enabled.
  3. Verify that the share is successfully created.
  4. Repeat for other share permission types

@alperozturk96
Copy link
Collaborator Author

/backport to stable-3.32

@alperozturk96 alperozturk96 added this to the Nextcloud App 3.32.2 milestone Jul 14, 2025
@alperozturk96 alperozturk96 force-pushed the bugfix/share-permission-handling branch from 853a79d to 9ac056d Compare July 14, 2025 10:58
@alperozturk96 alperozturk96 requested a review from mpivchev July 14, 2025 10:58
@alperozturk96 alperozturk96 force-pushed the bugfix/share-permission-handling branch from 9ac056d to 6126ee7 Compare July 14, 2025 13:26
@alperozturk96 alperozturk96 added ux-enhancement Improvements that refine user interactions, accessibility, or overall usability performance 🚀 Performance improvement opportunities (non-crash related) labels Jul 14, 2025
@alperozturk96 alperozturk96 force-pushed the bugfix/share-permission-handling branch from b0cfaec to e85dc70 Compare July 21, 2025 07:01
tobiasKaminsky

This comment was marked as resolved.

Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
…l share adapters sepearately previous sorting logic was wrong.

Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
Signed-off-by: alperozturk <alper_ozturk@proton.me>
@alperozturk96 alperozturk96 force-pushed the bugfix/share-permission-handling branch from d568245 to aacaa6e Compare July 22, 2025 07:21
@github-actions
Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/15107.apk

qrcode

To test this change/fix you can simply download above APK file and install and test it in parallel to your existing Nextcloud app.

@github-actions
Copy link

Codacy

Lint

TypemasterPR
Warnings4848
Errors1111

SpotBugs

CategoryBaseNew
Bad practice6564
Correctness6262
Dodgy code299299
Experimental11
Internationalization77
Malicious code vulnerability22
Multithreaded correctness3535
Performance4848
Security1818
Total537536

@tobiasKaminsky tobiasKaminsky merged commit 80e750f into master Jul 23, 2025
19 of 21 checks passed
@tobiasKaminsky tobiasKaminsky deleted the bugfix/share-permission-handling branch July 23, 2025 06:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review performance 🚀 Performance improvement opportunities (non-crash related) ux-enhancement Improvements that refine user interactions, accessibility, or overall usability

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants