resultCode is always 0 and request is always -1, . Activity.onActivityResult();
Asked Answered
P

6

14

I would like to add this to another list of questions about resultCode == 0 and requestCode == 0.

Here is the synopsis: NoteActivity calls NoteSettingsActivity using startActivityForResult().

I have searched the web and when I pressed back button when super.onBackPressed(), the resultCode == 0. Well after further researching, it seems it returns this code whenever the back button is pressed, but after botching that super.onBackPressed() call and just simply finish() the application the onActivityResult()'s resultCode is still equals to 0. It goes the same with requestCode.

Also, I tried manipulating the manifest file, I have done so many changes just to get this work but nothings works for me.

Here is the snippet. Note that I have reverted back to my previous commit so I have lost my recent modifications, but please take a look on the code I have wrote before I notice that the resultCode is always equals to 0 (ACTIVITY_CANCELED)

@Override
public void onNoteSettingsActivityCalled(Note note)
{
    Intent intent = new Intent(this, NoteSettingsActivity.class);
    intent.putExtra(NoteExtrasKey.EXTRA_NOTE_ID, note.getNoteID());

    startActivityForResult(intent, NoteRequest.REQUEST_UPDATE_SETTINGS);
}

Here is when the activity detected back press:

@Override
public void onBackPressed()
{   
    Log.i(NoteApplication.TAG, "NoteSettingsActivity.onBackPressed() has been called.");

    Intent intent = new Intent();
    intent.putExtra(NoteExtrasKey.EXTRA_NOTE_REMINDENABLED , mRemindEnabled);
    intent.putExtra(NoteExtrasKey.EXTRA_NOTE_REMINDEVERY   , mDaysSelected);
    intent.putExtra(NoteExtrasKey.EXTRA_NOTE_REMINDON      , String.valueOf(mRemindDateTime));
    intent.putExtra(NoteExtrasKey.EXTRA_NOTE_ID            , mTargetNoteID);

    if(getParent() != null)
        getParent().setResult(Activity.RESULT_OK, intent);
    else
        setResult(Activity.RESULT_OK, intent);

    super.onBackPressed();
}

Here's how NoteActivity received the resulting call.

    @Override
public void onActivityResult(int result, int request, Intent intent)
{
    super.onActivityResult(result, request, intent);

    Log.i(NoteApplication.TAG, "NoteActivity.onActivityResult() has been called.");
    Log.i(NoteApplication.TAG, "NoteActivity.onActivityResult() result = " + result + " request = " + request);

    if(result == Activity.RESULT_CANCELED)
        return;

    switch(request)
    {
        case NoteRequest.REQUEST_UPDATE_SETTINGS:

            if(intent == null) return;

            int noteID = intent.getIntExtra(NoteExtrasKey.EXTRA_NOTE_ID, -1);
            String remindOnString = intent.getStringExtra(NoteExtrasKey.EXTRA_NOTE_REMINDON);

            if(remindOnString != null && !remindOnString.equals(""))
                mRemindDateTime = Timestamp.valueOf(remindOnString);

            mHasSettingsEnabled = true;
            mRemindEnabled = intent.getBooleanExtra(NoteExtrasKey.EXTRA_NOTE_REMINDENABLED, false);
            mSelectedDays = intent.getIntegerArrayListExtra(NoteExtrasKey.EXTRA_NOTE_REMINDEVERY);

            if(noteID < 0)
            {
                Note note = mNoteDatabaseHelper.getNote(noteID);
                note.setRemindEnabled(mRemindEnabled);
                note.remindEvery(mSelectedDays);
                note.remindOn(mRemindDateTime);

                onNoteItemUpdated(note); 
            }

            Log.i(NoteApplication.TAG, "NoteActivity.onActivityResult() NoteRequest.REQUEST_UPDATE_SETTINGS called.");

            break;

        default:
            Log.i(NoteApplication.TAG, "NoteActivity.onActivityResult() : unknown request code = " + request);
            break;
    }
}

resultCode equals 0 and requestCode requals -1 when I ran this. I have checked the intent passed on this and it is not null.

Here are the questions very related to this question. None of them worked:

I am losing a lot of important hours working on my project just figuring out what makes the value for resultCode and requestCode lose the value I sent along the way.

Any help and guidance will be appreciated. Thank you very much!

Phane answered 30/4, 2015 at 15:44 Comment(1)
Any flags on that second Activity?Reyreyes
C
4

