Eliminated unused variables.

module/VuFind/src/VuFind/Config/PluginFactory.php

@@ -111,7 +111,7 @@ class PluginFactory implements AbstractFactoryInterface
                 ) {
                     $config->$section = $child->$section;
                 } else {
-                    foreach ($contents as $key => $value) {
+                    foreach (array_keys($contents->toArray()) as $key) {
                         $config->$section->$key = $child->$section->$key;
                     }
                 }

module/VuFind/src/VuFind/ILS/Logic/Holds.php

@@ -312,7 +312,6 @@ class Holds
                                 ? ($addlink && $copy['is_holdable']) : $addlink;
 
                             if ($addlink) {
-                                $holdLink = "";
                                 if ($checkHolds['function'] == "getHoldLink") {
                                     /* Build opac link */
                                     $holdings[$location_key][$copy_key]['link']

module/VuFind/src/VuFind/OAI/Server.php

@@ -512,9 +512,6 @@ class Server
             return;
         }
 
-        // Initialize the array of XML chunks to include in our response:
-        $xmlParts = array();
-
         // Copy the cursor from the parameters so we can track our current position
         // separately from our initial position!
         $currentCursor = $params['cursor'];

module/VuFind/src/VuFind/Recommend/SwitchQuery.php

@@ -151,7 +151,6 @@ class SwitchQuery implements RecommendInterface
 
         // Perform all checks (based on naming convention):
         $methods = get_class_methods($this);
-        $checks = array();
         foreach ($methods as $method) {
             if (substr($method, 0, 5) == 'check') {
                 $currentCheck = strtolower(substr($method, 5));