Cocoon
  1. Cocoon
  2. COCOON-2178

Array-based constructors of TreeSelectionEvent used.

    Details

    • Type: Improvement Improvement
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 2.1.11
    • Fix Version/s: 2.1.12
    • Component/s: Blocks: Forms
    • Labels:
      None
    • Urgency:
      Normal
    • Other Info:
      Patch available
    • Affects version (Component):
      Blocks: Forms - 1.0.0-RC3-SNAPSHOT
    • Fix version (Component):
      Blocks: Forms - 1.1.0-SNAPSHOT

      Description

      I'm facing a serious problem using the cforms tree.

      We added a new functionality: when a user doubleclicks an item, all subitems are selected as well. The result is that for every selected subitem an event is thrown. The problem, as mentioned in the source code, is that the array based constructor of the TreeSelectionEvent is not used. (Besides some other minor changes).

      I have adapted the org.apache.cocoon.forms.formmodel.tree.Tree class and it works fine. But we'll face a problem after updating, so please tell me how i could contribute my code.
      1. Tree.diff
        2 kB
        Harald Entner

        Activity

        Hide
        Vadim Gritsenko added a comment -
        Harald, this should help you get started:
        http://cocoon.apache.org/2.1/howto/howto-patch.html
        Show
        Vadim Gritsenko added a comment - Harald, this should help you get started: http://cocoon.apache.org/2.1/howto/howto-patch.html
        Hide
        Harald Entner added a comment -
        Unique Diff of the Tree class.

        It's not packaged, as it is only one file.
        Show
        Harald Entner added a comment - Unique Diff of the Tree class. It's not packaged, as it is only one file.
        Hide
        Jörg Heinicke added a comment -
        I applied a slightly different patch making the code more equal to the single selection. I think I didn't reintroduce the multiple events ;)
        Show
        Jörg Heinicke added a comment - I applied a slightly different patch making the code more equal to the single selection. I think I didn't reintroduce the multiple events ;)

          People

          • Assignee:
            Jörg Heinicke
            Reporter:
            Harald Entner
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development