diff --git a/module/VuFind/src/VuFind/RecordDriver/SolrMarc.php b/module/VuFind/src/VuFind/RecordDriver/SolrMarc.php index 1dd41657b98bd0ba88d8055fc987b6797517bc1a..9e788f26e8012c7ea77c5c6961ef77e8d58b990f 100644 --- a/module/VuFind/src/VuFind/RecordDriver/SolrMarc.php +++ b/module/VuFind/src/VuFind/RecordDriver/SolrMarc.php @@ -509,7 +509,7 @@ class SolrMarc extends SolrDefault // Loop through all subfields, collecting results that match the whitelist; // note that it is important to retain the original MARC order here! $allSubfields = $currentField->getSubfields(); - if (count($allSubfields) > 0) { + if (!empty($allSubfields)) { foreach ($allSubfields as $currentSubfield) { if (in_array($currentSubfield->getCode(), $subfields)) { // Grab the current subfield value and act on it if it is @@ -523,7 +523,7 @@ class SolrMarc extends SolrDefault } // Send back the data in a different format depending on $concat mode: - return $concat ? [implode($separator, $matches)] : $matches; + return $concat && $matches ? [implode($separator, $matches)] : $matches; } /**