# 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
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
use strict;
+use warnings;
use CGI;
use Mail::Sendmail;
template_name => "opac-userupdate.tmpl",
query => $query,
type => "opac",
- authnotrequired => 0,
+ authnotrequired => ( C4::Context->preference("OpacPublic") ? 1 : 0 ),
flagsrequired => { borrow => 1 },
debug => 1,
}
# get borrower information ....
my ( $borr ) = GetMemberDetails( $borrowernumber );
+my ( $patronemail ) = GetFirstValidEmailAddress($borrowernumber);
my $lib = GetBranchDetail($borr->{'branchcode'});
# handle the new information....
# collect the form values and send an email.
my @fields = (
- 'surname', 'firstname', 'phone',
- 'fax', 'address','address2','city','zipcode','phone','mobile','fax','phonepro', 'emailaddress','B_streetaddress','B_city','B_zipcode','dateofbirth','sex'
+ 'surname','firstname','othernames','streetnumber','address','address2','city','state','zipcode','country','phone','mobile','fax','phonepro', 'emailaddress','emailpro','B_streetnumber','B_address','B_address2','B_city','B_state','B_zipcode','B_country','B_phone','B_email','dateofbirth','sex'
);
my $update;
my $updateemailaddress = $lib->{'branchemail'};
$updateemailaddress = C4::Context->preference('KohaAdminEmailAddress') unless( $updateemailaddress =~ /\w+@\w+/);
-if ( $updateemailaddress eq '' ) {
+if ( !$updateemailaddress || $updateemailaddress eq '' ) {
warn
"KohaAdminEmailAddress system preference not set. Couldn't send patron update information for $borr->{'firstname'} $borr->{'surname'} (#$borrowernumber)\n";
my ($template) = get_template_and_user(
);
$template->param(
- errormessage => 'KohaAdminEmailAddress system preference
- is not set. Please visit the library to update your user record'
+ noadminemail => 1,
);
output_html_with_http_headers $query, $cookie, $template->output;
exit;
}
-if ( $query->{'modify'} ) {
+if ( !$patronemail || $patronemail eq '' ) {
+ $patronemail = $updateemailaddress;
+};
+
+if ( $query->param('modify') ) {
# get all the fields:
my $message = <<"EOF";
-Borrower $borr->{'cardnumber'}
+Patron $borr->{'cardnumber'}
has requested to change her/his personal details.
Please check these new details and make the changes:
EOF
+
+ my $streetnumber = $borr->{'streetnumber'} || '';
+ my $address = $borr->{'address'} || '';
+ my $address2 = $borr->{'address2'} || '';
+ my $B_streetnumber = $borr->{'B_streetnumber'} || '';
+ my $B_address = $borr->{'B_address'} || '';
+ my $B_address2 = $borr->{'B_address2'} || '';
+
foreach my $field (@fields) {
- my $newfield = $query->param($field);
- $message .= "$field : $borr->{$field} --> $newfield\n";
+ my $newfield = $query->param($field) || '';
+ my $borrowerfield = '';
+ if($borr->{$field}) {
+ $borrowerfield = $borr->{$field};
+ }
+
+ if($field eq "dateofbirth") {
+ $borrowerfield = format_date( $borr->{'dateofbirth'} ) || '';
+ }
+
+ if($borrowerfield eq $newfield) {
+ $message .= "$field : $borrowerfield --> $newfield\n";
+ } else {
+ $message .= uc($field) . " : $borrowerfield --> $newfield\n";
+ }
}
$message .= "\n\nThanks,\nKoha\n\n";
my %mail = (
To => $updateemailaddress,
- From => $updateemailaddress,
+ From => $patronemail,
Subject => "User Request for update of Record.",
Message => $message,
'Content-Type' => 'text/plain; charset="utf8"',
if ( sendmail %mail ) {
# do something if it works....
- warn "Mail sent ok\n";
print $query->redirect('/cgi-bin/koha/opac-user.pl?patronupdate=sent');
exit;
}
}
$borr->{'dateenrolled'} = format_date( $borr->{'dateenrolled'} );
-$borr->{'dateexpiry'} = format_date( $borr->{'dateexpiry'} );
+$borr->{'dateexpiry'} = format_date( $borr->{'dateexpiry'} );
$borr->{'dateofbirth'} = format_date( $borr->{'dateofbirth'} );
$borr->{'ethnicity'} = fixEthnicity( $borr->{'ethnicity'} );
+$borr->{'branchname'} = GetBranchName($borr->{'branchcode'});
if (C4::Context->preference('ExtendedPatronAttributes')) {
my $attributes = C4::Members::Attributes::GetBorrowerAttributes($borrowernumber, 'opac');