unyx / nyx

src/auth/id/protocols/oauth1/middlewares/Authorization.php

Doc Comments +1 added lines, -1 removed lines

@@ -104,7 +104,7 @@
      * See the class description for which kind of additional options are supported/mandatory.
      *
      * @param   callable    $handler
-     * @return  callable
+     * @return  \Closure
      */
     public function __invoke(callable $handler) : callable
     {

Spacing +5 added lines, -5 removed lines

@@ -89,7 +89,7 @@ 
      */
     public function setMethod(string $method) : Authorization
     {
-        switch($method) {
+        switch ($method) {
             case self::METHOD_HEADER:
             case self::METHOD_QUERY:
                 $this->method = $method;
@@ -108,7 +108,7 @@ 
      */
     public function __invoke(callable $handler) : callable
     {
-        return function ($request, array& $stackOptions) use ($handler) {
+        return function($request, array& $stackOptions) use ($handler) {
 
             // Skip to the next handler if we weren't asked to do any stuff.
             if (!isset($stackOptions['oauth1'])) {
@@ -119,7 +119,7 @@ 
             // so we might just as well make all the parameters we gather and append more easily accessible
             // by pushing into the $stackOptions directly in case there actually *are* handlers in the stack
             // that have to do some post-processing after us.
-            $handlerOptions =& $stackOptions['oauth1'] ?: $stackOptions['oauth1'] = [];
+            $handlerOptions = & $stackOptions['oauth1'] ?: $stackOptions['oauth1'] = [];
 
             $this->parseOptions($handlerOptions);
             $this->gatherAuthorizationParams($handlerOptions);
@@ -187,7 +187,7 @@ 
         // Unite our base parameters (which cannot be overridden) with the optional ones passed in.
         $options['params'] = [
             'oauth_version'          => '1.0',
-            'oauth_consumer_key'     => isset($options['client']) ? $options['client']->getId()     : $this->client->getId(),
+            'oauth_consumer_key'     => isset($options['client']) ? $options['client']->getId() : $this->client->getId(),
             'oauth_signature_method' => isset($options['signer']) ? $options['signer']->getMethod() : $this->signer->getMethod(),
             'oauth_nonce'            => utils\Random::string(6, utils\str\Character::CHARS_BASE64, utils\Random::STRENGTH_NONE),
             'oauth_timestamp'        => time(),
@@ -251,7 +251,7 @@ 
             $params[$key] = $key.'="'.rawurlencode($value).'"';
         }
 
-        return $request->withHeader('Authorization', 'OAuth ' . implode(', ', $params));
+        return $request->withHeader('Authorization', 'OAuth '.implode(', ', $params));
     }
 
     /**

src/notify/transports/mail/drivers/Postmark.php

Doc Comments +1 added lines, -1 removed lines

@@ -158,7 +158,7 @@
     /**
      * Converts an array of e-mail addresses into a string compliant with Postmark's API's format.
      *
-     * @param   array   $emails     The e-mail addresses to convert.
+     * @param   string[]   $emails     The e-mail addresses to convert.
      * @return  string
      */
     protected function emailsToString(array $emails) : string

Spacing +3 added lines, -3 removed lines

@@ -46,7 +46,7 @@ 
      */
     public function send(\Swift_Mime_Message $message, &$failures = null)
     {
-        $this->client->request('POST','https://api.postmarkapp.com/email', [
+        $this->client->request('POST', 'https://api.postmarkapp.com/email', [
             'headers' => [
                 'X-Postmark-Server-Token' => $this->key
             ],
@@ -102,7 +102,7 @@ 
 
                 // Special treatment for the 'X-PM-Tag' header if it's available, which we'll pass into the payload
                 // directly.
-                if ($fieldName === 'X-PM-Tag'){
+                if ($fieldName === 'X-PM-Tag') {
                     $payload["Tag"] = $header->getValue();
                 } else {
                     $payload['Headers'][] = [
@@ -166,7 +166,7 @@ 
         $addresses = [];
 
         foreach ($emails as $email => $name) {
-            $addresses[] = $name ? '"' . str_replace('"', '\\"', $name) . "\" <{$email}>" : $email;
+            $addresses[] = $name ? '"'.str_replace('"', '\\"', $name)."\" <{$email}>" : $email;
         }
 
         return implode(',', $addresses);

src/notify/transports/mail/drivers/Sendgrid.php

Doc Comments +2 added lines, -2 removed lines

@@ -152,7 +152,7 @@ 
      * Parses the first and only the first address from the given array of e-mail addresses into a structure
      * understood by Sendgrid's API.
      *
-     * @param   array   $addresses  The e-mail addresses to parse.
+     * @param   string[]   $addresses  The e-mail addresses to parse.
      * @return  array
      */
     protected function processFirstAddress(array $addresses) : array
@@ -172,7 +172,7 @@ 
     /**
      * Parses the given array of e-mail addresses into a structure understood by Sendgrid's API.
      *
-     * @param   array   $addresses  The e-mail addresses to parse.
+     * @param   string[]   $addresses  The e-mail addresses to parse.
      * @return  array
      */
     protected function processAllAddresses(array $addresses) : array

src/notify/transports/slack/Message.php

Doc Comments +1 added lines, -1 removed lines

@@ -403,7 +403,7 @@
     /**
      * Attempts to create a slack\Attachment instance based on the $attachment data given.
      *
-     * @param   mixed   $attachment
+     * @param   callable   $attachment
      * @return  mixed                   Either an instantiated slack\Attachment, or a passthrough of the input data.
      */
     protected function resolveAttachment($attachment)

Spacing +1 added lines, -1 removed lines

@@ -105,7 +105,7 @@
         }
 
         if (isset($attributes['attachments'])) {
-            foreach($attributes['attachments'] as $attachment) {
+            foreach ($attributes['attachments'] as $attachment) {
                 $this->attach($attachment);
             }
         }

src/utils/math/bigint/Gmp.php

Doc Comments +1 added lines, -1 removed lines

@@ -108,7 +108,7 @@
     /**
      * Constructs a new GMP BigInt instance.
      *
-     * @param   \GMP    $value  The GMP object holding the actual value.
+     * @param   resource    $value  The GMP object holding the actual value.
      */
     public function __construct(\GMP $value)
     {

Spacing +2 added lines, -2 removed lines

@@ -84,7 +84,7 @@ 
                 } else {
                     $matches[1] = '';
                 }
-                $number = str_pad(($matches[1] . $matches[2]), ($matches[3] + 1), '0', STR_PAD_RIGHT);
+                $number = str_pad(($matches[1].$matches[2]), ($matches[3] + 1), '0', STR_PAD_RIGHT);
             } else {
                 $base = 0;
             }
@@ -275,7 +275,7 @@ 
         $hex = gmp_strval($decimal, 16);
 
         if (strlen($hex) & 1) {
-            $hex = '0' . $hex;
+            $hex = '0'.$hex;
         }
 
         // Pack the hexadecimals into a string and remove the nullbyte.

src/utils/tests/FuncTest.php

Doc Comments +4 added lines

@@ -414,6 +414,10 @@
     }
 
     // Used by the when/unless tests.
+
+    /**
+     * @param boolean $exists
+     */
     protected function setWhenExists($exists)
     {
         $this->whenExists = (bool) $exists;

Spacing +31 added lines, -31 removed lines

@@ -24,7 +24,7 @@ 
         $runs  = 11;
 
         // Let's create a simple helper.
-        $test = function (callable $wrapper, callable $function, $value) use ($times, $runs) {
+        $test = function(callable $wrapper, callable $function, $value) use ($times, $runs) {
 
             // Invoke the function once to determine the expected value.
             $expected = $function($value);
@@ -39,8 +39,8 @@ 
         };
 
         // Prepare our functions.
-        $func1 = function ($value) {return 'foo'.$value;};
-        $func2 = function ($value) {return $value.'bar';};
+        $func1 = function($value) {return 'foo'.$value; };
+        $func2 = function($value) {return $value.'bar'; };
 
         // Run a few loops.
         $wrapper = Func::after($times, $func1);
@@ -54,7 +54,7 @@ 
     public function testHash()
     {
         // Prepare our functions.
-        $func1 = function ($arg1, $arg2) {return true;};
+        $func1 = function($arg1, $arg2) {return true; };
 
         // Run some tests.
         $this->assertEquals('8bdb4ed5e8d60590121851789f7c5366', Func::hash($func1, ['arg1', 'arg2']));
@@ -77,7 +77,7 @@ 
     public function testMemoize()
     {
         // Prepare our functions.
-        $func = function ($arg1, $arg2) {return $arg1.$arg2;};
+        $func = function($arg1, $arg2) {return $arg1.$arg2; };
 
         // Without resolver (automatic cache key).
         $memoized = Func::memoize($func);
@@ -88,7 +88,7 @@ 
         $this->assertNotEquals('foobar', $memoized('bar', 'foo'));
 
         // With resolver (key depends on args).
-        $resolver = function (callable $func, $args) {
+        $resolver = function(callable $func, $args) {
             return 'key_'.$args[0].$args[1];
         };
         $memoized = Func::memoize($func, $resolver);
@@ -100,7 +100,7 @@ 
 
         // With resolver (fixed key) - all calls should return the first result, regardless of the arguments
         // passed, since the result will be fetched from the fixed key.
-        $resolver = function (callable $func, $args) {return 'key';};
+        $resolver = function(callable $func, $args) {return 'key'; };
         $memoized = Func::memoize($func, $resolver);
 
         $this->assertEquals('foobar', $memoized('foo', 'bar'));
@@ -119,8 +119,8 @@ 
     public function testOnce()
     {
         // Prepare our functions.
-        $func1 = function ($value) {return 'foo'.$value;};
-        $func2 = function ($value) {return $value.'bar';};
+        $func1 = function($value) {return 'foo'.$value; };
+        $func2 = function($value) {return $value.'bar'; };
 
         // Run some tests.
         $wrapper = Func::once($func1);
@@ -138,8 +138,8 @@ 
     public function testOnly()
     {
         // Prepare our functions.
-        $func1 = function ($value) {return 'foo'.$value;};
-        $func2 = function ($value) {return $value.'bar';};
+        $func1 = function($value) {return 'foo'.$value; };
+        $func2 = function($value) {return $value.'bar'; };
 
         // Run some tests.
         $wrapper = Func::only(3, $func1);
@@ -160,7 +160,7 @@ 
     public function testPartial()
     {
         // Prepare our function.
-        $func = function ($val1, $val2, $val3) {return $val1.$val2.$val3;};
+        $func = function($val1, $val2, $val3) {return $val1.$val2.$val3; };
 
         // Run some tests.
         $wrapper = Func::partial($func, 'foo', 'bar');
@@ -176,7 +176,7 @@ 
     public function testPartialRight()
     {
         // Prepare our function.
-        $func = function ($val1, $val2, $val3) {return $val1.$val2.$val3;};
+        $func = function($val1, $val2, $val3) {return $val1.$val2.$val3; };
 
         // Run some tests.
         $wrapper = Func::partialRight($func, 'foo', 'bar');
@@ -192,7 +192,7 @@ 
     public function testThrottle()
     {
         $counter  = 0;
-        $function = function () use (&$counter) { $counter++; };
+        $function = function() use (&$counter) { $counter++; };
         $wrapper  = Func::throttle($function, 100);
 
         // Time it to invoke the function 5 times at most, even though we call the wrapper 7 times.
@@ -213,7 +213,7 @@ 
 
         // Single call.
         $counter  = 0;
-        $function = function () use (&$counter) { $counter++; };
+        $function = function() use (&$counter) { $counter++; };
         $wrapper  = Func::throttle($function, 100);
 
         $wrapper();
@@ -224,7 +224,7 @@ 
 
         // Double call.
         $counter  = 0;
-        $function = function () use (&$counter) { $counter++; };
+        $function = function() use (&$counter) { $counter++; };
         $wrapper  = Func::throttle($function, 100);
 
         $wrapper();
@@ -241,7 +241,7 @@ 
     public function testRetryAlwaysSuccessful()
     {
         $expected = 'foo';
-        $result   = Func::retry(function () use ($expected) {
+        $result   = Func::retry(function() use ($expected) {
             return $expected;
         });
 
@@ -252,7 +252,7 @@ 
     {
         $this->expectException('InvalidArgumentException');
 
-        Func::retry(function () {
+        Func::retry(function() {
             throw new \InvalidArgumentException;
         });
     }
@@ -260,7 +260,7 @@ 
     public function testRetryWithSingleException()
     {
         $shouldThrow = true;
-        $result = Func::retry(function () use (&$shouldThrow) {
+        $result = Func::retry(function() use (&$shouldThrow) {
             if ($shouldThrow) {
                 $shouldThrow = false;
                 throw new \Exception;
@@ -276,7 +276,7 @@ 
     {
         $count = 0;
         try {
-            Func::retry(function () use (&$count) {
+            Func::retry(function() use (&$count) {
                 $count++;
                 throw new \Exception;
             }, 5);
@@ -291,7 +291,7 @@ 
         $startTime = microtime(true);
 
         try {
-            Func::retry(function () use (&$count) {
+            Func::retry(function() use (&$count) {
                 throw new \Exception;
             }, 4, 0.25);
         } catch (\Exception $e) {
@@ -308,11 +308,11 @@ 
     public function testUnless()
     {
         // -- Basic test. No params for the test nor the callable.
-        $test = function () {
+        $test = function() {
             return true === $this->getWhenExists();
         };
 
-        $callable = function () {return 1;};
+        $callable = function() {return 1; };
 
         $unless = Func::unless($test, $callable);
         $this->setWhenExists(true);
@@ -321,7 +321,7 @@ 
         $this->assertEquals(1, $unless());
 
         // -- Params for the callable.
-        $callable = function ($val) {return $val;};
+        $callable = function($val) {return $val; };
         $unless = Func::unless($test, $callable);
         $this->setWhenExists(true);
         $this->assertNull($unless('foo'));
@@ -329,7 +329,7 @@ 
         $this->assertEquals('foo', $unless('foo'));
 
         // -- Same params for the callable and the test.
-        $test = function ($bool) {
+        $test = function($bool) {
             return $bool === $this->getWhenExists();
         };
 
@@ -342,7 +342,7 @@ 
         $this->assertTrue($unless(true));
 
         // -- Different params for the callable and the test.
-        $test = function ($bool) {
+        $test = function($bool) {
             return $bool === $this->getWhenExists();
         };
 
@@ -360,11 +360,11 @@ 
     public function testWhen()
     {
         // -- Basic test. No params for the test nor the callable.
-        $test = function () {
+        $test = function() {
             return true === $this->getWhenExists();
         };
 
-        $callable = function () {return 1;};
+        $callable = function() {return 1; };
         $when = Func::when($test, $callable);
         $this->setWhenExists(true);
         $this->assertEquals(1, $when());
@@ -372,7 +372,7 @@ 
         $this->assertNull($when());
 
         // -- Params for the callable.
-        $callable = function ($val) {return $val;};
+        $callable = function($val) {return $val; };
         $when = Func::when($test, $callable);
         $this->setWhenExists(true);
         $this->assertEquals('foo', $when('foo'));
@@ -380,7 +380,7 @@ 
         $this->assertNull($when('foo'));
 
         // -- Same params for the callable and the test.
-        $test = function ($bool) {
+        $test = function($bool) {
             return $bool === $this->getWhenExists();
         };
 
@@ -393,7 +393,7 @@ 
         $this->assertFalse($when(false));
 
         // -- Different params for the callable and the test.
-        $test = function ($bool) {
+        $test = function($bool) {
             return $bool === $this->getWhenExists();
         };
 

src/notify/Service.php

Spacing +3 added lines, -3 removed lines

@@ -21,7 +21,7 @@ 
      */
     public function register()
     {
-        $this->app->singleton(TransportManager::class, function (Application $app) {
+        $this->app->singleton(TransportManager::class, function(Application $app) {
             return new TransportManager($app);
         });
 
@@ -37,7 +37,7 @@ 
      */
     protected function registerMailer()
     {
-        $this->app->singleton('mailer', function (Application $app) {
+        $this->app->singleton('mailer', function(Application $app) {
 
             $mailer = new transports\mail\Mailer($app->make('mailer.transports')->driver(), $app->make('view'));
 
@@ -60,7 +60,7 @@ 
 
         // We are going to need to instantiate the Mailer with a proper Driver.
         // @todo Drop the DriverManager in favour of handling the logic in this very Service Provider itself?
-        $this->app->singleton('mailer.transports', function (Application $app) {
+        $this->app->singleton('mailer.transports', function(Application $app) {
             return (new transports\mail\DriverManager($app));
         });
     }

src/notify/transports/Slack.php

Spacing +1 added lines, -1 removed lines

@@ -147,7 +147,7 @@
         $message['endpoint']     = $message['endpoint'] ?? $this->endpoint;
         $message['username']     = $message['username'] ?? $this->username;
         $message['parse']        = $message['parse']    ?? $this->parse;
-        $message['link_names']   = $this->linkNames     ? 1 : 0;
+        $message['link_names']   = $this->linkNames ? 1 : 0;
         $message['unfurl_links'] = $this->unfurlLinks;
         $message['unfurl_media'] = $this->unfurlMedia;
         $message['mrkdwn']       = $this->allowMarkdown;

src/notify/transports/mail/drivers/Mailgun.php

Spacing +1 added lines, -1 removed lines

@@ -73,7 +73,7 @@
                     'api', $this->key
                 ],
                 'multipart' => [
-                    ['name' => 'to',      'contents' => implode(',', $recipients)],
+                    ['name' => 'to', 'contents' => implode(',', $recipients)],
                     ['name' => 'message', 'contents' => $message->toString(), 'filename' => 'message.mime'],
                 ]
             ]);