[
https://issues.apache.org/jira/browse/TAPESTRY-1562?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
]
Howard M. Lewis Ship updated TAPESTRY-1562:
-------------------------------------------
Summary: Spurious warnings about missing component ids due to
case-sensitive checking of template ids vs. declared components (was:
PageLoaderProcessor handles embeddedComponentIds case-sensitive -> should be
case-insensitive)
> Spurious warnings about missing component ids due to case-sensitive checking
> of template ids vs. declared components
> --------------------------------------------------------------------------------------------------------------------
>
> Key: TAPESTRY-1562
> URL: https://issues.apache.org/jira/browse/TAPESTRY-1562
> Project: Tapestry
> Issue Type: Bug
> Components: tapestry-core
> Affects Versions: 5.0.5
> Reporter: Kristian Marinkovic
> Priority: Minor
>
> The PageLoaderProcessor does not handle embedded componentIds as case
> insensitive.
> Therefore logging the error message "embeddedComponentsNotInTemplate".
> (Please see commented source snippet)
> private void loadTemplateForComponent(ComponentPageElement loadingElement) {
> ...
> // TODO: should make embedded ids eg. lowercase
> Set<String> embeddedIds =
> CollectionFactory.newSet(_loadingComponentModel
> .getEmbeddedComponentIds());
> _idAllocator.clear();
> for (String id : template.getComponentIds())
> {
> _idAllocator.allocateId(id);
> // TODO: should make id lowercase before remove
> embeddedIds.remove(id);
> }
> // TODO: because the set is case-sensitive this messages gets logged
> if the component
> // id is written in different cases in template and java class
> if (!embeddedIds.isEmpty())
> log.error(ServicesMessages.embeddedComponentsNotInTemplate(
> embeddedIds,
> componentClassName));
--
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]
