Bug 18881: [QA Follow-up] Remove even more
Removing the commented section from the template: If it does not work, it should not be here. When it works again, put it back in. Since @itemtypesloop is not used, remove it from the script too. Signed-off-by: Marcel de Rooy <m.de.rooy@rijksmuseum.nl> Signed-off-by: Jonathan Druart <jonathan.druart@bugs.koha-community.org>
This commit is contained in:
parent
f40c093c19
commit
e48a5fb560
|
|
@ -62,19 +62,6 @@ if ( $run_report ) {
|
|||
);
|
||||
}
|
||||
|
||||
# getting all itemtypes
|
||||
my $itemtypes = Koha::ItemTypes->search({}, {order_by => 'itemtype'});;
|
||||
my @itemtypesloop;
|
||||
while ( my $itemtype = $itemtypes->next ) {
|
||||
push @itemtypesloop, {
|
||||
value => $itemtype->itemtype,
|
||||
};
|
||||
}
|
||||
|
||||
$template->param( # FIXME Could be improved passing the $itemtypes iterator directly to the template
|
||||
itemtypeloop => \@itemtypesloop,
|
||||
);
|
||||
|
||||
# Checking if there is a Fast Cataloging Framework
|
||||
$template->param( fast_cataloging => 1 ) if Koha::BiblioFrameworks->find( 'FA' );
|
||||
|
||||
|
|
|
|||
|
|
@ -209,17 +209,6 @@ $(document).ready(function() {
|
|||
[% PROCESS options_for_libraries libraries => Branches.all() %]
|
||||
</select>
|
||||
</li>
|
||||
<!-- FIXME Not working yet
|
||||
<li><label for="itemtypeslimit">Item Type: </label><select name="itemtypeslimit" id="itemtypeslimit">
|
||||
<option value="">All</option>
|
||||
[% FOREACH itemtypeloo IN itemtypeloop %]
|
||||
[% IF ( itemtypeloo.selected ) %]<option value="[% itemtypeloo.value %]" selected="selected">[% itemtypeloo.description %]</option>
|
||||
[% ELSE %]
|
||||
<option value="[% itemtypeloo.value %]">[% itemtypeloo.description %]</option>
|
||||
[% END %]
|
||||
[% END %]
|
||||
</select></li>
|
||||
-->
|
||||
</ol></fieldset>
|
||||
<fieldset class="action"> <input type="submit" value="Submit" />
|
||||
<input type="hidden" name="run_report" value="1" /></fieldset>
|
||||
|
|
|
|||
Loading…
Reference in New Issue