From 90565d7ae478411bc4a47757fbcba9321f708e65 Mon Sep 17 00:00:00 2001
From: jdotcms <jonathan.sanchez@dotcms.com>
Date: Tue, 21 May 2024 22:15:09 -0600
Subject: [PATCH] #28563 removing template ajax

---
 .../api/v1/template/TemplateImageForm.java    |  21 ++
 .../api/v1/template/TemplateResource.java     |   7 +-
 .../portlets/templates/ajax/TemplateAjax.java | 291 ------------------
 dotCMS/src/main/webapp/WEB-INF/dwr.xml        |   3 -
 .../htmlpage_assets/template_custom_field.vtl |  37 ++-
 5 files changed, 58 insertions(+), 301 deletions(-)
 create mode 100644 dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateImageForm.java
 delete mode 100644 dotCMS/src/main/java/com/dotmarketing/portlets/templates/ajax/TemplateAjax.java

diff --git a/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateImageForm.java b/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateImageForm.java
new file mode 100644
index 000000000000..5489a3f8bcd4
--- /dev/null
+++ b/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateImageForm.java
@@ -0,0 +1,21 @@
+package com.dotcms.rest.api.v1.template;
+
+import com.fasterxml.jackson.annotation.JsonCreator;
+import com.fasterxml.jackson.annotation.JsonProperty;
+
+import java.io.Serializable;
+
+public class TemplateImageForm implements Serializable {
+
+    @JsonProperty("templateId")
+    private String templateId;
+
+    @JsonCreator
+    public TemplateImageForm(@JsonProperty("templateId") final String templateId) {
+        this.templateId = templateId;
+    }
+
+    public String getTemplateId() {
+        return templateId;
+    }
+}
diff --git a/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateResource.java b/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateResource.java
index 0d94cfa4c60e..0f4b80fd2e17 100644
--- a/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateResource.java
+++ b/dotCMS/src/main/java/com/dotcms/rest/api/v1/template/TemplateResource.java
@@ -816,20 +816,21 @@ public final Response delete(@Context final HttpServletRequest  request,
      * @throws DotSecurityException
      * @throws DotDataException
      */
-    @GET
-    @Path("/{templateId}/image")
+    @POST
+    @Path("/image")
     @JSONP
     @NoCache
     @Consumes(MediaType.APPLICATION_JSON)
     @Produces({MediaType.APPLICATION_JSON, "application/javascript"})
     public Map<String, Object> fetchTemplateImage(@Context final HttpServletRequest  httpRequest,
                                                   @Context final HttpServletResponse httpResponse,
-                                                  @PathParam("templateId") final String templateId) throws DotDataException, DotSecurityException {
+                                                  final TemplateImageForm templateImageForm) throws DotDataException, DotSecurityException {
 
         final InitDataObject initData = new WebResource.InitBuilder(webResource)
                 .requestAndResponse(httpRequest, httpResponse).rejectWhenNoUser(true).init();
         final User user     = initData.getUser();
         final PageMode mode = PageMode.get(httpRequest);
+        final String templateId = templateImageForm.getTemplateId();
 
         Logger.debug(this, ()-> "Getting the image working template by id: " + templateId);
         final Template template = this.templateAPI.findWorkingTemplate(templateId, user, mode.respectAnonPerms);
diff --git a/dotCMS/src/main/java/com/dotmarketing/portlets/templates/ajax/TemplateAjax.java b/dotCMS/src/main/java/com/dotmarketing/portlets/templates/ajax/TemplateAjax.java
deleted file mode 100644
index 882258185571..000000000000
--- a/dotCMS/src/main/java/com/dotmarketing/portlets/templates/ajax/TemplateAjax.java
+++ /dev/null
@@ -1,291 +0,0 @@
-package com.dotmarketing.portlets.templates.ajax;
-
-import com.dotmarketing.beans.Host;
-import com.dotmarketing.beans.Identifier;
-import com.dotmarketing.business.APILocator;
-import com.dotmarketing.business.DotStateException;
-import com.dotmarketing.business.FactoryLocator;
-import com.dotmarketing.business.web.UserWebAPI;
-import com.dotmarketing.business.web.WebAPILocator;
-import com.dotmarketing.exception.DotDataException;
-import com.dotmarketing.exception.DotRuntimeException;
-import com.dotmarketing.exception.DotSecurityException;
-import com.dotmarketing.portlets.contentlet.business.HostAPI;
-import com.dotmarketing.portlets.contentlet.model.Contentlet;
-import com.dotmarketing.portlets.templates.business.TemplateAPI;
-import com.dotmarketing.portlets.templates.factories.TemplateFactory;
-import com.dotmarketing.portlets.templates.model.Template;
-import com.dotmarketing.portlets.templates.model.TemplateWrapper;
-import com.dotmarketing.util.InodeUtils;
-import com.dotmarketing.util.Logger;
-import com.dotmarketing.util.RegEX;
-import com.dotmarketing.util.UtilMethods;
-import com.liferay.portal.PortalException;
-import com.liferay.portal.SystemException;
-import com.liferay.portal.language.LanguageException;
-import com.liferay.portal.language.LanguageUtil;
-import com.liferay.portal.model.User;
-import com.dotcms.repackage.org.directwebremoting.WebContextFactory;
-
-import javax.servlet.http.HttpServletRequest;
-import java.util.*;
-
-/**
- * @author David
- */
-public class TemplateAjax {
-
-	UserWebAPI userWebAPI;
-	TemplateAPI templateAPI;
-	HostAPI hostAPI;
-
-	public TemplateAjax () {
-		templateAPI = APILocator.getTemplateAPI();
-		userWebAPI = WebAPILocator.getUserWebAPI();
-		hostAPI = APILocator.getHostAPI();
-	}
-
-	public Map<String, Object> fetchTemplates (Map<String, String> query, Map<String, String> queryOptions, int start, int count,
-			List<String> sort) throws PortalException, SystemException, DotDataException, DotSecurityException {
-
-		HttpServletRequest req = WebContextFactory.get().getHttpServletRequest();
-		User user = userWebAPI.getLoggedInUser(req);
-		boolean respectFrontendRoles = userWebAPI.isLoggedToFrontend(req);
-		if(count<=0)count=10;
-
-		List<Template> fullListTemplates = new ArrayList<>();
-		List<Template> totalTemplates = new ArrayList<>();
-		Host host = hostAPI.find(query.get("hostId"), user, respectFrontendRoles);
-
-		try{
-			String filter = query.get("fullTitle");
-			if(UtilMethods.isSet(filter)){
-				filter = filter.replaceAll("\\*", "");
-				filter = filter.replaceAll("\\?", "");
-			}
-
-			if(UtilMethods.isSet(query.get("hostId"))) {
-			    int startF=start;
-			    int countF=count;
-			    if(start==0){
-                    Template t = new Template();
-                    t.setOwner(user.getUserId());
-                    t.setModUser(user.getUserId());
-                    t.setInode("0");
-                    t.setTitle("--- " + LanguageUtil.get(user, "All-Hosts") +" ---");
-                    t.setIdentifier("0");
-                    fullListTemplates.add(t);
-                    totalTemplates.add(t);
-                    countF=count-1;
-
-					if (UtilMethods.isSet(query.get("templateSelected"))) {
-						final Template templateSelected = templateAPI.findWorkingTemplate(query.get("templateSelected"),
-								APILocator.getUserAPI().getSystemUser(), respectFrontendRoles);
-
-						if (UtilMethods.isSet(templateSelected) && templateSelected.isAnonymous()) {
-							fullListTemplates.add(templateSelected);
-							totalTemplates.add(templateSelected);
-						}
-					}
-				}
-				else {
-					startF=start-1;
-				}
-
-				fullListTemplates.addAll(templateAPI.findTemplatesUserCanUse(user, host.getIdentifier(), filter, true, startF, countF));
-				totalTemplates.addAll(templateAPI.findTemplatesUserCanUse(user, host.getIdentifier(), filter, true, 0, 1000));
-
-			}
-
-			//doesn't currently respect archived
-			if(fullListTemplates.size() == 0){
-				fullListTemplates.addAll(templateAPI.findTemplatesUserCanUse(user,"", filter,true, start, start>0?count:count+1));
-				totalTemplates.addAll(templateAPI.findTemplatesUserCanUse(user,"", filter,true, 0, 1000));
-			}
-
-
-		}catch (Exception e) {
-			throw new DotRuntimeException(e.getMessage(), e);
-		}
-		//Collections.sort(fullListTemplates, new TemplateComparator(baseHostId));
-		Map<String, Object> results = new HashMap<>();
-		List<Map<String, Object>> list = new LinkedList<> ();
-
-		boolean shouldIncludeTemplate = true;
-		String toInclude = queryOptions.get("includeTemplate");
-		for(Template template : fullListTemplates) {
-			Map<String, Object> contMap = buildTemplateMap(template);
-			list.add(contMap);
-		}
-		if(toInclude != null && shouldIncludeTemplate) {
-			Template template = templateAPI.findWorkingTemplate(toInclude, APILocator.getUserAPI().getSystemUser(), false);
-			if(template != null) {
-				list.add(buildTemplateMap(template));
-			}
-		}
-
-		results.put("totalResults", totalTemplates.size());
-
-		results.put("list", list);
-
-		return results;
-	}
-
-	public Map<String, Object> fetchByIdentity(String id) throws DotDataException, DotSecurityException {
-		return buildTemplateMap(templateAPI.findWorkingTemplate(id, userWebAPI.getSystemUser(), false));
-	}
-
-	private Map<String, Object> buildTemplateMap(Template template) throws DotDataException, DotStateException, DotSecurityException {
-		if(template == null) return null;
-		Host parentHost = null;
-		if(template instanceof TemplateWrapper){
-			parentHost = ((TemplateWrapper) template).getHost();
-		}else{
-			try{
-			  parentHost = templateAPI.getTemplateHost(template);
-			}catch(DotDataException e){
-				Logger.warn(this, "Could not find host for template = " + template.getIdentifier());
-			}
-		}
-		final Map<String, Object> templateMap = template.getMap();
-
-		if (template.isAnonymous()){
-			try {
-				final String customLayoutTemplateLabel = LanguageUtil.get("editpage.properties.template.custom.label");
-				templateMap.put("fullTitle", customLayoutTemplateLabel);
-				templateMap.put("htmlTitle", customLayoutTemplateLabel);
-			} catch (LanguageException e) {
-				Logger.error(this.getClass(),
-						"Exception on buildTemplateMap exception message: " + e.getMessage(), e);
-			}
-		} else if(parentHost != null) {
-			templateMap.put("hostName", parentHost.getHostname());
-			templateMap.put("hostId", parentHost.getIdentifier());
-			templateMap.put("fullTitle",   templateMap.get("title") + " (" + parentHost.getHostname() + ")" );
-			templateMap.put("htmlTitle",  "<div>" + templateMap.get("title") + "</div><small>" + parentHost.getHostname() + "</div></small>" );
-		} else {
-			templateMap.put("fullTitle", templateMap.get("title"));
-			templateMap.put("htmlTitle", templateMap.get("title"));
-		}
-		return templateMap;
-	}
-
-
-
-	class TemplateComparator implements Comparator<Template> {
-
-		private String baseHostId;
-
-		public TemplateComparator(String baseHostId) {
-			this.baseHostId = baseHostId;
-		}
-
-		public int compare(Template o1, Template o2) {
-			try {
-				Identifier id1 = APILocator.getIdentifierAPI().find(o1.getIdentifier());
-				if(id1.getHostId() == null) id1 = null;
-				Identifier id2 = APILocator.getIdentifierAPI().find(o2.getIdentifier());
-				if(id2.getHostId() == null) id2 = null;
-				if(id1 != null && id2 != null && id1.getHostId().equals(baseHostId) && id2.getHostId().equals(baseHostId)) {
-					return o1.getTitle().compareTo(o2.getTitle());
-				}
-				if(id1 != null && id1.getHostId().equals(baseHostId)) {
-					return -1;
-				}
-				if(id2 != null && id2.getHostId().equals(baseHostId)) {
-					return 1;
-				}
-				return id1 == null || id2 == null || id1.getHostId().equals(id2.getHostId())?o1.getTitle().compareTo(o2.getTitle()):id1.getHostId().compareTo(id2.getHostId());
-
-			} catch (DotDataException e) {
-
-			}
-			return 0;
-		}
-
-	}
-
-
-
-	private boolean passFilter(Map<String, Object> item, Map<String, String> query) {
-		for(String key : item.keySet()) {
-			if(query.containsKey(key)) {
-				String filter = query.get(key);
-				filter = "^" + filter.replaceAll("\\*", ".*");
-				filter = filter.replaceAll("\\?", ".?");
-				String value = item.get(key).toString();
-				if(!RegEX.contains(value.toLowerCase(), filter.toLowerCase()))
-					return false;
-			}
-		}
-		return true;
-	}
-
-	public Map<String, Object> fetchTemplateImage(String id) throws DotDataException, DotSecurityException {
-		Map<String, Object> toReturn =  new HashMap<>();
-		Template template = null;
-		try{
-		   template = templateAPI.findWorkingTemplate(id, APILocator.getUserAPI().getSystemUser(), false);
-		}catch(DotSecurityException e){
-			Logger.error(this, e.getMessage());
-		}
-		if(template!=null){
-			Identifier imageIdentifier = APILocator.getIdentifierAPI().find(template.getImage());
-			if(UtilMethods.isSet(imageIdentifier.getAssetType()) && imageIdentifier.getAssetType().equals("contentlet")) {
-				Contentlet imageContentlet = TemplateFactory.getImageContentlet(template);
-				if(imageContentlet!=null){
-					toReturn.put("inode", imageContentlet.getInode());
-					toReturn.put("name", imageContentlet.getTitle());
-					toReturn.put("identifier", imageContentlet.getIdentifier());
-					toReturn.put("extension", com.dotmarketing.util.UtilMethods.getFileExtension(imageContentlet.getTitle()));
-				}
-			}
-		}
-		return toReturn;
-	}
-
-	public String checkDependencies(String templateInode) throws DotDataException, DotSecurityException, PortalException, SystemException{
-		HttpServletRequest req = WebContextFactory.get().getHttpServletRequest();
-		User user = userWebAPI.getLoggedInUser(req);
-		boolean respectFrontendRoles = userWebAPI.isLoggedToFrontend(req);
-		return templateAPI.checkDependencies(templateInode, user, respectFrontendRoles);
-	}
-
-    /**
-     * Method that will verify if a given template title is already used by another template
-     *
-     * @param title          template title to verify
-     * @param templateInode  template inode in case we are editing a template, null or empty in case of a new template
-     * @param hostIdentifier current host identifier
-     * @return
-     * @throws DotDataException
-     * @throws SystemException
-     * @throws PortalException
-     * @throws DotSecurityException
-     */
-    public boolean duplicatedTitle ( String title, String templateInode, String hostIdentifier ) throws DotDataException, SystemException, PortalException, DotSecurityException {
-
-        HttpServletRequest req = WebContextFactory.get().getHttpServletRequest();
-        User user = userWebAPI.getLoggedInUser( req );
-        boolean respectFrontendRoles = userWebAPI.isLoggedToFrontend( req );
-
-        //Getting the current host
-        Host host = hostAPI.find( hostIdentifier, user, respectFrontendRoles );
-
-        //The template name must be unique
-        Template foundTemplate = FactoryLocator.getTemplateFactory().findWorkingTemplateByName( title, host );
-        boolean duplicatedTitle = false;
-        if ( foundTemplate != null && InodeUtils.isSet( foundTemplate.getInode() ) ) {
-            if ( !UtilMethods.isSet( templateInode ) ) {
-                duplicatedTitle = true;
-            } else {
-                if ( !foundTemplate.getInode().equals( templateInode ) ) {
-                    duplicatedTitle = true;
-                }
-            }
-        }
-
-        return duplicatedTitle;
-    }
-
-}
\ No newline at end of file
diff --git a/dotCMS/src/main/webapp/WEB-INF/dwr.xml b/dotCMS/src/main/webapp/WEB-INF/dwr.xml
index 7bb7ad5cecee..d85feac1d805 100644
--- a/dotCMS/src/main/webapp/WEB-INF/dwr.xml
+++ b/dotCMS/src/main/webapp/WEB-INF/dwr.xml
@@ -54,9 +54,6 @@
       <param name="class" value="com.dotmarketing.portlets.containers.ajax.ContainerAjaxDrawedTemplate"/>
     </create>
     <!-- END GRAZIANO issue-12-dnd-template -->
-    <create creator="new" javascript="TemplateAjax" scope="application">
-      <param name="class" value="com.dotmarketing.portlets.templates.ajax.TemplateAjax"/>
-    </create>
     <create creator="new" javascript="HTMLPageViewAjax" scope="application">
       <param name="class" value="com.dotmarketing.portlets.htmlpageviews.ajax.HTMLPageViewAjax"/>
     </create>
diff --git a/dotCMS/src/main/webapp/WEB-INF/velocity/static/htmlpage_assets/template_custom_field.vtl b/dotCMS/src/main/webapp/WEB-INF/velocity/static/htmlpage_assets/template_custom_field.vtl
index 93cbe55a234c..3a5851ed72f3 100644
--- a/dotCMS/src/main/webapp/WEB-INF/velocity/static/htmlpage_assets/template_custom_field.vtl
+++ b/dotCMS/src/main/webapp/WEB-INF/velocity/static/htmlpage_assets/template_custom_field.vtl
@@ -5,6 +5,7 @@ dojo.require("dotcms.dojo.data.TemplateReadStore");
 function templateChanged() {
     var templateSel=dijit.byId("templateSel");
     var value=templateSel.get('value');
+    console.log("temlateChanged", value);
     if(value == "0") {
         var store=window.top._templateStore;
         store.hostId="";
@@ -13,7 +14,16 @@ function templateChanged() {
     }
     else if(value) {
         dojo.byId("template").value=value;
-        const response = fetch("/api/v1/templates/" + value + "/image");
+        console.log("templateChanged", value);
+        const response = fetch("/api/v1/templates/image", {
+            method: "POST",
+            headers: {
+                "Content-Type": "application/json"
+            },
+            body: JSON.stringify({
+                templateId: value
+            })
+        });
         getTemplateCallBack(response);
     }
 }
@@ -63,8 +73,17 @@ dojo.ready(function(){
     },"templateHolder");
     
     var val  = dojo.byId("template").value;
-    if (val != undefined && val != "0"){
-        const response = fetch("/api/v1/templates/" + val + "/image");
+    if (val != undefined && val != "" && val != "0"){
+        console.log("dojo.ready1", val);
+        const response = fetch("/api/v1/templates/image", {
+            method: "POST",
+            headers: {
+                "Content-Type": "application/json"
+            },
+            body: JSON.stringify({
+                templateId: val
+            })
+        });
         getTemplateCallBack(response);
     }
 
@@ -95,12 +114,22 @@ dojo.ready(function(){
             var obj = templatesMap[normalizedTemplateName];
             if(obj){
                 var value = obj.identifier;
+                console.log("dojo.ready", value, obj);
                 var fullTitle = obj.fullTitle;
                 // We set the values directly into the components because setting it direcrtly into`templateSel` fires another load operation.
                 dojo.byId("currentTemplateId").value = value;
                 dojo.byId("template").value = value;
                 dijit.byId('templateSel').set("displayedValue", fullTitle);
-                const response = fetch("/api/v1/templates/"+ value + "/image");
+                console.log("onTemplateFetchComplete", value, fullTitle);
+                const response = fetch("/api/v1/templates/image", {
+                    method: "POST",
+                    headers: {
+                        "Content-Type": "application/json"
+                    },
+                    body: JSON.stringify({
+                        templateId: value
+                    })
+                });
                 getTemplateCallBack(response);
             }
         }