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

Svein commented on TAP5-2489:
-----------------------------

I think you are focusing on the internal coding of the component. I thought 
that @Parameter.required() was a constraint for the users of the component. 
From the external point of view the parameter is not required, and normally 
should never be set on call to the component. A better way is to override the 
message:<STRING>. Internally the parameter is required, otherwise the button 
will not have any information.

The solution for IntelliJ is to check if the @Parameter.value is set in its 
analysis of the usage of the component.

if (required && value == null) then "Analysis error in IntelliJ" 

If you agree, should I report this to Jetbrains/IntelliJ, or would you like to 
do that?

> Palette component parameter availableLabel and selectedLabel should have 
> required = false 
> ------------------------------------------------------------------------------------------
>
>                 Key: TAP5-2489
>                 URL: https://issues.apache.org/jira/browse/TAP5-2489
>             Project: Tapestry 5
>          Issue Type: Wish
>          Components: tapestry-core
>    Affects Versions: 5.4
>            Reporter: Svein
>            Priority: Minor
>              Labels: component
>
> Having value = "message:core-palette-available-label" and value = 
> "message:core-palette-selected-label" then @Parameter(required = false) is 
> better! My IDE gives me error when selectedLabel or availableLabel is skipped.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to