Issue Details (XML | Word | Printable)

Key: SAK-3825
Type: Bug Bug
Status: Closed Closed
Resolution: Fixed
Priority: Minor Minor
Assignee: Beth Kirschner
Reporter: Stephen Marquard
Votes: 0
Watchers: 2
Operations

If you were logged in you would be able to see more operations.
Sakai

chef_dateselectionwidget makes assumptions about date format (I18N)

Created: 12-Feb-2006 12:37   Updated: 27-Oct-2009 15:12
Component/s: Global, Internationalization
Affects Version/s: 2.1.1, 2.1.2, 2.2.0, 2.2.1, 2.2.2, 2.2.3, 2.3.2, 2.3.x, 2.4.0, 2.4.1, 2.5.0, 2.5.2, 2.5.3
Fix Version/s: 2.5.5, 2.6.0

Time Tracking:
Not Specified

File Attachments: 1. File SAK-3825_2-5-x.diff (45 kB)

Environment: Linux, mysql
Issue Links:
Relate

2.6.x Status: None
2.5.x Status: Resolved
2.4.x Status: None


 Description  « Hide
chef_dateselectionwidget defined in VM_chef_library.vm makes assumptions about date formats in presenting date entry selectors as "month day year". In many regions, "day month year" is the format used. Affected tools are: announcement, assignment, calendar & content


 All   Comments   Work Log   Change History   Subversion Commits   git Commits      Sort Order: Ascending order - Click to sort in descending order
John Leasia added a comment - 17-Feb-2006 12:23
Gonzalo - is this something you could work on with Joanne?

Gonzalo Silverio added a comment - 22-Mar-2006 15:07
Joanne,

To go with recent work you have done in Schedule...

Yuanyuan Joanne Sui added a comment - 23-Mar-2006 10:12
This issue is like SAK-3824, related to the hardcoded date format in Sakai. The date format is not only in the Schedule tool, but in many places else, because the TimeService has contained the hardcoded date format, and multiple tools have used that.
John's comments on this are: "Doing all the date things as you say should be done after a design review/plan to make sure we are making the right changes. We can get to that maybe after schedule group work, maybe for 2.2 but I don't think (for us anyway) that it's critical. It might get to be so for some Sakai partners and then maybe they will take it on."
So I'd like to hold this for a while, until we are ready for this.

Beth Kirschner added a comment - 21-Jun-2006 17:05
There is currently an effort to design a new I18N aware date/time widget (see http://bugs.sakaiproject.org/confluence/pages/viewpage.action?pageId=20034)

Beth Kirschner added a comment - 05-Jul-2006 11:38

Stephen Marquard added a comment - 18-Sep-2008 03:42
Hi Beth,

In the absence of a new widget for velocity tools, is there any easy way to get the mmm / dd order of the current velocity template reversed based on locale info ?

We have this patch

http://source.cet.uct.ac.za/svn/sakai/patches/branches/vula_2-5-x/patch-22-datewidget.diff

which we have been maintaining since Sakai 2.1 and are keen to drop it.

Beth Kirschner added a comment - 29-Sep-2008 13:18
I've created a new (optional) parameter for the dateselectionwidget velocity macro to specify the date format. I've also created a new VelocityPortletPaneledAction.getDateFormatString() method which dynamically determines the date format based on the user's preferred language/locale.

It looks like the velocity tools which use this date picker are: announcement, assignment, calendar, & content

If I've missed any they will default to the US locale (e.g. no change in behavior). I'll close this issue when the above tools have been updated to use the new parameter and method.

QA instructions:
1. Change Language/Locale (Preferences -> Locale) to any non-US locale (e.g. "English (United Kingdom)" ) and verify the date picker modifies the month/date/year order appropriately (e.g. in the UK, the order is day/month/year).

Peter A. Knoop added a comment - 29-Sep-2008 13:50
Beth, i'm presuming this work is intended for 2.6, so I've set that as the Fix Version.

Beth Kirschner added a comment - 01-Oct-2008 09:40
The velocity date pickers are now localized (!) -- see above for QA instructions and details.

Jean-François Lévêque added a comment - 02-Oct-2008 03:05
Works fine with sakai/trunk on Oracle. Checked adding entries within Schedule, Announcements, Resources and Assignments.

Jean-François Lévêque added a comment - 10-Dec-2008 04:59
There's a 2.5.x merging conflict with the first revision changes:
svn merge --dry-run -c52912 https://source.sakaiproject.org/svn/velocity/trunk velocity/
U velocity/tool/src/java/org/sakaiproject/cheftool/VelocityPortletPaneledAction.java
C velocity/tool/src/templates/VM_chef_library.vm

Could a 2.5.x patch be provided?

Beth Kirschner added a comment - 02-Mar-2009 11:30
There were 2.5.x merge conflicts in assignment and velocity -- everything else merged cleanly. I hand merged assignment and velocity changes, but do not currently have a server to test them on. Attached is a _complete_ set of diffs for SAK-3825 for Sakai 2.5.x.

Beth Kirschner added a comment - 06-Mar-2009 06:00
Oops -- the BasicEmailService patch definitely did not belong with this. I've uploaded a revised patch file that has this removed. I've also removed some file deletions from the content directory, just to play it safe (these obsolete files were removed in the trunk for clean-up but aren't required to be deleted).

Jean-François Lévêque added a comment - 06-Mar-2009 07:16
2.5.x merge r58363

Nancy Wilkison added a comment - 01-May-2009 12:25
QA ENVIRONMENT -
vista, firefox, ie7
on 2.5.5-beta01 qa5 michigan server
http://qa5-us.sakaiproject.org/portal
Sakai QA Network qa5-us (svn tags/sakai-2.5.5-beta01) using Oracle - Built: 04/23/09 10:10 - Sakai sakai-2.5.5-beta01 - Server tartar

QA SUMMARY -
verify correct day, month, year format based on language.
1 - qa5-us server as nancy (IOR), preferences language=English US
announcement - shows month-day-year
assignment - all show as month-day-year
schedule - added assignment to schedule. details show due date month-day-year
resources - added file. date added is month-day-year
2 - same server as alice (student), preferences language=English UK
checked tools announcement, assignment, calendar(schedule) & content (resources)
announcement - below shows day-month-year
assignment - all show as day-month-year
schedule - details show due date day-month-year
resources - day-month-year

QA RESULTS -
pass verified.

Anthony Whyte added a comment - 24-Jun-2009 08:29