# WARRANTY; without even the implied warranty of MERCHANTABILITY or FITNESS FOR
# A PARTICULAR PURPOSE. See the GNU General Public License for more details.
#
-# You should have received a copy of the GNU General Public License along with
-# Koha; if not, write to the Free Software Foundation, Inc., 59 Temple Place,
-# Suite 330, Boston, MA 02111-1307 USA
+# You should have received a copy of the GNU General Public License along
+# with Koha; if not, write to the Free Software Foundation, Inc.,
+# 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA.
=head1 admin/itemtypes.pl
=cut
use strict;
+#use warnings; FIXME - Bug 2505
use CGI;
use List::Util qw/min/;
+use File::Spec;
use C4::Koha;
use C4::Context;
my $searchfield = $input->param('description');
my $script_name = "/cgi-bin/koha/admin/itemtypes.pl";
my $itemtype = $input->param('itemtype');
-my $pagesize = 10;
my $op = $input->param('op');
$searchfield =~ s/\,//g;
my ( $template, $borrowernumber, $cookie ) = get_template_and_user(
$data = $sth->fetchrow_hashref;
}
- # build list of images
- my $imagedir_filesystem = getitemtypeimagedir();
- my $imagedir_web = getitemtypeimagesrc();
- opendir( DIR, $imagedir_filesystem )
- or warn "cannot opendir " . $imagedir_filesystem . ": " . $!;
- my @imagelist;
- my $i = 0;
- my $image_per_line = 12;
- while ( my $line = readdir(DIR) ) {
- $i++;
- if ( $line =~ /\.(gif|png)$/i ) {
- if ( $i == $image_per_line ) {
- $i = 0;
- push @imagelist, { KohaImage => '', KohaImageSrc => '' };
- }
- else {
- push(
- @imagelist,
- {
- KohaImage => $line,
- KohaImageSrc => $imagedir_web . '/' . $line,
- checked => $line eq $data->{imageurl} ? 1 : 0,
- }
- );
- }
- }
- }
- closedir DIR;
+ my $imagesets = C4::Koha::getImageSets( checked => $data->{'imageurl'} );
my $remote_image = undef;
if ( defined $data->{imageurl} and $data->{imageurl} =~ /^http/i ) {
$template->param(
itemtype => $itemtype,
description => $data->{'description'},
- renewalsallowed => $data->{'renewalsallowed'},
rentalcharge => sprintf( "%.2f", $data->{'rentalcharge'} ),
notforloan => $data->{'notforloan'},
imageurl => $data->{'imageurl'},
template => C4::Context->preference('template'),
summary => $data->{summary},
- IMAGESLOOP => \@imagelist,
+ imagesets => $imagesets,
remote_image => $remote_image,
);
my $query2 = '
UPDATE itemtypes
SET description = ?
- , renewalsallowed = ?
, rentalcharge = ?
, notforloan = ?
, imageurl = ?
$sth = $dbh->prepare($query2);
$sth->execute(
$input->param('description'),
- $input->param('renewalsallowed'),
$input->param('rentalcharge'),
( $input->param('notforloan') ? 1 : 0 ),
(
else { # add a new itemtype & not modif an old
my $query = "
INSERT INTO itemtypes
- (itemtype,description,renewalsallowed,rentalcharge, notforloan, imageurl,summary)
+ (itemtype,description,rentalcharge, notforloan, imageurl,summary)
VALUES
- (?,?,?,?,?,?,?);
+ (?,?,?,?,?,?);
";
my $sth = $dbh->prepare($query);
my $image = $input->param('image');
$sth->execute(
$input->param('itemtype'),
$input->param('description'),
- $input->param('renewalsallowed'),
$input->param('rentalcharge'),
$input->param('notforloan') ? 1 : 0,
$image eq 'removeImage' ? '' :
# called by default form, used to confirm deletion of data in DB
}
elsif ( $op eq 'delete_confirm' ) {
- # Check both categoryitem and biblioitems, see Bug 199
- my $total = 0;
- for my $table ('biblioitems') {
- my $sth =
- $dbh->prepare(
- "select count(*) as total from $table where itemtype=?");
- $sth->execute($itemtype);
- $total += $sth->fetchrow_hashref->{total};
- }
+ # Check both items and biblioitems
+ my $sth = $dbh->prepare('
+ SELECT COUNT(*) AS total FROM (
+ SELECT itemtype AS t FROM biblioitems
+ UNION
+ SELECT itype AS t FROM items
+ ) AS tmp
+ WHERE tmp.t=?
+ ');
+ $sth->execute($itemtype);
+ my $total = $sth->fetchrow_hashref->{'total'};
my $sth =
$dbh->prepare(
-"select itemtype,description,renewalsallowed,rentalcharge from itemtypes where itemtype=?"
+"select itemtype,description,rentalcharge from itemtypes where itemtype=?"
);
$sth->execute($itemtype);
my $data = $sth->fetchrow_hashref;
$template->param(
itemtype => $itemtype,
description => $data->{description},
- renewalsallowed => $data->{renewalsallowed},
rentalcharge => sprintf( "%.2f", $data->{rentalcharge} ),
imageurl => $data->{imageurl},
total => $total
}
else { # DEFAULT
my ($results) = StringSearch( $searchfield, 'web' );
- my $page = $input->param('page') || 1;
- my $first = ( $page - 1 ) * $pagesize;
-
- # if we are on the last page, the number of the last word to display
- # must not exceed the length of the results array
- my $last = min( $first + $pagesize - 1, scalar @{$results} - 1, );
- my $toggle = 0;
my @loop;
- foreach my $itemtype ( @{$results}[ $first .. $last ] ) {
- $itemtype->{toggle} = ($toggle++ % 2) ? 0 : 1 ;
- $itemtype->{imageurl} = getitemtypeimagesrcfromurl( $itemtype->{imageurl} );
+ foreach my $itemtype ( @{$results} ) {
+ $itemtype->{imageurl} = getitemtypeimagelocation( 'intranet', $itemtype->{imageurl} );
$itemtype->{rentalcharge} = sprintf( '%.2f', $itemtype->{rentalcharge} );
push( @loop, $itemtype );
}
$template->param(
loop => \@loop,
- pagination_bar => pagination_bar(
- $script_name, getnbpages( scalar @{$results}, $pagesize ),
- $page, 'page'
- )
);
} #---- END $OP eq DEFAULT