You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ta...@apache.org on 2023/01/18 12:29:52 UTC

[myfaces] branch main updated: MYFACES-4505

This is an automated email from the ASF dual-hosted git repository.

tandraschko pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/myfaces.git


The following commit(s) were added to refs/heads/main by this push:
     new dfc385521 MYFACES-4505
dfc385521 is described below

commit dfc385521aa3ceccd0bbb56534a52f3ffcf11bf4
Author: Thomas Andraschko <ta...@apache.org>
AuthorDate: Wed Jan 18 13:29:44 2023 +0100

    MYFACES-4505
---
 .../org/apache/myfaces/view/facelets/tag/faces/core/AjaxHandler.java  | 4 +++-
 1 file changed, 3 insertions(+), 1 deletion(-)

diff --git a/impl/src/main/java/org/apache/myfaces/view/facelets/tag/faces/core/AjaxHandler.java b/impl/src/main/java/org/apache/myfaces/view/facelets/tag/faces/core/AjaxHandler.java
index 49770c3f7..73e6a4bcf 100644
--- a/impl/src/main/java/org/apache/myfaces/view/facelets/tag/faces/core/AjaxHandler.java
+++ b/impl/src/main/java/org/apache/myfaces/view/facelets/tag/faces/core/AjaxHandler.java
@@ -384,6 +384,7 @@ public class AjaxHandler extends TagHandler implements
             {
                 String separatorChar = String.valueOf(getSeparatorChar(context));
 
+                // execute is required
                 Collection<String> execute = ajaxBehavior.getExecute();              
                 if (execute.isEmpty() || execute.contains("@this"))
                 {
@@ -396,8 +397,9 @@ public class AjaxHandler extends TagHandler implements
                     ajaxBehavior.setExecute(newExecute);
                 }
 
+                // render is optional
                 Collection<String> render = ajaxBehavior.getRender();              
-                if (render.isEmpty() || render.contains("@this"))
+                if (render.contains("@this"))
                 {
                     Collection<String> newRender = new ArrayList<>(render);
                     newRender.remove("@this");