parsingphase / takeAticket

src/Phase/TakeATicket/Controller.php

Doc Comments +1 added lines, -1 removed lines

@@ -331,7 +331,7 @@
     }
 
     /**
-     * @param $requiredRole
+     * @param string $requiredRole
      */
     public function assertRole($requiredRole)
     {

Spacing +13 added lines, -13 removed lines

@@ -57,7 +57,7 @@ 
     public function upcomingAction()
     {
         $viewParams = $this->defaultViewParams();
-        $viewParams['freeze'] = (bool) $this->dataSource->getSetting('freeze');
+        $viewParams['freeze'] = (bool)$this->dataSource->getSetting('freeze');
         $viewParams['freezeMessage'] = $this->dataSource->getSetting('freezeMessage');
 
         return $this->app['twig']->render('upcoming.html.twig', $viewParams);
@@ -187,7 +187,7 @@ 
             throw new \InvalidArgumentException('Order must be array!');
         }
 
-        $this->app['logger']->debug('New order: '.print_r($idOrder, true));
+        $this->app['logger']->debug('New order: ' . print_r($idOrder, true));
 
         $res = true;
         foreach ($idOrder as $offset => $id) {
@@ -196,7 +196,7 @@ 
         if ($res) {
             $jsonResponse = new JsonResponse(['ok' => 'ok']);
         } else {
-            $this->app['logger']->warn('Failed to store track order: '.print_r($idOrder, true));
+            $this->app['logger']->warn('Failed to store track order: ' . print_r($idOrder, true));
             $jsonResponse = new JsonResponse(['ok' => 'fail', 'message' => 'Failed to store new sort order'], 500);
         }
 
@@ -281,11 +281,11 @@ 
     {
         $this->assertRole(self::MANAGER_REQUIRED_ROLE);
 
-        $rootDir = realpath(__DIR__.'/../../../');
+        $rootDir = realpath(__DIR__ . '/../../../');
         $map = [
-            'readme' => $rootDir.'/README.md',
-            'CONTRIBUTING' => $rootDir.'/docs/CONTRIBUTING.md',
-            'TODO' => $rootDir.'/docs/TODO.md',
+            'readme' => $rootDir . '/README.md',
+            'CONTRIBUTING' => $rootDir . '/docs/CONTRIBUTING.md',
+            'TODO' => $rootDir . '/docs/TODO.md',
         ];
 
         if (!isset($map[$section])) {
@@ -310,14 +310,14 @@ 
     {
         //$this->assertRole(self::MANAGER_REQUIRED_ROLE);
 
-        $rootDir = realpath(__DIR__.'/../../../');
-        $announceDir = $rootDir.'/docs/announcements';
+        $rootDir = realpath(__DIR__ . '/../../../');
+        $announceDir = $rootDir . '/docs/announcements';
 
         if (!preg_match('/^\w+$/', $section)) {
             throw new NotFoundHttpException(); // don't give access to anything but plain names
         }
 
-        $candidateFile = $announceDir.'/'.$section.'.md';
+        $candidateFile = $announceDir . '/' . $section . '.md';
 
         if (!file_exists($candidateFile)) {
             throw new NotFoundHttpException();
@@ -381,8 +381,8 @@ 
     {
         /* @noinspection PhpUnusedParameterInspection */
         $this->app->error(
-            function (\Exception $e, $code) {
-                $message = 'Threw '.get_class($e).': '.$e->getMessage();
+            function(\Exception $e, $code) {
+                $message = 'Threw ' . get_class($e) . ': ' . $e->getMessage();
 
                 return new JsonResponse(['error' => $message]);
             }
@@ -396,7 +396,7 @@ 
     {
         $protocol = $_SERVER['HTTPS'] ? 'https' : 'http';
         $viewParams = [
-            'serverInfo' => $protocol.'://'.$_SERVER['SERVER_ADDR'].':'.$_SERVER['SERVER_PORT'].'/',
+            'serverInfo' => $protocol . '://' . $_SERVER['SERVER_ADDR'] . ':' . $_SERVER['SERVER_PORT'] . '/',
         ];
         $viewParams['displayOptions'] = $this->getDisplayOptions();
 

src/Phase/TakeATicket/ControllerProvider.php

Spacing +2 added lines, -2 removed lines

@@ -39,7 +39,7 @@ 
         $app['twig'] = $app->share(
             $app->extend(
                 'twig',
-                function (\Twig_Environment $twig, $app) {
+                function(\Twig_Environment $twig, $app) {
                     // add custom globals, filters, tags, ...
                     $engine = new MichelfMarkdownEngine();
                     $twig->addExtension(new MarkdownExtension($engine));
@@ -49,7 +49,7 @@ 
         );
 
         $app['ticket.controller'] = $app->share(
-            function (Application $app) {
+            function(Application $app) {
                 return new Controller($app);
             }
         );

src/Phase/TakeATicket/PlaylistExporter.php

Spacing +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
 
         if (is_array($band) && !empty($band[$instrument])) {
             $performers = array_map(
-                function ($performer) {
+                function($performer) {
                     return $performer['performerName'];
                 },
                 $band[$instrument]

src/Phase/TakeATicket/DataSource/AbstractSql.php

Doc Comments +4 added lines, -1 removed lines

@@ -210,6 +210,9 @@ 
         }
     }
 
+    /**
+     * @param string $performerName
+     */
     public function getPerformerIdByName($performerName, $createMissing = false)
     {
         $conn = $this->getDbConn();
@@ -582,7 +585,7 @@ 
     /**
      * Return SQL in appropriate dialect to concatenate the listed values
      *
-     * @param array $fields
+     * @param string[] $fields
      *
      * @return string
      */

src/Phase/TakeATicket/DataSource/MySql.php

Doc Comments +2 added lines, -2 removed lines

@@ -24,7 +24,7 @@ 
      * @param $title
      * @param $songId
      *
-     * @return int|false
+     * @return string|false
      */
     public function storeNewTicket($title, $songId)
     {
@@ -48,7 +48,7 @@ 
      * @param $performerName
      * @param bool|false    $createMissing
      *
-     * @return mixed
+     * @return string|boolean
      */
     public function getPerformerIdByName($performerName, $createMissing = false)
     {

Spacing +1 added lines, -1 removed lines

@@ -15,7 +15,7 @@
      */
     protected function concatenateEscapedFields($fields)
     {
-        return 'CONCAT('.implode(', ', $fields).')';
+        return 'CONCAT(' . implode(', ', $fields) . ')';
     }
 
     /**

var/SymfonyRequirements.php

Doc Comments +2 added lines, -2 removed lines

@@ -223,7 +223,7 @@ 
      * Adds a mandatory requirement in form of a php.ini configuration.
      *
      * @param string        $cfgName           The configuration name used for ini_get()
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
+     * @param boolean|string $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.
@@ -241,7 +241,7 @@ 
      * Adds an optional recommendation in form of a php.ini configuration.
      *
      * @param string        $cfgName           The configuration name used for ini_get()
-     * @param bool|callback $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
+     * @param string|false $evaluation        Either a boolean indicating whether the configuration should evaluate to true or false,
      *                                         or a callback function receiving the configuration value as parameter to determine the fulfillment of the requirement
      * @param bool          $approveCfgAbsence If true the Requirement will be fulfilled even if the configuration option does not exist, i.e. ini_get() returns false.
      *                                         This is helpful for abandoned configs in later PHP versions or configs of an optional extension, like Suhosin.

Spacing +17 added lines, -17 removed lines

@@ -49,11 +49,11 @@ 
      */
     public function __construct($fulfilled, $testMessage, $helpHtml, $helpText = null, $optional = false)
     {
-        $this->fulfilled = (bool) $fulfilled;
-        $this->testMessage = (string) $testMessage;
-        $this->helpHtml = (string) $helpHtml;
-        $this->helpText = null === $helpText ? strip_tags($this->helpHtml) : (string) $helpText;
-        $this->optional = (bool) $optional;
+        $this->fulfilled = (bool)$fulfilled;
+        $this->testMessage = (string)$testMessage;
+        $this->helpHtml = (string)$helpHtml;
+        $this->helpText = null === $helpText ? strip_tags($this->helpHtml) : (string)$helpText;
+        $this->optional = (bool)$optional;
     }
 
     /**
@@ -417,13 +417,13 @@ 
         );
 
         $this->addRequirement(
-            is_dir(__DIR__.'/../vendor/composer'),
+            is_dir(__DIR__ . '/../vendor/composer'),
             'Vendor libraries must be installed',
-            'Vendor libraries are missing. Install composer following instructions from <a href="http://getcomposer.org/">http://getcomposer.org/</a>. '.
+            'Vendor libraries are missing. Install composer following instructions from <a href="http://getcomposer.org/">http://getcomposer.org/</a>. ' .
                 'Then run "<strong>php composer.phar install</strong>" to install them.'
         );
 
-        $cacheDir = is_dir(__DIR__.'/../var/cache') ? __DIR__.'/../var/cache' : __DIR__.'/cache';
+        $cacheDir = is_dir(__DIR__ . '/../var/cache') ? __DIR__ . '/../var/cache' : __DIR__ . '/cache';
 
         $this->addRequirement(
             is_writable($cacheDir),
@@ -431,7 +431,7 @@ 
             'Change the permissions of either "<strong>app/cache/</strong>" or  "<strong>var/cache/</strong>" directory so that the web server can write into it.'
         );
 
-        $logsDir = is_dir(__DIR__.'/../var/logs') ? __DIR__.'/../var/logs' : __DIR__.'/logs';
+        $logsDir = is_dir(__DIR__ . '/../var/logs') ? __DIR__ . '/../var/logs' : __DIR__ . '/logs';
 
         $this->addRequirement(
             is_writable($logsDir),
@@ -544,7 +544,7 @@ 
             );
         }
 
-        $pcreVersion = defined('PCRE_VERSION') ? (float) PCRE_VERSION : null;
+        $pcreVersion = defined('PCRE_VERSION') ? (float)PCRE_VERSION : null;
 
         $this->addRequirement(
             null !== $pcreVersion,
@@ -564,13 +564,13 @@ 
 
         /* optional recommendations follow */
 
-        if (file_exists(__DIR__.'/../vendor/composer')) {
-            require_once __DIR__.'/../vendor/autoload.php';
+        if (file_exists(__DIR__ . '/../vendor/composer')) {
+            require_once __DIR__ . '/../vendor/autoload.php';
 
             try {
                 $r = new ReflectionClass('Sensio\Bundle\DistributionBundle\SensioDistributionBundle');
 
-                $contents = file_get_contents(dirname($r->getFileName()).'/Resources/skeleton/app/SymfonyRequirements.php');
+                $contents = file_get_contents(dirname($r->getFileName()) . '/Resources/skeleton/app/SymfonyRequirements.php');
             } catch (ReflectionException $e) {
                 $contents = '';
             }
@@ -783,7 +783,7 @@ 
         $unit = '';
         if (!ctype_digit($size)) {
             $unit = strtolower(substr($size, -1, 1));
-            $size = (int) substr($size, 0, -1);
+            $size = (int)substr($size, 0, -1);
         }
         switch ($unit) {
             case 'g':
@@ -793,7 +793,7 @@ 
             case 'k':
                 return $size * 1024;
             default:
-                return (int) $size;
+                return (int)$size;
         }
     }
 
@@ -804,7 +804,7 @@ 
      */
     protected function getPhpRequiredVersion()
     {
-        if (!file_exists($path = __DIR__.'/../composer.lock')) {
+        if (!file_exists($path = __DIR__ . '/../composer.lock')) {
             return false;
         }
 
@@ -815,7 +815,7 @@ 
                 continue;
             }
 
-            return (int) $package['version'][1] > 2 ? self::REQUIRED_PHP_VERSION : self::LEGACY_REQUIRED_PHP_VERSION;
+            return (int)$package['version'][1] > 2 ? self::REQUIRED_PHP_VERSION : self::LEGACY_REQUIRED_PHP_VERSION;
         }
 
         return false;

web/app.php

Spacing +2 added lines, -2 removed lines

@@ -5,8 +5,8 @@
 /**
  * @var Composer\Autoload\ClassLoader
  */
-$loader = require __DIR__.'/../app/autoload.php';
-include_once __DIR__.'/../var/bootstrap.php.cache';
+$loader = require __DIR__ . '/../app/autoload.php';
+include_once __DIR__ . '/../var/bootstrap.php.cache';
 
 // Enable APC for autoloading to improve performance.
 // You should change the ApcClassLoader first argument to a unique prefix

web/app_dev.php

Spacing +2 added lines, -2 removed lines

@@ -15,13 +15,13 @@
     || !(in_array(@$_SERVER['REMOTE_ADDR'], ['127.0.0.1', 'fe80::1', '::1']) || php_sapi_name() === 'cli-server')
 ) {
     header('HTTP/1.0 403 Forbidden');
-    exit('You are not allowed to access this file. Check '.basename(__FILE__).' for more information.');
+    exit('You are not allowed to access this file. Check ' . basename(__FILE__) . ' for more information.');
 }
 
 /**
  * @var Composer\Autoload\ClassLoader $loader
  */
-$loader = require __DIR__.'/../app/autoload.php';
+$loader = require __DIR__ . '/../app/autoload.php';
 Debug::enable();
 
 $kernel = new AppKernel('dev', true);

src/Phase/TakeATicketBundle/Controller/ManagementController.php

Indentation +4 added lines, -4 removed lines

@@ -109,7 +109,7 @@ 
             $settingsForm->handleRequest($request);
 
             /**
- * @noinspection PhpUndefinedMethodInspection
+             * @noinspection PhpUndefinedMethodInspection
 */ // isClicked on Submit
             if ($settingsForm->isSubmitted()
                 && $settingsForm->isValid()
@@ -138,7 +138,7 @@ 
             $resetForm->handleRequest($request);
 
             /**
- * @noinspection PhpUndefinedMethodInspection
+             * @noinspection PhpUndefinedMethodInspection
 */ // isClicked on Submit
             if ($resetForm->isSubmitted()
                 && $resetForm->isValid()
@@ -169,7 +169,7 @@ 
             $songListForm->handleRequest($request);
 
             /**
- * @noinspection PhpUndefinedMethodInspection
+             * @noinspection PhpUndefinedMethodInspection
 */ // isClicked on Submit
             if ($songListForm->isSubmitted()
                 && $songListForm->isValid()
@@ -179,7 +179,7 @@ 
 
                 $file = $data['songListFile'];
                 /**
- * @var UploadedFile $file
+                 * @var UploadedFile $file
 */
 
                 $loader = new SongLoader();