[
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: [EMAIL PROTECTED]
For additional commands, e-mail: [EMAIL PROTECTED]
