Thank you for reporting this bug. We will make it our priority to review this report.
  1. #1
    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

  2. #2
    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.

  3. #3
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,008
    Vote Rating
    650
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      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!

  4. #4
    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();
        },
    });

  5. #5
    Sencha - Ext JS Dev Team evant's Avatar
    Join Date
    Apr 2007
    Location
    Sydney, Australia
    Posts
    17,008
    Vote Rating
    650
    evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute evant has a reputation beyond repute

      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!

Similar Threads

  1. 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

Thread Participants: 2