Bugzilla – Attachment 8147 Details for
Bug 7697
Clean up borrower.textmessaging leftover code
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 7697: Remove unused textmessaging parameter from OPAC main/user
Bug-7697-Remove-unused-textmessaging-parameter-fro.patch (text/plain), 2.01 KB, created by
Ian Walls
on 2012-03-10 17:06:48 UTC
(
hide
)
Description:
Bug 7697: Remove unused textmessaging parameter from OPAC main/user
Filename:
MIME Type:
Creator:
Ian Walls
Created:
2012-03-10 17:06:48 UTC
Size:
2.01 KB
patch
obsolete
>From 99703d077b05404c43f46d198a26df102bb13b99 Mon Sep 17 00:00:00 2001 >From: Ian Walls <koha.sekjal@gmail.com> >Date: Sat, 10 Mar 2012 12:04:20 -0500 >Subject: [PATCH] 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. >--- > opac/opac-main.pl | 6 ------ > opac/opac-user.pl | 1 - > 2 files changed, 0 insertions(+), 7 deletions(-) > >diff --git a/opac/opac-main.pl b/opac/opac-main.pl >index 61d288a..5d6c2e1 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 a88bffb..5b18cae 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, >-- >1.7.4.1
You cannot view the attachment while viewing its details because your browser does not support IFRAMEs.
View the attachment on a separate page
.
View Attachment As Diff
View Attachment As Raw
Actions:
View
|
Diff
|
Splinter Review
Attachments on
bug 7697
:
8147
|
8149