isackrubio / gwt-cal

Automatically exported from code.google.com/p/gwt-cal
0 stars 0 forks source link

Wrong setter for com.bradrydzewski.gwt.calendar.client.CalendarFormat.setTimeFormat(String formatPattern) #80

Closed GoogleCodeExporter closed 9 years ago

GoogleCodeExporter commented 9 years ago
What steps will reproduce the problem?
Setting timeFormat with CalenderFormat results in unexpected Format for 
Calendar date label.
CalendarFormat.INSTANCE.setTimeFormat("HH:00");

What is the expected output? What do you see instead?
TimeFormat should be "HH:00" instead dateFormat will be set to "HH:00"

What version of the product are you using? On what operating system?
gwt-cal 0.9.1

Please provide any additional information below.
Setter method for member private DateTimeFormat "timeFormat" sets member 
"dateFormat" instead.
  /**
    * Sets the pattern used to format the displayed hours and re-generates
    * all hour labels.
    *
    * @param formatPattern A legal format following the patterns
    * in {@link com.google.gwt.i18n.client.DateTimeFormat}
    */
   public void setTimeFormat(String formatPattern) {
      ***!!!dateFormat!!!*** = DateTimeFormat.getFormat(formatPattern);
      generateHourLabels();      
   }

Kind regards

Bernhard

Original issue reported on code.google.com by unger.be...@googlemail.com on 23 Aug 2010 at 2:17

GoogleCodeExporter commented 9 years ago
Will look into the issue ASAP. Thanks for the feedback!

- Carlos

Original comment by carlos.m...@gmail.com on 31 Jan 2011 at 4:00

GoogleCodeExporter commented 9 years ago
Fixed field set in 'setTimeFormat' setter method, revision 360. Thank you for 
the feedback, again.

- Carlos

Original comment by carlos.m...@gmail.com on 1 Feb 2011 at 12:54

GoogleCodeExporter commented 9 years ago

Original comment by ctasada on 3 Jan 2012 at 3:59