Bugzilla – Attachment 118154 Details for
Bug 27930
Move _escape_* functions from acq/parcel.tt to be re-usable
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 27930: [stables] Move _escape_* from acqui/parcel.tt to js/staff-global.js
Bug-27930-stables-Move-escape-from-acquiparceltt-t.patch (text/plain), 1.51 KB, created by
Jonathan Druart
on 2021-03-11 15:41:43 UTC
(
hide
)
Description:
Bug 27930: [stables] Move _escape_* from acqui/parcel.tt to js/staff-global.js
Filename:
MIME Type:
Creator:
Jonathan Druart
Created:
2021-03-11 15:41:43 UTC
Size:
1.51 KB
patch
obsolete
>From 240975af26b4846ac3bb54a982afb2eac4700fc0 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@ptfs-europe.com> >Date: Thu, 11 Mar 2021 14:22:54 +0000 >Subject: [PATCH] Bug 27930: [stables] Move _escape_* from acqui/parcel.tt to > js/staff-global.js > >This patch moves the _escape_str and _escape_price functions into the >staff-global.js include > >Test plan >1/ The acquisitions parcel page should look the same before and after >the patch. >2/ Signoff > >Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org> >Note: For stable branch the functions didn't exist. >--- > koha-tmpl/intranet-tmpl/prog/js/staff-global.js | 6 ++++++ > 1 file changed, 6 insertions(+) > >diff --git a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js >index 4f06af9465..5a7f7fff74 100644 >--- a/koha-tmpl/intranet-tmpl/prog/js/staff-global.js >+++ b/koha-tmpl/intranet-tmpl/prog/js/staff-global.js >@@ -26,6 +26,9 @@ String.prototype.escapeHtml = function() { > return HtmlCharsToEscape[c] || c; > }); > }; >+function escape_str(s){ >+ return s != null ? s.escapeHtml() : ""; >+} > > /* > * Void method for numbers, for consistency >@@ -33,6 +36,9 @@ String.prototype.escapeHtml = function() { > Number.prototype.escapeHtml = function() { > return this; > }; >+function escape_price(p){ >+ return p != null ? p.escapeHtml().format_price() : ""; >+} > > // http://stackoverflow.com/questions/14859281/select-tab-by-name-in-jquery-ui-1-10-0/16550804#16550804 > $.fn.tabIndex = function () { >-- >2.20.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 27930
:
118128
|
118153
| 118154