Project

General

Profile

Actions

Bug #24517

closed

Improve ExtDirect error handling

Added by Steffen Gebert almost 14 years ago. Updated over 13 years ago.

Status:
Closed
Priority:
Should have
Category:
-
Target version:
-
Start date:
2011-01-11
Due date:
% Done:

0%

Estimated time:
TYPO3 Version:
PHP Version:
Tags:
Complexity:
Is Regression:
Sprint Focus:

Description

ExtDirect resonds on timeout with

 Error parsing json response: null

This should instead end up with a human-readable error message and, if possible, more details, what went wrong.

Maybe this helps a bit http://svn.geoext.org/ext/3.3.1/docs/source/JsonProvider.html
Had a discussion with Stefan, he's aware of this issue and will think about it.
(issue imported from #M16971)


Files

16971_v1.diff (3.23 KB) 16971_v1.diff Administrator Admin, 2011-01-19 02:20
16971_v2.diff (2.46 KB) 16971_v2.diff Administrator Admin, 2011-01-19 11:08
Actions #2

Updated by Stefan Galinski almost 14 years ago

Hi Steffen,

I implemented a solution that triggers an information flash message with a nice information text. The backtrace information isn't shown in this case. Please review the patch and give me your opinion.

I'am not sure if this patch can be added to 4.5, because it adds two new language labels.

Actions #3

Updated by Steffen Gebert almost 14 years ago

Importance(Error Handling) > Importance(Labels) :-)

I'd suggest to call it "Connection problem"
Could the FlashMessage be red? (it's an error) ;-)

I've also adjusted it to the recent change of outputting LLL labels.

What a pity that the relogin doesn't use ExtDirect - those calls silently fail (before and after the patch).

Actions #4

Updated by Stefan Galinski almost 14 years ago

Thanks for your changes. I added the patch to the core mailinglist. Happy voting! :-)

Actions #5

Updated by Steffen Gebert almost 14 years ago

rev 10174

Actions #6

Updated by Susanne Moog over 13 years ago

  • Target version deleted (4.5.0)
Actions

Also available in: Atom PDF