Skip to content

Commit

Permalink
BUG Fix wrong date conversion from PHP format 'y' to jquery date and …
Browse files Browse the repository at this point in the history
…back.

While 'y' in PHP means 4 digits year (e.g. 2012), jquery date picker means 2 digits (e.g. 12). That's all fine until then but when you pass the 2 digit year value back to PHP it all goes awry.
For exmple, defatult date format in en_US is 'MMM d, y' so in jquery date picker it's something like this 'Aug 22, 12' and eventually PHP will convert that value to '0012-08-22' which completely wrong.
  • Loading branch information
phalkunz authored and chillu committed Aug 30, 2012
1 parent 14b8071 commit f3fcae3
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion forms/DateField.php
Expand Up @@ -591,7 +591,8 @@ static function convert_iso_to_jquery_format($format) {
'/l/' => '',
'/YYYY/' => 'yy',
'/yyyy/' => 'yy',
'/[^y]yy[^y]/' => 'y',
// See http://open.silverstripe.org/ticket/7669
'/y{1,3}/' => 'yy',
'/a/' => '',
'/B/' => '',
'/hh/' => '',
Expand Down

0 comments on commit f3fcae3

Please sign in to comment.