minor #13063 [2.3] [Bridge] [Propel] minor improvements to the Propel…

… bridge. (hhamon) This PR was merged into the 2.3 branch. Discussion ---------- [2.3] [Bridge] [Propel] minor improvements to the Propel bridge. | Q | A | ------------- | --- | Bug fix? | no | New feature? | no | BC breaks? | no | Deprecations? | no | Tests pass? | yes | Fixed tickets | ~ | License | MIT | Doc PR | ~ Commits ------- 6d9d460 [Bridge] [Propel] minor improvements to the Propel bridge.
symfony · Dec 24, 2014 · 9ea3f6a · 9ea3f6a
2 parents cad7f0e + 6d9d460
commit 9ea3f6a
Show file tree

Hide file tree

Showing 13 changed files with 75 additions and 64 deletions.
diff --git a/src/Symfony/Bridge/Propel1/DataCollector/PropelDataCollector.php b/src/Symfony/Bridge/Propel1/DataCollector/PropelDataCollector.php
@@ -26,7 +26,7 @@ class PropelDataCollector extends DataCollector
     /**
      * Propel logger.
      *
-     * @var \Symfony\Bridge\Propel1\Logger\PropelLogger
+     * @var PropelLogger
      */
     private $logger;
 
@@ -63,17 +63,17 @@ public function collect(Request $request, Response $response, \Exception $except
     /**
      * Returns the collector name.
      *
-     * @return string The collector name.
+     * @return string
      */
     public function getName()
     {
         return 'propel';
     }
 
     /**
-     * Returns queries.
+     * Returns the collected stats for all executed SQL queries.
      *
-     * @return array Queries
+     * @return array
      */
     public function getQueries()
     {
@@ -83,17 +83,17 @@ public function getQueries()
     /**
      * Returns the query count.
      *
-     * @return int The query count
+     * @return int
      */
     public function getQueryCount()
     {
         return $this->data['querycount'];
     }
 
     /**
-     * Returns the total time of queries.
+     * Returns the total time spent on running all queries.
      *
-     * @return float The total time of queries
+     * @return float
      */
     public function getTime()
     {
@@ -106,9 +106,9 @@ public function getTime()
     }
 
     /**
-     * Creates an array of Build objects.
+     * Computes the stats of all executed SQL queries.
      *
-     * @return array An array of Build objects
+     * @return array
      */
     private function buildQueries()
     {
@@ -136,9 +136,9 @@ private function buildQueries()
     }
 
     /**
-     * Count queries.
+     * Returns the total count of SQL queries.
      *
-     * @return int The number of queries.
+     * @return int
      */
     private function countQueries()
     {

diff --git a/src/Symfony/Bridge/Propel1/DependencyInjection/Security/UserProvider/PropelFactory.php b/src/Symfony/Bridge/Propel1/DependencyInjection/Security/UserProvider/PropelFactory.php
@@ -26,6 +26,12 @@ class PropelFactory implements UserProviderFactoryInterface
     private $key;
     private $providerId;
 
+    /**
+     * Constructor.
+     *
+     * @param string $key
+     * @param string $providerId
+     */
     public function __construct($key, $providerId)
     {
         $this->key = $key;
@@ -38,7 +44,7 @@ public function create(ContainerBuilder $container, $id, $config)
             ->setDefinition($id, new DefinitionDecorator($this->providerId))
             ->addArgument($config['class'])
             ->addArgument($config['property'])
-            ;
+        ;
     }
 
     public function getKey()
@@ -50,9 +56,14 @@ public function addConfiguration(NodeDefinition $node)
     {
         $node
             ->children()
-            ->scalarNode('class')->isRequired()->cannotBeEmpty()->end()
-            ->scalarNode('property')->defaultNull()->end()
+                ->scalarNode('class')
+                    ->isRequired()
+                    ->cannotBeEmpty()
+                ->end()
+                ->scalarNode('property')
+                    ->defaultNull()
+                ->end()
             ->end()
-            ;
+        ;
     }
 }
diff --git a/src/Symfony/Bridge/Propel1/Form/Type/ModelType.php b/src/Symfony/Bridge/Propel1/Form/Type/ModelType.php
@@ -55,18 +55,29 @@ class ModelType extends AbstractType
      */
     private $propertyAccessor;
 
+    /**
+     * Constructor.
+     *
+     * @param PropertyAccessorInterface|null $propertyAccessor
+     */
     public function __construct(PropertyAccessorInterface $propertyAccessor = null)
     {
         $this->propertyAccessor = $propertyAccessor ?: PropertyAccess::createPropertyAccessor();
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function buildForm(FormBuilderInterface $builder, array $options)
     {
         if ($options['multiple']) {
             $builder->addViewTransformer(new CollectionToArrayTransformer(), true);
         }
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function setDefaultOptions(OptionsResolverInterface $resolver)
     {
         $propertyAccessor = $this->propertyAccessor;
@@ -97,11 +108,17 @@ public function setDefaultOptions(OptionsResolverInterface $resolver)
         ));
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function getParent()
     {
         return 'choice';
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function getName()
     {
         return 'model';

diff --git a/src/Symfony/Bridge/Propel1/Form/Type/TranslationCollectionType.php b/src/Symfony/Bridge/Propel1/Form/Type/TranslationCollectionType.php
@@ -40,6 +40,9 @@ public function buildForm(FormBuilderInterface $builder, array $options)
         $builder->addEventSubscriber($listener);
     }
 
+    /**
+     * {@inheritdoc}
+     */
     public function getParent()
     {
         return 'collection';

diff --git a/src/Symfony/Bridge/Propel1/Tests/DataCollector/PropelDataCollectorTest.php b/src/Symfony/Bridge/Propel1/Tests/DataCollector/PropelDataCollectorTest.php
@@ -92,18 +92,19 @@ public function testCollectWithMultipleData()
     private function createCollector($queries)
     {
         $config = $this->getMock('\PropelConfiguration');
+
         $config
             ->expects($this->any())
             ->method('getParameter')
             ->will($this->returnArgument(1))
-            ;
+        ;
 
         $logger = $this->getMock('\Symfony\Bridge\Propel1\Logger\PropelLogger');
         $logger
             ->expects($this->any())
             ->method('getQueries')
             ->will($this->returnValue($queries))
-            ;
+        ;
 
         return new PropelDataCollector($logger, $config);
     }

diff --git a/src/Symfony/Bridge/Propel1/Tests/Fixtures/Column.php b/src/Symfony/Bridge/Propel1/Tests/Fixtures/Column.php
@@ -14,7 +14,6 @@
 class Column
 {
     private $name;
-
     private $type;
 
     public function __construct($name, $type)
@@ -35,13 +34,13 @@ public function isText()
         }
 
         switch ($this->type) {
-        case \PropelColumnTypes::CHAR:
-        case \PropelColumnTypes::VARCHAR:
-        case \PropelColumnTypes::LONGVARCHAR:
-        case \PropelColumnTypes::BLOB:
-        case \PropelColumnTypes::CLOB:
-        case \PropelColumnTypes::CLOB_EMU:
-            return true;
+            case \PropelColumnTypes::CHAR:
+            case \PropelColumnTypes::VARCHAR:
+            case \PropelColumnTypes::LONGVARCHAR:
+            case \PropelColumnTypes::BLOB:
+            case \PropelColumnTypes::CLOB:
+            case \PropelColumnTypes::CLOB_EMU:
+                return true;
         }
 
         return false;
@@ -54,6 +53,6 @@ public function getSize()
 
     public function isNotNull()
     {
-        return ('id' === $this->name);
+        return 'id' === $this->name;
     }
 }
diff --git a/src/Symfony/Bridge/Propel1/Tests/Fixtures/Item.php b/src/Symfony/Bridge/Propel1/Tests/Fixtures/Item.php
@@ -11,16 +11,11 @@
 
 namespace Symfony\Bridge\Propel1\Tests\Fixtures;
 
-use PropelPDO;
-
 class Item implements \Persistent
 {
     private $id;
-
     private $value;
-
     private $groupName;
-
     private $price;
 
     public function __construct($id = null, $value = null, $groupName = null, $price = null)
@@ -98,11 +93,11 @@ public function setDeleted($b)
     {
     }
 
-    public function delete(PropelPDO $con = null)
+    public function delete(\PropelPDO $con = null)
     {
     }
 
-    public function save(PropelPDO $con = null)
+    public function save(\PropelPDO $con = null)
     {
     }
 }
diff --git a/src/Symfony/Bridge/Propel1/Tests/Fixtures/TranslatableItem.php b/src/Symfony/Bridge/Propel1/Tests/Fixtures/TranslatableItem.php
@@ -11,16 +11,11 @@
 
 namespace Symfony\Bridge\Propel1\Tests\Fixtures;
 
-use PropelPDO;
-
 class TranslatableItem implements \Persistent
 {
     private $id;
-
     private $currentTranslations;
-
     private $groupName;
-
     private $price;
 
     public function __construct($id = null, $translations = array())
@@ -91,15 +86,15 @@ public function setDeleted($b)
     {
     }
 
-    public function delete(PropelPDO $con = null)
+    public function delete(\PropelPDO $con = null)
     {
     }
 
-    public function save(PropelPDO $con = null)
+    public function save(\PropelPDO $con = null)
     {
     }
 
-    public function getTranslation($locale = 'de', PropelPDO $con = null)
+    public function getTranslation($locale = 'de', \PropelPDO $con = null)
     {
         if (!isset($this->currentTranslations[$locale])) {
             $translation = new TranslatableItemI18n();

diff --git a/src/Symfony/Bridge/Propel1/Tests/Fixtures/TranslatableItemI18n.php b/src/Symfony/Bridge/Propel1/Tests/Fixtures/TranslatableItemI18n.php
@@ -11,18 +11,12 @@
 
 namespace Symfony\Bridge\Propel1\Tests\Fixtures;
 
-use PropelPDO;
-
 class TranslatableItemI18n implements \Persistent
 {
     private $id;
-
     private $locale;
-
     private $value;
-
     private $value2;
-
     private $item;
 
     public function __construct($id = null, $locale = null, $value = null)
@@ -84,11 +78,11 @@ public function setDeleted($b)
     {
     }
 
-    public function delete(PropelPDO $con = null)
+    public function delete(\PropelPDO $con = null)
     {
     }
 
-    public function save(PropelPDO $con = null)
+    public function save(\PropelPDO $con = null)
     {
     }
 

diff --git a/src/Symfony/Bridge/Propel1/Tests/Form/ChoiceList/ModelChoiceListTest.php b/src/Symfony/Bridge/Propel1/Tests/Form/ChoiceList/ModelChoiceListTest.php
@@ -249,14 +249,14 @@ public function testDontAllowInvalidChoiceValues()
      */
     public function testEmptyClass()
     {
-        $choiceList = new ModelChoiceList('');
+        new ModelChoiceList('');
     }
 
     /**
      * @expectedException \Symfony\Component\OptionsResolver\Exception\InvalidOptionsException
      */
     public function testInvalidClass()
     {
-        $choiceList = new ModelChoiceList('Foo\Bar\DoesNotExistClass');
+        new ModelChoiceList('Foo\Bar\DoesNotExistClass');
     }
 }
diff --git a/src/Symfony/Bridge/Propel1/Tests/Form/DataTransformer/CollectionToArrayTransformerTest.php b/src/Symfony/Bridge/Propel1/Tests/Form/DataTransformer/CollectionToArrayTransformerTest.php
@@ -11,7 +11,6 @@
 
 namespace Symfony\Bridge\Propel1\Tests\Form\DataTransformer;
 
-use PropelObjectCollection;
 use Symfony\Bridge\Propel1\Form\DataTransformer\CollectionToArrayTransformer;
 use Symfony\Bridge\Propel1\Tests\Propel1TestCase;
 
@@ -32,7 +31,7 @@ protected function setUp()
 
     public function testTransform()
     {
-        $result = $this->transformer->transform(new PropelObjectCollection());
+        $result = $this->transformer->transform(new \PropelObjectCollection());
 
         $this->assertTrue(is_array($result));
         $this->assertCount(0, $result);
@@ -56,7 +55,7 @@ public function testTransformThrowsExceptionIfNotPropelObjectCollection()
 
     public function testTransformWithData()
     {
-        $coll = new PropelObjectCollection();
+        $coll = new \PropelObjectCollection();
         $coll->setData(array('foo', 'bar'));
 
         $result = $this->transformer->transform($coll);

diff --git a/src/Symfony/Bridge/Propel1/Tests/Form/PropelTypeGuesserTest.php b/src/Symfony/Bridge/Propel1/Tests/Form/PropelTypeGuesserTest.php
@@ -18,16 +18,20 @@
 class PropelTypeGuesserTest extends Propel1TestCase
 {
     const CLASS_NAME = 'Symfony\Bridge\Propel1\Tests\Fixtures\Item';
-
     const UNKNOWN_CLASS_NAME = 'Symfony\Bridge\Propel1\Tests\Fixtures\UnknownItem';
 
     private $guesser;
 
-    public function setUp()
+    protected function setUp()
     {
         $this->guesser = new PropelTypeGuesser();
     }
 
+    protected function tearDown()
+    {
+        $this->guesser = null;
+    }
+
     public function testGuessMaxLengthWithText()
     {
         $value = $this->guesser->guessMaxLength(self::CLASS_NAME, 'value');