Uploaded image for project: 'Moodle'
  1. Moodle
  2. MDL-29519

Plagiarism API cron should be triggered after event queue

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.0, 2.1.1, 2.2
    • Fix Version/s: 2.0.5, 2.1.2
    • Component/s: Libraries
    • Labels:
    • Testing Instructions:
      Hide

      you can only really test this hasn't broken cron.

      a manual run of /admin/cron.php could be run to make sure that cron completes correctly.

      Show
      you can only really test this hasn't broken cron. a manual run of /admin/cron.php could be run to make sure that cron completes correctly.
    • Affected Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE, MOODLE_22_STABLE
    • Fixed Branches:
      MOODLE_20_STABLE, MOODLE_21_STABLE
    • Pull Master Branch:
      master_MDL-29519

      Description

      The Plagiarism plugins use event handlers for some events like assignment created/deleted/updated.

      These events should be handled before the plugins try to use the assignments/modules in those events.

      we need to change the order of the plagiarism cron to happen after the event queue.

        Gliffy Diagrams

          Activity

          Hide
          danmarsden Dan Marsden added a comment -

          NOTE FOR INTEGRATOR
          The master branch/commit will cherry-pick fine into 2.1stable and 2.0stable - happy for merge or cherry-pick

          Show
          danmarsden Dan Marsden added a comment - NOTE FOR INTEGRATOR The master branch/commit will cherry-pick fine into 2.1stable and 2.0stable - happy for merge or cherry-pick
          Hide
          salvetore Michael de Raadt added a comment -

          Thanks for sharing that, Dan.

          Show
          salvetore Michael de Raadt added a comment - Thanks for sharing that, Dan.
          Hide
          samhemelryk Sam Hemelryk added a comment -

          Thanks Dan - this has been integrated now

          Show
          samhemelryk Sam Hemelryk added a comment - Thanks Dan - this has been integrated now
          Hide
          ankit_frenz Ankit Agarwal added a comment -

          Hi,
          The cron completed successfully without any issues.
          Test passed.
          Thanks

          Show
          ankit_frenz Ankit Agarwal added a comment - Hi, The cron completed successfully without any issues. Test passed. Thanks
          Hide
          nebgor Aparup Banerjee added a comment -

          fixes have been rolled merrily up the stream! Thanks everybody!

          Show
          nebgor Aparup Banerjee added a comment - fixes have been rolled merrily up the stream! Thanks everybody!

            People

            • Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

              Dates

              • Created:
                Updated:
                Resolved:
                Fix Release Date:
                10/Oct/11