orocrm / platform

src/Oro/Bundle/EntityConfigBundle/Form/Type/ConfigScopeType.php

Doc Comments +6 added lines

@@ -218,6 +218,9 @@ 
         return $values;
     }
 
+    /**
+     * @param string $cssClass
+     */
     protected function appendClassAttr(array &$options, $cssClass)
     {
         if (isset($options['attr']['class'])) {
@@ -227,6 +230,9 @@ 
         }
     }
 
+    /**
+     * @param string $name
+     */
     protected function setAttr(array &$options, $name, $value)
     {
         if (!isset($options['attr'])) {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Oro\Component\PropertyAccess\Tests\Unit;
 
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
-
 use Oro\Component\PropertyAccess\PropertyAccessor;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;

src/Oro/Bundle/EntityConfigBundle/ImportExport/Strategy/EntityFieldImportStrategy.php

Doc Comments +2 added lines, -2 removed lines

@@ -107,7 +107,7 @@ 
     /**
      * @param FieldConfigModel $entity
      * @param array $searchContext
-     * @return null|FieldConfigModel
+     * @return FieldConfigModel
      */
     protected function findExistingEntity($entity, array $searchContext = [])
     {
@@ -119,7 +119,7 @@ 
 
     /**
      * @param FieldConfigModel $entity
-     * @return null|FieldConfigModel
+     * @return FieldConfigModel
      */
     protected function validateAndUpdateContext(FieldConfigModel $entity)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Oro\Component\PropertyAccess\Tests\Unit;
 
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
-
 use Oro\Component\PropertyAccess\PropertyAccessor;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;

src/Oro/Bundle/EntityConfigBundle/Migration/RemoveEnumFieldQuery.php

Doc Comments +1 added lines, -1 removed lines

@@ -119,7 +119,7 @@
 
     /**
      * @param LoggerInterface $logger
-     * @param $sql
+     * @param string $sql
      * @param array $parameters
      * @throws \Doctrine\DBAL\DBALException
      */

Unused Use Statements -2 removed lines

@@ -4,9 +4,7 @@
 
 use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Driver\Connection;
-
 use Symfony\Component\Filesystem\Filesystem;
-
 use Oro\Component\TestUtils\ORM\Mocks\DriverMock;
 use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock;
 

src/Oro/Bundle/EntityConfigBundle/Provider/ConfigProvider.php

Doc Comments +1 added lines, -1 removed lines

@@ -196,7 +196,7 @@
      * Applies the callback to configuration data to all configurable entities (if $className is not specified)
      * or all configurable fields of the given entity.
      *
-     * @param callable    $callback The callback function to run for configuration data for each object
+     * @param \Closure    $callback The callback function to run for configuration data for each object
      * @param string|null $className
      * @param bool        $withHidden
      *

Unused Use Statements -2 removed lines

@@ -4,9 +4,7 @@
 
 use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Driver\Connection;
-
 use Symfony\Component\Filesystem\Filesystem;
-
 use Oro\Component\TestUtils\ORM\Mocks\DriverMock;
 use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock;
 

src/Oro/Bundle/EntityConfigBundle/Tests/Unit/ImportExport/Strategy/EntityFieldImportStrategyTest.php

Doc Comments +1 added lines, -1 removed lines

@@ -186,7 +186,7 @@
     }
 
     /**
-     * @param object $object
+     * @param EntityFieldImportStrategy $object
      * @param string $property
      * @return mixed $value
      */

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Oro\Component\PropertyAccess\Tests\Unit;
 
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
-
 use Oro\Component\PropertyAccess\PropertyAccessor;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;

src/Oro/Bundle/EntityConfigBundle/Tools/ConfigHelper.php

Doc Comments +1 added lines, -1 removed lines

@@ -94,7 +94,7 @@
      *
      * @param string $className
      *
-     * @return array [$moduleName, $entityName]
+     * @return string[] [$moduleName, $entityName]
      */
     public static function getModuleAndEntityNames($className)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Oro\Component\PropertyAccess\Tests\Unit;
 
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
-
 use Oro\Component\PropertyAccess\PropertyAccessor;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;

src/Oro/Bundle/EntityConfigBundle/Tools/ConfigLoader.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      * Load entity configs from annotations to a database
      *
      * @param bool                 $force  Force overwrite config's option values
-     * @param callable|null        $filter function (ClassMetadata[] $doctrineAllMetadata)
+     * @param null|\Closure        $filter function (ClassMetadata[] $doctrineAllMetadata)
      * @param LoggerInterface|null $logger
      * @param bool                 $dryRun Log modifications without apply them
      *

Unused Use Statements -2 removed lines

@@ -4,9 +4,7 @@
 
 use Doctrine\Common\EventManager;
 use Doctrine\DBAL\Driver\Connection;
-
 use Symfony\Component\Filesystem\Filesystem;
-
 use Oro\Component\TestUtils\ORM\Mocks\DriverMock;
 use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock;
 

src/Oro/Bundle/EntityExtendBundle/Entity/Manager/AssociationManager.php

Doc Comments +2 added lines, -2 removed lines

@@ -103,7 +103,7 @@ 
      * @param string $attribute The name of the entity config attribute which indicates
      *                          whether the association is enabled or not
      *
-     * @return callable
+     * @return \Closure
      */
     public function getSingleOwnerFilter($scope, $attribute = 'enabled')
     {
@@ -121,7 +121,7 @@ 
      * @param string $attribute The name of the entity config attribute which is used to store
      *                          enabled associations
      *
-     * @return callable
+     * @return \Closure
      */
     public function getMultiOwnerFilter($scope, $attribute)
     {

Unused Use Statements -1 removed lines

@@ -3,7 +3,6 @@
 namespace Oro\Component\PropertyAccess\Tests\Unit;
 
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
-
 use Oro\Component\PropertyAccess\PropertyAccessor;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;

src/Oro/Bundle/EntityExtendBundle/Form/Guesser/ExtendFieldTypeGuesser.php

Unused Use Statements -4 removed lines

@@ -2,10 +2,7 @@ 
 
 namespace Oro\Bundle\EntityExtendBundle\Form\Guesser;
 
-use Symfony\Component\Form\Guess;
-
 use Doctrine\Common\Persistence\ManagerRegistry;
-
 use Oro\Bundle\EntityBundle\Form\Guesser\AbstractFormGuesser;
 use Oro\Bundle\EntityConfigBundle\Config\ConfigInterface;
 use Oro\Bundle\EntityConfigBundle\Config\Id\FieldConfigId;
@@ -15,7 +12,6 @@ 
 use Oro\Bundle\EntityExtendBundle\Tools\ExtendConfigDumper;
 use Oro\Bundle\EntityExtendBundle\Tools\ExtendHelper;
 use Oro\Bundle\EntityExtendBundle\Validator\Constraints\Decimal;
-
 use Symfony\Component\Validator\Constraints\Length;
 
 class ExtendFieldTypeGuesser extends AbstractFormGuesser