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

Exception raised when processing portfolio exports via cron

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Reopened
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 2.4.7, 2.5.3
    • Fix Version/s: BACKEND
    • Component/s: Portfolio
    • Labels:
    • Testing Instructions:
      Hide
      1. Set up the portfolio Flickr
      2. Set portfolio_moderate_filesize_threshold and portfolio_high_filesize_threshold to 10kb
      3. Create a forum post with an image (more than 10kb)
      4. Export the image to Flickr portfolio and choose Don't wait on the export page
      5. Run the Cron and make sure the file was exported successfully
      1. Back to the forum post, export the image again
      2. Again, choose "Don't wait"
      3. Navigate to Home / My profile settings / Portfolios / Transfer logs and cancel the export
      4. Run the cron
      5. Make sure you don't see an nasty exception, but you see "Skipping unknown portfolio export with ID X".
      Show
      Set up the portfolio Flickr Set portfolio_moderate_filesize_threshold and portfolio_high_filesize_threshold to 10kb Create a forum post with an image (more than 10kb) Export the image to Flickr portfolio and choose Don't wait on the export page Run the Cron and make sure the file was exported successfully Back to the forum post, export the image again Again, choose "Don't wait" Navigate to Home / My profile settings / Portfolios / Transfer logs and cancel the export Run the cron Make sure you don't see an nasty exception, but you see "Skipping unknown portfolio export with ID X".
    • Affected Branches:
      MOODLE_24_STABLE, MOODLE_25_STABLE
    • Pull from Repository:
    • Pull Master Branch:
      MDL-42802-master
    • Pull Master Diff URL:

      Description

      I'm still working on reproducing this bug manually, but when a portfolio has been set to be exporter during cron, and it doesn't exist (I suppose because the user deleted it), an exception is raised and the cron stops.

      ++ Handler function of component moodle: s:22:"portfolio_handle_event"; threw exception :Invalid export id. Maybe it has expired
      		 
      * line 944 of /lib/portfoliolib.php: call to portfolio_exporter::rewaken_object()
      		 
      * line ? of unknownfile: call to portfolio_handle_event()
      		 
      * line 299 of /lib/eventslib.php: call to call_user_func()
      		 
      * line 343 of /lib/eventslib.php: call to events_dispatch()
      		 
      * line 450 of /lib/eventslib.php: call to events_process_queued_handler()
      		 
      * line 340 of /lib/cronlib.php: call to events_cron()
      		 
      * line 61 of /admin/cli/cron.php: call to cron_run()
      		 
       ++
      		 
      * line 366 of /lib/eventslib.php: call to debugging()
      		 
      * line 450 of /lib/eventslib.php: call to events_process_queued_handler()
      		 
      * line 340 of /lib/cronlib.php: call to events_cron()
      		 
      * line 61 of /admin/cli/cron.php: call to cron_run()

      This is highlighted by the fact that MDL-42542 has been fixed.

        Attachments

          Issue Links

          caused a regression

          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
          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

            Activity

              People

              Assignee:
              Unassigned Unassigned
              Reporter:
              fred Frédéric Massart
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Ferran Recio, Ilya Tregubov, Sara Arjona (@sarjona)
              Votes:
              0 Vote for this issue
              Watchers:
              4 Start watching this issue

                Dates

                Created:
                Updated: