BroadcastReceiver doesn't work for the second time

Question

I'm trying to schedule notifications with AlarmManager It works perfectly when I schedule one notification but when I schedule two notification, the first notification is okay but the second one not works.

I figured out opening the app after few minutes will notify the second notification. I think something is wrong with my BroadcastReceiver

MainActivity.java

Intent intent = new Intent(context,NotificationClass.class);
intent.putExtra("notification_id", id);
PendingIntent pendingIntent = PendingIntent.getBroadcast(context,id,intent,PendingIntent.FLAG_UPDATE_CURRENT);
alarmManager.setExactAndAllowWhileIdle(AlarmManager.RTC_WAKEUP, calendar.getTimeInMillis(),pendingIntent);

Notification.java

public class NotificationClass extends BroadcastReceiver {
    @Override
    public void onReceive(Context context, Intent intent) {
        int id = intent.getIntExtra("notification_id",0);
        NotificationCompat.Builder builder = new NotificationCompat.Builder(context,"1")
            .setContentTitle("Notification")
            .setContentText("Content")
            .setSmallIcon(R.drawable.notif_ic);

        Notification notification = builder.build();
        NotificationManagerCompat notificationManager = NotificationManagerCompat.from(context);

        if (android.os.Build.VERSION.SDK_INT >= android.os.Build.VERSION_CODES.O) {
            NotificationChannel channel = new NotificationChannel("1","test", NotificationManager.IMPORTANCE_HIGH);
            notificationManager.createNotificationChannel(channel);
        }
        notificationManager.notify(id,notification);
}

AndroidManifest.xml

<receiver android:name=".NotificationClass" ></receiver>

I don't know what is wrong with my code. Can anybody help me with this?

I don't see anything obviously wrong. You should add some logging in your `BroadcastReceiver` and see if it is getting triggered and if the "extra" has the value you expect. — David Wasser, Apr 09 '20 at 12:39
Could you please add the piece of code that triggers the two notifications? — Diego Marcher, Apr 13 '20 at 17:59
Could be related to doze mode restrictions - https://stackoverflow.com/a/33150005/891373 — Froyo, Apr 15 '20 at 14:51

iamnaran · Answer 1 · 2020-04-14T09:40:43.607

Broadcast receiver to receive the data:

private BroadcastReceiver mReceiver = new BroadcastReceiver() {

    @Override
    public void onReceive(Context context, Intent intent) {
        String alertMessage = intent.getStringExtra("type");

        doNotificationAlertWorkHere(alertMessage);
    }
};

Register & Unregister your broadcast to avoid static leaks.

via the Android manifest file. (Statically)

<receiver android:name="YourBroadcastReceiverName"> </receiver>

via the Context.registerReceiver() and Context.unregisterReceiver() methods. (Dynamically)

 @Override
    protected void onPause() {
        super.onPause();
        // unregister broadcast
        LocalBroadcastManager.getInstance(this).unregisterReceiver(mReceiver);
    }

    @Override
    protected void onResume() {
        super.onResume();

        // register broadcast
        IntentFilter filter = new IntentFilter(Constants.ACTION);
        LocalBroadcastManager.getInstance(this).registerReceiver(mReceiver, filter);
    }

Send Broadcast like:

// public static final String ACTION = "ALERT";

Intent intent = new Intent(Constants.ACTION);
intent.putExtra("type", "SUP BRO. Stay Inside");
LocalBroadcastManager.getInstance(this).sendBroadcast(intent);

Knowledge Note :- Broadcast receiver is like a Cannon-fire to score a hit, you have to determine what to fire (eg. msg), where to fire (eg. activity). Load & unload the cannon to score another hit. (eg. Register & Unregister)

score 1 · Answer 2 · answered Apr 15 '20 at 05:16

I have tried it and it is working. Add your notification code inside onReceive.

Broadcast Receiver

class AlarmReceiver : BroadcastReceiver() {

override fun onReceive(context: Context, intent: Intent) {

 /*
  Your implementation

   */
 }
}

Mainfest

    <receiver
        android:name=".AlarmReceiver"
        android:exported="true"
        android:enabled="true" />

Creating pending intents

val alarmManager = activity.getSystemService(Activity.ALARM_SERVICE) as AlarmManager
        val alarmIntent = Intent(activity.applicationContext, AlarmReceiver::class.java) // AlarmReceiver1 = broadcast receiver

        val calendar = Calendar.getInstance()
        calendar.timeInMillis = timeInMilliSeconds

        val pendingIntent = PendingIntent.getBroadcast(activity, timeInMilliSeconds.toInt(), alarmIntent, PendingIntent.FLAG_CANCEL_CURRENT)
        alarmManager.set(AlarmManager.RTC_WAKEUP, calendar.timeInMillis, pendingIntent)

score 0 · Answer 3 · answered Apr 09 '20 at 01:59

0

First, make sure your notification Id is difference every single time you create a notification

Second, you miss tag intent-filter inside tag receive in manifest. pls check this https://developer.android.com/guide/components/broadcasts.

Hope this help!

answered Apr 09 '20 at 01:59

Chuong Le Van

274
1
8

Hi, thanks for your answer, As I said in my comment id is different each time, if it wasn't the second notification would be notified and for the second, what intent-filter i missed? can you help please? – Daniel Apr 09 '20 at 08:15
@Daniel Example in link above: ``` ``` that tag intent-filter sir. – Chuong Le Van Apr 09 '20 at 09:28
I don't think a intent-filter would be necessary for `AlarmManager` – Daniel Apr 09 '20 at 11:30
But did you try this? – Chuong Le Van Apr 09 '20 at 11:31
No, which intent-filter's action you suggest? – Daniel Apr 09 '20 at 11:42
intent-filter will receive action by which action you put in intent when you sent broadcast. So it can be customize. – Chuong Le Van Apr 09 '20 at 11:59
You don't need an `` because the `PendingIntent` contains an **explicit** `Intent` (ie: the component is specified in the `Intent`). You only need an `` if your `Intent` is **implicit** (ie: ACTION or DATA based) – David Wasser Apr 09 '20 at 12:35

BroadcastReceiver doesn't work for the second time

3 Answers3