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
    • Pull from Repository:
    • Pull Master Branch:
      MDL-41041-master

      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

            Activity

              People

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

                Dates

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