From 00bd97eeee24376fe7c2663d522c1bb54b814a20 Mon Sep 17 00:00:00 2001
From: Ere Maijala <ere.maijala@helsinki.fi>
Date: Thu, 27 Sep 2018 05:52:57 +0300
Subject: [PATCH] Use empty() instead of count() fixing a typo.
The comparison was misplaced.
---
module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php b/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php
index 92921883d2c..a338cd0293d 100644
--- a/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php
+++ b/module/VuFind/src/VuFind/ILS/Driver/VoyagerRestful.php
@@ -362,7 +362,7 @@ class VoyagerRestful extends Voyager implements \VuFindHttp\HttpServiceAwareInte
$valid_hold_statuses_array
= explode(':', $this->config['Holds']['valid_hold_statuses']);
- if (count($valid_hold_statuses_array > 0)) {
+ if (!empty($valid_hold_statuses_array)) {
foreach ($statusArray as $status) {
if (!in_array($status, $valid_hold_statuses_array)) {
$is_holdable = false;
--
GitLab