refactor: return status instead of boolean on Publish

Author: isc-shuliu

src/cls/IPM/Repo/IPublishService.cls

@@ -1,7 +1,7 @@
 Class %IPM.Repo.IPublishService [ Abstract ]
 {
 
-Method PublishModule(pModule As %IPM.Repo.Remote.ModuleInfo) As %Boolean [ Abstract ]
+Method PublishModule(pModule As %IPM.Repo.Remote.ModuleInfo) As %Status [ Abstract ]
 {
 }
 

src/cls/IPM/Repo/Remote/PublishService.cls

@@ -1,12 +1,17 @@
 Class %IPM.Repo.Remote.PublishService Extends (%IPM.Repo.Remote.PackageService, %IPM.Repo.IPublishService)
 {
 
-Method PublishModule(pModule As %IPM.Repo.Remote.ModuleInfo) As %Boolean
+Method PublishModule(pModule As %IPM.Repo.Remote.ModuleInfo) As %Status
 {
-  Quit ..Publish(pModule)
+  Try {
+    Do ..Publish(pModule)
+  } Catch Ex {
+    Return Ex.AsStatus()
+  }
+  Return $$$OK
 }
 
-Method Publish(pModule As %IPM.Repo.Remote.ModuleInfo) As %Boolean
+Method Publish(pModule As %IPM.Repo.Remote.ModuleInfo)
 {
   If ((..Username="") || (..Password="")) && (..Token="") {
     $$$ThrowStatus($$$ERROR($$$GeneralError, "Publishing module, authorization required."))
@@ -28,7 +33,6 @@ Method Publish(pModule As %IPM.Repo.Remote.ModuleInfo) As %Boolean
       $$$ThrowStatus($$$ERROR($$$GeneralError, tMessage))
     }
   }
-  Return 1
 }
 
 Method CheckUnpublishEnabled(packageName As %String, packageVersion As %String) As %Boolean