A notificação passa antigos extras de intenção

134

Estou criando uma notificação dentro de um BroadcastReceiver através deste código:

String ns = Context.NOTIFICATION_SERVICE;
        NotificationManager mNotificationManager = (NotificationManager) context.getSystemService(ns);
        int icon = R.drawable.ic_stat_notification;
        CharSequence tickerText = "New Notification";
        long when = System.currentTimeMillis();

        Notification notification = new Notification(icon, tickerText, when);
        notification.defaults |= Notification.DEFAULT_VIBRATE;
        long[] vibrate = {0,100,200,200,200,200};
        notification.vibrate = vibrate;
        notification.flags |= Notification.FLAG_AUTO_CANCEL;

        CharSequence contentTitle = "Title";
        CharSequence contentText = "Text";
        Intent notificationIntent = new Intent(context, NotificationActivity.class);
        notificationIntent.putExtra(Global.INTENT_EXTRA_FOO_ID, foo_id);
PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, 0);

        notification.setLatestEventInfo(context, contentTitle, contentText, contentIntent);

        int mynotification_id = 1;

        mNotificationManager.notify(mynotification_id, notification);

Quando clico na notificação, ela abre o NotificationActivity e, dentro da Activity, posso recuperar o foo_id do Intent-Bundle (por exemplo, 1)

No entanto, se outra notificação for acionada e eu clicar nela novamente, a atividade ainda receberá o valor "antigo" (1) do Intent-Bundle. Tentei limpar o pacote com clear (), mas estou recebendo o mesmo efeito. Eu acho que sth está errado com o meu código ..

BrianM
fonte
por favor, você pode me dizer como você obter os dados a partir pendente intenção
user49557
perceber que estava enviando extras antigos, facilitou minha triagem.
precisa

Respostas:

268

Você está enviando o mesmo código de solicitação para o intens pendente. Mude isso:

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, 0);

Para:

PendingIntent contentIntent = PendingIntent.getActivity(context, UNIQUE_INT_PER_CALL, notificationIntent, 0);

intenções não são criadas se você enviar os mesmos parâmetros. Eles são reutilizados.

IncrediApp
fonte
1
então o UNIQUE_INT_PER_CALL é um número inteiro que preciso fornecer? ou é uma variável estática declarada em algum lugar?
BrianM
23
android pegadinha # 147 - para um Intentque tem diferentes extras (via putExtra) são considerados o mesmo e re-utilizado, porque eu não forneceu um ID único para alguns chamam intenção pendente - api terrível
wal
você sabe, eu era tão descuidada. Apenas pensando como ele poderia permanecer 0 em um bloco (no meu caso) :(
Exigente05
3
Isso foi incrivelmente útil para mim, apenas uma dica para outras pessoas, é provável que você esteja criando sua notificação no mesmo método e, portanto, você pode definir o ID da nova intenção pendente da mesma forma que você deseja. use para o ID exclusivo das notificações!
precisa saber é o seguinte
1
@IncrediApp, é o mesmo com PendingIntent.getBroadcast (); ?
Shruti
139

Como alternativa, você pode usar o seguinte código para gerar seu PendingIntent:

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT);

Do documento para PendingIntent.FLAG_UPDATE_CURRENT:

Se o PendingIntent descrito já existir, mantenha-o, mas substitua seus dados extras pelos que estão neste novo Intent. Isso pode ser usado se você estiver criando intenções nas quais apenas os extras mudam e não se importa que quaisquer entidades que receberam seu PendingIntent anterior possam iniciá-lo com seus novos extras, mesmo que não sejam explicitamente dados a ele.

ChristophK
fonte
Graças ... perfeitamente trabalhar para esta bandeira que é adicionar "PendingIntent.FLAG_UPDATE_CURRENT" :)
Najib Ahmed Puthawala
1
Trabalhou para mim, usando a intenção pendente de transferir o estado da configuração de um alarme para o receptor de transmissão.
William T. Mallard
(!) Eu só gostaria de saber sobre o que essas bandeiras realmente fez antes de eu enviou notificações para meus usuários Contente isso resolve os meus problemas ...
James Andrew
42

Você está passando o mesmo ID. Nesse tipo de situação, crie um ID exclusivo a partir de um momento como este:

int iUniqueId = (int) (System.currentTimeMillis() & 0xfffffff);

E coloque assim:

PendingIntent contentIntent = PendingIntent.getActivity(getApplicationContext(),iUniqueId, intentForNotification, 0);
hderanga
fonte
3
por que não usar new Random (). nextInt ()
exloong
@hderanga O que a adição de "& 0xfffffff" faz ao int acima?
AJW
3
@AJW System.currentTimeMillis()retorna um longo, enquanto o requestIdparâmetro de PendingIntent.getActivity()leva um int. 0xffffffffé uma máscara de bit. Embora exista um pouco mais, a explicação simples é que fazer 'long & 0xffffffff' fornece os 32 bits mais baixos dos longos e descarta os 32 bits mais altos, deixando-o essencialmente com um int de 32 bits. Isso é melhor do que simplesmente converter para um int, porque ele não estraga o bit de sinal (se você converter um comprimento maior que um int para um int, o bit de sinal transbordará e você poderá acabar com um valor negativo )
Jordan Bondo 12/12
8

Para quem procura a melhor abordagem após um longo período de tempo, é necessário passar o PendingIntent.FLAG_UPDATE_CURRENT como o último argumento, como mostrado abaixo

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT);

você nem precisa fornecer um novo ID exclusivo.

Você precisa fazer isso da próxima vez em diante, não pela primeira vez

□ Gentil
fonte
1
Isso não funciona, cheguei aqui porque era isso que eu estava fazendo.
Brill Pappin
Você precisa fazer isso nas próximas vezes, não pela primeira vez, ele funcionará.
Suave
0

Seu código de solicitação é 0 para toda a notificação. Altere a seguinte linha:

PendingIntent contentIntent = PendingIntent.getActivity(context, 0, notificationIntent, 0);

Com:

PendingIntent contentIntent = PendingIntent.getActivity(context, new Random().nextInt(), notificationIntent, 0);
Faisal Shaikh
fonte
1
Existe algum benefício em usar "new Random (). NextInt ()" em vez de "System.currentTimeMillis ()"?
AJW 23/05
O uso aleatório pode facilmente regenerar o mesmo valor inteiro novamente por acidente, causando um erro muito difícil de encontrar nas intenções antigas.
Sam
@AJW houve no meu caso. Criei duas notificações diferentes exatamente no mesmo milissegundo, então uma delas errou extras.
artman
0

Só queria adicionar outra opção

 PendingIntent.getBroadcast(context, 0, intent, PendingIntent.FLAG_ONE_SHOT | PendingIntent.FLAG_IMMUTABLE);
pelucida
fonte