@@ -, +, @@ --- C4/Record.pm | 3 +- circ/circulation.pl | 2 - .../intranet-tmpl/prog/en/js/pages/circulation.js | 11 +- .../prog/en/modules/circ/circulation.tt | 119 +++++++++++++-------- .../prog/en/modules/members/moremember.tt | 13 ++- tools/export.pl | 2 +- 6 files changed, 90 insertions(+), 60 deletions(-) --- a/C4/Record.pm +++ a/C4/Record.pm @@ -366,6 +366,7 @@ C<$itemnumbers> - a list of itemnumbers to export sub marc2csv { my ($biblios, $id, $itemnumbers) = @_; + $itemnumbers ||= []; my $output; my $csv = Text::CSV::Encoded->new(); @@ -380,7 +381,7 @@ sub marc2csv { eval $preprocess if ($preprocess); my $firstpass = 1; - if ( $itemnumbers ) { + if ( @$itemnumbers ) { for my $itemnumber ( @$itemnumbers) { my $biblionumber = GetBiblionumberFromItemnumber $itemnumber; $output .= marcrecord2csv( $biblionumber, $id, $firstpass, $csv, $fieldprocessing, [$itemnumber] ); --- a/circ/circulation.pl +++ a/circ/circulation.pl @@ -596,8 +596,6 @@ $template->param( SpecifyDueDate => $duedatespec_allow, CircAutocompl => C4::Context->preference("CircAutocompl"), AllowRenewalLimitOverride => C4::Context->preference("AllowRenewalLimitOverride"), - export_remove_fields => C4::Context->preference("ExportRemoveFields"), - export_with_csv_profile => C4::Context->preference("ExportWithCsvProfile"), canned_bor_notes_loop => $canned_notes, debarments => GetDebarments({ borrowernumber => $borrowernumber }), todaysdate => dt_from_string()->set(hour => 23)->set(minute => 59), --- a/koha-tmpl/intranet-tmpl/prog/en/js/pages/circulation.js +++ a/koha-tmpl/intranet-tmpl/prog/en/js/pages/circulation.js @@ -41,15 +41,10 @@ $(document).ready(function() { minute: 59 }); $("#export_submit").on("click",function(){ - var export_format = $("#export_formats").val(); - export_checkouts(export_format); + var output_format = $("#output_format").val(); + export_checkouts(output_format); return false; }); - - toggle_onsite_checkout(); - $("#onsite_checkout").click(function(){ - toggle_onsite_checkout(); - }); }); function export_checkouts(format) { @@ -74,7 +69,7 @@ function export_checkouts(format) { $("#dont_export_item").val(1); } - document.getElementById("export_format").value = format; + document.getElementById("output_format").value = format; document.issues.submit(); } --- a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/circ/circulation.tt @@ -1,7 +1,7 @@ [% USE Koha %] [% USE Branches %] [% USE KohaDates %] -[% IF ( export_remove_fields OR export_with_csv_profile ) %] +[% IF Koha.Preference('ExportRemoveFields') OR Koha.Preference('ExportWithCsvProfile') %] [% SET exports_enabled = 1 %] [% END %] [% USE AuthorisedValues %] @@ -45,17 +45,6 @@ var MSG_EXPORT_SELECT_CHECKOUTS = _("You must select checkout(s) to export"); [% IF ( borrowernumber ) %]if($.cookie("holdfor") != [% borrowernumber %]){ $.cookie("holdfor",null, { path: "/", expires: 0 }); }[% ELSE %]$.cookie("holdfor",null, { path: "/", expires: 0 });[% END %] [% UNLESS ( borrowernumber ) %][% UNLESS ( selectborrower ) %]window.onload=function(){ $('#findborrower').focus(); };[% END %][% END %] -$(document).ready(function() { - [% IF !( CircAutoPrintQuickSlip == 'clear' ) %] - // listen submit to trigger qslip on empty checkout - $('#mainform').bind('submit',function() { - if ($('#barcode').val() == '') { - return printx_window( '[% CircAutoPrintQuickSlip %]' ); - } - }); - [% END %] -}); - // On-site checkout function toggle_onsite_checkout(){ if ( $("#onsite_checkout").attr('checked') ) { @@ -70,6 +59,21 @@ function toggle_onsite_checkout(){ } } +$(document).ready(function() { + [% IF !( CircAutoPrintQuickSlip == 'clear' ) %] + // listen submit to trigger qslip on empty checkout + $('#mainform').bind('submit',function() { + if ($('#barcode').val() == '') { + return printx_window( '[% CircAutoPrintQuickSlip %]' ); + } + }); + [% END %] + toggle_onsite_checkout(); + $("#onsite_checkout").click(function(){ + toggle_onsite_checkout(); + }); +}); + //]]> @@ -770,41 +774,66 @@ No patron matched [% message %]

- - - - - - - - - - - - - - - - - - - - [% INCLUDE 'checkouts-table-footer.inc' %] -
  Due dateDue dateTitleItem typeChecked out onChecked out fromCall noChargePriceRenew

select all | none

Check in

select all | none

Export

select all | none

- - - - -
- [% IF ( CAN_user_circulate_override_renewals ) %] - [% IF ( AllowRenewalLimitOverride ) %] - - +
+ + + + + + + + + + + + + + + + + + + + [% INCLUDE 'checkouts-table-footer.inc' %] +
  Due dateDue dateTitleItem typeChecked out onChecked out fromCall noChargePriceRenew

select all | none

Check in

select all | none

Export

select all | none

+ + + + +
+ [% IF ( CAN_user_circulate_override_renewals ) %] + [% IF ( AllowRenewalLimitOverride ) %] + + + [% END %] + [% END %] + + +
+ + [% IF issuecount %] + [% IF ( exports_enabled ) %] +
+ + + + + + + + + +
[% END %] [% END %] - - -
+ + [% ELSE %]

Patron has nothing checked out.

[% END %] --- a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt +++ a/koha-tmpl/intranet-tmpl/prog/en/modules/members/moremember.tt @@ -1,5 +1,8 @@ [% USE Koha %] [% USE KohaDates %] +[% IF Koha.Preference('ExportRemoveFields') OR Koha.Preference('ExportWithCsvProfile') %] + [% SET exports_enabled = 1 %] +[% END %] [% INCLUDE 'doc-head-open.inc' %] Koha › Patrons › [% IF ( unknowuser ) %] @@ -17,6 +20,7 @@ <script type="text/javascript" src="[% interface %]/lib/jquery/plugins/jquery-ui-timepicker-addon.min.js"></script> [% INCLUDE 'timepicker.inc' %] <script type="text/javascript" src="[% interface %]/lib/jquery/plugins/jquery.dataTables.rowGrouping.js"></script> +<script type="text/javascript" src="[% themelang %]/js/pages/circulation.js"></script> <script type="text/javascript" src="[% themelang %]/js/checkouts.js"></script> <script type="text/javascript" src="[% themelang %]/js/holds.js"></script> <script type="text/JavaScript"> @@ -34,6 +38,9 @@ var relatives_borrowernumbers = new Array(); relatives_borrowernumbers.push("[% b %]"); [% END %] +var MSG_ADD_MESSAGE = _("Add a new message"); +var MSG_EXPORT_SELECT_CHECKOUTS = _("You must select checkout(s) to export"); + $(document).ready(function() { $('#finesholdsissues').tabs({ // Correct table sizing for tables hidden in tabs @@ -459,11 +466,11 @@ function validate1(date) { [% IF ( exports_enabled ) %] <fieldset> - <label for="export_formats"><b>Export checkouts using format:</b></label> - <select name="export_formats" id="export_formats"> + <label for="output_format"><b>Export checkouts using format:</b></label> + <select name="output_format" id="output_format"> <option value="iso2709_995">ISO2709 with items</option> <option value="iso2709">ISO2709 without items</option> - [% IF ( export_with_csv_profile ) %] + [% IF Koha.Preference('ExportWithCsvProfile') %] <option value="csv">CSV</option> [% END %] </select> --- a/tools/export.pl +++ a/tools/export.pl @@ -44,7 +44,7 @@ my $op = $query->param("op") || ''; my $filename = $query->param("filename") || 'koha.mrc'; my $dbh = C4::Context->dbh; my $marcflavour = C4::Context->preference("marcflavour"); -my $format = $query->param("format") || 'iso2709'; +my $format = $query->param("format") || $query->param("output_format") || 'iso2709'; # Checks if the script is called from commandline my $commandline = not defined $ENV{GATEWAY_INTERFACE}; --