BCRM / www

src/BCRM/WebBundle/Controller/WebController.php

Braces +3 added lines, -1 removed lines

@@ -131,7 +131,9 @@
         }
 
         $p = $this->reader->getPage($path . '.md');
-        if ($p->isHidden()) throw new NotFoundHttpException();
+        if ($p->isHidden()) {
+            throw new NotFoundHttpException();
+        }
 
         return $this->renderer->renderResponse('BCRMWebBundle:Web:page.html.twig', array(
             'page'     => $p,

src/BCRM/WebBundle/Tests/Functional/ConciergeControllerTest.php

Spacing +2 added lines, -2 removed lines

@@ -48,7 +48,7 @@ 
         $ticket = $em->getRepository('BCRMBackendBundle:Event\Ticket')->findOneBy(array('code' => 'WOOT'));
         $this->assertEquals(false, $ticket->isCheckedIn());
 
-        $client   = static::createClient(array(), array(
+        $client = static::createClient(array(), array(
             'PHP_AUTH_USER' => 'concierge',
             'PHP_AUTH_PW'   => 'letmein',
         ));
@@ -172,7 +172,7 @@ 
      */
     public function theTicketSearchApiShouldWork()
     {
-        $client    = static::createClient(array(), array(
+        $client = static::createClient(array(), array(
             'PHP_AUTH_USER' => 'concierge',
             'PHP_AUTH_PW'   => 'letmein',
         ));

src/BCRM/WebBundle/Tests/Functional/EventControllerTest.php

Spacing +2 added lines, -2 removed lines

@@ -313,7 +313,7 @@ 
             'day'   => Ticket::DAY_SATURDAY,
         ));
         $this->assertEquals(3, count($tickets));
-        $this->assertInArray('john.doe.1981@domain.com', array_map(function (Ticket $t) {
+        $this->assertInArray('john.doe.1981@domain.com', array_map(function(Ticket $t) {
             return $t->getEmail();
         }, $tickets));
 
@@ -340,7 +340,7 @@ 
             'day'   => Ticket::DAY_SATURDAY,
         ));
         $this->assertEquals(3, count($tickets));
-        $this->assertNotInArray('john.doe.1981@domain.com', array_map(function (Ticket $t) {
+        $this->assertNotInArray('john.doe.1981@domain.com', array_map(function(Ticket $t) {
             return $t->getEmail();
         }, $tickets));
     }

src/BCRM/WebBundle/Tests/Functional/StatsControllerTest.php

Braces +6 added lines, -2 removed lines

@@ -67,8 +67,12 @@
 
     protected function createTicket($day, $checkedIn = null, $email = null)
     {
-        if ($email === null) $email = sprintf('stats.doe.198%s@domain.com', $this->ticketCounter);
-        if ($checkedIn === null) $checkedIn = true;
+        if ($email === null) {
+            $email = sprintf('stats.doe.198%s@domain.com', $this->ticketCounter);
+        }
+        if ($checkedIn === null) {
+            $checkedIn = true;
+        }
         $client    = static::createClient();
         $container = $client->getContainer();
 

Doc Comments +3 added lines

@@ -66,6 +66,9 @@
 
     protected $ticketCounter = 1;
 
+    /**
+     * @param false $checkedIn
+     */
     protected function createTicket($day, $checkedIn = null, $email = null)
     {
         if ($email === null) $email = sprintf('stats.doe.198%s@domain.com', $this->ticketCounter);

Unused Use Statements -3 removed lines

@@ -7,11 +7,8 @@
 
 namespace BCRM\WebBundle\Tests\Functional;
 
-use BCRM\BackendBundle\Entity\Event\Registration;
 use BCRM\BackendBundle\Entity\Event\Ticket;
 use BCRM\BackendBundle\Entity\Payment;
-use Symfony\Component\BrowserKit\Cookie;
-use Symfony\Component\DependencyInjection\ContainerInterface;
 
 class StatsControllerTest extends Base
 {

Spacing +1 added lines, -1 removed lines

@@ -80,7 +80,7 @@
 
         // Create a payment
         $txId = sha1('someid' . $email);
-        $payment  = $em->getRepository('BCRMBackendBundle:Payment')->findOneByTxId($txId);
+        $payment = $em->getRepository('BCRMBackendBundle:Payment')->findOneByTxId($txId);
         if (!$payment) {
             $payment = new Payment();
             $payment->setTransactionId($txId);