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

Felix Scheffer commented on TAP5-2268:
--------------------------------------

Sorry, I misunderstood the issue.

The following code throws the exception above:

{code}
    @Property
    private TestObject2 row;

    public TestObject2 getRow() {
        return this.row;
    }
{code}

While the following code is also illegal but does not throw an exception:

{code}
    @Property
    private TestObject2 row;

    public int getRow() {
        return 5;
    }
{code}

So you're right...5.4 is affected

> @Property may generate a method that conflicts with an existing one, without 
> error, and with unpredictable results
> ------------------------------------------------------------------------------------------------------------------
>
>                 Key: TAP5-2268
>                 URL: https://issues.apache.org/jira/browse/TAP5-2268
>             Project: Tapestry 5
>          Issue Type: Bug
>          Components: tapestry-core
>    Affects Versions: 5.3.7
>            Reporter: Alexey Chalov
>            Priority: Minor
>         Attachments: 
> 0001-TAP5-2268-add-test-to-verify-that-this-issue-is-no-l.patch
>
>
> Page class can contain 2 conflicting pieces for displaying html table like 
> that:
> @Property(write=false)
> private List<Item> items;
> or
> public GridDataSource getItems() {
>     ....
> }
> This causes no error then these 2 constructs present in page class, but 
> behaviour is unpredictable: either accessor for GridDataSource or property 
> generated accessor can be used when page is generated. Restarting/redeploying 
> application can change this behaviour to any of these 2 choices in 
> unpredictable manner. More oftenly (as I noticed 90% data source accessor is 
> used when table content is generated), but about ~10% is left on generated 
> accessor. Tested under Tomcat 7.
> It would be nice if exception will rise in such situaеions to prevent 
> accidental programming error.



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

Reply via email to