[MAGNOLIA-4497] Date Control: timeFormat and jsTimeFormat not working Created: 30/Jul/12 Updated: 14/Jan/14 Resolved: 05/Apr/13 |
|
| Status: | Closed |
| Project: | Magnolia |
| Component/s: | gui |
| Affects Version/s: | 4.4.6 |
| Fix Version/s: | 4.4.11, 4.5.9 |
| Type: | Bug | Priority: | Neutral |
| Reporter: | Richard Unger | Assignee: | Milan Divilek |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | datecontrol, dialogs, gui | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Environment: |
Magnolia EE 4.4.8 |
||
| Issue Links: |
|
||||||||||||||||||||||||
| Template: |
|
||||||||||||||||||||||||
| Acceptance criteria: |
Empty
|
||||||||||||||||||||||||
| Task DoD: |
[ ]*
Doc/release notes changes? Comment present?
[ ]*
Downstream builds green?
[ ]*
Solution information and context easily available?
[ ]*
Tests
[ ]*
FixVersion filled and not yet released
[ ] 
Architecture Decision Record (ADR)
|
||||||||||||||||||||||||
| Bug DoR: |
[ ]*
Steps to reproduce, expected, and actual results filled
[ ]*
Affected version filled
|
||||||||||||||||||||||||
| Date of First Response: | |||||||||||||||||||||||||
| Description |
|
Since upgrading to Magnolia 4.4.6 we are experiencing the following problem: We have a customer requirement that dates in certain dialogs must be input with times, but without seconds. The problems is when saving the dialog: |
| Comments |
| Comment by Richard Unger [ 30/Jul/12 ] |
|
The exception generated is: ERROR info.magnolia.module.admininterface.SaveHandlerImpl 30.07.2012 12:05:40 – Could not update date value of node:/lmat/kalender/umwelt/xxx/veranstaltung of property:enddate |
| Comment by Richard Unger [ 30/Jul/12 ] |
|
Following the code for saving, it seems obvious that it can't work: SaveHandlerImpl.processDate() simply uses DateUtil.getUTCCalendarFromDialogString() This method doesn't take the custom date or time format into account in any way, insisting on one of 2 standardized formats (with or without time). |
| Comment by Richard Unger [ 02/Aug/12 ] |
|
Seems we're not the only ones with this issue: |
| Comment by Milan Divilek [ 03/Aug/12 ] |
|
Thanks for reporting issue Richard. It should be fixed for the next version. |
| Comment by Milan Divilek [ 03/Aug/12 ] |
|
Before resolving this issue in 4.5.x branch, it's necessary solved issue |