Drop Resque\Logger class (#77)

* Drop Resque\Logger class

This removes the internal PSR-3 logger. Now, if no custom
logger is set up in the APP_INCLUDE, logging is disabled.

* Relax psr/log requirement so projects can use a newer version

Author: pprkut

bin/resque

@@ -68,9 +68,10 @@ if($APP_INCLUDE) {
 }
 
 // See if the APP_INCLUDE containes a logger object,
-// If none exists, fallback to internal logger
+// If none exists, use the NullLogger instead, which
+// effectively disables logging.
 if (!isset($logger) || !is_object($logger)) {
-    $logger = new \Resque\Logger($logLevel);
+    $logger = new \Psr\Log\NullLogger();
 }
 
 $BLOCKING = getenv('BLOCKING') !== FALSE;

composer.json

@@ -30,7 +30,7 @@
 	"require": {
 		"php": ">=5.6.0",
 		"colinmollenhour/credis": "~1.7",
-		"psr/log": "~1.0"
+		"psr/log": ">=1.1.0"
 	},
 	"suggest": {
 		"ext-pcntl": "REQUIRED for forking processes on platforms that support it (so anything but Windows).",

lib/Logger.php

@@ -4,10 +4,10 @@
 
 namespace Resque\Worker;
 
-use Resque\Logger;
 use Resque\Resque;
 use CredisException;
 use Psr\Log\LogLevel;
+use Psr\Log\NullLogger;
 use Resque\Job\PID;
 use Resque\Event;
 use Resque\Exceptions\DirtyExitException;
@@ -91,7 +91,7 @@ class ResqueWorker
 	 */
 	public function __construct($queues)
 	{
-		$this->logger = new Logger();
+		$this->logger = new NullLogger();
 
 		if (!is_array($queues)) {
 			$queues = array($queues);