#5576 closed bug (fixed)

Opened May 03, 2010 06:39PM UTC

Closed May 18, 2010 12:49AM UTC

Last modified May 20, 2010 11:13AM UTC

invalid increment operand

Reported by: joernroeder Owned by:
Priority: blocker Milestone: 1.8.2
Component: [meta] ui.build Version: 1.8.1
Keywords: Cc:
Blocked by: Blocking:
Description

I get an invalid increment operand error when i minimize the code.

Attachments (0)
Change History (11)

Changed May 03, 2010 06:46PM UTC by joernroeder comment:1

i looked at the code and it looks as if it was already fixed.

you can close this ticked.

Changed May 04, 2010 08:56PM UTC by nyro comment:2

This bug reappears in the 1.8.1 release

The code mentioned here http://dev.jqueryui.com/ticket/3434 is reused in 4 different places in the jQuery UI code.

Just search for "+ ++" and you'll find it.

wrapping the second ++var with parentheses seems to fix the problem.

Changed May 05, 2010 01:43AM UTC by scottgonzalez comment:3

component: ui.tabs → [meta] ui.build
milestone: TBD → 1.9
priority: major → blocker

Changed May 05, 2010 03:10AM UTC by Rwhitbeck comment:4

@joernroeder - you say you are minimizing the code. Can you be specific and let us know exactly what you are doing?

Thanks

Changed May 05, 2010 03:17AM UTC by rdworth comment:5

Fix for Datepicker in 0ad5ac7. Fix still needed for Tabs.

Changed May 05, 2010 08:06AM UTC by nyro comment:6

@Rwhitbeck I experienced the same problem by simply downloading jQuery UI 1.8.1 from the main page. The minified version was already done on the zip.

Changed May 07, 2010 12:28AM UTC by rdworth comment:7

@nyro We haven't been able to reproduce with the version in the zip or either of these versions

http://jquery-ui.googlecode.com/svn/tags/1.8.1/ui/minified/jquery-ui.min.js

http://ajax.googleapis.com/ajax/libs/jqueryui/1.8.1/jquery-ui.min.js

Can you post a sample page or reproduce on http://jsbin.com/ ? Thanks.

Changed May 07, 2010 09:40AM UTC by nyro comment:8

@rdworth: after checking again and some testing, it looks it might be related to the minifier engine I'm using.

I created a demo page with everything explained: http://temp.nyro.com/jqueryUIBug5576/

As I said, it might be related to MoxieCompressor. I didn't test with another tool and I will check for an other engine.

Changed May 07, 2010 09:57AM UTC by nyro comment:9

BTW, if we look here: http://www.crockford.com/javascript/jsmin.html in the Caution section, there is an explanation about the problem.

Changed May 18, 2010 12:49AM UTC by scottgonzalez comment:10

resolution: → fixed
status: new → closed

Tabs fixed in fcbf4c0.

Changed May 20, 2010 11:13AM UTC by rdworth comment:11

milestone: 1.9 → 1.8.2