Skip to content

Commit 7b9ece7

Browse files
Pass multireddit data back to SearchActivity after selecting a multireddit in the original SearchActivity and opening SearchResultActivity.
1 parent 8625b3b commit 7b9ece7

File tree

2 files changed

+20
-30
lines changed

2 files changed

+20
-30
lines changed

app/src/main/java/ml/docilealligator/infinityforreddit/activities/SearchActivity.java

Lines changed: 11 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -309,6 +309,14 @@ public void onDismissed(Snackbar transientBottomBar, int event) {
309309
setSearchInThingText();
310310
} else {
311311
query = getIntent().getStringExtra(EXTRA_QUERY);
312+
searchInSubredditOrUserName = getIntent().getStringExtra(EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME);
313+
searchInMultiReddit = getIntent().getParcelableExtra(EXTRA_SEARCH_IN_MULTIREDDIT);
314+
searchInThingType = getIntent().getIntExtra(EXTRA_SEARCH_IN_THING_TYPE, SelectThingReturnKey.THING_TYPE.SUBREDDIT);
315+
if (searchInSubredditOrUserName != null) {
316+
binding.subredditNameTextViewSearchActivity.setText(searchInSubredditOrUserName);
317+
} else if (searchInMultiReddit != null) {
318+
binding.subredditNameTextViewSearchActivity.setText(searchInMultiReddit.getDisplayName());
319+
}
312320
}
313321
bindView();
314322

@@ -339,16 +347,6 @@ public void onDismissed(Snackbar transientBottomBar, int event) {
339347
requestThingSelectionForCurrentActivityLauncher.launch(intent);
340348
});
341349
}
342-
343-
Intent intent = getIntent();
344-
searchInSubredditOrUserName = intent.getStringExtra(EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME);
345-
searchInMultiReddit = intent.getParcelableExtra(EXTRA_SEARCH_IN_MULTIREDDIT);
346-
searchInThingType = intent.getIntExtra(EXTRA_SEARCH_IN_THING_TYPE, SelectThingReturnKey.THING_TYPE.SUBREDDIT);
347-
if (searchInSubredditOrUserName != null) {
348-
binding.subredditNameTextViewSearchActivity.setText(searchInSubredditOrUserName);
349-
} else if (searchInMultiReddit != null) {
350-
binding.subredditNameTextViewSearchActivity.setText(searchInMultiReddit.getDisplayName());
351-
}
352350
}
353351

354352
private void bindView() {
@@ -450,11 +448,9 @@ private void openSearchResult(String query) {
450448
} else {
451449
Intent intent = new Intent(SearchActivity.this, SearchResultActivity.class);
452450
intent.putExtra(SearchResultActivity.EXTRA_QUERY, query);
453-
if (searchInSubredditOrUserName != null) {
454-
intent.putExtra(SearchResultActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, searchInSubredditOrUserName);
455-
intent.putExtra(SearchResultActivity.EXTRA_SEARCH_IN_MULTIREDDIT, searchInMultiReddit);
456-
intent.putExtra(SearchResultActivity.EXTRA_SEARCH_IN_THING_TYPE, searchInThingType);
457-
}
451+
intent.putExtra(SearchResultActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, searchInSubredditOrUserName);
452+
intent.putExtra(SearchResultActivity.EXTRA_SEARCH_IN_MULTIREDDIT, searchInMultiReddit);
453+
intent.putExtra(SearchResultActivity.EXTRA_SEARCH_IN_THING_TYPE, searchInThingType);
458454
startActivity(intent);
459455
finish();
460456
}

app/src/main/java/ml/docilealligator/infinityforreddit/activities/SearchResultActivity.java

Lines changed: 9 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -345,11 +345,9 @@ public void onPageSelected(int position) {
345345
}
346346
case SharedPreferencesUtils.OTHER_ACTIVITIES_BOTTOM_APP_BAR_FAB_SEARCH: {
347347
Intent intent = new Intent(this, SearchActivity.class);
348-
if (mSearchInSubredditOrUserName != null && !mSearchInSubredditOrUserName.equals("")) {
349-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, mSearchInSubredditOrUserName);
350-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_MULTIREDDIT, mSearchInMultiReddit);
351-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_THING_TYPE, mSearchInThingType);
352-
}
348+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, mSearchInSubredditOrUserName);
349+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_MULTIREDDIT, mSearchInMultiReddit);
350+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_THING_TYPE, mSearchInThingType);
353351
startActivity(intent);
354352
break;
355353
}
@@ -435,11 +433,9 @@ public boolean onOptionsItemSelected(@NonNull MenuItem item) {
435433
return true;
436434
} else if (itemId == R.id.action_search_search_result_activity) {
437435
Intent intent = new Intent(this, SearchActivity.class);
438-
if (mSearchInSubredditOrUserName != null && !mSearchInSubredditOrUserName.equals("")) {
439-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, mSearchInSubredditOrUserName);
440-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_MULTIREDDIT, mSearchInMultiReddit);
441-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_THING_TYPE, mSearchInThingType);
442-
}
436+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, mSearchInSubredditOrUserName);
437+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_MULTIREDDIT, mSearchInMultiReddit);
438+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_THING_TYPE, mSearchInThingType);
443439
intent.putExtra(SearchActivity.EXTRA_QUERY, mQuery);
444440
finish();
445441
startActivity(intent);
@@ -549,11 +545,9 @@ public void fabOptionSelected(int option) {
549545
break;
550546
case FABMoreOptionsBottomSheetFragment.FAB_OPTION_SEARCH:
551547
Intent intent = new Intent(this, SearchActivity.class);
552-
if (mSearchInSubredditOrUserName != null && !mSearchInSubredditOrUserName.equals("")) {
553-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, mSearchInSubredditOrUserName);
554-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_MULTIREDDIT, mSearchInMultiReddit);
555-
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_THING_TYPE, mSearchInThingType);
556-
}
548+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_SUBREDDIT_OR_USER_NAME, mSearchInSubredditOrUserName);
549+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_MULTIREDDIT, mSearchInMultiReddit);
550+
intent.putExtra(SearchActivity.EXTRA_SEARCH_IN_THING_TYPE, mSearchInThingType);
557551
startActivity(intent);
558552
break;
559553
case FABMoreOptionsBottomSheetFragment.FAB_OPTION_GO_TO_SUBREDDIT: {

0 commit comments

Comments
 (0)