Merge pull request #12474 from nextcloud/fix/comments_loading

Fix comments loading and screenshot test

app/src/androidTest/java/com/owncloud/android/ui/dialog/DialogFragmentIT.java

@@ -248,6 +248,7 @@ public class DialogFragmentIT extends AbstractIT {
         accountManager.setUserData(newAccount, AccountUtils.Constants.KEY_USER_ID, "test");
         accountManager.setAuthToken(newAccount, AccountTypeUtils.getAuthTokenTypePass(newAccount.type), "password");
         User newUser = userAccountManager.getUser(newAccount.name).orElseThrow(RuntimeException::new);
+        userAccountManager.setCurrentOwnCloudAccount(newAccount.name);
 
         Account newAccount2 = new Account("user1@nextcloud.localhost", MainApp.getAccountType(targetContext));
         accountManager.addAccountExplicitly(newAccount2, "password", null);

app/src/debug/java/com/nextcloud/test/TestActivity.kt

@@ -126,8 +126,8 @@ class TestActivity :
         TODO("Not yet implemented")
     }
 
-    override fun getFileUploaderHelper(): FileUploadHelper? {
-        return null
+    override fun getFileUploaderHelper(): FileUploadHelper {
+        return FileUploadHelper.instance()
     }
 
     override fun getFileDownloadProgressListener(): FileDownloadWorker.FileDownloadProgressListener? {

app/src/main/java/com/owncloud/android/ui/activity/FileDisplayActivity.java

@@ -1921,28 +1921,29 @@ public class FileDisplayActivity extends FileActivity
         // the execution is slightly delayed to allow the activity get the window focus if it's being started
         // or if the method is called from a dialog that is being dismissed
         if (TextUtils.isEmpty(searchQuery) && getUser().isPresent()) {
-            getHandler().postDelayed(new Runnable() {
-                @Override
-                public void run() {
-                    if (ignoreFocus || hasWindowFocus()) {
-                        long currentSyncTime = System.currentTimeMillis();
-                        mSyncInProgress = true;
+            getHandler().postDelayed(() -> {
+                Optional<User> user = getUser();
 
-                        // perform folder synchronization
-                        RemoteOperation synchFolderOp = new RefreshFolderOperation(folder, currentSyncTime, false, ignoreETag, getStorageManager(), getUser().orElseThrow(RuntimeException::new), getApplicationContext());
-                        synchFolderOp.execute(getAccount(), MainApp.getAppContext(), FileDisplayActivity.this, null, null);
+                if (!ignoreFocus && !hasWindowFocus() || !user.isPresent()) {
+                    // do not refresh if the user rotates the device while another window has focus
+                    // or if the current user is no longer valid
+                    return;
+                }
 
-                        OCFileListFragment fragment = getListOfFilesFragment();
+                long currentSyncTime = System.currentTimeMillis();
+                mSyncInProgress = true;
 
-                        if (fragment != null && !(fragment instanceof GalleryFragment)) {
-                            fragment.setLoading(true);
-                        }
+                // perform folder synchronization
+                RemoteOperation refreshFolderOperation = new RefreshFolderOperation(folder, currentSyncTime, false, ignoreETag, getStorageManager(), user.get(), getApplicationContext());
+                refreshFolderOperation.execute(getAccount(), MainApp.getAppContext(), FileDisplayActivity.this, null, null);
 
-                        setBackgroundText();
+                OCFileListFragment fragment = getListOfFilesFragment();
 
-                    }   // else: NOTHING ; lets' not refresh when the user rotates the device but there is
-                    // another window floating over
+                if (fragment != null && !(fragment instanceof GalleryFragment)) {
+                    fragment.setLoading(true);
                 }
+
+                setBackgroundText();
             }, DELAY_TO_REQUEST_REFRESH_OPERATION_LATER);
         }
     }

app/src/main/java/com/owncloud/android/ui/fragment/FileDetailActivitiesFragment.java

@@ -147,6 +147,7 @@ public class FileDetailActivitiesFragment extends Fragment implements
         viewThemeUtils.androidx.themeSwipeRefreshLayout(binding.swipeContainingEmpty);
         viewThemeUtils.androidx.themeSwipeRefreshLayout(binding.swipeContainingList);
 
+        isLoadingActivities = true;
         fetchAndSetData(-1);
 
         binding.swipeContainingList.setOnRefreshListener(() -> {

app/src/main/java/com/owncloud/android/ui/fragment/FileDetailSharingFragment.java

@@ -452,7 +452,11 @@ public class FileDetailSharingFragment extends Fragment implements ShareeListAda
      * before reading database.
      */
     public void refreshSharesFromDB() {
-        file = fileDataStorageManager.getFileById(file.getFileId());
+        OCFile newFile = fileDataStorageManager.getFileById(file.getFileId());
+        if (newFile != null) {
+            file = newFile;
+        }
+
         ShareeListAdapter adapter = (ShareeListAdapter) binding.sharesList.getAdapter();
 
         if (adapter == null) {