Bug 37728: (follow-up) Tell the test that opac-illrequests.tt has an op

xt/find-missing-op-in-forms.t wants to see name="op" value="cud-..." in any
form with method="post", but opac-illrequests.tt inserts its op input by
passing around whole and unpack and repacking whole.keys which includes
the op. We just need to tell the test that it really exists.

At first, I thought of this approach as a joke, faking out the test, but
when I tried to be more responsible and put a whole <input type="hidden" etc.
in the comment, I realized that would be more likely to confuse someone
who thought that was uncommented and that they were making real changes than
it would be to future-proof against changes in the test's approach.

Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
Signed-off-by: Pedro Amorim <pedro.amorim@ptfs-europe.com>
Signed-off-by: Katrin Fischer <katrin.fischer@bsz-bw.de>
This commit is contained in:
Phil Ringnalda 2024-09-23 19:40:32 -07:00 committed by Katrin Fischer
parent 1f8b6d5daa
commit adbbbb1d8c
No known key found for this signature in database
GPG Key ID: 0EF6E2C03357A834
1 changed files with 2 additions and 0 deletions

View File

@ -258,6 +258,7 @@
<h1>Interlibrary loan item availability</h1>
<div id="results">
<form method="POST" action="/cgi-bin/koha/opac-illrequests.pl">
[%# This will get its name="op" value="cud-" from the loop over whole.keys below %]
[% INCLUDE 'csrf-token.inc' %]
<legend><h2>Displaying availability results</h2></legend>
[% FOREACH key IN whole.keys %]
@ -291,6 +292,7 @@
<div id="results" class="page-section">
[% INCLUDE messages %]
<form method="post" id="typedisclaimer-form">
[%# This will get its name="op" value="cud-" from the loop over whole.keys below %]
[% INCLUDE 'csrf-token.inc' %]
<fieldset class="rows">
[% disclaimer.text | $raw %]