From 8b8f795f8a420c3cb7785996fb5cacb9ea5df64d Mon Sep 17 00:00:00 2001
From: Felipe Martinez <pipernene@gmail.com>
Date: Mon, 3 Jun 2019 13:33:43 +0200
Subject: [PATCH] Fix U.GG provider and a crash

---
 .../Data/Providers/UGGProvider.cs             | 29 +++++++------------
 Legendary Rune Maker/Data/Rune.cs             |  2 +-
 2 files changed, 11 insertions(+), 20 deletions(-)

diff --git a/Legendary Rune Maker/Data/Providers/UGGProvider.cs b/Legendary Rune Maker/Data/Providers/UGGProvider.cs
index aed0db9..30c2575 100644
--- a/Legendary Rune Maker/Data/Providers/UGGProvider.cs	
+++ b/Legendary Rune Maker/Data/Providers/UGGProvider.cs	
@@ -7,6 +7,7 @@
 using System.Linq;
 using System.Net;
 using System.Text;
+using System.Text.RegularExpressions;
 using System.Threading.Tasks;
 
 namespace Legendary_Rune_Maker.Data.Providers
@@ -21,6 +22,7 @@ internal class UGGProvider : Provider
 
         private const string UGGApiVersion = "1.1";
         private const string UGGDataVersion = "1.2";
+        private const string UGGOverviewVersion = "1.2.5";
 
         private readonly static IDictionary<int, Position> IdToPosition = new Dictionary<int, Position>
         {
@@ -36,28 +38,17 @@ private async Task<string> GetLolUGGVersion()
         {
             if (_LolUGGVersion == null)
             {
-                var url = $"https://u.gg/json/new_ugg_versions/{UGGDataVersion}.json";
-                var json = JObject.Parse(await WebCache.String(url));
-                _LolUGGVersion = (json.Children().First().Next as JProperty).Name;
-            }
-
-            return _LolUGGVersion;
-        }
+                var page = await WebCache.String("https://u.gg");
+                var scriptUrl = Regex.Match(page, @"(?<=<script src="").*main.*\.js").Value;
 
-        private string _UGGOverviewVersion;
-        private async Task<string> GetUGGOverviewVersion()
-        {
-            if (_UGGOverviewVersion == null)
-            {
-                var url = $"https://u.gg/json/new_ugg_versions/{UGGDataVersion}.json";
-
-                var json = JObject.Parse(await Client.DownloadStringTaskAsync(url));
+                var scriptText = await WebCache.String(scriptUrl);
+                var versionsJson = Regex.Match(scriptText, @"(?<=Ln=)\[.*?\]").Value;
+                var versions = JArray.Parse(versionsJson);
 
-                _UGGOverviewVersion = json["latest"]["overview"].ToObject<string>();
+                _LolUGGVersion = versions[0]["value"].ToObject<string>();
             }
 
-            return _UGGOverviewVersion;
-
+            return _LolUGGVersion;
         }
 
         private IDictionary<int, JObject> ChampionData = new Dictionary<int, JObject>();
@@ -65,7 +56,7 @@ protected async Task<JObject> GetChampionData(int championId)
         {
             if (!ChampionData.TryGetValue(championId, out var data))
             {
-                string url = $"https://stats2.u.gg/lol/{UGGApiVersion}/overview/{await GetLolUGGVersion()}/ranked_solo_5x5/{championId}/{await GetUGGOverviewVersion()}.json";
+                string url = $"https://stats2.u.gg/lol/{UGGApiVersion}/overview/{await GetLolUGGVersion()}/ranked_solo_5x5/{championId}/{UGGOverviewVersion}.json";
 
                 var json = JObject.Parse(await WebCache.String(url));
                 ChampionData[championId] = data = (JObject)json[OverviewWorld.ToString()][OverviewPlatPlus.ToString()];
diff --git a/Legendary Rune Maker/Data/Rune.cs b/Legendary Rune Maker/Data/Rune.cs
index a57e8b3..db4a10d 100644
--- a/Legendary Rune Maker/Data/Rune.cs	
+++ b/Legendary Rune Maker/Data/Rune.cs	
@@ -43,7 +43,7 @@ private static string Richify(string desc)
                 ("<br>", "<LineBreak/>"),
                 (@"<b>(.*?)<\/b>", "<Bold>$1</Bold>"),
                 (@"<i>(.*?)<\/i>", "<Italic>$1</Italic>"),
-                (@"<hr><\/hr>", "<Line/>"),
+                (@"<hr\/?>(<\/hr>)?", "<Line/>"),
                 (@"<\/li>", "</li> "),
                 (@"<font color='(.*?)'>(.*?)<\/font>", "<Run Foreground=\"$1\">$2</Run>"),
                 (@"<(.*?)( .*?)?>(?<content>.*?)<\/\1>", "${content}"), //Replace all remaining XML tags with just their content