http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=7067

--- Comment #32 from Kyle M Hall <k...@bywatersolutions.com> ---
> - A new dependency (String::Random) is required and has to be added to the
> C4/Installer/PerlDependencies.pm file

Fixed.

> - The page opac-user.pl does not have a register link.

Fixed

> - There is no verification on the date of birth entry and could be caused a
> warning

Good catch. I told myself I would take care of it later and then forgot ; )

> 
> * misc/cronjobs/delete_expired_opac_registrations.pl                      
> FAIL
>       critic                      FAIL
>               "require" statement with library name as string at line 28, 
> column 12. Use
> a bareword instead.
>               Code before strictures are enabled at line 23, column 1. See 
> page 429 of
> PBP.
> 
> * misc/cronjobs/delete_unverified_opac_registrations.pl                   
> FAIL
>       critic                      FAIL
>               "require" statement with library name as string at line 28, 
> column 12. Use
> a bareword instead.
>               Code before strictures are enabled at line 23, column 1. See 
> page 429 of
> PBP.

These are not valid. I use 'use Modern::Perl' which enables both warnings and
strictures. The rest of the errors you saw should be fixed.

-- 
You are receiving this mail because:
You are the QA Contact for the bug.
You are watching all bug changes.
_______________________________________________
Koha-bugs mailing list
Koha-bugs@lists.koha-community.org
http://lists.koha-community.org/cgi-bin/mailman/listinfo/koha-bugs
website : http://www.koha-community.org/
git : http://git.koha-community.org/
bugs : http://bugs.koha-community.org/

Reply via email to