[Koha-bugs] [Bug 17855] Updated Koha web installer and new onboarding tool feature to guide users through setting up Koha

[bugzilla-daemon at bugs.koha-community.org]

https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=17855

--- Comment #113 from M. Tompsett <mtompset at hotmail.com> ---
Comment on attachment 59279
  --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=59279
[SIGNED-OFF] bug 17855 - Updated web installer finished

Review of attachment 59279:
 --> (https://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=17855&attachment=59279)
-----------------------------------------------------------------

Sorry I didn't catch this the first time.

::: installer/onboarding.pl
@@ +381,5 @@
> +            $newdata{categorycode}   = $input->param('categorycode_entry');
> +            $newdata{userid}         = $input->param('userid');
> +            $newdata{password}       = $input->param('password');
> +            $newdata{password2}      = $input->param('password2');
> +            $newdata{dateexpiry}     = '12/10/2016';

No no no! I was trying to test under Xenial Kohadevbox, and it blew up!
"DBIx::Class::Storage::DBI::_dbh_execute(): Incorrect date value: '12/10/2016'
for column 'dateexpiry' at row 1 at /home/vagrant/kohaclone/Koha/Object.pm line
120"
Dates that will be used directly with SQL queries need to be YYYY-MM-DD.
Plus hard coded?! YUCK!

-- 
You are receiving this mail because:
You are watching all bug changes.