Bug 7697: Remove unused textmessaging parameter from OPAC main/user
authorIan Walls <koha.sekjal@gmail.com>
Sat, 10 Mar 2012 17:04:20 +0000 (12:04 -0500)
committerPaul Poulain <paul.poulain@biblibre.com>
Tue, 20 Mar 2012 15:49:36 +0000 (16:49 +0100)
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>
opac/opac-main.pl
opac/opac-user.pl

index 61d288a..5d6c2e1 100755 (executable)
@@ -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);
index a88bffb..5b18cae 100755 (executable)
@@ -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,