[RELEASE] 3.0

[jesmrec]

Synchronization engine with new architecture

TASKS:

• [GIT] Create branch release/3.0 in owncloud/android from new_arch/synchronization
• [DEV] Update version number and name in build.gradle in owncloudApp module
• [DIS] Create a folder for the new version like $majorVersion.$minorVersion.$patchVersion_YYYY-MM-DD
• [DIS] Move all changelog files from the unreleased folder to the new version folder
• [DIS] Update screenshots, if needed, in README.md
• [DIS] Add ReleaseNotes replacing emptyList with listOf and adding inside ReleaseNote() with String resources
• [QA] Design Test plan
• [QA] Regression Test plan
• [GIT] Create and sign tag 'oc-android-3.0' in HEAD commit of release branch, in owncloud/android
• [GIT] Create and sign tag '2.0' in HEAD commit of release branch, in owncloud/android-library
• [DIS] Generate final bundle from signed commit in owncloud/android
• [GIT] Merge branch release/3.0 in owncloud/android, into master
• [DIS] Upload & publish release bundle and changelog in Play Store
• [DIS] Update screenshots and store listing, if needed, in Play Store
• [GIT] Publish a new release in owncloud/android
• [DIS] Create post in central.owncloud.org (Category:News + Tag:android)
• [COM] Inform #updates and #marketing in internal chat
• [DIS] Upload release APK and bundle to internal owncloud instance
• [GIT] Merge new_arch/synchronization branch into master, in owncloud/android-library
• [GIT] Merge new_arch/synchronization branch into master, in owncloud/android
• [GIT] Merge master branch into stable, in owncloud/android-library
• [GIT] Merge master branch into stable, in owncloud/android
• [DOC] Update documentation with new stuff (notify rocketchat #documentation-internal in advance!)

---

QA (pending reports)

S1:

• (1) Refresh new content in previews [RELEASE] 3.0 #3724 (comment) [FIXED]
• (2) Smooth authentication process [RELEASE] 3.0 #3724 (comment) [WONT FIX ftm]
• (35) Crash copying using document provider to root folder [RELEASE] 3.0 #3724 (comment) [FIXED]
• (36) App does not behave correctly in certificate change [RELEASE] 3.0 #3724 (comment) [FIXED]

S2:

• (3) Clickable snackbar after revoking token [RELEASE] 3.0 #3724 (comment) [FIXED - file/folder sync]
• (4) Sync badge is gone [RELEASE] 3.0 #3724 (comment) [FIXED]
• (6) Download cancelling in file list's toolbar [RELEASE] 3.0 #3724 (comment)
• (16) Failed authentication in OAuth2 [RELEASE] 3.0 #3724 (comment)
• clean up paths before uploading #3788
• (18) Uploaded files out of line are not listed till connection is recovered [RELEASE] 3.0 #3724 (comment) [OUT OF SCOPE]
• (20) Unknown error in oCIS when big file is uploaded [RELEASE] 3.0 #3724 (comment)
• (21) Cancelled uploads with oCIS in file list [RELEASE] 3.0 #3724 (comment) [NOT REGRESSION -> CREATE NEW ISSUE]
• (24) Public link not in the list if item is not in root folder [RELEASE] 3.0 #3724 (comment) [MAYBE SERVER RELATED] -> POST response does not return correct path when creating public link ocis#5139
• (25) Remove public link in oCIS [RELEASE] 3.0 #3724 (comment) [FIXED]
• (30) Download bar in image preview [RELEASE] 3.0 #3724 (comment) [NOT REP]
• (31) Ugly error when copying [RELEASE] 3.0 #3724 (comment) [MAYBE SERVER RELATED] -> OC-FileId missing in COPY operation response ocis#5039
• (32) Conflict when moving file [RELEASE] 3.0 #3724 (comment)
• (34) No way to cancel folder sync / av. offline [RELEASE] 3.0 #3724 (comment) [TO IMPROVE]
• (37) Download stuff is always synced [RELEASE] 3.0 #3724 (comment) [wont fix now]
• [38] Sync action with folders triggers spinner / progress bar [RELEASE] 3.0 #3724 (comment)
• (40) Crash resuming upload with credentials error [RELEASE] 3.0 #3724 (comment) [FIXED]
• (41) Conflicted file not opened [RELEASE] 3.0 #3724 (comment) [FIXED]

S3:

• (5) Upload stucked after running sc. st. wizard [RELEASE] 3.0 #3724 (comment)
• (7) Glitch in toolbar when navigating [RELEASE] 3.0 #3724 (comment)
• (8) Bad browsing - toolbar [RELEASE] 3.0 #3724 (comment)
• (9) Long message in copy error [RELEASE] 3.0 #3724 (comment)
• (10) Folder recreated after being removed [RELEASE] 3.0 #3724 (comment)
• (11) Uploads very fast [RELEASE] 3.0 #3724 (comment)
• (12) New status for enqueued [RELEASE] 3.0 #3724 (comment)
• (13) Clear button for enqueued list in upload [RELEASE] 3.0 #3724 (comment)
• (14) Unav. offline option in top bar [RELEASE] 3.0 #3724 (comment)
• (15) Disable FAB if folder lacks of writing permission [RELEASE] 3.0 #3724 (comment)
• (17) Way to display quota in oCIS [RELEASE] 3.0 #3724 (comment)
• (19) Downloading offline image [RELEASE] 3.0 #3724 (comment)
• (22) App closed while uploading shows Unknown error [RELEASE] 3.0 #3724 (comment)
• (23) Behaviour after non-allowed streaming [RELEASE] 3.0 #3724 (comment)
• (26) Error incorrect when sharing with existing recipient [RELEASE] 3.0 #3724 (comment)
• (27) Error incorrect when resharing without permissions [RELEASE] 3.0 #3724 (comment) [MAYBE SERVER RELATED]
• (28) Error incorrect when editing file without permissions [RELEASE] 3.0 #3724 (comment)
• (29) Error incorrect when uploading stuff to non-permission folder [RELEASE] 3.0 #3724 (comment)
• (33) Syncing removed file in details view does not lead to list of files [RELEASE] 3.0 #3724 (comment)
• (39) Navigation to a non-existent folder after rejecting "Open with" or "Send" [RELEASE] 3.0 #3724 (comment)

[jesmrec]

(1) [S1] [FIXED]

1. Download a text file, or set it as av. offline
2. Go back to the list of files
3. In server, modify the text file
4. Open txt file in app

Current: download enqueued but old content displayed. By reopening the file, content is correct.
Expected: download done and new content displayed

Happens in:

• oCIS
• oC10

[jesmrec]

(2) [S1] [WONT FIX]

(unfortunately, not always reproducible)

1. Install app from scratch
2. Add basic auth account

Current: After entering credentials, app is frozen. Reopening the app not always fix the problem. Sounds like a problem in dependency injection
Expected: Smooth authentication and file list process

Samsung A51 Android 12
d470bd968

Happens in:

• oCIS
• oC10 (only with basic auth)

[jesmrec]

(3) [S2] [FIXED]

1. Enter in an OAuth2 account
2. In the server, remove the token
3. In the app, download a file

Current: notification not clickable in snackbar
Expected: notification clickable to redirect to login view

master	3.0-beta.2

Happens in:

• oCIS (not able to reproduce)
• oC10

[jesmrec]

(4) [S2] [FIXED]

1. Select a file in file list
2. Click on download or select av. offline option

Current: Sync badge is gone
Expected: Sync badge displayed

Happens in:

• oCIS
• oC10

Update:
Fixed in #3790

[jesmrec]

(5) [S3]

1. Install v2.18.3 (or any version before Scoped Storage release)
2. Add account
3. Upload a very big file
4. Before the upload ends, upgrade to current and run wizard
5. Open uploads view

Current: upload stucked and never ends.
Expected: upload resumed and finished

Happens in:

• oCIS
• oC10

[jesmrec]

(6) [S2] [FIXED]

1. Select a big file from the file list
2. Click on the down arrow on the toolbar to download it

File starts to download (badge is gone as (14)), the way to check it is the Details view.

By cancelling the selection mode, and selecting the file again, the (x) to cancel is in the toolbar. By clicking the (x), the download is cancelled but the (x) remains in the toolbar.

Expected: Download and cancel buttons should be handled dinamically in toolbar, as in master. That means:

• If user clicks on down arrow, the down arrow turns in (x) to cancel
• If the download is running and user clicks on (x), (x) turns into down arrow again.

Sometimes, by clicking several times in (x) after cancelling, app crashes

Behaviour is correctly implemented in Details view

Happens in:

• oCIS
• oC10

[jesmrec]

(7) [S3]

Related to #3498

1. Create a link on a pdf file
2. Open Links tab
3. Clink on the pdf file -> open details view
4. Select Open with and open the file with an external app
5. Go back to oC app

Current: Toolbar shows Shared by link in the details view

Happens in:

• oCIS
• oC10

[jesmrec]

(8) [S3]

1. Open Details view of any file in the root folder
2. Click on Share icon on the top
3. Click on back arrow

Current: toolbar is not correct. It includes items of root folder instead of Details view
Expected: Details view items

Happens in:

• oCIS
• oC10

[jesmrec]

(9) [S3]

1. Select a folder
2. Select Copy
3. Select as target folder, same folder as 1.

Current: error message is too long, and could not be read completely:

Expected (master): message is reduced: It is not possible to copy a folder into a descendant

NOTE: same for move

Happens in:

• oCIS
• oC10

[jesmrec]

(10) [S3]

1. Browse to folder Photos
2. Select a file to upload inside
3. Before submitting, remove the folder Photos in another client
4. Submit the upload

Current: Photos folder is created again with the picture inside
Expected: Folder was deleted, so it should trigger a Folder error. Maybe the current behaviour is not too evil, open to discussion

Happens in:

• oCIS
• oC10

[jesmrec]

(11) [S3]

Open to discussion: when a bunch of files is uploaded and uploads run fast. it's difficult to check what's going on in the uploads view because items move up and down very fast in a non-readable way:

device-2022-08-18-145000.mp4

Happens in:

• oCIS
• oC10

[jesmrec]

(12) [S3]

1. Set to upload a very big file
2. When file is uploading, set to upload other files

Current: Pending files with the status Uploading, same as the file that is really being uploaded:

Expected: different status. File uploading with Uploading and others with Waiting or similar

Happens in:

• oCIS
• oC10

[jesmrec]

(13) [S3] (improvement)

1. Upload many files manually
2. Remove device connection

Current: it may happen that the amount of enqueued files is very big. For example:

Do you think it'd be interesting to add a Clear button to the enqueued? is it feasible to cancel works at that point?

Happens in:

• oCIS
• oC10

[jesmrec]

(14) (improvement) [S3]

1. Set a folder with many items inside as av. offline (so, the sync operation will take more time)

Current: The option Unset as av. offline is available in three dot menu while the folder is being synced.
Expected (current master): while folder is being synced, the option to Unset as av. offline is available on the top bar, so only one click is required to cancel sync:

Happens in:

• oCIS
• oC10

[jesmrec]

(15) [S3]

Comes from #2653

When a folder lacks of writing permission, FAB button could be hidden/disabled. In that way, we will avoid users to perform actions that will be banned.

Permissions available in property <oc:permissions> with the following values:

https://github.com/owncloud/client/blob/cedf72825bc0db8e3c147958a9382ee5b411d9ea/src/common/remotepermissions.h#L42-L54

Samsung A52
8f2d519d

Happens in:

• oCIS
• oC10

[jesmrec]

(16) [S2]

1. Log in a OAuth2 server

Current: after first authentication, a credentials error is returned. By re-signing, everything OK
Expected: No error after first authentication

Some logs were caught over cloud.damken.com:

owncloud.2022-11-03.log

d470bd9685

Happens in:

• oCIS
• oC10

[jesmrec]

(17) [S3]

1. In oCIS user dashboard, set any user's quota to 1GB
2. Open app drawer of such user

Current: xMB of 982.2 MB used
Expected: xMB of 1 GB used

Maybe not a client issue, maybe yes... open to discussion
Same issue with other values.

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(18) [S2] [OUT OF SCOPE]

Maybe a feature, not a bug

1. Set several files to upload. I used 20, to have tome to perform the following steps
2. While files are uploading, remove connection. Some of them have finished, the rest will fail
3. Switch to the containing folder

Current: none of the uploaded (finished) files are there
Expected: the uploaded files are there

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

In master, after each upload, the file is stored in the database.
In the new architecture, uploads won't be shown in the file list till the user refreshes

FMPOV is not a feature, nor a bug since it does not break anything, files will be there after the next refresh. It is the expected behavior when it was developed. Open to discuss, but it would be out of the scope of 3.0

Update 11/11/2022:

Taken as expected, but could be confusing for users that will expect files in the list. Let's wait for feedback

[jesmrec]

(19) [S3]

1. Set device to offline
2. Set an image to download

Current: Downloading... status forever
Expected (master): Download error view

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(20) [S2]

1. Upload a big file to oCIS (i used a 2GB one)
2. Wait till upload ends

Current: in uploads view, it shows Unknown error, but file is uploaded and listed in the list view
Expected: Finished in uploads view

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(21) [S2] (server could be involved)

1. in oCIS, upload several big files (to have time to perform next steps). For this example, upload 5 files
2. Cancel some of them, for example 3, in uploads view

Current: All 5 files appear in the target folder. The cancelled ones with 0B size
Expected: Cancelled files not appearing in the file list

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

maybe server involved... not sure. Propfind returns 0B files, but, they shouldn't be created from client side.

Notes:
Upload is never cancelled actually. The PUT operation isn't aborted, so the HTTP request always returns 201 and it is considered a success. Therefore, a new file is created, sometimes with the whole size of the file and sometimes with 0 B.
It happens in master too.

[jesmrec]

(22) [S3] [FIXED]

1. in oCIS, upload a big file (to have time to perform next steps)
2. Before file finishes, kill the app
3. Reopen the app and check the uploads view

Current: upload with Unknown error
Expected (msater): upload with App was terminated

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(27) [S3] [No regression]

1. Share item with user without sharing permission
2. Recipient shares the item with another user

Current: Error displayed: It was not posible to share this file or folder because it was not found
Expected: More accurate error, or prevent sharing for shared items without such permission.

Samsung A51, Android 12
f14caeff2

• oCIS
• oC10

We receive a 404 when trying to do that, and the expected message for 404 errors is not found.
Shouldn't we get a 403 if resharing is not allowed?

[jesmrec]

(28) [S3]

1. User1 shares txt file with User2 without edit permission
2. User2 receives the file and try to edit it
3. User2 saves changes

Current: Unknown error in uploads view
Expected (master): Permission error

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(29) [S3] [No regression]

1. User1 shares a folder with User2, with no create permission
2. User2 tries to upload something inside

Current: Unknown error in uploads view
Expected: More accurate error

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(30) [S2] [Not always reproducible]

1. Set a large image to download
2. Cancel the download in the preview

Current: Download progress bar is not continuos, so, cancelling sometimes comes later and the cancelled image is finally downloaded:

device-2022-11-09-090637.mp4

Expected: Download progress bar with progress information

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(31) [S2] (moved to server)

1. In oCIS, select some files from the list
2. Copy the files to another location

Current: ugly error in snackbar. First file is copied to the location, but the other files don't

Expected: files copied with no error

device-2022-11-09-170057.mp4

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

When copying a file, we expect the fileId of the copied file in the response headers OC-FileId In oCIS we don't receive that data which makes it fail.

Server issue: owncloud/ocis#5039

[jesmrec]

(32) [S2]

1. In app, download a txt file
2. In server, modify the txt file
3. In app, move the file to another location

Current: file is shown as conflicted
Expected: file was always remote-modified, no local modification. So, no conflict should happen.

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

Reproducible if the file is synced while picking the destination folder, otherwise it does not happen. Moving the file uses the latest state before moving, so if it is updated by the way, recent changes won't be applied and conflict may appear

[jesmrec]

(33) [S3]

1. Open details view of any file
2. Click on sync -> OK
3. In server, remove the file
4. Click again on sync

current: nothing happens
expected: move back to the list of files

Samsung A51, Android 12
f14caeff2

Happens in:

• oCIS
• oC10

[jesmrec]

(34) [S2] [FIRST FIX, improvements pending]

There is no way to cancel a folder sync / av. offline.

Samsung A51, Android 12
f14caef

Happens in:

• oCIS
• oC10

Update:
Fixed in #3790

[jesmrec]

(35) [S1] [FIXED]

1. Create a new document using a 3rd-party-app compatible with document provider
2. Save it in a oC account's root folder

Current: app crashes. File is always save in the parent folder. That means, by selecting A/B/C to save the file, it is saved in A/B instead. So, root folder does not have parent and make the app crash

Expected: No crash

Samsung A51, Android 12
f14caef

Happens in:

• oCIS
• oC10

Update:
Fixed in #3790

[jesmrec]

(36) [S1] [FIXED]

1. Connect a server with https insecure (needs user approval for custom certificate in login view)
2. Once app is connected to the server, change the certificate in server side
3. Refresh app / Reopen app

Current: Warning dialog missing and app not working. Ugly error instead.
Expected: Warning displayed to be accepted by the user.

Here it is a simple guide how to simulate a certificate change: https://github.com/owncloud/ios-sdk/blob/master/doc/TESTING.md#simulating-certificate-changes

Samsung A51 Android 12
f14caeff2a

Happens in:

• oCIS (did not find a way to simulate cert change here. Anyway same scenario as oC10)
• oC10

[jesmrec]

(37) [S2] [No regression] [To discuss]

Take a look together with (1)

1. Download any file to the device
2. Using web or any other client, modify the file
3. In app, pull to refresh

Current: new version of the file is automatically downloaded.
Expected: new version of the file not downloaded. It should be downloaded when file is opened. Automatic download when refreshing/syncing folder is for available offline stuff.

This is not a bad behaviour, but could lead to confusion for scenarios with lots of modifications and/or av. offline files. All downloaded and av. offline will behave in the same way, breaking any difference between them. Since Download status is intended to be removed , this problem should not make us to spend a lot of time. Need to discuss.

Samsung A51 Android 12
f14caeff2a

Happens in:

• oCIS
• oC10

[jesmrec]

(38) [S2]

1. Select a folder with much content
2. Select to sync / Av. offline (no matter which one)

Current: Progress bar on the top and spinner in the screen till the sync action finishes
Expected: Progress bar and spinner only displayed in pull to refresh action. For syncing, just setting status badge should be enough

Samsung A51 Android 12
492645dcf

Happens in:

• oCIS
• oC10

[JuancaG05]

(39) [S3]

1. In a non-downloaded file, select "Open with" or "Send"
2. Before it finishes downloading, navigate to its details view (or a preview of any file that is already downloaded)
3. Go back to the main file list before it finishes downloading
4. When it finishes downloading, close the bottom sheet

Current: After closing the bottom sheet, the app navigates to a non-existent file with the name of the file whose details (or whose preview) has been seen in step 2
Expected: No navigation, staying in the same folder

Samsung Galaxy S9 Android 9
7d6fb80

Happens in:

• oCIS
• oC10

Notes:
It happens due to the property file of FileActivity, from where many activities extend. When navigating to a preview or to a details view, the variable file is overwritten with the current file being previewed or whose details are being seen, and when closing the bottom sheet, the method onResume of FileDisplayActivity tries to navigate to file expecting it as a folder.

[jesmrec]

(40) [FIXED]

1. In a OAuth2 account, upload a big file
2. While the upload is being done, remove session from admin dashboard -> Upload will fail with Credentials error
3. Tap on the failed upload in the uploads view -> redirected to login
4. Authorize again
5. Open uploads view and click in the cell to resume

Current: crash
Expected: moved to login view

Stack trace:

FATAL EXCEPTION: main
Process: com.owncloud.android, PID: 9479
java.lang.IllegalStateException: Cannot access database on the main thread since it may potentially lock the UI for a long period of time.  at androidx.room.RoomDatabase.assertNotMainThread(RoomDatabase.java:469)
at androidx.room.RoomDatabase.query(RoomDatabase.java:525)
at androidx.room.util.DBUtil.query(DBUtil.java:86)
at com.owncloud.android.data.transfers.db.TransferDao_Impl.getTransfersWithStatus(TransferDao_Impl.java:504)
at com.owncloud.android.data.transfers.datasources.implementation.OCLocalTransferDataSource.getFailedTransfers(OCLocalTransferDataSource.kt:127)
at com.owncloud.android.data.transfers.repository.OCTransferRepository.getFailedTransfers(OCTransferRepository.kt:99)
at com.owncloud.android.usecases.transfers.uploads.RetryFailedUploadsForAccountUseCase.run(RetryFailedUploadsForAccountUseCase.kt:40)
at com.owncloud.android.usecases.transfers.uploads.RetryFailedUploadsForAccountUseCase.run(RetryFailedUploadsForAccountUseCase.kt:32)
at com.owncloud.android.domain.BaseUseCase.execute(BaseUseCase.kt:30)
at com.owncloud.android.ui.activity.UploadListActivity.onRemoteOperationFinish(UploadListActivity.java:156)
at com.owncloud.android.services.OperationsService.lambda$dispatchResultToOperationListeners$0(OperationsService.java:398)
at com.owncloud.android.services.OperationsService$$ExternalSyntheticLambda0.run(Unknown Source:6)
at android.os.Handler.handleCallback(Handler.java:938)
at android.os.Handler.dispatchMessage(Handler.java:99)
at android.os.Looper.loopOnce(Looper.java:226)
at android.os.Looper.loop(Looper.java:313)
at android.app.ActivityThread.main(ActivityThread.java:8663)
at java.lang.reflect.Method.invoke(Native Method)
at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:567)
at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:1135)

