Returning from an activity using navigateUpFromSameTask()

Question

I have two activities, A and B. When activity A is first started, it accesses the Intent passed to it (because the Bundle is null, as it should be the first time through), and displays information accordingly:

CustInfo m_custInfo;
...
protected void onCreate(Bundle savedInstanceState)
{
    ...
    Bundle bundle = (savedInstanceState == null) ? getIntent().getExtras() : savedInstanceState;
    m_custInfo = (CustInfo) m_bundle.getSerializable("CustInfo");
    if (m_custInfo != null
        ...
}

This works fine the first time through. The EditText controls and ListView are filled out correctly.

Now, when an item in the list is clicked, activity B is started to show the details:

m_custInfo = m_arrCustomers.get(pos);

Intent intent = new Intent(A.this, B.class);
intent.putExtra("CustInfo", m_custInfo); // CustInfo is serializable
// printing this intent, it shows to have extras and no flags

startActivityForResult(intent, 1);

Right before acivity B is started, the framework calls A's overridden onSaveInstanceState():

protected void onSaveInstanceState(Bundle outState)
{
    super.onSaveInstanceState(outState);

    outState.putSerializable("CustInfo", m_custInfo);
}

In activity B, when the Up button is pressed in the action bar, I want to return to activity A and have it be in the same state as it was before:

public boolean onOptionsItemSelected(MenuItem item)
{
    if (item.getItemId() == android.R.id.home)
    {
        Intent intent = NavUtils.getParentActivityIntent(this);
        // printing this intent, it shows to have flags but no extras

        NavUtils.navigateUpFromSameTask(this); // tried finish() here but that created an even bigger mess
        return true;
    }
    ...
}

Herein lies the problem, when in onCreate() of activity A the second time, the Bundle parameter is null and getExtras() returns null. Since onSaveInstanceState() was called, I would have expected the Bundle parameter to be non-null.

I've read about this issue on other web sites, have tried the suggestions, but nothing works.

finish() is the way to go with Activity B. Yes, onCreate should have a bundle with it if you saved in in onSaveInstanceState(). Triple check your code as there must be a bug in it somewhere. You might also consider posting more complete code. — jsmith, Jan 22 '13 at 17:10
jsmith -- I was trying to keep the code brief but relevant. What else can I add that would help? — DavidCrow, Jan 23 '13 at 15:48

score 133 · Answer 1 · edited May 23 '17 at 11:54

If you want your application to react this way, you should declare the launch mode of your activity A as:

android:launchMode="singleTop"

in your AndroidManifest.xml.

Otherwise android uses standard launch mode, which means

"The system always creates a new instance of the activity in the target task"

and your activity is recreated (see Android documentation).

With singleTop the system returns to your existing activity (with the original extra), if it is on the top of the back stack of the task. There is no need to implement onSaveInstanceState in this situation.

savedInstanceState is null in your case, because your activity was not previously being shut down by the OS.

Notice (thanks, android developer for pointing to this):

While this is a solution to the question, it will not work if the activity one returns to is not on the top of the back stack. Consider the case of activity A starting B, which is starting C, and C's parent activity is configured to be A. If you call NavigateUpFromSameTask() from C, the activity will be recreated, because A is not on top of the stack.

In this case one can use this piece of code instead:

Intent intent = NavUtils.getParentActivityIntent(this); 
intent.setFlags(Intent.FLAG_ACTIVITY_CLEAR_TOP|Intent.FLAG_ACTIVITY_SINGLE_TOP); 
NavUtils.navigateUpTo(this, intent);

this forces you to have the activity always launch this way. it might be better to do it only for those specific times that you need it, as shown here: http://stackoverflow.com/a/20306670/878126 — android developer, Dec 05 '13 at 13:00
@androiddeveloper Thanks for pointing to this, I have edited the answer accordingly — yonojoy, Dec 05 '13 at 15:39
Another good reference: http://developer.android.com/training/implementing-navigation/ancestral.html#NavigateUp — James Liu, Aug 26 '14 at 03:02
The second part of your answer solved my issue since I needed more then one instance of the activity I needed to navigate back. This should be the accepted answer! — Markus, Mar 06 '16 at 09:56

Mark Doliner · Answer 2 · 2020-08-10T01:05:02.200

This happens because NavUtils.navigateUpFromSameTask() basically just calls startActivity(intentForActivityA). If activity A uses the default android:launchMode="standard" then a new instance of the activity is created and the saved state is not used. There are three ways to fix this, with various advantages and disadvantages.

Option 1

Override getParentActivityIntent() in activity B and specify the appropriate extras needed by activity A:

@Override
public Intent getParentActivityIntent() {
    Intent intent = super.getParentActivityIntent();
    intent.putSerializable("CustInfo", m_custInfo);
    return intent;
}

Note: If you're using AppCompatActivity and the toolbar from androidx or the support library then you'll need to override getSupportParentActivityIntent(), instead.

I feel that this is the most elegant solution as it works regardless of how the user navigated to activity B. The two options listed below don't work correctly if the user navigates directly to activity B without going through activity A, for example if activity B is launched from a notification or URL handler. I think this scenario is the reason the 'up' navigation API is complicated and doesn't simply act as a dumb back button.

One downside with this solution is that the standard starting-a-new-activity transition animation is used rather than the returning-to-previous-activity animation.

Option 2

Intercept the up button and treat it as a dumb back button by overriding onNavigateUp():

@Override
public boolean onNavigateUp() {
    onBackPressed();
    return true;
}

Note: If you're using AppCompatActivity and the toolbar from androidx or the support library then you'll need to override onSupportNavigateUp(), instead.

This solution is a bit hacky and only works for applications where "up" and "back" should behave the same. This is probably rare, because if "up" and "back" should behave the same then why bother having an up button? One advantage of this solution is that the standard returning-to-previous-activity animation is used.

Option 3

As suggested by yonojoy, set android:launchMode="singleTop" on activity A. See his answer for details. Note that singleTop isn't appropriate for all applications. You'll have to try it and/or spend some time reading the documentation to decide for yourself.

I think this is the correct answer. Thanks I don't want my parent activity to always be SingleTop, i just want UP to pass some state. — Jared Kells, Dec 19 '14 at 04:53
Love the solution to call onBackPressed. It works perfectly for me, thanks! — batoutofhell, Apr 28 '15 at 20:43
Option 1. How can I extract data if A - extends Fragment, B - extends AppCompatActivity ? I've overrided ``getParentActivityIntent() in B``, but how to use it in A? — user25, Jun 11 '16 at 06:38
user25: Are you asking how A can extract data from the Intent returned by `getParentActivityIntent()` from B? You should be able to use the `getIntent()` method on fragment A's parent Activity. This page discusses how fragments can access intents: https://stackoverflow.com/questions/11387740/where-how-to-getintent-getextras-in-an-android-fragment Note that if you're using android:launchMode="singleTop" then the Activity hosting fragment A might need to override onNewIntent() and call this.setIntent(). — Mark Doliner, Jun 13 '16 at 17:18
Option 2 should not be used. Useful presentation that explains it really well. https://speakerdeck.com/jgilfelt/this-way-up-implementing-effective-navigation-on-android — TheLibrarian, Sep 19 '17 at 09:06
@TheLibrarianCz That's fair. I edited my post and added a note to try to steer people away from it. There's good info in that presentation! — Mark Doliner, Sep 24 '17 at 06:28

score 4 · Answer 3 · answered Dec 19 '14 at 05:09

Instead of calling NavUtils.navigateUpFromSameTask

You can get the parent intent and modify it before navigating. For example:

Intent intent = NavUtils.getParentActivityIntent(this);
intent.putExtra("CustInfo", m_custInfo); // CustInfo is serializable
NavUtils.navigateUpTo(this, intent);

This will behave exactly the same as NavUtils.navigateUpFromSameTask but will allow you to add some extra properties to the intent. You can have a look at the code for NavUtils.navigateUpFromSameTask it's very simple.

public static void navigateUpFromSameTask(Activity sourceActivity) {
    Intent upIntent = getParentActivityIntent(sourceActivity);

    if (upIntent == null) {
        throw new IllegalArgumentException("Activity " +
                sourceActivity.getClass().getSimpleName() +
                " does not have a parent activity name specified." +
                " (Did you forget to add the android.support.PARENT_ACTIVITY <meta-data> " +
                " element in your manifest?)");
    }

    navigateUpTo(sourceActivity, upIntent);
}

Making the parent activity SINGLE_TOP may work for some simple applications, but what if this activity wasn't launched directly from it's parent? OR you might legitimately want the parent to exist in multiple places in the back stack.

android developer · Answer 4 · 2018-12-26T08:21:30.733

I think this is a more general solution, to be added to your base Activity class:

@Override
public boolean onOptionsItemSelected(final MenuItem item) {
    if (item.getItemId() == android.R.id.home) {
        final Intent upIntent = NavUtils.getParentActivityIntent(this);
        if (upIntent == null)
            onBackPressed();
        else
            //optionally add this flag to the intent: Intent.FLAG_ACTIVITY_SINGLE_TOP
            NavUtils.navigateUpTo(this, upIntent);
        return true;
    }
    return super.onOptionsItemSelected(item);
}

And for each Activity that you wish that would navigate to some specific parent Activity, use this on the manifest:

    <activity android:parentActivityName="pathToParentActivity" ...>
        <meta-data
            android:name="android.support.PARENT_ACTIVITY"
            android:value="pathToParentActivity"/>
    </activity>

And, if your minSdk is from API 16, remove the "meta-data" tag, so you have this instead in the manifest:

    <activity android:parentActivityName="pathToParentActivity" ...>
    </activity>

More information here

Stan Smith · Answer 5 · 2014-02-18T15:07:28.413

-1

You mention:

In activity B, when the Up button is pressed in the action bar, I want to return to activity A and have it be in the same state as it was before.

If your content on Activity A resides in a Fragment then call setRetainInstance(true) in the onCreate() of the Fragment.

Then the Fragment instance will be retained across Activity recreation.

edited Feb 18 '14 at 15:07

answered Feb 17 '14 at 22:00

Stan Smith

896
7
19

Returning from an activity using navigateUpFromSameTask()

5 Answers5

Option 1

Option 2

Option 3

Linked