diff --git a/module/VuFind/src/VuFind/Recommend/EuropeanaResults.php b/module/VuFind/src/VuFind/Recommend/EuropeanaResults.php index 0e57dcdf4abfc30d1e1aded084f90e2126072bbb..bbe8d472256c1d091339b60ff699d024ed2b155c 100644 --- a/module/VuFind/src/VuFind/Recommend/EuropeanaResults.php +++ b/module/VuFind/src/VuFind/Recommend/EuropeanaResults.php @@ -283,7 +283,7 @@ class EuropeanaResults implements RecommendInterface, } $parsedFeed = FeedReader::import($this->targetUrl); $resultsProcessed = array(); - foreach ($parsedFeed as $key => $value) { + foreach ($parsedFeed as $value) { $link = (string)$value->link; if (!empty($link)) { $resultsProcessed[] = array( diff --git a/module/VuFind/src/VuFind/Search/Base/Options.php b/module/VuFind/src/VuFind/Search/Base/Options.php index 59e60766eafa300a8e737e604a9dbb3ae339fc2f..100119408eefa03efd7ebfa6d6790bef886ef929 100644 --- a/module/VuFind/src/VuFind/Search/Base/Options.php +++ b/module/VuFind/src/VuFind/Search/Base/Options.php @@ -172,7 +172,6 @@ abstract class Options implements TranslatorAwareInterface { $label = empty($label) ? false : $this->translate($label); - $targetHandler = false; foreach ($this->getBasicHandlers() as $id => $currentLabel) { if ($this->translate($currentLabel) == $label) { return $id; diff --git a/module/VuFind/src/VuFind/Search/Solr/Params.php b/module/VuFind/src/VuFind/Search/Solr/Params.php index d5e7f89097bacc64a045983d086cd7768793bc88..4454b808943a2cf19527d6d2ea4934775986965d 100644 --- a/module/VuFind/src/VuFind/Search/Solr/Params.php +++ b/module/VuFind/src/VuFind/Search/Solr/Params.php @@ -136,7 +136,7 @@ class Params extends \VuFind\Search\Base\Params $facetSet = array(); if (!empty($this->facetConfig)) { $facetSet['limit'] = $this->facetLimit; - foreach ($this->facetConfig as $facetField => $facetName) { + foreach (array_keys($this->facetConfig) as $facetField) { if ($this->getFacetOperator($facetField) == 'OR') { $facetField = '{!ex=' . $facetField . '_filter}' . $facetField; }