Search code examples
androidbroadcastreceiver

Alarm Manager does not fire a broadcast


I've got a service which setups an AlarmManager as shown:

private void setupInsuranceNotification(@NonNull String plate, @NonNull LocalDate insuranceDeadline)
    {
        LocalDate now = LocalDate.now();
        int daysToDeadline = getDaysBetweenDates(now, insuranceDeadline);
        String notificationTitle = "Assicurazione " + plate;
        String notificationMessage;
        int millsBetweenNotifications = 1000 * 60 * 60 * 24; // 1 day default
        LocalDate notificationStart = now;

        if (daysToDeadline > 0)
        {
            notificationMessage = daysToDeadline + " giorni alla scadenza.";

            if (daysToDeadline > 30)
            {
                // show the notification only when they are 30 days left
                notificationStart = now.withFieldAdded(DurationFieldType.days(), daysToDeadline - 30);
            }
        }
        else if (daysToDeadline == 0)
        {
            notificationMessage = "Assicurazione scaduta oggi.";
        }
        else
        {
            if (daysToDeadline >= -15)
            {
                // tolerance period
                notificationMessage = Math.abs(daysToDeadline) + " alla fine del periodo di tolleranza!";
                millsBetweenNotifications = 1000 * 60 * 60 * 6; // 6 hours
            }
            else
            {
                // insurance expired
                notificationMessage = "Periodo di tolleranza finito! L'assicurazione è scaduta!";
                millsBetweenNotifications = 1000 * 60 * 60; // 1 hour
            }
        }

        Intent notificationIntent = new Intent(this, NotificationPublisher.class);
        notificationIntent.putExtra("notification_title", notificationTitle);
        notificationIntent.putExtra("notification_message", notificationMessage);
        PendingIntent alarmIntent = PendingIntent.getBroadcast(this, 0, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT);
        AlarmManager alarmManager = (AlarmManager) getSystemService(Context.ALARM_SERVICE);
        alarmManager.setRepeating(AlarmManager.RTC_WAKEUP, notificationStart.toDate().getTime(), millsBetweenNotifications, alarmIntent);
    }

this is my NotificationPublisher class:

public class NotificationPublisher extends BroadcastReceiver
{
    private static final int NOTIFICATION_ID = 1;

    @Override
    public void onReceive(Context context, Intent intent)
    {
        String title = intent.getStringExtra("notification_title");
        String message = intent.getStringExtra("notification_message");
        Notification.Builder builder = new Notification.Builder(context);
        NotificationManager manager = (NotificationManager) context.getSystemService(Context.NOTIFICATION_SERVICE);

        builder.setContentTitle(title);
        builder.setContentText(message);
        builder.setSmallIcon(R.drawable.ic_notification);
        manager.notify(NOTIFICATION_ID, builder.build());
    }
}

My problem is that the setupInsuranceNotification is correctly executed untill the end (there isn't any exception), but my NotificationPublisher is never fired (I've put a breakpoint on in to check).

EDIT

this is my onHandleIntent from NotificationService class:

@Override
    protected void onHandleIntent(@Nullable Intent intent)
    {
        Debug.waitForDebugger();

        if (Utilities.vehicleFileExists(this))
        {
            try
            {
                DocumentBuilderFactory dbFactory = DocumentBuilderFactory.newInstance();
                DocumentBuilder dBuilder = dbFactory.newDocumentBuilder();
                Document doc = dBuilder.parse(Utilities.getVehicleListFile(this));
                Element root = doc.getDocumentElement();

                root.normalize();

                NodeList vehicleNodeList = root.getElementsByTagName("vehicle");

                for (int i = 0; i < vehicleNodeList.getLength(); i++)
                {
                    Element vehicleElement = (Element) vehicleNodeList.item(i);
                    Element matriculationDateElement = (Element) vehicleElement.getFirstChild();
                    Element endInsuranceDateElement = (Element) matriculationDateElement.getNextSibling();
                    Element lastInspectionDateElement = (Element) endInsuranceDateElement.getNextSibling();

                    String plate = vehicleElement.getAttribute("plate");
                    LocalDate matriculationDate = LocalDate.parse(matriculationDateElement.getFirstChild().getNodeValue(), DateTimeFormat.forPattern(Utilities.Formats.Date.ITALIAN));
                    LocalDate endInsuranceDate = LocalDate.parse(endInsuranceDateElement.getFirstChild().getNodeValue(), DateTimeFormat.forPattern(Utilities.Formats.Date.ITALIAN));
                    LocalDate lastInspectionDate = null;

                    if (lastInspectionDateElement.getFirstChild() != null) // <inspection_dates/>
                    {
                        lastInspectionDate = LocalDate.parse(lastInspectionDateElement.getFirstChild().getNodeValue(), DateTimeFormat.forPattern(Utilities.Formats.Date.ITALIAN));
                    }

                    LocalDate inspectionDeadline = getInspectionDeadline(matriculationDate, lastInspectionDate);

                    setupInsuranceNotification(plate, endInsuranceDate);
                    setupInspectionNotification(plate, inspectionDeadline);
                }
            }
            catch (ParserConfigurationException | SAXException | IOException ignored) {}
        }
    }

Solution

  • Your NotificationPublisher is fired only for last iteration because of same requestCode that you used in alarm PendingIntent.

    As you are using requestCode as 0 for all alarm, its just overrides the previous alarm data. That's why only for last alarm your NotificationPublisher's onReceive() executed.

    SOLUTION:

    Use different REQUEST_CODE for different alarm PendingIntent.

    PendingIntent alarmIntent = PendingIntent.getBroadcast(this, REQUEST_CODE, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT);
    

    Hope this will help~