[ICE-8819] ace:dateTimeEntry - add new "lenientParsing" attribute (was: Incorrect date converted on client side) - ICEsoft JIRA Issue Tracker

Details

Type: Improvement
Status: Closed
Priority: Major
Resolution: Fixed
Affects Version/s: 3.2
Fix Version/s: EE-3.2.0.BETA, EE-3.2.0.GA, 3.3
Component/s: ACE-Components
Labels:
None
Environment:
All

Assignee Priority:
P1
Salesforce Case Reference:
Affects:

Documentation (User Guide, Ref. Guide, etc.), Compatibility/Configuration
Workaround Exists:

Yes
Workaround Description:
Using the "dd/MM/yyyy hh:mm" pattern seems to help to keep the correct pattern.

Description

When using the following date pattern, dd/MM/yyyy, selecting a date displays the correct patter. Upon manually changing the day value to an incorrect value, the date is still converted to a valid day.

For example:
- Set a date via the popup calendar: 11/12/2012
- Manually change the day value to: 211/12/2012
- The date is converted to: 29/06/2013

This is done on the client side so there is no chance to validate the date.

Options
- Sort By Name
- Sort By Date
- Ascending
- Descending
- Download All

Attachments

Hide
Case11790.zip

11/Dec/12 6:03 PM

22 kB

Arran Mccullough
Case11790/build.xml 3 kB

Case11790/nbproject/ant-deploy.xml 3 kB

Case11790/nbproject/build-impl.xml 58 kB

Case11790/nbproject/genfiles.properties 0.5 kB

Case11790/nbproject/.../private.properties 0.5 kB

Case11790/nbproject/private/private.xml 0.2 kB

Case11790/nbproject/project.properties 3 kB

Case11790/nbproject/project.xml 0.9 kB

Case11790/src/conf/MANIFEST.MF 0.0 kB

Case11790/src/.../managedBeans/PanelTab.java 0.6 kB

Case11790/src/java/.../managedBeans/Tab.java 0.1 kB

Case11790/src/.../managedBeans/TestBean.java 6 kB

Case11790/src/.../TestTabsManagerFB.java 1.0 kB

Case11790/src/java/.../model/TestDTO.java 0.6 kB

Case11790/web/index.xhtml 2 kB

Case11790/web/META-INF/context.xml 0.1 kB

Case11790/web/page2.xhtml 1 kB

Case11790/web/WEB-INF/web.xml 2 kB

Download Zip
Show

Case11790.zip

11/Dec/12 6:03 PM

22 kB

Arran Mccullough
Hide
Case11790WAR.zip

11/Dec/12 6:03 PM

9.36 MB

Arran Mccullough
Case11790.war 10.17 MB

Download Zip
Show

Case11790WAR.zip

11/Dec/12 6:03 PM

9.36 MB

Arran Mccullough

screenshot-01.png

328 kB

13/Dec/12 1:04 AM

Activity

Ascending order - Click to sort in descending order

Arran Mccullough created issue - 11/Dec/12 6:01 PM

Arran Mccullough made changes - 11/Dec/12 6:01 PM

Field	Original Value	New Value
Salesforce Case Reference		5007000000PXTHwAAP

Hide

Permalink

Arran Mccullough added a comment - 11/Dec/12 6:03 PM

Attached test case that shows the issue. Follow similar steps as mentioned in the issue description.

Show

Arran Mccullough added a comment - 11/Dec/12 6:03 PM Attached test case that shows the issue. Follow similar steps as mentioned in the issue description.

Arran Mccullough made changes - 11/Dec/12 6:03 PM

Attachment		Case11790.zip [ 15101 ]
Attachment		Case11790WAR.zip [ 15102 ]

Ken Fyten made changes - 12/Dec/12 4:29 PM

