Bugzilla – Attachment 184034 Details for
Bug 39522
Add hooks to allow 'Valuebuilder' plugins to be installable
Home
|
New
|
Browse
|
Search
|
[?]
|
Reports
|
Help
|
New Account
|
Log In
[x]
|
Forgot Password
Login:
[x]
[patch]
Bug 39522: (follow-up) Remove closing </html> to allow tidy
Bug-39522-follow-up-Remove-closing-html-to-allow-t.patch (text/plain), 2.88 KB, created by
Martin Renvoize (ashimema)
on 2025-07-12 12:03:01 UTC
(
hide
)
Description:
Bug 39522: (follow-up) Remove closing </html> to allow tidy
Filename:
MIME Type:
Creator:
Martin Renvoize (ashimema)
Created:
2025-07-12 12:03:01 UTC
Size:
2.88 KB
patch
obsolete
>From 0ebf3121478661b334634452be0879a2ef7bd413 Mon Sep 17 00:00:00 2001 >From: Martin Renvoize <martin.renvoize@openfifth.co.uk> >Date: Sat, 12 Jul 2025 13:00:41 +0100 >Subject: [PATCH] Bug 39522: (follow-up) Remove closing </html> to allow tidy > >I don't understand this.. for the tidy script to run and the qa script >to be happy I had to remove the closing html tag. >--- > .../test_valuebuilder_popup.tt | 52 +++++++++---------- > 1 file changed, 26 insertions(+), 26 deletions(-) > >diff --git a/t/lib/plugins/Koha/Plugin/TestValuebuilder/test_valuebuilder_popup.tt b/t/lib/plugins/Koha/Plugin/TestValuebuilder/test_valuebuilder_popup.tt >index 1db60a9777f..bb3f26d5667 100644 >--- a/t/lib/plugins/Koha/Plugin/TestValuebuilder/test_valuebuilder_popup.tt >+++ b/t/lib/plugins/Koha/Plugin/TestValuebuilder/test_valuebuilder_popup.tt >@@ -1,29 +1,29 @@ > <!DOCTYPE html> > <html> >-<head> >- <title>[% plugin_name | html %] - Valuebuilder</title> >- <meta charset="UTF-8"> >-</head> >-<body> >- <h1>[% plugin_name | html %]</h1> >- <p>This is a test valuebuilder popup for field: [% field_id | html %]</p> >+ <head> >+ <title>[% plugin_name | html %] - Valuebuilder</title> >+ <meta charset="UTF-8" /> >+ </head> >+ <body> >+ <h1>[% plugin_name | html %]</h1> >+ <p>This is a test valuebuilder popup for field: [% field_id | html %]</p> > >- <form> >- <label for="test_value">Enter a value:</label> >- <input type="text" id="test_value" value="Test Value from Plugin"> >- <br><br> >- <button type="button" onclick="setValueAndClose()">Set Value</button> >- <button type="button" onclick="window.close()">Cancel</button> >- </form> >- >- <script> >- function setValueAndClose() { >- var value = document.getElementById('test_value').value; >- if (window.opener && window.opener.document.getElementById('[% field_id | html %]')) { >- window.opener.document.getElementById('[% field_id | html %]').value = value; >- } >- window.close(); >- } >- </script> >-</body> >-</html> >\ No newline at end of file >+ <form> >+ <label for="test_value">Enter a value:</label> >+ <input type="text" id="test_value" value="Test Value from Plugin" /> >+ <br /><br /> >+ <button type="button" onclick="setValueAndClose()">Set Value</button> >+ <button type="button" onclick="window.close()">Cancel</button> >+ </form> >+ <script> >+ function setValueAndClose() { >+ var value = document.getElementById('test_value').value; >+ if (window.opener && window.opener.document.getElementById('[% field_id | html %]')) { >+ window.opener.document.getElementById('[% field_id | html %]').value = value; >+ } >+ window.close(); >+ } >+ </script> >+ </body> >+ </head></html >+> >-- >2.50.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 39522
:
180283
|
180284
|
180301
|
180302
|
180337
|
180338
|
183467
|
183468
|
183469
|
184030
|
184031
|
184032
|
184033
|
184034
|
184071
|
184072