Skip to content

Commit

Permalink
Removed menuItem from class property to local variable in onCreateMen…
Browse files Browse the repository at this point in the history
…uOptions. Also the same with searchView in FolderPickerActivity
  • Loading branch information
joragua committed Mar 15, 2024
1 parent 508de23 commit d51b707
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 8 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -57,9 +57,6 @@ open class FolderPickerActivity : FileActivity(),

private var isInSpacesScreen: Boolean = false

private lateinit var menuItem: MenuItem

private lateinit var searchView: SearchView

override fun onCreate(savedInstanceState: Bundle?) {
Timber.d("onCreate() start")
Expand Down Expand Up @@ -162,8 +159,8 @@ open class FolderPickerActivity : FileActivity(),

override fun onCreateOptionsMenu(menu: Menu?): Boolean {
menuInflater.inflate(R.menu.main_menu, menu)
menuItem = menu!!.findItem(R.id.action_search)
searchView = menuItem.getActionView() as SearchView
val menuItem = menu!!.findItem(R.id.action_search)
val searchView = menuItem.getActionView() as SearchView
searchView.maxWidth = Int.MAX_VALUE
searchView.queryHint = resources.getString(R.string.actionbar_search)
if (isInSpacesScreen) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -181,8 +181,6 @@ public class ReceiveExternalFilesActivity extends FileActivity

private boolean isInSpacesList;

private MenuItem menuItem;

Pattern pattern = Pattern.compile("[/\\\\]");

private ReceiveExternalFilesViewModel mReceiveExternalFilesViewModel;
Expand Down Expand Up @@ -758,7 +756,7 @@ private void initTargetFolder() {
public boolean onCreateOptionsMenu(Menu menu) {
MenuInflater inflater = getMenuInflater();
inflater.inflate(R.menu.main_menu, menu);
menuItem = menu.findItem(R.id.action_search);
MenuItem menuItem = menu.findItem(R.id.action_search);
mSearchView = (SearchView) menuItem.getActionView();
mSearchView.setMaxWidth(Integer.MAX_VALUE);
mSearchView.setQueryHint(getResources().getString(R.string.actionbar_search));
Expand Down

0 comments on commit d51b707

Please sign in to comment.