From 74f2a9032475dcfd18141655bd2ef608c4c65801 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 Signed-off-by: Nick Clemens --- 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 a2e898a67c..3a97969152 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 1fa1ce386c..1b7807cb34 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.39.5