Bugzilla – Attachment 42621 Details for
Bug 12426
Allow resending of emails from the notices tab in the patron account
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12426: Allow resend for sent messages
Bug-12426-Allow-resend-for-sent-messages.patch (text/plain), 4.23 KB, created by
Lari Taskula
on 2015-09-16 16:49:39 UTC
(
hide
)
Description:
Bug 12426: Allow resend for sent messages
Filename:
MIME Type:
Creator:
Lari Taskula
Created:
2015-09-16 16:49:39 UTC
Size:
4.23 KB
patch
obsolete
>From f7c4d237a16398e99710971b29ad62c07d048e0a Mon Sep 17 00:00:00 2001 >From: Lari Taskula <larit@student.uef.fi> >Date: Wed, 16 Sep 2015 18:31:57 +0300 >Subject: [PATCH] Bug 12426: Allow resend for sent messages > >This patch allows to resend both sent and failed messages. > >With messages in 'sent' status, we have to be careful not to accidentally send >sent messages again. With the previous patch using GET request, this was likely >to happen because of browser storing the GET parameters. > >This patch changes request method from GET to POST. Instead of a simple link, >we now have a form element. > >In notices.pl we redirect back to notices.pl, because with POST there is a risk >of resending the message accidentally by form resubmission at refresh. > >To test, find/create a Patron that has sent or failed notices in message_queue: >1. Enable EnchancedMessagingPreferences system preference >2. Go to Patrons -> Notices >3. In the Notice column, click the title of the sent or failed message >4. Observe that there is nothing for resending the sent or failed message >5. Apply the patches. >6. Reload Notices page and repeat step 3 >7. Observe that there is now a link "Resend" in the Status-column >8. Click Resend >9. Observe that the message gets into 'pending' status >--- > C4/Letters.pm | 2 +- > koha-tmpl/intranet-tmpl/prog/en/modules/members/notices.tt | 12 +++++++++++- > members/notices.pl | 2 ++ > 3 files changed, 14 insertions(+), 2 deletions(-) > >diff --git a/C4/Letters.pm b/C4/Letters.pm >index f0d7aa3..824d621 100644 >--- a/C4/Letters.pm >+++ b/C4/Letters.pm >@@ -1096,7 +1096,7 @@ sub ResendMessage { > > my $message = GetMessage( $message_id ); > return unless $message; >- if ( $message->{status} eq 'failed' ) { >+ if ( $message->{status} ne 'pending' ) { > return ((C4::Letters::_set_message_status( { > message_id => $message_id, > status => 'pending', >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/members/notices.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/members/notices.tt >index 189209c..3dcb059 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/notices.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/members/notices.tt >@@ -70,9 +70,19 @@ > <td> > [% IF ( QUEUED_MESSAGE.status == 'sent' ) %]sent > [% ELSIF ( QUEUED_MESSAGE.status == 'pending' ) %]pending >- [% ELSIF ( QUEUED_MESSAGE.status == 'failed' ) %]failed <div class="notice"><a href="/cgi-bin/koha/members/notices.pl?borrowernumber=[% borrowernumber %]&op=resend_notice&message_id=[% QUEUED_MESSAGE.message_id %]" title="Attempt to resend the notice">Resend</a></div> >+ [% ELSIF ( QUEUED_MESSAGE.status == 'failed' ) %]failed > [% ELSIF ( QUEUED_MESSAGE.status == 'deleted' ) %]deleted > [% ELSE %][% QUEUED_MESSAGE.status %][% END %] >+ [% IF ( QUEUED_MESSAGE.status != 'pending' ) %] >+ <div class="notice"> >+ <form action="/cgi-bin/koha/members/notices.pl?borrowernumber=[% borrowernumber %]" method="POST"> >+ <input type="hidden" name="op" value="resend_notice" /> >+ <input type="hidden" name="borrowernumber" value="[% borrowernumber %]" /> >+ <input type="hidden" name="message_id" value="[% QUEUED_MESSAGE.message_id %]" /> >+ <a href="#" onclick="$(this).closest('form').submit();return false;" title="Attempt to resend the notice">Resend</a> >+ </form> >+ </div> >+ [% END %] > </td> > <td><span title="[% QUEUED_MESSAGE.time_queued %]">[% QUEUED_MESSAGE.time_queued | $KohaDates with_hours => 1 %]</span></td> > </tr> >diff --git a/members/notices.pl b/members/notices.pl >index 2fdbe88..95595ef 100755 >--- a/members/notices.pl >+++ b/members/notices.pl >@@ -58,6 +58,8 @@ if ( $op eq 'resend_notice' ) { > my $message = C4::Letters::GetMessage( $message_id ); > if ( $message->{borrowernumber} = $borrowernumber ) { > C4::Letters::ResendMessage( $message_id ); >+ # redirect to self to avoid form submission on refresh >+ print $input->redirect("/cgi-bin/koha/members/notices.pl?borrowernumber=$borrowernumber"); > } > } > >-- >1.9.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 12426
:
42295
|
42346
|
42401
|
42409
|
42418
|
42512
|
42615
|
42618
|
42620
|
42621
|
47563
|
47564
|
47565
|
47588
|
47589
|
47590
|
47956
|
47957
|
47958
|
48049
|
48570
|
48571
|
48572
|
48573