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

(another) try..catch(Exception $e)

XMLWordPrintable

    • Icon: Improvement Improvement
    • Resolution: Unresolved
    • Icon: Minor Minor
    • None
    • 3.11.3
    • Course
    • MOODLE_311_STABLE

      I'm possibly a bit obsessed but try...catch($e) {...generic error...} is both pointless and masks the actual error for no good reason. 

      This time course/classes/task/course_delete_modules.php lines 65 to 69

      Something went wrong with a module delete for me but (for some reason) I'm being prevented from seeing the underlying error. try...catch should only even be used to trap specific exceptions and then do something to handle them. 

            Unassigned Unassigned
            howardsmiller Howard Miller
            Votes:
            0 Vote for this issue
            Watchers:
            3 Start watching this issue

              Created:
              Updated:

                Error rendering 'clockify-timesheets-time-tracking-reports:timer-sidebar'. Please contact your Jira administrators.