You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tapestry.apache.org by "Lukasz Jazgar (JIRA)" <ji...@apache.org> on 2009/06/04 17:55:07 UTC

[jira] Reopened: (TAP5-737) Parameter "translate" of TextField is specified as required, but it's realy not required

     [ https://issues.apache.org/jira/browse/TAP5-737?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Lukasz Jazgar reopened TAP5-737:
--------------------------------


It's completly illogical.
Required, but not required.

If you doesn't have to specify value of parameter and component works without it, that parameter is NOT REQUIRED.

> So, the documentation is already clear about required parameters for which defaults are provided, and what those defaults are. 
It's not clear, because it is inconsistent with flag. 

Translate parameter is marked as required. In description there is text "If not specified". I am programmer, exact mind. I have one conclusion from that. This implication will never happen. Somebody made mistake, either in flag (most probably), either in description, writing redundant implication. 

Another point.
If somebody will want to write IDE plugin automaticaly  validating correctness of templates, he will have a problem. From what plugin will get to know, is parameter is required (it has to be written in component tag) or not. Attribute "required" of annotation is meaningless now. Maybe by natural language processing of description? ;) 




> Parameter "translate" of TextField is specified as required, but it's realy not required
> ----------------------------------------------------------------------------------------
>
>                 Key: TAP5-737
>                 URL: https://issues.apache.org/jira/browse/TAP5-737
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.1.0.5
>            Reporter: Lukasz Jazgar
>            Priority: Minor
>
> Components TextField, TextArea and PasswordField have parameter "translate", which is specified by annotation and in documentation as required.
> This parameter should not be required and it isn't really. 
> Even in examples of using TextField parameter "translate" is not used (http://tapestry.apache.org/tapestry5.1/tapestry-core/ref/org/apache/tapestry5/corelib/components/TextField.html).
> Specification in documentation, specification in annotations,  examples and real behaviour should be consistent.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.