mahara-contributors team mailing list archive

[Ruslan Kabalin <675564@xxxxxxxxxxxxxxxxxx>]

Hi Richard,

I have removed that JS bit only (2a5ec5b7bc). Also decided to remove
formError function, so that error is not shown on background window
(23c3b899a). It looks like we do not use $form->reply or
$form->json_reply with PIEFORM_ERR anywhere in block config related
functions, so the only case when blockConfigError is called is field
error and there is really no point to show general "check the marked
fields" in the main window behind the active one. Correct me if I am
wrong.

** Changed in: mahara
       Status: Confirmed => Fix Committed

-- 
Error processing in External feed blocktype
https://bugs.launchpad.net/bugs/675564
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.

Status in Mahara ePortfolio: Fix Committed

Bug description:
As it was pointed out in https://bugs.launchpad.net/mahara/+bug/675385, "processing form" never ends in some cases.

Brief investigation has shown that even if form processing ever ends with any of feed URL failure, no detailed information about the error apart of standard pieform error header (i.e. "...Please check the marked fields and try again") in the main window (as block config form is closed by that time) is displayed.