]> git.sesse.net Git - vlc/blobdiff - src/interface/interaction.c
interface/interaction.c: Set the interaction object name.
[vlc] / src / interface / interaction.c
index 76472110020ca51bf307ecb835cae27bf975d518..bad5fa48edeca60beed6c02bac582352ac1ca389 100644 (file)
@@ -1,7 +1,7 @@
 /*****************************************************************************
  * interaction.c: User interaction functions
  *****************************************************************************
- * Copyright (C) 2005-2006 VideoLAN
+ * Copyright (C) 2005-2006 the VideoLAN team
  * $Id$
  *
  * Authors: ClĂ©ment Stenac <zorglub@videolan.org>
 /*****************************************************************************
  * Preamble
  *****************************************************************************/
-#include <stdlib.h>                                      /* free(), strtol() */
-#include <stdio.h>                                                   /* FILE */
-#include <string.h>                                            /* strerror() */
+
+#ifdef HAVE_CONFIG_H
+# include "config.h"
+#endif
 
 #include <vlc/vlc.h>
-#include <vlc/input.h>
 
-#include "vlc_interaction.h"
-#include "vlc_interface.h"
-#include "vlc_playlist.h"
+#include <stdlib.h>                                      /* free(), strtol() */
+#include <stdio.h>                                                   /* FILE */
+#include <string.h>
+
+#include <vlc_interface.h>
+#include <vlc_playlist.h>
+#include "interface.h"
 
 /*****************************************************************************
  * Local prototypes
  *****************************************************************************/
