tylercd100 / lern

src/Notifications/MonologHandlerFactory.php

Doc Comments +4 added lines, -5 removed lines

@@ -13,7 +13,7 @@ 
      * Creates a handler for a specified driver
      * @param  string $driver                   Lowercase driver string that is also in the config/lern.php file
      * @param  array  $subject                  Title or Subject line for the notification
-     * @return Monolog\Handler\HandlerInterface A handler to use with a Monolog\Logger instance
+     * @return \Monolog\Handler\HandlerInterface A handler to use with a Monolog\Logger instance
      */
     public function create($driver,$subject = null)
     {
@@ -25,7 +25,7 @@ 
     /**
      * Creates Pushover Monolog Handler
      * @param  array $subject  Title or Subject line for the notification
-     * @return PushoverHandler A handler to use with a Monolog\Logger instance
+     * @return \Monolog\Handler\PushoverHandler A handler to use with a Monolog\Logger instance
      */
     protected function pushover($subject)
     {
@@ -41,7 +41,7 @@ 
     /**
      * Creates Mail Monolog Handler
      * @param  array $subject      Title or Subject line for the notification
-     * @return NativeMailerHandler A handler to use with a Monolog\Logger instance
+     * @return \Monolog\Handler\NativeMailerHandler A handler to use with a Monolog\Logger instance
      */
     protected function mail($subject)
     {
@@ -55,8 +55,7 @@ 
 
     /**
      * Creates Slack Monolog Handler
-     * @param  array $subject Title or Subject line for the notification
-     * @return SlackHandler   A handler to use with a Monolog\Logger instance
+     * @return \Monolog\Handler\SlackHandler   A handler to use with a Monolog\Logger instance
      */
     protected function slack()
     {

Spacing +6 added lines, -6 removed lines

@@ -15,10 +15,10 @@ 
      * @param  array  $subject                  Title or Subject line for the notification
      * @return Monolog\Handler\HandlerInterface A handler to use with a Monolog\Logger instance
      */
-    public function create($driver,$subject = null)
+    public function create($driver, $subject = null)
     {
-        $this->config = config('lern.notify.'.$driver);
-        if(is_array($this->config))
+        $this->config = config('lern.notify.' . $driver);
+        if (is_array($this->config))
             return $this->{$driver}($subject);
     }
 
@@ -72,12 +72,12 @@ 
      * @param  mixed $subject [description]
      * @return [type]          [description]
      */
-    private function checkSubject($subject){
-        if(empty($subject)) {
+    private function checkSubject($subject) {
+        if (empty($subject)) {
             throw new Exception('$subject must not be empty!');
         }
 
-        if(!is_string($subject)) {
+        if (!is_string($subject)) {
             throw new Exception('$subject must be a string!');
         }
     }

Braces +3 added lines, -2 removed lines

@@ -18,8 +18,9 @@
     public function create($driver,$subject = null)
     {
         $this->config = config('lern.notify.'.$driver);
-        if(is_array($this->config))
-            return $this->{$driver}($subject);
+        if(is_array($this->config)) {
+                    return $this->{$driver}($subject);
+        }
     }
 
     /**

migrations/2016_03_17_000000_create_lern_tables.php

Spacing +1 added lines, -1 removed lines

@@ -12,7 +12,7 @@
      */
     public function up()
     {
-        Schema::create(config('lern.record.table'), function(Blueprint $table){
+        Schema::create(config('lern.record.table'), function(Blueprint $table) {
             $table->increments('id', true)->unsigned();
             $table->string('class');
             $table->string('file');

Unused Use Statements +1 added lines, -1 removed lines

@@ -1,7 +1,7 @@
 <?php
 
-use Illuminate\Database\Schema\Blueprint;
 use Illuminate\Database\Migrations\Migration;
+use Illuminate\Database\Schema\Blueprint;
 
 class CreateLERNTables extends Migration {
 

src/LERNServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -6,10 +6,10 @@
 
 class LERNServiceProvider extends ServiceProvider
 {
-    public function register(){
+    public function register() {
         $this->mergeConfigFrom(__DIR__ . '/../config/lern.php', 'lern');
 
-        $this->app->singleton('lern', function () {
+        $this->app->singleton('lern', function() {
             return new LERN;
         });
     }

src/Notifications/Notifier.php

Spacing +16 added lines, -16 removed lines

@@ -17,8 +17,8 @@ 
      * You can provide a Monolog Logger instance to use in the constructor 
      * @param Logger|null $log Logger instance to use
      */
-    public function __construct(Logger $log = null){
-        if($log === null){
+    public function __construct(Logger $log = null) {
+        if ($log === null) {
             $log = new Logger(config('lern.notify.channel'));
         }
 
@@ -32,9 +32,9 @@ 
      */
     public function setMessage($cb)
     {
-        if(is_string($cb)) {
+        if (is_string($cb)) {
             $this->messageCb = function() use ($cb) { return $cb; };
-        } else if(is_callable($cb)) {
+        } else if (is_callable($cb)) {
             $this->messageCb = $cb;
         }
 
@@ -46,13 +46,13 @@ 
      * @param  Exception $e The Exception instance that you want to build the message around
      * @return string       The message string
      */
-    public function getMessage(Exception $e){
-        if(is_callable($this->messageCb)){
+    public function getMessage(Exception $e) {
+        if (is_callable($this->messageCb)) {
             return $this->messageCb->__invoke($e);
         } else {
-            $msg = get_class($e) . " was thrown! \n".$e->getMessage();
-            if($this->config['includeExceptionStackTrace']===true){
-                $msg.="\n\n".$e->getTraceAsString();
+            $msg = get_class($e) . " was thrown! \n" . $e->getMessage();
+            if ($this->config['includeExceptionStackTrace'] === true) {
+                $msg .= "\n\n" . $e->getTraceAsString();
             }
             return $msg;
         }
@@ -64,9 +64,9 @@ 
      */
     public function setSubject($cb)
     {
-        if(is_string($cb)) {
+        if (is_string($cb)) {
             $this->subjectCb = function() use ($cb) { return $cb; };
-        } else if(is_callable($cb)) {
+        } else if (is_callable($cb)) {
             $this->subjectCb = $cb;
         }
 
@@ -78,8 +78,8 @@ 
      * @param  Exception $e The Exception instance that you want to build the subject around
      * @return string       The subject string
      */
-    public function getSubject(Exception $e){
-        if(is_callable($this->subjectCb)){
+    public function getSubject(Exception $e) {
+        if (is_callable($this->subjectCb)) {
             return $this->subjectCb->__invoke($e);
         } else {
             return get_class($e);
@@ -91,7 +91,7 @@ 
      * @param  HandlerInterface $handler The handler instance to add on
      * @return Notifier                  Returns this
      */
-    public function pushHandler(HandlerInterface $handler){
+    public function pushHandler(HandlerInterface $handler) {
         $this->log->pushHandler($handler);
         return $this;
     }
@@ -101,7 +101,7 @@ 
      * @param  Exception $e The exception to use
      * @return Notifier     Returns this
      */
-    public function send(Exception $e){
+    public function send(Exception $e) {
         $factory = new MonologHandlerFactory();
         $drivers = $this->config['drivers'];
 
@@ -109,7 +109,7 @@ 
         $subject = $this->getSubject($e);
         
         foreach ($drivers as $driver) {
-            $handler = $factory->create($driver,$subject);
+            $handler = $factory->create($driver, $subject);
             $this->log->pushHandler($handler);
         }
 

Unused Use Statements +1 added lines, -1 removed lines

@@ -3,8 +3,8 @@
 namespace Tylercd100\LERN\Notifications;
 
 use Exception;
-use Monolog\Logger;
 use Monolog\Handler\HandlerInterface;
+use Monolog\Logger;
 use Tylercd100\LERN\Notifications\MonologHandlerFactory;
 
 class Notifier {

src/LERN.php

Spacing +2 added lines, -2 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function __construct(Notifier $notifier = null)
     {
-        if(empty($notifier))
+        if (empty($notifier))
             $notifier = new Notifier();
         $this->notifier = $notifier;
     }
@@ -55,7 +55,7 @@ 
             'trace'       => $e->getTraceAsString(),
         ];
 
-        if($e instanceof HttpExceptionInterface)
+        if ($e instanceof HttpExceptionInterface)
             $opts['status_code'] = $e->getStatusCode();
 
         return ExceptionModel::create($opts);

Braces +6 added lines, -4 removed lines

@@ -21,8 +21,9 @@ 
      */
     public function __construct(Notifier $notifier = null)
     {
-        if(empty($notifier))
-            $notifier = new Notifier();
+        if(empty($notifier)) {
+                    $notifier = new Notifier();
+        }
         $this->notifier = $notifier;
     }
 
@@ -55,8 +56,9 @@ 
             'trace'       => $e->getTraceAsString(),
         ];
 
-        if($e instanceof HttpExceptionInterface)
-            $opts['status_code'] = $e->getStatusCode();
+        if($e instanceof HttpExceptionInterface) {
+                    $opts['status_code'] = $e->getStatusCode();
+        }
 
         return ExceptionModel::create($opts);
     }

src/Handler.php

Unused Use Statements +1 added lines, -1 removed lines

@@ -3,8 +3,8 @@
 namespace Tylercd100\LERN;
 
 use Exception;
-use Tylercd100\LERN\Facades\LERN;
 use Illuminate\Foundation\Exceptions\Handler as ExceptionHandler;
+use Tylercd100\LERN\Facades\LERN;
     
 class Handler extends ExceptionHandler {