[MGNLUI-3123] Date property change in JCR browser doesn't work Created: 26/Aug/14 Updated: 05/Dec/14 Resolved: 01/Oct/14 |
|
| Status: | Closed |
| Project: | Magnolia UI |
| Component/s: | content app |
| Affects Version/s: | 5.2.7, 5.3.2 |
| Fix Version/s: | 5.2.10, 5.3.4 |
| Type: | New Feature | Priority: | Neutral |
| Reporter: | Frank Sommer | Assignee: | Christopher Zimmermann |
| Resolution: | Fixed | Votes: | 0 |
| Labels: | support | ||
| Remaining Estimate: | Not Specified | ||
| Time Spent: | Not Specified | ||
| Original Estimate: | Not Specified | ||
| Attachments: |
|
||||||||
| Issue Links: |
|
||||||||
| Template: |
|
||||||||
| Acceptance criteria: |
Empty
|
||||||||
| Release notes required: |
Yes
|
||||||||
| Date of First Response: | |||||||||
| Description |
|
If you change a date property in website jcr browser, the change won't be saved. Instead the current time will be set. There is no corresponding error message in the log. It is reproducable on the demoauthor instance. |
| Comments |
| Comment by Christian Ringele [ 26/Aug/14 ] |
|
The date property was in the past never meant to be directly edited via a JCR browser. I will add this issue to the customer care, where it will be decided if this functionality will be added in the future. |
| Comment by Eric Hechinger [ 03/Sep/14 ] |
|
Possible solution. |
| Comment by Eric Hechinger [ 03/Sep/14 ] |
|
Just need test classes |
| Comment by Christopher Zimmermann [ 01/Oct/14 ] |
|
The jcrPropertyAdapter#updateProperty VALUE_PROPERTY clause was unnecessarily converting the date object to a string in a format that createValue(String, int, factory) could not process properly. It now calls PropertyUtil.createValue with the actual value. |