Merge pull request #364 from petermcneil/channelmapper-sorting

Change channel mapper to sort list
jellyfin · Jun 26, 2019 · c9e70d9 · c9e70d9
2 parents 6a0672a + f7376f5
commit c9e70d9
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 10 deletions.
diff --git a/CONTRIBUTORS.md b/CONTRIBUTORS.md
@@ -22,6 +22,7 @@
  - [LogicalPhallacy](https://github.com/LogicalPhallacy)
  - [thornbill](https://github.com/thornbill)
  - [Oddstr13](https://github.com/oddstr13)
+ - [petermcneil](https://github.com/petermcneil)
 
 # Emby Contributors
 

diff --git a/src/components/channelmapper/channelmapper.js b/src/components/channelmapper/channelmapper.js
@@ -28,15 +28,18 @@ define(["dialogHelper", "loading", "connectionManager", "globalize", "actionshee
         function onChannelsElementClick(e) {
             var btnMap = parentWithClass(e.target, "btnMap");
             if (btnMap) {
-                var channelId = btnMap.getAttribute("data-id"),
-                    providerChannelId = btnMap.getAttribute("data-providerid"),
-                    menuItems = currentMappingOptions.ProviderChannels.map(function(m) {
-                        return {
-                            name: m.Name,
-                            id: m.Id,
-                            selected: m.Id.toLowerCase() === providerChannelId.toLowerCase()
-                        }
-                    });
+                var channelId = btnMap.getAttribute("data-id");
+                var providerChannelId = btnMap.getAttribute("data-providerid");
+                var menuItems = currentMappingOptions.ProviderChannels.map(function(m) {
+                    return {
+                        name: m.Name,
+                        id: m.Id,
+                        selected: m.Id.toLowerCase() === providerChannelId.toLowerCase()
+                    }
+                }).sort(function (a, b) {
+                    return a.name.localeCompare(b.name);
+                });
+
                 actionsheet.show({
                     positionTo: btnMap,
                     items: menuItems
@@ -93,4 +96,4 @@ define(["dialogHelper", "loading", "connectionManager", "globalize", "actionshee
             })
         }
     }
-});
+});