Android getIntent() devuelve el primer intento

Question

He desarrollado una aplicación para descargar un archivo de video y almacenarlo en la tarjeta SD. En el proceso, también actualizo el progreso y el estado de la descarga como una notificación de la barra de estado usando el NotificationManager.

Mi clase llamada DownloadTask.java extiende el AsyncTask. Así que aquí actualizo el progreso usando el método onProgressUpdate(), donde utilizo el NotificationManager para este propósito. Todo funciona a las mil maravillas excepto que, al completar la descarga, deseo hacer clic en la notificación para abrir el archivo de video específico. Así que esto es lo que he hecho:

mNotificationManager = (NotificationManager) mContext.getSystemService(Context.NOTIFICATION_SERVICE); 

int icon = android.R.drawable.stat_sys_download_done; 
long when = System.currentTimeMillis(); 

mNotification = new Notification(icon, "", when); 
mContentTitle_complete = mContext.getString(R.string.download_complete); 
notificationIntent = new Intent(mContext,OpenDownloadedVideo.class); 
notificationIntent.putExtra("fileName", file); 
mContentIntent = PendingIntent.getActivity(mContext, 0, notificationIntent, 0); 
mNotification.setLatestEventInfo(mContext, file, mContentTitle_complete, mContentIntent); 
mNotification.flags = Notification.FLAG_AUTO_CANCEL; 
mNotificationManager.notify(NOTIFICATION_ID, mNotification); 

Tenga en cuenta que la fileName y NOTIFICATION_ID son únicos en mi caso.

El ActivityOpenDownloadedVideo.java abre el archivo por:

public void onCreate(Bundle savedInstanceState) { 
    super.onCreate(savedInstanceState); 
    try { 
     fileName = getIntent().getExtras().getString("fileName"); 
     Intent i = new Intent(Intent.ACTION_VIEW); 
     File videoFileToPlay = new File(Environment.getExternalStorageDirectory()+"/MyFolder"+"/"+fileName); 
     i.setDataAndType(Uri.fromFile(videoFileToPlay), "video/*"); 
     startActivity(i); 
     finish(); 
    } catch(Exception e) { 
     // 
    } 
} 

Así que cuando me descargo un vídeo por primera vez y hacer clic en la notificación se abrirá el archivo de vídeo adecuado. Sin embargo, la próxima vez que descargue otro video y haga clic en la notificación, se abrirá nuevamente el primer archivo que se descargó.

Esto es porque getIntent dentro de OpenDownloadedVideo devuelve el primer Intent creado y no el más reciente. ¿Cómo puedo corregir esto?

Además, tenga en cuenta que existe el problema cuando descargo más de un video, p. Ej. si descargo cinco archivos de video diferentes y hay cinco notificaciones en la barra de estado. El mismo archivo se abrirá cada vez que se haga clic en una notificación.

Answer 1

@Alex y @Codinguser, gracias por sus respuestas. Muy apreciado. Sin embargo, encontré una respuesta diferente que funcionó para mí. Al crear un PendingIntent para el Intent, pasele un valor único. En mi caso yo estaba haciendo esto:

mContentIntent = PendingIntent.getActivity(mContext, 0, notificationIntent, 0); 

pero ahora estoy usando NOTIFICATION_ID ya que era único. Cambié la llamada anterior a:

mContentIntent = PendingIntent.getActivity(mContext, NOTIFICATION_ID, 
              notificationIntent, 0); 

Eso es todo y funciona.

me encontré con algo de información sobre el mismo en la pregunta Mulitple Instances of Pending Intent

Answer 2

¿Qué sucede si realiza su actividad OpenDownloadedVideo SINGLE_TOP y anula onNewIntent?

Answer 3

Desde el Android Activity.onNewIntent() documentation

Tenga en cuenta que getIntent() sigue devolviendo la intención original. Puede usar setIntent (Intención) para actualizarlo a este nuevo Intento.

Así que cuando obtenga el nuevo Intento, debe establecerlo explícitamente como el objetivo de la actividad.

Answer 4

realidad sólo necesitas crear PendingIntent con PendingIntent.FLAG_UPDATE_CURRENT, así:

mContentIntent = PendingIntent.getActivity(mContext, 0, notificationIntent, PendingIntent.FLAG_UPDATE_CURRENT); 

Answer 5

/** 
* Override super.onNewIntent() so that calls to getIntent() will return the 
* latest intent that was used to start this Activity rather than the first 
* intent. 
*/ 
@Override 
public void onNewIntent(Intent intent){ 
    super.onNewIntent(intent); // Propagate. 
    setIntent(intent); // Passing the new intent to setIntent() means this new intent will be the one returned whenever getIntent() is called. 
}