tapestry-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ted Steen (JIRA)" <...@tapestry.apache.org>
Subject [jira] Assigned: (TAPESTRY-2315) InjectContainerWorker should use this or other local variable name than "container"
Date Sun, 30 Mar 2008 14:01:24 GMT

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

Ted Steen reassigned TAPESTRY-2315:
-----------------------------------

    Assignee: Ted Steen

> InjectContainerWorker should use this or other local variable name than "container"
> -----------------------------------------------------------------------------------
>
>                 Key: TAPESTRY-2315
>                 URL: https://issues.apache.org/jira/browse/TAPESTRY-2315
>             Project: Tapestry
>          Issue Type: Improvement
>          Components: tapestry-core
>    Affects Versions: 5.0.11
>         Environment: any
>            Reporter: Wolfram Koska
>            Assignee: Ted Steen
>            Priority: Minor
>
> What I did:
> @InjectContainer
> private Field container;
> Resulted in: container == null
> The InjectContainerWorker uses "container" as the name of a temporary local variable,
and the assignment is done to the field name:
>   builder.addln("%s = (%s) container;", fieldName, fieldType);
> This resulted in:
>   container = (org.apache.tapestry.Field) container;
> Thus my instance variable was never assigned to. Should probably be
>   builder.addln("this.%s = (%s) container;", fieldName, fieldType);
> or use any other name for the temporary variable - might not be so uncommon for people
to name the container "container", and can be very confusing.

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


---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tapestry.apache.org
For additional commands, e-mail: dev-help@tapestry.apache.org


Mime
View raw message