Overview

This report was build with JSLint, the JavaScript Verifier. JSLint is a JavaScript program that looks for problems in other JavaScript programs.

Here is the list of files in this report:

AG.Param.js

Lint at line 17 character 12: 'AG' is not defined.
if (typeof AG === "undefined") {

Lint at line 18 character 12: 'AG' was used before it was defined.
var AG = {};

ajs.clone.js

No problems found

AG.Gadget.js

Lint at line 24 character 17: Bad line breaking before '+'.
+ "");

Lint at line 63 character 30: Use '===' to compare with ''.
if (this == "") {

Lint at line 100 character 26: Be careful when making functions within a loop. Consider putting the function in a closure.
});

Lint at line 108 character 70: Use '===' to compare with '0'.
if (AJS.$("ul.faux-list", this.container).length == 0) {

Lint at line 241 character 41: Expected '{' and instead saw 'that'.
if (that.minimized) that.maximize();

Lint at line 242 character 26: Expected '{' and instead saw 'that'.
else that.minimize();

Lint at line 245 character 45: Expected '{' and instead saw 'that'.
if (that.minimized) that.maximize();

Lint at line 246 character 30: Expected '{' and instead saw 'that'.
else that.minimize();

Lint at line 367 character 39: Expected a conditional expression and instead saw an assignment.
if (pref_name = arguments[i]) {

Lint at line 784 character 25: Bad line breaking before '+'.
+ "either 'default' or 'canvas'");

zparse.js

Lint at line 91 character 43: The Function constructor is eval.
this.functionScript = new Function('$data', this.header + this.functionText + this.footer);

Lint at line 123 character 9: Expected '{' and instead saw 'return'.
return ZParse.clean(this.functionScript.apply(bind, [data]));

Lint at line 125 character 9: Expected '{' and instead saw 'return'.
return ZParse.clean(this.functionScript(data));

Lint at line 145 character 13: Expected '{' and instead saw 'source'.
source =  source.replace(new RegExp(list[i][0],'gi'), list[i][1]);

Lint at line 147 character 13: Expected '{' and instead saw 'source'.
source = source.replace(new RegExp('\\'+list[i],'gi'), '\\'+list[i]);

Lint at line 183 character 17: Expected '{' and instead saw 'delimiter'.
delimiter = i;

Lint at line 185 character 17: Expected '{' and instead saw 'if'.
if(source.indexOf(imp[delimiter].opener) > source.indexOf(imp[i].opener))

Lint at line 186 character 21: Expected '{' and instead saw 'delimiter'.
delimiter = i;

Lint at line 181 character 9: The body of a for in should be wrapped in an if statement to filter unwanted properties from the prototype.
for(var i in imp) {

Lint at line 202 character 14: 'i' is already defined.
for(var i=0; i',

gadgets-dashboard.js

Lint at line 43 character 23: Use '===' to compare with '0'.
if (--pending == 0) {

Lint at line 47 character 6: Be careful when making functions within a loop. Consider putting the function in a closure.
};

AG.Render.js

Lint at line 34 character 69: Unescaped '}'.
var matches = template.match(/(?:\$[\W]*)(\w+)(?:\s*\([^\}]+})/gi);

Lint at line 133 character 18: Unnecessary semicolon.
};

Lint at line 137 character 18: Unnecessary semicolon.
};

AG.DashboardManager.js

Lint at line 14 character 1: 'AG' is not defined.
AG = {}; // set namespace

