From 855b685eeea58d403f86c84f7ad12f28f3d17785 Mon Sep 17 00:00:00 2001 From: Andrew Isherwood Date: Mon, 8 Oct 2018 13:33:46 +0100 Subject: [PATCH] Bug 20941: (follow-up) Switch 'media' -> 'types' Clearing up the inconsistency mentioned in comment #35. There is one place where we use the term 'media' for a template variable, everywhere we refer to material types as 'types'. NOTE: This is a breaking change for existing backends that still use 'media'. Of the PTFS Europe backends, only the Koha backend uses it, this will be modified as necessary. Generally backends will supply this variable themselves, so the breaking-ness of this change should be minimal. No test plan as it's backend dependent. Signed-off-by: Josef Moravec --- ill/ill-requests.pl | 2 +- opac/opac-illrequests.pl | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/ill/ill-requests.pl b/ill/ill-requests.pl index a2e898a..3a97969 100755 --- a/ill/ill-requests.pl +++ b/ill/ill-requests.pl @@ -273,7 +273,7 @@ if ( $backends_available ) { $template->param( backends => $backends, - media => [ "Book", "Article", "Journal" ], + types => [ "Book", "Article", "Journal" ], query_type => $op, branches => scalar Koha::Libraries->search, ); diff --git a/opac/opac-illrequests.pl b/opac/opac-illrequests.pl index 1fa1ce3..1b7807c 100755 --- a/opac/opac-illrequests.pl +++ b/opac/opac-illrequests.pl @@ -121,7 +121,7 @@ if ( $op eq 'list' ) { ); } else { $template->param( - media => [ "Book", "Article", "Journal" ], + types => [ "Book", "Article", "Journal" ], branches => Koha::Libraries->search->unblessed, whole => $backend_result, request => $request -- 2.1.4