Fixed a bug where skill templates would try to load the image for No Skill.

Fixed a bug where build template manager would delete saved template.

Daybreak/Controls/Templates/SkillTemplate.xaml.cs

@@ -48,9 +48,9 @@ namespace Daybreak.Controls
 
         private void SkillTemplate_DataContextChanged(object sender, DependencyPropertyChangedEventArgs e)
         {
-            if (e.NewValue is Skill)
+            if (e.NewValue is Skill skill && skill != Skill.NoSkill)
             {
-                Task.Run(() => GetImageStream(e.NewValue.As<Skill>())).ContinueWith((previousTask) =>
+                Task.Run(() => GetImageStream(skill)).ContinueWith((previousTask) =>
                 {
                     this.Dispatcher.Invoke(() =>
                     {

Daybreak/Daybreak.csproj

@@ -9,7 +9,7 @@
     <CopyLocalLockFileAssemblies>false</CopyLocalLockFileAssemblies>
     <LangVersion>preview</LangVersion>
     <ApplicationIcon>Daybreak.ico</ApplicationIcon>
-    <Version>0.7.5</Version>
+    <Version>0.7.6</Version>
     <ApplicationManifest>app.manifest</ApplicationManifest>
   </PropertyGroup>
 

Daybreak/Services/BuildTemplates/BuildTemplateManager.cs

@@ -68,13 +68,12 @@ namespace Daybreak.Services.BuildTemplates
         public void SaveBuild(BuildEntry buildEntry)
         {
             var encodedBuild = this.EncodeTemplate(buildEntry.Build);
-            File.WriteAllText($"{BuildsPath}\\{buildEntry.Name}.txt", encodedBuild);
             if (string.IsNullOrWhiteSpace(buildEntry.PreviousName))
             {
-                return;
+                File.Delete($"{BuildsPath}\\{buildEntry.PreviousName}.txt");
             }
 
-            File.Delete($"{BuildsPath}\\{buildEntry.PreviousName}.txt");
+            File.WriteAllText($"{BuildsPath}\\{buildEntry.Name}.txt", encodedBuild);
         }
 
         public void RemoveBuild(BuildEntry buildEntry)