Broadcast receiver is not called

Question

In my Fragment, I have a button, when button pressed, I broadcast a custom intent in following way:

package com.my.store.fragments.shopping;

public class ShoppingFragment extends Fragment{
    ...
    @Override
    public void onStart(){
       super.onStart()

       myButton.setOnClickListener(new OnClickListener(){
             @Override
             public void onClick(View v){
                broadcastMyIntent(v);
             }
       });
    }

    public void broadcastMyIntent(View view){
     Intent intent = new Intent();
     intent.setAction("com.my.store.fragments.shopping.CUSTOM_INTENT");
     getActivity().sendBroadcast(intent);
    }
}

Then, I defined a broadcast receiver:

package com.my.store.utils;

public class MyReceiver extends BroadcastReceiver{

    @Override
    public void onReceive(Context context, Intent intent) {
        Toast.makeText(context, "Receive my intent", Toast.LENGTH_LONG).show();
    }
}

I register the receiver in AndroidManifest.xml:

<application
    ...>
    <activity ...>
       ...
    </activity>

    <!--this is the receiver which doesn't work-->
    <receiver android:name="com.my.store.utils.MyReceiver"> 
          <action android:name="com.my.store.fragments.shopping.CUSTOM_INTENT"/>
    </receiver>

    <!--I have another receiver here, it is working fine-->
   <receiver android:name="com.my.store.utils.AnotherReceiver">
        <intent-filter>
            <action android:name="android.intent.action.BOOT_COMPLETED"/>
        </intent-filter>
    </receiver>
</application>

I run my app, when I press the button, my receiver isn't called. Why?

score 2 · Accepted Answer · answered Jul 08 '13 at 14:06

2

You forgot to surround your <action> element with an <intent-filter> container.

answered Jul 08 '13 at 14:06

woot

3,671
4
26
39

I didn't forget that, I just didn't include because this link suggests me not to do that (because I got the same warning as the link mentioned): http://stackoverflow.com/questions/16112470/android-exported-receiver-does-not-require-permission-on-receivers-meant-to – Mellon Jul 08 '13 at 14:07
In the example you showed the answer is stating only to do that if you're using an `Intent` with the class name rather than an action string. You can do that by using Intent it = new Intent(context, MyReceiver.class) - So either you use this syntax or you surround your action element with an intent-filter container. – woot Jul 08 '13 at 14:13

score 0 · Answer 2 · edited Jul 09 '13 at 04:44

0

AndroidManifest.xml

<!--this is the receiver which doesn't work-->
<receiver android:name="com.my.store.utils.MyReceiver"> 
  <intent-filter>
   <action android:name="com.my.store.fragments.shopping.CUSTOM_INTENT"/>
  </intent-filter>
</receiver>

edited Jul 09 '13 at 04:44

Paul

26,170
12
85
119

answered Jul 08 '13 at 14:12

iStar

1,112
12
20

I got "Exported receiver does not require permission" in my AndroidManifest.xml when I include the that's why I removed that – Mellon Jul 08 '13 at 14:14
@Mellon Maybe you need add `````` under ``````. – iStar Jul 08 '13 at 14:16

score 0 · Answer 3 · answered Jul 08 '13 at 14:14

Try this

<receiver android:name="com.my.store.utils.MyReceiver"> 
   <intent-filter>
      <action android:name="com.my.store.fragments.shopping.CUSTOM_INTENT"/>
      <category android:name="android.intent.category.DEFAULT" />
   </intent-filter>
</receiver>

Broadcast receiver is not called

3 Answers3