Support for custom "date_format" and "time_format" arguments#498
Support for custom "date_format" and "time_format" arguments#498mweimerskirch wants to merge 1 commit into
Conversation
…text_date, text_date_timestamp and text_datetime_timestamp fields
|
I downloaded the patch file for #318 directly from GitHub because I didn't want to pull in 23 individual commits and there were hundreds of whitespace changes as well as changes in a translation file: https://github.com/WebDevStudios/CMB2/pull/318.patch |
|
yivi here. I see that in your version you are are back to using DateTime::createFromFormat, whereas we had a workaround for PHP versions < 5.3. Also, my work continued in #446, on a different branch that @jtsternberg created to carry on this work. I updated it a couple weeks ago, and it does have support for time_picker as well... I think I messed up with the PRs, by opening a new one instead of continuing work in #318, but I can merge trunk into #446 again to see if everything keeps working as intended. Regards. |
|
@yivi Could you squash the commits for the current pull request? That way I could test it as well and let you know if any of my use cases fail. |
|
@jtsternberg , #446 is the current one. |
Support for custom "date_format" and "time_format" arguments for the text_date, text_date_timestamp and text_datetime_timestamp fields.
Original fixes by @yivi, based on #318
I removed all the unrelated changes, and added support for the time picker. To make merging easier, the pull request also doesn't include a minified JS file.