Stanvk / Magister

src/Magister/Models/Enrollment/Enrollment.php

Doc Comments +1 added lines, -1 removed lines

@@ -44,7 +44,7 @@
     /**
      * Define a relationship.
      *
-     * @return mixed
+     * @return \Magister\Services\Database\Elegant\Relations\HasMany
      */
     public function counsellors()
     {

src/Magister/Services/Auth/Guard.php

Doc Comments +2 added lines, -2 removed lines

@@ -153,7 +153,7 @@ 
     /**
      * Determine if the user matches the credentials.
      *
-     * @param mixed $user
+     * @param \Magister\Services\Database\Elegant\Model|null $user
      *
      * @return bool
      */
@@ -286,7 +286,7 @@ 
     /**
      * Return the currently cached user.
      *
-     * @return mixed|null
+     * @return UserContract
      */
     public function getUser()
     {

Spacing +6 added lines, -6 removed lines

@@ -65,7 +65,7 @@ 
      */
     public function check()
     {
-        return !is_null($this->user());
+        return ! is_null($this->user());
     }
 
     /**
@@ -75,7 +75,7 @@ 
      */
     public function guest()
     {
-        return !$this->check();
+        return ! $this->check();
     }
 
     /**
@@ -92,7 +92,7 @@ 
         // If we have already retrieved the user for the current request we can just
         // return it back immediately. We do not want to pull the user data every
         // request into the method because that would tremendously slow an app.
-        if (!is_null($this->user)) {
+        if ( ! is_null($this->user)) {
             return $this->user;
         }
 
@@ -100,7 +100,7 @@ 
 
         $user = null;
 
-        if (!is_null($id)) {
+        if ( ! is_null($id)) {
             $user = $this->provider->retrieveByToken();
         }
 
@@ -159,7 +159,7 @@ 
      */
     protected function hasValidCredentials($user)
     {
-        return !is_null($user);
+        return ! is_null($user);
     }
 
     /**
@@ -314,7 +314,7 @@ 
      */
     public function getCookieJar()
     {
-        if (!isset($this->cookie)) {
+        if ( ! isset($this->cookie)) {
             throw new \RuntimeException('Cookie jar has not been set.');
         }
 

src/Magister/Services/Container/Container.php

Doc Comments +3 added lines, -3 removed lines

@@ -36,7 +36,7 @@ 
      * Register a service as a singleton.
      *
      * @param string $abstract
-     * @param mixed  $concrete
+     * @param Closure  $concrete
      *
      * @return void
      */
@@ -209,7 +209,7 @@ 
      *
      * @param string $abstract
      *
-     * @return bool
+     * @return boolean|null
      */
     public function __isset($abstract)
     {
@@ -258,7 +258,7 @@ 
      *
      * @param string $abstract
      *
-     * @return bool
+     * @return boolean|null
      */
     public function offsetExists($abstract)
     {

Spacing +7 added lines, -7 removed lines

@@ -56,7 +56,7 @@ 
      */
     public function make($abstract)
     {
-        if (!$this->bound($abstract)) {
+        if ( ! $this->bound($abstract)) {
             throw new InvalidArgumentException(sprintf('Identifier "%s" is not defined.', $abstract));
         }
 
@@ -99,7 +99,7 @@ 
      */
     public static function share(Closure $callable)
     {
-        return function ($c) use ($callable) {
+        return function($c) use ($callable) {
             static $object;
 
             if (null === $object) {
@@ -119,7 +119,7 @@ 
      */
     public static function protect(Closure $callable)
     {
-        return function ($c) use ($callable) {
+        return function($c) use ($callable) {
             return $callable;
         };
     }
@@ -135,7 +135,7 @@ 
      */
     public function raw($abstract)
     {
-        if (!array_key_exists($abstract, $this->bindings)) {
+        if ( ! array_key_exists($abstract, $this->bindings)) {
             throw new InvalidArgumentException(sprintf('Identifier "%s" is not defined.', $abstract));
         }
 
@@ -154,17 +154,17 @@ 
      */
     public function extend($abstract, Closure $callable)
     {
-        if (!array_key_exists($abstract, $this->bindings)) {
+        if ( ! array_key_exists($abstract, $this->bindings)) {
             throw new InvalidArgumentException(sprintf('Identifier "%s" is not defined.', $abstract));
         }
 
         $factory = $this->bindings[$abstract];
 
-        if (!($factory instanceof Closure)) {
+        if ( ! ($factory instanceof Closure)) {
             throw new InvalidArgumentException(sprintf('Identifier "%s" does not contain an object definition.', $abstract));
         }
 
-        return $this->bindings[$abstract] = function ($c) use ($callable, $factory) {
+        return $this->bindings[$abstract] = function($c) use ($callable, $factory) {
             return $callable($factory($c), $c);
         };
     }

src/Magister/Services/Database/Query/Builder.php

Doc Comments +2 added lines, -2 removed lines

@@ -72,7 +72,7 @@ 
      * @param string $column
      * @param string $key
      *
-     * @return array
+     * @return Collection
      */
     public function lists($column, $key = null)
     {
@@ -200,7 +200,7 @@ 
      *
      * @throws \BadMethodCallException
      *
-     * @return mixed
+     * @return Builder
      */
     public function __call($method, $parameters)
     {

Spacing +1 added lines, -1 removed lines

@@ -95,7 +95,7 @@
     {
         $select = is_null($key) ? [$column] : [$column, $key];
 
-        return array_map(function ($column) {
+        return array_map(function($column) {
             $dot = strpos($column, '.');
 
             return $dot === false ? $column : substr($column, $dot + 1);

src/Magister/Services/Events/Dispatcher.php

Doc Comments +1 added lines, -1 removed lines

@@ -155,7 +155,7 @@
     /**
      * Create a class based listener using the IoC container.
      *
-     * @param mixed $listener
+     * @param string $listener
      *
      * @return \Closure
      */

Spacing +4 added lines, -4 removed lines

@@ -79,7 +79,7 @@ 
     {
         $responses = [];
 
-        if (!is_array($payload)) {
+        if ( ! is_array($payload)) {
             $payload = [$payload];
         }
 
@@ -88,7 +88,7 @@ 
         foreach ($this->getListeners($event) as $listener) {
             $response = call_user_func_array($listener, $payload);
 
-            if (!is_null($response) && $halt) {
+            if ( ! is_null($response) && $halt) {
                 return $response;
             }
 
@@ -111,7 +111,7 @@ 
      */
     public function getListeners($eventName)
     {
-        if (!isset($this->sorted[$eventName])) {
+        if ( ! isset($this->sorted[$eventName])) {
             $this->sortListeners($eventName);
         }
 
@@ -163,7 +163,7 @@ 
     {
         $container = $this->container;
 
-        return function () use ($listener, $container) {
+        return function() use ($listener, $container) {
             $segments = explode('@', $listener);
 
             $method = count($segments) == 2 ? $segments[1] : 'handle';

src/Magister/Services/Support/Collection.php

Doc Comments +1 added lines, -1 removed lines

@@ -202,7 +202,7 @@
     /**
      * Remove an item from the collection by key.
      *
-     * @param mixed $key
+     * @param string $key
      *
      * @return void
      */

Spacing +8 added lines, -8 removed lines

@@ -64,13 +64,13 @@ 
     public function contains($key, $value = null)
     {
         if (func_num_args() == 2) {
-            return $this->contains(function ($k, $item) use ($key, $value) {
+            return $this->contains(function($k, $item) use ($key, $value) {
                 return data_get($item, $key) == $value;
             });
         }
 
         if ($this->useAsCallable($key)) {
-            return !is_null($this->first($key));
+            return ! is_null($this->first($key));
         }
 
         return in_array($key, $this->items);
@@ -129,7 +129,7 @@ 
      */
     public function where($key, $value, $strict = true)
     {
-        return $this->filter(function ($item) use ($key, $value, $strict) {
+        return $this->filter(function($item) use ($key, $value, $strict) {
             return $strict ? data_get($item, $key) === $value : data_get($item, $key) == $value;
         });
     }
@@ -157,7 +157,7 @@ 
      */
     public function find($key, $default = null)
     {
-        return array_first($this->items, function ($itemKey, $model) use ($key) {
+        return array_first($this->items, function($itemKey, $model) use ($key) {
             return $itemKey == $key;
         }, $default);
     }
@@ -245,7 +245,7 @@ 
         foreach ($this->items as $key => $value) {
             $groupKey = $groupBy($value, $key);
 
-            if (!array_key_exists($groupKey, $results)) {
+            if ( ! array_key_exists($groupKey, $results)) {
                 $results[$groupKey] = new static();
             }
 
@@ -337,7 +337,7 @@ 
      */
     protected function useAsCallable($value)
     {
-        return !is_string($value) && is_callable($value);
+        return ! is_string($value) && is_callable($value);
     }
 
     /**
@@ -651,7 +651,7 @@ 
             return $value;
         }
 
-        return function ($item) use ($value) {
+        return function($item) use ($value) {
             return data_get($item, $value);
         };
     }
@@ -683,7 +683,7 @@ 
      */
     public function toArray()
     {
-        return array_map(function ($value) {
+        return array_map(function($value) {
             return $value instanceof Arrayable ? $value->toArray() : $value;
         }, $this->items);
     }

src/Magister/Magister.php

Spacing +2 added lines, -2 removed lines

@@ -160,7 +160,7 @@ 
             return;
         }
 
-        array_walk($this->serviceProviders, function ($p) {
+        array_walk($this->serviceProviders, function($p) {
             $this->bootProvider($p);
         });
 
@@ -247,7 +247,7 @@ 
     {
         $name = is_string($provider) ? $provider : get_class($provider);
 
-        return array_first($this->serviceProviders, function ($key, $value) use ($name) {
+        return array_first($this->serviceProviders, function($key, $value) use ($name) {
             return $value instanceof $name;
         });
     }

src/Magister/Services/Auth/AuthServiceProvider.php

Spacing +2 added lines, -2 removed lines

@@ -26,7 +26,7 @@ 
      */
     protected function registerAuthenticator()
     {
-        $this->app->singleton('auth', function ($app) {
+        $this->app->singleton('auth', function($app) {
             return new AuthManager($app);
         });
     }
@@ -41,7 +41,7 @@ 
         if ($this->app->bound('credentials')) {
             $auth = $this->app->auth;
 
-            if (!$auth->check()) {
+            if ( ! $auth->check()) {
                 $auth->attempt($this->app['credentials']);
             }
         }

src/Magister/Services/Encryption/EncryptionServiceProvider.php

Spacing +1 added lines, -1 removed lines

@@ -16,7 +16,7 @@
      */
     public function register()
     {
-        $this->app->singleton('encrypter', function ($app) {
+        $this->app->singleton('encrypter', function($app) {
             return new Encrypter($app['config']['app.key']);
         });
     }