Bugzilla – Attachment 63800 Details for
Bug 18595
Move C4::Members::Messaging to Koha namespace
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 18595: Remove references to C4::Members::Messaging
Bug-18595-Remove-references-to-C4MembersMessaging.patch (text/plain), 7.76 KB, created by
Lari Taskula
on 2017-05-29 11:18:02 UTC
(
hide
)
Description:
Bug 18595: Remove references to C4::Members::Messaging
Filename:
MIME Type:
Creator:
Lari Taskula
Created:
2017-05-29 11:18:02 UTC
Size:
7.76 KB
patch
obsolete
>From 0d993bf0c1bff776ede45c0468ccc7355435461e Mon Sep 17 00:00:00 2001 >From: Lari Taskula <lari.taskula@jns.fi> >Date: Thu, 27 Oct 2016 16:25:05 +0300 >Subject: [PATCH] Bug 18595: Remove references to C4::Members::Messaging > >$ grep -r 'C4::Members::Messaging' > >This command should not produce any results in Koha source code after this >patch. > >To test: >1. Check the pages touched by this commit >2. Try setting and viewing patron and patron category messaging preferences > >Finally, run unit tests: >1. prove t/db_dependent/Circulation.t >2. prove t/db_dependent/Reserves.t >3. prove t/db_dependent/Koha/Patron/Categories.t >--- > C4/Auth_with_ldap.pm | 1 - > C4/Form/MessagingPreferences.pm | 3 +- > C4/Members/Messaging.pm | 97 ---------------------- > C4/Message.pm | 2 +- > C4/Reserves.pm | 1 - > Koha/Patron/Category.pm | 2 - > circ/returns.pl | 1 - > misc/cronjobs/advance_notices.pl | 1 - > .../maintenance/borrowers-force-messaging-defaults | 1 - > opac/opac-messaging.pl | 1 - > t/Members_Messaging.t | 14 ---- > tools/import_borrowers.pl | 1 - > 12 files changed, 2 insertions(+), 123 deletions(-) > delete mode 100644 C4/Members/Messaging.pm > delete mode 100755 t/Members_Messaging.t > >diff --git a/C4/Auth_with_ldap.pm b/C4/Auth_with_ldap.pm >index 00732a7..817a66f 100644 >--- a/C4/Auth_with_ldap.pm >+++ b/C4/Auth_with_ldap.pm >@@ -26,7 +26,6 @@ use C4::Context; > use C4::Members qw(AddMember); > use C4::Members::Attributes; > use C4::Members::AttributeTypes; >-use C4::Members::Messaging; > use C4::Auth qw(checkpw_internal); > use Koha::Patrons; > use Koha::AuthUtils qw(hash_password); >diff --git a/C4/Form/MessagingPreferences.pm b/C4/Form/MessagingPreferences.pm >index 28c073b..1aaedd5 100644 >--- a/C4/Form/MessagingPreferences.pm >+++ b/C4/Form/MessagingPreferences.pm >@@ -22,7 +22,6 @@ use warnings; > > use CGI qw ( -utf8 ); > use C4::Context; >-use C4::Members::Messaging; > use C4::Debug; > use Koha::Patron::Message::Preferences; > use Koha::Patrons; >@@ -178,7 +177,7 @@ sub set_form_values { > > =head1 SEE ALSO > >-L<C4::Members::Messaging>, F<admin/categories.pl>, F<opac/opac-messaging.pl>, F<members/messaging.pl> >+L<Koha::Patron::Message::Preference>, F<admin/categories.pl>, F<opac/opac-messaging.pl>, F<members/messaging.pl> > > =head1 AUTHOR > >diff --git a/C4/Members/Messaging.pm b/C4/Members/Messaging.pm >deleted file mode 100644 >index 923ed76..0000000 >--- a/C4/Members/Messaging.pm >+++ /dev/null >@@ -1,97 +0,0 @@ >-package C4::Members::Messaging; >- >-# Copyright (C) 2008 LibLime >-# >-# This file is part of Koha. >-# >-# Koha is free software; you can redistribute it and/or modify it >-# under the terms of the GNU General Public License as published by >-# the Free Software Foundation; either version 3 of the License, or >-# (at your option) any later version. >-# >-# Koha is distributed in the hope that it will be useful, but >-# WITHOUT ANY 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, see <http://www.gnu.org/licenses>. >- >-use strict; >-use warnings; >-use C4::Context; >- >-=head1 NAME >- >-C4::Members::Messaging - manage patron messaging preferences >- >-=head1 SYNOPSIS >- >- use C4::Members::Messaging >- >-=head1 DESCRIPTION >- >-This module lets you modify a patron's messaging preferences. >- >-=head1 FUNCTIONS >- >-=head1 TABLES >- >-=head2 message_queue >- >-The actual messages which will be sent via a cron job running >-F<misc/cronjobs/process_message_queue.pl>. >- >-=head2 message_attributes >- >-What kinds of messages can be sent? >- >-=head2 message_transport_types >- >-What transports can messages be sent vith? (email, sms, etc.) >- >-=head2 message_transports >- >-How are message_attributes and message_transport_types correlated? >- >-=head2 borrower_message_preferences >- >-What messages do the borrowers want to receive? >- >-=head2 borrower_message_transport_preferences >- >-What transport should a message be sent with? >- >-=head1 CONFIG >- >-=head2 Adding a New Kind of Message to the System >- >-=over 4 >- >-=item 1. >- >-Add a new template to the `letter` table. >- >-=item 2. >- >-Insert a row into the `message_attributes` table. >- >-=item 3. >- >-Insert rows into `message_transports` for each message_transport_type. >- >-=back >- >-=head1 SEE ALSO >- >-L<C4::Letters> >- >-=head1 AUTHOR >- >-Koha Development Team <http://koha-community.org/> >- >-Andrew Moore <andrew.moore@liblime.com> >- >-=cut >- >-1; >diff --git a/C4/Message.pm b/C4/Message.pm >index 30b71c8..b2b6c15 100644 >--- a/C4/Message.pm >+++ b/C4/Message.pm >@@ -402,7 +402,7 @@ sub DESTROY { } > > =head1 SEE ALSO > >-L<C4::Circulation>, L<C4::Letters>, L<C4::Members::Messaging> >+L<C4::Circulation>, L<C4::Letters>, L<Koha::Patron::Message::Preference> > > =head1 AUTHOR > >diff --git a/C4/Reserves.pm b/C4/Reserves.pm >index 3054e3e..a9f75f1 100644 >--- a/C4/Reserves.pm >+++ b/C4/Reserves.pm >@@ -31,7 +31,6 @@ use C4::Circulation; > use C4::Accounts; > > # for _koha_notify_reserve >-use C4::Members::Messaging; > use C4::Members qw(); > use C4::Letters; > use C4::Log; >diff --git a/Koha/Patron/Category.pm b/Koha/Patron/Category.pm >index 8e9b590..9e9b657 100644 >--- a/Koha/Patron/Category.pm >+++ b/Koha/Patron/Category.pm >@@ -19,8 +19,6 @@ use Modern::Perl; > > use Carp; > >-use C4::Members::Messaging; >- > use Koha::Database; > use Koha::DateUtils; > use Koha::Patron::Message::Preferences; >diff --git a/circ/returns.pl b/circ/returns.pl >index 942dd11..868006e 100755 >--- a/circ/returns.pl >+++ b/circ/returns.pl >@@ -44,7 +44,6 @@ use C4::Reserves; > use C4::Biblio; > use C4::Items; > use C4::Members; >-use C4::Members::Messaging; > use C4::Koha; # FIXME : is it still useful ? > use C4::RotatingCollections; > use Koha::AuthorisedValues; >diff --git a/misc/cronjobs/advance_notices.pl b/misc/cronjobs/advance_notices.pl >index a753007..49cb07d 100755 >--- a/misc/cronjobs/advance_notices.pl >+++ b/misc/cronjobs/advance_notices.pl >@@ -53,7 +53,6 @@ use C4::Biblio; > use C4::Context; > use C4::Letters; > use C4::Members; >-use C4::Members::Messaging; > use C4::Overdues; > use Koha::DateUtils; > use C4::Log; >diff --git a/misc/maintenance/borrowers-force-messaging-defaults b/misc/maintenance/borrowers-force-messaging-defaults >index 081e0f5..9cd0b4f 100755 >--- a/misc/maintenance/borrowers-force-messaging-defaults >+++ b/misc/maintenance/borrowers-force-messaging-defaults >@@ -27,7 +27,6 @@ BEGIN { > } > > use C4::Context; >-use C4::Members::Messaging; > use Koha::Patrons; > use Getopt::Long; > use Pod::Usage; >diff --git a/opac/opac-messaging.pl b/opac/opac-messaging.pl >index 2493f21..d6c9486 100755 >--- a/opac/opac-messaging.pl >+++ b/opac/opac-messaging.pl >@@ -28,7 +28,6 @@ use C4::Koha; > use C4::Circulation; > use C4::Output; > use C4::Members; >-use C4::Members::Messaging; > use C4::Form::MessagingPreferences; > use Koha::SMS::Providers; > >diff --git a/t/Members_Messaging.t b/t/Members_Messaging.t >deleted file mode 100755 >index 9395008..0000000 >--- a/t/Members_Messaging.t >+++ /dev/null >@@ -1,14 +0,0 @@ >-#!/usr/bin/perl >-# >-# This Koha test module is a stub! >-# Add more tests here!!! >- >-use strict; >-use warnings; >- >-use Test::More tests => 1; >- >-BEGIN { >- use_ok('C4::Members::Messaging'); >-} >- >diff --git a/tools/import_borrowers.pl b/tools/import_borrowers.pl >index c79fdce..b82d19d 100755 >--- a/tools/import_borrowers.pl >+++ b/tools/import_borrowers.pl >@@ -43,7 +43,6 @@ use C4::Context; > use C4::Members; > use C4::Members::Attributes qw(:all); > use C4::Members::AttributeTypes; >-use C4::Members::Messaging; > use C4::Reports::Guided; > use C4::Templates; > use Koha::Patron::Debarments; >-- >2.7.4
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 18595
:
63435
|
63436
|
63437
|
63438
|
63439
|
63440
|
63441
|
63794
|
63795
|
63796
|
63797
|
63798
|
63799
|
63800
|
63813
|
63815
|
63816
|
63817
|
63818
|
63819
|
63820
|
63821
|
63822
|
63823
|
63824
|
63825
|
63826
|
63827
|
63828
|
63829
|
69441
|
69442
|
69443
|
69444
|
69445
|
69446
|
69447
|
69448
|
69449
|
69450
|
69451
|
69452
|
69453
|
69667
|
69668
|
69669
|
69670
|
69671
|
69672
|
69673
|
69674
|
93873
|
93874
|
93875
|
93876
|
93877
|
93878
|
93879
|
93880
|
93908
|
94753
|
94754
|
103444
|
103445
|
103446
|
103447
|
103448
|
103449
|
103450
|
103451
|
103452
|
110151
|
110152
|
110153
|
110154
|
110155
|
110156
|
110157
|
110158
|
110159
|
112147
|
112148
|
112149
|
112150
|
112151
|
112152
|
112153
|
112154
|
112155
|
112690
|
113126
|
113192
|
113221
|
113222
|
113769
|
146541
|
146542
|
146543
|
146544
|
146545
|
146546
|
146547
|
146548
|
146549
|
146550
|
146551
|
146552