fix: production items are exported for source control when production is not in source control

required now that production class is not a separate source controlled item

Author: isc-pbarton

cls/SourceControl/Git/Extension.cls

@@ -276,7 +276,7 @@ Method OnAfterSave(InternalName As %String, Object As %RegisteredObject = {$$$NU
     try {
         set InternalName = ##class(SourceControl.Git.Utils).NormalizeInternalName(.InternalName)
         set context = ##class(SourceControl.Git.PackageManagerContext).ForInternalName(InternalName)
-        if ##class(SourceControl.Git.Utils).IsNamespaceInGit() && ..IsInSourceControl(InternalName) {
+        if ##class(SourceControl.Git.Utils).IsNamespaceInGit() {
             // If this is a production class and production decomposition is enabled, call recursively on all modified production items.
             if ##class(SourceControl.Git.Utils).ItemIsProductionToDecompose(InternalName) {
                 do ##class(SourceControl.Git.Production).GetModifiedItemsAfterSave(InternalName, .productionItems)
@@ -296,11 +296,13 @@ Method OnAfterSave(InternalName As %String, Object As %RegisteredObject = {$$$NU
                     set key = $order(productionItems(key))
                 }
             }
-            set filename = ##class(SourceControl.Git.Utils).FullExternalName(InternalName)
-            $$$ThrowOnError(##class(SourceControl.Git.Utils).RemoveRoutineTSH(InternalName))
-            $$$ThrowOnError(##class(SourceControl.Git.Utils).ExportItem(InternalName))
-            if '##class(SourceControl.Git.Change).IsUncommitted(filename) {
-                $$$ThrowOnError(##class(SourceControl.Git.Change).SetUncommitted(filename, "edit", InternalName, $username, "", 1, "", "", 0))
+            if ..IsInSourceControl(InternalName) {
+                set filename = ##class(SourceControl.Git.Utils).FullExternalName(InternalName)
+                $$$ThrowOnError(##class(SourceControl.Git.Utils).RemoveRoutineTSH(InternalName))
+                $$$ThrowOnError(##class(SourceControl.Git.Utils).ExportItem(InternalName))
+                if '##class(SourceControl.Git.Change).IsUncommitted(filename) {
+                    $$$ThrowOnError(##class(SourceControl.Git.Change).SetUncommitted(filename, "edit", InternalName, $username, "", 1, "", "", 0))
+                }
             }
         }
     } catch e {

cls/SourceControl/Git/Utils.cls

@@ -1534,18 +1534,21 @@ ClassMethod ExportItem(InternalName As %String, expand As %Boolean = 1, force As
                 write "Mapping to another database found. Skipping export"
                 quit $$$OK
             }
-            set filenames($I(filenames)) = filename
             write !, "exporting new version of ", InternalName, " to ", filename
             if (type = "ptd") {
                 $$$QuitOnError(##class(SourceControl.Git.Production).ExportPTD(InternalName,"FullExternalName"))
             } elseif (..ItemIsProductionToDecompose(InternalName, .productionName)) {
                 write !, "Production decomposition enabled, skipping export of production class"
+                set filename = ""
             } else {
                 $$$QuitOnError($system.OBJ.ExportUDL(InternalName, filename,"-d/diff"))
             }
-            $$$QuitOnError(..UpdateRoutineTSH(InternalName, $h))
-            if '##class(SourceControl.Git.Change).IsUncommitted(filename) {
-                $$$ThrowOnError(##class(SourceControl.Git.Change).SetUncommitted(filename, "add", InternalName, $username, "", 1, "", "", 0))
+            if (filename '= "") {
+                set filenames($I(filenames)) = filename
+                $$$QuitOnError(..UpdateRoutineTSH(InternalName, $h))
+                if '##class(SourceControl.Git.Change).IsUncommitted(filename) {
+                    $$$ThrowOnError(##class(SourceControl.Git.Change).SetUncommitted(filename, "add", InternalName, $username, "", 1, "", "", 0))
+                }
             }
         }
     }