From: Ian Walls Date: Sat, 10 Mar 2012 17:04:20 +0000 (-0500) Subject: Bug 7697: Remove unused textmessaging parameter from OPAC main/user X-Git-Url: http://git.rot13.org/?a=commitdiff_plain;h=f7a7b34bdfe85d00eaea42cc4b5462e3a8633ccc;p=koha.git Bug 7697: Remove unused textmessaging parameter from OPAC main/user Removes the textmessaging parameter from opac/opac-main.pl and opac/opac-user.pl, since it's not utilized anywhere in the templates, and is sourced from a field that no longer exists in the borrower table (textmessaging). This also removes a dependency on C4/Members.pm from opac-main.pl To Test: Login to the OPAC, and view both opac-user.pl and opac-main.pl. Both pages should still render properly. Signed-off-by: Jared Camins-Esakov Signed-off-by: Paul Poulain --- diff --git a/opac/opac-main.pl b/opac/opac-main.pl index 61d288af0f..5d6c2e1c9a 100755 --- a/opac/opac-main.pl +++ b/opac/opac-main.pl @@ -21,7 +21,6 @@ use warnings; use CGI; use C4::Auth; # get_template_and_user use C4::Output; -use C4::Members; # GetMember use C4::NewsChannels; # get_opac_news use C4::Languages qw(getTranslatedLanguages accept_language); @@ -44,11 +43,6 @@ $template->param( ); -my $borrower = GetMember( borrowernumber=>$borrowernumber ); -$template->param( - textmessaging => $borrower->{textmessaging}, -) if (ref($borrower) eq "HASH"); - # display news # use cookie setting for language, bug default to syspref if it's not set my ($theme, $news_lang) = C4::Templates::themelanguage(C4::Context->config('opachtdocs'),'opac-main.tt','opac',$input); diff --git a/opac/opac-user.pl b/opac/opac-user.pl index a88bffbae7..5b18cae4c0 100755 --- a/opac/opac-user.pl +++ b/opac/opac-user.pl @@ -362,7 +362,6 @@ if ( $borr->{'opacnote'} ) { $template->param( bor_messages_loop => GetMessages( $borrowernumber, 'B', 'NONE' ), waiting_count => $wcount, - textmessaging => $borr->{textmessaging}, patronupdate => $patronupdate, OpacRenewalAllowed => C4::Context->preference("OpacRenewalAllowed"), userview => 1,