@@ -187,7 +187,7 @@ |
||
| 187 | 187 | /** |
| 188 | 188 | * Returns if the given $value is considered empty by the field type. |
| 189 | 189 | * |
| 190 | - * @param mixed $value |
|
| 190 | + * @param SPIValue $value |
|
| 191 | 191 | * |
| 192 | 192 | * @return bool |
| 193 | 193 | */ |
@@ -113,7 +113,7 @@ |
||
| 113 | 113 | * It will be used to generate content name and url alias if current field is designated |
| 114 | 114 | * to be used in the content name/urlAlias pattern. |
| 115 | 115 | * |
| 116 | - * @param mixed $value |
|
| 116 | + * @param SPIValue $value |
|
| 117 | 117 | * |
| 118 | 118 | * @return string |
| 119 | 119 | */ |
@@ -187,7 +187,7 @@ |
||
| 187 | 187 | /** |
| 188 | 188 | * Returns if the given $value is considered empty by the field type. |
| 189 | 189 | * |
| 190 | - * @param mixed $value |
|
| 190 | + * @param SPIValue $value |
|
| 191 | 191 | * |
| 192 | 192 | * @return bool |
| 193 | 193 | */ |
@@ -65,7 +65,7 @@ |
||
| 65 | 65 | /** |
| 66 | 66 | * Returns a mock for Storage. |
| 67 | 67 | * |
| 68 | - * @return Storage |
|
| 68 | + * @return \PHPUnit\Framework\MockObject\MockObject |
|
| 69 | 69 | */ |
| 70 | 70 | protected function getFieldValueConverterMock() |
| 71 | 71 | { |
@@ -67,7 +67,7 @@ |
||
| 67 | 67 | /** |
| 68 | 68 | * Returns a mock for Storage. |
| 69 | 69 | * |
| 70 | - * @return Storage |
|
| 70 | + * @return \PHPUnit\Framework\MockObject\MockObject |
|
| 71 | 71 | */ |
| 72 | 72 | protected function getStorageMock() |
| 73 | 73 | { |
@@ -130,7 +130,7 @@ |
||
| 130 | 130 | /** |
| 131 | 131 | * Returns a mock for persistence field type. |
| 132 | 132 | * |
| 133 | - * @return \eZ\Publish\SPI\Persistence\FieldType |
|
| 133 | + * @return \PHPUnit\Framework\MockObject\MockObject |
|
| 134 | 134 | */ |
| 135 | 135 | protected function getFieldTypeMock() |
| 136 | 136 | { |
@@ -231,7 +231,7 @@ discard block |
||
| 231 | 231 | /** |
| 232 | 232 | * Returns data for the first level children of the location identified by given $locationId. |
| 233 | 233 | * |
| 234 | - * @param mixed $locationId |
|
| 234 | + * @param integer $locationId |
|
| 235 | 235 | * |
| 236 | 236 | * @return array |
| 237 | 237 | */ |
@@ -358,6 +358,9 @@ discard block |
||
| 358 | 358 | } |
| 359 | 359 | } |
| 360 | 360 | |
| 361 | + /** |
|
| 362 | + * @param string $pathString |
|
| 363 | + */ |
|
| 361 | 364 | private function isHiddenByParent($pathString, array $rows) |
| 362 | 365 | { |
| 363 | 366 | $parentNodeIds = explode('/', trim($pathString, '/')); |
@@ -558,8 +561,8 @@ discard block |
||
| 558 | 561 | * Make the location identified by $locationId1 refer to the Content |
| 559 | 562 | * referred to by $locationId2 and vice versa. |
| 560 | 563 | * |
| 561 | - * @param mixed $locationId1 |
|
| 562 | - * @param mixed $locationId2 |
|
| 564 | + * @param integer $locationId1 |
|
| 565 | + * @param integer $locationId2 |
|
| 563 | 566 | * |
| 564 | 567 | * @return bool |
| 565 | 568 | */ |
@@ -735,7 +738,7 @@ discard block |
||
| 735 | 738 | * Create an entry in the node assignment table. |
| 736 | 739 | * |
| 737 | 740 | * @param \eZ\Publish\SPI\Persistence\Content\Location\CreateStruct $createStruct |
| 738 | - * @param mixed $parentNodeId |
|
| 741 | + * @param string $parentNodeId |
|
| 739 | 742 | * @param int $type |
| 740 | 743 | */ |
| 741 | 744 | public function createNodeAssignment(CreateStruct $createStruct, $parentNodeId, $type = self::NODE_ASSIGNMENT_OP_CODE_CREATE_NOP) |
@@ -863,8 +866,8 @@ discard block |
||
| 863 | 866 | * |
| 864 | 867 | * Convert existing node assignments into real locations. |
| 865 | 868 | * |
| 866 | - * @param mixed $contentId |
|
| 867 | - * @param mixed $versionNo |
|
| 869 | + * @param integer $contentId |
|
| 870 | + * @param integer $versionNo |
|
| 868 | 871 | */ |
| 869 | 872 | public function createLocationsFromNodeAssignments($contentId, $versionNo) |
| 870 | 873 | { |
@@ -932,8 +935,8 @@ discard block |
||
| 932 | 935 | /** |
| 933 | 936 | * Updates all Locations of content identified with $contentId with $versionNo. |
| 934 | 937 | * |
| 935 | - * @param mixed $contentId |
|
| 936 | - * @param mixed $versionNo |
|
| 938 | + * @param integer $contentId |
|
| 939 | + * @param integer $versionNo |
|
| 937 | 940 | */ |
| 938 | 941 | public function updateLocationsContentVersionNo($contentId, $versionNo) |
| 939 | 942 | { |
@@ -1092,8 +1095,8 @@ discard block |
||
| 1092 | 1095 | * the node identified by given $locationId (current main node). |
| 1093 | 1096 | * Assumes that content has more than one location. |
| 1094 | 1097 | * |
| 1095 | - * @param mixed $contentId |
|
| 1096 | - * @param mixed $locationId |
|
| 1098 | + * @param integer $contentId |
|
| 1099 | + * @param integer $locationId |
|
| 1097 | 1100 | * |
| 1098 | 1101 | * @return array |
| 1099 | 1102 | */ |
@@ -1129,9 +1132,9 @@ discard block |
||
| 1129 | 1132 | * |
| 1130 | 1133 | * The associated content object is left untouched. |
| 1131 | 1134 | * |
| 1132 | - * @param mixed $locationId |
|
| 1135 | + * @param integer $locationId |
|
| 1133 | 1136 | * |
| 1134 | - * @return bool |
|
| 1137 | + * @return boolean|null |
|
| 1135 | 1138 | */ |
| 1136 | 1139 | public function trashLocation($locationId) |
| 1137 | 1140 | { |
@@ -1161,8 +1164,8 @@ discard block |
||
| 1161 | 1164 | * at the old position. If this is not possible ( because the old location |
| 1162 | 1165 | * does not exist any more) and exception is thrown. |
| 1163 | 1166 | * |
| 1164 | - * @param mixed $locationId |
|
| 1165 | - * @param mixed|null $newParentId |
|
| 1167 | + * @param integer $locationId |
|
| 1168 | + * @param integer $newParentId |
|
| 1166 | 1169 | * |
| 1167 | 1170 | * @return \eZ\Publish\SPI\Persistence\Content\Location |
| 1168 | 1171 | */ |
@@ -1415,10 +1418,10 @@ discard block |
||
| 1415 | 1418 | * Updates ezcontentobject_tree table for the given $contentId and eznode_assignment table for the given |
| 1416 | 1419 | * $contentId, $parentLocationId and $versionNo |
| 1417 | 1420 | * |
| 1418 | - * @param mixed $contentId |
|
| 1419 | - * @param mixed $locationId |
|
| 1420 | - * @param mixed $versionNo version number, needed to update eznode_assignment table |
|
| 1421 | - * @param mixed $parentLocationId parent location of location identified by $locationId, needed to update |
|
| 1421 | + * @param integer $contentId |
|
| 1422 | + * @param integer $locationId |
|
| 1423 | + * @param integer $versionNo version number, needed to update eznode_assignment table |
|
| 1424 | + * @param integer $parentLocationId parent location of location identified by $locationId, needed to update |
|
| 1422 | 1425 | * eznode_assignment table |
| 1423 | 1426 | */ |
| 1424 | 1427 | public function changeMainLocation($contentId, $locationId, $versionNo, $parentLocationId) |
@@ -1548,7 +1551,7 @@ discard block |
||
| 1548 | 1551 | * |
| 1549 | 1552 | * @todo Align with createNodeQueryBuilder, removing the need for both(?) |
| 1550 | 1553 | * |
| 1551 | - * @param array $columns list of columns to fetch |
|
| 1554 | + * @param string[] $columns list of columns to fetch |
|
| 1552 | 1555 | * |
| 1553 | 1556 | * @return \Doctrine\DBAL\Query\QueryBuilder |
| 1554 | 1557 | */ |
@@ -212,7 +212,7 @@ discard block |
||
| 212 | 212 | * |
| 213 | 213 | * @param string $name |
| 214 | 214 | * |
| 215 | - * @return \eZ\Publish\SPI\FieldType\ValidationError|null |
|
| 215 | + * @return ValidationError|null |
|
| 216 | 216 | */ |
| 217 | 217 | private function validateSettingName($name) |
| 218 | 218 | { |
@@ -236,7 +236,7 @@ discard block |
||
| 236 | 236 | * @param string $name |
| 237 | 237 | * @param string $value |
| 238 | 238 | * |
| 239 | - * @return \eZ\Publish\SPI\FieldType\ValidationError|null |
|
| 239 | + * @return ValidationError|null |
|
| 240 | 240 | */ |
| 241 | 241 | private function validateDefaultAuthorSetting($name, $value) |
| 242 | 242 | { |
@@ -59,7 +59,7 @@ |
||
| 59 | 59 | /** |
| 60 | 60 | * Returns relative image path. |
| 61 | 61 | * |
| 62 | - * @param $url string |
|
| 62 | + * @param string $url string |
|
| 63 | 63 | * @return string |
| 64 | 64 | */ |
| 65 | 65 | protected function rewriteUrl($url) |