yahoo-eng-team team mailing list archive

[David Lyle <david.lyle@xxxxxx>]

the version of the django-compressor package you are using is old

see https://bugs.launchpad.net/horizon/+bug/1130610 for more details

** Changed in: horizon
       Status: New => Invalid

-- 
You received this bug notification because you are a member of Yahoo!
Engineering Team, which is subscribed to OpenStack Dashboard (Horizon).
https://bugs.launchpad.net/bugs/1172690

Title:
  jquery-ui-1.9.2.custom.min.js fails when template compiled with
  invalid increment operand

Status in OpenStack Dashboard (Horizon):
  Invalid

Bug description:
  On my system, /opt/stack/horizon/horizon/static/horizon/lib/jquery/jquery-ui-1.9.2.custom.min.js was compiled to /opt/stack/horizon/static/dashboard/js/5f5b4136f892.js 
  The compiled file misses a blank and causes a syntax error in Firefox 20.0's JS engine.

  The original JS segment goes "uniqueId:function(){return this.each(function(){this.id||(this.id="ui-id-"+ ++n)})}"
  and is compiled to: "uniqueId:function(){return this.each(function(){this.id||(this.id="ui-id-"+++n)})}"
  Note the missing blank when incrementing n.
  The triple + raises the syntax error.

  I'm not sure where the blank is removed, but I tested 
  - tag 2013.1
  - branch stable/grizzly
  - and current master
  (though of course resulting in different compiled js file names in 'static/dashboard/js/')
  It's more difficult to locate by browser in current master branch (as it hides js file paths).

  I hope this helps. Maybe other browsers don't have the +++n issue.
  Simply wrapping increment in parentheses would solve the 
  issue.

To manage notifications about this bug go to:
https://bugs.launchpad.net/horizon/+bug/1172690/+subscriptions