Bug 29093: Interface changes
Show the TOC checkbox on OPAC and staff. Test plan: Add new article request on OPAC or staff. Tick checkbox. Verify if TOC is Yes on opac-user or staff patron details. Check the list view on circ/article-requests.pl. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: David Nind <david@davidnind.com> Signed-off-by: Nick Clemens <nick@bywatersolutions.com> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
parent
13501fe2d7
commit
8c6281f985
7 changed files with 28 additions and 0 deletions
|
@ -71,6 +71,7 @@ if ( $action eq 'create' ) {
|
|||
my $chapters = $cgi->param('chapters') || undef;
|
||||
my $patron_notes = $cgi->param('patron_notes') || undef;
|
||||
my $format = $cgi->param('format') || undef;
|
||||
my $toc_request = $cgi->param('toc_request');
|
||||
|
||||
try {
|
||||
my $ar = Koha::ArticleRequest->new(
|
||||
|
@ -88,6 +89,7 @@ if ( $action eq 'create' ) {
|
|||
chapters => $chapters,
|
||||
patron_notes => $patron_notes,
|
||||
format => $format,
|
||||
toc_request => $toc_request ? 1 : 0,
|
||||
}
|
||||
)->request;
|
||||
} catch {
|
||||
|
|
|
@ -12,6 +12,7 @@
|
|||
<th class="article-request-volume">Volume</th>
|
||||
<th class="article-request-issue">Issue</th>
|
||||
<th class="article-request-date">Date</th>
|
||||
<th class="article-request-toc">TOC</th>
|
||||
<th class="article-request-pages">Pages</th>
|
||||
<th class="article-request-chapters">Chapters</th>
|
||||
<th class="article-request-notes">Notes</th>
|
||||
|
@ -57,6 +58,10 @@
|
|||
[% ar.date | html %]
|
||||
</td>
|
||||
|
||||
<td class="article-request-toc">
|
||||
[% IF ar.toc_request %]Yes[% END %]
|
||||
</td>
|
||||
|
||||
<td class="article-request-pages">
|
||||
[% ar.pages | html %]
|
||||
</td>
|
||||
|
|
|
@ -353,6 +353,7 @@
|
|||
</p>
|
||||
</td>
|
||||
<td class="ar-request">
|
||||
[% IF ar.toc_request %] <p><strong>Include TOC</strong> </p> [% END %]
|
||||
[% IF ar.title %] <p><strong>Title:</strong> [% ar.title | html %] </p> [% END %]
|
||||
[% IF ar.author %] <p><strong>Author:</strong> [% ar.author | html %] </p> [% END %]
|
||||
[% IF ar.volume %] <p><strong>Volume:</strong> [% ar.volume | html %] </p> [% END %]
|
||||
|
@ -456,6 +457,7 @@
|
|||
</p>
|
||||
</td>
|
||||
<td class="ar-request">
|
||||
[% IF ar.toc_request %] <p><strong>Include TOC</strong> </p> [% END %]
|
||||
[% IF ar.title %] <p><strong>Title:</strong> [% ar.title | html %] </p> [% END %]
|
||||
[% IF ar.author %] <p><strong>Author:</strong> [% ar.author | html %] </p> [% END %]
|
||||
[% IF ar.volume %] <p><strong>Volume:</strong> [% ar.volume | html %] </p> [% END %]
|
||||
|
|
|
@ -177,6 +177,11 @@
|
|||
<input type="text" name="date" id="date" size="50"/>
|
||||
</li>
|
||||
|
||||
<li>
|
||||
<label for="toc_request">Table of contents:</label>
|
||||
<input type="checkbox" name="toc_request" value="1"/>
|
||||
</li>
|
||||
|
||||
<li>
|
||||
<label for="pages">Pages:</label>
|
||||
<input type="text" name="pages" id="pages" size="50"/>
|
||||
|
@ -299,6 +304,7 @@
|
|||
<th>Volume</th>
|
||||
<th>Issue</th>
|
||||
<th>Date</th>
|
||||
<th>TOC</th>
|
||||
<th>Pages</th>
|
||||
<th>Chapters</th>
|
||||
<th>Patron notes</th>
|
||||
|
@ -318,6 +324,7 @@
|
|||
<td>[% ar.volume | html %]</td>
|
||||
<td>[% ar.issue | html %]</td>
|
||||
<td>[% ar.date | html %]</td>
|
||||
<td>[% IF ar.toc_request %]Yes[% END %]</td>
|
||||
<td>[% ar.pages | html %]</td>
|
||||
<td>[% ar.chapters | html %]</td>
|
||||
<td>[% ar.patron_notes | html %]</td>
|
||||
|
|
|
@ -114,6 +114,11 @@
|
|||
[% END %]
|
||||
</li>
|
||||
|
||||
<li>
|
||||
<label for="toc_request">Table of contents:</label>
|
||||
<input type="checkbox" name="toc_request" value="1"/>
|
||||
</li>
|
||||
|
||||
<li>
|
||||
[% IF mandatory_fields.search('pages') %]
|
||||
<label for="pages" class="required">Pages:</label>
|
||||
|
|
|
@ -777,6 +777,7 @@
|
|||
<th class="article-request-volume">Volume</th>
|
||||
<th class="article-request-issue">Issue</th>
|
||||
<th class="article-request-date">Date</th>
|
||||
<th class="article-request-toc">TOC</th>
|
||||
<th class="article-request-pages">Pages</th>
|
||||
<th class="article-request-chapters">Chapters</th>
|
||||
<th class="article-request-patron-notes">Notes</th>
|
||||
|
@ -822,6 +823,10 @@
|
|||
[% ar.date | html %]
|
||||
</td>
|
||||
|
||||
<td class="article-request-toc">
|
||||
[% IF ar.toc_request %]Yes[% END %]
|
||||
</td>
|
||||
|
||||
<td class="article-request-pages">
|
||||
[% ar.pages | html %]
|
||||
</td>
|
||||
|
|
|
@ -62,6 +62,7 @@ if ( $action eq 'create' ) {
|
|||
my $chapters = $cgi->param('chapters') || undef;
|
||||
my $patron_notes = $cgi->param('patron_notes') || undef;
|
||||
my $format = $cgi->param('format') || undef;
|
||||
my $toc_request = $cgi->param('toc_request');
|
||||
|
||||
|
||||
my $success;
|
||||
|
@ -82,6 +83,7 @@ if ( $action eq 'create' ) {
|
|||
chapters => $chapters,
|
||||
patron_notes => $patron_notes,
|
||||
format => $format,
|
||||
toc_request => $toc_request ? 1 : 0,
|
||||
}
|
||||
)->request;
|
||||
$success = 1;
|
||||
|
|
Loading…
Reference in a new issue