macosx: add new pl items to the end if they are dropped to a node

diff --git a/modules/gui/macosx/PLModel.m b/modules/gui/macosx/PLModel.m
index 6fae2ccbe81055b52fb94e59d5b427d7ea11b822..41055eebba48be0bd1bf76fbc7f82d7d82a3934f 100644 (file)
--- a/modules/gui/macosx/PLModel.m
+++ b/modules/gui/macosx/PLModel.m
@@ -26,9 +26,14 @@
 #import "playlist.h"
 #import "StringUtility.h"
 
+#ifdef HAVE_CONFIG_H
+# import "config.h"
+#endif
+#include <assert.h>
+
 #include <vlc_playlist.h>
 #include <vlc_input_item.h>
-#import <vlc_input.h>
+#include <vlc_input.h>
 #include <vlc_url.h>
 
 #define TRACKNUM_COLUMN @"tracknumber"
@@ -448,10 +453,6 @@
 {
     NSPasteboard *o_pasteboard = [info draggingPasteboard];
 
-    // this is no valid target, sanitize to top of table
-    if (index == NSOutlineViewDropOnItemIndex)
-        index = 0;
-
     if (targetItem == nil) {
         targetItem = _rootItem;
     }
@@ -498,6 +499,10 @@
                 pp_items[j++] = p_item;
         }
 
+        // drop on a node itself will append entries at the end
+        if (index == NSOutlineViewDropOnItemIndex)
+            index = p_new_parent->i_children;
+
         if (playlist_TreeMoveMany(p_playlist, j, pp_items, p_new_parent, index) != VLC_SUCCESS) {
             PL_UNLOCK;
             free(pp_items);
@@ -565,6 +570,9 @@
             [o_array addObject: o_dic];
         }
 
+        // drop on a node itself will append entries at the end
+        static_assert(NSOutlineViewDropOnItemIndex == -1, "Expect NSOutlineViewDropOnItemIndex to be -1");
+
         [_playlist addPlaylistItems:o_array withParentItemId:[targetItem plItemId] atPos:index startPlayback:NO];
         return YES;
     }