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

Replace Legacy events - portfolio_send

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Minor
    • Resolution: Fixed
    • Affects Version/s: 2.6
    • Fix Version/s: 2.6
    • Component/s: Events API
    • Labels:
    • Testing Instructions:
      Hide
      1. Set portfolio_moderate_filesize_threshold and portfolio_high_filesize_threshold to very low limits
      2. Upload a forum posts with a picture larger than the limit you've set above
      3. Export the forum post to box.net, and choose not to wait
      4. Make sure the export has been delayed to a cronjob
        • Check that there is an entry in mdl_portfolio_log and mdl_portfolio_tempdata
        • Go to your profile, then My profile settings /Portfolios /Transfer logs and check you can see an entry there

      We do not want to check if the portfolio can run over cron, because it's broken. Cf. MDL-42542

      Show
      Set portfolio_moderate_filesize_threshold and portfolio_high_filesize_threshold to very low limits Upload a forum posts with a picture larger than the limit you've set above Export the forum post to box.net, and choose not to wait Make sure the export has been delayed to a cronjob Check that there is an entry in mdl_portfolio_log and mdl_portfolio_tempdata Go to your profile, then My profile settings /Portfolios /Transfer logs and check you can see an entry there We do not want to check if the portfolio can run over cron, because it's broken. Cf. MDL-42542
    • Affected Branches:
      MOODLE_26_STABLE
    • Fixed Branches:
      MOODLE_26_STABLE
    • Epic Link:
    • Pull from Repository:
    • Pull Master Branch:
      MDL-41041-master
    • Pull Master Diff URL:

      Description

      Replace the legacy call to the event portfolio_send.
      This is in it's own issue as it requires an update to the code to remove the use of cron in the event handler.
      Event handlers are being replaced with observers and the old code won't work.

      This can be solved by waiting for MDL-25508 to be completed, or remove delayed uploading of portfolio content.

        Attachments

          Issue Links

          has been marked as being related by

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-42542 The portfolio cronjob does not work

          • Blocker - Blocks development and/or testing, prevents Moodle from running (applicable to bugs only)
          • Closed

          Bug - A problem which impairs or prevents Moodle from functioning correctly. MDL-42541 Refactor portfolio queue system to prevent usage of events_trigger()

          • Minor - Minor loss of function where workaround is possible
          • Closed

          Improvement - An enhancement to an existing Moodle feature. MDL-25508 Update portfolio code to use once off tasks rather than events API

          • Major - Major loss of function, incorrect output
          • Open

            Activity

              People

              Assignee:
              fred Frédéric Massart
              Reporter:
              abgreeve Adrian Greeve
              Peer reviewer:
              Ankit Agarwal Ankit Agarwal
              Integrator:
              Dan Poltawski Dan Poltawski
              Tester:
              Dan Poltawski Dan Poltawski
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              2 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Fix Release Date:
                18/Nov/13