Skip to content

Commit 4b3416b

Browse files
committed
remove duplicated calls
Signed-off-by: alperozturk <[email protected]>
1 parent bbc054e commit 4b3416b

File tree

3 files changed

+25
-59
lines changed

3 files changed

+25
-59
lines changed

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

Lines changed: 13 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -533,7 +533,7 @@ class FileDisplayActivity :
533533
}
534534

535535
/** reset views */
536-
resetTitleBarAndScrolling()
536+
resetScrollingAndUpdateActionBar()
537537
}
538538

539539
// region Handle Intents
@@ -632,6 +632,8 @@ class FileDisplayActivity :
632632
}
633633
}
634634
}
635+
636+
listOfFilesFragment?.setCurrentSearchType(searchEvent)
635637
}
636638
// endregion
637639

@@ -789,8 +791,8 @@ class FileDisplayActivity :
789791
return null
790792
}
791793

792-
protected fun resetTitleBarAndScrolling() {
793-
updateActionBarTitleAndHomeButton(null)
794+
protected fun resetScrollingAndUpdateActionBar(searchType: SearchType = SearchType.NO_SEARCH) {
795+
updateActionBarTitleAndHomeButton(file, searchType)
794796
resetScrolling(true)
795797
}
796798

@@ -1233,7 +1235,7 @@ class FileDisplayActivity :
12331235
listOfFiles.registerFabListener()
12341236
}
12351237

1236-
resetTitleBarAndScrolling()
1238+
resetScrollingAndUpdateActionBar(listOfFilesFragment?.currentSearchType ?: SearchType.NO_SEARCH)
12371239
configureMenuItem()
12381240
startMetadataSyncForCurrentDir()
12391241
}
@@ -1519,7 +1521,7 @@ class FileDisplayActivity :
15191521

15201522
private fun handleRemovedFileFromServer(currentFile: OCFile?, currentDir: OCFile?): OCFile? {
15211523
if (currentFile == null && file?.isFolder == false) {
1522-
resetTitleBarAndScrolling()
1524+
resetScrollingAndUpdateActionBar()
15231525
return currentDir
15241526
}
15251527

@@ -1797,12 +1799,12 @@ class FileDisplayActivity :
17971799
startSyncFolderOperation(root, false)
17981800
}
17991801
binding.fabMain.setImageResource(R.drawable.ic_plus)
1800-
resetTitleBarAndScrolling()
1802+
resetScrollingAndUpdateActionBar(listOfFilesFragment?.currentSearchType ?: SearchType.NO_SEARCH)
18011803
}
18021804

18031805
override fun onBrowsedDownTo(directory: OCFile?) {
18041806
file = directory
1805-
resetTitleBarAndScrolling()
1807+
resetScrollingAndUpdateActionBar()
18061808
startSyncFolderOperation(directory, false)
18071809
startMetadataSyncForCurrentDir()
18081810
}
@@ -2099,7 +2101,7 @@ class FileDisplayActivity :
20992101
val fileAvailable = storageManager.fileExists(removedFile.fileId)
21002102
if (leftFragment is FileFragment && !fileAvailable && removedFile == leftFragment.file) {
21012103
file = storageManager.getFileById(removedFile.parentId)
2102-
resetTitleBarAndScrolling()
2104+
resetScrollingAndUpdateActionBar()
21032105
}
21042106
val parentFile = storageManager.getFileById(removedFile.parentId)
21052107
if (parentFile != null && parentFile == getCurrentDir()) {
@@ -2305,7 +2307,7 @@ class FileDisplayActivity :
23052307
leftFragment.updateFileDetails(file, currentUser)
23062308
} else {
23072309
if (!file.fileExists()) {
2308-
resetTitleBarAndScrolling()
2310+
resetScrollingAndUpdateActionBar()
23092311
} else {
23102312
leftFragment.updateFileDetails(false, true)
23112313
}
@@ -2735,7 +2737,8 @@ class FileDisplayActivity :
27352737
}
27362738

27372739
listOfFilesFragment?.setCurrentSearchType(event)
2738-
listOfFilesFragment?.setActionBarTitle()
2740+
updateActionBarTitleAndHomeButton(null)
2741+
//listOfFilesFragment?.setActionBarTitle()
27392742
}
27402743

27412744
@Subscribe(threadMode = ThreadMode.MAIN)

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

Lines changed: 5 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -155,12 +155,14 @@ public void setupHomeSearchToolbarWithSortAndListButtons() {
155155
}
156156

