Skip to main content

Search and Top Navigation

#8031 closed bug (fixed)

Opened January 19, 2012 06:08PM UTC

Closed January 20, 2012 12:48PM UTC

Bad character in jquery.i18n.js (UI release 1.8.17 zip)

Reported by: regisc Owned by: regisc
Priority: minor Milestone: 1.8.17
Component: [meta] ui.build Version: 1.8.17
Keywords: Cc:
Blocked by: Blocking:
Description

/development-bundle/ui/i18n/jquery-ui-i18n.js contains a bad character (looks like a BOM) in the middle of the file.

(look for "Dutch (Belgium)", bad char is just before the "/*")

Attachments (0)
Change History (9)

Changed January 19, 2012 07:13PM UTC by scottgonzalez comment:1

owner: → regisc
status: newpending

Indeed. This was actually fixed last week with eee63a2e2435a6ecbcd0daa15f41b7bd5901e307. Did you just download this or did you download it a while ago? The 1.8.17 zips should be fixed. If they're not, please let me know where you got the bad zip from so we can fix it.

Changed January 19, 2012 07:16PM UTC by regisc comment:2

status: pendingnew

I downloaded the zip today from http://jqueryui.com/download/jquery-ui-1.8.17.custom.zip

Thanks for looking into this.

Changed January 19, 2012 08:31PM UTC by scottgonzalez comment:3

Can you try now and verify whether it works for you?

Changed January 19, 2012 08:37PM UTC by regisc comment:4

Replying to [comment:3 scott.gonzalez]:

Can you try now and verify whether it works for you?

Redownloaded the zip (url from [comment:2]), opened in vim and looked for "Dutch":

    $.datepicker.setDefaults($.datepicker.regional['ms']);
});<feff>/* Dutch (Belgium) initialisation for the jQuery UI date picker plugin.
/* David De Sloovere @DavidDeSloovere */
jQuery(function($){

Note the "<feff>", so still not good.

Changed January 19, 2012 09:11PM UTC by scottgonzalez comment:5

This seems to be broken in places that should have been fixed already. I'm trying to figure out what's going on.

Changed January 19, 2012 09:20PM UTC by scottgonzalez comment:6

Ok, it looks like only the minified files were fixed. I'm going to manually fix the other source nl-BE and concatenated file in the SVN tag and then regenerate the quick download. That should fix all the problems with the download builder. I'll let you know when it's done.

Changed January 19, 2012 09:37PM UTC by scottgonzalez comment:7

Download should be working now. I tested with a custom download and the quick download, but would appreciate if you can confirm as well.

Changed January 20, 2012 05:07AM UTC by regisc comment:8

> Confirmed

Thanks

Changed January 20, 2012 12:48PM UTC by scottgonzalez comment:9

milestone: 1.91.8.17
resolution: → fixed
status: newclosed