Samsung A51, Android 12
b15316ff7

Happens in:

• oCIS
• oC10

[jesmrec]

(41) [S2] [FIXED]

1. Cause a conflict in a file in root folder (i used txt file)
2. Tap on the file to fix the conflict

Current: File is not opened
Expected: File opened to fix the conflict

NOTE: if the file is not in root, conflict dialog appears but app crashes after choosing any option

Samsung A51, Android 12
cfa459804

Happens in:

• oCIS
• oC10

[jesmrec]

(42) [FIXED]

1. Download and install 2.21.2
2. Add account
3. Download and set as av. offline some files and folders
4. Update to current

Current: av. offline and downloaded files are missing
Expected: av. offline and downloaded files are locally downloaded

Samsung A51, Android 12
1c8ac816a

Happens in:

• oCIS
• oC10

[JuancaG05]

Before 3.0, we are releasing a 3.0 beta.3 with the following fixes:

• (1) Refresh new content in previews [RELEASE] 3.0 #3724 (comment) [FIXED]
• (35) Crash copying using document provider to root folder [RELEASE] 3.0 #3724 (comment) [FIXED]
• (36) App does not behave correctly in certificate change [RELEASE] 3.0 #3724 (comment) [FIXED]
• (3) Clickable snackbar after revoking token [RELEASE] 3.0 #3724 (comment) [FIXED - file/folder sync]
• (4) Sync badge is gone [RELEASE] 3.0 #3724 (comment) [FIXED]
• (25) Remove public link in oCIS [RELEASE] 3.0 #3724 (comment) [FIXED]
• (40) Crash resuming upload with credentials error [RELEASE] 3.0 #3724 (comment) [FIXED]
• (41) Conflicted file not opened [RELEASE] 3.0 #3724 (comment) [FIXED]