Time Field minutes not saving correctly ->solved, hopefully!

edited December 2012 in Bug Reports
Hello,
I had noticed that when using a Time field type, that the minutes were always saving as 00 in the database and also were then 00 in the Time field minutes value on the form.

I discovered on line 2155 of modules/fuel/libraries/form_builder.php that a ')' was in the wrong place, in Fuel 1.0 beta.

Was:
$min = (isset($_POST["'.$params['key'].'_min"]) AND is_numeric($_POST["'.$params['key'].'_min"] AND $hr)) ? $_POST["'.$params['key'].'_min"] : "00";

Should be:
$min = (isset($_POST["'.$params['key'].'_min"]) AND is_numeric($_POST["'.$params['key'].'_min"]) AND $hr) ? $_POST["'.$params['key'].'_min"] : "00";

I hope this helps someone else / can be fixed in the mother-code!

Many thanks for the great CMS - I am enjoying being involved at the beta stage too!

Guy

Comments

  • edited 5:34AM
    Thanks for the report. I'll post an update to the repo very soon.
Sign In or Register to comment.