@@ -96,7 +96,7 @@ |
||
96 | 96 | * @param \PHPUnit_Framework_MockObject_MockObject|null $emMock |
97 | 97 | * @param boolean $isReturnManager |
98 | 98 | * |
99 | - * @return \PHPUnit_Framework_MockObject_MockObject |
|
99 | + * @return \Doctrine\Common\Persistence\ManagerRegistry |
|
100 | 100 | */ |
101 | 101 | protected function getDoctrineMock($emMock = null, $isReturnManager = true) |
102 | 102 | { |
@@ -3,7 +3,6 @@ |
||
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; |
@@ -107,7 +107,7 @@ |
||
107 | 107 | } |
108 | 108 | |
109 | 109 | /** |
110 | - * @return UserInterface|null |
|
110 | + * @return UserInterface |
|
111 | 111 | */ |
112 | 112 | protected function getUser() |
113 | 113 | { |
@@ -4,9 +4,7 @@ |
||
4 | 4 | |
5 | 5 | use Doctrine\Common\EventManager; |
6 | 6 | use Doctrine\DBAL\Driver\Connection; |
7 | - |
|
8 | 7 | use Symfony\Component\Filesystem\Filesystem; |
9 | - |
|
10 | 8 | use Oro\Component\TestUtils\ORM\Mocks\DriverMock; |
11 | 9 | use Oro\Component\TestUtils\ORM\Mocks\EntityManagerMock; |
12 | 10 |
@@ -118,7 +118,7 @@ |
||
118 | 118 | } |
119 | 119 | |
120 | 120 | /** |
121 | - * @return ConfigLogDiff[]|ArrayCollection |
|
121 | + * @return ConfigLogDiff[] |
|
122 | 122 | */ |
123 | 123 | public function getDiffs() |
124 | 124 | { |
@@ -3,7 +3,6 @@ |
||
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; |
@@ -174,7 +174,7 @@ |
||
174 | 174 | /** |
175 | 175 | * @param array $classMap [class_name => entity_config_id, ...] |
176 | 176 | * |
177 | - * @return array [entity_config_id => [field_name => is_configurable, ...], ...] |
|
177 | + * @return string [entity_config_id => [field_name => is_configurable, ...], ...] |
|
178 | 178 | */ |
179 | 179 | protected function loadConfigurableFields(array $classMap) |
180 | 180 | { |
@@ -3,7 +3,6 @@ |
||
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; |
@@ -150,7 +150,7 @@ discard block |
||
150 | 150 | /** |
151 | 151 | * @param string $className |
152 | 152 | * |
153 | - * @return EntityMetadata|null |
|
153 | + * @return \Metadata\ClassMetadata|null |
|
154 | 154 | */ |
155 | 155 | public function getEntityMetadata($className) |
156 | 156 | { |
@@ -558,7 +558,7 @@ discard block |
||
558 | 558 | } |
559 | 559 | |
560 | 560 | /** |
561 | - * @param array $models |
|
561 | + * @param ConfigModel[] $models |
|
562 | 562 | * |
563 | 563 | * @SuppressWarnings(PHPMD.NPathComplexity) |
564 | 564 | */ |
@@ -4,17 +4,13 @@ |
||
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 |
@@ -200,7 +200,7 @@ |
||
200 | 200 | * |
201 | 201 | * @param string $scope |
202 | 202 | * @param string $code |
203 | - * @param mixed $value |
|
203 | + * @param string $value |
|
204 | 204 | * @return ConfigModelIndexValue |
205 | 205 | */ |
206 | 206 | abstract protected function createIndexedValue($scope, $code, $value); |
@@ -262,7 +262,7 @@ discard block |
||
262 | 262 | |
263 | 263 | /** |
264 | 264 | * @param DatagridConfiguration $config |
265 | - * @param $itemType |
|
265 | + * @param string|null $itemType |
|
266 | 266 | */ |
267 | 267 | protected function addEntityConfigProperties(DatagridConfiguration $config, $itemType) |
268 | 268 | { |
@@ -321,7 +321,7 @@ discard block |
||
321 | 321 | * @param array $filters |
322 | 322 | * @param array $actions |
323 | 323 | * |
324 | - * @return callable |
|
324 | + * @return \Closure |
|
325 | 325 | */ |
326 | 326 | public function getActionConfigurationClosure($filters, $actions) |
327 | 327 | { |
@@ -4,17 +4,13 @@ |
||
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 |
@@ -218,6 +218,9 @@ discard block |
||
218 | 218 | return $values; |
219 | 219 | } |
220 | 220 | |
221 | + /** |
|
222 | + * @param string $cssClass |
|
223 | + */ |
|
221 | 224 | protected function appendClassAttr(array &$options, $cssClass) |
222 | 225 | { |
223 | 226 | if (isset($options['attr']['class'])) { |
@@ -227,6 +230,9 @@ discard block |
||
227 | 230 | } |
228 | 231 | } |
229 | 232 | |
233 | + /** |
|
234 | + * @param string $name |
|
235 | + */ |
|
230 | 236 | protected function setAttr(array &$options, $name, $value) |
231 | 237 | { |
232 | 238 | if (!isset($options['attr'])) { |
@@ -3,7 +3,6 @@ |
||
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; |
@@ -107,7 +107,7 @@ discard block |
||
107 | 107 | /** |
108 | 108 | * @param FieldConfigModel $entity |
109 | 109 | * @param array $searchContext |
110 | - * @return null|FieldConfigModel |
|
110 | + * @return FieldConfigModel |
|
111 | 111 | */ |
112 | 112 | protected function findExistingEntity($entity, array $searchContext = []) |
113 | 113 | { |
@@ -119,7 +119,7 @@ discard block |
||
119 | 119 | |
120 | 120 | /** |
121 | 121 | * @param FieldConfigModel $entity |
122 | - * @return null|FieldConfigModel |
|
122 | + * @return FieldConfigModel |
|
123 | 123 | */ |
124 | 124 | protected function validateAndUpdateContext(FieldConfigModel $entity) |
125 | 125 | { |
@@ -3,7 +3,6 @@ |
||
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; |