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
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]
