Tapestry
  1. Tapestry
  2. TAPESTRY-2220

In some cases, component event requests are incorrectly interpretted as render requests (with a page activation context)

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Blocker Blocker
    • Resolution: Fixed
    • Affects Version/s: 5.0.10
    • Fix Version/s: 5.0.11
    • Component/s: tapestry-core
    • Labels:
      None

      Description

      The PageRenderDispatcher runs first, and it sees a URL like /folder/name.componentid (clearly, an action URL) as page "/folder/index" activation context "name.componentId".

        Activity

        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open In Progress In Progress
        3m 30s 1 Howard M. Lewis Ship 02/Mar/08 20:24
        In Progress In Progress Closed Closed
        6m 51s 1 Howard M. Lewis Ship 02/Mar/08 20:31
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12568584 ] jira [ 12591624 ]
        Mark Thomas made changes -
        Workflow jira [ 12424899 ] Default workflow, editable Closed status [ 12568584 ]
        Howard M. Lewis Ship made changes -
        Status In Progress [ 3 ] Closed [ 6 ]
        Fix Version/s 5.0.11 [ 12312968 ]
        Resolution Fixed [ 1 ]
        Hide
        Howard M. Lewis Ship added a comment -

        Moving the CED before the PRD solves the problem.

        Show
        Howard M. Lewis Ship added a comment - Moving the CED before the PRD solves the problem.
        Howard M. Lewis Ship made changes -
        Field Original Value New Value
        Status Open [ 1 ] In Progress [ 3 ]
        Hide
        Howard M. Lewis Ship added a comment -

        The ComponentEventDispatcher is currently wired to run after the PageRenderDispatcher. Perhaps these should be reversed? CED was rewritten a ways back to be much pickier about what paths it processes, based on a regular expression.

        Show
        Howard M. Lewis Ship added a comment - The ComponentEventDispatcher is currently wired to run after the PageRenderDispatcher. Perhaps these should be reversed? CED was rewritten a ways back to be much pickier about what paths it processes, based on a regular expression.
        Howard M. Lewis Ship created issue -

          People

          • Assignee:
            Howard M. Lewis Ship
            Reporter:
            Howard M. Lewis Ship
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development