-static void                  intf_InteractionInit( playlist_t *p_playlist );
-static interaction_t *       intf_InteractionGet( vlc_object_t *p_this );
-static void                  intf_InteractionSearchInterface( interaction_t *
+static void                  InteractionInit( playlist_t *p_playlist );
+static interaction_t *       InteractionGet( vlc_object_t *p_this );
+static void                  InteractionSearchInterface( interaction_t *
                                                           p_interaction );
-static int                   intf_WaitAnswer( interaction_t *p_interact,
-                             interaction_dialog_t *p_dialog );
-static int                   intf_Send( interaction_t *p_interact,
-                             interaction_dialog_t *p_dialog );
-static interaction_dialog_t *intf_InteractionGetById( vlc_object_t* , int );
-static void                  intf_InteractionDialogDestroy(
-                                              interaction_dialog_t *p_dialog );
-
-/**
- * Send an interaction element to the user
- *
- * \param p_this the calling vlc_object_t
- * \param p_interact the interaction element
- * \return VLC_SUCCESS or an error code
- */
-int  __intf_Interact( vlc_object_t *p_this, interaction_dialog_t *
-                                    p_dialog )
-{
-    interaction_t *p_interaction = intf_InteractionGet( p_this );
-
-    /* Get an id, if we don't already have one */
-    if( p_dialog->i_id == 0 )
-    {
-        p_dialog->i_id = ++p_interaction->i_last_id;
-    }
-
-    if( p_this->i_flags & OBJECT_FLAGS_NOINTERACT )
-    {
-       return VLC_EGENERIC;
-    }
-
-    p_dialog->p_interaction = p_interaction;
-    p_dialog->p_parent = p_this;
-
-    if( p_dialog->i_type == INTERACT_DIALOG_TWOWAY )
-    {
-        return intf_WaitAnswer( p_interaction, p_dialog );
-    }
-    else
-    {
-        p_dialog->i_flags |=  DIALOG_GOT_ANSWER;
-        return intf_Send( p_interaction, p_dialog );
-    }
-}
+static interaction_dialog_t *DialogGetById( interaction_t* , int );
+static void                  DialogDestroy( interaction_dialog_t *p_dialog );
+static int DialogSend( vlc_object_t *p_this, interaction_dialog_t *p_dialog );
 
 /**
  * Destroy the interaction system
@@ -101,17 +64,14 @@ int  __intf_Interact( vlc_object_t *p_this, interaction_dialog_t *
 void intf_InteractionDestroy( interaction_t *p_interaction )
 {
     int i;
-
     // Remove all dialogs - Interfaces must be able to clean up their data
-
     for( i = p_interaction->i_dialogs -1 ; i >= 0; i-- )
     {
         interaction_dialog_t * p_dialog = p_interaction->pp_dialogs[i];
-        intf_InteractionDialogDestroy( p_dialog );
+        DialogDestroy( p_dialog );
         REMOVE_ELEM( p_interaction->pp_dialogs, p_interaction->i_dialogs, i );
     }
-
-    vlc_object_destroy( p_interaction );
+    vlc_object_release( p_interaction );
 }
 
 /**
@@ -125,44 +85,31 @@ void intf_InteractionManage( playlist_t *p_playlist )
 {
     vlc_value_t val;
     int i_index;
-    interaction_t *p_interaction;
-
-    p_interaction = p_playlist->p_interaction;
+    interaction_t *p_interaction = p_playlist->p_interaction;
 
     // Nothing to do
     if( p_interaction->i_dialogs == 0 ) return;
 
     vlc_mutex_lock( &p_interaction->object_lock );
 
-    intf_InteractionSearchInterface( p_interaction );
-
+    InteractionSearchInterface( p_interaction );
     if( !p_interaction->p_intf )
     {
         // We mark all dialogs as answered with their "default" answer
         for( i_index = 0 ; i_index < p_interaction->i_dialogs; i_index ++ )
         {
             interaction_dialog_t *p_dialog = p_interaction->pp_dialogs[i_index];
-
-            // Give default answer
-            p_dialog->i_return = DIALOG_DEFAULT;
-            if( p_dialog->i_flags & DIALOG_OK_CANCEL )
-                p_dialog->i_return = DIALOG_CANCELLED;
+            p_dialog->i_return = DIALOG_DEFAULT; // Give default answer
 
             // Pretend we have hidden and destroyed it
             if( p_dialog->i_status == HIDDEN_DIALOG )
-            {
                 p_dialog->i_status = DESTROYED_DIALOG;
-            }
             else
-            {
                 p_dialog->i_status = HIDING_DIALOG;
-            }
         }
     }
     else
-    {
         vlc_object_yield( p_interaction->p_intf );
-    }
 
     for( i_index = 0 ; i_index < p_interaction->i_dialogs; i_index ++ )
     {
@@ -186,20 +133,17 @@ void intf_InteractionManage( playlist_t *p_playlist )
             break;
         case HIDDEN_DIALOG:
             if( !(p_dialog->i_flags & DIALOG_GOT_ANSWER) ) break;
-            if( !(p_dialog->i_flags & DIALOG_REUSABLE) )
-            {
-                p_dialog->i_action = INTERACT_DESTROY;
-                val.p_address = p_dialog;
-                if( p_interaction->p_intf )
-                    var_Set( p_interaction->p_intf, "interaction", val );
-            }
+            p_dialog->i_action = INTERACT_DESTROY;
+            val.p_address = p_dialog;
+            if( p_interaction->p_intf )
+                var_Set( p_interaction->p_intf, "interaction", val );
             break;
         case DESTROYED_DIALOG:
             // Interface has now destroyed it, remove it
             REMOVE_ELEM( p_interaction->pp_dialogs, p_interaction->i_dialogs,
                          i_index);
             i_index--;
-            intf_InteractionDialogDestroy( p_dialog );
+            DialogDestroy( p_dialog );
             break;
         case NEW_DIALOG:
             // This is truly a new dialog, send it.
@@ -220,162 +164,141 @@ void intf_InteractionManage( playlist_t *p_playlist )
     vlc_mutex_unlock( &p_playlist->p_interaction->object_lock );
 }
 
-
-
-#define INTERACT_INIT( new )                                            \
-        new = (interaction_dialog_t*)malloc(                            \
-                        sizeof( interaction_dialog_t ) );               \
-        new->i_widgets = 0;                                             \
-        new->pp_widgets = NULL;                                         \
-        new->psz_title = NULL;                                          \
-        new->psz_description = NULL;                                    \
-        new->p_private = NULL;                                          \
-        new->i_id = 0;                                                  \
-        new->i_flags = 0;                                               \
-        new->i_status = NEW_DIALOG;
-
-#define INTERACT_FREE( new )                                            \
-        if( new->psz_title ) free( new->psz_title );                    \
-        if( new->psz_description ) free( new->psz_description );
-
-/** Helper function to send an error message
+#define DIALOG_INIT( type ) \
+        DECMALLOC_ERR( p_new, interaction_dialog_t );                     \
+        memset( p_new, 0, sizeof( interaction_dialog_t ) );               \
+        p_new->b_cancelled = VLC_FALSE;                                   \
+        p_new->i_status = NEW_DIALOG;                                     \
+        p_new->i_flags = 0; \
+        p_new->i_type = INTERACT_DIALOG_##type;                           \
+        p_new->psz_returned[0] = NULL;                                    \
+        p_new->psz_returned[1] = NULL;
+
+#define FORMAT_DESC \
+        va_start( args, psz_format ); \
+        vasprintf( &p_new->psz_description, psz_format, args ); \
+        va_end( args );
+
+/** Send an error message, both in a blocking and non-blocking way
  *  \param p_this     Parent vlc_object
- *  \param i_id       A predefined ID, 0 if not applicable
+ *  \param b_blocking Is this dialog blocking or not?
  *  \param psz_title  Title for the dialog
  *  \param psz_format The message to display
  *  */
-void __intf_UserFatal( vlc_object_t *p_this,
+int __intf_UserFatal( vlc_object_t *p_this, vlc_bool_t b_blocking,
                        const char *psz_title,
                        const char *psz_format, ... )
 {
     va_list args;
-    interaction_dialog_t *p_new = NULL;
-    user_widget_t *p_widget = NULL;
-    int i_id = DIALOG_ERRORS;
-
-    if( i_id > 0 )
-    {
-        p_new = intf_InteractionGetById( p_this, i_id );
-    }
-    if( !p_new )
-    {
-        INTERACT_INIT( p_new );
-        if( i_id > 0 ) p_new->i_id = i_id ;
-    }
-    else
-    {
-        p_new->i_status = UPDATED_DIALOG;
-    }
+    DIALOG_INIT( ONEWAY );
 
-    p_new->i_flags |= DIALOG_REUSABLE;
-
-    p_new->i_type = INTERACT_DIALOG_ONEWAY;
     p_new->psz_title = strdup( psz_title );
+    FORMAT_DESC;
 
-    p_widget = (user_widget_t* )malloc( sizeof( user_widget_t ) );
+    if( b_blocking )
+        p_new->i_flags = DIALOG_BLOCKING_ERROR;
+    else
+        p_new->i_flags = DIALOG_NONBLOCKING_ERROR;
 
-    p_widget->i_type = WIDGET_TEXT;
-    p_widget->val.psz_string = NULL;
+    return DialogSend( p_this, p_new );
+}
 
-    va_start( args, psz_format );
-    vasprintf( &p_widget->psz_text, psz_format, args );
-    va_end( args );
+/** Helper function to send an warning, which is always shown non-blocking
+ *  \param p_this     Parent vlc_object
+ *  \param psz_title  Title for the dialog
+ *  \param psz_format The message to display
+ *  */
+int __intf_UserWarn( vlc_object_t *p_this,
+                     const char *psz_title,
+                     const char *psz_format, ... )
+{
+    va_list args;
+    DIALOG_INIT( ONEWAY );
 
-    INSERT_ELEM ( p_new->pp_widgets,
-                  p_new->i_widgets,
-                  p_new->i_widgets,
-                  p_widget );
+    p_new->psz_title = strdup( psz_title );
+    FORMAT_DESC
 
-    p_new->i_flags |= DIALOG_CLEAR_NOSHOW;
+    p_new->i_flags = DIALOG_WARNING;
 
-    intf_Interact( p_this, p_new );
+    return DialogSend( p_this, p_new );
 }
 
-/** Helper function to ask a yes-no question
+/** Helper function to ask a yes-no-cancel question
  *  \param p_this           Parent vlc_object
  *  \param psz_title        Title for the dialog
  *  \param psz_description  A description
+ *  \param psz_default      caption for the default button
+ *  \param psz_alternate    caption for the alternate button
+ *  \param psz_other        caption for the optional 3rd button (== cancel)
  *  \return                 Clicked button code
  */
 int __intf_UserYesNo( vlc_object_t *p_this,
                       const char *psz_title,
-                      const char *psz_description )
+                      const char *psz_description,
+                      const char *psz_default,
+                      const char *psz_alternate,
+                      const char *psz_other )
 {
-    int i_ret;
-    interaction_dialog_t *p_new = NULL;
-    user_widget_t *p_widget = NULL;
-
-    INTERACT_INIT( p_new );
+    DIALOG_INIT( TWOWAY );
 
-    p_new->i_type = INTERACT_DIALOG_TWOWAY;
     p_new->psz_title = strdup( psz_title );
-
-    /* Text */
-    p_widget = (user_widget_t* )malloc( sizeof( user_widget_t ) );
-    p_widget->i_type = WIDGET_TEXT;
-    p_widget->psz_text = strdup( psz_description );
-    p_widget->val.psz_string = NULL;
-    INSERT_ELEM ( p_new->pp_widgets, p_new->i_widgets,
-                  p_new->i_widgets,  p_widget );
-
+    p_new->psz_description = strdup( psz_description );
     p_new->i_flags = DIALOG_YES_NO_CANCEL;
+    p_new->psz_default_button = strdup( psz_default );
+    p_new->psz_alternate_button = strdup( psz_alternate );
+    if( psz_other )
+        p_new->psz_other_button = strdup( psz_other );
 
-    i_ret = intf_Interact( p_this, p_new );
-
-    return i_ret;
+    return DialogSend( p_this, p_new );
 }
 
-/** Helper function to make a progressbar box
+/** Helper function to create a dialogue showing a progress-bar with some info
  *  \param p_this           Parent vlc_object
- *  \param psz_title        Title for the dialog
+ *  \param psz_title        Title for the dialog (NULL implies main intf )
  *  \param psz_status       Current status
  *  \param f_position       Current position (0.0->100.0)
+ *  \param i_timeToGo       Time (in sec) to go until process is finished
  *  \return                 Dialog id, to give to UserProgressUpdate
  */
-int __intf_UserProgress( vlc_object_t *p_this,
-                         const char *psz_title,
-                         const char *psz_status,
-                         float f_pos )
+int __intf_Progress( vlc_object_t *p_this, const char *psz_title,
+                     const char *psz_status, float f_pos, int i_time )
 {
-    int i_ret;
-    interaction_dialog_t *p_new = NULL;
-    user_widget_t *p_widget = NULL;
-
-    INTERACT_INIT( p_new );
-
-    p_new->i_type = INTERACT_DIALOG_ONEWAY;
-    p_new->psz_title = strdup( psz_title );
-
-    /* Progress bar */
-    p_widget = (user_widget_t* )malloc( sizeof( user_widget_t ) );
-    p_widget->i_type = WIDGET_PROGRESS;
-    p_widget->psz_text = strdup( psz_status );
-    p_widget->val.f_float = f_pos;
-    INSERT_ELEM ( p_new->pp_widgets, p_new->i_widgets,
-                  p_new->i_widgets,  p_widget );
+    DIALOG_INIT( ONEWAY );
+    p_new->psz_description = strdup( psz_status );
+    p_new->val.f_float = f_pos;
+    p_new->i_timeToGo = i_time;
+    p_new->psz_alternate_button = strdup( _( "Cancel" ) );
 
-    i_ret = intf_Interact( p_this, p_new );
+    if( psz_title )
+    {
+        p_new->psz_title = strdup( psz_title );
+        p_new->i_flags = DIALOG_USER_PROGRESS;
+    }
+    else
+        p_new->i_flags = DIALOG_INTF_PROGRESS;
 
+    DialogSend( p_this, p_new );
     return p_new->i_id;
 }
 
-/** Update a progress bar
+/** Update a progress bar in a dialogue
  *  \param p_this           Parent vlc_object
  *  \param i_id             Identifier of the dialog
  *  \param psz_status       New status
  *  \param f_position       New position (0.0->100.0)
+ *  \param i_timeToGo       Time (in sec) to go until process is finished
  *  \return                 nothing
  */
-void __intf_UserProgressUpdate( vlc_object_t *p_this, int i_id,
-                                const char *psz_status, float f_pos )
+void __intf_ProgressUpdate( vlc_object_t *p_this, int i_id,
+                            const char *psz_status, float f_pos, int i_time )
 {
-    interaction_t *p_interaction = intf_InteractionGet( p_this );
+    interaction_t *p_interaction = InteractionGet( p_this );
     interaction_dialog_t *p_dialog;
 
     if( !p_interaction ) return;
 
     vlc_mutex_lock( &p_interaction->object_lock );
-    p_dialog  =  intf_InteractionGetById( p_this, i_id );
+    p_dialog  =  DialogGetById( p_interaction, i_id );
 
     if( !p_dialog )
     {
@@ -383,17 +306,46 @@ void __intf_UserProgressUpdate( vlc_object_t *p_this, int i_id,
         return;
     }
 
-    if( p_dialog->pp_widgets[0]->psz_text )
-        free( p_dialog->pp_widgets[0]->psz_text );
-    p_dialog->pp_widgets[0]->psz_text = strdup( psz_status );
+    free( p_dialog->psz_description );
+    p_dialog->psz_description = strdup( psz_status );
 
-    p_dialog->pp_widgets[0]->val.f_float = f_pos;
+    p_dialog->val.f_float = f_pos;
+    p_dialog->i_timeToGo = i_time;
 
     p_dialog->i_status = UPDATED_DIALOG;
     vlc_mutex_unlock( &p_interaction->object_lock) ;
+
+    playlist_Signal( pl_Get( p_this ) );
 }
 
-/** Helper function to make a login/password box
+/** Helper function to communicate dialogue cancellations between the
+ *  interface module and the caller
+ *  \param p_this           Parent vlc_object
+ *  \param i_id             Identifier of the dialogue
+ *  \return                 Either true or false
+ */
+vlc_bool_t __intf_UserProgressIsCancelled( vlc_object_t *p_this, int i_id )
+{
+    interaction_t *p_interaction = InteractionGet( p_this );
+    interaction_dialog_t *p_dialog;
+    vlc_bool_t b_cancel;
+
+    if( !p_interaction ) return VLC_TRUE;
+
+    vlc_mutex_lock( &p_interaction->object_lock );
+    p_dialog  =  DialogGetById( p_interaction, i_id );
+    if( !p_dialog )
+    {
+        vlc_mutex_unlock( &p_interaction->object_lock ) ;
+        return VLC_TRUE;
+    }
+
+    b_cancel = p_dialog->b_cancelled;
+    vlc_mutex_unlock( &p_interaction->object_lock );
+    return b_cancel;
+}
+
+/** Helper function to make a login/password dialogue
  *  \param p_this           Parent vlc_object
  *  \param psz_title        Title for the dialog
  *  \param psz_description  A description
@@ -407,24 +359,54 @@ int __intf_UserLoginPassword( vlc_object_t *p_this,
                               char **ppsz_login,
                               char **ppsz_password )
 {
-
     int i_ret;
-    interaction_dialog_t *p_new = NULL;
+    DIALOG_INIT( TWOWAY );
+    p_new->i_type = INTERACT_DIALOG_TWOWAY;
+    p_new->psz_title = strdup( psz_title );
+    p_new->psz_description = strdup( psz_description );
+    p_new->psz_default_button = strdup( _("Ok" ) );
+    p_new->psz_alternate_button = strdup( _("Cancel" ) );
 
-    INTERACT_INIT( p_new );
+    p_new->i_flags = DIALOG_LOGIN_PW_OK_CANCEL;
+
+    i_ret = DialogSend( p_this, p_new );
+
+    if( i_ret != DIALOG_CANCELLED && i_ret != VLC_EGENERIC )
+    {
+        *ppsz_login = p_new->psz_returned[0]?
+                                strdup( p_new->psz_returned[0] ) : NULL;
+        *ppsz_password = p_new->psz_returned[1]?
+                                strdup( p_new->psz_returned[1] ) : NULL;
+    }
+    return i_ret;
+}
 
+/** Helper function to make a dialogue asking the user for !password string
+ *  \param p_this           Parent vlc_object
+ *  \param psz_title        Title for the dialog
+ *  \param psz_description  A description
+ *  \param ppsz_usersString Returned login
+ *  \return                 Clicked button code
+ */
+int __intf_UserStringInput( vlc_object_t *p_this,
+                              const char *psz_title,
+                              const char *psz_description,
+                              char **ppsz_usersString )
+{
+    int i_ret;
+    DIALOG_INIT( TWOWAY );
     p_new->i_type = INTERACT_DIALOG_TWOWAY;
     p_new->psz_title = strdup( psz_title );
     p_new->psz_description = strdup( psz_description );
 
-    p_new->i_flags = DIALOG_LOGIN_PW_OK_CANCEL;
+    p_new->i_flags = DIALOG_PSZ_INPUT_OK_CANCEL;
 
-    i_ret = intf_Interact( p_this, p_new );
+    i_ret = DialogSend( p_this, p_new );
 
     if( i_ret != DIALOG_CANCELLED )
     {
-        *ppsz_login = strdup( p_new->psz_returned[0] );
-        *ppsz_password = strdup( p_new->psz_returned[1] );
+        *ppsz_usersString = p_new->psz_returned[0]?
+                                    strdup( p_new->psz_returned[0] ) : NULL;
     }
     return i_ret;
 }
@@ -436,13 +418,13 @@ int __intf_UserLoginPassword( vlc_object_t *p_this,
  */
 void __intf_UserHide( vlc_object_t *p_this, int i_id )
 {
-    interaction_t *p_interaction = intf_InteractionGet( p_this );
+    interaction_t *p_interaction = InteractionGet( p_this );
     interaction_dialog_t *p_dialog;
 
     if( !p_interaction ) return;
 
     vlc_mutex_lock( &p_interaction->object_lock );
-    p_dialog  =  intf_InteractionGetById( p_this, i_id );
+    p_dialog = DialogGetById( p_interaction, i_id );
 
     if( !p_dialog )
     {
@@ -454,63 +436,50 @@ void __intf_UserHide( vlc_object_t *p_this, int i_id )
     vlc_mutex_unlock( &p_interaction->object_lock );
 }
 
-
-
 /**********************************************************************
  * The following functions are local
  **********************************************************************/
 
 /* Get the interaction object. Create it if needed */
-static interaction_t * intf_InteractionGet( vlc_object_t *p_this )
+static interaction_t * InteractionGet( vlc_object_t *p_this )
 {
-    playlist_t *p_playlist;
     interaction_t *p_interaction;
+    playlist_t *p_playlist = pl_Yield( p_this );
 
-    p_playlist = (playlist_t*) vlc_object_find( p_this, VLC_OBJECT_PLAYLIST,
-                                                FIND_ANYWHERE );
-
-    if( !p_playlist )
-    {
-        return NULL;
-    }
-
+    PL_LOCK;
     if( p_playlist->p_interaction == NULL )
-    {
-        intf_InteractionInit( p_playlist );
-    }
+       InteractionInit( p_playlist );
 
     p_interaction = p_playlist->p_interaction;
+    PL_UNLOCK;
 
-    vlc_object_release( p_playlist );
-
+    pl_Release( p_this );
     return p_interaction;
 }
 
 /* Create the interaction object in the given playlist object */
-static void intf_InteractionInit( playlist_t *p_playlist )
+static void InteractionInit( playlist_t *p_playlist )
 {
-    interaction_t *p_interaction;
-
-    p_interaction = vlc_object_create( VLC_OBJECT( p_playlist ),
-                                       sizeof( interaction_t ) );
+    interaction_t *p_interaction = vlc_object_create( VLC_OBJECT( p_playlist ),
+                                                      sizeof( interaction_t ) );
     if( !p_interaction )
     {
         msg_Err( p_playlist,"out of memory" );
         return;
     }
 
+    p_interaction->psz_object_name = "interaction";
     p_interaction->i_dialogs = 0;
     p_interaction->pp_dialogs = NULL;
     p_interaction->p_intf = NULL;
-    p_interaction->i_last_id = DIALOG_LAST_PREDEFINED + 1;
+    p_interaction->i_last_id = 0;
 
     vlc_mutex_init( p_interaction , &p_interaction->object_lock );
-
     p_playlist->p_interaction  = p_interaction;
 }
 
 /* Look for an interface suitable for interaction */
-static void intf_InteractionSearchInterface( interaction_t *p_interaction )
+static void InteractionSearchInterface( interaction_t *p_interaction )
 {
     vlc_list_t  *p_list;
     int          i_index;
@@ -537,121 +506,101 @@ static void intf_InteractionSearchInterface( interaction_t *p_interaction )
     vlc_list_release ( p_list );
 }
 
-/* Add a dialog to the queue and wait for answer */
-static int intf_WaitAnswer( interaction_t *p_interact,
-                            interaction_dialog_t *p_dialog )
+/* Find an interaction dialog by its id */
+static interaction_dialog_t *DialogGetById( interaction_t *p_interaction,
+                                            int i_id )
 {
     int i;
-    vlc_bool_t b_found = VLC_FALSE;
-    vlc_mutex_lock( &p_interact->object_lock );
-    for( i = 0 ; i< p_interact->i_dialogs; i++ )
-    {
-        if( p_interact->pp_dialogs[i]->i_id == p_dialog->i_id )
-        {
-            b_found = VLC_TRUE;
-        }
-    }
-    if( ! b_found )
-    {
-        INSERT_ELEM( p_interact->pp_dialogs,
-                     p_interact->i_dialogs,
-                     p_interact->i_dialogs,
-                     p_dialog );
-    }
-    else
-        p_dialog->i_status = UPDATED_DIALOG;
-    vlc_mutex_unlock( &p_interact->object_lock );
-
-    /// \todo Check that the initiating object is not dying
-    while( p_dialog->i_status != ANSWERED_DIALOG &&
-           p_dialog->i_status != HIDING_DIALOG &&
-           p_dialog->i_status != HIDDEN_DIALOG &&
-           !p_dialog->p_parent->b_die )
-    {
-        msleep( 100000 );
-    }
-    /// \todo locking
-    if( p_dialog->p_parent->b_die )
+    for( i = 0 ; i< p_interaction->i_dialogs; i++ )
     {
-        p_dialog->i_return = DIALOG_CANCELLED;
-        p_dialog->i_status = ANSWERED_DIALOG;
+        if( p_interaction->pp_dialogs[i]->i_id == i_id )
+            return p_interaction->pp_dialogs[i];
     }
-    p_dialog->i_flags |= DIALOG_GOT_ANSWER;
-    return p_dialog->i_return;
+    return NULL;
 }
 
-/* Add a dialog to the queue and return */
-static int intf_Send( interaction_t *p_interact,
-                      interaction_dialog_t *p_dialog )
+/* Destroy a dialog */
+static void DialogDestroy( interaction_dialog_t *p_dialog )
 {
-    int i;
-    vlc_bool_t b_found = VLC_FALSE;
-    if( p_interact == NULL ) return VLC_ENOOBJ;
-    vlc_mutex_lock( &p_interact->object_lock );
-
-    for( i = 0 ; i< p_interact->i_dialogs; i++ )
-    {
-        if( p_interact->pp_dialogs[i]->i_id == p_dialog->i_id )
-        {
-            b_found = VLC_TRUE;
-        }
-    }
-    if( !b_found )
-    {
-        INSERT_ELEM( p_interact->pp_dialogs,
-                     p_interact->i_dialogs,
-                     p_interact->i_dialogs,
-                     p_dialog );
-    }
-    else
-        p_dialog->i_status = UPDATED_DIALOG;
-    // Pretend we already retrieved the "answer"
-    p_dialog->i_flags |= DIALOG_GOT_ANSWER;
-    vlc_mutex_unlock( &p_interact->object_lock );
-    return VLC_SUCCESS;
+    free( p_dialog->psz_title );
+    free( p_dialog->psz_description );
+    free( p_dialog->psz_default_button );
+    free( p_dialog->psz_alternate_button );
+    free( p_dialog->psz_other_button );
+    free( p_dialog );
 }
 
-/* Find an interaction dialog by its id */
-static interaction_dialog_t *intf_InteractionGetById( vlc_object_t* p_this,
-                                                       int i_id )
+/* Ask for the dialog to be sent to the user. Wait for answer
+ * if required */
+static int DialogSend( vlc_object_t *p_this, interaction_dialog_t *p_dialog )
 {
-    interaction_t *p_interaction = intf_InteractionGet( p_this );
-    int i;
+    interaction_t *p_interaction = InteractionGet( p_this );
 
-    if( !p_interaction ) return NULL;
+    /* Get an id, if we don't already have one */
+    if( p_dialog->i_id == 0 )
+        p_dialog->i_id = ++p_interaction->i_last_id;
 
-    for( i = 0 ; i< p_interaction->i_dialogs; i++ )
+    if( p_this->i_flags & OBJECT_FLAGS_NOINTERACT ) return VLC_EGENERIC;
+
+    if( config_GetInt(p_this, "interact") ||
+        p_dialog->i_flags & DIALOG_BLOCKING_ERROR ||
+        p_dialog->i_flags & DIALOG_NONBLOCKING_ERROR )
     {
-        if( p_interaction->pp_dialogs[i]->i_id == i_id )
+        vlc_bool_t b_found = VLC_FALSE;
+        int i;
+        p_dialog->p_interaction = p_interaction;
+        p_dialog->p_parent = p_this;
+
+        /* Check if we have already added this dialog */
+        vlc_mutex_lock( &p_interaction->object_lock );
+        for( i = 0 ; i< p_interaction->i_dialogs; i++ )
         {
-            return p_interaction->pp_dialogs[i];
+            if( p_interaction->pp_dialogs[i]->i_id == p_dialog->i_id )
+                b_found = VLC_TRUE;
         }
-    }
-    return NULL;
-}
-
-#define FREE( i ) { if( i ) free( i ); i = NULL; }
-
-static void intf_InteractionDialogDestroy( interaction_dialog_t *p_dialog )
-{
-    int i;
-    for( i = p_dialog->i_widgets - 1 ; i >= 0 ; i-- )
-    {
-        user_widget_t *p_widget = p_dialog->pp_widgets[i];
-        FREE( p_widget->psz_text );
-        if( p_widget->i_type == WIDGET_INPUT_TEXT )
+        /* Add it to the queue, the main loop will send the orders to the
+         * interface */
+        if( ! b_found )
         {
-            FREE( p_widget->val.psz_string );
+            INSERT_ELEM( p_interaction->pp_dialogs,
+                         p_interaction->i_dialogs,
+                         p_interaction->i_dialogs,
+                         p_dialog );
         }
+        else
+            p_dialog->i_status = UPDATED_DIALOG;
 
-        REMOVE_ELEM( p_dialog->pp_widgets, p_dialog->i_widgets, i );
-        free( p_widget );
+        if( p_dialog->i_type == INTERACT_DIALOG_TWOWAY ) // Wait for answer
+        {
+            playlist_Signal( pl_Get( p_this ) );
+            while( p_dialog->i_status != ANSWERED_DIALOG &&
+                   p_dialog->i_status != HIDING_DIALOG &&
+                   p_dialog->i_status != HIDDEN_DIALOG &&
+                   !p_dialog->p_parent->b_die )
+            {
+                vlc_mutex_unlock( &p_interaction->object_lock );
+                msleep( 100000 );
+                vlc_mutex_lock( &p_interaction->object_lock );
+            }
+            if( p_dialog->p_parent->b_die )
+            {
+                p_dialog->i_return = DIALOG_CANCELLED;
+                p_dialog->i_status = ANSWERED_DIALOG;
+            }
+            p_dialog->i_flags |= DIALOG_GOT_ANSWER;
+            vlc_mutex_unlock( &p_interaction->object_lock );
+            playlist_Signal( pl_Get( p_this ) );
+            return p_dialog->i_return;
+        }
+        else
+        {
+            // Pretend we already retrieved the "answer"
+            p_dialog->i_flags |=  DIALOG_GOT_ANSWER;
+            vlc_mutex_unlock( &p_interaction->object_lock );
+            playlist_Signal( pl_Get( p_this ) );
+            return VLC_SUCCESS;
+        }
     }
-    FREE( p_dialog->psz_title );
-    FREE( p_dialog->psz_description );
-    
-    FREE( p_dialog->psz_returned[0] );
-    FREE( p_dialog->psz_returned[1] );
-
-    free( p_dialog );
+    else
+        return VLC_EGENERIC;
 }