The correct signature is

public void onActivityResult(int requestCode, int resultCode, Intent data) {

You have request and result mixed up.

Cibis answered 30/4, 2015 at 16:13 Comment(2)
I was just a bit tired. I could have noticed! Thank you very much! You saved my life.Phane
I did not mix up the signature, but I kept checking resultCode and thought I was looking requestCode, hope it would help someone who comes to look for the answer later.Variola
B
29

In my case, I got the resultCode == 0 error because of two reasons.

Firstly, I copied the method from a fragment instead of from another activity:

protected void onActivityResult(int requestCode, int resultCode, Intent data) {

A good way to prevent this from happening is to @Override your method so that you know you are overriding the default activity onActivityResult method signature.

Use the correct signature:

public void onActivityResult(int requestCode, int resultCode, Intent data) {

Secondly, I was doing this:

    super.onBackPressed();
    Intent intent = new Intent();
    intent.putExtra(getString(R.string.like_count), people.size());
    Log.e(TAG, people.size() +" people ");
    setResult(RESULT_OK, intent);
    finish();

The super.onBackPressed() set the resultCode before it was actually set hence it resulted in the 0.

A simple fix cause it to work properly:

    Intent intent = new Intent();
    intent.putExtra(getString(R.string.like_count), people.size());
    Log.e(TAG, people.size() +" people ");
    setResult(RESULT_OK, intent);
    super.onBackPressed();
Bourassa answered 3/3, 2016 at 17:55 Comment(3)
It saves my so much time and tention..:) use super.onBackPressed();Keynesianism
I am not sure if IIRC, there was once a time I trusted the intellisense in previous version in Android Studio and by god I swear it highlighted the first param to be filled in as 'resultCode' and then followed by 'requestCode' so maybe that is why.Phane
nice and clean answer +1Norton
D
4

The call to super.onBackPressed() is the one that will in the end set the result code and send it back to the calling activity. Replace that call with a call to finish() and you should get the result code that you are looking for.

Hope this helps!

Dividivi answered 30/4, 2015 at 15:47 Comment(5)
On the link I have posted, I did similar thing. That doesn't work.Phane
Yep, confirmed, I have tried as suggested. It did not work as well. I will try to see if this is reproducible when I create a button and setResult from there.Phane
Is the getParent() call really necessary here? If NoteActivity is the calling activity and NoteSettingsActivity is the called one, you can just call setResult() and not getParent().setResult() and then call finish() to terminate the NoteSettingsActivity. Have you tried that?Dividivi
I personally always used setResult() and then finish() and it worked. Do not use super.onBackPressed().Reyreyes
I have done it as well. I mentioned that was on previous commit. But that doesn't work as well.Phane
C
4

The correct signature is

public void onActivityResult(int requestCode, int resultCode, Intent data) {

You have request and result mixed up.

Cibis answered 30/4, 2015 at 16:13 Comment(2)
I was just a bit tired. I could have noticed! Thank you very much! You saved my life.Phane
I did not mix up the signature, but I kept checking resultCode and thought I was looking requestCode, hope it would help someone who comes to look for the answer later.Variola
V
1

Was facing the same issue while doing uninstalling an application from the device. It got resolved by putting: intentObject.putExtra(Intent.EXTRA_RETURN_RESULT, true); before you do startActivityForResult.

Vacua answered 25/1, 2016 at 7:20 Comment(0)
B
0

For the comers from The Android Big Nerd Ranch Book, it's probably a typo in your code, make sure you're comparing correctly:

if (requestCode == REQUEST_CODE_CHEAT) {
if (RESULT_OK == resultCode) {...} 
...
}

Reminder: RESULT_OK is always -1 and it's existence means setResult() was called from the child Activity.

Babbler answered 2/4, 2018 at 22:45 Comment(0)
O
0

I know I am late to the party, but: I had the sam problem, for me it did not work because I was calling onBackPressedDispatcher.onBackPressed() before setting the result.

Like this:

override fun onBackPressed() {
    onBackPressedDispatcher.onBackPressed() // <---- PROBLEM
    val intent = Intent()
    intent.putExtra("KEY", "VALUE"))
    setResult(RESULT_OK, intent);
    // AND here I tried with this calls without luck. -> requireActivity().finish() or super.onBackPressed()
}
Obscene answered 10/3, 2021 at 19:59 Comment(0)

© 2022 - 2024 — McMap. All rights reserved.