Completed
Pull Request — master (#361)
by Ramūnas
37:16
created
src/Oro/Bundle/EntityExtendBundle/Entity/Manager/AssociationManager.php 2 patches
Doc Comments   +2 added lines, -2 removed lines patch added patch discarded remove patch
@@ -103,7 +103,7 @@  discard block
 block discarded – undo
103 103
      * @param string $attribute The name of the entity config attribute which indicates
104 104
      *                          whether the association is enabled or not
105 105
      *
106
-     * @return callable
106
+     * @return \Closure
107 107
      */
108 108
     public function getSingleOwnerFilter($scope, $attribute = 'enabled')
109 109
     {
@@ -121,7 +121,7 @@  discard block
 block discarded – undo
121 121
      * @param string $attribute The name of the entity config attribute which is used to store
122 122
      *                          enabled associations
123 123
      *
124
-     * @return callable
124
+     * @return \Closure
125 125
      */
126 126
     public function getMultiOwnerFilter($scope, $attribute)
127 127
     {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Oro\Component\PropertyAccess\Tests\Unit;
4 4
 
5 5
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
6
-
7 6
 use Oro\Component\PropertyAccess\PropertyAccessor;
8 7
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
9 8
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;
Please login to merge, or discard this patch.
src/Oro/Bundle/EntityExtendBundle/Form/Guesser/ExtendFieldTypeGuesser.php 1 patch
Unused Use Statements   -4 removed lines patch added patch discarded remove patch
@@ -2,10 +2,7 @@  discard block
 block discarded – undo
2 2
 
3 3
 namespace Oro\Bundle\EntityExtendBundle\Form\Guesser;
4 4
 
5
-use Symfony\Component\Form\Guess;
6
-
7 5
 use Doctrine\Common\Persistence\ManagerRegistry;
8
-
9 6
 use Oro\Bundle\EntityBundle\Form\Guesser\AbstractFormGuesser;
10 7
 use Oro\Bundle\EntityConfigBundle\Config\ConfigInterface;
11 8
 use Oro\Bundle\EntityConfigBundle\Config\Id\FieldConfigId;
@@ -15,7 +12,6 @@  discard block
 block discarded – undo
15 12
 use Oro\Bundle\EntityExtendBundle\Tools\ExtendConfigDumper;
16 13
 use Oro\Bundle\EntityExtendBundle\Tools\ExtendHelper;
17 14
 use Oro\Bundle\EntityExtendBundle\Validator\Constraints\Decimal;
18
-
19 15
 use Symfony\Component\Validator\Constraints\Length;
20 16
 
21 17
 class ExtendFieldTypeGuesser extends AbstractFormGuesser
Please login to merge, or discard this patch.
src/Oro/Bundle/EntityExtendBundle/Form/Type/TargetType.php 2 patches
Doc Comments   +12 added lines patch added patch discarded remove patch
@@ -69,6 +69,10 @@  discard block
 block discarded – undo
69 69
         );
70 70
     }
71 71
 
72
+    /**
73
+     * @param string $entityClassName
74
+     * @param string|null $relationType
75
+     */
72 76
     protected function getEntityChoiceList($entityClassName, $relationType)
73 77
     {
74 78
         $choices       = [];
@@ -82,6 +86,10 @@  discard block
 block discarded – undo
82 86
         if (in_array($relationType, [RelationTypeBase::ONE_TO_MANY, RelationTypeBase::MANY_TO_MANY], true)) {
83 87
             $entityIds = array_filter(
84 88
                 $entityIds,
89
+
90
+                /**
91
+                 * @param string $configId
92
+                 */
85 93
                 function (EntityConfigId $configId) {
86 94
                     $config = $this->configManager->getConfig($configId);
87 95
 
@@ -92,6 +100,10 @@  discard block
 block discarded – undo
92 100
 
93 101
         $entityIds = array_filter(
94 102
             $entityIds,
103
+
104
+            /**
105
+             * @param string $configId
106
+             */
95 107
             function (EntityConfigId $configId) {
96 108
                 $config = $this->configManager->getConfig($configId);
97 109
 
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Oro\Component\PropertyAccess\Tests\Unit;
4 4
 
5 5
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
6
-
7 6
 use Oro\Component\PropertyAccess\PropertyAccessor;
8 7
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
9 8
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;
Please login to merge, or discard this patch.
src/Oro/Bundle/EntityExtendBundle/Migration/ExtendOptionsBuilder.php 1 patch
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -234,7 +234,7 @@
 block discarded – undo
234 234
      *
235 235
      * @param array  $options
236 236
      * @param string $name
237
-     * @return mixed
237
+     * @return string|null
238 238
      */
239 239
     protected function getAndRemoveOption(array &$options, $name)
240 240
     {
Please login to merge, or discard this patch.
src/Oro/Bundle/EntityExtendBundle/Migration/Extension/ExtendExtension.php 2 patches
Doc Comments   +3 added lines, -3 removed lines patch added patch discarded remove patch
@@ -270,7 +270,7 @@  discard block
 block discarded – undo
270 270
      * @param string        $enumCode        The target enum identifier
271 271
      * @param bool          $isMultiple      Indicates whether several options can be selected for this enum
272 272
      *                                       or it supports only one selected option
273
-     * @param bool|string[] $immutable       Indicates whether the changing the list of enum values and
273
+     * @param boolean $immutable       Indicates whether the changing the list of enum values and
274 274
      *                                       public flag is allowed or not. More details can be found
275 275
      *                                       in entity_config.yml
276 276
      * @param array         $options
@@ -341,7 +341,7 @@  discard block
 block discarded – undo
341 341
      * Adds one-to-many relation
342 342
      *
343 343
      * @param Schema       $schema
344
-     * @param Table|string $table                     A Table object or table name
344
+     * @param Table $table                     A Table object or table name
345 345
      * @param string       $associationName           A relation name
346 346
      * @param Table|string $targetTable               A Table object or table name
347 347
      * @param string[]     $targetTitleColumnNames    Column names are used to show a title of related entity
@@ -525,7 +525,7 @@  discard block
 block discarded – undo
525 525
      * Adds many-to-one relation
526 526
      *
527 527
      * @param Schema       $schema
528
-     * @param Table|string $table            A Table object or table name
528
+     * @param Table $table            A Table object or table name
529 529
      * @param string       $associationName  A relation name
530 530
      * @param Table|string $targetTable      A Table object or table name
531 531
      * @param string       $targetColumnName A column name is used to show related entity
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Oro\Component\PropertyAccess\Tests\Unit;
4 4
 
5 5
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
6
-
7 6
 use Oro\Component\PropertyAccess\PropertyAccessor;
8 7
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
9 8
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;
Please login to merge, or discard this patch.
EntityExtendBundle/Migrations/Schema/v1_0/RenameExtendTablesAndColumns.php 2 patches
Doc Comments   +6 added lines patch added patch discarded remove patch
@@ -226,6 +226,9 @@  discard block
 block discarded – undo
226 226
         }
227 227
     }
228 228
 
229
+    /**
230
+     * @param string $associationName
231
+     */
229 232
     protected function renameManyToOneExtendField(
230 233
         Schema $schema,
231 234
         QueryBag $queries,
@@ -247,6 +250,9 @@  discard block
 block discarded – undo
247 250
         }
248 251
     }
249 252
 
253
+    /**
254
+     * @param string $associationName
255
+     */
250 256
     protected function renameOneToManyExtendField(
251 257
         Schema $schema,
252 258
         QueryBag $queries,
Please login to merge, or discard this patch.
Unused Use Statements   -4 removed lines patch added patch discarded remove patch
@@ -4,17 +4,13 @@
 block discarded – undo
4 4
 
5 5
 use Symfony\Component\HttpFoundation\JsonResponse;
6 6
 use Symfony\Bundle\FrameworkBundle\Controller\Controller;
7
-
8 7
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Route;
9 8
 use Sensio\Bundle\FrameworkExtraBundle\Configuration\Template;
10
-
11 9
 use Oro\Bundle\SecurityBundle\Annotation\AclAncestor;
12 10
 use Oro\Bundle\SecurityBundle\Annotation\Acl;
13 11
 use Oro\Bundle\SecurityBundle\Authentication\Token\UsernamePasswordOrganizationToken;
14
-
15 12
 use Oro\Bundle\UserBundle\Entity\User;
16 13
 use Oro\Bundle\UserBundle\Entity\UserApi;
17
-
18 14
 use Oro\Bundle\OrganizationBundle\Entity\Manager\BusinessUnitManager;
19 15
 use Oro\Bundle\OrganizationBundle\Entity\Organization;
20 16
 
Please login to merge, or discard this patch.
Oro/Bundle/EntityExtendBundle/Tests/Unit/Form/Type/EnumValueTypeTest.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -267,7 +267,7 @@
 block discarded – undo
267 267
     }
268 268
 
269 269
     /**
270
-     * @return \PHPUnit_Framework_MockObject_MockObject|ConstraintValidatorFactoryInterface
270
+     * @return ConstraintValidatorFactoryInterface
271 271
      */
272 272
     protected function getConstraintValidatorFactory()
273 273
     {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Oro\Component\PropertyAccess\Tests\Unit;
4 4
 
5 5
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
6
-
7 6
 use Oro\Component\PropertyAccess\PropertyAccessor;
8 7
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
9 8
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;
Please login to merge, or discard this patch.
Bundle/EntityExtendBundle/Tests/Unit/Grid/AdditionalFieldsExtensionTest.php 1 patch
Doc Comments   +5 added lines patch added patch discarded remove patch
@@ -490,6 +490,11 @@
 block discarded – undo
490 490
         );
491 491
     }
492 492
 
493
+    /**
494
+     * @param string $className
495
+     * @param string $fieldName
496
+     * @param string $fieldType
497
+     */
493 498
     protected function setExpectationForGetFields($className, $fieldName, $fieldType, array $extendFieldConfig = [])
494 499
     {
495 500
         $extendConfig = new Config(new FieldConfigId('extend', $className, $fieldName, $fieldType));
Please login to merge, or discard this patch.
src/Oro/Bundle/EntityExtendBundle/Validator/FieldNameValidationHelper.php 2 patches
Doc Comments   +1 added lines, -1 removed lines patch added patch discarded remove patch
@@ -65,7 +65,7 @@
 block discarded – undo
65 65
      * @param string $className
66 66
      * @param string $fieldName
67 67
      *
68
-     * @return Config|null
68
+     * @return \Oro\Bundle\EntityConfigBundle\Config\ConfigInterface|null
69 69
      */
70 70
     public function findExtendFieldConfig($className, $fieldName)
71 71
     {
Please login to merge, or discard this patch.
Unused Use Statements   -1 removed lines patch added patch discarded remove patch
@@ -3,7 +3,6 @@
 block discarded – undo
3 3
 namespace Oro\Component\PropertyAccess\Tests\Unit;
4 4
 
5 5
 use Symfony\Component\PropertyAccess\Exception\NoSuchPropertyException;
6
-
7 6
 use Oro\Component\PropertyAccess\PropertyAccessor;
8 7
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClass;
9 8
 use Oro\Component\PropertyAccess\Tests\Unit\Fixtures\TestClassMagicCall;
Please login to merge, or discard this patch.