mahara-contributors team mailing list archive

[Mahara Bot <1318432@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/3510
Committed: http://gitorious.org/mahara/mahara/commit/2e873d2a5251b905ddd2543ba53f40d19749b28c
Submitter: Aaron Wells (aaronw@xxxxxxxxxxxxxxx)
Branch:    master

commit 2e873d2a5251b905ddd2543ba53f40d19749b28c
Author: Nigel Cunningham <nigelc@xxxxxxxxxxxxxxx>
Date:   Mon Jul 14 16:04:00 2014 +1000

(Bug 1318432) Ensure errorhandler called

In sendjsonrequest, if the request completes but returns an error,
we should invoke the caller's error handling function.

Change-Id: I52ef8ae900796c5db4532fa11c2a152d8a75b838
Signed-off-by: Nigel Cunningham <nigelc@xxxxxxxxxxxxxxx>

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1318432

Title:
  Improve error reporting in Ajax installer/upgrader

Status in Mahara ePortfolio:
  In Progress
Status in Mahara 1.10 series:
  New
Status in Mahara 15.04 series:
  In Progress

Bug description:
  We use an Ajax-based page to install & upgrade Mahara. For each
  upgrade task it:

  - fires off a separate Ajax request to run the task
  - shows a "progress" spinner while it waits for a response
  - and then once it gets a response it updates the page and fires off the next task

  The problem is that this system doesn't handle errors well. If the
  Ajax script errors out, it continues showing the "progress" spinner
  forever. Consequently we get a LOT of error reports where people
  complain that "installation takes forever", when in fact it errored
  out after 30 seconds or whatever and it just never told them.

  We need to update this process so that it actually lets you know when
  it failed, and why.

  The simplest solution, suggested by Robert, is that we put a timeout
  on waiting for the Ajax response. Mahara sets max_execution_time
  itself, so we could:

  1. Make the ajax time out after max_execution_time
  2. Show an error icon
  3. Fire off a second ajax response to a script that gets any error messages from the session and displays those on the page.

  That should be pretty robust against any type of error.

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1318432/+subscriptions