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

async publishing: the polling javascript can get mixed-up and give bad results

    Details

      Description

      Reproduced on Chrome, might happen on other browsers as well:

      1. added to the async publication two sleep (30) calls, just before and after object publication, to simulate a long-publication event
      2. tail -f | grep the apache access log, to see incoming requests
      2. send an object to publish
      3. open 2 more tabs, on the same url: ezflow_site_admin/content/queued/149/14
      4. verify that 3 ajax requests per second come in
      5. what happens at the end: when object was published, tabs 2 and 3 stopped polling, and showed link to published object, while tab 1 did not and kept going on

      Note that I did not reproduce it, but a customer has problems where sometimes the opposite happens: a page stops polling for a time going up to 30 minutes, leading the editor to think that a page is not published when it really is

      1. pic_ok.png
        64 kB
      2. pic.png
        31 kB

        Issue Links

          Activity

          Gaetano Giunta (Inactive) created issue -
          Hide
          Gaetano Giunta (Inactive) added a comment -

          Apart from fixing the cases where the polling stops working because of javascript problems, I think that it is important to give feedback to the editor if the polling fails for any reason, such as e.g. network errors, so that he can try reloading the page or taking other corrective action.

          F.e. showing the timestamp of the last succesfull check done via javascript, or displaying a warning message when polling gets back unexpected responses

          Show
          Gaetano Giunta (Inactive) added a comment - Apart from fixing the cases where the polling stops working because of javascript problems, I think that it is important to give feedback to the editor if the polling fails for any reason, such as e.g. network errors, so that he can try reloading the page or taking other corrective action. F.e. showing the timestamp of the last succesfull check done via javascript, or displaying a warning message when polling gets back unexpected responses
          Damien Pobel (Inactive) made changes -
          Field Original Value New Value
          Status Open [ 1 ] Confirmed [ 10037 ]
          Damien Pobel (Inactive) made changes -
          Status Confirmed [ 10037 ] InputQ [ 10001 ]
          Damien Pobel (Inactive) made changes -
          Status InputQ [ 10001 ] Development [ 3 ]
          Assignee Damien Pobel [ damien.pobel@ez.no ]
          Damien Pobel (Inactive) made changes -
          Affects Version/s 5.2 [ 12582 ]
          Affects Version/s 5.1 [ 11280 ]
          Affects Version/s 5.3 [ 11282 ]
          Affects Version/s 5.0 [ 10300 ]
          Affects Version/s 5.4-dev [ 13485 ]
          Damien Pobel (Inactive) made changes -
          Component/s Administration Interface [ 10785 ]
          Show
          Damien Pobel (Inactive) added a comment - PR: https://github.com/ezsystems/ezpublish-legacy/pull/1048
          Damien Pobel (Inactive) made changes -
          Status Development [ 3 ] Development review [ 10006 ]
          Damien Pobel (Inactive) made changes -
          Remote Link This issue links to "PR (Web Link)" [ 14244 ]
          Damien Pobel (Inactive) made changes -
          Fix Version/s Customer request [ 11018 ]
          Pedro Resende (Inactive) logged work - 06/Aug/14 10:13 AM
          • Time Spent:
            2 hours
             

            Worked on Story

          André Rømcke made changes -
          Assignee Damien Pobel [ damien.pobel@ez.no ] Yannick Roger [ yannick.roger@ez.no ]
          Show
          Yannick Roger (Inactive) added a comment - - edited Fixed in master: https://github.com/ezsystems/ezpublish-legacy/commit/72b677ac7db624dd2635bad3fd4d072b89e81af5 https://github.com/ezsystems/ezpublish-legacy/commit/ce59725504c3a2c83aacc72b6c1105289f80d23e https://github.com/ezsystems/ezpublish-legacy/commit/54892d128ae4b653615b473ca4e32e6a407fd104 https://github.com/ezsystems/ezpublish-legacy/commit/097977ff8324fce5f3769aa8c956dd3527eb2760 https://github.com/ezsystems/ezpublish-legacy/commit/1f54b6251a85c3c85e41ece43f35421de9fc0bdb
          Yannick Roger (Inactive) made changes -
          Status Development review [ 10006 ] Development Review done [ 10028 ]
          Affects Version/s 2014.07 [ 13481 ]
          Fix Version/s 4.7 Maintenance [ 12583 ]
          Fix Version/s 5.0 Maintenance [ 11287 ]
          Fix Version/s 5.1 Maintenance [ 12301 ]
          Fix Version/s 5.2 Maintenance [ 12782 ]
          Fix Version/s 5.4 [ 13180 ]
          Fix Version/s 5.3.3 [ 13484 ]
          Fix Version/s 2014.09 [ 13681 ]
          Yannick Roger (Inactive) made changes -
          Status Development Review done [ 10028 ] Documentation done [ 10011 ]
          Marcos Loureiro (Inactive) made changes -
          Status Documentation done [ 10011 ] QA [ 10008 ]
          Assignee Yannick Roger [ yannick.roger@ez.no ] Marcos Loureiro [ marcos.loureiro@ez.no ]
          Paulo Nunes (Inactive) made changes -
          Rank Ranked higher
          Pedro Resende (Inactive) made changes -
          Assignee Marcos Loureiro [ marcos.loureiro@ez.no ] Pedro Resende [ pedro.resende@ez.no ]
          Pedro Resende (Inactive) logged work - 07/Aug/14 2:00 AM
          • Time Spent:
            3 hours
             

            Worked on Story

          Pedro Resende (Inactive) made changes -
          Rank Ranked lower
          Pedro Resende (Inactive) made changes -
          Rank Ranked lower
          Pedro Resende (Inactive) made changes -
          Attachment pic.png [ 18605 ]
          Pedro Resende (Inactive) made changes -
          Flagged Impediment [ 10000 ]
          Pedro Resende (Inactive) made changes -
          Attachment pic_ok.png [ 18606 ]
          Pedro Resende (Inactive) made changes -
          Flagged Impediment [ 10000 ]
          Hide
          Pedro Resende (Inactive) added a comment -

          Tested and approved by Q.A.

          Show
          Pedro Resende (Inactive) added a comment - Tested and approved by Q.A.
          Pedro Resende (Inactive) made changes -
          Assignee Pedro Resende [ pedro.resende@ez.no ]
          Status QA [ 10008 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Pedro Resende (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Pedro Resende (Inactive) made changes -
          Remaining Estimate 0 minutes [ 0 ]
          Time Spent 2 hours [ 7200 ]
          Worklog Id 49942 [ 49942 ]
          Pedro Resende (Inactive) made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Pedro Resende (Inactive) made changes -
          Resolution Fixed [ 1 ]
          Status Closed [ 6 ] Reopened [ 4 ]
          Pedro Resende (Inactive) made changes -
          Time Spent 2 hours [ 7200 ] 5 hours [ 18000 ]
          Worklog Id 50056 [ 50056 ]
          Pedro Resende (Inactive) made changes -
          Status Reopened [ 4 ] Closed [ 6 ]
          Resolution Fixed [ 1 ]
          Gaetano Giunta (Inactive) made changes -
          Link This issue relates to EZP-19139 [ EZP-19139 ]
          André Rømcke made changes -
          Workflow eZ Engineering Scrumban Workflow [ 64042 ] EZ* Development Workflow [ 85256 ]
          Alex Schuster made changes -
          Workflow EZ* Development Workflow [ 85256 ] EZEE Development Workflow [ 124055 ]
          Transition Time In Source Status Execution Times Last Executer Last Execution Date
          Open Open Confirmed Confirmed
          1d 11h 22m 1 damien.pobel@ez.no 30/Jul/14 12:16 PM
          Confirmed Confirmed InputQ InputQ
          24s 1 damien.pobel@ez.no 30/Jul/14 12:16 PM
          InputQ InputQ Development Development
          2s 1 damien.pobel@ez.no 30/Jul/14 12:16 PM
          Development Development Development Review Development Review
          23h 1m 1 damien.pobel@ez.no 31/Jul/14 11:18 AM
          Development Review Development Review Development Review done Development Review done
          6d 3h 4m 1 yannick.roger@ez.no 06/Aug/14 2:22 PM
          Development Review done Development Review done Documentation Review done Documentation Review done
          46s 1 yannick.roger@ez.no 06/Aug/14 2:23 PM
          Documentation Review done Documentation Review done QA QA
          39m 36s 1 Marcos Loureiro (Inactive) 06/Aug/14 3:03 PM
          QA QA Closed Closed
          23h 51m 1 pedro.resende@ez.no 07/Aug/14 2:54 PM
          Closed Closed Reopened Reopened
          3d 18h 52m 2 pedro.resende@ez.no 11/Aug/14 9:47 AM
          Reopened Reopened Closed Closed
          2s 2 pedro.resende@ez.no 11/Aug/14 9:47 AM

            People

            • Assignee:
              Unassigned
              Reporter:
              Gaetano Giunta (Inactive)
            • Votes:
              0 Vote for this issue
              Watchers:
              6 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:

                Time Tracking

                Estimated:
                Original Estimate - Not Specified
                Not Specified
                Remaining:
                Remaining Estimate - 0 minutes
                0m
                Logged:
                Time Spent - 5 hours
                5h