Details

    • Type: Sub-task Sub-task
    • Status: Closed
    • Priority: High High
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: Customer request
    • Component/s: Legacy stack
    • Labels:
      None

      Description

      During the autosave, the status checks are performed (https://github.com/ezsystems/ezautosave/blob/master/classes/ezjscserverfunctionsautosave.php#L107) before the transaction begins (https://github.com/ezsystems/ezautosave/blob/master/classes/ezjscserverfunctionsautosave.php#L205). This means that if this version is published in the meantime (or archived, basically any status change) then during the transaction the status will be set back to draft, which leads to inconsistencies (no published version).

        Activity

        Hide
        Gunnstein Lye added a comment -

        Moving to review. It is not fully complete, meaning we're not using select-for-update for all queries. That would require major rewriting, which we shouldn't do on legacy. Still I hope this improves things.

        Show
        Gunnstein Lye added a comment - Moving to review. It is not fully complete, meaning we're not using select-for-update for all queries. That would require major rewriting, which we shouldn't do on legacy. Still I hope this improves things.
        Show
        Gunnstein Lye added a comment - - edited PR: https://github.com/ezsystems/ezautosave/pull/27 and https://github.com/ezsystems/ezpublish-legacy/pull/1403

          People

          • Assignee:
            Unassigned
            Reporter:
            Gunnstein Lye
          • Votes:
            0 Vote for this issue
            Watchers:
            1 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 - 3 days
              3d