Cocoon
  1. Cocoon
  2. COCOON-1786

BrowserUpdateTransformer can get into invalid state - must override recycle()

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.8, 2.1.9
    • Fix Version/s: None
    • Component/s: Blocks: Ajax
    • Labels:
      None
    • Urgency:
      Urgent
    • Other Info:
      Patch available

      Description

      If a form throws an exception during transformation (see https://issues.apache.org/jira/browse/COCOON-1785) then the BrowserUpdateTransformer gets into an invalid state, and futher request by any user or session that happens to use the invalid transformer receive the entire form document inside of the bu:document tag. The client side ajax javascript is then unable to process the resulting update.

      The attached patch (license granted to ASF) overrides recycle() and fixes this problem.

        Activity

        Eric Meyer created issue -
        Antonio Gallardo made changes -
        Field Original Value New Value
        Assignee Antonio Gallardo [ antonio ]
        Hide
        Antonio Gallardo added a comment -
        Thanks for the patch!
        The patch was applied. Please cross check and close the bug.
        Show
        Antonio Gallardo added a comment - Thanks for the patch! The patch was applied. Please cross check and close the bug.
        Antonio Gallardo made changes -
        Status Open [ 1 ] On Hold [ 10000 ]
        Hide
        Eric Meyer added a comment -
        Upon reflection, this is more a task for the setup() method. Recycle should deal with releasing references. I've created a new patch that moves the initialization to setup(). I added nulling the reference to the Locator to the recycle() method.
        Show
        Eric Meyer added a comment - Upon reflection, this is more a task for the setup() method. Recycle should deal with releasing references. I've created a new patch that moves the initialization to setup(). I added nulling the reference to the Locator to the recycle() method.
        Eric Meyer made changes -
        Hide
        Antonio Gallardo added a comment -
        Improved Second patch applied. Would you cross check the patch and close the issue?
        Show
        Antonio Gallardo added a comment - Improved Second patch applied. Would you cross check the patch and close the issue?
        Eric Meyer made changes -
        Resolution Fixed [ 1 ]
        Status On Hold [ 10000 ] Closed [ 6 ]
        Transition Time In Source Status Execution Times Last Executer Last Execution Date
        Open Open Blocked Blocked
        51m 45s 1 Antonio Gallardo 01/Mar/06 14:02
        Blocked Blocked Closed Closed
        14h 40m 1 Eric Meyer 02/Mar/06 04:42

          People

          • Assignee:
            Antonio Gallardo
            Reporter:
            Eric Meyer
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development