suzanne.soy/zotero
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Make varDump more resilient to property access errors

Browse Source
This commit is contained in:
aurimasv 2012-11-25 05:05:52 -06:00 committed by Simon Kornblith
parent 5f66482537
commit 63f1dd163d
1 changed files with 15 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

21
chrome/content/zotero/xpcom/utilities.js
View File

@ -1092,11 +1092,11 @@ Zotero.Utilities = {
*/
"varDump":function(arr,level,maxLevel,parentObjects,path) {
var dumped_text = "";
if (!level){
if (level === undefined){
level = 0;
}
if (!maxLevel) {
if (maxLevel === undefined) {
maxLevel = 4;
}
@ -1119,7 +1119,12 @@ Zotero.Utilities = {
}
for (var item in arr) {
var value = arr[item];
try {
var value = arr[item];
} catch(e) {
dumped_text += level_padding + "'" + item + "' => <<Access Denied>>\n";
continue;
}
if (typeof(value) == 'object') { // If it is an array
//check for recursion
@ -1139,9 +1144,13 @@ Zotero.Utilities = {
dumped_text += level_padding + "'" + item + "' => " + openBrace;
//only recurse if there's anything in the object, purely cosmetical
for(var i in value) {
dumped_text += "\n" + Zotero.Utilities.varDump(value,level+1,maxLevel,parentObjects.concat([value]),path.concat([item])) + level_padding;
break;
try {
for(var i in value) {
dumped_text += "\n" + Zotero.Utilities.varDump(value,level+1,maxLevel,parentObjects.concat([value]),path.concat([item])) + level_padding;
break;
}
} catch(e) {
dumped_text += "<<Error processing object:\n" + e + ">>\n";
}
dumped_text += closeBrace + "\n";
}