apiaryhq / silex-sms-login-provider

src/SmsLoginController.php

Doc Comments -1 removed lines

@@ -56,7 +56,6 @@
      * @param \Apiary\SmsLoginProvider\SmsHandler\SmsHandlerInterface $handler
      * @param string|NULL $view
      * @param string|NULL $message
-     * @param null $debug
      */
     public function __construct(
         SmsHandlerInterface $handler,

Spacing +2 added lines, -2 removed lines

@@ -65,7 +65,7 @@ 
     )
     {
         $this->messageTemplate = $message ?: 'Security code: %s';
-        $countryData = file_get_contents(__DIR__ . '/../data/countries.json');
+        $countryData = file_get_contents(__DIR__.'/../data/countries.json');
         $this->countries = json_decode($countryData);
         $this->handler = $handler;
         $this->viewName = $view ?: 'login.twig';
@@ -114,7 +114,7 @@ 
 
         return $app['twig']->render($this->viewName, [
             'mobile' => $number,
-            'form_action' => $app['url_generator']->generate('sms.login') . '/check',
+            'form_action' => $app['url_generator']->generate('sms.login').'/check',
         ]);
     }
 

src/SmsHandler/MockSmsHandler.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the Apiary SMS Login Provider package.
- *
- * (c) Darren Mothersele <darren@darrenmothersele.com>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * This file is part of the Apiary SMS Login Provider package.
+     *
+     * (c) Darren Mothersele <darren@darrenmothersele.com>
+     *
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace Apiary\SmsLoginProvider\SmsHandler;
 

