[ 
https://issues.apache.org/jira/browse/MYFACES-4656?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17828445#comment-17828445
 ] 

Thomas Andraschko edited comment on MYFACES-4656 at 3/19/24 4:53 PM:
---------------------------------------------------------------------

TBH i think your case is just invalid
the base setRequired uses the enum as key, your overwritten getRequired a 
string as key
this just cant match - getter and setter need to be in sync

if you add

{code:java}
    @Override
    public void setRequired(boolean required)
    {
        getStateHelper().put("required", required );
    }
{code}


its working fine


was (Author: tandraschko):
TBH i think your case is just invalid
the base setRequired uses the enum as key, your overwritten as string as key
this just cant match

if you add

{code:java}
    @Override
    public void setRequired(boolean required)
    {
        getStateHelper().put("required", required );
    }
{code}


its working fine

> Component with overriden isRequired method does not evaluate plain string 
> value
> -------------------------------------------------------------------------------
>
>                 Key: MYFACES-4656
>                 URL: https://issues.apache.org/jira/browse/MYFACES-4656
>             Project: MyFaces Core
>          Issue Type: Bug
>    Affects Versions: 4.0.2
>            Reporter: chaloma1
>            Priority: Minor
>             Fix For: 5.0.0, 4.1.0-RC2, 4.0.3
>
>         Attachments: primefaces-test-git.zip
>
>
> Component with overriden isRequired() method using "required" instead 
> PropertyKeys.required string as in UIInput parent class (you can add default 
> value here if attribute is not found or set). Works with #\{true} or 
> #\{xxxController.isComponentRequired} {*}but not with plain string like 
> "true"{*}.
> See ThirdComponent in attached example. You can execute the sample with mvn 
> jetty:run -Pmyfaces40 command and hit [http://localhost:8080/primefaces-test] 
> to run the page.



--
This message was sent by Atlassian Jira
(v8.20.10#820010)

Reply via email to