sub returnlost{
my ( $borrowernumber, $itemnum ) = @_;
C4::Circulation::MarkIssueReturned( $borrowernumber, $itemnum );
- my $borrower = C4::Members::GetMember( $borrowernumber, 'borrowernumber' );
+ my $borrower = C4::Members::GetMember( 'borrowernumber'=>$borrowernumber );
my @datearr = localtime(time);
my $date = ( 1900 + $datearr[5] ) . "-" . ( $datearr[4] + 1 ) . "-" . $datearr[3];
my $bor = "$borrower->{'firstname'} $borrower->{'surname'} $borrower->{'cardnumber'}";
# and parse borrower ...
my $innerletter = $letter;
- my $borinfo = GetMember( $_->{'borrowernumber'}, 'borrowernumber' );
+ my $borinfo = GetMember( 'borrowernumber' => $_->{'borrowernumber'});
parseletter( $innerletter, 'borrowers', $_->{'borrowernumber'} );
# ... then send mail
my $to_address = $message->{to_address};
unless ($to_address) {
- my $member = C4::Members::GetMember( $message->{'borrowernumber'} );
+ my $member = C4::Members::GetMember( 'borrowernumber' => $message->{'borrowernumber'} );
unless ($member) {
warn "FAIL: No 'to_address' and INVALID borrowernumber ($message->{borrowernumber})";
_set_message_status( { message_id => $message->{'message_id'},
sub _send_message_by_sms ($) {
my $message = shift or return undef;
- my $member = C4::Members::GetMember( $message->{'borrowernumber'} );
+ my $member = C4::Members::GetMember( 'borrowernumber' => $message->{'borrowernumber'} );
return unless $member->{'smsalertnumber'};
my $success = C4::SMS->send_sms( { destination => $member->{'smsalertnumber'},
use C4::Reserves;
use C4::Accounts;
use C4::Biblio;
+use C4::SQLHelper qw(InsertInTable UpdateInTable);
our ($VERSION,@ISA,@EXPORT,@EXPORT_OK,$debug);
=head2 GetMember
- $borrower = &GetMember($information, $type);
+ $borrower = &GetMember(%information);
Looks up information about a patron (borrower) by either card number
,firstname, or borrower number, depending on $type value.
#'
sub GetMember {
- my ( $information, $type ) = @_;
+ my ( %information ) = @_;
my $dbh = C4::Context->dbh;
my $sth;
my $select = "
FROM borrowers
LEFT JOIN categories on borrowers.categorycode=categories.categorycode
";
- if (defined($type) and ( $type eq 'cardnumber' || $type eq 'firstname'|| $type eq 'userid'|| $type eq 'borrowernumber' ) ){
- $information = uc $information;
- $sth = $dbh->prepare("$select WHERE $type=?");
- } else {
- $sth = $dbh->prepare("$select WHERE borrowernumber=?");
- }
- $sth->execute($information);
- my $data = $sth->fetchrow_hashref;
- ($data) and return ($data);
-
- if (defined($type) and ($type eq 'cardnumber' || $type eq 'firstname')) { # otherwise, try with firstname
- $sth = $dbh->prepare("$select WHERE firstname like ?");
- $sth->execute($information);
- $data = $sth->fetchrow_hashref;
- ($data) and return ($data);
- }
- return undef;
+ $select.=" WHERE ".join(" AND ",map {"$_ = ?"}keys %information);
+ $select=~s/AND $//;
+ $debug && warn $select, " ",values %information;
+ $sth = $dbh->prepare("$select");
+ $sth->execute(map{$information{$_}} keys %information);
+ my $data = $sth->fetchall_arrayref({});
+ return undef if (scalar(@$data)==0);
+ if (scalar(@$data)==1) {return $$data[0];}
+ ($data) and return $data;
}
+
=head2 IsMemberBlocked
=over 4
=back
=cut
-
sub ModMember {
my (%data) = @_;
- my $dbh = C4::Context->dbh;
- my $iso_re = C4::Dates->new()->regexp('iso');
- foreach (qw(dateofbirth dateexpiry dateenrolled)) {
- if (my $tempdate = $data{$_}) { # assignment, not comparison
- ($tempdate =~ /$iso_re/) and next; # Congatulations, you sent a valid ISO date.
- warn "ModMember given $_ not in ISO format ($tempdate)";
- my $tempdate2 = format_date_in_iso($tempdate);
- if (!$tempdate2 or $tempdate2 eq '0000-00-00') {
- warn "ModMember cannot convert '$tempdate' (from syspref to ISO)";
- next;
- }
- $data{$_} = $tempdate2;
- }
- }
- if (!$data{'dateofbirth'}){
- delete $data{'dateofbirth'};
- }
- my @columns = &columns;
- my %hashborrowerfields = (map {$_=>1} @columns);
- my $query = "UPDATE borrowers SET \n";
- my $sth;
- my @parameters;
-
# test to know if you must update or not the borrower password
if (exists $data{password}) {
if ($data{password} eq '****' or $data{password} eq '') {
$data{password} = md5_base64($data{password});
}
}
- my @badkeys;
- foreach (keys %data) {
- next if ($_ eq 'borrowernumber' or $_ eq 'flags');
- if ($hashborrowerfields{$_}){
- $query .= " $_=?, ";
- push @parameters,$data{$_};
- } else {
- push @badkeys, $_;
- delete $data{$_};
- }
- }
- (@badkeys) and warn scalar(@badkeys) . " Illegal key(s) passed to ModMember: " . join(',',@badkeys);
- $query =~ s/, $//;
- $query .= " WHERE borrowernumber=?";
- push @parameters, $data{'borrowernumber'};
- $debug and print STDERR "$query (executed w/ arg: $data{'borrowernumber'})";
- $sth = $dbh->prepare($query);
- my $execute_success = $sth->execute(@parameters);
- $sth->finish;
-
+ my $execute_success=UpdateInTable("borrowers",\%data);
# ok if its an adult (type) it may have borrowers that depend on it as a guarantor
# so when we update information for an adult we should check for guarantees and update the relevant part
# of their records, ie addresses and phone numbers
# is adult check guarantees;
UpdateGuarantees(%data);
}
- logaction("MEMBERS", "MODIFY", $data{'borrowernumber'}, "$query (executed w/ arg: $data{'borrowernumber'})")
+ logaction("MEMBERS", "MODIFY", $data{'borrowernumber'}, "UPDATE (executed w/ arg: $data{'borrowernumber'})")
if C4::Context->preference("BorrowersLog");
return $execute_success;
my $dbh = C4::Context->dbh;
$data{'userid'} = '' unless $data{'password'};
$data{'password'} = md5_base64( $data{'password'} ) if $data{'password'};
-
- # WE SHOULD NEVER PASS THIS SUBROUTINE ANYTHING OTHER THAN ISO DATES
- # IF YOU UNCOMMENT THESE LINES YOU BETTER HAVE A DARN COMPELLING REASON
-# $data{'dateofbirth'} = format_date_in_iso( $data{'dateofbirth'} );
-# $data{'dateenrolled'} = format_date_in_iso( $data{'dateenrolled'});
-# $data{'dateexpiry'} = format_date_in_iso( $data{'dateexpiry'} );
- # This query should be rewritten to use "?" at execute.
- if (!$data{'dateofbirth'}){
- undef ($data{'dateofbirth'});
- }
- my $query =
- "insert into borrowers set cardnumber=" . $dbh->quote( $data{'cardnumber'} )
- . ",surname=" . $dbh->quote( $data{'surname'} )
- . ",firstname=" . $dbh->quote( $data{'firstname'} )
- . ",title=" . $dbh->quote( $data{'title'} )
- . ",othernames=" . $dbh->quote( $data{'othernames'} )
- . ",initials=" . $dbh->quote( $data{'initials'} )
- . ",streetnumber=". $dbh->quote( $data{'streetnumber'} )
- . ",streettype=" . $dbh->quote( $data{'streettype'} )
- . ",address=" . $dbh->quote( $data{'address'} )
- . ",address2=" . $dbh->quote( $data{'address2'} )
- . ",zipcode=" . $dbh->quote( $data{'zipcode'} )
- . ",country=" . $dbh->quote( $data{'country'} )
- . ",city=" . $dbh->quote( $data{'city'} )
- . ",phone=" . $dbh->quote( $data{'phone'} )
- . ",email=" . $dbh->quote( $data{'email'} )
- . ",mobile=" . $dbh->quote( $data{'mobile'} )
- . ",phonepro=" . $dbh->quote( $data{'phonepro'} )
- . ",opacnote=" . $dbh->quote( $data{'opacnote'} )
- . ",guarantorid=" . $dbh->quote( $data{'guarantorid'} )
- . ",dateofbirth=" . $dbh->quote( $data{'dateofbirth'} )
- . ",branchcode=" . $dbh->quote( $data{'branchcode'} )
- . ",categorycode=" . $dbh->quote( $data{'categorycode'} )
- . ",dateenrolled=" . $dbh->quote( $data{'dateenrolled'} )
- . ",contactname=" . $dbh->quote( $data{'contactname'} )
- . ",borrowernotes=" . $dbh->quote( $data{'borrowernotes'} )
- . ",dateexpiry=" . $dbh->quote( $data{'dateexpiry'} )
- . ",contactnote=" . $dbh->quote( $data{'contactnote'} )
- . ",B_address=" . $dbh->quote( $data{'B_address'} )
- . ",B_address2=" . $dbh->quote( $data{'B_address2'} )
- . ",B_zipcode=" . $dbh->quote( $data{'B_zipcode'} )
- . ",B_country=" . $dbh->quote( $data{'B_country'} )
- . ",B_city=" . $dbh->quote( $data{'B_city'} )
- . ",B_phone=" . $dbh->quote( $data{'B_phone'} )
- . ",B_email=" . $dbh->quote( $data{'B_email'} )
- . ",password=" . $dbh->quote( $data{'password'} )
- . ",userid=" . $dbh->quote( $data{'userid'} )
- . ",sort1=" . $dbh->quote( $data{'sort1'} )
- . ",sort2=" . $dbh->quote( $data{'sort2'} )
- . ",contacttitle=" . $dbh->quote( $data{'contacttitle'} )
- . ",emailpro=" . $dbh->quote( $data{'emailpro'} )
- . ",contactfirstname=" . $dbh->quote( $data{'contactfirstname'} )
- . ",sex=" . $dbh->quote( $data{'sex'} )
- . ",fax=" . $dbh->quote( $data{'fax'} )
- . ",relationship=" . $dbh->quote( $data{'relationship'} )
- . ",B_streetnumber=" . $dbh->quote( $data{'B_streetnumber'} )
- . ",B_streettype=" . $dbh->quote( $data{'B_streettype'} )
- . ",gonenoaddress=" . $dbh->quote( $data{'gonenoaddress'} )
- . ",lost=" . $dbh->quote( $data{'lost'} )
- . ",debarred=" . $dbh->quote( $data{'debarred'} )
- . ",ethnicity=" . $dbh->quote( $data{'ethnicity'} )
- . ",ethnotes=" . $dbh->quote( $data{'ethnotes'} )
- . ",altcontactsurname=" . $dbh->quote( $data{'altcontactsurname'} )
- . ",altcontactfirstname=" . $dbh->quote( $data{'altcontactfirstname'} )
- . ",altcontactaddress1=" . $dbh->quote( $data{'altcontactaddress1'} )
- . ",altcontactaddress2=" . $dbh->quote( $data{'altcontactaddress2'} )
- . ",altcontactaddress3=" . $dbh->quote( $data{'altcontactaddress3'} )
- . ",altcontactzipcode=" . $dbh->quote( $data{'altcontactzipcode'} )
- . ",altcontactcountry=" . $dbh->quote( $data{'altcontactcountry'} )
- . ",altcontactphone=" . $dbh->quote( $data{'altcontactphone'} ) ;
- $debug and print STDERR "AddMember SQL: ($query)\n";
- my $sth = $dbh->prepare($query);
- # print "Executing SQL: $query\n";
- $sth->execute();
- $sth->finish;
- $data{'borrowernumber'} = $dbh->{'mysql_insertid'}; # unneeded w/ autoincrement ?
+ $data{'borrowernumber'}=InsertInTable("borrowers",\%data);
# mysql_insertid is probably bad. not necessarily accurate and mysql-specific at best.
-
logaction("MEMBERS", "CREATE", $data{'borrowernumber'}, "") if C4::Context->preference("BorrowersLog");
# check for enrollment fee & add it if needed
- $sth = $dbh->prepare("SELECT enrolmentfee FROM categories WHERE categorycode=?");
+ my $sth = $dbh->prepare("SELECT enrolmentfee FROM categories WHERE categorycode=?");
$sth->execute($data{'categorycode'});
my ($enrolmentfee) = $sth->fetchrow;
if ($enrolmentfee && $enrolmentfee > 0) {
return $data{'borrowernumber'};
}
+
sub Check_Userid {
my ($uid,$member) = @_;
my $dbh = C4::Context->dbh;
sub ExtendMemberSubscriptionTo {
my ( $borrowerid,$date) = @_;
my $dbh = C4::Context->dbh;
- my $borrower = GetMember($borrowerid,'borrowernumber');
+ my $borrower = GetMember('borrowernumber'=>$borrowerid);
unless ($date){
$date=POSIX::strftime("%Y-%m-%d",localtime());
- my $borrower = GetMember($borrowerid,'borrowernumber');
$date = GetExpiryDate( $borrower->{'categorycode'}, $date );
}
my $sth = $dbh->do(<<EOF);
foreach (@$result){
$_->{date_created} = format_date($_->{date_created});
- my $member = C4::Members::GetMember($_->{borrowernumber});
+ my $member = C4::Members::GetMember(borrowernumber=>$_->{borrowernumber});
$_->{borrowerfirstname} = $member->{firstname};
$_->{borrowersurname} = $member->{surname};
}
my $dbh = C4::Context->dbh;
my $biblio = GetBiblioData($biblionumber);
- my $borrower = C4::Members::GetMember($borrowernumber);
+ my $borrower = C4::Members::GetMember(borrowernumber=>$borrowernumber);
my $controlbranch = C4::Context->preference('ReservesControlBranch');
my $itype = C4::Context->preference('item-level_itypes');
my $reservesrights= 0;
# we retrieve borrowers and items informations #
my $item = GetItem($itemnumber);
- my $borrower = C4::Members::GetMember($borrowernumber);
+ my $borrower = C4::Members::GetMember('borrowernumber'=>$borrowernumber);
# we retrieve user rights on this itemtype and branchcode
my $sth = $dbh->prepare("SELECT categorycode, itemtype, branchcode, reservesallowed
# check if its on issue and if so get the borrower
my $issue = GetItemIssue($item->{'itemnumber'});
- my $borrower = GetMember($issue->{'borrowernumber'},'borrowernumber');
+ my $borrower = GetMember(borrowernumber=>$issue->{'borrowernumber'});
$item->{patron} = $borrower->{'cardnumber'};
my ($whatever, $arrayref) = GetReservesFromBiblionumber($item->{biblionumber});
$item->{hold_queue} = [ sort priority_sort @$arrayref ];
my $self = shift or return;
# return $self->{hold_patron_name} if $self->{hold_patron_name}; TODO: consider caching
my $borrowernumber = (@_ ? shift: $self->hold_patron_id()) or return;
- my $holder = GetMember($borrowernumber, 'borrowernumber');
+ my $holder = GetMember(borrowernumber=>$borrowernumber);
unless ($holder) {
syslog("LOG_ERR", "While checking hold, GetMember failed for borrowernumber '$borrowernumber'");
return;
sub _barcode_to_borrowernumber ($) {
my $known = shift;
(defined($known)) or return undef;
- my $member = GetMember($known,'cardnumber') or return undef;
+ my $member = GetMember(cardnumber=>$known) or return undef;
return $member->{borrowernumber};
}
sub barcode_is_borrowernumber ($$$) { # because hold_queue only has borrowernumber...
my ($class, $patron_id) = @_;
my $type = ref($class) || $class;
my $self;
- $kp = GetMember($patron_id,'cardnumber');
+ $kp = GetMember(cardnumber=>$patron_id);
$debug and warn "new Patron (GetMember): " . Dumper($kp);
unless (defined $kp) {
syslog("LOG_DEBUG", "new ILS::Patron(%s): no such patron", $patron_id);
$self->ok(0);
return $self;
}
- my $borrower = GetMember( $self->{patron}->id, 'cardnumber');
+ my $borrower = GetMember( 'cardnumber'=>$self->{patron}->id);
unless ($borrower) {
$self->screen_msg('No borrower matches cardnumber "' . $self->{patron}->id . '".');
$self->ok(0);
$self->ok(0);
return $self;
}
- my $borrower = GetMember( $self->{patron}->id, 'cardnumber');
+ my $borrower = GetMember( 'cardnumber'=>$self->{patron}->id);
unless ($borrower) {
$self->screen_msg('No borrower matches cardnumber "' . $self->{patron}->id . '".');
$self->ok(0);
$self->ok(0);
return $self;
}
- my $borrower = GetMember( $self->{patron}->id, 'cardnumber');
+ my $borrower = GetMember( 'cardnumber'=>$self->{patron}->id);
unless ($borrower) {
$self->screen_msg('No borrower matches cardnumber "' . $self->{patron}->id . '".');
$self->ok(0);
sub do_renew {
my $self = shift;
- my $borrower = GetMember( $self->{patron}->id, 'cardnumber');
+ my $borrower = GetMember( 'cardnumber'=>$self->{patron}->id);
return $self->do_renew_for($borrower);
}
sub do_renew_all {
my $self = shift;
my $patron = $self->{patron}; # SIP's patron
- my $borrower = GetMember($patron->id, 'cardnumber'); # Koha's patron
+ my $borrower = GetMember('cardnumber'=>$patron->id); # Koha's patron
my $all_ok = 1;
foreach my $itemx (@{$patron->{items}}) {
my $item_id = $itemx->{barcode};
defined($in) or last;
chomp($in);
last unless $in;
- print "GetMember : \n", Dumper(GetMember($in, 'cardnumber'));
+ print "GetMember : \n", Dumper(GetMember('cardnumber'=>$in));
my ($member) = GetMemberDetails(undef, $in);
my $flags = $member->{authflags};
print "GetMemberDetails (member) : \n", Dumper($member);
--- /dev/null
+package C4::SQLHelper;
+
+# Copyright 2009 Biblibre SARL
+#
+# This file is part of Koha.
+#
+# Koha is free software; you can redistribute it and/or modify it under the
+# terms of the GNU General Public License as published by the Free Software
+# Foundation; either version 2 of the License, or (at your option) any later
+# version.
+#
+# Koha is distributed in the hope that it will be useful, but WITHOUT ANY
+# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
+# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License along with
+# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
+# Suite 330, Boston, MA 02111-1307 USA
+
+
+use List::MoreUtils qw(first_value);
+use C4::Context;
+use C4::Dates qw(format_date_in_iso);
+use C4::Debug;
+use strict;
+use warnings;
+require Exporter;
+use vars qw($VERSION @ISA @EXPORT_OK %EXPORT_TAGS);
+
+BEGIN {
+ # set the version for version checking
+ $VERSION = 0.5;
+ require Exporter;
+ @ISA = qw(Exporter);
+@EXPORT_OK=qw(
+ InsertInTable
+ SearchInTable
+ UpdateInTable
+ GetPrimaryKey
+);
+ %EXPORT_TAGS = ( all =>[qw( InsertInTable SearchInTable UpdateInTable GetPrimaryKey)]
+ );
+}
+
+my $tablename;
+my $hash;
+
+=head1 NAME
+
+C4::SQLHelper - Perl Module containing convenience functions for SQL Handling
+
+=head1 SYNOPSIS
+
+use C4::SQLHelper;
+
+=head1 DESCRIPTION
+
+This module contains routines for adding, modifying and Searching Data in MysqlDB
+
+=head1 FUNCTIONS
+
+=over 2
+
+=back
+
+
+=head2 SearchInTable
+
+=over 4
+
+ $hashref = &SearchInTable($tablename,$data, $orderby);
+
+=back
+
+$data may contain
+ - string
+ - data_hashref : will be considered as an AND of all the data searched
+ - data_array_ref on hashrefs : Will be considered as an OR of Datahasref elements
+
+$orderby is a hashref with fieldnames as key and 0 or 1 as values (ASCENDING or DESCENDING order)
+
+=cut
+
+sub SearchInTable{
+ my ($tablename,$filters,$orderby) = @_;
+ my $dbh = C4::Context->dbh;
+ my $sql = "SELECT * from $tablename";
+ my $row;
+ my $sth;
+ my ($keys,$values)=_filter_fields($filters,$tablename, "search");
+ if ($filters) {
+ $sql.= do { local $"=' AND ';
+ qq{ WHERE @$keys }
+ };
+ }
+ if ($orderby){
+ my @orders=map{ "$_".($$orderby{$_}? " DESC" : "") } keys %$orderby;
+ $sql.= do { local $"=', ';
+ qq{ ORDER BY @orders}
+ };
+ }
+
+ $debug && warn $sql," ",join(",",@$values);
+ $sth = $dbh->prepare($sql);
+ $sth->execute(@$values);
+ my $results = $sth->fetchall_arrayref( {} );
+ return $results;
+}
+
+=head2 InsertInTable
+
+=over 4
+
+ $data_id_in_table = &InsertInTable($tablename,$data_hashref);
+
+=back
+
+ Insert Data in table
+ and returns the id of the row inserted
+=cut
+
+sub InsertInTable{
+ my ($tablename,$data) = @_;
+ my $dbh = C4::Context->dbh;
+ my ($keys,$values)=_filter_fields($data,$tablename);
+
+ my $query = do { local $"=',';
+ qq{
+ INSERT $tablename
+ SET @$keys
+ };
+ };
+
+ $debug && warn $query, join(",",@$values);
+ my $sth = $dbh->prepare($query);
+ $sth->execute( @$values);
+
+ return $dbh->last_insert_id(undef, undef, $tablename, undef);
+}
+
+=head2 UpdateInTable
+
+=over 4
+
+ $status = &UpdateInTable($tablename,$data_hashref);
+
+=back
+
+ Update Data in table
+ and returns the status of the operation
+=cut
+
+sub UpdateInTable{
+ my ($tablename,$data) = @_;
+ my $field_id=GetPrimaryKey($tablename);
+ my $id=$$data{$field_id};
+ my $dbh = C4::Context->dbh;
+ my ($keys,$values)=_filter_fields($data,$tablename);
+
+ my $query = do { local $"=',';
+ qq{
+ UPDATE $tablename
+ SET @$keys
+ WHERE $field_id=?
+ };
+ };
+ $debug && warn $query, join(",",@$values,$id);
+
+ my $sth = $dbh->prepare($query);
+ return $sth->execute( @$values,$id);
+
+}
+
+=head2 GetPrimaryKey
+
+=over 4
+
+ $primarykeyname = &GetPrimaryKey($tablename)
+
+=back
+
+ Get the Primary Key field name of the table
+=cut
+
+sub GetPrimaryKey($) {
+ my $tablename=shift;
+ my $hash_columns=_columns($tablename);
+ return first_value { $$hash_columns{$_}{'Key'} =~/PRI/} keys %$hash_columns;
+}
+
+=head2 _get_columns
+
+=over 4
+
+_get_columns($tablename)
+
+=back
+
+Given a tablename
+Returns a hashref of all the fieldnames of the table
+With
+ Key
+ Type
+ Default
+
+=cut
+
+sub _columns($) {
+ my $tablename=shift;
+ $debug && warn $tablename;
+ my $dbh=C4::Context->dbh;
+ my $sth=$dbh->prepare(qq{SHOW COLUMNS FROM $tablename });
+ $sth->execute;
+ return $sth->fetchall_hashref(qw(Field));
+}
+
+=head2 _filter_fields
+
+=over 4
+
+_filter_fields
+
+=back
+
+Given
+ - a tablename
+ - a hashref of data
+ - an indicator on operation
+
+Returns a ref of key array to use in SQL functions
+and a ref to value array
+
+=cut
+
+sub _filter_fields{
+ my ($data_to_filter,$tablename,$research)=@_;
+ warn "$tablename";
+ my @keys;
+ my @values;
+ my $columns= _columns($tablename);
+ #Filter Primary Keys of table
+ my $elements=join "|",grep {$$columns{$_}{'Key'} ne "PRI"} keys %$columns;
+ if (ref($data_to_filter) eq "HASH"){
+ foreach my $field (grep {/\b($elements)\b/} keys %$data_to_filter){
+ ## supposed to be a hash of simple values, hashes of arrays could be implemented
+ $$data_to_filter{$field}=format_date_in_iso($$data_to_filter{$field}) if ($$columns{$field}{Type}=~/date/ && $$data_to_filter{$field} !~C4::Dates->regexp("iso"));
+ my $strkeys= " $field = ? ";
+ if ($field=~/code/ && $research){
+ $strkeys="( $strkeys OR $field='' OR $field IS NULL) ";
+ }
+ push @values, $$data_to_filter{$field};
+ push @keys, $strkeys;
+ }
+ } elsif (ref($data_to_filter) eq "ARRAY"){
+ foreach my $element (@$data_to_filter){
+ my (@localkeys,@localvalues)=_filter_fields($element);
+ push @keys, join(' AND ',@localkeys);
+ push @values, @localvalues;
+ }
+ }
+ else{
+ my @operands=split / /,$data_to_filter;
+ foreach my $operand (@operands){
+ my @localvalues=($operand,"\%$operand\%") ;
+ foreach my $field (keys %$columns){
+ my $strkeys= " ( $field = ? OR $field LIKE ? )";
+ if ($field=~/code/){
+ $strkeys="( $strkeys OR $field='' OR $field IS NULL) ";
+ }
+ push @values, @localvalues;
+ push @keys, $strkeys;
+ }
+ }
+ }
+
+ return (\@keys,\@values);
+}
+
+1;
+
} elsif ( $op eq 'modif' ) {
my ( $shelfnumber2, $shelfname, $owner, $category, $sortfield ) =GetShelf( $shelfnumber );
- my $member = GetMember($owner,'borrowernumber');
+ my $member = GetMember('borrowernumber'=>$owner);
my $ownername = defined($member) ? $member->{firstname} . " " . $member->{surname} : '';
$template->param(
edit => 1,
if ($owner eq $loggedinuser or $canmanage) {
$shelflist->{$element}->{'mine'} = 1;
}
- my $member = GetMember($owner,'borrowernumber');
+ my $member = GetMember('borrowernumber' => $owner);
$shelflist->{$element}->{ownername} = defined($member) ? $member->{firstname} . " " . $member->{surname} : '';
$numberCanManage++ if $canmanage; # possibly outmoded
if ($shelflist->{$element}->{'category'} eq '1') {
);
# budget
-my $borrower= GetMember($loggedinuser);
+my $borrower= GetMember('borrowernumber' => $loggedinuser);
my ( $flags, $homebranch )= ($borrower->{'flags'},$borrower->{'branchcode'});
my @results = GetBookFunds($homebranch);
$template->param( branchloop => \@branchloop , itypeloop => \@itemtypesloop );
# build bookfund list
-my $borrower= GetMember($loggedinuser);
+my $borrower= GetMember('borrowernumber' => $loggedinuser);
my ( $flags, $homebranch )= ($borrower->{'flags'},$borrower->{'branchcode'});
my @select_bookfund;
elsif ( $code eq 'WasReturned' ) {
$err{errwasreturned} = 1;
$err{borrowernumber} = $messages->{'WasReturned'};
- my $borrower = GetMember($messages->{'WasReturned'},'borrowernumber');
+ my $borrower = GetMember('borrowernumber'=>$messages->{'WasReturned'});
$err{title} = $borrower->{'title'};
$err{firstname} = $borrower->{'firstname'};
$err{surname} = $borrower->{'surname'};
my $action = $input->param('action') || '';
#get borrower details
-my $data=GetMember($borrowernumber,'borrowernumber');
+my $data=GetMember('borrowernumber' => $borrowernumber);
if ( $action eq 'reverse' ) {
ReversePayment( $borrowernumber, $input->param('accountno') );
my $description=$input->param('description');
my $category_type=$input->param('category_type');
-$data=GetMember($borrowernumber,'borrowernumber');
+$data=GetMember('borrowernumber' => $borrowernumber);
$template->param( borrowernumber => $borrowernumber,#register number
#transform value in capital or capital for first letter of the word
my $borrowernumber=$input->param('borrowernumber');
#get borrower details
-my $data=GetMember($borrowernumber,'borrowernumber');
+my $data=GetMember('borrowernumber' => $borrowernumber);
my $add=$input->param('add');
if ($add){
# get borrower details
-my $data=GetMember($borrowernumber,'borrowernumber');
+my $data=GetMember('borrowernumber'=>$borrowernumber);
my $add=$input->param('add');
if ($add){
# print $input->header;
my $cardnumber = $input->param('cardnumber');
my $destination = $input->param('destination');
my $errormsg;
-my ($bor)=GetMember($member);
+my ($bor)=GetMember('borrowernumber' => $member);
if(( $member ne $loggedinuser ) && ($bor->{'category_type'} eq 'S' ) ) {
$errormsg = 'NOPERMISSION' unless($staffflags->{'superlibrarian'} || $staffflags->{'staffaccess'} );
# need superlibrarian for koha-conf.xml fakeuser.
}
$template->param("add"=>1) if ($op eq 'add');
$template->param("checked" => 1) if (defined($nodouble) && $nodouble eq 1);
-($borrower_data = GetMember($borrowernumber,'borrowernumber')) if ($op eq 'modify' or $op eq 'save');
+($borrower_data = GetMember( 'borrowernumber'=>$borrowernumber )) if ($op eq 'modify' or $op eq 'save');
my $categorycode = $input->param('categorycode') || $borrower_data->{'categorycode'};
my $category_type = $input->param('category_type');
my $new_c_type = $category_type; #if we have input param, then we've already chosen the cat_type.
( $category_type eq 'C' || $category_type eq 'P' ) and
$guarantorid ne '' and
$guarantorid ne '0' ) {
- if (my $guarantordata=GetMember($guarantorid)) {
+ if (my $guarantordata=GetMember(borrowernumber => $guarantorid)) {
$guarantorinfo=$guarantordata->{'surname'}." , ".$guarantordata->{'firstname'};
if ( !defined($data{'contactname'}) or $data{'contactname'} eq '' or
$data{'contactname'} ne $guarantordata->{'surname'} ) {
}
);
my $borrowernumber = $query->param('borrowernumber');
-my $borrower = GetMember( $borrowernumber ,'borrowernumber');
+my $borrower = GetMember( 'borrowernumber' => $borrowernumber );
my $branch = C4::Context->userenv->{'branch'};
$template->param( $borrower );
my $borrowernumber = $input->param('borrowernumber');
#start the page and read in includes
-my $data = GetMember( $borrowernumber ,'borrowernumber');
+my $data = GetMember( 'borrowernumber' => $borrowernumber );
my $reregistration = $input->param('reregistration');
if ( not defined $data ) {
my $catcode;
if ( $category_type eq 'C') {
if ($data->{'guarantorid'} ne '0' ) {
- my $data2 = GetMember( $data->{'guarantorid'} ,'borrowernumber');
+ my $data2 = GetMember( 'borrowernumber' => $data->{'guarantorid'} );
foreach (qw(address city B_address B_city phone mobile zipcode country B_country)) {
$data->{$_} = $data2->{$_};
}
}
else {
if ($data->{'guarantorid'}){
- my ($guarantor) = GetMember( $data->{'guarantorid'},'biblionumber');
+ my ($guarantor) = GetMember( 'borrowernumber' =>$data->{'guarantorid'});
$template->param(guarantor => 1);
foreach (qw(borrowernumber cardnumber firstname surname)) {
$template->param("guarantor$_" => $guarantor->{$_});
my $borrowernumber = $input->param('borrowernumber');
#get borrower details
-my $borrower = GetMember(borrowernumber=>$borrowernumber);
+my $borrower = GetMember(borrowernumber => $borrowernumber);
my ($template, $loggedinuser, $cookie)
= get_template_and_user({template_name => "members/notices.tmpl",
}
# get borrower details
-my $data = GetMember( $borrowernumber,'borrowernumber' );
+my $data = GetMember( borrowernumber => $borrowernumber );
my $user = $input->remote_user;
# get account details
my $borrowernumber=$input->param('borrowernumber');
#get borrower details
-my $data=GetMember($borrowernumber,'borrowernumber');
+my $data=GetMember('borrowernumber'=>$borrowernumber);
my $order=$input->param('order') || '';
my $order2=$order;
if ($order2 eq ''){
}
elsif ( $op eq 'update' ) {
- my $member = GetMember($borrowernumber);
+ my $member = GetMember('borrowernumber'=>$borrowernumber);
$member->{'guarantorid'} = '0';
$member->{'categorycode'} = $catcode;
my $borcat = GetBorrowercategory($catcode);
my $barcode = $item->{barcode};
my $itemcallnumber = $item->{itemcallnumber};
- my $borrower = GetMember($borrowernumber);
+ my $borrower = GetMember('borrowernumber'=>$borrowernumber);
my $cardnumber = $borrower->{'cardnumber'};
my $surname = $borrower->{'surname'};
my $firstname = $borrower->{'firstname'};
$circ->{ 'barcode' } = barcodedecode($circ->{'barcode'}) if( $circ->{'barcode'} && C4::Context->preference('itemBarcodeInputFilter'));
my $branchcode = C4::Context->userenv->{branch};
- my $borrower = GetMember( $circ->{ 'cardnumber' }, 'cardnumber' );
+ my $borrower = GetMember( 'cardnumber'=>$circ->{ 'cardnumber' } );
my $item = GetBiblioFromItemNumber( undef, $circ->{ 'barcode' } );
my $issue = GetItemIssue( $item->{'itemnumber'} );
#warn( Data::Dumper->Dump( [ $circ, $item ], [ qw( circ item ) ] ) );
my $borrowernumber = _get_borrowernumber_from_barcode( $circ->{'barcode'} );
if ( $borrowernumber ) {
- my $borrower = GetMember( $borrowernumber, 'borrowernumber' );
+ my $borrower = GetMember( 'borrowernumber' =>$borrowernumber );
C4::Circulation::MarkIssueReturned( $borrowernumber,
$item->{'itemnumber'},
undef,
sub kocMakePayment {
my ( $circ ) = @_;
- my $borrower = GetMember( $circ->{ 'cardnumber' }, 'cardnumber' );
+ my $borrower = GetMember( 'cardnumber'=>$circ->{ 'cardnumber' } );
recordpayment( $borrower->{'borrowernumber'}, $circ->{'amount'} );
push( @output, { payment => 1,
amount => $circ->{'amount'},
my $reviews = getreviews( $biblionumber, 1 );
foreach ( @$reviews ) {
my $borrower_number_review = $_->{borrowernumber};
- my $borrowerData = GetMember($borrower_number_review,'borrowernumber');
+ my $borrowerData = GetMember('borrowernumber' =>$borrower_number_review);
# setting some borrower info into this hash
$_->{title} = $borrowerData->{'title'};
$_->{surname} = $borrowerData->{'surname'};
my $reviews = getreviews( $biblionumber, 1 );
my $loggedincommenter;
foreach ( @$reviews ) {
- my $borrowerData = GetMember($_->{borrowernumber},'borrowernumber');
+ my $borrowerData = GetMember('borrowernumber' => $_->{borrowernumber});
# setting some borrower info into this hash
$_->{title} = $borrowerData->{'title'};
$_->{surname} = $borrowerData->{'surname'};
}
);
-my $borrower = GetMember( $borrowernumber, 'borrowernumber' );
+my $borrower = GetMember( 'borrowernumber'=> $borrowernumber );
$template->param(
textmessaging => $borrower->{textmessaging},
);
my @rank=$input->param('rank-request');
my $type=$input->param('type');
my $title=$input->param('title');
-my $borrowernumber=GetMember($borrower,'cardnumber');
+my $borrowernumber=GetMember('cardnumber'=>$borrower);
my $checkitem=$input->param('checkitem');
my $multi_hold = $input->param('multi_hold');
foreach ( @$reviews ) {
my $borrowernumber = $_->{borrowernumber};
- my $borrowerData = GetMember($borrowernumber,'borrowernumber');
+ my $borrowerData = GetMember('borrowernumber' => $borrowernumber);
my $biblioData = GetBiblioData($_->{biblionumber});
# setting some borrower info into this hash
$_->{bibliotitle} = $biblioData->{'title'};
my @results;
my $data;
for(my $i=0;$i<$routing;$i++){
- $data=GetMember($routinglist[$i]->{'borrowernumber'},'borrowernumber');
+ $data=GetMember('borrowernumber' => $routinglist[$i]->{'borrowernumber'});
$data->{'location'}=$data->{'branchcode'};
$data->{'name'}="$data->{'firstname'} $data->{'surname'}";
$data->{'routingid'}=$routinglist[$i]->{'routingid'};
my @results;
my $data;
for(my $i=0;$i<$routing;$i++){
- $data=GetMember($routinglist[$i]->{'borrowernumber'},'borrowernumber');
+ $data=GetMember('borrowernumber' => $routinglist[$i]->{'borrowernumber'});
$data->{'location'}=$data->{'branchcode'};
if ($data->{firstname} ) {
$data->{name} = $data->{firstname} . q| |;
--- /dev/null
+#!/usr/bin/perl
+#
+# This Koha test module is a stub!
+# Add more tests here!!!
+
+use strict;
+use warnings;
+use Data::Dumper;
+
+use C4::SQLHelper qw(:all);
+
+use Test::More tests => 3;
+
+BEGIN {
+ use_ok('C4::SQLHelper');
+}
+
+my $borrid=InsertInTable("borrowers",{firstname=>"Jean",surname=>"Valjean",city=>" ",zipcode=>" ",email=>"email",categorycode=>"EL"});
+my $status=UpdateInTable("borrowers",{borrowernumber=>$borrid,firstname=>"Jean",surname=>"Valjean",city=>"ma6tVaCracker ",zipcode=>" ",email=>"email"});
+my $borrowers=SearchInTable("borrowers",{firstname=>"Jean"});
sub enable_independant_branches {
my $self = shift;
- my $member = GetMember( $self->{'memberid'} );
+ my $member = GetMember( 'borrowernumber' =>$self->{'memberid'} );
C4::Context::set_userenv( 0, # usernum
$self->{'memberid'}, # userid
}
# provide some branch information
- my $member = GetMember( $self->{'memberid'} );
+ my $member = GetMember( borrowernumber=>$self->{'memberid'} );
# diag( Data::Dumper->Dump( [ $member ], [ 'member' ] ) );
{
my @orders = GetLateOrders( 0, $self->{'booksellerid'}, $member->{'branchcode'} );
GetAge
get_institutions
add_member_orgs
- GetCities
- GetSortDetails
MoveMemberToDeleted
DelMember
ExtendMemberSubscriptionTo
- GetRoadTypes
GetTitles
GetPatronImage
PutPatronImage
RmPatronImage
- GetRoadTypeDetails
GetBorrowersWhoHaveNotBorrowedSince
GetBorrowersWhoHaveNeverBorrowed
GetBorrowersWithIssuesHistoryOlderThan
address => 'address' . $self->random_string(),
city => 'city' . $self->random_string(),
cardnumber => 'card' . $self->random_string(),
- branchcode => 'CPL',
- categorycode => 'B', # B => Board
+ branchcode => 'U1BCG',
+ categorycode => 'D', # B => Board
dateexpiry => '2020-01-01',
password => 'testpassword',
userid => 'testuser',
#search by borrowernumber
my $results =
- C4::Members::GetMember( $self->{get_new_borrowernumber}, 'borrowernumber' );
+ C4::Members::GetMember( borrowernumber=>$self->{get_new_borrowernumber});
ok( $results, 'we successfully called GetMember searching by borrowernumber' );
ok( exists $results->{borrowernumber},
"we have a valid cardnumber $self->{get_new_cardnumber} to test with" );
#search by cardnumber
- my $results = C4::Members::GetMember( $self->{get_new_cardnumber}, 'cardnumber' );
+ my $results = C4::Members::GetMember( 'cardnumber'=>$self->{get_new_cardnumber} );
ok( $results, 'we successfully called GetMember searching by cardnumber' );
ok( exists $results->{cardnumber}, 'member details has a "cardnumber" attribute' );
"we have a valid firstname $self->{get_new_firstname} to test with" );
##search by firstname
- my $results = C4::Members::GetMember( $self->{get_new_firstname}, 'firstname' );
+ my $results = C4::Members::GetMember( 'firstname'=>$self->{get_new_firstname} );
ok( $results, 'we successfully called GetMember searching by firstname' );
ok( exists $results->{firstname}, 'member details has a "firstname" attribute' );
"we have a valid userid $self->{get_new_userid} to test with" );
#search by userid
- my $results = C4::Members::GetMember( $self->{get_new_userid}, 'userid' );
+ my $results = C4::Members::GetMember( 'userid'=>$self->{get_new_userid} );
ok( $results, 'we successfully called GetMember searching by userid' );
ok( exists $results->{'userid'}, 'member details has a "userid" attribute' );
# diag( Data::Dumper->Dump( [ $item_from_barcode ], [ 'item_from_barcode' ] ) );
ok( $self->{'memberid'}, 'memberid' );
- my $borrower = C4::Members::GetMember( $self->{'memberid'} );
+ my $borrower = C4::Members::GetMember( borrowernumber=>$self->{'memberid'} );
ok( $borrower->{'borrowernumber'}, 'borrowernumber' );
my ( $issuingimpossible, $needsconfirmation ) = C4::Circulation::CanBookBeIssued( $borrower, $item->{'barcode'}, $duedate, 0 );
# my $item_from_barcode = C4::Items::GetItem( undef, $item->{'barcode'} );
# diag( Data::Dumper->Dump( [ $item_from_barcode ], [ 'item_from_barcode' ] ) );
- my $borrower = C4::Members::GetMember( $self->{'memberid'} );
+ my $borrower = C4::Members::GetMember( borrowernumber => $self->{'memberid'} );
ok( $borrower->{'borrowernumber'}, 'borrowernumber' );
my ( $issuingimpossible, $needsconfirmation ) = C4::Circulation::CanBookBeIssued( $borrower, $item->{'barcode'}, $duedate, 0 );
my $borrowernumber;
my $member;
if ( ($matchpoint eq 'cardnumber') && ($borrower{'cardnumber'}) ) {
- $member = GetMember( $borrower{'cardnumber'}, 'cardnumber' );
+ $member = GetMember( 'cardnumber' => $borrower{'cardnumber'} );
if ($member) {
$borrowernumber = $member->{'borrowernumber'};
}
if ($src eq 'circ') { # if we were called from circulation, use the circulation menu and get data to populate it -fbcit
use C4::Members;
my $borrowernumber = $object;
- my $data = GetMember($borrowernumber,'borrowernumber');
+ my $data = GetMember('borrowernumber'=>$borrowernumber);
my ($picture, $dberror) = GetPatronImage($data->{'cardnumber'});
$template->param( picture => 1 ) if $picture;
$template->param( menu => 1,