| @@ -209,8 +209,8 @@ discard block | ||
| 209 | 209 | * @todo Either move to async/batch or find ways toward optimizing away operations per object. | 
| 210 | 210 | * @todo Optionally retain dates and set creator | 
| 211 | 211 | * | 
| 212 | - * @param mixed $sourceId | |
| 213 | - * @param mixed $destinationParentId | |
| 212 | + * @param integer $sourceId | |
| 213 | + * @param integer $destinationParentId | |
| 214 | 214 | * @param int|null $newOwnerId | 
| 215 | 215 | * | 
| 216 | 216 | * @return Location the newly created Location. | 
| @@ -368,10 +368,10 @@ discard block | ||
| 368 | 368 | * destination, identified by $destinationParentId. Relations do not need | 
| 369 | 369 | * to be updated, since they refer to Content. URLs are not touched. | 
| 370 | 370 | * | 
| 371 | - * @param mixed $sourceId | |
| 372 | - * @param mixed $destinationParentId | |
| 371 | + * @param integer $sourceId | |
| 372 | + * @param integer $destinationParentId | |
| 373 | 373 | * | 
| 374 | - * @return bool | |
| 374 | + * @return boolean|null | |
| 375 | 375 | */ | 
| 376 | 376 | public function move($sourceId, $destinationParentId) | 
| 377 | 377 |      { | 
| @@ -414,7 +414,7 @@ discard block | ||
| 414 | 414 | /** | 
| 415 | 415 | * Sets a location to be hidden, and it self + all children to invisible. | 
| 416 | 416 | * | 
| 417 | - * @param mixed $id Location ID | |
| 417 | + * @param integer $id Location ID | |
| 418 | 418 | */ | 
| 419 | 419 | public function hide($id) | 
| 420 | 420 |      { | 
| @@ -427,7 +427,7 @@ discard block | ||
| 427 | 427 | * Sets a location to be unhidden, and self + children to visible unless a parent is hiding the tree. | 
| 428 | 428 | * If not make sure only children down to first hidden node is marked visible. | 
| 429 | 429 | * | 
| 430 | - * @param mixed $id | |
| 430 | + * @param integer $id | |
| 431 | 431 | */ | 
| 432 | 432 | public function unHide($id) | 
| 433 | 433 |      { | 
| @@ -442,10 +442,10 @@ discard block | ||
| 442 | 442 | * Make the location identified by $locationId1 refer to the Content | 
| 443 | 443 | * referred to by $locationId2 and vice versa. | 
| 444 | 444 | * | 
| 445 | - * @param mixed $locationId1 | |
| 446 | - * @param mixed $locationId2 | |
| 445 | + * @param integer $locationId1 | |
| 446 | + * @param integer $locationId2 | |
| 447 | 447 | * | 
| 448 | - * @return bool | |
| 448 | + * @return boolean|null | |
| 449 | 449 | */ | 
| 450 | 450 | public function swap($locationId1, $locationId2) | 
| 451 | 451 |      { | 
| @@ -492,9 +492,9 @@ discard block | ||
| 492 | 492 | * main Location will get the first of its other Locations assigned as the | 
| 493 | 493 | * new main Location. | 
| 494 | 494 | * | 
| 495 | - * @param mixed $locationId | |
| 495 | + * @param integer $locationId | |
| 496 | 496 | * | 
| 497 | - * @return bool | |
| 497 | + * @return boolean|null | |
| 498 | 498 | */ | 
| 499 | 499 | public function removeSubtree($locationId) | 
| 500 | 500 |      { | 
| @@ -505,7 +505,7 @@ discard block | ||
| 505 | 505 | * Set section on all content objects in the subtree. | 
| 506 | 506 | * | 
| 507 | 507 | * @param mixed $locationId | 
| 508 | - * @param mixed $sectionId | |
| 508 | + * @param integer $sectionId | |
| 509 | 509 | */ | 
| 510 | 510 | public function setSectionForSubtree($locationId, $sectionId) | 
| 511 | 511 |      { |