From 4c7195fed11ae79a9cb0ac26868f66ea02fb892d 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. --- 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 002b60f8f1..e7db6c492a 100755 --- a/opac/opac-illrequests.pl +++ b/opac/opac-illrequests.pl @@ -120,7 +120,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.11.0