Why is onActivityResult not called in Android?

Question

When I my app is launched I am showing a splash screen. That page is been shown for 10 sec, running on a thread.

When it switches over to new activity on a result I want o hit a URL in server and I will be getting a return value which I can use for my further implements.

Here is my code:

private final int SPLASH_DISPLAY_LENGHT = 1000;

new Handler().postDelayed(new Runnable()
        {
            @Override
            public void run() 
            {
                Log.e("Handler ","run");
                Intent myIntent = new Intent(getApplicationContext(), CaptureActivity.class);
                startActivityForResult(myIntent, imgDL);
                finish();
            }
        }, SPLASH_DISPLAY_LENGHT);



public void onActivityResult(int requestCode, int resultCode, final Intent data) 
      {
          super.onActivityResult(requestCode, resultCode, data);
          if (requestCode == imgDL) 
          {     
              Log.e("onActivity Result","");
              urlHitMethod("http://XXXXXXXXXXXXXXXXXX.com/banner_scan");
          }
      }

But here the onActivityResult is not called. How to fix this?

Secondhand

Why are you finishing your caller activity? – Diarthrosis

in my next activity when the user clicks the back button i want the app to get closed.... – Secondhand

when u using finish then it will never come in onActivityResult so remove it – Moran

Can you post more codes of caller Activity and some lines of called Activity where you finishing that? – Diarthrosis

now i am trying this for a sample purpose, the above code is my caller activity and in my called activity i am just showing a layout without any codes. just typed as setContentView(R.layout.image); – Secondhand

Answer 1

In the CaptureActivity.class you have to set the result and then check in the onActivityResult in the First Activity the result code

In the CaptureActivity.class it should be like the following

 Intent in = new Intent();
    setResult(1,in);//Here I am Setting the Requestcode 1, you can put according to your requirement
    finish();

Exclamation

Actually your comment is wrong. You don't set the requestCode, but the resultCode. And if I'm right, you could use Activity.RESULT_CANCELLED and Activity.RESULT_OK – Tupi

My problem is that when you set the result of the activity to RESULT_OK, onActivityResult on the waiting activity was never called.When i changed to result value to 1 it worked. – Wadley

-1. @Tupi is right - you aren't setting the requestCode, you're passing a literal value in for resultCode, which actually corresponds to RESULT_FIRST_USER, not RESULT_OK – Ungenerous

Answer 2

Also, please note than if you base activity (the one calling startActivityForResult) can't use the flag noHitory in the manifest.

If you do so, onActivityResult will never be called.

Marianamariand

Answer 3

try this

Intent myIntent = new Intent(activity.this, CaptureActivity.class);

and

@Override
public void onActivityResult(int requestCode, int resultCode, final Intent data) 
      {
          super.onActivityResult(requestCode, resultCode, data);
          if (requestCode == imgDL) 
          {     
              Log.e("onActivity Result","");
              urlHitMethod("http://XXXXXXXXXXXXXXXXXX.com/banner_scan");
          }
          if(resultCode==RESULT_OK)
      {
    Log.e("onActivity Result","come in onactivity result ok"); 

      }
          else
          {
    Log.e("onActivity Result","come in onactivity result with error"); 

      }



      }

Moran

when u using finish then it will never come in onActivityResult so remove it – Moran

Answer 4

If you are using onActivityResult, then you should not finish the activity when starting with intent otherwise it will crash the app. Thanks.

Miquelon