From 6040175eb02f56a6da86f90a8a79840abd4774bc Mon Sep 17 00:00:00 2001
From: isc-akadoo <anagha.kadoo@intersystems.com>
Date: Fri, 27 Sep 2024 08:58:00 -0400
Subject: [PATCH 1/4] Added link back to IRIS management portal

---
 CHANGELOG.md                                  |  4 +++
 cls/SourceControl/Git/WebUIDriver.cls         |  8 +++++
 csp/gitprojectsettings.csp                    | 31 +++++++++++++++++--
 .../share/git-webui/webui/css/git-webui.css   | 20 +++++++++---
 .../share/git-webui/webui/img/home.svg        |  3 ++
 .../share/git-webui/webui/js/git-webui.js     | 31 ++++++++++++++-----
 .../share/git-webui/webui/css/git-webui.less  | 23 +++++++++++---
 .../src/share/git-webui/webui/img/home.svg    |  3 ++
 .../src/share/git-webui/webui/js/git-webui.js | 28 ++++++++++++++---
 9 files changed, 127 insertions(+), 24 deletions(-)
 create mode 100644 git-webui/release/share/git-webui/webui/img/home.svg
 create mode 100644 git-webui/src/share/git-webui/webui/img/home.svg

diff --git a/CHANGELOG.md b/CHANGELOG.md
index 052a2e6f..d8c53359 100644
--- a/CHANGELOG.md
+++ b/CHANGELOG.md
@@ -7,9 +7,13 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [2.5.1] - Unreleased
 
