Merge pull request #4601 from jmarshallnz/play_media_sort_order

[builtins] PlayMedia() should use a sort order appropriate for the items
xbmc · Apr 28, 2014 · bed92f6 · bed92f6
2 parents fcb9a0e + b434835
commit bed92f6
Showing 1 changed file with 7 additions and 2 deletions.
diff --git a/xbmc/interfaces/Builtins.cpp b/xbmc/interfaces/Builtins.cpp
@@ -675,7 +675,6 @@ int CBuiltins::Execute(const CStdString& execString)
       CFileItemList items;
       CStdString extensions = g_advancedSettings.m_videoExtensions + "|" + g_advancedSettings.m_musicExtensions;
       CDirectory::GetDirectory(item.GetPath(),items,extensions);
-      items.Sort(SortByLabel, SortOrderAscending);
 
       bool containsMusic = false, containsVideo = false;
       for (int i = 0; i < items.Size(); i++)
@@ -687,7 +686,13 @@ int CBuiltins::Execute(const CStdString& execString)
         if (containsMusic && containsVideo)
           break;
       }
-
+
+      CGUIViewState *state = CGUIViewState::GetViewState(containsVideo ? WINDOW_VIDEO_NAV : WINDOW_MUSIC, items);
+      if (state)
+        items.Sort(state->GetSortMethod());
+      else
+        items.Sort(SortByLabel, SortOrderAscending);
+
       int playlist = containsVideo? PLAYLIST_VIDEO : PLAYLIST_MUSIC;;
       if (containsMusic && containsVideo) //mixed content found in the folder
       {