ICEpush
  1. ICEpush
  2. PUSH-362

ice.pushid params not being cleaned up and growing in number

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: EE-3.3.0.GA_P02, EE-4.0.0.GA, EE-3.3.0.GA_P03
    • Fix Version/s: 4.1, EE-3.3.0.GA_P04
    • Component/s: Push Library
    • Labels:
      None
    • Environment:
      All
    • Assignee Priority:
      P1
    • Support Case References:
    • Workaround Exists:
      Yes
    • Workaround Description:
      Using a redirect navigation instead of a forward keeps a single ice.pushid param.

      Description

      In this scenario an ace:fileEntry component is included in a template used by two pages. When navigating from one page to another, new ice:pushid params are being added to the listen.icepush.xml calls. The old ones are not cleaned up and new ones are created.

        Activity

        Hide
        Arran Mccullough added a comment -

        Attached test case.

        Steps:

        • Load main.jsf
        • Note the ice.pushid params that are in the listen.icepush.xml calls.
        • Click the commandLink and navigate to page 2.
        • The previous ice.pushid param is still there and a new one is there as well.
        Show
        Arran Mccullough added a comment - Attached test case. Steps: Load main.jsf Note the ice.pushid params that are in the listen.icepush.xml calls. Click the commandLink and navigate to page 2. The previous ice.pushid param is still there and a new one is there as well.
        Hide
        Arran Mccullough added a comment -

        The customer is reporting that backporting fixes from the following JIRA to the EE 3.3.0 code base has resolved the issue for them:

        http://jira.icesoft.org/browse/PUSH-260
        http://jira.icesoft.org/browse/PUSH-269
        http://jira.icesoft.org/browse/PUSH-343

        Show
        Arran Mccullough added a comment - The customer is reporting that backporting fixes from the following JIRA to the EE 3.3.0 code base has resolved the issue for them: http://jira.icesoft.org/browse/PUSH-260 http://jira.icesoft.org/browse/PUSH-269 http://jira.icesoft.org/browse/PUSH-343
        Hide
        Mircea Toma added a comment - - edited

        Cleanup pushID used by the file entry component on iframe unload. Also modified FileEntryFormSubmit listener to keep the generated pushID in the window scope to ensure that subsequent rendering phases will not generate new IDs every time.

        Show
        Mircea Toma added a comment - - edited Cleanup pushID used by the file entry component on iframe unload. Also modified FileEntryFormSubmit listener to keep the generated pushID in the window scope to ensure that subsequent rendering phases will not generate new IDs every time.
        Hide
        Carmen Cristurean added a comment - - edited

        ICEfaces4 trunk/ EE-3.3.0-maintenance r45810: verified with attached test case in Chrome43, FF34; verified showcase > ace:fileEntry.

        Show
        Carmen Cristurean added a comment - - edited ICEfaces4 trunk/ EE-3.3.0-maintenance r45810: verified with attached test case in Chrome43, FF34; verified showcase > ace:fileEntry.
        Hide
        Mircea Toma added a comment -

        Back ported fix to 3.3.0 maintenance branch.

        Show
        Mircea Toma added a comment - Back ported fix to 3.3.0 maintenance branch.
        Hide
        Carmen Cristurean added a comment -

        Verified EE-3.3.0-maintenance branch r45846; icefaces-ee/patches/icefaces-ee-3.3.0_P02_ICE10088/icefaces r45846 with attached test case, Chrome43.

        Show
        Carmen Cristurean added a comment - Verified EE-3.3.0-maintenance branch r45846; icefaces-ee/patches/icefaces-ee-3.3.0_P02_ICE10088/icefaces r45846 with attached test case, Chrome43.
        Hide
        Mircea Toma added a comment -

        Committed fix for Fedex 3.3.0_P03 branch.

        Show
        Mircea Toma added a comment - Committed fix for Fedex 3.3.0_P03 branch.

          People

          • Assignee:
            Mircea Toma
            Reporter:
            Arran Mccullough
          • Votes:
            0 Vote for this issue
            Watchers:
            4 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: