google-code-export / smartgwt

Automatically exported from code.google.com/p/smartgwt
1 stars 3 forks source link

First day of week in German locale is Sunday, but correct is Monday #679

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
1. Set German locale in module configuration.
2. Create a form with a DateItem.
3. Run sample, click calendar icon to show date picker.

What is the expected output? What do you see instead?
Expected is to see first day of week Monday, but week starts with Sunday.

What version of the product are you using? On what operating system?
SmartGWT version: 3.0
Operating system: Windows 7

What browser(s) does this happen in?  Are there any browsers where the
issue does not occur?
Issue is same in all browsers: FF, IE 8, Chrome

Please provide any additional information below.

I think the value dateChooser_firstDayOfWeek in SmartGwtMessages_de.properties 
should be change from 0 to 1.

Picture of incorrect behavior is attached.

Sample code:

Locale set in module configuration (Sample.gwt.xml):

<extend-property name="locale" values="de_DE" />
<set-property name="locale" value="de_DE" />

Sample.java:

package com.piag.poc.client;

import com.google.gwt.core.client.EntryPoint;
import com.smartgwt.client.widgets.form.DynamicForm;
import com.smartgwt.client.widgets.form.fields.DateItem;

public class Sample implements EntryPoint {

    @Override
    public void onModuleLoad() {
        DynamicForm form = new DynamicForm();
        DateItem dateItem = new DateItem();
        form.setItems(dateItem);
        form.draw();
    }
}

Original issue reported on code.google.com by kub...@gmail.com on 25 Jul 2012 at 9:32

Attachments:

GoogleCodeExporter commented 9 years ago
Patching SmartGwtMessages_de.properties fixes the problem.

Original comment by marcel@frightanic.com on 21 Sep 2012 at 8:06

GoogleCodeExporter commented 9 years ago
It appears to be fixed in 3.1 (2012-12-20).

Original comment by marcel@frightanic.com on 20 Dec 2012 at 12:09