src/SmsHandler/MockSmsHandlerProvider.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
     public function register(Application $app)
     {
-        $app['sms.handler'] = $app->share(function () use ($app) {
+        $app['sms.handler'] = $app->share(function() use ($app) {
             return new MockSmsHandler($app);
         });
     }

src/SmsHandler/SmsHandlerInterface.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the Apiary SMS Login Provider package.
- *
- * (c) Darren Mothersele <darren@darrenmothersele.com>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * This file is part of the Apiary SMS Login Provider package.
+     *
+     * (c) Darren Mothersele <darren@darrenmothersele.com>
+     *
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace Apiary\SmsLoginProvider\SmsHandler;
 

src/SmsHandler/TwilioSmsHandlerProvider.php

Indentation +7 added lines, -7 removed lines

@@ -1,12 +1,12 @@
 <?php
 /**
- * This file is part of the Apiary SMS Login Provider package.
- *
- * (c) Darren Mothersele <darren@darrenmothersele.com>
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- */
+     * This file is part of the Apiary SMS Login Provider package.
+     *
+     * (c) Darren Mothersele <darren@darrenmothersele.com>
+     *
+     * For the full copyright and license information, please view the LICENSE
+     * file that was distributed with this source code.
+     */
 
 namespace Apiary\SmsLoginProvider\SmsHandler;
 

Spacing +1 added lines, -1 removed lines

@@ -21,7 +21,7 @@
         if (!isset($app['sms.handler.from'])) {
             $app['sms.handler.from'] = 'Apiary';
         }
-        $app['sms.handler'] = $app->share(function () use ($app) {
+        $app['sms.handler'] = $app->share(function() use ($app) {
             $lookupClient = new \Lookups_Services_Twilio($app['sms.handler.twilio_sid'],
                 $app['sms.handler.twilio_auth_token']);
             $client = new \Services_Twilio($app['sms.handler.twilio_sid'],

src/SmsLoginProvider.php

Spacing +19 added lines, -19 removed lines

@@ -35,14 +35,14 @@ 
         $controllers->post('/login', 'login.controller:verifyAction');
 
         // Convert Twilio exception to Auth exception
-        $app->error(function (\Services_Twilio_RestException $e) use ($app) {
-            $app['monolog']->addDebug('Converting Twilio Exception: ' . $e->getMessage());
+        $app->error(function(\Services_Twilio_RestException $e) use ($app) {
+            $app['monolog']->addDebug('Converting Twilio Exception: '.$e->getMessage());
             $app['session']->set(Security::AUTHENTICATION_ERROR,
               new BadCredentialsException('Invalid phone number'));
             return $app->redirect($app['url_generator']->generate('sms.login'));
         });
         // Store Auth exception and go back to login form
-        $app->error(function (AuthenticationException $e) use ($app) {
+        $app->error(function(AuthenticationException $e) use ($app) {
             $app['session']->set(Security::AUTHENTICATION_ERROR, $e);
             return $app->redirect($app['url_generator']->generate('sms.login'));
         });
@@ -53,17 +53,17 @@ 
 
     public function register(Application $app)
     {
-        $app['login.controller'] = $app->share(function () use ($app) {
+        $app['login.controller'] = $app->share(function() use ($app) {
             // TODO: Pass in arguments for view template and message template
             return new SmsLoginController($app['sms.handler'], null, null);
         });
 
-        $app['security.authentication_listener.factory.sms'] = $app->protect(function (
+        $app['security.authentication_listener.factory.sms'] = $app->protect(function(
           $name,
           $options
         ) use ($app) {
 
-            $app['security.entry_point.' . $name . '.sms'] = $app->share(function (
+            $app['security.entry_point.'.$name.'.sms'] = $app->share(function(
             ) use ($app, $options) {
                 $loginPath = $app['url_generator']->generate('sms.login');
                 $useForward = isset($options['use_forward']) ? $options['use_forward'] : false;
@@ -71,17 +71,17 @@ 
                   $app['security.http_utils'], $loginPath, $useForward);
             });
 
-            $app['security.authentication_provider.' . $name . '.sms'] = $app->share(function (
+            $app['security.authentication_provider.'.$name.'.sms'] = $app->share(function(
             ) use ($app, $name) {
                 return new SmsAuthenticator($name, $app['session']->get('code'),
                   $app['monolog']);
             });
 
-            $app['security.authentication_listener.' . $name . '.sms'] = $app->share(function (
+            $app['security.authentication_listener.'.$name.'.sms'] = $app->share(function(
             ) use ($app, $name, $options) {
 
                 // Create fake route for login check
-                $loginCheckPath = $app['url_generator']->generate('sms.login') . '/check';
+                $loginCheckPath = $app['url_generator']->generate('sms.login').'/check';
                 $options['check_path'] = $loginCheckPath;
                 $app->match($loginCheckPath)->run(null)->bind(str_replace('/',
                   '_', ltrim($loginCheckPath, '/')));
@@ -90,8 +90,8 @@ 
                 $options['username_parameter'] = empty($options['username_parameter']) ? 'mobile' : $options['username_parameter'];
                 $options['password_parameter'] = empty($options['password_parameter']) ? 'code' : $options['password_parameter'];
 
-                if (!isset($app['security.authentication.success_handler.' . $name])) {
-                    $app['security.authentication.success_handler.' . $name] = $app->share(function (
+                if (!isset($app['security.authentication.success_handler.'.$name])) {
+                    $app['security.authentication.success_handler.'.$name] = $app->share(function(
                     ) use ($name, $options, $app) {
                         $handler = new DefaultAuthenticationSuccessHandler($app['security.http_utils'],
                           $options);
@@ -100,8 +100,8 @@ 
                     });
                 }
 
-                if (!isset($app['security.authentication.failure_handler.' . $name])) {
-                    $app['security.authentication.failure_handler.' . $name] = $app->share(function (
+                if (!isset($app['security.authentication.failure_handler.'.$name])) {
+                    $app['security.authentication.failure_handler.'.$name] = $app->share(function(
                     ) use ($name, $options, $app) {
                         return new DefaultAuthenticationFailureHandler($app,
                           $app['security.http_utils'], $options,
@@ -112,11 +112,11 @@ 
                 return new UsernamePasswordFormAuthenticationListener(
                   $app['security.token_storage'],
                   $app['security.authentication_manager'],
-                  isset($app['security.session_strategy.' . $name]) ? $app['security.session_strategy.' . $name] : $app['security.session_strategy'],
+                  isset($app['security.session_strategy.'.$name]) ? $app['security.session_strategy.'.$name] : $app['security.session_strategy'],
                   $app['security.http_utils'],
                   $name,
-                  $app['security.authentication.success_handler.' . $name],
-                  $app['security.authentication.failure_handler.' . $name],
+                  $app['security.authentication.success_handler.'.$name],
+                  $app['security.authentication.failure_handler.'.$name],
                   $options,
                   $app['logger'],
                   $app['dispatcher'],
@@ -127,11 +127,11 @@ 
 
             return [
                 // the authentication provider id
-              'security.authentication_provider.' . $name . '.sms',
+              'security.authentication_provider.'.$name.'.sms',
                 // the authentication listener id
-              'security.authentication_listener.' . $name . '.sms',
+              'security.authentication_listener.'.$name.'.sms',
                 // the entry point id
-              'security.entry_point.' . $name . '.sms',
+              'security.entry_point.'.$name.'.sms',
                 // the position of the listener in the stack
               'form'
             ];