plone/plone.app.jquerytools — pull request #2 by thet (2012-04-30 12:30:48+00:00)
Title: Update to jQueryTools 1.2.7

for the records, when building it i got following jslint output... some errors included:

```
$ ./jqt_checkout_build
--------------------------------------------------------------------------------
FILE: jslint.js
--------------------------------------------------------------------------------
FILE: dateinput/skin1.css
|-> plone: jquery.tools.dateinput.css
--------------------------------------------------------------------------------
FILE: dateinput/dateinput.js
|-> jslint status: ERROR
Lint at line 12 character 14: Expected an identifier and instead saw 'undefined' (a reserved word).
(function($, undefined) {

Lint at line 182 character 28: Missing '()' invoking a constructor.
var date = new Date;

Lint at line 194 character 28: Missing '()' invoking a constructor.
now = new Date,

Lint at line 224 character 26: 'original' is already defined.
var original = input.clone(),

Lint at line 228 character 29: Expected '{' and instead saw 'clone'.
if (conf.value) clone.val(conf.value); // jquery 1.6.2 val(undefined) will clear val()

Lint at line 278 character 57: Expected an assignment or function call and instead saw an expression.
conf.toggle ? self.toggle() : self.show();

Lint at line 300 character 37: Expected an assignment or function call and instead saw an expression.
e || (e = $.Event("api"));

Lint at line 497 character 103: Use '===' to compare with 'undefined'.
var date = integer(month) >= -1 ? new Date(integer(year), integer(month), integer(day == undefined || isNaN(day) ? 1 : day)) :

|-> minified to: dateinput/dateinput.min.js
|-> plone: jquery.tools.dateinput.js
|-> plone: jquery.tools.dateinput.min.js
--------------------------------------------------------------------------------
FILE: overlay/overlay.apple.js
|-> jslint status: OK
|-> minified to: overlay/overlay.apple.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: overlay/overlay.js
|-> jslint status: OK
|-> minified to: overlay/overlay.min.js
|-> plone: jquery.tools.js
|-> plone: jquery.tools.min.js
--------------------------------------------------------------------------------
FILE: rangeinput/rangeinput.js
|-> jslint status: OK
|-> minified to: rangeinput/rangeinput.min.js
|-> plone: jquery.tools.rangeinput.js
|-> plone: jquery.tools.rangeinput.min.js
--------------------------------------------------------------------------------
FILE: scrollable/scrollable.autoscroll.js
|-> jslint status: ERROR
Lint at line 48 character 20: Expected '{' and instead saw 'clearTimeout'.
if (timer) clearTimeout(timer); // reset timeout, especially for onSeek event

Lint at line 73 character 37: Expected an assignment or function call and instead saw an expression.
stopped || api.play();

|-> minified to: scrollable/scrollable.autoscroll.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: scrollable/scrollable.js
|-> jslint status: OK
|-> minified to: scrollable/scrollable.min.js
|-> plone: jquery.tools.js
|-> plone: jquery.tools.min.js
--------------------------------------------------------------------------------
FILE: scrollable/scrollable.navigator.js
|-> jslint status: ERROR
Lint at line 106 character 34: Use '===' to compare with '0'.
if (i % size == 0) addItem(i);

Lint at line 106 character 40: Expected '{' and instead saw 'addItem'.
if (i % size == 0) addItem(i);

Lint at line 125 character 30: Use '===' to compare with '0'.
if (i % size == 0) addItem(i);

Lint at line 125 character 36: Expected '{' and instead saw 'addItem'.
if (i % size == 0) addItem(i);

|-> minified to: scrollable/scrollable.navigator.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: tabs/tabs.js
|-> jslint status: ERROR
Lint at line 111 character 22: Expected '{' and instead saw 'return'.
if (animating) return; // don't allow other animations

Lint at line 117 character 50: Expected an assignment or function call and instead saw an expression.
w || ( w = this.getPanes().eq(0).width() );

|-> minified to: tabs/tabs.min.js
|-> plone: jquery.tools.js
|-> plone: jquery.tools.min.js
--------------------------------------------------------------------------------
FILE: tabs/tabs.slideshow.js
|-> jslint status: ERROR
Lint at line 116 character 38: Expected an assignment or function call and instead saw an expression.
stopped || self.play();

|-> minified to: tabs/tabs.slideshow.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: toolbox/toolbox.expose.js
|-> jslint status: OK
|-> minified to: toolbox/toolbox.expose.min.js
|-> plone: jquery.tools.js
|-> plone: jquery.tools.min.js
--------------------------------------------------------------------------------
FILE: toolbox/toolbox.flashembed.js
|-> jslint status: OK
|-> minified to: toolbox/toolbox.flashembed.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: toolbox/toolbox.history.js
|-> jslint status: ERROR
Lint at line 29 character 57: Script URL.
iframe = $("").attr("src", "javascript:false;").hide().get(0);

|-> minified to: toolbox/toolbox.history.min.js
|-> plone: jquery.tools.js
|-> plone: jquery.tools.min.js
--------------------------------------------------------------------------------
FILE: toolbox/toolbox.mousewheel.js
|-> jslint status: OK
|-> minified to: toolbox/toolbox.mousewheel.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: tooltip/tooltip.dynamic.js
|-> jslint status: OK
|-> minified to: tooltip/tooltip.dynamic.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: tooltip/tooltip.js
|-> jslint status: ERROR
Lint at line 270 character 36: Expected '{' and instead saw 'tip'.
if (!conf.tip) tip.data("__set", true);

|-> minified to: tooltip/tooltip.min.js
|-> plone: jquery.tools.js
|-> plone: jquery.tools.min.js
--------------------------------------------------------------------------------
FILE: tooltip/tooltip.slide.js
|-> jslint status: OK
|-> minified to: tooltip/tooltip.slide.min.js
|-> plone: jquery.tools.plugins.js
|-> plone: jquery.tools.plugins.min.js
--------------------------------------------------------------------------------
FILE: validator/validator.js
|-> jslint status: ERROR
Lint at line 581 character 22: Use '!==' to compare with 'null'.
if ((els != null) && (els.length != 0)) {

Lint at line 581 character 46: Use '!==' to compare with '0'.
if ((els != null) && (els.length != 0)) {

|-> minified to: validator/validator.min.js
|-> plone: jquery.tools.validator.js
|-> plone: jquery.tools.validator.min.js
--------------------------------------------------------------------------------
FILE: README.md
--------------------------------------------------------------------------------
FILE: jquery.form.js
|-> jslint status: ERROR
Lint at line 13 character 1: Unnecessary semicolon.
;(function($) {

Lint at line 14 character 1: Expected an assignment or function call and instead saw an expression.
"use strict";

Lint at line 92 character 65: Script URL.
iframeSrc: /^https/i.test(window.location.href || '') ? 'javascript:false' : 'about:blank'

Lint at line 206 character 43: Line breaking error ')'.
for (var k=0; k < elements.length; k++)

Lint at line 207 character 9: Expected '{' and instead saw 'elements'.
elements[k] = null;

Lint at line 214 character 5: Inner functions should be listed at the top of the outer function.
function fileUploadXhr(a) {

Lint at line 214 character 27: 'fileUploadXhr' was used before it was defined.
function fileUploadXhr(a) {

Lint at line 222 character 44: Line breaking error ')'.
for (var p in options.extraData)

Lint at line 222 character 44: Line breaking error ')'.
for (var p in options.extraData)

Lint at line 223 character 56: Line breaking error ')'.
if (options.extraData.hasOwnProperty(p))

Lint at line 224 character 21: Expected '{' and instead saw 'formdata'.
formdata.append(p, options.extraData[p]);

Lint at line 259 character 26: Line breaking error ')'.
if(beforeSend)

Lint at line 260 character 17: Expected '{' and instead saw 'beforeSend'.
beforeSend.call(o, xhr, options);

Lint at line 266 character 30: 'fileUploadIframe' was used before it was defined.
function fileUploadIframe(a) {

Lint at line 281 character 30: Line breaking error ')'.
if ( useProp )

Lint at line 282 character 21: Expected '{' and instead saw 'el'.
el.prop('disabled', false);

Lint at line 284 character 21: Expected '{' and instead saw 'el'.
el.removeAttr('disabled');

Lint at line 290 character 27: Parens are not needed here.
id = 'jqFormIO' + (new Date().getTime());

Lint at line 294 character 19: Line breaking error ')'.
if (!n)

Lint at line 295 character 18: Expected '{' and instead saw '$io'.
$io.attr('name', id);

Lint at line 297 character 17: Expected '{' and instead saw 'id'.
id = n;

Lint at line 321 character 28: Line breaking error ')'.
if (s.error)

Lint at line 322 character 21: Expected '{' and instead saw 's'.
s.error.call(s.context, xhr, e, status);

Lint at line 323 character 22: Line breaking error ')'.
if (g)

Lint at line 324 character 21: Expected '{' and instead saw '$'.
$.event.trigger("ajaxError", [xhr, s, e]);

Lint at line 325 character 31: Line breaking error ')'.
if (s.complete)

Lint at line 326 character 21: Expected '{' and instead saw 's'.
s.complete.call(s.context, xhr, e);

Lint at line 411 character 72: Line breaking error ')'.
if (state && state.toLowerCase() == 'uninitialized')

Lint at line 412 character 25: Expected '{' and instead saw 'setTimeout'.
setTimeout(checkState,50);

Lint at line 417 character 38: Line breaking error ')'.
if (timeoutHandle)

Lint at line 418 character 25: Expected '{' and instead saw 'clearTimeout'.
clearTimeout(timeoutHandle);

Lint at line 439 character 39: Line breaking error ')'.
if (io.attachEvent)

Lint at line 440 character 25: Expected '{' and instead saw 'io'.
io.attachEvent('onload', cb);

Lint at line 442 character 25: Expected '{' and instead saw 'io'.
io.addEventListener('load', cb, false);

Lint at line 490 character 30: Line breaking error ')'.
if (!timedOut)

Lint at line 491 character 21: Expected '{' and instead saw 'return'.
return;

Lint at line 493 character 31: Line breaking error ')'.
if (io.detachEvent)

Lint at line 494 character 17: Expected '{' and instead saw 'io'.
io.detachEvent('onload', cb);

Lint at line 496 character 17: Expected '{' and instead saw 'io'.
io.removeEventListener('load', cb, false);

Lint at line 523 character 26: Line breaking error ')'.
if (isXml)

Lint at line 524 character 21: Expected '{' and instead saw 's'.
s.dataType = 'xml';

Lint at line 565 character 24: 'e' is already defined.
catch (e) {

Lint at line 570 character 20: 'e' is already defined.
catch (e) {

Lint at line 587 character 30: Line breaking error ')'.
if (s.success)

Lint at line 588 character 21: Expected '{' and instead saw 's'.
s.success.call(s.context, data, 'success', xhr);

Lint at line 589 character 22: Line breaking error ')'.
if (g)

Lint at line 590 character 21: Expected '{' and instead saw '$'.
$.event.trigger("ajaxSuccess", [xhr, s]);

Lint at line 593 character 41: Line breaking error ')'.
if (errMsg === undefined)

Lint at line 594 character 21: Expected '{' and instead saw 'errMsg'.
errMsg = xhr.statusText;

Lint at line 595 character 28: Line breaking error ')'.
if (s.error)

Lint at line 595 character 28: Too many errors. (55% scanned).

/tmp/build--jquery.formQ4XxQY/malsup-form-5904d38-build/jquery.form.js:14: WARNING - Suspicious code. Is there a missing '+' on the previous line?
"use strict";
^

0 error(s), 1 warning(s)
|-> minified to: jquery.form.min.js
|-> plone: jquery.form.js
|-> plone: jquery.form.min.js
```