[ http://issues.apache.org/jira/browse/COCOON-1692?page=all ]

Antonio Gallardo updated COCOON-1692:
-------------------------------------


In the privided patch, I guess there is a typo in:

@@ -259,9 +260,8 @@

             if (this.expandSelectedPath) {
                 expandPath(path);
             }
-            if (this.selectionListener != null) {
-                this.selectionListener.selectionChanged(new 
TreeSelectionEvent(this, path, true));
-            }
+            
+            this.getForm().addWidgetEvent(new TreeSelectionEvent(this, path, 
false));
         }
     }

I suspect it should be:

+            this.getForm().addWidgetEvent(new TreeSelectionEvent(this, path, 
true));

(I just changed te last paramter to true).

Would you confirm if this is correct?

> [PATCH] Tree widget not handling on-selection-change events correctly.
> ----------------------------------------------------------------------
>
>                 Key: COCOON-1692
>                 URL: http://issues.apache.org/jira/browse/COCOON-1692
>             Project: Cocoon
>          Issue Type: Bug
>          Components: Blocks: Forms
>    Affects Versions: 2.1.9, 2.1.8
>            Reporter: Suzan Foster
>         Assigned To: Antonio Gallardo
>         Attachments: Tree.java.patch
>
>
> In the current implementation field widgets that have their value changed 
> from an on-selection-changed handler do not propagate these changes to the 
> client. The patch, applied to current SVN, resolves this issue.

-- 
This message is automatically generated by JIRA.
-
If you think it was sent incorrectly contact one of the administrators: 
http://issues.apache.org/jira/secure/Administrators.jspa
-
For more information on JIRA, see: http://www.atlassian.com/software/jira

        

Reply via email to