Move event details to service implementation

Author: BryanMLima

api/src/main/java/org/apache/cloudstack/api/command/user/gui/themes/CreateGuiThemeCmd.java

@@ -111,8 +111,6 @@ public Boolean getRecursiveDomains() {
 
     @Override
     public void execute() {
-        CallContext.current().setEventDetails(String.format("Name: %s, AccountIDs: %s, DomainIDs: %s, RecursiveDomains: %s, CommonNames: %s", getName(), getAccountIds(),
-                getDomainIds(), getRecursiveDomains(), getCommonNames()));
         GuiThemeJoinVO guiTheme = guiThemeService.createGuiTheme(this);
 
         if (guiTheme == null) {

api/src/main/java/org/apache/cloudstack/api/command/user/gui/themes/RemoveGuiThemeCmd.java

@@ -46,7 +46,6 @@ public Long getId() {
 
     @Override
     public void execute() {
-        CallContext.current().setEventDetails(String.format("ID: %s", getId()));
         guiThemeService.removeGuiTheme(this);
         final SuccessResponse response = new SuccessResponse();
         response.setResponseName(getCommandName());

api/src/main/java/org/apache/cloudstack/api/command/user/gui/themes/UpdateGuiThemeCmd.java

@@ -118,8 +118,6 @@ public Boolean getIsPublic() {
 
     @Override
     public void execute() {
-        CallContext.current().setEventDetails(String.format("ID: %s, Name: %s, AccountIDs: %s, DomainIDs: %s, RecursiveDomains: %s, CommonNames: %s", getId(), getName(),
-                getAccountIds(), getDomainIds(), getRecursiveDomains(), getCommonNames()));
         GuiThemeJoinVO guiThemeJoinVO = guiThemeService.updateGuiTheme(this);
 
         if (guiThemeJoinVO == null) {

server/src/main/java/org/apache/cloudstack/gui/theme/GuiThemeServiceImpl.java

@@ -142,6 +142,9 @@ public GuiThemeJoinVO createGuiTheme(CreateGuiThemeCmd cmd) {
         boolean isPublic = cmd.getPublic();
         Boolean recursiveDomains = cmd.getRecursiveDomains();
 
+        CallContext.current().setEventDetails(String.format("Name: %s, AccountIDs: %s, DomainIDs: %s, RecursiveDomains: %s, CommonNames: %s", name, providedAccountIds,
+                providedDomainIds, recursiveDomains, commonNames));
+
         if (StringUtils.isAllBlank(css, jsonConfiguration)) {
             throw new CloudRuntimeException("Either the `css` or `jsonConfiguration` parameter must be informed.");
         }
@@ -369,6 +372,9 @@ public GuiThemeJoinVO updateGuiTheme(UpdateGuiThemeCmd cmd) {
         Boolean isPublic = cmd.getIsPublic();
         Boolean recursiveDomains = cmd.getRecursiveDomains();
 
+        CallContext.current().setEventDetails(String.format("ID: %s, Name: %s, AccountIDs: %s, DomainIDs: %s, RecursiveDomains: %s, CommonNames: %s", guiThemeId, name,
+                providedAccountIds, providedDomainIds, recursiveDomains, commonNames));
+
         validateParameters(jsonConfiguration, providedDomainIds, providedAccountIds, commonNames, guiThemeId);
 
         if (shouldSetGuiThemeToPrivate(providedDomainIds, providedAccountIds)) {
@@ -429,6 +435,7 @@ protected String ifBlankReturnNull(String value) {
     public void removeGuiTheme(RemoveGuiThemeCmd cmd) {
         Long guiThemeId = cmd.getId();
         GuiThemeVO guiThemeVO = guiThemeDao.findById(guiThemeId);
+        CallContext.current().setEventDetails(String.format("ID: %s", guiThemeId));
 
         if (guiThemeVO != null) {
             guiThemeDao.remove(guiThemeId);