From 74d235cfab94bceae69e0df1888e52268f728b67 Mon Sep 17 00:00:00 2001
From: Jalinson Diaz <zjaaaldev@gmail.com>
Date: Tue, 20 Aug 2024 19:22:01 -0300
Subject: [PATCH] chore(edit content): Revert Binary Field changes (#29675)

### Proposed Changes
* Leave as it was
---
 .../ext/contentlet/edit_contentlet.jsp        |  2 --
 .../ext/contentlet/edit_contentlet_js_inc.jsp |  5 -----
 .../ext/contentlet/field/edit_field.jsp       | 21 +++++--------------
 3 files changed, 5 insertions(+), 23 deletions(-)

diff --git a/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet.jsp b/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet.jsp
index 8fc2678cd17f..aabb03f620fb 100644
--- a/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet.jsp
+++ b/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet.jsp
@@ -108,8 +108,6 @@
 <% } %>
 
 <script type="text/javascript">
-
-	var binaryFieldCallbacksMap = {} // This is a map to store the callbacks for the binary fields, used to reload them when we publish or save a contentlet, look on dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet_js_inc.jsp on saveContentCallback function
 	const relationsLoadedMap = {};
 
 	function waitForRelation() {
diff --git a/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet_js_inc.jsp b/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet_js_inc.jsp
index 03070448d4e6..457c03623601 100644
--- a/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet_js_inc.jsp
+++ b/dotCMS/src/main/webapp/html/portlet/ext/contentlet/edit_contentlet_js_inc.jsp
@@ -731,11 +731,6 @@
         customEvent.initCustomEvent('ng-event', false, false, customEventDetail);
         document.dispatchEvent(customEvent);
 
-        const binaryCallbacks = Object.keys(window.binaryFieldCallbacksMap || {}).map(key => window.binaryFieldCallbacksMap[key]); // Get the callbacks in an array
-
-        binaryCallbacks.forEach(callback => {
-            callback(); // To reload all binary fields
-        });
     }
 
     function refreshPermissionsTab(){
diff --git a/dotCMS/src/main/webapp/html/portlet/ext/contentlet/field/edit_field.jsp b/dotCMS/src/main/webapp/html/portlet/ext/contentlet/field/edit_field.jsp
index 9b89e4af6144..74ac3e92a89f 100644
--- a/dotCMS/src/main/webapp/html/portlet/ext/contentlet/field/edit_field.jsp
+++ b/dotCMS/src/main/webapp/html/portlet/ext/contentlet/field/edit_field.jsp
@@ -62,8 +62,6 @@
     Object inodeObj =(Object) request.getAttribute("inode");
     String inode = inodeObj != null ? inodeObj.toString() : "";
 
-    String contentletIdentifier = contentlet.getIdentifier();
-
     String counter = (String) request.getAttribute("counter");
 
     boolean fullScreenField = Try.of(()->(boolean)request.getAttribute("DOT_FULL_SCREEN_FIELD")).getOrElse(false);
@@ -115,6 +113,7 @@
         }
     }
 </style>
+
 <div class="fieldWrapper" >
     <div class="fieldName" id="<%=field.getVelocityVarName()%>_tag">
         <% if (hint != null) {%>
@@ -213,6 +212,7 @@
             String textValue = UtilMethods.isSet(value) ? value.toString(): (UtilMethods.isSet(defaultValue) ? defaultValue : "");
             String safeTextValue = "`" + StringEscapeUtils.escapeJavaScript(textValue.replaceAll("`", "&#96;").replaceAll("\\$", "&#36;")) + "`";
 
+            String contentletIdentifier = contentlet.getIdentifier();
             String jsonField = "{}";
             String contentletObj = "{}";
             Boolean showVideoThumbnail = Config.getBooleanProperty("SHOW_VIDEO_THUMBNAIL", true);
@@ -774,7 +774,7 @@
             </script>
             <script>
                 // Create a new scope so that variables defined here can have the same name without being overwritten.
-                function setBinaryFieldInputs() {
+                (function autoexecute() {
                     const binaryFieldContainer = document.getElementById("container-binary-field-<%=field.getVelocityVarName()%>");
 
                     /**
@@ -782,10 +782,7 @@
                      * This is a workaround to get the contentlet from the API
                      * because there is no way to get the same contentlet the AP retreive from the dwr call.
                      */
-
-                    const identifier =  "<%=contentletIdentifier%>";
-
-                    fetch(`/api/v1/content/${identifier}`, {
+                    fetch('/api/v1/content/<%=inode%>', {
                         method: 'GET',
                         headers: {
                             'Content-Type': 'application/json'
@@ -872,15 +869,7 @@
                     .catch(() => {
                         binaryFieldContainer.innerHTMl = '<div class="callOutBox">Error loading the binary field</div>';
                     })
-                };
-
-                setBinaryFieldInputs(); // Initialize the binary Field
-
-
-                // This code covers the multiple binaries in a contentlet
-                if(!binaryFieldCallbacksMap["<%=field.getVelocityVarName()%>"]) {
-                    binaryFieldCallbacksMap["<%=field.getVelocityVarName()%>"] = setBinaryFieldInputs; // Register the function to be called when the contentlet is saved
-                }
+                })();
             </script>
         <%}else{%>