+### Added
+- Added link back to IRIS management portal from Settings, Git WebUI pages (#449)
+
 ### Fixed
 - Changed prompts in configure from 0/1 to no/yes (#461)
 - Added warnings when user is using incompatible git version (#488)
+- Fixed the back button navigation between WebUI and Settings page (#361)
 
 ## [2.5.0] - 2024-09-24
 
diff --git a/cls/SourceControl/Git/WebUIDriver.cls b/cls/SourceControl/Git/WebUIDriver.cls
index 5aead7ab..4e4352bb 100644
--- a/cls/SourceControl/Git/WebUIDriver.cls
+++ b/cls/SourceControl/Git/WebUIDriver.cls
@@ -22,6 +22,8 @@ ClassMethod HandleRequest(pagePath As %String, InternalName As %String = "", Out
             set responseJSON = ..GetPackageVersion()
         } elseif $extract(pagePath, 6, *) = "git-version" {
             set responseJSON = ..GetGitVersion()
+        } elseif $extract(pagePath,6,*) = "home"{
+            set responseJSON = ..GetHomeURL()
         } else {
             set %response.Status = ##class(%CSP.REST).#HTTP404NOTFOUND
             set responseJSON = {"error":("invalid URI: " _ pagePath)}
@@ -340,4 +342,10 @@ ClassMethod GetGitVersion() As %Library.DynamicObject
     quit {"version": (version)}
 }
 
+ClassMethod GetHomeURL() As %SystemBase
+{
+    set homeURL = "/csp/sys/UtilHome.csp"
+    quit {"url": (homeURL)}
+}
+
 }
diff --git a/csp/gitprojectsettings.csp b/csp/gitprojectsettings.csp
index c116f6e7..a5c772ed 100644
--- a/csp/gitprojectsettings.csp
+++ b/csp/gitprojectsettings.csp
@@ -40,6 +40,9 @@ body {
 #webuiURL {
     display: none;
 }
+#homeURL {
+    display: none;
+}
 
 .custom-switch {
     padding: 0.4rem 0.5rem 0 2.75rem;
@@ -73,6 +76,7 @@ body {
  set version = ##class(SourceControl.Git.Utils).GetPackageVersion()
  set webuiURL = "/isc/studio/usertemplates/gitsourcecontrol/webuidriver.csp/"_namespace_"/?CSPSHARE=1"
  set webuiURL = ##class(SourceControl.Git.WebUIDriver).GetURLPrefix(%request, webuiURL)
+ set homeURL = "/csp/sys/UtilHome.csp"
 
  set settings = ##class(SourceControl.Git.Settings).%New()
  if $Data(%request.Data("gitsettings",1)) {
@@ -128,12 +132,17 @@ body {
         <input type="hidden" name="gitsettings" value="1" />
         <div class="col-sm-12"><br></div>
         <div class="row">
-            <div class="offset-sm-1 col-sm-8">
+            <div class="offset-sm-1 col-sm-7" id="settingsTitle">
                 <h1>Git Project Settings</h1>
                 <h3> Package version: #(version)# </h3>
             </div>
 
-            <div class="col-sm-2">
+            <div class="col-sm-4" id="settingsNavBtn">
+                <button class="btn btn-lg btn-outline-dark" id="goToHome">Home
+                </button>
+                <div id="homeURL">
+                    #(homeURL)#
+                </div>
                 <button class="btn btn-lg btn-outline-dark" id="goToWebUI">Go to WebUI
                 </button>
                 <div id="webuiURL">
@@ -542,6 +551,17 @@ body {
 <script src="js/popper.min.js"></script>
 <script src="js/bootstrap.min.js"></script>
 <script type="text/javascript">
+
+// Removing the home button if not a top-level page
+if ((window !== window.parent) || (navigator.userAgent.indexOf('MSIE 7') > -1) || (navigator.userAgent.indexOf(" Code/") > -1)) {
+    $("#goToHome", self.element).remove();
+    $("#homeURL", self.element).remove();
+    $('#settingsTitle').removeClass('col-sm-7');
+    $('#settingsNavBtn').removeClass('col-sm-4');
+    $('#settingsTitle').addClass('col-sm-8');
+    $('#settingsNavBtn').addClass('col-sm-2');
+}
+
 function getSocket(urlPostfix) {
     var socketURL = window.location.href.replace('http','ws').replace('gitprojectsettings.csp','_zpkg.isc.sc.git.Socket.cls') + "&" + urlPostfix;
 	return new WebSocket(socketURL);
@@ -634,7 +654,12 @@ $(function()
     $(document).on('click', "#goToWebUI", function(e){
         e.preventDefault();
         console.log($("#webuiURL")[0].innerText);
-        window.location.replace($("#webuiURL")[0].innerText);
+        window.location.href = $("#webuiURL")[0].innerText;
+    })
+
+    $(document).on('click', "#goToHome", function(e){
+        e.preventDefault();
+        window.location.href = $("#homeURL")[0].innerText;
     })
 });
 
diff --git a/git-webui/release/share/git-webui/webui/css/git-webui.css b/git-webui/release/share/git-webui/webui/css/git-webui.css
index 0c75c1f4..a455bf8b 100644
--- a/git-webui/release/share/git-webui/webui/css/git-webui.css
+++ b/git-webui/release/share/git-webui/webui/css/git-webui.css
@@ -183,7 +183,7 @@ body {
   padding-bottom: 100px;
 }
 #sidebar #sidebar-content > :first-child,
-#sidebar #sidebar-content > :nth-last-child(3) {
+#sidebar #sidebar-content > :last-child {
   border-top: 1px solid #5e5e5e;
 }
 #sidebar #sidebar-content h4:before {
@@ -227,7 +227,7 @@ body {
 }
 #sidebar #sidebar-content #sidebar-vscode {
   position: absolute;
-  bottom: 80px;
+  bottom: 90px;
   width: 16.7em;
   background-color: #333333;
 }
@@ -239,7 +239,7 @@ body {
 }
 #sidebar #sidebar-content #sidebar-context {
   position: absolute;
-  bottom: 40px;
+  bottom: 50px;
   width: 16.7em;
   background-color: #333333;
 }
@@ -251,7 +251,19 @@ body {
 }
 #sidebar #sidebar-content #sidebar-settings {
   position: absolute;
-  bottom: 0px;
+  bottom: 10px;
+  width: 16.7em;
+  background-color: #333333;
+}
+#sidebar #sidebar-content #sidebar-home h4 {
+  padding: 0px;
+}
+#sidebar #sidebar-content #sidebar-home h4:before {
+  content: url(../img/home.svg);
+}
+#sidebar #sidebar-content #sidebar-home {
+  position: absolute;
+  bottom: 130px;
   width: 16.7em;
   background-color: #333333;
 }
