fixed CS

fabpot · fabpot · commit a6af39b830bf · 2014-09-22T10:32:35.000+02:00
diff --git a/DataCollector/ConfigDataCollector.php b/DataCollector/ConfigDataCollector.php
@@ -70,7 +70,7 @@ public function collect(Request $request, Response $response, \Exception $except
             'wincache_enabled'     => extension_loaded('wincache') && ini_get('wincache.ocenabled'),
             'zend_opcache_enabled' => extension_loaded('Zend OPcache') && ini_get('opcache.enable'),
             'bundles'              => array(),
-            'sapi_name'            => php_sapi_name()
+            'sapi_name'            => php_sapi_name(),
         );
 
         if (isset($this->kernel)) {
diff --git a/DataCollector/LoggerDataCollector.php b/DataCollector/LoggerDataCollector.php
@@ -41,7 +41,7 @@ public function collect(Request $request, Response $response, \Exception $except
             $this->data = array(
                 'error_count'       => $this->logger->countErrors(),
                 'logs'              => $this->sanitizeLogs($this->logger->getLogs()),
-                'deprecation_count' => $this->computeDeprecationCount()
+                'deprecation_count' => $this->computeDeprecationCount(),
             );
         }
     }
diff --git a/Debug/TraceableEventDispatcher.php b/Debug/TraceableEventDispatcher.php
@@ -277,7 +277,7 @@ private function getListenerInfo($listener, $eventId, $eventName)
         if ($listener instanceof \Closure) {
             $info += array(
                 'type' => 'Closure',
-                'pretty' => 'closure'
+                'pretty' => 'closure',
             );
         } elseif (is_string($listener)) {
             try {
@@ -406,7 +406,8 @@ private function preDispatch($eventName, $eventId, Event $event)
                 // which must be caught.
                 try {
                     $this->stopwatch->openSection($token);
-                } catch (\LogicException $e) {}
+                } catch (\LogicException $e) {
+                }
                 break;
         }
     }
@@ -432,7 +433,8 @@ private function postDispatch($eventName, $eventId, Event $event)
                 // does not exist, then closing it throws an exception which must be caught.
                 try {
                     $this->stopwatch->stopSection($token);
-                } catch (\LogicException $e) {}
+                } catch (\LogicException $e) {
+                }
                 // The children profiles have been updated by the previous 'kernel.response'
                 // event. Only the root profile need to be updated with the 'kernel.terminate'
                 // timing information.
diff --git a/HttpCache/HttpCache.php b/HttpCache/HttpCache.php
@@ -152,7 +152,6 @@ public function getKernel()
         return $this->kernel;
     }
 
