Bug 6679 - [SIGNED-OFF] fix 3 perlcritic violations in C4/SQLHelper.pm
Subroutine prototypes used at line 251, column 1. See page 194 of PBP. (Severity: 5) Subroutine prototypes used at line 287, column 1. See page 194 of PBP. (Severity: 5) Subroutine prototypes used at line 317, column 1. See page 194 of PBP. (Severity: 5) Signed-off-by: Jonathan Druart <jonathan.druart@biblibre.com> Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
This commit is contained in:
parent
162d81d536
commit
16cc80d2bd
|
|
@ -248,7 +248,7 @@ Get the Primary Key field names of the table
|
|||
|
||||
=cut
|
||||
|
||||
sub GetPrimaryKeys($) {
|
||||
sub GetPrimaryKeys {
|
||||
my $tablename=shift;
|
||||
my $hash_columns=_get_columns($tablename);
|
||||
return grep { $hash_columns->{$_}->{'Key'} =~/PRI/i} keys %$hash_columns;
|
||||
|
|
@ -284,7 +284,7 @@ With
|
|||
|
||||
=cut
|
||||
|
||||
sub _get_columns($) {
|
||||
sub _get_columns {
|
||||
my ($tablename) = @_;
|
||||
unless ( exists( $hashref->{$tablename} ) ) {
|
||||
my $dbh = C4::Context->dbh;
|
||||
|
|
@ -314,7 +314,7 @@ If it is not for research purpose, filter primary keys
|
|||
|
||||
=cut
|
||||
|
||||
sub _filter_columns ($$;$) {
|
||||
sub _filter_columns {
|
||||
my ($tablename,$research, $filtercolumns)=@_;
|
||||
if ($filtercolumns){
|
||||
return (@$filtercolumns);
|
||||
|
|
|
|||
Loading…
Reference in New Issue