Bugzilla – Attachment 48036 Details for
Bug 9021
Add SMS via email as an alternative to SMS services via SMS::Send drivers
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
[SIGNED-OFF] Bug 9021 [QA Followup] - Add additional message to deletion warning if provider is in use.
SIGNED-OFF-Bug-9021-QA-Followup---Add-additional-m.patch (text/plain), 2.09 KB, created by
Mark Tompsett
on 2016-02-12 20:09:25 UTC
(
hide
)
Description:
[SIGNED-OFF] Bug 9021 [QA Followup] - Add additional message to deletion warning if provider is in use.
Filename:
MIME Type:
Creator:
Mark Tompsett
Created:
2016-02-12 20:09:25 UTC
Size:
2.09 KB
patch
obsolete
>From 44932e4234286d6046c791384e9012225f0403e3 Mon Sep 17 00:00:00 2001 >From: Kyle M Hall <kyle@bywatersolutions.com> >Date: Thu, 11 Feb 2016 10:43:40 +0000 >Subject: [PATCH] [SIGNED-OFF] Bug 9021 [QA Followup] - Add additional message > to deletion warning if provider is in use. > >Signed-off-by: Mark Tompsett <mtompset@hotmail.com> >--- > .../intranet-tmpl/prog/en/modules/admin/sms_providers.tt | 13 ++++++++++--- > 1 file changed, 10 insertions(+), 3 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/sms_providers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/sms_providers.tt >index 706db5c..8fbd415 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/sms_providers.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/sms_providers.tt >@@ -36,8 +36,15 @@ function cancel_edit() { > > } > >-function delete_provider( id ) { >- if ( confirm( _("Are you sure you want to delete ") + $("#name_" + id).html() + _("?") ) ) { >+function delete_provider( id, users ) { >+ var c; >+ if ( users ) { >+ c = confirm( _("Are you sure you want to delete %s? %s patron(s) are using it!").format( $("#name_" + id).html(), users ) ); >+ } else { >+ c = confirm( _("Are you sure you want to delete %s?").format( $("#name_" + id).html() ) ); >+ } >+ >+ if ( c ) { > $("#op").val('delete'); > $("#id").val( id ); > $("#sms_form").submit(); >@@ -79,7 +86,7 @@ function delete_provider( id ) { > </td> > <td> > [% IF p.patrons_using %] >- <a class="btn btn-danger" href="#" id="delete_[% p.id %]" class="delete" onclick="delete_provider( [% p.id %] );"> >+ <a class="btn btn-danger" href="#" id="delete_[% p.id %]" class="delete" onclick="delete_provider( [% p.id %], [% p.patrons_using %] );"> > <span style="color:white"><i class="fa fa-trash"></i> Delete</span> > </a> > [% ELSE %] >-- >2.1.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 9021
:
13290
|
13291
|
13293
|
13294
|
14166
|
14167
|
14363
|
15685
|
15687
|
15689
|
15691
|
16628
|
17311
|
21729
|
22478
|
28488
|
29927
|
32453
|
34180
|
34181
|
35816
|
40720
|
44331
|
45152
|
46163
|
46164
|
46165
|
46168
|
46169
|
46170
|
46171
|
47128
|
47129
|
47130
|
47133
|
47134
|
47135
|
47338
|
47339
|
47814
|
47815
|
47816
|
47817
|
47818
|
47845
|
47846
|
47847
|
47848
|
47995
|
48024
|
48025
|
48026
|
48027
|
48028
|
48029
|
48030
|
48031
|
48032
|
48033
|
48034
|
48035
|
48036
|
48052
|
48053
|
48054
|
48055
|
48056
|
48057
|
48058
|
48059
|
48060
|
48061
|
48062
|
48063
|
48064
|
48065
|
48066