Bugzilla – Attachment 76658 Details for
Bug 12365
Allow descriptive notes to be added to circulation and fine rules
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 12365: [FOLLOW-UP] Adding maxlength attribute and popover
Bug-12365-FOLLOW-UP-Adding-maxlength-attribute-and.patch (text/plain), 2.97 KB, created by
Aleisha Amohia
on 2018-07-04 00:34:27 UTC
(
hide
)
Description:
Bug 12365: [FOLLOW-UP] Adding maxlength attribute and popover
Filename:
MIME Type:
Creator:
Aleisha Amohia
Created:
2018-07-04 00:34:27 UTC
Size:
2.97 KB
patch
obsolete
>From 36e43c8ef0213af26afba27b9a8ee214246c5f87 Mon Sep 17 00:00:00 2001 >From: Aleisha Amohia <aleishaamohia@hotmail.com> >Date: Wed, 4 Jul 2018 00:12:21 +0000 >Subject: [PATCH] Bug 12365: [FOLLOW-UP] Adding maxlength attribute and popover > >Test that you can't add a note of more than 100 characters. >Test notes with rules, notes are hidden by 'Click to view note' and the >popover correctly displays note. >--- > koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt | 10 ++++++++-- > 1 file changed, 8 insertions(+), 2 deletions(-) > >diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt >index 0a10714..48dfd0a 100644 >--- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt >+++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/smart-rules.tt >@@ -213,7 +213,11 @@ > [% END %] > </td> > <td>[% rule.rentaldiscount %]</td> >- <td>[% rule.note %]</td> >+ <td> >+ [% IF rule.note %] >+ <a data-toggle="popover" title="Note" data-content="[% rule.note %]">Click to view note</a> >+ [% ELSE %] [% END %] >+ </td> > <td class="actions"> > <a href="#" class="editrule btn btn-default btn-xs"><i class="fa fa-pencil"></i> Edit</a> > <a class="btn btn-default btn-xs delete" href="/cgi-bin/koha/admin/smart-rules.pl?op=delete&itemtype=[% rule.itemtype %]&categorycode=[% rule.categorycode %]&branch=[% rule.current_branch %]"><i class="fa fa-trash"></i> Delete</a> >@@ -314,7 +318,7 @@ > </select> > </td> > <td><input type="text" name="rentaldiscount" id="rentaldiscount" size="2" /></td> >- <td><input type="text" name="note" id="note" size="15" value=""></td> >+ <td><input type="text" name="note" id="note" size="15" value="" maxlength="100"></td> > <td class="actions"> > <input type="hidden" name="branch" value="[% current_branch %]"/> > <button type="submit" class="btn btn-default btn-xs"><i class="fa fa-save"></i> Save</button> >@@ -767,6 +771,8 @@ > var MSG_CONFIRM_DELETE = _("Are you sure you want to delete this rule? This cannot be undone."); > > $(document).ready(function() { >+ $('[data-toggle="popover"]').popover(); >+ > $(".delete").on("click",function(){ > return confirmDelete(MSG_CONFIRM_DELETE); > }); >-- >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 12365
:
61865
|
63902
|
64441
|
64443
|
64565
|
66829
|
66830
|
67802
|
76609
|
76637
|
76658
|
77147
|
77275
|
78685
|
78686
|
80790
|
80956
|
80957
|
80958
|
80997
|
80998
|
81603
|
81604
|
81605
|
81606
|
81647
|
81648
|
81649
|
81650
|
82081