-
     /**
      * Gets the Esi instance
      *
diff --git a/Kernel.php b/Kernel.php
@@ -519,7 +519,6 @@ protected function initializeBundles()
                 array_pop($bundleMap);
             }
         }
-
     }
 
     /**
diff --git a/Profiler/BaseMemcacheProfilerStorage.php b/Profiler/BaseMemcacheProfilerStorage.php
@@ -53,12 +53,11 @@ public function find($ip, $url, $limit, $method, $start = null, $end = null)
         $result = array();
 
         foreach ($profileList as $item) {
-
             if ($limit === 0) {
                 break;
             }
 
-            if ($item=='') {
+            if ($item == '') {
                 continue;
             }
 
@@ -166,7 +165,6 @@ public function write(Profile $profile)
         $profileIndexed = false !== $this->getValue($this->getItemName($profile->getToken()));
 
         if ($this->setValue($this->getItemName($profile->getToken()), $data, $this->lifetime)) {
-
             if (!$profileIndexed) {
                 // Add to index
                 $indexName = $this->getIndexName();
@@ -304,5 +302,4 @@ private function isItemNameValid($name)
 
         return true;
     }
-
 }
diff --git a/Profiler/FileProfilerStorage.php b/Profiler/FileProfilerStorage.php
@@ -70,7 +70,7 @@ public function find($ip, $url, $limit, $method, $start = null, $end = null)
             }
 
             if (!empty($start) && $csvTime < $start) {
-               continue;
+                continue;
             }
 
             if (!empty($end) && $csvTime > $end) {
diff --git a/Profiler/MemcacheProfilerStorage.php b/Profiler/MemcacheProfilerStorage.php
@@ -91,7 +91,6 @@ protected function appendValue($key, $value, $expiration = 0)
         $memcache = $this->getMemcache();
 
         if (method_exists($memcache, 'append')) {
-
             // Memcache v3.0
             if (!$result = $memcache->append($key, $value, false, $expiration)) {
                 return $memcache->set($key, $value, false, $expiration);
diff --git a/Profiler/MongoDbProfilerStorage.php b/Profiler/MongoDbProfilerStorage.php
@@ -82,7 +82,7 @@ public function write(Profile $profile)
             'ip' => $profile->getIp(),
             'method' => $profile->getMethod(),
             'url' => $profile->getUrl(),
-            'time' => $profile->getTime()
+            'time' => $profile->getTime(),
         );
 
         $result = $this->getMongo()->update(array('_id' => $profile->getToken()), array_filter($record, function ($v) { return !empty($v); }), array('upsert' => true));
diff --git a/Profiler/RedisProfilerStorage.php b/Profiler/RedisProfilerStorage.php
@@ -171,7 +171,6 @@ public function write(Profile $profile)
         $profileIndexed = false !== $this->getValue($this->getItemName($profile->getToken()));
 
         if ($this->setValue($this->getItemName($profile->getToken()), $data, $this->lifetime, self::REDIS_SERIALIZER_PHP)) {
-
             if (!$profileIndexed) {
                 // Add to index
                 $indexName = $this->getIndexName();
diff --git a/Tests/Bundle/BundleTest.php b/Tests/Bundle/BundleTest.php
@@ -43,6 +43,5 @@ public function testRegisterCommands()
         $bundle2 = new ExtensionAbsentBundle();
 
         $this->assertNull($bundle2->registerCommands($app));
-
     }
 }
diff --git a/Tests/ClientTest.php b/Tests/ClientTest.php
@@ -75,7 +75,7 @@ public function testFilterResponseConvertsCookies()
 
         $expected = array(
             'foo=bar; expires=Sun, 15 Feb 2009 20:00:00 GMT; domain=http://example.com; path=/foo; secure; httponly',
-            'foo1=bar1; expires=Sun, 15 Feb 2009 20:00:00 GMT; domain=http://example.com; path=/foo; secure; httponly'
+            'foo1=bar1; expires=Sun, 15 Feb 2009 20:00:00 GMT; domain=http://example.com; path=/foo; secure; httponly',
         );
 
         $response = new Response();
diff --git a/Tests/DataCollector/ExceptionDataCollectorTest.php b/Tests/DataCollector/ExceptionDataCollectorTest.php
@@ -43,5 +43,4 @@ public function testCollect()
         $this->assertSame('exception',$c->getName());
         $this->assertSame($trace,$c->getTrace());
     }
-
 }
diff --git a/Tests/DataCollector/LoggerDataCollectorTest.php b/Tests/DataCollector/LoggerDataCollectorTest.php
@@ -50,28 +50,28 @@ public function getCollectTestData()
                 1,
                 array(array('message' => 'foo', 'context' => array())),
                 null,
-                0
+                0,
             ),
             array(
                 1,
                 array(array('message' => 'foo', 'context' => array('foo' => fopen(__FILE__, 'r')))),
                 array(array('message' => 'foo', 'context' => array('foo' => 'Resource(stream)'))),
-                0
+                0,
             ),
             array(
                 1,
                 array(array('message' => 'foo', 'context' => array('foo' => new \stdClass()))),
                 array(array('message' => 'foo', 'context' => array('foo' => 'Object(stdClass)'))),
-                0
+                0,
             ),
             array(
                 1,
                 array(
                     array('message' => 'foo', 'context' => array('type' => ErrorHandler::TYPE_DEPRECATION)),
-                    array('message' => 'foo2', 'context' => array('type' => ErrorHandler::TYPE_DEPRECATION))
+                    array('message' => 'foo2', 'context' => array('type' => ErrorHandler::TYPE_DEPRECATION)),
                 ),
                 null,
-                2
+                2,
             ),
         );
     }
diff --git a/Tests/DataCollector/RequestDataCollectorTest.php b/Tests/DataCollector/RequestDataCollectorTest.php
@@ -74,7 +74,7 @@ public function testControllerInspection(Request $request, Response $response)
                     'class' => 'Symfony\Component\HttpKernel\Tests\DataCollector\RequestDataCollectorTest',
                     'method' => 'testControllerInspection',
                     'file' => __FILE__,
-                    'line' => $r1->getStartLine()
+                    'line' => $r1->getStartLine(),
                 ),
             ),
 
@@ -102,7 +102,7 @@ function () { return 'foo'; },
                     'class' => 'Symfony\Component\HttpKernel\Tests\DataCollector\RequestDataCollectorTest',
                     'method' => 'staticControllerMethod',
                     'file' => __FILE__,
-                    'line' => $r2->getStartLine()
+                    'line' => $r2->getStartLine(),
                 ),
             ),
 
@@ -113,7 +113,7 @@ function () { return 'foo'; },
                     'class' => 'Symfony\Component\HttpKernel\Tests\DataCollector\RequestDataCollectorTest',
                     'method' => 'staticControllerMethod',
                     'file' => __FILE__,
-                    'line' => $r2->getStartLine()
+                    'line' => $r2->getStartLine(),
                 ),
             ),
 
@@ -124,7 +124,7 @@ function () { return 'foo'; },
                     'class' => 'Symfony\Component\HttpKernel\Tests\DataCollector\RequestDataCollectorTest',
                     'method' => 'magicMethod',
                     'file' => 'n/a',
-                    'line' => 'n/a'
+                    'line' => 'n/a',
                 ),
             ),
 
@@ -135,7 +135,7 @@ function () { return 'foo'; },
                     'class' => 'Symfony\Component\HttpKernel\Tests\DataCollector\RequestDataCollectorTest',
                     'method' => 'magicMethod',
                     'file' => 'n/a',
-                    'line' => 'n/a'
+                    'line' => 'n/a',
                 ),
             ),
         );
@@ -166,7 +166,7 @@ public function provider()
         $response->headers->setCookie(new Cookie('bazz','foo','2000-12-12'));
 
         return array(
-            array($request, $response)
+            array($request, $response),
         );
     }
 
@@ -204,5 +204,4 @@ public static function __callStatic($method, $args)
     {
         throw new \LogicException('Unexpected method call');
     }
-
 }
diff --git a/Tests/Debug/TraceableEventDispatcherTest.php b/Tests/Debug/TraceableEventDispatcherTest.php
@@ -38,7 +38,7 @@ public function testAddRemoveListener()
         $dispatcher = new EventDispatcher();
         $tdispatcher = new TraceableEventDispatcher($dispatcher, new Stopwatch());
 
-        $tdispatcher->addListener('foo', $listener = function () { ; });
+        $tdispatcher->addListener('foo', $listener = function () {; });
         $listeners = $dispatcher->getListeners('foo');
         $this->assertCount(1, $listeners);
         $this->assertSame($listener, $listeners[0]);
@@ -52,7 +52,7 @@ public function testGetListeners()
         $dispatcher = new EventDispatcher();
         $tdispatcher = new TraceableEventDispatcher($dispatcher, new Stopwatch());
 
-        $tdispatcher->addListener('foo', $listener = function () { ; });
+        $tdispatcher->addListener('foo', $listener = function () {; });
         $this->assertSame($dispatcher->getListeners('foo'), $tdispatcher->getListeners('foo'));
     }
 
@@ -64,7 +64,7 @@ public function testHasListeners()
         $this->assertFalse($dispatcher->hasListeners('foo'));
         $this->assertFalse($tdispatcher->hasListeners('foo'));
 
-        $tdispatcher->addListener('foo', $listener = function () { ; });
+        $tdispatcher->addListener('foo', $listener = function () {; });
         $this->assertTrue($dispatcher->hasListeners('foo'));
         $this->assertTrue($tdispatcher->hasListeners('foo'));
     }
@@ -89,7 +89,7 @@ public function testGetCalledListeners()
     {
         $dispatcher = new EventDispatcher();
         $tdispatcher = new TraceableEventDispatcher($dispatcher, new Stopwatch());
-        $tdispatcher->addListener('foo', $listener = function () { ; });
+        $tdispatcher->addListener('foo', $listener = function () {; });
 
         $this->assertEquals(array(), $tdispatcher->getCalledListeners());
         $this->assertEquals(array('foo.closure' => array('event' => 'foo', 'type' => 'Closure', 'pretty' => 'closure')), $tdispatcher->getNotCalledListeners());
@@ -106,8 +106,8 @@ public function testLogger()
 
         $dispatcher = new EventDispatcher();
         $tdispatcher = new TraceableEventDispatcher($dispatcher, new Stopwatch(), $logger);
-        $tdispatcher->addListener('foo', $listener1 = function () { ; });
-        $tdispatcher->addListener('foo', $listener2 = function () { ; });
+        $tdispatcher->addListener('foo', $listener1 = function () {; });
+        $tdispatcher->addListener('foo', $listener2 = function () {; });
 
         $logger->expects($this->at(0))->method('debug')->with("Notified event \"foo\" to listener \"closure\".");
         $logger->expects($this->at(1))->method('debug')->with("Notified event \"foo\" to listener \"closure\".");
@@ -122,7 +122,7 @@ public function testLoggerWithStoppedEvent()
         $dispatcher = new EventDispatcher();
         $tdispatcher = new TraceableEventDispatcher($dispatcher, new Stopwatch(), $logger);
         $tdispatcher->addListener('foo', $listener1 = function (Event $event) { $event->stopPropagation(); });
-        $tdispatcher->addListener('foo', $listener2 = function () { ; });
+        $tdispatcher->addListener('foo', $listener2 = function () {; });
 
         $logger->expects($this->at(0))->method('debug')->with("Notified event \"foo\" to listener \"closure\".");
         $logger->expects($this->at(1))->method('debug')->with("Listener \"closure\" stopped propagation of the event \"foo\".");
diff --git a/Tests/DependencyInjection/RegisterListenersPassTest.php b/Tests/DependencyInjection/RegisterListenersPassTest.php
@@ -133,5 +133,7 @@ public function testAbstractEventListener()
 
 class SubscriberService implements \Symfony\Component\EventDispatcher\EventSubscriberInterface
 {
-    public static function getSubscribedEvents() {}
+    public static function getSubscribedEvents()
+    {
+    }
 }
diff --git a/Tests/EventListener/ExceptionListenerTest.php b/Tests/EventListener/ExceptionListenerTest.php
@@ -108,7 +108,7 @@ public function provider()
         $event2 = new GetResponseForExceptionEvent(new TestKernelThatThrowsException(), $request, 'foo', $exception);
 
         return array(
-            array($event, $event2)
+            array($event, $event2),
         );
     }
 
diff --git a/Tests/EventListener/ResponseListenerTest.php b/Tests/EventListener/ResponseListenerTest.php
@@ -36,7 +36,6 @@ protected function setUp()
         $this->dispatcher->addListener(KernelEvents::RESPONSE, array($listener, 'onKernelResponse'));
 
         $this->kernel = $this->getMock('Symfony\Component\HttpKernel\HttpKernelInterface');
-
     }
 
     protected function tearDown()
diff --git a/Tests/Fixtures/ExtensionPresentBundle/Command/BarCommand.php b/Tests/Fixtures/ExtensionPresentBundle/Command/BarCommand.php
@@ -13,6 +13,5 @@ class BarCommand extends Command
 {
     public function __construct($example, $name = 'bar')
     {
-
     }
 }
diff --git a/Tests/Fixtures/ExtensionPresentBundle/Command/FooCommand.php b/Tests/Fixtures/ExtensionPresentBundle/Command/FooCommand.php
@@ -19,5 +19,4 @@ protected function configure()
     {
         $this->setName('foo');
     }
-
 }
diff --git a/Tests/Fixtures/KernelForOverrideName.php b/Tests/Fixtures/KernelForOverrideName.php
@@ -20,11 +20,9 @@ class KernelForOverrideName extends Kernel
 
     public function registerBundles()
     {
-
     }
 
     public function registerContainerConfiguration(LoaderInterface $loader)
     {
-
     }
 }
diff --git a/Tests/Fragment/RoutableFragmentRendererTest.php b/Tests/Fragment/RoutableFragmentRendererTest.php
@@ -62,8 +62,12 @@ private function getRenderer()
 
 class Renderer extends RoutableFragmentRenderer
 {
-    public function render($uri, Request $request, array $options = array()) {}
-    public function getName() {}
+    public function render($uri, Request $request, array $options = array())
+    {
+    }
+    public function getName()
+    {
+    }
 
     public function doGenerateFragmentUri(ControllerReference $reference, Request $request, $absolute = false)
     {
diff --git a/Tests/KernelTest.php b/Tests/KernelTest.php
@@ -497,7 +497,7 @@ public function testLocateResourceReturnsAllMatches()
 
         $this->assertEquals(array(
             __DIR__.'/Fixtures/Bundle2Bundle/foo.txt',
-            __DIR__.'/Fixtures/Bundle1Bundle/foo.txt'),
+            __DIR__.'/Fixtures/Bundle1Bundle/foo.txt',),
             $kernel->locateResource('@Bundle1Bundle/foo.txt', null, false));
     }
 
@@ -509,7 +509,7 @@ public function testLocateResourceReturnsAllMatchesBis()
             ->method('getBundle')
             ->will($this->returnValue(array(
                 $this->getBundle(__DIR__.'/Fixtures/Bundle1Bundle'),
-                $this->getBundle(__DIR__.'/Foobar')
+                $this->getBundle(__DIR__.'/Foobar'),
             )))
         ;
 
@@ -560,7 +560,7 @@ public function testLocateResourceReturnsTheDirOneForResourcesAndBundleOnes()
 
         $this->assertEquals(array(
             __DIR__.'/Fixtures/Resources/Bundle1Bundle/foo.txt',
-            __DIR__.'/Fixtures/Bundle1Bundle/Resources/foo.txt'),
+            __DIR__.'/Fixtures/Bundle1Bundle/Resources/foo.txt',),
             $kernel->locateResource('@Bundle1Bundle/Resources/foo.txt', __DIR__.'/Fixtures/Resources', false)
         );
     }
diff --git a/Tests/Profiler/Mock/RedisMock.php b/Tests/Profiler/Mock/RedisMock.php
@@ -18,7 +18,6 @@
  */
 class RedisMock
 {
-
     private $connected;
     private $storage;
 
diff --git a/Tests/Profiler/MongoDbProfilerStorageTest.php b/Tests/Profiler/MongoDbProfilerStorageTest.php

Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public function collect(Request $request, Response $response, \Exception $except`
`41`	`41`	`$this->data = array(`
`42`	`42`	`'error_count' => $this->logger->countErrors(),`
`43`	`43`	`'logs' => $this->sanitizeLogs($this->logger->getLogs()),`
`44`		`- 'deprecation_count' => $this->computeDeprecationCount()`
	`44`	`+ 'deprecation_count' => $this->computeDeprecationCount(),`
`45`	`45`	`);`
`46`	`46`	`}`
`47`	`47`	`}`
Original file line number	Diff line number	Diff line change
`@@ -152,7 +152,6 @@ public function getKernel()`
`152`	`152`	`return $this->kernel;`
`153`	`153`	`}`
`154`	`154`
`155`		`-`
`156`	`155`	`/**`
`157`	`156`	`* Gets the Esi instance`
`158`	`157`	`*`
Original file line number	Diff line number	Diff line change
`@@ -519,7 +519,6 @@ protected function initializeBundles()`
`519`	`519`	`array_pop($bundleMap);`
`520`	`520`	`}`
`521`	`521`	`}`
`522`		`-`
`523`	`522`	`}`
`524`	`523`
`525`	`524`	`/**`
Original file line number	Diff line number	Diff line change
`@@ -53,12 +53,11 @@ public function find($ip, $url, $limit, $method, $start = null, $end = null)`
`53`	`53`	`$result = array();`
`54`	`54`
`55`	`55`	`foreach ($profileList as $item) {`
`56`		`-`
`57`	`56`	`if ($limit === 0) {`
`58`	`57`	`break;`
`59`	`58`	`}`
`60`	`59`
`61`		`- if ($item=='') {`
	`60`	`+ if ($item == '') {`
`62`	`61`	`continue;`
`63`	`62`	`}`
`64`	`63`
`@@ -166,7 +165,6 @@ public function write(Profile $profile)`
`166`	`165`	`$profileIndexed = false !== $this->getValue($this->getItemName($profile->getToken()));`
`167`	`166`
`168`	`167`	`if ($this->setValue($this->getItemName($profile->getToken()), $data, $this->lifetime)) {`
`169`		`-`
`170`	`168`	`if (!$profileIndexed) {`
`171`	`169`	`// Add to index`
`172`	`170`	`$indexName = $this->getIndexName();`
`@@ -304,5 +302,4 @@ private function isItemNameValid($name)`
`304`	`302`
`305`	`303`	`return true;`
`306`	`304`	`}`
`307`		`-`
`308`	`305`	`}`
Original file line number	Diff line number	Diff line change
`@@ -70,7 +70,7 @@ public function find($ip, $url, $limit, $method, $start = null, $end = null)`
`70`	`70`	`}`
`71`	`71`
`72`	`72`	`if (!empty($start) && $csvTime < $start) {`
`73`		`- continue;`
	`73`	`+ continue;`
`74`	`74`	`}`
`75`	`75`
`76`	`76`	`if (!empty($end) && $csvTime > $end) {`