Bug Tracker

Opened 6 years ago

Closed 6 years ago

#13181 closed bug (notabug)

The non-ascii character en dash and em dash and WebLogic Application Server and CVS.

Reported by: pgmpjl Owned by:
Priority: undecided Milestone: None
Component: unfiled Version: git
Keywords: Cc: scott_gonzalez
Blocked by: Blocking:

Description

The current version of JQuery jquery-ui-1.9.2.custom.js has 2 characters which require the file to be UTF-8 encoded JavaScript characters.

The characters are are on line 9563:

en dash or \u2013 em dash or \u2014

These characters appear on 1 line in the file, they could be easily be changed to avoid the problems.

Initialize unlinked menu-items containing spaces and/or dashes only as dividers

menus.children( ":not(.ui-menu-item)" ).each(function() {

var item = $( this ); hyphen, em dash, en dash if ( !/[\-\u2013\u2014\s]/.test( item.text() ) ) {

item.addClass( "ui-widget-content ui-menu-divider" );

}

});

This creates a problem with 2 different systems. Older versions of CVS running on UNIX systems do not have good support for UTF-8 files.

The English version of the WebLogic Application Server chokes on this file. It requires the Application server to be configured for International characters.

Oracle documents http://docs.oracle.com/cd/E13222_01/wls/docs81/webserv/i18n.html

Change History (3)

comment:1 Changed 6 years ago by dmethvin

This belongs in the jQuery UI bug tracker, per the instructions in that big red box over the bug reporting form.

I'll just leave it open here for now and they can move it over.

Scott, if you implement this suggested change be sure to add the Uglify option beautify: { ascii_only: true } during build or Uglify will change it back during compression.

This is the code I'm using to ensure nothing sneaks through: https://github.com/jquery/jquery/blob/2f6b3f818fc51ee6ae44be69dc6b15b3d7a2dad4/Gruntfile.js#L411 . We talked about making it a task a few months back but I'm not sure it's general enough. I added the line ending check just a couple of days ago.

comment:2 Changed 6 years ago by timmywil

Cc: scott_gonzalez added

comment:3 Changed 6 years ago by dmethvin

Resolution: notabug
Status: newclosed
Note: See TracTickets for help on using tickets.