Lint at line 17 character 1: 'AG' is not defined.
AG.DashboardManager = (function () {

Lint at line 45 character 22: 'AG' is not defined.
descriptor = AG.render.getDescriptor("dashboardMenu", json);

Lint at line 48 character 35: 'AG' is not defined.
dashboard.contents.append(AG.render(descriptor));

Lint at line 57 character 13: 'AG' is not defined.
AG.DashboardManager.editLayout();

Lint at line 139 character 22: 'AG' is not defined.
layout = AG.LayoutManager(descriptor);

Lint at line 142 character 17: Expected 'layouts' to have an indentation of 12 instead of 16.
layouts.push(layout);

Lint at line 144 character 17: Expected 'if' to have an indentation of 12 instead of 16.
if (descriptor.active !== false) {

Lint at line 146 character 21: Expected 'this' to have an indentation of 16 instead of 20.
this.setLayout(layout);

Lint at line 147 character 17: Expected '}' to have an indentation of 12 instead of 16.
}

Lint at line 149 character 17: Expected 'layout' to have an indentation of 12 instead of 16.
layout.init();

Lint at line 179 character 26: 'AG' is not defined.
descriptor = AG.render.getDescriptor("layoutDialog", {

Lint at line 180 character 29: 'AG' is not defined.
layoutType: AG.DashboardManager.getLayout().getLayout()

Lint at line 189 character 27: 'AG' is not defined.
.html(AG.render(descriptor))

Lint at line 190 character 31: 'AG' is not defined.
.addClass(AG.LayoutManager.getLayoutAttrName(descriptor.layoutType));

Lint at line 203 character 24: 'AG' is not defined.
AJS.$.each(AG.LayoutManager.layouts, function () {

Lint at line 206 character 34: 'AG' is not defined.
layoutAttrName = AG.LayoutManager.getLayoutAttrName(layout);

Lint at line 211 character 40: 'AG' is not defined.
var activeLayout = AG.DashboardManager.getLayout();

Lint at line 216 character 42: 'AG' is not defined.
.removeClass(AG.LayoutManager.getLayoutAttrName(activeLayout.getLayout()))

Lint at line 223 character 21: 'AG' is not defined.
AG.Sortable.update();

Lint at line 333 character 38: 'gadgets' is not defined.
AJS.$.aop.after({target: gadgets.IfrGadgetService.prototype, method: "setHeight"}, function () {

Lint at line 340 character 13: 'AG' is not defined.
AG.param.set(options.params);

Lint at line 342 character 13: 'AG' is not defined.
AG.render.ready(function () {

Lint at line 346 character 21: 'AG' is not defined.
AG.Sortable.init();

Lint at line 359 character 24: Missing space after 'if'.
if(options.menu.items && options.menu.items.length > 0) {

Lint at line 374 character 21: 'window' is not defined.
window.setTimeout(function() {

Lint at line 374 character 47: Missing space after 'function'.
window.setTimeout(function() {

Lint at line 375 character 25: 'window' is not defined.
window.location.reload();

Lint at line 381 character 13: 'AG' is not defined.
AG.render.initialize();

atl-browser.js

Lint at line 56 character 20: 'button' is already defined.
var button = $(button);

Lint at line 68 character 20: 'button' is already defined.
var button = $(button);

Lint at line 92 character 20: 'button' is already defined.
var button = $(button);

Lint at line 112 character 48: Use '!==' to compare with ''.
if ($('#add-gadget-url').attr('value') != "") {

Lint at line 190 character 56: Use '!==' to compare with ''.
if (gadget.thumbnailUri && gadget.thumbnailUri != "") {

Lint at line 194 character 54: Use '!==' to compare with ''.
if (gadget.description && gadget.description != "") {

Lint at line 200 character 52: Use '!==' to compare with ''.
if (gadget.authorName && gadget.authorName != "") {

Lint at line 209 character 48: Use '!==' to compare with ''.
if (gadget.titleUri && gadget.titleUri != "") {

Lint at line 217 character 49: Bad line breaking before '+'.
+ gadget.gadgetSpecUri);

Lint at line 317 character 35: Use '===' to compare with ''.
if (searchInput.val() == "") {

Lint at line 372 character 71: ['all'] is better written in dot notation.
mb.page[0].addPanel(AG.param.get("all") + " (" + categoryDivs["all"].children().length + ")", categoryDivs["all"]);

Lint at line 372 character 116: ['all'] is better written in dot notation.
mb.page[0].addPanel(AG.param.get("all") + " (" + categoryDivs["all"].children().length + ")", categoryDivs["all"]);

Lint at line 386 character 22: Use '!==' to compare with 'null'.
if(gadgetUrl != null) {

ajs.parseUri.js

Lint at line 18 character 21: Expected '{' and instead saw 'uri'.
while (i--) uri[o.key[i]] = m[i] || "";

Lint at line 22 character 21: Expected '{' and instead saw 'uri'.
if ($1) uri[o.q.name][$1] = $2;

Lint at line 26 character 6: Unnecessary semicolon.
};

cookiebaseduserprefstore.js

Lint at line 57 character 3: The body of a for in should be wrapped in an if statement to filter unwanted properties from the prototype.
for (var name in gadget.getUserPrefs()) {

cookies.js

Lint at line 88 character 25: Use '===' to compare with '0'.
} else if (opt_maxAge == 0) {

Lint at line 97 character 40: Missing '()' invoking a constructor.
var futureDate = new Date((new Date).getTime() + opt_maxAge * 1000);

Lint at line 168 character 24: Expected a conditional expression and instead saw an assignment.
for (var i = 0; part = parts[i]; i++) {

Lint at line 206 character 26: Use '===' to compare with ''.
return document.cookie == '';

Lint at line 216 character 14: Use '===' to compare with ''.
if (cookie == '') {

AG.LayoutManager.js

Lint at line 61 character 40: Expected '===' and instead saw '=='.
if (request.status == 403 || request.status == 401) {

Lint at line 61 character 65: Expected '===' and instead saw '=='.
if (request.status == 403 || request.status == 401) {

Lint at line 218 character 79: 'jQuery' is not defined.
var that = this, isIE6OrBelow = !!(AJS.$.browser.msie && parseInt(jQuery.browser.version) <= 6);

Lint at line 218 character 70: Missing radix parameter.
var that = this, isIE6OrBelow = !!(AJS.$.browser.msie && parseInt(jQuery.browser.version) <= 6);

Lint at line 281 character 33: Unexpected space after 'hideGadget'.
function hideGadget () {

Lint at line 287 character 33: Unexpected space after 'showGadget'.
function showGadget () {

Lint at line 294 character 34: Unexpected space after 'validateAdd'.
function validateAdd (numGadgets) {

Lint at line 295 character 21: Missing radix parameter.
if (parseInt(numGadgets) >= AG.param.get("maxGadgets")) {

Lint at line 303 character 37: Unexpected space after 'appendToColumn'.
function appendToColumn () {

Lint at line 312 character 44: Unexpected space after 'isFromDifferentLayout'.
function isFromDifferentLayout () {

Lint at line 316 character 46: Unexpected space after 'ensureIframeDoesntCache'.
function ensureIframeDoesntCache () {

Lint at line 348 character 14: Unnecessary semicolon.
};

Lint at line 453 character 52: Unexpected space after 'getCanvasGadgetRepresentation'.
function getCanvasGadgetRepresentation (gadgets) {

Lint at line 455 character 18: Expected 'AJS' to have an indentation of 16 instead of 17.
AJS.$.each(that.options.gadgets, function () {

Lint at line 486 character 32: Unexpected space after 'appendTab'.
function appendTab () {

Lint at line 508 character 32: Missing radix parameter.
parseInt(labelStrong.css('padding-left')) -

Lint at line 509 character 32: Missing radix parameter.
parseInt(labelStrong.css('padding-right'));

Lint at line 514 character 20: Missing space after 'if'.
if(that.isWritable() && that.options.active === false) {

Lint at line 521 character 36: Unexpected space after 'appendColumns'.
function appendColumns () {

Lint at line 529 character 36: Unexpected space after 'appendGadgets'.
function appendGadgets () {

Lint at line 535 character 37: Unexpected space after 'setInitialized'.
function setInitialized () {

AG.Sortable.js

No problems found

AG.Cookie.js

Lint at line 14 character 24: Use '===' to compare with 'null'.
if(cookieValue == null) {

Lint at line 25 character 34: Use '===' to compare with '0'.
if (cp.indexOf(name) == 0) {

Lint at line 37 character 24: Use '===' to compare with 'null'.
if(cookieValue == null) {

Lint at line 46 character 19: Use '!==' to compare with ''.
if(cp != "") {

Lint at line 51 character 38: Use '!==' to compare with '0'.
if (cp.indexOf(name) != 0) {

Lint at line 58 character 18: Use '!==' to compare with 'null'.
if(value != null && value != '') {

Lint at line 58 character 35: Use '!==' to compare with ''.
if(value != null && value != '') {

Lint at line 77 character 31: Use '===' to compare with '0'.
if (c.indexOf(eq) == 0) {

Lint at line 107 character 22: Use '!==' to compare with 'null'.
if(value != null) {

jquery.selection.js

Lint at line 12 character 23: Use '===' to compare with 'null'.
if (value == null) {

Lint at line 37 character 23: Use '===' to compare with 'null'.
if (start == null) {

Lint at line 51 character 23: Use '===' to compare with 'null'.
if (value == null) {

Lint at line 70 character 23: Use '===' to compare with 'null'.
if (start == null) {

jquery.cookie.js

Lint at line 83 character 48: Use '!==' to compare with ''.
if (document.cookie && document.cookie != '') {

json.js

No problems found