static void input_item_subitem_added( const vlc_event_t * p_event,
void * user_data )
{
- playlist_t * p_playlist = user_data;
+ playlist_item_t *p_parent_playlist_item = user_data;
+ playlist_t * p_playlist = p_parent_playlist_item->p_playlist;
input_item_t * p_parent, * p_child;
- vlc_bool_t b_play = var_CreateGetBool( p_playlist, "playlist-autostart" );
- playlist_item_t *p_item_in_category;
- playlist_item_t *p_current;
+ playlist_item_t * p_child_in_category;
+ playlist_item_t * p_item_in_category;
+ vlc_bool_t b_play;
p_parent = p_event->p_obj;
p_child = p_event->u.input_item_subitem_added.p_new_child;
- p_current = playlist_ItemGetByInput( p_playlist, p_parent, VLC_FALSE );
- if( p_current->i_children == -1 )
- p_item_in_category = playlist_ItemToNode( p_playlist, p_current,
- VLC_FALSE );
- else
- p_item_in_category = p_current;
- p_item_in_category = p_current;
- b_play = b_play && p_current == p_playlist->status.p_item;
+ PL_LOCK;
+ b_play = var_CreateGetBool( p_playlist, "playlist-autostart" );
- playlist_NodeAddInput( p_playlist, p_child, p_item_in_category,
- PLAYLIST_APPEND | PLAYLIST_SPREPARSE , PLAYLIST_END,
- VLC_FALSE );
+ /* This part is really hakish, but this playlist system isn't simple */
+ /* First check if we haven't already added the item as we are
+ * listening using the onelevel and the category representent
+ * (Because of the playlist design) */
+ p_child_in_category = playlist_ItemFindFromInputAndRoot(
+ p_playlist, p_child->i_id,
+ p_playlist->p_root_category,
+ VLC_FALSE /* Only non-node */ );
- if( b_play )
+ if( !p_child_in_category )
{
- playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
+ /* Then, transform to a node if needed */
+ p_item_in_category = playlist_ItemFindFromInputAndRoot(
+ p_playlist, p_parent->i_id,
+ p_playlist->p_root_category,
+ VLC_FALSE /* Only non-node */ );
+ if( !p_item_in_category )
+ {
+ /* Item may have been removed */
+ PL_UNLOCK;
+ return;
+ }
+
+ b_play = b_play && p_item_in_category == p_playlist->status.p_item;
+
+ /* If this item is already a node don't transform it */
+ if( p_item_in_category->i_children == -1 )
+ {
+ p_item_in_category = playlist_ItemToNode( p_playlist,
+ p_item_in_category, VLC_TRUE );
+ p_item_in_category->p_input->i_type = ITEM_TYPE_PLAYLIST;
+ }
+
+ playlist_BothAddInput( p_playlist, p_child, p_item_in_category,
+ PLAYLIST_APPEND | PLAYLIST_SPREPARSE , PLAYLIST_END,
+ NULL, NULL, VLC_TRUE );
+
+ if( b_play )
+ {
+ playlist_Control( p_playlist, PLAYLIST_VIEWPLAY,
VLC_TRUE, p_item_in_category, NULL );
- vlc_object_release( p_playlist );
+ }
}
+
+ PL_UNLOCK;
+
}
/*****************************************************************************
* Listen to vlc_InputItemAddSubItem event
*****************************************************************************/
-static void install_input_item_observer( playlist_t * p_playlist,
+static void install_input_item_observer( playlist_item_t * p_item,
input_item_t * p_input )
{
- msg_Dbg( p_playlist, "Listening to %s with %p", p_input->psz_name, p_playlist);
-
vlc_event_attach( &p_input->event_manager, vlc_InputItemSubItemAdded,
input_item_subitem_added,
- p_playlist );
+ p_item );
}
-static void uninstall_input_item_observer( playlist_t * p_playlist,
+static void uninstall_input_item_observer( playlist_item_t * p_item,
input_item_t * p_input )
{
- msg_Dbg( p_playlist, "Not Listening to %s with %p", p_input->psz_name, p_playlist);
-
vlc_event_detach( &p_input->event_manager, vlc_InputItemSubItemAdded,
input_item_subitem_added,
- p_playlist );
-
+ p_item );
+
}
/*****************************************************************************
input_item_t *p_input )
{
DECMALLOC_NULL( p_item, playlist_item_t );
- playlist_t *p_playlist = p_obj->p_libvlc->p_playlist;
+ playlist_t *p_playlist = pl_Yield( p_obj );
p_item->p_input = p_input;
vlc_gc_incref( p_item->p_input );
p_item->i_flags = 0;
p_item->p_playlist = p_playlist;
- install_input_item_observer( p_playlist, p_input );
+ install_input_item_observer( p_item, p_input );
+
+ pl_Release( p_item->p_playlist );
return p_item;
}
/** Delete a playlist item and detach its input item */
int playlist_ItemDelete( playlist_item_t *p_item )
{
- uninstall_input_item_observer( p_item->p_playlist, p_item->p_input );
+ uninstall_input_item_observer( p_item, p_item->p_input );
vlc_gc_decref( p_item->p_input );
free( p_item );
return VLC_EGENERIC;
}
+/** Remove an input item when it appears from a root playlist item */
+int playlist_DeleteInputInParent( playlist_t *p_playlist, int i_input_id,
+ playlist_item_t *p_root, vlc_bool_t b_locked )
+{
+ int i_ret;
+ if( !b_locked ) PL_LOCK;
+ i_ret = DeleteFromInput( p_playlist, i_input_id,
+ p_root, VLC_TRUE );
+ if( !b_locked ) PL_UNLOCK;
+ return i_ret;
+}
+
/** Remove an input item from ONELEVEL and CATEGORY */
int playlist_DeleteFromInput( playlist_t *p_playlist, int i_input_id,
vlc_bool_t b_locked )
p_playlist, p_item->p_input->i_id,
p_playlist->p_root_onelevel,
VLC_TRUE );
+ assert( p_item_in_one );
+
/* We already have it, and there is nothing more to do */
ChangeToNode( p_playlist, p_item_in_category );
- if( !p_item_in_one )
- return p_item_in_category;
-
/* Item in one is a root, change it to node */
if( p_item_in_one->p_parent == p_playlist->p_root_onelevel )
ChangeToNode( p_playlist, p_item_in_one );
{
if( psz_name && p_item )
{
- input_ItemSetName( p_item->p_input, psz_name );
+ input_item_SetName( p_item->p_input, psz_name );
return VLC_SUCCESS;
}
return VLC_EGENERIC;
vlc_cond_signal( &p_playlist->object_wait );
}
/* Preparse if PREPARSE or SPREPARSE & not enough meta */
+ char *psz_artist = input_item_GetArtist( p_item_cat->p_input );
+ char *psz_album = input_item_GetAlbum( p_item_cat->p_input );
if( p_playlist->b_auto_preparse &&
(i_mode & PLAYLIST_PREPARSE ||
( i_mode & PLAYLIST_SPREPARSE &&
- ( EMPTY_STR( input_item_GetArtist( p_item_cat->p_input ) ) ||
- ( EMPTY_STR( input_item_GetAlbum( p_item_cat->p_input ) ) ) )
+ ( EMPTY_STR( psz_artist ) || ( EMPTY_STR( psz_album ) ) )
) ) )
playlist_PreparseEnqueue( p_playlist, p_item_cat->p_input );
/* If we already have it, signal it */
- else if( !EMPTY_STR( input_item_GetArtist( p_item_cat->p_input ) ) &&
- !EMPTY_STR( input_item_GetAlbum( p_item_cat->p_input ) ) )
+ else if( !EMPTY_STR( psz_artist ) && !EMPTY_STR( psz_album ) )
input_item_SetPreparsed( p_item_cat->p_input, VLC_TRUE );
+ free( psz_artist );
+ free( psz_album );
}
/* Add the playlist item to the requested node and fire a notification */