You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@myfaces.apache.org by "Matt Cooper (JIRA)" <de...@myfaces.apache.org> on 2007/10/05 23:35:50 UTC

[jira] Issue Comment Edited: (TRINIDAD-757) partialTriggers Contract Fails if direct parent component is not a NamingContainer

    [ https://issues.apache.org/jira/browse/TRINIDAD-757?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel#action_12532765 ] 

mattcooper edited comment on TRINIDAD-757 at 10/5/07 2:34 PM:
---------------------------------------------------------------

This test page has 4 buttons:
- button 1 is what all of the other buttons' partialTriggers will be pointing to
- button 2's partialTrigger works, has the incorrect number of colons, and has a direct NamingContainer parent
- button 3's partialTrigger fails, has the incorrect number of colons, and has a direct non-NamingContainer parent
- button 4's partialTrigger works, has the correct number of colons, and has a direct non-NamingContainer parent

When you run the page and click on any of the buttons, you will see an error logged to the console like this:

Oct 5, 2007 2:04:50 PM org.apache.myfaces.trinidadinternal.context.RequestContextImpl addPartialTriggerListeners
WARNING: Could not find partial trigger :::button1WrapperA:button1WrapperB:button1 from CoreCommandButton[UIXFacesBeanImpl, id=button3]

This is wrong, Andrew and I feel that button 2 and 3's partialTriggers should fail.

      was (Author: mattcooper):
    This test page has 4 buttons:
- button 1 is what all of the other buttons' partialTriggers will be pointing to
- button 2's partialTrigger works, has the correct number of colons, and has a direct NamingContainer parent
- button 3's partialTrigger fails, has the correct number of colons, and has a direct non-NamingContainer parent
- button 4's partialTrigger works, has the incorrect number of colons, and has a direct non-NamingContainer parent

When you run the page and click on any of the buttons, you will see an error logged to the console like this:

Oct 5, 2007 2:04:50 PM org.apache.myfaces.trinidadinternal.context.RequestContextImpl addPartialTriggerListeners
WARNING: Could not find partial trigger :::button1WrapperA:button1WrapperB:button1 from CoreCommandButton[UIXFacesBeanImpl, id=button3]

This is wrong, button 3's partialTrigger should work.  Instead, the error should be for button 4.
  
> partialTriggers Contract Fails if direct parent component is not a NamingContainer
> ----------------------------------------------------------------------------------
>
>                 Key: TRINIDAD-757
>                 URL: https://issues.apache.org/jira/browse/TRINIDAD-757
>             Project: MyFaces Trinidad
>          Issue Type: Bug
>          Components: Components
>    Affects Versions: 1.0.3-core, 1.2.2-core
>            Reporter: Matt Cooper
>            Assignee: Matt Cooper
>         Attachments: partialTriggers.jspx
>
>
> The definition of partialTriggers is:
> The IDs of the components that should trigger a partial update. This component will listen on the trigger components. If one of the trigger components receives an event that will cause it to update in some way, this component will request to be updated too. Identifiers are relative to the source component, and must account for NamingContainers. If your component is already inside of a naming container, you can use a single colon to start the search from the root, or multiple colons to move up through the NamingContainers - "::" will search from the parent naming container, ":::" will search from the grandparent naming container, etc.
> I am seeing that the multiple-colon use case will work as long as the direct parent component is a NamingContainer.  If the direct parent component is not a NamingContainer then one additional colon is incorrectly required to make it work.  I will upload a test page that demonstrates this shortly.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.