@@ -125,6 +125,9 @@ |
||
| 125 | 125 | $this->assertNull($this->provider->loadMainLocation($contentId)); |
| 126 | 126 | } |
| 127 | 127 | |
| 128 | + /** |
|
| 129 | + * @param integer $mainLocationId |
|
| 130 | + */ |
|
| 128 | 131 | private function getContentMock(int $contentId, ?int $mainLocationId = null, bool $published = false): Content |
| 129 | 132 | { |
| 130 | 133 | $contentInfo = new APIContentInfo([ |
@@ -15,6 +15,9 @@ |
||
| 15 | 15 | |
| 16 | 16 | abstract class AbstractViewTest extends TestCase |
| 17 | 17 | { |
| 18 | + /** |
|
| 19 | + * @param string $template |
|
| 20 | + */ |
|
| 18 | 21 | abstract protected function createViewUnderTest($template = null, array $parameters = [], $viewType = 'full'): View; |
| 19 | 22 | |
| 20 | 23 | /** |
@@ -68,6 +68,7 @@ |
||
| 68 | 68 | |
| 69 | 69 | /** |
| 70 | 70 | * {@inheritdoc} |
| 71 | + * @param string $key |
|
| 71 | 72 | */ |
| 72 | 73 | public function getItem($key) |
| 73 | 74 | { |
@@ -150,7 +150,7 @@ discard block |
||
| 150 | 150 | * Will contain always a complete list of fields. |
| 151 | 151 | * |
| 152 | 152 | * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct Content creation struct. |
| 153 | - * @param mixed $versionNo Used by self::copy() to maintain version numbers |
|
| 153 | + * @param integer $versionNo Used by self::copy() to maintain version numbers |
|
| 154 | 154 | * |
| 155 | 155 | * @return \eZ\Publish\SPI\Persistence\Content Content value object |
| 156 | 156 | */ |
@@ -432,7 +432,7 @@ discard block |
||
| 432 | 432 | /** |
| 433 | 433 | * Returns the metadata object for a content identified by $remoteId. |
| 434 | 434 | * |
| 435 | - * @param mixed $remoteId |
|
| 435 | + * @param string $remoteId |
|
| 436 | 436 | * |
| 437 | 437 | * @return \eZ\Publish\SPI\Persistence\Content\ContentInfo |
| 438 | 438 | */ |
@@ -593,7 +593,7 @@ discard block |
||
| 593 | 593 | * |
| 594 | 594 | * @param int $contentId |
| 595 | 595 | * |
| 596 | - * @return bool |
|
| 596 | + * @return boolean|null |
|
| 597 | 597 | */ |
| 598 | 598 | public function deleteContent($contentId) |
| 599 | 599 | { |
@@ -625,7 +625,7 @@ discard block |
||
| 625 | 625 | * @param int $contentId |
| 626 | 626 | * @param int $versionNo |
| 627 | 627 | * |
| 628 | - * @return bool |
|
| 628 | + * @return boolean|null |
|
| 629 | 629 | */ |
| 630 | 630 | public function deleteVersion($contentId, $versionNo) |
| 631 | 631 | { |
@@ -750,7 +750,7 @@ discard block |
||
| 750 | 750 | * |
| 751 | 751 | * @todo Should the existence verifications happen here or is this supposed to be handled at a higher level? |
| 752 | 752 | * |
| 753 | - * @param mixed $relationId |
|
| 753 | + * @param integer $relationId |
|
| 754 | 754 | * @param int $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON, |
| 755 | 755 | * \eZ\Publish\API\Repository\Values\Content\Relation::EMBED, |
| 756 | 756 | * \eZ\Publish\API\Repository\Values\Content\Relation::LINK, |
@@ -764,7 +764,7 @@ discard block |
||
| 764 | 764 | /** |
| 765 | 765 | * Loads relations from $sourceContentId. Optionally, loads only those with $type and $sourceContentVersionNo. |
| 766 | 766 | * |
| 767 | - * @param mixed $sourceContentId Source Content ID |
|
| 767 | + * @param integer $sourceContentId Source Content ID |
|
| 768 | 768 | * @param mixed|null $sourceContentVersionNo Source Content Version, null if not specified |
| 769 | 769 | * @param int|null $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON, |
| 770 | 770 | * \eZ\Publish\API\Repository\Values\Content\Relation::EMBED, |
@@ -785,7 +785,7 @@ discard block |
||
| 785 | 785 | * |
| 786 | 786 | * Only loads relations against published versions. |
| 787 | 787 | * |
| 788 | - * @param mixed $destinationContentId Destination Content ID |
|
| 788 | + * @param integer $destinationContentId Destination Content ID |
|
| 789 | 789 | * @param int|null $type {@see \eZ\Publish\API\Repository\Values\Content\Relation::COMMON, |
| 790 | 790 | * \eZ\Publish\API\Repository\Values\Content\Relation::EMBED, |
| 791 | 791 | * \eZ\Publish\API\Repository\Values\Content\Relation::LINK, |
@@ -50,7 +50,7 @@ |
||
| 50 | 50 | * @see \eZ\Publish\API\Repository\PermissionResolver::canUser() |
| 51 | 51 | * |
| 52 | 52 | * @param TokenInterface $token A TokenInterface instance |
| 53 | - * @param object $object The object to secure |
|
| 53 | + * @param \stdClass $object The object to secure |
|
| 54 | 54 | * @param array $attributes An array of attributes associated with the method being invoked |
| 55 | 55 | * |
| 56 | 56 | * @return int either ACCESS_GRANTED, ACCESS_ABSTAIN, or ACCESS_DENIED |
@@ -236,7 +236,6 @@ discard block |
||
| 236 | 236 | * |
| 237 | 237 | * @param \eZ\Publish\API\Repository\Values\ValueObject $expectedValues |
| 238 | 238 | * @param \eZ\Publish\API\Repository\Values\ValueObject $actualObject |
| 239 | - * @param array $propertyNames |
|
| 240 | 239 | */ |
| 241 | 240 | protected function assertStructPropertiesCorrect(ValueObject $expectedValues, ValueObject $actualObject, array $additionalProperties = []) |
| 242 | 241 | { |
@@ -297,6 +296,7 @@ discard block |
||
| 297 | 296 | |
| 298 | 297 | /** |
| 299 | 298 | * Create a user in editor user group. |
| 299 | + * @param string $email |
|
| 300 | 300 | */ |
| 301 | 301 | protected function createUserVersion1(string $login = 'user', ?string $email = null): User |
| 302 | 302 | { |
@@ -525,7 +525,7 @@ discard block |
||
| 525 | 525 | /** |
| 526 | 526 | * Create role of a given name with the given policies described by an array. |
| 527 | 527 | * |
| 528 | - * @param $roleName |
|
| 528 | + * @param string $roleName |
|
| 529 | 529 | * @param array $policiesData [['module' => 'content', 'function' => 'read', 'limitations' => []] |
| 530 | 530 | * |
| 531 | 531 | * @return \eZ\Publish\API\Repository\Values\User\Role |
@@ -69,7 +69,7 @@ discard block |
||
| 69 | 69 | /** |
| 70 | 70 | * Create a Connection Handler with corresponding Doctrine connection from DSN. |
| 71 | 71 | * |
| 72 | - * @param string|array $dsn |
|
| 72 | + * @param string $dsn |
|
| 73 | 73 | * |
| 74 | 74 | * @return ConnectionHandler |
| 75 | 75 | */ |
@@ -460,8 +460,9 @@ discard block |
||
| 460 | 460 | * |
| 461 | 461 | * This is an optimization and works around the ezcDB implementation. |
| 462 | 462 | * |
| 463 | - * @param string $identifier |
|
| 464 | 463 | * |
| 464 | + * @param string $name |
|
| 465 | + * @param string $alias |
|
| 465 | 466 | * @return string |
| 466 | 467 | */ |
| 467 | 468 | public function alias($name, $alias) |
@@ -496,7 +497,7 @@ discard block |
||
| 496 | 497 | * @param string $table |
| 497 | 498 | * @param string $column |
| 498 | 499 | * |
| 499 | - * @return mixed |
|
| 500 | + * @return string |
|
| 500 | 501 | */ |
| 501 | 502 | public function getAutoIncrementValue($table, $column) |
| 502 | 503 | { |
@@ -63,7 +63,7 @@ |
||
| 63 | 63 | * Inserts a new content object. |
| 64 | 64 | * |
| 65 | 65 | * @param \eZ\Publish\SPI\Persistence\Content\CreateStruct $struct |
| 66 | - * @param mixed $currentVersionNo |
|
| 66 | + * @param integer $currentVersionNo |
|
| 67 | 67 | * |
| 68 | 68 | * @return int ID |
| 69 | 69 | */ |
@@ -20,6 +20,9 @@ discard block |
||
| 20 | 20 | { |
| 21 | 21 | private const URL_CONTENT_TYPE_IDENTIFIER = 'link_ct'; |
| 22 | 22 | |
| 23 | + /** |
|
| 24 | + * @param integer $expectedTotalCount |
|
| 25 | + */ |
|
| 23 | 26 | protected function doTestFindUrls(URLQuery $query, array $expectedUrls, $expectedTotalCount = null, $ignoreOrder = true) |
| 24 | 27 | { |
| 25 | 28 | $repository = $this->getRepository(); |
@@ -37,6 +40,9 @@ discard block |
||
| 37 | 40 | $this->assertSearchResultItems($searchResult, $expectedUrls, $ignoreOrder); |
| 38 | 41 | } |
| 39 | 42 | |
| 43 | + /** |
|
| 44 | + * @param boolean $ignoreOrder |
|
| 45 | + */ |
|
| 40 | 46 | protected function assertSearchResultItems(SearchResult $searchResult, array $expectedUrls, $ignoreOrder) |
| 41 | 47 | { |
| 42 | 48 | $this->assertCount(count($expectedUrls), $searchResult->items); |