yahoo-eng-team team mailing list archive

[Thierry Carrez <thierry.carrez+lp@xxxxxxxxx>]

** Changed in: horizon
       Status: Fix Committed => Fix Released

** Changed in: horizon
    Milestone: None => juno-1

-- 
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/1308952

Title:
  Traling commas in JS files able to break Horizon on old web browsers.

Status in OpenStack Dashboard (Horizon):
  Fix Released

Bug description:
  In some JS files, there are trailing commas able to break Horizon on
  old web browsers.

  Example (in horizon/static/horizon/js/horizon.d3linechart.js, line
  187):

    domain: function($super) {
      var ret = $super();
      // If y axis wants to have static range, not based on data
      if (this.yMin !== undefined && this.yMax !== undefined){
        ret.y = [this.yMin, this.yMax];
      }
      return ret;
    }, <-- can cause an error " Expected identifier, string or number".
  });

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