search

OpenESignForms / openesignforms

Open eSignForms is the first open source SaaS web contracting platform
https://open.esignforms.com
107 stars 50 forks source link

Deployment screen fails if default date format is DD FullMonthName YYYY format #159

Closed OpenESignForms closed 9 years ago

OpenESignForms commented 9 years ago

See forum discussion https://groups.google.com/forum/#!topic/openesignformsdev/XN32rp1MSsQ for details. If the deployment screen date format is changed to "DD FullMonthName YYYY" format, the installation date gets a format error. It appears we cannot parse that date string back to a valid EsfDate object (it is an atypical date format for date input which is generally mm/dd/yyyy in the U.S. or yyyy-mm-dd).

Want to see if we can use Java date routines to parse that long string back to a date, and also why the installation date is being converted at all since it should be read-only and not be updated whatsoever.

OpenESignForms commented 9 years ago

Accept dates in DD MonthName YYYY format. Fixed in the 15.8.23_p0930 patch release.

OpenESignForms commented 9 years ago

Another user used the date format MonthName DD, YYYY format, which then also broke this. There are all sorts of date output formats that we don't expect to ever accept as input, hence the validation issues. Instead, fixed the deployment view to no longer show the installation date as an EsfDate field, but just let it be a String field so there's no issue with the Vaadin bug that attempts to convert the string date back to an EsfDate object (even though the field is read-only and has no corresponding setting method).