You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by pa...@apache.org on 2022/10/22 02:54:07 UTC

[myfaces] branch main updated: MYFACES-4486: don't update model

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

paulnicolucci 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 7e3ab6d62 MYFACES-4486: don't update model
     new c17b32dd2 Merge pull request #358 from pnicolucci/MYFACES-4486-Faces40
7e3ab6d62 is described below

commit 7e3ab6d6297cdce06b804380b8ab082b283548d1
Author: Paul Nicolucci <pn...@gmail.com>
AuthorDate: Fri Oct 21 14:45:18 2022 -0400

    MYFACES-4486: don't update model
---
 .../myfaces/component/validate/ValidateWholeBeanComponent.java    | 8 +++++++-
 1 file changed, 7 insertions(+), 1 deletion(-)

diff --git a/impl/src/main/java/org/apache/myfaces/component/validate/ValidateWholeBeanComponent.java b/impl/src/main/java/org/apache/myfaces/component/validate/ValidateWholeBeanComponent.java
index 68dc77e53..510b692c1 100644
--- a/impl/src/main/java/org/apache/myfaces/component/validate/ValidateWholeBeanComponent.java
+++ b/impl/src/main/java/org/apache/myfaces/component/validate/ValidateWholeBeanComponent.java
@@ -83,7 +83,13 @@ public class ValidateWholeBeanComponent extends UIInput
     {
         // No-op. It does not make sense to allow a converter to be installed.
     }
-    
+
+    @Override
+    public void updateModel(FacesContext context)
+    {
+        // Do nothing! See the following specification issue: https://github.com/eclipse-ee4j/mojarra/issues/4313
+    }
+
     @JSFProperty
     public String getValidationGroups()
     {