157157
public void updateActionBarTitleAndHomeButton(OCFile chosenFile, SearchType searchType) {
158+
if (mAppBar == null) {
159+
return;
160+
}
161+
158162
boolean isRoot = isRoot(chosenFile);
159163
String title = getActionBarTitle(chosenFile, searchType);
160164
updateActionBarTitleAndHomeButtonByString(title);
161-
if (mAppBar != null) {
162-
showHomeSearchToolbar(title, isRoot);
163-
}
165+
showHomeSearchToolbar(title, isRoot);
164166
}
165167

166168
protected void updateActionBarTitleAndHomeButton(OCFile chosenFile) {

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

Lines changed: 7 additions & 46 deletions
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,6 @@ public void onAttach(@NonNull Context context) {
290290
Log_OC.i(TAG, "onAttach");
291291
try {
292292
mContainerActivity = (FileFragment.ContainerActivity) context;
293-
setActionBarTitle();
294293
} catch (ClassCastException e) {
295294
throw new IllegalArgumentException(context.toString() + " must implement " +
296295
FileFragment.ContainerActivity.class.getSimpleName(), e);
@@ -434,10 +433,8 @@ public void onActivityCreated(Bundle savedInstanceState) {
434433
});
435434
}
436435

437-
setActionBarTitle();
438-
439-
if (getActivity() instanceof FileDisplayActivity fileDisplayActivity) {
440-
fileDisplayActivity.updateActionBarTitleAndHomeButton(fileDisplayActivity.getCurrentDir(), currentSearchType);
436+
if (getActivity() instanceof FileDisplayActivity fda) {
437+
fda.updateActionBarTitleAndHomeButton(fda.getCurrentDir(), currentSearchType);
441438
}
442439
listDirectory(MainApp.isOnlyOnDevice(), false);
443440
}
@@ -1062,7 +1059,6 @@ public int onBrowseUp() {
10621059
Pair<Integer, OCFile> result = futureResult.get();
10631060
mFile = result.second;
10641061
setFileDepth(mFile);
1065-
setActionBarTitle();
10661062
updateFileList();
10671063
return result.first;
10681064
} catch (Exception e) {
@@ -1810,28 +1806,18 @@ protected void setEmptyView(SearchEvent event) {
18101806
@Subscribe(threadMode = ThreadMode.MAIN)
18111807
public void onMessageEvent(ChangeMenuEvent changeMenuEvent) {
18121808
resetSearchAttributes();
1813-
18141809
resetMenuItems();
1815-
Activity activity = getActivity();
1816-
if (activity != null) {
1817-
activity.invalidateOptionsMenu();
1818-
1819-
if (activity instanceof FileDisplayActivity) {
1820-
((FileDisplayActivity) activity).initSyncBroadcastReceiver();
1821-
}
1822-
1823-
setActionBarTitle(themeUtils.getDefaultDisplayNameForRootFolder(getContext()), false);
1824-
activity.getIntent().removeExtra(OCFileListFragment.SEARCH_EVENT);
1825-
}
18261810

1827-
Bundle arguments = getArguments();
1828-
if (arguments != null) {
1829-
arguments.putParcelable(OCFileListFragment.SEARCH_EVENT, null);
1811+
if (getActivity() instanceof FileDisplayActivity fda) {
1812+
fda.invalidateOptionsMenu();
1813+
fda.getIntent().removeExtra(OCFileListFragment.SEARCH_EVENT);
1814+
fda.updateActionBarTitleAndHomeButton(null, NO_SEARCH);
18301815
}
18311816

18321817
if (mFile != null) {
18331818
setFabVisible(mFile.canCreateFileAndFolder());
18341819
}
1820+
18351821
slideHideBottomBehaviourForBottomNavigationView(true);
18361822
}
18371823

@@ -2088,31 +2074,6 @@ public void onMessageEvent(FileLockEvent event) {
20882074
}
20892075
}
20902076

2091-
public void setActionBarTitle() {
2092-
if (!(getActivity() instanceof FileDisplayActivity fda) || currentSearchType == null) {
2093-
return;
2094-
}
2095-
2096-
String title = fda.getActionBarTitle(mFile, currentSearchType);
2097-
boolean isRoot = (getFileDepth() == OCFileDepth.Root) || fda.isRoot(mFile);
2098-
setActionBarTitle(title, isRoot);
2099-
}
2100-
2101-
protected void setActionBarTitle(final String title, Boolean showBackAsMenu) {
2102-
if (!(getActivity() instanceof FileDisplayActivity fda)) {
2103-
return;
2104-
}
2105-
2106-
final var actionBar = fda.getSupportActionBar();
2107-
if (actionBar == null) {
2108-
return;
2109-
}
2110-
2111-
fda.runOnUiThread(() -> {
2112-
viewThemeUtils.files.themeActionBar(fda, actionBar, title, showBackAsMenu);
2113-
});
2114-
}
2115-
21162077
@Override
21172078
public void onStart() {
21182079
super.onStart();

0 commit comments

Comments
 (0)