fix: remove instance-wide uncommitted check for decomposed production class

minor performance improvement in instance-wide uncommitted check
fixed lock warning when uncommited check is disabled

Author: isc-pbarton

CHANGELOG.md

@@ -12,6 +12,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ### Fixed
 - Fixed errors on production page when item settings need to be XML escaped (#667)
+- No longer display instance-wide uncommitted warning with decomposed production (#547)
 
 ## [2.8.0] - 2024-12-06
 

cls/SourceControl/Git/Change.cls

@@ -202,15 +202,15 @@ Query InstanceUncommitted() As %Query(ROWSPEC = "InternalName:%String,User:%Stri
 
 ClassMethod InstanceUncommittedExecute(ByRef qHandle As %Binary) As %Status
 {
-    set qHandle("q") = "SELECT InternalName, ChangedBy FROM SourceControl_Git.Change"
+    set initNS = $namespace
     &sql(DECLARE InstanceCursor CURSOR FOR SELECT InternalName, ChangedBy into :internalName, :changedBy from SourceControl_Git.Change)
     set namespaces = ##class(SourceControl.Git.Utils).GetGitEnabledNamespaces()
     set tPtr = 0
     set qHandle("i") = 1
     new $namespace
     while $LISTNEXT(namespaces, tPtr, tValue) {
         set namespace = $ZCONVERT(tValue, "U")
-        if '(namespace [ "^") {
+        if '(namespace [ "^") && (namespace '= initNS) {
             set $NAMESPACE = namespace
 
                 &sql(OPEN InstanceCursor)

cls/SourceControl/Git/Extension.cls

@@ -56,23 +56,23 @@ Method UserAction(Type As %Integer, Name As %String, InternalName As %String, Se
     }
 
     if (Type = 1) && (Name = 3) {
-        if settings.warnInstanceWideUncommitted {
-            // if item is being edited in a different namespace, opening it will display an alert. Editing in this namespace will remove the alert.
-            set filename = ##class(SourceControl.Git.Utils).FullExternalName(.InternalName)
-            do ##class(SourceControl.Git.Change).GetUncommitted(filename,.tAction)
-            do ..GetStatus(.InternalName, .isInSourceControl, .isEditable,.isCheckedOut,.userCheckedOut)
-            if '$data(tAction) {
-                set user = "", inNamespace = ""
-                if 'isEditable || ##class(SourceControl.Git.Utils).Locked() {
-                    set Target = "Warning: Attempting to edit read-only file"
-                    write !, Target
-                    set Action = 6
-                } elseif ##class(SourceControl.Git.Utils).InstanceWideUncommittedCheck(InternalName, .user, .inNamespace) {
-                    set Target = "Warning: Item " _ InternalName _ " is currently being modified by " _ user _ " in namespace " _ inNamespace _ "."
-                    write !, Target
-                    set Action = 6
-                } 
-            }
+        set filename = ##class(SourceControl.Git.Utils).FullExternalName(.InternalName)
+        do ##class(SourceControl.Git.Change).GetUncommitted(filename,.tAction)
+        do ..GetStatus(.InternalName, .isInSourceControl, .isEditable,.isCheckedOut,.userCheckedOut)
+        if '$data(tAction) {
+            set user = "", inNamespace = ""
+            if 'isEditable || ##class(SourceControl.Git.Utils).Locked() {
+                set Target = "Warning: Attempting to edit read-only file"
+                write !, Target
+                set Action = 6
+            } elseif settings.warnInstanceWideUncommitted 
+                    && ##class(SourceControl.Git.Utils).InstanceWideUncommittedCheck(InternalName, .user, .inNamespace) 
+                    && '(##class(SourceControl.Git.Utils).ItemIsProductionToDecompose(InternalName)) {
+                // if item is being edited in a different namespace, opening it will display an alert. Editing in this namespace will remove the alert.
+                set Target = "Warning: Item " _ InternalName _ " is currently being modified by " _ user _ " in namespace " _ inNamespace _ "."
+                write !, Target
+                set Action = 6
+            } 
         }
     }