Assignee		yip.ng [ yip.ng ]
Fix Version/s		EE-3.2.0.GA [ 10332 ]
Fix Version/s		3.3 [ 10370 ]
Assignee Priority		P1 [ 10010 ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#32793	Wed Dec 12 18:07:21 MST 2012	yip.ng	~~ICE-8819~~: ace:dateTimeEntry - Incorrect date converted on client side.
				Files Changed
				MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryMeta.java MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryRenderer.java

Hide

Permalink

yip.ng added a comment - 12/Dec/12 11:41 PM

This has nothing to do with jQuery datepicker. It is standard behavior with lenient Java date parsing. (Search the web to understand more.) If we set the leniency to false, then a stricter parsing would be done, resulting in a conversion error. See video http://screencast.com/t/QdBuCU6qF2.

PF uses default leniency of true in Java date parsing. Change it to use false? Add an attribute for DateTimeEntry? Tell the user to use his own converter to do strict parsing?

Show

yip.ng added a comment - 12/Dec/12 11:41 PM This has nothing to do with jQuery datepicker. It is standard behavior with lenient Java date parsing. (Search the web to understand more.) If we set the leniency to false, then a stricter parsing would be done, resulting in a conversion error. See video http://screencast.com/t/QdBuCU6qF2 . PF uses default leniency of true in Java date parsing. Change it to use false? Add an attribute for DateTimeEntry? Tell the user to use his own converter to do strict parsing?

yip.ng made changes - 13/Dec/12 1:04 AM

Attachment

screenshot-01.png [ 15116 ]

Hide

Permalink

yip.ng added a comment - 13/Dec/12 1:07 AM - edited

Done. See screenshot-01.

Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java
Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java
Completed: At revision: 32793

Show

yip.ng added a comment - 13/Dec/12 1:07 AM - edited Done. See screenshot-01. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java Completed: At revision: 32793

yip.ng made changes - 13/Dec/12 1:07 AM

Status	Open [ 1 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Repository	Revision	Date	User	Message
ICEsoft Public SVN Repository	#32811	Thu Dec 13 17:20:09 MST 2012	yip.ng	~~ICE-8819~~: ace:dateTimeEntry - Incorrect date converted on client side.
				Files Changed
				MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryMeta.java MODIFY /icefaces3/trunk/icefaces/ace/component/src/org/icefaces/ace/component/datetimeentry/DateTimeEntryRenderer.java

Hide

Permalink

Ken Fyten added a comment - 13/Dec/12 11:47 PM

Let's rename the new attribute to "lenientParsing".

Show

Ken Fyten added a comment - 13/Dec/12 11:47 PM Let's rename the new attribute to "lenientParsing".

Ken Fyten made changes - 13/Dec/12 11:47 PM

Resolution	Fixed [ 1 ]
Status	Resolved [ 5 ]	Reopened [ 4 ]

Ken Fyten made changes - 13/Dec/12 11:48 PM

Summary

ace:dateTimeEntry - Incorrect date converted on client side

ace:dateTimeEntry - add new "lenientParsing" attribute (was: Incorrect date converted on client side)

Ken Fyten made changes - 13/Dec/12 11:48 PM

Issue Type	Bug [ 1 ]	Improvement [ 4 ]
Affects		Documentation (User Guide, Ref. Guide, etc.),Compatibility/Configuration [ 10003, 10002 ]

Hide

Permalink

yip.ng added a comment - 14/Dec/12 12:20 AM

Done.

Show

yip.ng added a comment - 14/Dec/12 12:20 AM Done. Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryMeta.java Modified: C:\svn\ossrepo\icefaces3\trunk\icefaces\ace\component\src\org\icefaces\ace\component\datetimeentry\DateTimeEntryRenderer.java Completed: At revision: 32811

yip.ng made changes - 14/Dec/12 12:20 AM

Status	Reopened [ 4 ]	Resolved [ 5 ]
Resolution		Fixed [ 1 ]

Ken Fyten made changes - 20/Dec/12 5:11 PM

Fix Version/s

EE-3.2.0.BETA [ 10573 ]

Ken Fyten made changes - 14/Nov/14 5:39 PM

Status

Resolved [ 5 ]

Closed [ 6 ]

People

Assignee:

yip.ng

Reporter:

Arran Mccullough

Votes:

3 Vote for this issue

Watchers:

4 Start watching this issue

Dates

Created:

11/Dec/12 6:01 PM

Updated:

14/Nov/14 5:39 PM

Resolved:

14/Dec/12 12:20 AM