Wierd behavior with pending intent changing the order of code changes everything

Question

hello guys i am fairly new to widgets and pending intent i am getting a weird behavior while clicking buttons I got 3 buttons on my widget Play /Pause

the broadcast for Play is working fine but when i click on the next button It is doing what previous button is supposed to do while previous button doesn't do anything at all

I made sure that id's weren't swapped.

here is the code

Intent playIntent = new Intent(BROADCAST_PLAYPAUSE);

        PendingIntent playPendingIntent = PendingIntent.getBroadcast(
                context, REQUEST_CODE, playIntent, PendingIntent.FLAG_UPDATE_CURRENT);
        controlButtons.setOnClickPendingIntent(
                R.id.bPlay, playPendingIntent);

        Intent nextIntent     = new Intent(BROADCAST_SWAP);
        nextIntent.putExtra("nextprev", 1);

        PendingIntent nextPendingIntent = PendingIntent.getBroadcast(
                context, REQUEST_CODE, nextIntent, PendingIntent.FLAG_UPDATE_CURRENT);
        controlButtons.setOnClickPendingIntent(
                R.id.bNext, nextPendingIntent);



        Intent previousIntent = new Intent(BROADCAST_SWAP);
        previousIntent.putExtra("nextprev", -1);
        PendingIntent previousPendingIntent = PendingIntent.getBroadcast(
                context, REQUEST_CODE, previousIntent, PendingIntent.FLAG_UPDATE_CURRENT);
        controlButtons.setOnClickPendingIntent(
                R.id.bPrevious, previousPendingIntent);
              appWidgetManager.updateAppWidget(appWidgetIds, controlButtons);

EDIT :I put the code of next button after the previous button and now nothing works...only play button works..has it something to do with their intents using the same string??and is this an error of the flags of the intent ??

by the way here is how my app looks

enter image description here

score 0 · Answer 1 · answered May 13 '14 at 17:16

You create the PendingIntent for NEXT button like this:

   Intent nextIntent     = new Intent(BROADCAST_SWAP);
   nextIntent.putExtra("nextprev", 1);
   PendingIntent nextPendingIntent = PendingIntent.getBroadcast(
            context, REQUEST_CODE, nextIntent, PendingIntent.FLAG_UPDATE_CURRENT);

Then you create the PendingIntent for PREV button like this:

   Intent previousIntent = new Intent(BROADCAST_SWAP);
   previousIntent.putExtra("nextprev", -1);
   PendingIntent previousPendingIntent = PendingIntent.getBroadcast(
            context, REQUEST_CODE, previousIntent, PendingIntent.FLAG_UPDATE_CURRENT);

Since the PendingIntent for PREV has exactly the same action (BROADCAST_SWAP) and requestCode (REQUEST_CODE) they are considered matching, therefore they point to the same PendingIntent objects. Since you specify PendingIntent.FLAG_UPDATE_CURRENT when you call getBroadcast() for the PREV button, this replaces the extras parameters in the PendingIntent with the ones for PREV button. This means that both PREV and NEXT button will act like PREV button.

To fix this you need to make sure that the PendingIntent for NEXT and PREV are unique. You can do this by providing either different requestCode or a different Intent action. Just making the extras different does not make them different!

hi,actually i did try changing the request code but the weird part is that too didn't help... — Ankit Srivastava, May 14 '14 at 08:51
i checked this out and it didn't help http://stackoverflow.com/questions/3009059/android-pending-intent-notification-problem?rq=1 — Ankit Srivastava, May 14 '14 at 08:52
Strange. That should work. Try changing the ACTION and see if that helps. That means passing a unique `String` in each call to `new Intent(...)` — David Wasser, May 14 '14 at 09:29

Wierd behavior with pending intent changing the order of code changes everything

I made sure that id's weren't swapped.

EDIT :I put the code of next button after the previous button and now nothing works...only play button works..has it something to do with their intents using the same string??and is this an error of the flags of the intent ??

1 Answers1