Completed
Push — ezp-26146_location_swap_incons... ( ba812b...4a02af )
by
unknown
83:29 queued 58:32
created
Core/Persistence/Legacy/Content/Location/Gateway/DoctrineDatabase.php 1 patch
Doc Comments   +26 added lines, -23 removed lines patch added patch discarded remove patch
@@ -91,7 +91,7 @@  discard block
 block discarded – undo
91 91
      *
92 92
      * @todo optimize
93 93
      *
94
-     * @param mixed $remoteId
94
+     * @param string $remoteId
95 95
      *
96 96
      * @return array
97 97
      */
@@ -246,7 +246,7 @@  discard block
 block discarded – undo
246 246
     /**
247 247
      * Returns data for the first level children of the location identified by given $locationId.
248 248
      *
249
-     * @param mixed $locationId
249
+     * @param integer $locationId
250 250
      *
251 251
      * @return array
252 252
      */
@@ -373,6 +373,9 @@  discard block
 block discarded – undo
373 373
         }
374 374
     }
375 375
 
376
+    /**
377
+     * @param string $pathString
378
+     */
376 379
     private function isHiddenByParent($pathString, array $rows)
377 380
     {
378 381
         $parentNodeIds = explode('/', trim($pathString, '/'));
@@ -573,10 +576,10 @@  discard block
 block discarded – undo
573 576
      * Make the location identified by $locationId1 refer to the Content
574 577
      * referred to by $locationId2 and vice versa.
575 578
      *
576
-     * @param mixed $locationId1
577
-     * @param mixed $locationId2
579
+     * @param integer $locationId1
580
+     * @param integer $locationId2
578 581
      *
579
-     * @return bool
582
+     * @return false|null
580 583
      */
581 584
     public function swap($locationId1, $locationId2)
582 585
     {
@@ -777,7 +780,7 @@  discard block
 block discarded – undo
777 780
      * Create an entry in the node assignment table.
778 781
      *
779 782
      * @param \eZ\Publish\SPI\Persistence\Content\Location\CreateStruct $createStruct
780
-     * @param mixed $parentNodeId
783
+     * @param string $parentNodeId
781 784
      * @param int $type
782 785
      */
783 786
     public function createNodeAssignment(CreateStruct $createStruct, $parentNodeId, $type = self::NODE_ASSIGNMENT_OP_CODE_CREATE_NOP)
@@ -905,8 +908,8 @@  discard block
 block discarded – undo
905 908
      *
906 909
      * Convert existing node assignments into real locations.
907 910
      *
908
-     * @param mixed $contentId
909
-     * @param mixed $versionNo
911
+     * @param integer $contentId
912
+     * @param integer $versionNo
910 913
      */
911 914
     public function createLocationsFromNodeAssignments($contentId, $versionNo)
912 915
     {
@@ -974,8 +977,8 @@  discard block
 block discarded – undo
974 977
     /**
975 978
      * Updates all Locations of content identified with $contentId with $versionNo.
976 979
      *
977
-     * @param mixed $contentId
978
-     * @param mixed $versionNo
980
+     * @param integer $contentId
981
+     * @param integer $versionNo
979 982
      */
980 983
     public function updateLocationsContentVersionNo($contentId, $versionNo)
981 984
     {
@@ -1134,8 +1137,8 @@  discard block
 block discarded – undo
1134 1137
      * the node identified by given $locationId (current main node).
1135 1138
      * Assumes that content has more than one location.
1136 1139
      *
1137
-     * @param mixed $contentId
1138
-     * @param mixed $locationId
1140
+     * @param integer $contentId
1141
+     * @param integer $locationId
1139 1142
      *
1140 1143
      * @return array
1141 1144
      */
@@ -1171,9 +1174,9 @@  discard block
 block discarded – undo
1171 1174
      *
1172 1175
      * The associated content object is left untouched.
1173 1176
      *
1174
-     * @param mixed $locationId
1177
+     * @param integer $locationId
1175 1178
      *
1176
-     * @return bool
1179
+     * @return boolean|null
1177 1180
      */
1178 1181
     public function trashLocation($locationId)
1179 1182
     {
@@ -1202,8 +1205,8 @@  discard block
 block discarded – undo
1202 1205
      * at the old position. If this is not possible ( because the old location
1203 1206
      * does not exist any more) and exception is thrown.
1204 1207
      *
1205
-     * @param mixed $locationId
1206
-     * @param mixed|null $newParentId
1208
+     * @param integer $locationId
1209
+     * @param integer $newParentId
1207 1210
      *
1208 1211
      * @return \eZ\Publish\SPI\Persistence\Content\Location
1209 1212
      */
@@ -1377,10 +1380,10 @@  discard block
 block discarded – undo
1377 1380
     /**
1378 1381
      * Set section on all content objects in the subtree.
1379 1382
      *
1380
-     * @param mixed $pathString
1381
-     * @param mixed $sectionId
1383
+     * @param string $pathString
1384
+     * @param integer $sectionId
1382 1385
      *
1383
-     * @return bool
1386
+     * @return boolean|null
1384 1387
      */
1385 1388
     public function setSectionForSubtree($pathString, $sectionId)
1386 1389
     {
@@ -1446,10 +1449,10 @@  discard block
 block discarded – undo
1446 1449
      * Updates ezcontentobject_tree table for the given $contentId and eznode_assignment table for the given
1447 1450
      * $contentId, $parentLocationId and $versionNo
1448 1451
      *
1449
-     * @param mixed $contentId
1450
-     * @param mixed $locationId
1451
-     * @param mixed $versionNo version number, needed to update eznode_assignment table
1452
-     * @param mixed $parentLocationId parent location of location identified by $locationId, needed to update
1452
+     * @param integer $contentId
1453
+     * @param integer $locationId
1454
+     * @param integer $versionNo version number, needed to update eznode_assignment table
1455
+     * @param integer $parentLocationId parent location of location identified by $locationId, needed to update
1453 1456
      *        eznode_assignment table
1454 1457
      */
1455 1458
     public function changeMainLocation($contentId, $locationId, $versionNo, $parentLocationId)
Please login to merge, or discard this patch.