From 0d91a09e1c315c6c4ec883a641dc02962cf42527 Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Rene=CC=81=20Bock?= <bock@salient.de>
Date: Tue, 5 Nov 2024 17:51:49 +0100
Subject: [PATCH] Set the default type only if the property
 "er.extensions.ERXResponseRewriter.javascriptTypeAttribute" is true

---
 .../er/extensions/appserver/ERXRedirect.java        |  9 ++++++++-
 .../components/javascript/ERXJavaScript.java        | 13 ++++++++++++-
 2 files changed, 20 insertions(+), 2 deletions(-)

diff --git a/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXRedirect.java b/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXRedirect.java
index 186e0bdf82d..21a3ad6bf70 100644
--- a/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXRedirect.java
+++ b/Frameworks/Core/ERExtensions/Sources/er/extensions/appserver/ERXRedirect.java
@@ -10,6 +10,7 @@
 
 import er.extensions.appserver.ajax.ERXAjaxApplication;
 import er.extensions.foundation.ERXMutableURL;
+import er.extensions.foundation.ERXProperties;
 
 /**
  * <span class="en">
@@ -364,8 +365,14 @@ else if (_directActionName != null) {
 	
 			if (ERXAjaxApplication.isAjaxRequest(context.request())) {
 				boolean hasUpdateContainer = context.request().stringFormValueForKey(ERXAjaxApplication.KEY_UPDATE_CONTAINER_ID) != null;
+				boolean appendDefaultTypeAttribute = ERXProperties.booleanForKeyWithDefault("er.extensions.ERXResponseRewriter.javascriptTypeAttribute", false);
+				
 				if (hasUpdateContainer) {
-					response.appendContentString("<script type=\"text/javascript\">");
+					if(appendDefaultTypeAttribute) {
+						response.appendContentString("<script type=\"text/javascript\">");
+					} else {
+						response.appendContentString("<script>");
+					}
 				}
 				else {
 					response.setHeader("text/javascript", "Content-Type");
diff --git a/Frameworks/Core/ERExtensions/Sources/er/extensions/components/javascript/ERXJavaScript.java b/Frameworks/Core/ERExtensions/Sources/er/extensions/components/javascript/ERXJavaScript.java
index 885e41046dc..921532d01ce 100644
--- a/Frameworks/Core/ERExtensions/Sources/er/extensions/components/javascript/ERXJavaScript.java
+++ b/Frameworks/Core/ERExtensions/Sources/er/extensions/components/javascript/ERXJavaScript.java
@@ -46,6 +46,7 @@
  * @binding scriptKey if set, the content will get rendered into an external script src
  * @binding hideInComment boolean that specifies if the script content should
  *   be included in HTML comments, true by default of the script tag contains a script
+ * @binding type defaults to <i>"text/javascript"</i>, if property er.extensions.ERXResponseRewriter.javascriptTypeAttribute is <code>true</code>
  *   
  * @property er.extensions.ERXJavaScript.hideInComment sets globally if the script
  *   content should be included within HTML comments, defaults to <code>true</code>
@@ -86,6 +87,7 @@ public WOActionResults performActionNamed(String name) {
 	WOAssociation _scriptKey;
 	WOAssociation _hideInComment;
 	WOAssociation _language;
+	WOAssociation _type;
 
 	public ERXJavaScript(String s, NSDictionary<String, WOAssociation> nsdictionary, WOElement woelement) {
 		super("script", nsdictionary, woelement);
@@ -98,6 +100,7 @@ public ERXJavaScript(String s, NSDictionary<String, WOAssociation> nsdictionary,
 		_hideInComment = _associations.removeObjectForKey("hideInComment");
 		_scriptFramework = _associations.removeObjectForKey("scriptFramework");
 		_framework = _associations.removeObjectForKey("framework");
+		_type = _associations.removeObjectForKey("type");
 		if((_scriptFile != null && _scriptString != null) 
 				|| (_scriptFile != null && (_scriptSource != null || _filename != null)) 
 				|| (_scriptString != null && (_scriptSource != null || _filename != null))) {
@@ -113,8 +116,16 @@ public ERXJavaScript(String s, NSDictionary<String, WOAssociation> nsdictionary,
 
 	@Override
 	public void appendAttributesToResponse(WOResponse woresponse, WOContext wocontext) {
+		boolean appendDefaultTypeAttribute = ERXProperties.booleanForKeyWithDefault("er.extensions.ERXResponseRewriter.javascriptTypeAttribute", false);
+
 		WOComponent wocomponent = wocontext.component();
-		woresponse._appendContentAsciiString(" type=\"text/javascript\"");
+		
+		if(_type != null) {
+			woresponse._appendContentAsciiString(" type=\""+(String)_type.valueInComponent(wocomponent) +"\"");
+		}
+		else if(appendDefaultTypeAttribute) {
+			woresponse._appendContentAsciiString(" type=\"text/javascript\"");
+		}
 		
 		String framework = null;
 		String scriptName = null;