From 41649012c93fd9e591b61421b8666bde250ca4aa Mon Sep 17 00:00:00 2001 From: Kyle M Hall Date: Thu, 3 Dec 2015 14:56:30 +0000 Subject: [PATCH] Bug 8352 [QA Followup] --- installer/data/mysql/atomicupdate/bug_8352.sql | 6 +++--- installer/data/mysql/en/mandatory/sample_notices.sql | 5 +++-- .../intranet-tmpl/prog/en/includes/printers-toolbar.inc | 4 ++-- .../intranet-tmpl/prog/en/modules/admin/admin-home.tt | 2 +- koha-tmpl/intranet-tmpl/prog/en/modules/admin/printers.tt | 4 ++-- .../intranet-tmpl/prog/en/modules/help/admin/printers.tt | 6 +++--- misc/cronjobs/holds/print_holds.pl | 13 ++++++++----- svc/printer | 1 - 8 files changed, 22 insertions(+), 19 deletions(-) diff --git a/installer/data/mysql/atomicupdate/bug_8352.sql b/installer/data/mysql/atomicupdate/bug_8352.sql index 3b7308d..85d48cd 100644 --- a/installer/data/mysql/atomicupdate/bug_8352.sql +++ b/installer/data/mysql/atomicupdate/bug_8352.sql @@ -1,12 +1,12 @@ ALTER TABLE reserves ADD printed DATETIME NULL AFTER suspend_until; ALTER TABLE old_reserves ADD printed DATETIME NULL AFTER suspend_until; -INSERT INTO letter ( module, code, branchcode, name, is_html, title, content ) VALUES ( - 'reserves', 'HOLD_PLACED_PRINT', '', 'Hold Placed ( Auto-Print )', '0', 'Hold Placed ( Auto-Print )', 'Hold to pull at <> +INSERT INTO letter ( module, code, branchcode, name, is_html, title, content, message_transport_type ) VALUES ( + 'reserves', 'HOLDPLACED', '', 'Hold Placed ( Auto-Print )', '0', 'Hold Placed ( Auto-Print )', 'Hold to pull at <> For <> <> ( <> ) -<> by <>'); +<> by <>', 'print'); ALTER TABLE printers DROP PRIMARY KEY, diff --git a/installer/data/mysql/en/mandatory/sample_notices.sql b/installer/data/mysql/en/mandatory/sample_notices.sql index f589bcb..f877fdb 100644 --- a/installer/data/mysql/en/mandatory/sample_notices.sql +++ b/installer/data/mysql/en/mandatory/sample_notices.sql @@ -162,5 +162,6 @@ VALUES ( 'circulation', 'OVERDUES_SLIP', '', 'Overdues Slip', '0', 'OVERDUES_SLI "<>" by <>, <>, Barcode: <> Fine: <> ', 'print' ); -INSERT INTO `letter` (`module`,`code`,`branchcode`,`name`,`is_html`,`title`,`content`) -VALUES ('reserves', 'HOLD_PLACED_PRINT', 'Hold Placed ( Auto-Print )', 'Hold Placed ( Auto-Print )', 'Hold to pull at <>\r\nFor <> <> ( <> )\r\n<> by <>'); + +INSERT INTO letter ( module, code, branchcode, name, is_html, title, content, message_transport_type ) +VALUES ('reserves', 'HOLDPLACED', 'Hold Placed ( Auto-Print )', 'Hold Placed ( Auto-Print )', 'Hold to pull at <>\r\nFor <> <> ( <> )\r\n<> by <>', 'print'); diff --git a/koha-tmpl/intranet-tmpl/prog/en/includes/printers-toolbar.inc b/koha-tmpl/intranet-tmpl/prog/en/includes/printers-toolbar.inc index 416de09..6a07794 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/includes/printers-toolbar.inc +++ b/koha-tmpl/intranet-tmpl/prog/en/includes/printers-toolbar.inc @@ -1,4 +1,4 @@ diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/admin-home.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/admin-home.tt index ecdb388..990f55a 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/admin-home.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/admin-home.tt @@ -106,7 +106,7 @@
Printers
Define your configured network printers
-
Z39.50 client targets
+
Z39.50/SRU servers
Define which servers to query for MARC data in the integrated Z39.50 client.
Did you mean?
diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/printers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/printers.tt index 57926b5..aa2cc0c 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/admin/printers.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/admin/printers.tt @@ -1,5 +1,5 @@ [% INCLUDE 'doc-head-open.inc' %] -Koha › Administration › Printer editor +Koha › Administration › Printers [% INCLUDE 'doc-head-close.inc' %] [% INCLUDE 'datatables.inc' %] @@ -147,7 +147,7 @@ $(document).ready(function() { if (!idsToDelete) { alert(_("No printers selected!")); } - else if (confirm(_("Are you sure you wish to delete printer(s) ") + idsToDelete + "?")) { + else if (confirm(_("Are you sure you wish to delete printer(s) %s?").format(idsToDelete))) { oTable.$('.selected').each(function(){ var printerID = $(this).attr('id'); $.ajax({ diff --git a/koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/printers.tt b/koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/printers.tt index 36abeed..29e14dc 100644 --- a/koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/printers.tt +++ b/koha-tmpl/intranet-tmpl/prog/en/modules/help/admin/printers.tt @@ -8,7 +8,7 @@

Click on any field to edit the contents; Press the <Enter> key to save edit.

Click on one or more printer IDs to select entire printers for deletion; Click the 'Delete printer(s)' button to delete selected printers.

-

Printers defined in this area can be used to print automatic reserve notices

+

Printers defined in this area can be used to print automatic hold notices

How printers are defined

Printers can be utilized in two ways, either by sending the data directly to a networked printer via lpr, or by printing to a printer set up on the Koha server via CUPS.

@@ -25,10 +25,10 @@

Caveats

- Printing plain text is supported by both methods. When printing HTML, the output will be sent to the printer as PostScript. If you are printing via LPR, and you want to print HTML, you will need to ensure that your printer can recieve and print PostScript data. + Printing plain text is supported by both methods. When printing HTML, the output will be sent to the printer as PostScript. If you are printing via LPR, and you want to print HTML, you will need to ensure that your printer can receive and print PostScript data.

- For printing slips and notices, this behavior can be controlled via the "HTML message" flag of the HOLD_PLACED_PRINT notice. Leaving this checkbox unchecked will result in plain text being printed, whereas checking this checkbox will result in HTML converted to PostScript being printed. + For printing slips and notices, this behavior can be controlled via the "HTML message" flag of the HOLDPLACED print notice. Leaving this checkbox unchecked will result in plain text being printed, whereas checking this checkbox will result in HTML converted to PostScript being printed.

[% INCLUDE 'help-bottom.inc' %] diff --git a/misc/cronjobs/holds/print_holds.pl b/misc/cronjobs/holds/print_holds.pl index 826e5be..8117c43 100755 --- a/misc/cronjobs/holds/print_holds.pl +++ b/misc/cronjobs/holds/print_holds.pl @@ -76,10 +76,11 @@ while ( my $hold = $holds->next() ) { my $borrower = Koha::Borrowers->find( $hold->borrowernumber ); my $letter = GetPreparedLetter( - module => 'reserves', - letter_code => 'HOLD_PLACED_PRINT', - branchcode => $hold->branchcode, - tables => { + module => 'reserves', + letter_code => 'HOLDPLACED', + branchcode => $hold->branchcode, + message_transport_type => 'print', + tables => { branches => $hold->branchcode, biblio => $hold->biblionumber, biblioitems => $hold->biblionumber, @@ -87,7 +88,7 @@ while ( my $hold = $holds->next() ) { borrowers => $borrower->unblessed, reserves => $hold->unblessed, - } + }, ); if ( defined( $printers{ $hold->branchcode } ) ) { @@ -155,6 +156,8 @@ Run in test mode. Holds will not actually be printed. Run in production mode. Holds will be printed to printers. +=back + =head1 DESCRIPTION B will print on-demand notices for items in the holds queue as they appear. diff --git a/svc/printer b/svc/printer index 04cd775..ba78af3 100755 --- a/svc/printer +++ b/svc/printer @@ -1,7 +1,6 @@ #!/usr/bin/perl # Copyright 2014 ByWater Solutions -# Copyright 2012 Foundations Bible College Inc. # # This file is part of Koha. # -- 1.7.10.4