Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.
Bug 357303 - Days of the week are improperly truncated with Hebrew locale
Summary: Days of the week are improperly truncated with Hebrew locale
Status: RESOLVED FIXED
Alias: None
Product: z_Archived
Classification: Eclipse Foundation
Component: Nebula (show other bugs)
Version: unspecified   Edit
Hardware: PC Windows 7
: P3 normal (vote)
Target Milestone: ---   Edit
Assignee: Emil Crumhorn CLA
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2011-09-10 03:18 EDT by Caleb CLA
Modified: 2021-07-05 11:40 EDT (History)
0 users

See Also:

Attachments
screenshot of the problem (4.34 KB, image/png)
2011-09-10 03:18 EDT, Caleb CLA 		wim.jongman: iplog-
Details
proposed patch (986 bytes, patch)
2011-09-10 03:20 EDT, Caleb CLA 		wim.jongman: iplog+
Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Caleb CLA 2011-09-10 03:18:17 EDT 
Build Identifier: 

In Hebrew, the days of the week contain the word "day" (or יום in Hebrew), for example:
Sunday - יום ראשון
Monday - יום שני
...
Friday - יום שישי
Saturday - שבת

As you can see the days are called "Day One", "Day Two", and Saturday is just called Sabbath.

This is problematic because the Calendar displays the first letter of the day of the week, and it ends up being "י" for all the days except Saturday.

Reproducible: Always
Comment 1 Caleb CLA 2011-09-10 03:18:38 EDT 
Created attachment 203107 [details]
screenshot of the problem
Comment 2 Caleb CLA 2011-09-10 03:20:31 EDT 
Created attachment 203108 [details]
proposed patch

The patch proposes to use getShortWeekdays() which returns the proper results for Hebrew, and shouldn't interfere with other locales (for example, in English it would return Sun for Sunday, but it's OK since we're going to use the first letter anyway).
Comment 3 Emil Crumhorn CLA 2011-09-24 12:17:11 EDT 
Good fix, thanks! Now patched.