[Toaster] [review-request] [PATCH] dreyna/dir_expand_7810
Reyna, David
david.reyna at windriver.com
Wed Aug 19 00:34:21 PDT 2015
Hi Belén,
I have pushed a fix for 7810:
dreyna/dir_expand_7810
It is very strange. The problem was that the AJAX call in "dirinfo.html" expects the response data to be a JSON-formatted string, and uses "parseJSON()" to convert that into a JS object for display computation.
However, in debugging the code I found that the JS debugger recognized the "response" to now _already_ be a proper JS object, and the observed failure in the defect is that "parseJSON()" tries to convert it and immediately hits a syntax error.
The fix is to simply skip the "parseJSON()", and now it all works. But I do not know why the AJAX operation changed, unless it was updated to take the JSON format indicator and perform the conversion automatically now, which would explain why it works the way it does.
In any case, I have kept a note in the the patch indicated how "parseJSON()" is no longer needed, in case someone in the future puzzles over this.
- David
More information about the toaster
mailing list