Merge pull request 'release/0.4.1.0217' (#4) from release/0.4.1.0217 into master

Reviewed-on: #4

DTOs/SaveFile/App3Options.cs

@@ -3,7 +3,7 @@ using System.Diagnostics;
 using System.Drawing;
 using System.Text.Json.Serialization;
 
-namespace RokettoLaunch.Models.SaveFile
+namespace RokettoLaunch.DTOs.SaveFile
 {
     public class App3Options : AppOptionsBase
     {

DTOs/SaveFile/App4Options.cs

@@ -6,7 +6,7 @@ using System.Linq;
 using System.Text.Json;
 using System.Text.Json.Serialization;
 
-namespace RokettoLaunch.Models.SaveFile
+namespace RokettoLaunch.DTOs.SaveFile
 {
     public class App4Options : AppOptionsBase
     {

DTOs/SaveFile/AppOptionsBase.cs

@@ -1,6 +1,6 @@
 using System.Drawing;
 
-namespace RokettoLaunch.Models.SaveFile
+namespace RokettoLaunch.DTOs.SaveFile
 {
     public class AppOptionsBase
     {

Forms/CopyToTileForm.cs

@@ -5,7 +5,7 @@ using System.Windows.Forms;
 using RyzStudio.Windows.Forms;
 using RyzStudio.Windows.ThemedForms;
 
-namespace RokettoLaunch
+namespace RokettoLaunch.Forms
 {
     public class CopyToTileForm : Form
     {

Forms/EditFolderForm.cs

@@ -5,12 +5,12 @@ using System.Drawing;
 using System.IO;
 using System.Linq;
 using System.Windows.Forms;
-using RokettoLaunch.Models.SaveFile;
+using RokettoLaunch.DTOs.SaveFile;
 using RyzStudio.Windows.Forms;
 using RyzStudio.Windows.ThemedForms;
 using RyzStudio.Windows.ThemedForms.ButtonTextBox;
 
-namespace RokettoLaunch
+namespace RokettoLaunch.Forms
 {
     public class EditFolderForm : Form
     {

Forms/EditGroupForm.cs

@@ -1,12 +1,12 @@
 using System;
 using System.Windows.Forms;
-using RokettoLaunch.Models.SaveFile;
+using RokettoLaunch.DTOs.SaveFile;
 using RokettoLaunch.Windows.Forms;
 using RyzStudio.Windows.Forms;
 using RyzStudio.Windows.ThemedForms.ButtonTextBox;
 using RyzStudio.Windows.ThemedForms.PickerBox;
 
-namespace RokettoLaunch
+namespace RokettoLaunch.Forms
 {
     public class EditGroupForm : Form
     {

Forms/EditTileForm.cs

@@ -2,13 +2,13 @@
 using System.Diagnostics;
 using System.Drawing;
 using System.Windows.Forms;
-using RokettoLaunch.Models.SaveFile;
+using RokettoLaunch.DTOs.SaveFile;
 using RyzStudio.Windows.Forms;
 using RyzStudio.Windows.ThemedForms;
 using RyzStudio.Windows.ThemedForms.ButtonTextBox;
 using RyzStudio.Windows.ThemedForms.PickerBox;
 
-namespace RokettoLaunch
+namespace RokettoLaunch.Forms
 {
     public class EditTileForm : Form
     {

Forms/LoadingForm.cs

@@ -3,10 +3,10 @@ using System.Collections.Generic;
 using System.IO;
 using System.Threading.Tasks;
 using System.Windows.Forms;
-using RokettoLaunch.Models.SaveFile;
+using RokettoLaunch.DTOs.SaveFile;
 using RyzStudio.Windows.Forms;
 
-namespace RokettoLaunch
+namespace RokettoLaunch.Forms
 {
     public class LoadingForm : Form
     {

Forms/OptionsForm.cs

@@ -1,12 +1,12 @@
 using System;
 using System.Windows.Forms;
-using RokettoLaunch.Models.SaveFile;
+using RokettoLaunch.DTOs.SaveFile;
 using RyzStudio.Windows.Forms;
 using RyzStudio.Windows.ThemedForms;
 using RyzStudio.Windows.ThemedForms.ButtonTextBox;
 using RyzStudio.Windows.ThemedForms.PickerBox;
 
-namespace RokettoLaunch
+namespace RokettoLaunch.Forms
 {
     public class OptionsForm : Form
     {

MainForm.cs

@@ -6,8 +6,8 @@ using System.IO;
 using System.Linq;
 using System.Threading.Tasks;
 using System.Windows.Forms;
-using Microsoft.Extensions.DependencyInjection;
+using RokettoLaunch.DTOs.SaveFile;
-using RokettoLaunch.Models.SaveFile;
+using RokettoLaunch.Forms;
 using RokettoLaunch.Windows.Forms;
 using RyzStudio;
 using RyzStudio.Windows.Forms;
@@ -18,11 +18,11 @@ namespace RokettoLaunch
 {
     public partial class MainForm : Form
     {
-        private readonly IServiceProvider _provider;
         private readonly IFileSessionManager _fileSessionManager;
 
-        private bool _isBusy = false;
+        private App4Options currentSession = null;
-        private bool _requestExit = false;
+        private bool isBusy = false;
+        private bool requestExit = false;
 
 
         public MainForm()
@@ -30,22 +30,10 @@ namespace RokettoLaunch
             InitializeComponent();
 
             this.AutoScaleMode = AutoScaleMode.None;
+            this.StartPosition = FormStartPosition.WindowsDefaultLocation;
             this.Text = Application.ProductName;
 
-            this.CurrentSession = new App4Options();
+            _fileSessionManager = new FileSessionManager();
-
-//#if DEBUG
-            //flowLayoutPanel1.BackColor = Color.LightGreen;
-//#endif
-
-            notifyIcon1.Text = Application.ProductName;
-        }
-
-        public MainForm(IServiceProvider provider, IFileSessionManager fileSessionManager) : this()
-        {
-            _provider = provider;
-            _fileSessionManager = fileSessionManager;
-
             _fileSessionManager.OpenFileDialog = openFileDialog1;
             _fileSessionManager.SaveFileDialog = saveFileDialog1;
             _fileSessionManager.OnNewing += fileSessionManager_OnNewSession;
@@ -53,6 +41,8 @@ namespace RokettoLaunch
             _fileSessionManager.OnSaving += fileSessionManager_OnSaveSession;
             _fileSessionManager.OnClearing += fileSessionManager_OnClearSession;
             _fileSessionManager.OnFilenameChanged += fileSessionManager_OnFilenameChanged;
+
+            notifyIcon1.Text = Application.ProductName;
         }
 
         protected async override void OnShown(EventArgs e)
@@ -88,7 +78,7 @@ namespace RokettoLaunch
                 this.CurrentSession = new App4Options();
             }
 
-            if (this.CurrentSession.HideOnClose && !_requestExit)
+            if (this.CurrentSession.HideOnClose && !requestExit)
             {
                 this.Hide();
 
@@ -96,7 +86,7 @@ namespace RokettoLaunch
                 return;
             }
 
-            _requestExit = false;
+            requestExit = false;
 
             var result = await _fileSessionManager.CloseSession();
             if (!result)
@@ -125,7 +115,7 @@ namespace RokettoLaunch
 
                     break;
                 case RyzStudio.Runtime.InteropServices.User32.WM_QUERYENDSESSION:
-                    _requestExit = true;
+                    requestExit = true;
 
                     Application.Exit();
 
@@ -139,8 +129,19 @@ namespace RokettoLaunch
 
 
         [Browsable(false), DesignerSerializationVisibility(DesignerSerializationVisibility.Hidden)]
-        public App4Options CurrentSession { get; set; } = null;
+        public App4Options CurrentSession
+        {
+            get
+            {
+                if (currentSession == null)
+                {
+                    currentSession = new App4Options();
+                }
 
+                return currentSession;
+            }
+            set => currentSession = value;
+        }
 
         private void InvalidateOptions(bool resize)
         {
@@ -205,7 +206,7 @@ namespace RokettoLaunch
         /// <param name="e"></param>
         private async void NewToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
@@ -220,7 +221,7 @@ namespace RokettoLaunch
         /// <param name="e"></param>
         private async void OpenToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
@@ -235,7 +236,7 @@ namespace RokettoLaunch
         /// <param name="e"></param>
         private async void CloseToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
@@ -250,7 +251,7 @@ namespace RokettoLaunch
         /// <param name="e"></param>
         private async void SaveToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
@@ -265,7 +266,7 @@ namespace RokettoLaunch
         /// <param name="e"></param>
         private async void SaveAsToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
@@ -280,12 +281,12 @@ namespace RokettoLaunch
         /// <param name="e"></param>
         private void ExitToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
 
-            _requestExit = true;
+            requestExit = true;
 
             this.Close();
         }
@@ -404,12 +405,12 @@ namespace RokettoLaunch
 
         private void NotifyExitToolStripMenuItem_Click(object sender, EventArgs e)
         {
-            if (_isBusy)
+            if (isBusy)
             {
                 return;
             }
 
-            _requestExit = true;
+            requestExit = true;
 
             this.Close();
         }
@@ -435,8 +436,7 @@ namespace RokettoLaunch
 
         private async Task<bool> fileSessionManager_OnLoadSession(FileSessionManager sender, string filename, int formatType)
         {
-            // Progress indicator
+            var loadingForm = new LoadingForm();
-            var loadingForm = _provider.GetRequiredService<LoadingForm>();
             var result = await loadingForm.ShowDialog(filename);
             if (result != DialogResult.OK)
             {
@@ -473,12 +473,12 @@ namespace RokettoLaunch
 
             return await Task.Run(async () =>
             {
-                if (_isBusy)
+                if (isBusy)
                 {
                     return false;
                 }
 
-                _isBusy = true;
+                isBusy = true;
 
                 // update session
                 if (this.CurrentSession == null)
@@ -530,7 +530,7 @@ namespace RokettoLaunch
                     }
                 }
 
-                _isBusy = false;
+                isBusy = false;
 
                 return result.IsSuccess;
             });
@@ -724,7 +724,7 @@ namespace RokettoLaunch
                 return;
             }
 
-            _isBusy = true;
+            isBusy = true;
 
             await Task.Run(() =>
             {
@@ -735,7 +735,7 @@ namespace RokettoLaunch
             });
 
             _fileSessionManager.HasChanged = true;
-            _isBusy = false;
+            isBusy = false;
         }
 
         /// <summary>

Models/NewFormModel.cs

@@ -1,10 +0,0 @@
-namespace RokettoLaunch.Models
-{
-    public class NewFormModel
-    {
-        public int ColumnCount { get; set; } = 8;
-
-        public int GroupCount { get; set; } = 1;
-
-    }
-}

Program.cs

@@ -1,7 +1,5 @@
 using System;
 using System.Windows.Forms;
-using Microsoft.Extensions.DependencyInjection;
-using RyzStudio.Windows.Forms;
 
 namespace RokettoLaunch
 {
@@ -13,19 +11,10 @@ namespace RokettoLaunch
         [STAThread]
         static void Main()
         {
-            var services = new ServiceCollection();
-            services.AddSingleton<IFileSessionManager, FileSessionManager>();
-            services.AddTransient<MainForm>();
-            services.AddTransient<LoadingForm>();
-
-            var provider = services.BuildServiceProvider();
-
             Application.SetHighDpiMode(HighDpiMode.SystemAware);
             Application.EnableVisualStyles();
             Application.SetCompatibleTextRenderingDefault(false);
-            //Application.Run(new MainForm());
+            Application.Run(new MainForm());
-
-            Application.Run(provider.GetRequiredService<MainForm>());
         }
     }
 }

RokettoLaunch.csproj

@@ -14,7 +14,7 @@
     <Copyright>Ray Lam</Copyright>
     <AssemblyVersion>1.0.0.0</AssemblyVersion>
     <FileVersion>1.0.0.0</FileVersion>
-    <Version>0.4.1.0202</Version>
+    <Version>0.4.1.0217</Version>
     <EnableNETAnalyzers>False</EnableNETAnalyzers>
     <PlatformTarget>x64</PlatformTarget>
     <PackageIcon>icon-128.png</PackageIcon>
@@ -31,15 +31,15 @@
 
   <ItemGroup>
     <Compile Remove="#\**" />
-    <Compile Remove="Resources\UI\**" />
+    <Compile Remove="Resources\**" />
     <Compile Remove="RyzStudio\**" />
     <Compile Remove="Text\**" />
     <EmbeddedResource Remove="#\**" />
-    <EmbeddedResource Remove="Resources\UI\**" />
+    <EmbeddedResource Remove="Resources\**" />
     <EmbeddedResource Remove="RyzStudio\**" />
     <EmbeddedResource Remove="Text\**" />
     <None Remove="#\**" />
-    <None Remove="Resources\UI\**" />
+    <None Remove="Resources\**" />
     <None Remove="RyzStudio\**" />
     <None Remove="Text\**" />
   </ItemGroup>
@@ -79,7 +79,6 @@
   </ItemGroup>
 
   <ItemGroup>
-    <PackageReference Include="Microsoft.Extensions.Hosting" Version="10.0.7" />
     <PackageReference Include="RyzStudio" Version="10.1.0.128" />
     <PackageReference Include="RyzStudio.Windows.Forms" Version="10.1.0.140" />
   </ItemGroup>
@@ -89,10 +88,10 @@
       <AutoGen>True</AutoGen>
       <DependentUpon>AppResource.resx</DependentUpon>
     </Compile>
-    <Compile Update="CopyToTileForm.cs">
+    <Compile Update="Forms\CopyToTileForm.cs">
       <SubType>Form</SubType>
     </Compile>
-    <Compile Update="LoadingForm.cs">
+    <Compile Update="Forms\LoadingForm.cs">
       <SubType>Form</SubType>
     </Compile>
   </ItemGroup>
@@ -104,11 +103,4 @@
     </EmbeddedResource>
   </ItemGroup>
 
-  <ItemGroup>
-    <None Update="Resources\icon-128.png">
-      <Pack>True</Pack>
-      <PackagePath>\</PackagePath>
-    </None>
-  </ItemGroup>
-
 </Project>