20

I have this code:

Intent intent = new Intent();
        intent.setAction(Intent.ACTION_PICK);
        intent.setData(ContactsContract.Contacts.CONTENT_URI);
        intent.putExtra(EXTRA_ONLINE_ID, (String) v.getTag());
        startActivityForResult(intent, PICK_CONTACT);

Then on response:

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


        switch (reqCode) {
            case (PICK_CONTACT):
                if (resultCode == Activity.RESULT_OK) {
                    try {
                        Uri contactData = data.getData();
                        String onlineid = data.getStringExtra(EXTRA_ONLINE_ID);
                    } catch (Exception e) {
                        e.printStackTrace();
                    }
                }
                break;
        }
        super.onActivityResult(reqCode, resultCode, data);
    }

the onlineid variable is null. How can I pass a value and then to receive it back?

EDIT

I even tried,

Bundle extras = data.getExtras(); // returns null
Carl Manaster
  • 39,912
  • 17
  • 102
  • 155
Pentium10
  • 204,586
  • 122
  • 423
  • 502

4 Answers4

6

This is done by design; system activities will not send back the extras with which they're called, so you have to manage the data elsewhere.

Luckily, the resultCode parameter is fully controlled by yourself, which means that you can use it to index your data.

private final int PICK_CONTACT = 0;
private Bundle[] myDataTransfer = { null };

...

Bundle myData = new Bundle();
myData.putString(EXTRA_ONLINE_ID, (String) v.getTag());
myDataTransfer[PICK_CONTACT] = myData;
// create intent and all
startActivityForResult(intent, PICK_CONTACT);

...

public void onActivityResult(int reqCode, int resultCode, Intent data) {
    if (resultCode == PICK_CONTACT) {
        Bundle myData = myDataTransfer[resultCode];
        String onlineid = myData.getString(EXTRA_ONLINE_ID);
    }
}

I'm not a Java programmer, there must be a nicer way to implement a map of Bundles, but this works :)

Ja͢ck
  • 170,779
  • 38
  • 263
  • 309
  • 1
    When you start an activity, your activity may get killed. So the index will be gone. I think you’ll need to persist the Bundle somewhere outside your App’s memory. – rumpel Feb 11 '15 at 15:53
  • To add to the comment by @rumpel, you can test this by going into Settings > Developer Options and enabling "Don't keep activities". This way you'll be able to test this usecase and find that the data in myDataTransfer will get wiped out unless you save / restore manually in onSaveInstanceState – strangetimes Sep 03 '16 at 10:46
  • this doesn't even work in the best case scenario because in you mixed requestCode and resultCode. The correct signature is: onActivityResult(int requestCode, int resultCode, Intent data) And the requestCode will always be PICK_CONTACT, and the resultCode will usually be Activity.RESULT_OK==-1. – Gavriel Nov 13 '19 at 14:02
  • @Gavriel it’s been a long time since I touched this answer; you’re probably right, in which case you could consider writing a new answer when you find out or edit this one :) – Ja͢ck Nov 13 '19 at 14:17
0

ok Check if your Activity android:launchMode is configured as SingleTask or SingleInstance! that must be the problem :)

Jorgesys
  • 124,308
  • 23
  • 334
  • 268
0

The EXTRA_ONLINE_ID field will have to be set in the activity that you launched using setResult. If it's not setting that value in the returned Intent (which is different from what you sent) then you will get a null value.

Abhinav
  • 38,516
  • 9
  • 41
  • 49
-1

I was running into some problems with this as well.

Instead of this line

 intent.putExtra(EXTRA_ONLINE_ID, (String) v.getTag());

Try

 intent.putExtra(EXTRA_ONLINE_ID, "" + v.getTag());
Dom
  • 1