PageLoaderProcessor handles embeddedComponentIds case-sensitive -> should be 
case-insensitive
---------------------------------------------------------------------------------------------

                 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]

Reply via email to