orocrm / platform

src/Oro/Bundle/EntityExtendBundle/Form/Type/TargetType.php

Doc Comments +12 added lines

@@ -69,6 +69,10 @@ 
         );
     }
 
+    /**
+     * @param string $entityClassName
+     * @param string|null $relationType
+     */
     protected function getEntityChoiceList($entityClassName, $relationType)
     {
         $choices       = [];
@@ -82,6 +86,10 @@ 
         if (in_array($relationType, [RelationTypeBase::ONE_TO_MANY, RelationTypeBase::MANY_TO_MANY], true)) {
             $entityIds = array_filter(
                 $entityIds,
+
+                /**
+                 * @param string $configId
+                 */
                 function (EntityConfigId $configId) {
                     $config = $this->configManager->getConfig($configId);
 
@@ -92,6 +100,10 @@ 
 
         $entityIds = array_filter(
             $entityIds,
+
+            /**
+             * @param string $configId
+             */
             function (EntityConfigId $configId) {
                 $config = $this->configManager->getConfig($configId);
 

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/Migration/ExtendOptionsBuilder.php

Doc Comments +1 added lines, -1 removed lines

@@ -234,7 +234,7 @@
      *
      * @param array  $options
      * @param string $name
-     * @return mixed
+     * @return string|null
      */
     protected function getAndRemoveOption(array &$options, $name)
     {

src/Oro/Bundle/EntityExtendBundle/Migrations/Schema/v1_0/RenameExtendTablesAndColumns.php

Doc Comments +6 added lines

@@ -226,6 +226,9 @@ 
         }
     }
 
+    /**
+     * @param string $associationName
+     */
     protected function renameManyToOneExtendField(
         Schema $schema,
         QueryBag $queries,
@@ -247,6 +250,9 @@ 
         }
     }
 
+    /**
+     * @param string $associationName
+     */
     protected function renameOneToManyExtendField(
         Schema $schema,
         QueryBag $queries,

Unused Use Statements -4 removed lines

@@ -4,17 +4,13 @@
 
 use Symfony\Component\HttpFoundation\JsonResponse;
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
-
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
-
 use Oro\Bundle\SecurityBundle\Annotation\AclAncestor;
 use Oro\Bundle\SecurityBundle\Annotation\Acl;
 use Oro\Bundle\SecurityBundle\Authentication\Token\UsernamePasswordOrganizationToken;
-
 use Oro\Bundle\UserBundle\Entity\User;
 use Oro\Bundle\UserBundle\Entity\UserApi;
-
 use Oro\Bundle\OrganizationBundle\Entity\Manager\BusinessUnitManager;
 use Oro\Bundle\OrganizationBundle\Entity\Organization;
 

src/Oro/Bundle/EntityExtendBundle/Tests/Unit/Grid/AdditionalFieldsExtensionTest.php

Doc Comments +5 added lines

@@ -490,6 +490,11 @@
         );
     }
 
+    /**
+     * @param string $className
+     * @param string $fieldName
+     * @param string $fieldType
+     */
     protected function setExpectationForGetFields($className, $fieldName, $fieldType, array $extendFieldConfig = [])
     {
         $extendConfig = new Config(new FieldConfigId('extend', $className, $fieldName, $fieldType));

src/Oro/Bundle/EntityMergeBundle/Controller/MergeController.php

Unused Use Statements -4 removed lines

@@ -3,15 +3,11 @@
 namespace Oro\Bundle\EntityMergeBundle\Controller;
 
 use Doctrine\ORM\EntityManager;
-
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
-
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
-use Symfony\Component\Validator\Constraint;
 use Symfony\Component\Validator\ConstraintViolation;
 use Symfony\Component\Validator\ValidatorInterface;
-
 use Oro\Bundle\EntityMergeBundle\Exception\ValidationException;
 use Oro\Bundle\DataGridBundle\Extension\MassAction\MassActionDispatcher;
 use Oro\Bundle\EntityMergeBundle\Data\EntityData;

src/Oro/Bundle/EntityMergeBundle/Model/Accessor/DefaultAccessor.php

Doc Comments +2 added lines, -2 removed lines

@@ -15,7 +15,7 @@ 
     protected $propertyAccessor;
 
     /**
-     * @return PropertyAccessor
+     * @return string
      */
     public function getName()
     {
@@ -27,7 +27,7 @@ 
      *
      * @param string        $entity
      * @param FieldMetadata $metadata
-     * @return string
+     * @return boolean
      */
     public function supports($entity, FieldMetadata $metadata)
     {

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/EntityMergeBundle/Model/Accessor/InverseAssociationAccessor.php

Doc Comments +1 added lines, -1 removed lines

@@ -37,7 +37,7 @@
      *
      * @param string        $entity
      * @param FieldMetadata $metadata
-     * @return string
+     * @return boolean
      */
     public function supports($entity, FieldMetadata $metadata)
     {

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/EntityMergeBundle/Tests/Unit/Model/Accessor/DelegateAccessorTest.php

Doc Comments +6 added lines

@@ -178,6 +178,9 @@ 
         $this->accessor->setValue($entity, $metadata, $value);
     }
 
+    /**
+     * @param string $name
+     */
     protected function createAccessor($name)
     {
         $result = $this->getMock('Oro\Bundle\EntityMergeBundle\Model\Accessor\AccessorInterface');
@@ -195,6 +198,9 @@ 
             ->getMock();
     }
 
+    /**
+     * @param integer $id
+     */
     protected function createTestEntity($id)
     {
         $result     = new \stdClass();

src/Oro/Bundle/EntityMergeBundle/Tests/Unit/Model/Accessor/InverseAssociationAccessorTest.php

Doc Comments +18 added lines

@@ -160,6 +160,9 @@ 
         ];
     }
 
+    /**
+     * @param string $fieldName
+     */
     protected function getFieldMetadata($fieldName = null, array $options = [])
     {
         $result = $this->createFieldMetadata();
@@ -179,6 +182,10 @@ 
             ->method('get')
             ->will(
                 $this->returnCallback(
+
+                    /**
+                     * @param string $code
+                     */
                     function ($code) use ($options) {
                         $this->assertArrayHasKey($code, $options);
                         return $options[$code];
@@ -190,6 +197,10 @@ 
             ->method('has')
             ->will(
                 $this->returnCallback(
+
+                    /**
+                     * @param string $code
+                     */
                     function ($code) use ($options) {
                         return isset($options[$code]);
                     }
@@ -211,11 +222,18 @@ 
             ->disableOriginalConstructor()->getMock();
     }
 
+    /**
+     * @param string $id
+     * @param EntityStub $parent
+     */
     protected function createEntity($id = null, $parent = null)
     {
         return new EntityStub($id, $parent);
     }
 
+    /**
+     * @param string $id
+     */
     protected function createRelatedEntity($id = null)
     {
         return new CollectionItemStub($id);