From be2f28f47b95527a3995f5e43ee368924a01e0e1 Mon Sep 17 00:00:00 2001
From: Emanuele Filardo <dev@n4notech.it>
Date: Fri, 18 Oct 2019 22:32:41 +0200
Subject: [PATCH 1/3] reset pages list

---
 Oqtane.Client/Themes/Controls/ControlPanel.razor | 10 ++++++----
 1 file changed, 6 insertions(+), 4 deletions(-)

diff --git a/Oqtane.Client/Themes/Controls/ControlPanel.razor b/Oqtane.Client/Themes/Controls/ControlPanel.razor
index c370012b..bc193f20 100644
--- a/Oqtane.Client/Themes/Controls/ControlPanel.razor
+++ b/Oqtane.Client/Themes/Controls/ControlPanel.razor
@@ -48,7 +48,7 @@
                                         }
                                     </select>
                                     <select class="form-control" @bind="@moduledefinitionname">
-                                        <option value="">&lt;Select Module&gt;</option>
+                                        <option value="-">&lt;Select Module&gt;</option>
                                         @foreach (var moduledefinition in moduledefinitions)
                                         {
                                             if (moduledefinition.Permissions == "[]" || UserSecurity.IsAuthorized(PageState.User, "Utilize", moduledefinition.Permissions))
@@ -148,11 +148,11 @@
     string moduletype = "new";
     List<string> categories = new List<string>();
     List<ModuleDefinition> moduledefinitions;
-    List<Page> pages = new List<Page>();
+    List<Page> pages;
     string moduleid = "";
     List<Module> modules = new List<Module>();
     Dictionary<string, string> containers = new Dictionary<string, string>();
-    string moduledefinitionname = "";
+    string moduledefinitionname = "-";
     string pane = "";
     string title = "";
     string containertype = "";
@@ -174,6 +174,8 @@
 
         if (UserSecurity.IsAuthorized(PageState.User, "Edit", PageState.Page.Permissions))
         {
+            pages = new List<Page>();
+
             foreach (ModuleDefinition moduledefinition in PageState.ModuleDefinitions)
             {
                 if (moduledefinition.Categories != "")
@@ -213,7 +215,7 @@
         {
             moduledefinitions = PageState.ModuleDefinitions.Where(item => item.Categories.Contains(e.Value.ToString())).ToList();
         }
-        moduledefinitionname = "";
+        moduledefinitionname = "-";
         StateHasChanged();
     }
 

From 3ab47c63e7676a734f0c50dab445ba1ae12bf674 Mon Sep 17 00:00:00 2001
From: Emanuele Filardo <dev@n4notech.it>
Date: Fri, 18 Oct 2019 22:36:15 +0200
Subject: [PATCH 2/3] Add Existing module fix for pageId

---
 Oqtane.Client/Themes/Controls/ControlPanel.razor | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/Oqtane.Client/Themes/Controls/ControlPanel.razor b/Oqtane.Client/Themes/Controls/ControlPanel.razor
index bc193f20..2322ff34 100644
--- a/Oqtane.Client/Themes/Controls/ControlPanel.razor
+++ b/Oqtane.Client/Themes/Controls/ControlPanel.razor
@@ -149,6 +149,7 @@
     List<string> categories = new List<string>();
     List<ModuleDefinition> moduledefinitions;
     List<Page> pages;
+    string pageid = "";
     string moduleid = "";
     List<Module> modules = new List<Module>();
     Dictionary<string, string> containers = new Dictionary<string, string>();
@@ -221,7 +222,7 @@
 
     private void PageChanged(ChangeEventArgs e)
     {
-        string pageid = (string)e.Value;
+        pageid = (string)e.Value;
         if (pageid != "")
         {
             foreach(Module module in PageState.Modules.Where(item => item.PageId == int.Parse(pageid)))
@@ -251,7 +252,7 @@
             }
 
             PageModule pagemodule = new PageModule();
-            pagemodule.PageId = PageState.Page.PageId;
+            pagemodule.PageId = string.IsNullOrWhiteSpace(pageid) ? PageState.Page.PageId : int.Parse(pageid);
             pagemodule.ModuleId = int.Parse(moduleid);
             pagemodule.Title = title;
             if (pagemodule.Title == "")

From 6cb64b5ad6673ce509d1ad0d826f46283af39567 Mon Sep 17 00:00:00 2001
From: Emanuele Filardo <dev@n4notech.it>
Date: Fri, 18 Oct 2019 22:48:09 +0200
Subject: [PATCH 3/3] reset control to default value after module added

---
 Oqtane.Client/Themes/Controls/ControlPanel.razor | 14 ++++++++++----
 1 file changed, 10 insertions(+), 4 deletions(-)

diff --git a/Oqtane.Client/Themes/Controls/ControlPanel.razor b/Oqtane.Client/Themes/Controls/ControlPanel.razor
index 2322ff34..568b0c70 100644
--- a/Oqtane.Client/Themes/Controls/ControlPanel.razor
+++ b/Oqtane.Client/Themes/Controls/ControlPanel.razor
@@ -69,7 +69,7 @@
                                     }
                                 </select>
                                 <select class="form-control" @bind="@moduleid">
-                                    <option value="">&lt;Select Module&gt;</option>
+                                    <option value="-">&lt;Select Module&gt;</option>
                                     @foreach (Module module in modules)
                                     {
                                         <option value="@module.ModuleId">@module.Title</option>
@@ -150,7 +150,7 @@
     List<ModuleDefinition> moduledefinitions;
     List<Page> pages;
     string pageid = "";
-    string moduleid = "";
+    string moduleid = "-";
     List<Module> modules = new List<Module>();
     Dictionary<string, string> containers = new Dictionary<string, string>();
     string moduledefinitionname = "-";
@@ -225,7 +225,7 @@
         pageid = (string)e.Value;
         if (pageid != "")
         {
-            foreach(Module module in PageState.Modules.Where(item => item.PageId == int.Parse(pageid)))
+            foreach(Module module in PageState.Modules.Where(item => item.PageId == int.Parse(pageid) && !item.IsDeleted))
             {
                 if (UserSecurity.IsAuthorized(PageState.User, "View", module.Permissions))
                 {
@@ -233,7 +233,7 @@
                 }
             }
         }
-        moduleid = "";
+        moduleid = "-";
         StateHasChanged();
     }
 
@@ -280,6 +280,12 @@
 
             message = "<br /><div class=\"alert alert-success\" role=\"alert\">Module Added To Page</div>";
 
+            moduledefinitionname = "-";
+            pane = "";
+            title = "";
+            containertype = "";
+            moduleid = "-";
+
             NavigationManager.NavigateTo(NavigateUrl(Reload.Page));
         }
     }