Koha-community/Koha
13
7
Fork 0
Code Issues Releases Wiki Activity

Bug 7697: Remove unused textmessaging parameter from OPAC main/user

Browse source 
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 <jcamins@cpbibliography.com>
Signed-off-by: Paul Poulain <paul.poulain@biblibre.com>
This commit is contained in:
Ian Walls 2012-03-10 12:04:20 -05:00 committed by Paul Poulain
parent 692ee0e9da
commit f7a7b34bdf
2 changed files with 0 additions and 7 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
opac/opac-main.pl
View file

@ -21,7 +21,6 @@ use warnings;
use CGI; use CGI;
use C4::Auth; # get_template_and_user use C4::Auth; # get_template_and_user
use C4::Output; use C4::Output;
use C4::Members; # GetMember
use C4::NewsChannels; # get_opac_news use C4::NewsChannels; # get_opac_news
use C4::Languages qw(getTranslatedLanguages accept_language); 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 # display news
# use cookie setting for language, bug default to syspref if it's not set # 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); my ($theme, $news_lang) = C4::Templates::themelanguage(C4::Context->config('opachtdocs'),'opac-main.tt','opac',$input);

1
opac/opac-user.pl
View file

@ -362,7 +362,6 @@ if ( $borr->{'opacnote'} ) {
$template->param( $template->param(
bor_messages_loop => GetMessages( $borrowernumber, 'B', 'NONE' ), bor_messages_loop => GetMessages( $borrowernumber, 'B', 'NONE' ),
waiting_count => $wcount, waiting_count => $wcount,
textmessaging => $borr->{textmessaging},
patronupdate => $patronupdate, patronupdate => $patronupdate,
OpacRenewalAllowed => C4::Context->preference("OpacRenewalAllowed"), OpacRenewalAllowed => C4::Context->preference("OpacRenewalAllowed"),
userview => 1, userview => 1,