← Back to team overview

mahara-contributors team mailing list archive

[Bug 675564] Re: Error processing in External feed blocktype

 

Hi Ruslan,

I think in the case of feed errors the block config window should remain
open and maybe the timeout reduced, as you suggest.  So feel free to do
whatever you want with f032d6f1d93, including reverting it altogether if
you like, or maybe just removing the part that gets rid of the config
block.

f032d6f1d93 is there because I was doing a bit of a proof of concept for
a fix of bug #547773, but it won't fix that bug anyway with stock
htmlpurifier.  I was testing it with a hacked htmlpurifier which threw
exceptions when it was running too long.  I left it in the code in case
htmlpurifier exceptions do get thrown sometimes, but I haven't actually
seen that happen, so it's probably safe to get rid of it.

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

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.





References