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

Portfolio should give more information during errors.

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Open
    • Priority: Major
    • Resolution: Unresolved
    • Affects Version/s: 3.2.3
    • Fix Version/s: None
    • Component/s: Portfolio
    • Labels:
    • Difficulty:
      Easy
    • Affected Branches:
      MOODLE_32_STABLE

      Description

      So No matter what the error is portfolio throws bogus error messages which is of no use at all.

      So I had APIs disabled in Google developer console.

      And this is the error that I get in Moodle -

      Debug info:
      Error code: failedtosendpackage
      Stack trace:

      line 516 of /lib/portfolio/exporter.php: portfolio_export_exception thrown
      line 227 of /lib/portfolio/exporter.php: call to portfolio_exporter->process_stage_send()
      line 233 of /lib/portfolio/exporter.php: call to portfolio_exporter->process_stage()
      line 233 of /lib/portfolio/exporter.php: call to portfolio_exporter->process_stage()
      line 233 of /lib/portfolio/exporter.php: call to portfolio_exporter->process_stage()
      line 233 of /lib/portfolio/exporter.php: call to portfolio_exporter->process_stage()
      line 265 of /portfolio/add.php: call to portfolio_exporter->process_stage()

      Looking futher we have this -
      throw new portfolio_plugin_exception('sendfailed', 'portfolio_gdocs', $file->get_filename());

      Which should be improved to show the actual error.

        Attachments

          Activity

            People

            • Assignee:
              Unassigned
              Reporter:
              ankit_frenz Ankit Agarwal
              Participants:
              Component watchers:
              Amaia Anabitarte, Carlos Escobedo, Sara Arjona (@sarjona), Víctor Déniz Falcón
            • Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

              Dates

              • Created:
                Updated: