Load logger through service manager.

module/VuFind/config/module.config.php

@@ -262,12 +262,12 @@ $config = array(
         'factories' => array(
             'worldcatconnection' => function ($sm) {
                 $wc = new \VuFind\Connection\WorldCat();
-                $wc->setLogger(\VuFind\Log\Logger::getInstance());
+                $wc->setLogger($sm->get('Logger'));
                 return $wc;
             },
             'worldcatutils' => function ($sm) {
                 $wcu = new \VuFind\Connection\WorldCatUtils();
-                $wcu->setLogger(\VuFind\Log\Logger::getInstance());
+                $wcu->setLogger($sm->get('Logger'));
                 return $wcu;
             },
         ),
@@ -275,6 +275,7 @@ $config = array(
             'authmanager' => 'VuFind\Auth\Manager',
             'cart' => 'VuFind\Cart',
             'cachemanager' => 'VuFind\Cache\Manager',
+            'logger' => 'VuFind\Log\Logger',
             'recordloader' => 'VuFind\Record\Loader',
             'searchspecsreader' => 'VuFind\Config\SearchSpecsReader',
             'sessionmanager' => 'Zend\Session\SessionManager',

module/VuFind/src/VuFind/Connection/Manager.php

@@ -117,7 +117,7 @@ class Manager
         $index->setServiceLocator(self::$serviceLocator);
 
         // Set the logger:
-        $index->setLogger(\VuFind\Log\Logger::getInstance());
+        $index->setLogger(self::$serviceLocator->get('Logger'));
 
         return $index;
     }

module/VuFind/src/VuFind/Controller/CoverController.php

@@ -54,7 +54,7 @@ class CoverController extends AbstractBase
                 ConfigReader::getConfig(),
                 $this->getServiceLocator()->get('CacheManager')->getCacheDir()
             );
-            $this->loader->setLogger(\VuFind\Log\Logger::getInstance());
+            $this->loader->setLogger($this->getServiceLocator()->get('Logger'));
         }
         return $this->loader;
     }

module/VuFind/src/VuFind/Log/Logger.php

@@ -204,20 +204,6 @@ class Logger extends BaseLogger
         }
     }
 
-    /**
-     * Get the logger instance.
-     *
-     * @return Logger
-     */
-    public static function getInstance()
-    {
-        static $instance;
-        if (!$instance) {
-            $instance = new Logger();
-        }
-        return $instance;
-    }
-
     /**
      * Is one of the log writers listening for debug messages?  (This is useful to
      * know, since some code can save time that would be otherwise wasted generating

module/VuFind/src/VuFind/Search/Summon/Results.php

@@ -60,7 +60,7 @@ class Results extends BaseResults
             $id = isset($config->Summon->apiId) ? $config->Summon->apiId : null;
             $key = isset($config->Summon->apiKey) ? $config->Summon->apiKey : null;
             $conn = new SummonConnection($id, $key);
-            $conn->setLogger(\VuFind\Log\Logger::getInstance());
+            $conn->setLogger($this->getServiceLocator()->get('Logger'));
         }
         return $conn;
     }