From dacfc1f8eacb98933e96c5625c7d05ee9d115943 Mon Sep 17 00:00:00 2001
From: Kwoth <srki0509@gmail.com>
Date: Fri, 14 Oct 2016 23:18:07 +0200
Subject: [PATCH] Fixed crash

---
 .../Utility/Commands/UnitConversion.cs        | 55 ++++++++++---------
 1 file changed, 30 insertions(+), 25 deletions(-)

diff --git a/src/NadekoBot/Modules/Utility/Commands/UnitConversion.cs b/src/NadekoBot/Modules/Utility/Commands/UnitConversion.cs
index 0f86bcbc..d4f9df0c 100644
--- a/src/NadekoBot/Modules/Utility/Commands/UnitConversion.cs
+++ b/src/NadekoBot/Modules/Utility/Commands/UnitConversion.cs
@@ -66,35 +66,40 @@ namespace NadekoBot.Modules.Utility
             }
 
             public async Task UpdateCurrency()
-            {
-                var currencyRates = await UpdateCurrencyRates();
-                var unitTypeString = "currency";
-                var range = currencyRates.ConversionRates.Select(u => new ConvertUnit()
+            {try
                 {
-                    InternalTrigger = u.Key,
-                    Modifier = u.Value,
-                    UnitType = unitTypeString
-                }).ToArray();
-                var baseType = new ConvertUnit()
-                {
-                    Triggers = new[] { currencyRates.Base },
-                    Modifier = decimal.One,
-                    UnitType = unitTypeString
-                };
-                var toRemove = Units.Where(u => u.UnitType == unitTypeString);
+                    var currencyRates = await UpdateCurrencyRates();
+                    var unitTypeString = "currency";
+                    var range = currencyRates.ConversionRates.Select(u => new ConvertUnit()
+                    {
+                        InternalTrigger = u.Key,
+                        Modifier = u.Value,
+                        UnitType = unitTypeString
+                    }).ToArray();
+                    var baseType = new ConvertUnit()
+                    {
+                        Triggers = new[] { currencyRates.Base },
+                        Modifier = decimal.One,
+                        UnitType = unitTypeString
+                    };
+                    var toRemove = Units.Where(u => u.UnitType == unitTypeString);
 
-                using (var uow = DbHandler.UnitOfWork())
-                {
-                    uow.ConverterUnits.RemoveRange(toRemove.ToArray());
-                    uow.ConverterUnits.Add(baseType);
-                    uow.ConverterUnits.AddRange(range);
+                    using (var uow = DbHandler.UnitOfWork())
+                    {
+                        uow.ConverterUnits.RemoveRange(toRemove.ToArray());
+                        uow.ConverterUnits.Add(baseType);
+                        uow.ConverterUnits.AddRange(range);
 
-                    await uow.CompleteAsync().ConfigureAwait(false);
+                        await uow.CompleteAsync().ConfigureAwait(false);
+                    }
+                    Units.RemoveAll(u => u.UnitType == unitTypeString);
+                    Units.Add(baseType);
+                    Units.AddRange(range);
+                    _log.Info("Updated Currency");
+                }
+                catch {
+                    _log.Warn("Failed updating currency.");
                 }
-                Units.RemoveAll(u => u.UnitType == unitTypeString);
-                Units.Add(baseType);
-                Units.AddRange(range);
-                _log.Info("Updated Currency");
             }
             [NadekoCommand, Usage, Description, Aliases]
             [RequireContext(ContextType.Guild)]