diff --git a/git-webui/release/share/git-webui/webui/img/home.svg b/git-webui/release/share/git-webui/webui/img/home.svg
new file mode 100644
index 00000000..54cab057
--- /dev/null
+++ b/git-webui/release/share/git-webui/webui/img/home.svg
@@ -0,0 +1,3 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="#eee" class="bi bi-house-door-fill" viewBox="0 0 16 16">
+  <path d="M6.5 14.5v-3.505c0-.245.25-.495.5-.495h2c.25 0 .5.25.5.5v3.5a.5.5 0 0 0 .5.5h4a.5.5 0 0 0 .5-.5v-7a.5.5 0 0 0-.146-.354L13 5.793V2.5a.5.5 0 0 0-.5-.5h-1a.5.5 0 0 0-.5.5v1.293L8.354 1.146a.5.5 0 0 0-.708 0l-6 6A.5.5 0 0 0 1.5 7.5v7a.5.5 0 0 0 .5.5h4a.5.5 0 0 0 .5-.5"/>
+</svg>
\ No newline at end of file
diff --git a/git-webui/release/share/git-webui/webui/js/git-webui.js b/git-webui/release/share/git-webui/webui/js/git-webui.js
index 49006e1b..016021db 100644
--- a/git-webui/release/share/git-webui/webui/js/git-webui.js
+++ b/git-webui/release/share/git-webui/webui/js/git-webui.js
@@ -69,6 +69,12 @@ $.get("api/settings", function(settingsURL){
     webui.settingsURL = url.url;
 });
 
