Uploaded image for project: 'eZ Publish / Platform'
  1. eZ Publish / Platform
  2. EZP-23478

ezinfo/about redirected after each form post

    Details

      Description

      After every post in frontend you'll be redirected to ezinfo/about

      Steps to reproduce:

      1- Clean installation
      2- Login into eZ Demo
      3- Access to /ezinfo/about
      4- Click on eZ Logo
      5- Choose "Folder" from the ezwt
      6- Click on "Create here" button
      7- Click on "Send for publishing"
      8- You'll notice you'll be redirected to /ezinfo/about and not the homepage

        Issue Links

          Activity

          Hide
          André Rømcke added a comment - - edited

          This most likely happens on any legacy url.
          I assume the following happens: new stack does not keep track of browsed urls, so last legacy url will be redirected to after legacy edit/action.

          It is more of a Story / Improvement then a bug technically.

          Show
          André Rømcke added a comment - - edited This most likely happens on any legacy url. I assume the following happens: new stack does not keep track of browsed urls, so last legacy url will be redirected to after legacy edit/action. It is more of a Story / Improvement then a bug technically.
          Hide
          André Rømcke added a comment - - edited

          Issue is clear, it's obviously the code in legacy keeping track of which view page you used last*, and use that for redirect after actions. This affects users on 5.3+ where ezwt is exposed in frontend.

          Proper fix: Make sure we always pass uri to redirect to in form values instead, and make sure the legacy parts involved accepts that (most does). This is much more stable, especially when working with several tabs open.

          * This was done in index.php in the past, now moved to the kernel stuff.

          Show
          André Rømcke added a comment - - edited Issue is clear, it's obviously the code in legacy keeping track of which view page you used last*, and use that for redirect after actions. This affects users on 5.3+ where ezwt is exposed in frontend. Proper fix: Make sure we always pass uri to redirect to in form values instead, and make sure the legacy parts involved accepts that (most does). This is much more stable, especially when working with several tabs open. * This was done in index.php in the past, now moved to the kernel stuff.
          Hide
          Bertrand Dunogier added a comment -

          I don't see any existing variable in the wt template for a redirect uri after publishing. But I guess it could be added.

          The only thing is to get the current uri in the website toolbar template, and send it to the legacy template engine.

          Show
          Bertrand Dunogier added a comment - I don't see any existing variable in the wt template for a redirect uri after publishing. But I guess it could be added. The only thing is to get the current uri in the website toolbar template, and send it to the legacy template engine.
          Show
          Bertrand Dunogier added a comment - - edited Pull requests: https://github.com/ezsystems/ezdemo/pull/38 https://github.com/ezsystems/ezpublish-kernel/pull/1102 https://github.com/ezsystems/ezpublish-legacy/pull/1118 https://github.com/ezsystems/ezwt/pull/15
          Hide
          Bertrand Dunogier added a comment -

          All PR are now up-to-date.

          Show
          Bertrand Dunogier added a comment - All PR are now up-to-date.
          Show
          Bertrand Dunogier added a comment - - edited Merged to: ezdemo/master@2131212c ezpublish-legacy/master@31760652 ezpublish-kernel/master@171971c0 ezwt/master@c86ce909
          Hide
          Bertrand Dunogier added a comment -

          Note that there is a doc task here: custom content/edit.tpl overrides must include content/edit_redirection.tpl.

          Show
          Bertrand Dunogier added a comment - Note that there is a doc task here: custom content/edit.tpl overrides must include content/edit_redirection.tpl .
          Hide
          Bertrand Dunogier added a comment -

          Doc: added this paragraph.

          Show
          Bertrand Dunogier added a comment - Doc: added this paragraph .
          Hide
          Rui Silva (Inactive) added a comment -

          Approved by QA.

          Show
          Rui Silva (Inactive) added a comment - Approved by QA.

            People

            • Assignee:
              Unassigned
              Reporter:
              Pedro Resende (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              5 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved: