From 9a1a13ba81f601919f87e4a24dc63d6183ee77c8 Mon Sep 17 00:00:00 2001
From: Warren Buckley <warren@umbraco.com>
Date: Wed, 24 Oct 2018 14:55:01 +0100
Subject: [PATCH] Fix the ordering logic - it helps if you actually assign the
 ordered items back to the list *facepalm*

---
 .../Models/Mapping/ContentItemDisplayVariationResolver.cs       | 2 +-
 src/Umbraco.Web/Models/Mapping/LanguageMapperProfile.cs         | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/src/Umbraco.Web/Models/Mapping/ContentItemDisplayVariationResolver.cs b/src/Umbraco.Web/Models/Mapping/ContentItemDisplayVariationResolver.cs
index 562797e3e3..7db491ad2e 100644
--- a/src/Umbraco.Web/Models/Mapping/ContentItemDisplayVariationResolver.cs
+++ b/src/Umbraco.Web/Models/Mapping/ContentItemDisplayVariationResolver.cs
@@ -59,7 +59,7 @@ namespace Umbraco.Web.Models.Mapping
                 variants.Remove(defaultLang);
 
                 //Sort the remaining languages a-z
-                variants.OrderBy(x => x.Language.Name);
+                variants = variants.OrderBy(x => x.Name).ToList();
 
                 //Insert the default lang as the first item
                 variants.Insert(0, defaultLang);
diff --git a/src/Umbraco.Web/Models/Mapping/LanguageMapperProfile.cs b/src/Umbraco.Web/Models/Mapping/LanguageMapperProfile.cs
index 6dd6135329..f820d5ae54 100644
--- a/src/Umbraco.Web/Models/Mapping/LanguageMapperProfile.cs
+++ b/src/Umbraco.Web/Models/Mapping/LanguageMapperProfile.cs
@@ -37,7 +37,7 @@ namespace Umbraco.Web.Models.Mapping
                 langs.Remove(defaultLang);
 
                 //Sort the remaining languages a-z
-                langs.OrderBy(x => x.Name);
+                langs = langs.OrderBy(x => x.Name).ToList();
 
                 //Insert the default lang as the first item
                 langs.Insert(0, defaultLang);