+webui.homeURL = "";
+$.get("api/home", function(homeURL){
+    var url = JSON.parse(homeURL);
+    webui.homeURL = url.url;
+});
+
 webui.showSuccess = function(message) {
     var messageBox = $("#message-box");
     messageBox.empty();
@@ -913,7 +919,11 @@ webui.SideBarView = function(mainView, noEventHandlers) {
     }
 
     self.goToSettingsPage = function() {
-        window.location.replace(webui.settingsURL);
+        window.location.href = webui.settingsURL;
+    }
+
+    self.goToHomePage = function() {
+        window.location.href = webui.homeURL;
     }
 
     self.fetchSection = function(section, title, id, gitCommand) {
@@ -983,17 +993,17 @@ webui.SideBarView = function(mainView, noEventHandlers) {
                                         webui.refreshIcon+
                                       '</button>' +'</h4>' +
                                 '</section>' +
-                                '<section id="sidebar-tags">' +
-                                    '<h4>Tags</h4>' +
-                                '</section>' +
-                                '<section id="sidebar-vscode">' +
-                                    '<h4><a href="vscode-workspace" target="_blank">Code Workspace</a></h4>' +
+                                '<section id="sidebar-settings">' +
+                                    '<h4>Settings</h4>' +
                                 '</section>' +
                                 '<section id="sidebar-context" data-toggle="tooltip" data-placement="right" title="' + self.currentContext + '">' + 
                                     '<h4>Change Context</h4>' +
                                 '</section>' +
-                                '<section id="sidebar-settings">' +
-                                    '<h4>Settings</h4>' +
+                                '<section id="sidebar-vscode">' +
+                                    '<h4><a href="vscode-workspace" target="_blank">Code Workspace</a></h4>' +
+                                '</section>' +
+                                '<section id="sidebar-home">' +
+                                    '<h4>Home</h4>' +
                                 '</section>' +
                             '</div>' +
                         '</div>')[0];
@@ -1019,8 +1029,13 @@ webui.SideBarView = function(mainView, noEventHandlers) {
         $('.btn-prune-remote-branches', self.element).click(self.pruneRemoteBranches);
         $("#sidebar-settings", self.element).click(self.goToSettingsPage);
         $("#sidebar-context", self.element).click(self.changeContextGet);
+        $("#sidebar-home", self.element).click(self.goToHomePage);
     }
 
+    // Removing the link to home if not a top-level page
+    if ((window !== window.parent) || (navigator.userAgent.indexOf('MSIE 7') > -1) || (navigator.userAgent.indexOf(" Code/") > -1)) {
+        $("#sidebar-home", self.element).remove();
+    }
     
     self.getPackageVersion();
     self.fetchSection($("#sidebar-local-branches", self.element)[0], "Local Branches", "local-branches", "branch --verbose --verbose");
diff --git a/git-webui/src/share/git-webui/webui/css/git-webui.less b/git-webui/src/share/git-webui/webui/css/git-webui.less
index 8fb86733..45122c72 100644
--- a/git-webui/src/share/git-webui/webui/css/git-webui.less
+++ b/git-webui/src/share/git-webui/webui/css/git-webui.less
@@ -257,7 +257,7 @@ body {
         color: @gray-lighter;
         padding-bottom: 100px;
 
-        > :first-child, >:nth-last-child(3) {
+        > :first-child, >:last-child {
             border-top: 1px solid darken(@separator-line, 50%);
         }
 
@@ -315,7 +315,7 @@ body {
 
         #sidebar-vscode {
             position: absolute;
-            bottom: 80px;
+            bottom: 90px;
             width: 16.7em;
             background-color: rgba(51, 51, 51, 1);
         }
@@ -330,7 +330,7 @@ body {
 
         #sidebar-context {
             position: absolute;
-            bottom: 40px;
+            bottom: 50px;
             width: 16.7em;
             background-color: rgba(51, 51, 51, 1);
         }
@@ -345,7 +345,22 @@ body {
 
         #sidebar-settings {
             position: absolute;
-            bottom: 0px;
+            bottom: 10px;
+            width: 16.7em;
+            background-color: rgba(51, 51, 51, 1);
+        }
+
+        #sidebar-home h4 {
+            padding: 0px;
+        }
+
+        #sidebar-home h4:before {
+            content: url(../img/home.svg);
+        }
+
+        #sidebar-home {
+            position: absolute;
+            bottom: 130px;
             width: 16.7em;
             background-color: rgba(51, 51, 51, 1);
         }
diff --git a/git-webui/src/share/git-webui/webui/img/home.svg b/git-webui/src/share/git-webui/webui/img/home.svg
new file mode 100644
index 00000000..54cab057
--- /dev/null
+++ b/git-webui/src/share/git-webui/webui/img/home.svg
@@ -0,0 +1,3 @@
+<svg xmlns="http://www.w3.org/2000/svg" width="16" height="16" fill="#eee" class="bi bi-house-door-fill" viewBox="0 0 16 16">
+  <path d="M6.5 14.5v-3.505c0-.245.25-.495.5-.495h2c.25 0 .5.25.5.5v3.5a.5.5 0 0 0 .5.5h4a.5.5 0 0 0 .5-.5v-7a.5.5 0 0 0-.146-.354L13 5.793V2.5a.5.5 0 0 0-.5-.5h-1a.5.5 0 0 0-.5.5v1.293L8.354 1.146a.5.5 0 0 0-.708 0l-6 6A.5.5 0 0 0 1.5 7.5v7a.5.5 0 0 0 .5.5h4a.5.5 0 0 0 .5-.5"/>
+</svg>
\ No newline at end of file
diff --git a/git-webui/src/share/git-webui/webui/js/git-webui.js b/git-webui/src/share/git-webui/webui/js/git-webui.js
index 49006e1b..05c650c5 100644
--- a/git-webui/src/share/git-webui/webui/js/git-webui.js
+++ b/git-webui/src/share/git-webui/webui/js/git-webui.js
@@ -69,6 +69,12 @@ $.get("api/settings", function(settingsURL){
     webui.settingsURL = url.url;
 });
 
+webui.homeURL = "";
+$.get("api/home", function(homeURL){
+    var url = JSON.parse(homeURL);
+    webui.homeURL = url.url;
+});
+
 webui.showSuccess = function(message) {
     var messageBox = $("#message-box");
     messageBox.empty();
@@ -913,7 +919,11 @@ webui.SideBarView = function(mainView, noEventHandlers) {
     }
 
     self.goToSettingsPage = function() {
-        window.location.replace(webui.settingsURL);
+        window.location.href = webui.settingsURL;
+    }
+
+    self.goToHomePage = function() {
+        window.location.href = webui.homeURL;
     }
 
     self.fetchSection = function(section, title, id, gitCommand) {
@@ -986,14 +996,17 @@ webui.SideBarView = function(mainView, noEventHandlers) {
                                 '<section id="sidebar-tags">' +
                                     '<h4>Tags</h4>' +
                                 '</section>' +
-                                '<section id="sidebar-vscode">' +
-                                    '<h4><a href="vscode-workspace" target="_blank">Code Workspace</a></h4>' +
+                                '<section id="sidebar-settings">' +
+                                    '<h4>Settings</h4>' +
                                 '</section>' +
                                 '<section id="sidebar-context" data-toggle="tooltip" data-placement="right" title="' + self.currentContext + '">' + 
                                     '<h4>Change Context</h4>' +
                                 '</section>' +
-                                '<section id="sidebar-settings">' +
-                                    '<h4>Settings</h4>' +
+                                '<section id="sidebar-vscode">' +
+                                    '<h4><a href="vscode-workspace" target="_blank">Code Workspace</a></h4>' +
+                                '</section>' +
+                                '<section id="sidebar-home">' +
+                                    '<h4>Home</h4>' +
                                 '</section>' +
                             '</div>' +
                         '</div>')[0];
@@ -1019,8 +1032,13 @@ webui.SideBarView = function(mainView, noEventHandlers) {
         $('.btn-prune-remote-branches', self.element).click(self.pruneRemoteBranches);
         $("#sidebar-settings", self.element).click(self.goToSettingsPage);
         $("#sidebar-context", self.element).click(self.changeContextGet);
+        $("#sidebar-home", self.element).click(self.goToHomePage);
     }
 
+    // Removing the link to home if not a top-level page
+    if ((window !== window.parent) || (navigator.userAgent.indexOf('MSIE 7') > -1) || (navigator.userAgent.indexOf(" Code/") > -1)) {
+        $("#sidebar-home", self.element).remove();
+    }
     
     self.getPackageVersion();
     self.fetchSection($("#sidebar-local-branches", self.element)[0], "Local Branches", "local-branches", "branch --verbose --verbose");

From 7434007e2a21145460cd55e1ffb348f9a51c7f8e Mon Sep 17 00:00:00 2001
From: isc-akadoo <anagha.kadoo@intersystems.com>
Date: Fri, 27 Sep 2024 09:48:07 -0400
Subject: [PATCH 2/4] Re-added tags element

---
 git-webui/release/share/git-webui/webui/js/git-webui.js | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/git-webui/release/share/git-webui/webui/js/git-webui.js b/git-webui/release/share/git-webui/webui/js/git-webui.js
index 016021db..05c650c5 100644
--- a/git-webui/release/share/git-webui/webui/js/git-webui.js
+++ b/git-webui/release/share/git-webui/webui/js/git-webui.js
@@ -993,6 +993,9 @@ webui.SideBarView = function(mainView, noEventHandlers) {
                                         webui.refreshIcon+
                                       '</button>' +'</h4>' +
                                 '</section>' +
+                                '<section id="sidebar-tags">' +
+                                    '<h4>Tags</h4>' +
+                                '</section>' +
                                 '<section id="sidebar-settings">' +
                                     '<h4>Settings</h4>' +
                                 '</section>' +

From 6dfbd9a3ef246056d669f65d7211f3e6d9c6d4f3 Mon Sep 17 00:00:00 2001
From: isc-akadoo <anagha.kadoo@intersystems.com>
Date: Fri, 27 Sep 2024 10:06:33 -0400
Subject: [PATCH 3/4] Removed console log

---
 csp/gitprojectsettings.csp | 1 -
 1 file changed, 1 deletion(-)

diff --git a/csp/gitprojectsettings.csp b/csp/gitprojectsettings.csp
index a5c772ed..857170c3 100644
--- a/csp/gitprojectsettings.csp
+++ b/csp/gitprojectsettings.csp
@@ -653,7 +653,6 @@ $(function()
 
     $(document).on('click', "#goToWebUI", function(e){
         e.preventDefault();
-        console.log($("#webuiURL")[0].innerText);
         window.location.href = $("#webuiURL")[0].innerText;
     })
 

From e4acf51f8f09ccd1c9c69a794c963d1a329219d6 Mon Sep 17 00:00:00 2001
From: isc-tleavitt <73311181+isc-tleavitt@users.noreply.github.com>
Date: Fri, 27 Sep 2024 14:13:36 -0400
Subject: [PATCH 4/4] fix: support CSPInstanceName prefix when used

Also refactors to a single method to return home URL. (From working session/touchpoint.)
---
 cls/SourceControl/Git/WebUIDriver.cls | 13 +++++++++----
 csp/gitprojectsettings.csp            |  2 +-
 2 files changed, 10 insertions(+), 5 deletions(-)

diff --git a/cls/SourceControl/Git/WebUIDriver.cls b/cls/SourceControl/Git/WebUIDriver.cls
index 4e4352bb..0767ed86 100644
--- a/cls/SourceControl/Git/WebUIDriver.cls
+++ b/cls/SourceControl/Git/WebUIDriver.cls
@@ -23,7 +23,9 @@ ClassMethod HandleRequest(pagePath As %String, InternalName As %String = "", Out
         } elseif $extract(pagePath, 6, *) = "git-version" {
             set responseJSON = ..GetGitVersion()
         } elseif $extract(pagePath,6,*) = "home"{
-            set responseJSON = ..GetHomeURL()
+            set responseJSON = {
+                "url": (..GetHomeURL())
+            }
         } else {
             set %response.Status = ##class(%CSP.REST).#HTTP404NOTFOUND
             set responseJSON = {"error":("invalid URI: " _ pagePath)}
@@ -342,10 +344,13 @@ ClassMethod GetGitVersion() As %Library.DynamicObject
     quit {"version": (version)}
 }
 
-ClassMethod GetHomeURL() As %SystemBase
+ClassMethod GetHomeURL() As %String
 {
-    set homeURL = "/csp/sys/UtilHome.csp"
-    quit {"url": (homeURL)}
+    set prefix = ""
+    if $IsObject($Get(%request)) {
+        set prefix = $Piece(%request.Application,%request.AppMatch)
+    }
+    quit prefix_"/csp/sys/UtilHome.csp"
 }
 
 }
diff --git a/csp/gitprojectsettings.csp b/csp/gitprojectsettings.csp
index 857170c3..aa792a2e 100644
--- a/csp/gitprojectsettings.csp
+++ b/csp/gitprojectsettings.csp
@@ -76,7 +76,7 @@ body {
  set version = ##class(SourceControl.Git.Utils).GetPackageVersion()
  set webuiURL = "/isc/studio/usertemplates/gitsourcecontrol/webuidriver.csp/"_namespace_"/?CSPSHARE=1"
  set webuiURL = ##class(SourceControl.Git.WebUIDriver).GetURLPrefix(%request, webuiURL)
- set homeURL = "/csp/sys/UtilHome.csp"
+ set homeURL = ##class(SourceControl.Git.WebUIDriver).GetHomeURL()
 
  set settings = ##class(SourceControl.Git.Settings).%New()
  if $Data(%request.Data("gitsettings",1)) {