[
https://issues.apache.org/jira/browse/TEXEN-17?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12850992#action_12850992
]
Fedor Jutte edited comment on TEXEN-17 at 3/29/10 4:37 PM:
-----------------------------------------------------------
Attached patch that resolves this issue.
The patch lets Generator.fillContextProperties() use
TexenUtils.fillContextProperties(), and TexenUtils.fillContextProperties()
supports creating objects from the properties. This patch also cleans up the
TexenUtils.fillContextProperties() method to make it better readable.
was (Author: fejuto):
Attached patch that resolves this issue.
The patch let's Generator.fillContextProperties() use
TexenUtils.fillContextProperties(), and TexenUtils.fillContextProperties()
supports creating objects from the properties. This patch also cleans up the
TexenUtils.fillContextProperties() method to make it better readable.
> Default properties of Generator are processed differently than the propertie
> file(s) given in the Ant-task
> ----------------------------------------------------------------------------------------------------------
>
> Key: TEXEN-17
> URL: https://issues.apache.org/jira/browse/TEXEN-17
> Project: Texen
> Issue Type: Bug
> Affects Versions: 1.1
> Reporter: Fedor Jutte
> Fix For: 1.1
>
> Attachments: texen17-patch.txt
>
> Original Estimate: 24h
> Remaining Estimate: 24h
>
> The TexenTask uses TexenUtils.populateContext() to populate the context.
> Instead of reusing TexenUtils.populateContext(),
> Generator.fillContextProperties() uses it's own, slightly different,
> algorithm to fill the context with it's default properties.
> Difference:
> TexenUtils.populateContext() provides additional support for entries like
> "license.file.contents = license.txt", but Generator.fillContextProperties()
> doesn't.
> Generator.fillContextProperties() provides additional supports for entries
> like "context.objects.strings=org.apache.velocity.util.StringUtils", but
> TexenUtils.populateContext() doesn't.
> SOLUTION:
> Instead of giving Generator.fillContextProperties() it's own algorithm, it
> should use TexenUtils.populateContext(). And the additional support for
> objects in Generator.fillContextProperties() should be added to
> TexenUtils.populateContext().
--
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]
