[Koha-bugs] [Bug 13364] Add a call number browser in item edition page

[bugzilla-daemon at bugs.koha-community.org]

http://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=13364

--- Comment #42 from Jonathan Druart <jonathan.druart at biblibre.com> ---
Comment on attachment 36600
  --> http://bugs.koha-community.org/bugzilla3/attachment.cgi?id=36600
Bug 13364 - Add a call number browser in item edition page

Review of attachment 36600:
 --> (http://bugs.koha-community.org/bugzilla3/page.cgi?id=splinter.html&bug=13364&attachment=36600)
-----------------------------------------------------------------

Is there a good reason to have the template file in the tools directory,
instead of cataloguing/value_builder?

The green line is always at the top of the table. If I am remember correctly,
in the previous version it was inserted correctly.

::: cataloguing/value_builder/cn_browser.pl
@@ +1,4 @@
> +#!/usr/bin/perl
> +
> +use strict;
> +use warnings;

Prefer use Modern::Perl;

@@ +9,5 @@
> +use C4::Output;
> +use POSIX;
> +use C4::ClassSource;
> +use URI::Escape;
> +use Data::Dumper;

No needed.

::: koha-tmpl/intranet-tmpl/prog/en/modules/tools/cn_browser.tt
@@ +12,5 @@
> +<script type="text/javascript" src="[% interface %]/lib/jquery/plugins/jquery.dataTables.min.js"></script>
> +<script language="JavaScript">
> +$(document).ready(function()
> +{
> +    $("#course_reserves").DataTable({"paging":   false, "bFilter": false, "info":     false});

Hum? Why did you choose this id?

-- 
You are receiving this mail because:
You are watching all bug changes.