Skip to main content

Bug Tracker

Side navigation

#13181 closed bug (notabug)

Opened January 08, 2013 08:15PM UTC

Closed January 26, 2013 07:56PM UTC

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

Attachments (0)
Change History (3)

Changed January 08, 2013 08:45PM UTC by dmethvin comment:1

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.

Changed January 21, 2013 07:25PM UTC by timmywil comment:2

cc: → scott_gonzalez

Changed January 26, 2013 07:56PM UTC by dmethvin comment:3

resolution: → notabug
status: newclosed