Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature Compatible File Names #13220

Merged
merged 65 commits into from
Aug 23, 2024
Merged

Conversation

alperozturk96
Copy link
Collaborator

@alperozturk96 alperozturk96 commented Jul 2, 2024

  • Tests written, or not not needed

How to Test?

For all tests should do checks according to OC Capability. e.g. If forbiddenFilenames is false or unknown, no need to check reserved Windows names.

  1. [Create a New Document] Tap the plus button on the home screen, select "New Document," and enter a file name for the new document.
  2. [Rename a File] Tap the three dots icon at the end of any file in the list, and select "Rename."
  3. [Rename a File From File Details] Go to "File Details". From the top right, tap the three dots icon, select "Rename," and test it.
  4. [Create a New Folder] Tap the plus button on the home screen, select "New Folder," and enter a name for the new folder.
  5. [Move or Copy file to Any Folder] Tap the three dots icon at the end of any file in the list, and select "Move" or "Copy." Attempt to move or copy to both an invalid and a valid folder path.
  6. [Share a File From Other Any App] From any other app, try to share a file with the Nextcloud Files app. If the shared file contains invalid characters or reserved name, user must receive feedback, and the operation should not be allowed.
  7. [Upload Contents From Other Apps] Tap the plus button on the home screen, select "Upload Contents from Other Apps." If the contents include a path, the user must receive feedback, and the operation should not be allowed.
  8. [Create a New File From Other File Manager Apps] User can create forbidden file names from native files app such as Google Files.
  9. [Create a New Template] User may have a template for creating office files such as new presentation and new text document etc.

@alperozturk96 alperozturk96 marked this pull request as draft July 2, 2024 13:37
@alperozturk96 alperozturk96 force-pushed the feature/compatible-file-names branch 3 times, most recently from e5c8c9e to b670044 Compare July 8, 2024 13:27
@alperozturk96 alperozturk96 force-pushed the feature/compatible-file-names branch 2 times, most recently from 449a315 to acec2dd Compare July 12, 2024 13:58
@alperozturk96 alperozturk96 requested review from tobiasKaminsky and removed request for tobiasKaminsky July 15, 2024 09:40
@alperozturk96 alperozturk96 force-pushed the feature/compatible-file-names branch 2 times, most recently from 7a362f5 to 5a1028e Compare July 19, 2024 07:26
@alperozturk96 alperozturk96 force-pushed the feature/compatible-file-names branch 3 times, most recently from 4051595 to 20c34dc Compare August 19, 2024 08:44
alperozturk96 and others added 16 commits August 23, 2024 13:24
…py. And check button states

Signed-off-by: alperozturk <[email protected]>
… folder name empty in initial phase

Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: tobiasKaminsky <[email protected]>
Signed-off-by: tobiasKaminsky <[email protected]>
Signed-off-by: tobiasKaminsky <[email protected]>
Signed-off-by: alperozturk <[email protected]>
@alperozturk96 alperozturk96 marked this pull request as ready for review August 23, 2024 11:33
alperozturk96 and others added 3 commits August 23, 2024 13:47
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: tobiasKaminsky <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Signed-off-by: alperozturk <[email protected]>
Copy link

Codacy

Lint

TypemasterPR
Warnings6260
Errors33

SpotBugs

CategoryBaseNew
Bad practice6363
Correctness6464
Dodgy code305299
Experimental11
Internationalization77
Multithreaded correctness66
Performance5253
Security1818
Total516511

Copy link

APK file: https://www.kaminsky.me/nc-dev/android-artifacts/13220.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.

@tobiasKaminsky tobiasKaminsky merged commit 6d32493 into master Aug 23, 2024
21 checks passed
@AndyScherzinger AndyScherzinger added this to the Nextcloud App 3.30.0 milestone Aug 23, 2024
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.

3 participants