You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by we...@apache.org on 2022/12/12 08:14:24 UTC
[myfaces] branch 2.3-next updated: Fix for https://issues.apache.org/jira/browse/MYFACES-4040
This is an automated email from the ASF dual-hosted git repository.
werpu pushed a commit to branch 2.3-next
in repository https://gitbox.apache.org/repos/asf/myfaces.git
The following commit(s) were added to refs/heads/2.3-next by this push:
new b0c0d838b Fix for https://issues.apache.org/jira/browse/MYFACES-4040
b0c0d838b is described below
commit b0c0d838b4dac8f7976785e24169fc04811b9b3f
Author: Werner Punz <we...@apache.org>
AuthorDate: Mon Dec 12 09:14:19 2022 +0100
Fix for https://issues.apache.org/jira/browse/MYFACES-4040
cherry pick from main
---
.../main/javascript/META-INF/resources/myfaces/api/jsf.js | 3 ++-
.../myfaces/renderkit/html/util/AjaxScriptBuilder.java | 13 ++++++++++---
2 files changed, 12 insertions(+), 4 deletions(-)
diff --git a/api/src/main/javascript/META-INF/resources/myfaces/api/jsf.js b/api/src/main/javascript/META-INF/resources/myfaces/api/jsf.js
index e2a6519cb..c06a4e834 100644
--- a/api/src/main/javascript/META-INF/resources/myfaces/api/jsf.js
+++ b/api/src/main/javascript/META-INF/resources/myfaces/api/jsf.js
@@ -469,7 +469,8 @@ if (!myfaces.ab) {
}
if (eventName) {
- options["javax.faces.behavior.event"] = eventName;
+ options["params"] = options.params || {};
+ options.params["jakarta.faces.behavior.event"] = eventName;
}
if (execute) {
options["execute"] = execute;
diff --git a/impl/src/main/java/org/apache/myfaces/renderkit/html/util/AjaxScriptBuilder.java b/impl/src/main/java/org/apache/myfaces/renderkit/html/util/AjaxScriptBuilder.java
index 1ac187fae..6f61701c0 100644
--- a/impl/src/main/java/org/apache/myfaces/renderkit/html/util/AjaxScriptBuilder.java
+++ b/impl/src/main/java/org/apache/myfaces/renderkit/html/util/AjaxScriptBuilder.java
@@ -196,8 +196,11 @@ public class AjaxScriptBuilder
{
appendProperty(sb, "resetValues", resetValues, false);
}
+
if ((params != null && !params.isEmpty()) || (uiParams != null && !uiParams.isEmpty()))
{
+ StringBuilder paramsBuilder = new StringBuilder();
+ paramsBuilder.append('{');
if (params != null && !params.isEmpty())
{
if (params instanceof RandomAccess)
@@ -206,14 +209,14 @@ public class AjaxScriptBuilder
for (int i = 0, size = list.size(); i < size; i++)
{
ClientBehaviorContext.Parameter param = list.get(i);
- appendProperty(sb, param.getName(), param.getValue(), true);
+ appendProperty(paramsBuilder, param.getName(), param.getValue(), true);
}
}
else
{
for (ClientBehaviorContext.Parameter param : params)
{
- appendProperty(sb, param.getName(), param.getValue(), true);
+ appendProperty(paramsBuilder, param.getName(), param.getValue(), true);
}
}
}
@@ -223,9 +226,12 @@ public class AjaxScriptBuilder
for (int i = 0, size = uiParams.size(); i < size; i++)
{
UIParameter param = uiParams.get(i);
- appendProperty(sb, param.getName(), param.getValue(), true);
+ appendProperty(paramsBuilder, param.getName(), param.getValue(), true);
}
}
+ paramsBuilder.append('}');
+ sb.append("params: ");
+ sb.append(paramsBuilder);
}
sb.append('}');
@@ -260,6 +266,7 @@ public class AjaxScriptBuilder
sb.append('\'');
}
+
public static void appendProperty(StringBuilder builder,
String name,
Object value,