Thank you for reporting this bug. We will make it our priority to review this report.
  1. 7 Oct 2010 3:51 AM #1
    Kask
    Kask is offline
    Ext User
    Join Date
    Aug 2010
    Location
    Sweden, Västerås
    Posts
    85
    Vote Rating
    0
    Kask is on a distinguished road
      0  

    Default [FIXED-405] A trailing comma

    [FIXED-405] A trailing comma


    On line 34712 in the ext-touch-debug-w-comments.js file there is an extra trailing comma sign.

    I know im really picky, but my obfuscator started to complain
    Reply With Quote Reply With Quote
  2. 7 Oct 2010 10:40 AM #2
    BrendanC
    BrendanC is offline
    Sencha - GXT Dev Team BrendanC's Avatar
    Join Date
    Aug 2010
    Posts
    534
    Vote Rating
    3
    BrendanC is on a distinguished road
      0  

    Default


    Thanks for the formatting report.
    Reply With Quote Reply With Quote
  3. 7 Oct 2010 12:52 PM #3
    evant
    evant is offline
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    15,103
    Vote Rating
    97
    evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold
      0  

    Default


    It would help if you specify which version, on 0.96:

    Code:
    var items = this.getLayoutItems(), // <34712
            ln = items.length, i;
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!
    Reply With Quote Reply With Quote
  4. 7 Oct 2010 2:04 PM #4
    Kask
    Kask is offline
    Ext User
    Join Date
    Aug 2010
    Location
    Sweden, Västerås
    Posts
    85
    Vote Rating
    0
    Kask is on a distinguished road
      0  

    Default


    Really sorry about that ...

    Its in 0.96 version, but i forgot that I had modified the file and added some code ...
    The correct linenumber is 34695.

    Code:
        getTarget : function() {
        return this.owner.getTargetEl();
    },
});
    Reply With Quote Reply With Quote
  5. 7 Oct 2010 2:22 PM #5
    evant
    evant is offline
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    15,103
    Vote Rating
    97
    evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold evant is a splendid one to behold
      0  

    Default


    Seems it's already corrected in source control, will be part of the next release, thanks!
    Evan Trimboli
    Sencha Developer
    Twitter - @evantrimboli
    Don't be afraid of the source code!
    Reply With Quote Reply With Quote

Similar Threads

  1. [DUPE] When defining a Store, baseParam and paramName don't have trailing comma
    By devtig in forum Ext Designer: Bugs
    Replies: 1
    Last Post: 21 May 2010, 9:43 AM
  2. [FIXED-138] JSONStore baseParams missing trailing comma in code
    By michael_w_rickman in forum Ext Designer: Bugs
    Replies: 1
    Last Post: 13 May 2010, 9:12 AM
  3. works in ff, not in ie - not trailing comma
    By djfiii in forum Ext 2.x: Help & Discussion
    Replies: 6
    Last Post: 26 Feb 2008, 9:10 AM
  4. Leading comma or Trailing comma: that is the question.
    By jsakalos in forum Community Discussion
    Replies: 19
    Last Post: 1 Dec 2007, 3:48 PM