More small bugfixes from demo planning/recording

Author: isc-tleavitt

cls/SourceControl/Git/Extension.cls

@@ -127,11 +127,13 @@ Method OnSourceMenuContextItem(itemName As %String, menuItemName As %String, ByR
 	} elseif $F(itemName,",") > 0 { //if more than one item is selected, we can only add/remove, no diff or blame
 		set Enabled = $case(menuItemName,"%AddToSC":1,"%RemoveFromSC":1,:-1)
 	} elseif menuItemName = "%Revert" {
+		set Enabled = 1
 		do ..GetStatus(itemName, .isInSourceControl, .isEditable,.isCheckedOut,.userCheckedOut)
 		if '(##class(Change).IsUncommitted(##class(Utils).FullExternalName(itemName))) || ($username '= userCheckedOut) {
 			set Enabled = 0
 		}
 	} elseif menuItemName = "%Commit" {
+		set Enabled = 1
 		do ..GetStatus(itemName, .isInSourceControl, .isEditable,.isCheckedOut,.userCheckedOut)
 		if '(##class(Change).IsUncommitted(##class(Utils).FullExternalName(itemName))) || ($username '= userCheckedOut) {
 			set Enabled = 0

cls/SourceControl/Git/PullEventHandler/IncrementalLoad.cls

@@ -12,8 +12,8 @@ Method OnPull() As %Status
     set loadSC = $$$OK
     set nFiles = 0
     for i=1:1:$Get(..ModifiedFiles) {
-        if $Data(files(i))#2 {
-            set internalName = ##class(SourceControl.Git.Utils).NameToInternalName(..ModifiedFiles(i)) 
+        if $Data(..ModifiedFiles(i))#2 {
+            set internalName = ##class(SourceControl.Git.Utils).NameToInternalName(..ModifiedFiles(i),,0) 
             if (internalName = "") {
                 write files(i), " was not imported into the database and will not be compiled. ", !
             } else {

cls/SourceControl/Git/Utils.cls

@@ -310,7 +310,7 @@ ClassMethod Pull(remote As %String = "origin") As %Status
     merge event.ModifiedFiles = files
     for i=1:1:$Get(files) {
         if $Data(files(i))#2 {
-            set internalName = ##class(SourceControl.Git.Utils).NameToInternalName(event.ModifiedFiles(i)) 
+            set internalName = ##class(SourceControl.Git.Utils).NameToInternalName(event.ModifiedFiles(i),,0) 
             if (internalName '= "") {
                 set event.ModifiedItems(internalName) = ""
             }
@@ -1310,7 +1310,7 @@ ClassMethod NameToInternalName(Name, IgnorePercent = 1, IgnoreNonexistent = 1) A
 	}
 	If ((IgnorePercent)&&($extract(InternalName)="%")) { Set InternalName = "" }	; don't return a result for % items if instructed to ignore them
 	If ((IgnoreNonexistent)&&('##class(%RoutineMgr).Exists(InternalName))) { Set InternalName = "" }	; only return item names which exist in the DB
-	Quit ..NormalizeInternalName(InternalName)
+    Quit ..NormalizeInternalName(InternalName)
 }
 
 ClassMethod OutputConfigureMessage()