[BUGFIX] Remove wrong break; statement in rootline check
[Packages/TYPO3.CMS.git] / typo3 / sysext / frontend / Classes / Hooks / TreelistCacheUpdateHooks.php
index 4094cad..34a8560 100644 (file)
@@ -67,19 +67,18 @@ class TreelistCacheUpdateHooks
      * @param mixed $recordId The record's uid for update records, a string to look the record's uid up after it has been created
      * @param array $updatedFields Array of changed fields and their new values
      * @param DataHandler $dataHandler DataHandler parent object
-     * @return void
      */
     public function processDatamap_afterDatabaseOperations($status, $table, $recordId, array $updatedFields, DataHandler $dataHandler)
     {
-        if ($table == 'pages' && $this->requiresUpdate($updatedFields)) {
+        if ($table === 'pages' && $this->requiresUpdate($updatedFields)) {
             $affectedPagePid = 0;
             $affectedPageUid = 0;
-            if ($status == 'new') {
+            if ($status === 'new') {
                 // Detect new pages
                 // Resolve the uid
                 $affectedPageUid = $dataHandler->substNEWwithIDs[$recordId];
                 $affectedPagePid = $updatedFields['pid'];
-            } elseif ($status == 'update') {
+            } elseif ($status === 'update') {
                 // Detect updated pages
                 $affectedPageUid = $recordId;
                 // When updating a page the pid is not directly available so we
@@ -101,7 +100,6 @@ class TreelistCacheUpdateHooks
      * @param int $recordId The record's uid
      * @param array $commandValue The commands value, typically an array with more detailed command information
      * @param DataHandler $dataHandler The DataHandler parent object
-     * @return void
      */
     public function processCmdmap_postProcess($command, $table, $recordId, $commandValue, DataHandler $dataHandler)
     {
@@ -138,11 +136,10 @@ class TreelistCacheUpdateHooks
      * @param array $movedRecord The record that moved
      * @param array $updatedFields Array of changed fields
      * @param DataHandler $dataHandler DataHandler parent object
-     * @return void
      */
     public function moveRecord_firstElementPostProcess($table, $recordId, $destinationPid, array $movedRecord, array $updatedFields, DataHandler $dataHandler)
     {
-        if ($table == 'pages' && $this->requiresUpdate($updatedFields)) {
+        if ($table === 'pages' && $this->requiresUpdate($updatedFields)) {
             $affectedPageUid = $recordId;
             $affectedPageOldPid = $movedRecord['pid'];
             $affectedPageNewPid = $updatedFields['pid'];
@@ -165,11 +162,10 @@ class TreelistCacheUpdateHooks
      * @param array $movedRecord The record that moved
      * @param array $updatedFields Array of changed fields
      * @param DataHandler $dataHandler DataHandler parent object
-     * @return void
      */
     public function moveRecord_afterAnotherElementPostProcess($table, $recordId, $destinationPid, $originalDestinationPid, array $movedRecord, array $updatedFields, DataHandler $dataHandler)
     {
-        if ($table == 'pages' && $this->requiresUpdate($updatedFields)) {
+        if ($table === 'pages' && $this->requiresUpdate($updatedFields)) {
             $affectedPageUid = $recordId;
             $affectedPageOldPid = $movedRecord['pid'];
             $affectedPageNewPid = $updatedFields['pid'];
@@ -192,7 +188,7 @@ class TreelistCacheUpdateHooks
         $requiresUpdate = false;
         $updatedFieldNames = array_keys($updatedFields);
         foreach ($updatedFieldNames as $updatedFieldName) {
-            if (in_array($updatedFieldName, $this->updateRequiringFields)) {
+            if (in_array($updatedFieldName, $this->updateRequiringFields, true)) {
                 $requiresUpdate = true;
                 break;
             }
@@ -201,13 +197,12 @@ class TreelistCacheUpdateHooks
     }
 
     /**
-     * Calls the cache maintainance functions according to the determined actions
+     * Calls the cache maintenance functions according to the determined actions
      *
      * @param int $affectedPage uid of the affected page
      * @param int $affectedParentPage parent uid of the affected page
      * @param array $updatedFields Array of updated fields and their new values
      * @param array $actions Array of actions to carry out
-     * @return void
      */
     protected function processClearCacheActions($affectedPage, $affectedParentPage, $updatedFields, array $actions)
     {
@@ -230,7 +225,7 @@ class TreelistCacheUpdateHooks
         // From time to time clean the cache from expired entries
         // (theoretically every 1000 calls)
         $randomNumber = rand(1, 1000);
-        if ($randomNumber == 500) {
+        if ($randomNumber === 500) {
             $this->removeExpiredCacheEntries();
         }
     }
@@ -240,7 +235,6 @@ class TreelistCacheUpdateHooks
      * gets called after creating a new page and after moving a page
      *
      * @param int $affectedParentPage Parent page id of the changed page, the page to start clearing from
-     * @return void
      */
     protected function clearCacheForAllParents($affectedParentPage)
     {
@@ -271,7 +265,6 @@ class TreelistCacheUpdateHooks
      * in the treelist
      *
      * @param int $affectedPage ID of the changed page
-     * @return void
      */
     protected function clearCacheWhereUidInTreelist($affectedPage)
     {
@@ -280,7 +273,7 @@ class TreelistCacheUpdateHooks
         $queryBuilder
             ->delete('cache_treelist')
             ->where(
-                $queryBuilder->expr()->inSet('treelist', (int)$affectedPage)
+                $queryBuilder->expr()->inSet('treelist', $queryBuilder->quote($affectedPage))
             )
             ->execute();
     }
@@ -291,7 +284,6 @@ class TreelistCacheUpdateHooks
      *
      * @param int $affectedPage Uid of the changed page
      * @param int $expirationTime
-     * @return void
      */
     protected function setCacheExpiration($affectedPage, $expirationTime)
     {
@@ -300,7 +292,7 @@ class TreelistCacheUpdateHooks
         $queryBuilder
             ->update('cache_treelist')
             ->where(
-                $queryBuilder->expr()->inSet('treelist', (int)$affectedPage)
+                $queryBuilder->expr()->inSet('treelist', $queryBuilder->quote($affectedPage))
             )
             ->set('expires', $expirationTime)
             ->execute();
@@ -308,8 +300,6 @@ class TreelistCacheUpdateHooks
 
     /**
      * Removes all expired treelist cache entries
-     *
-     * @return void
      */
     protected function removeExpiredCacheEntries()
     {
@@ -337,10 +327,10 @@ class TreelistCacheUpdateHooks
     protected function determineClearCacheActions($status, $updatedFields)
     {
         $actions = [];
-        if ($status == 'new') {
+        if ($status === 'new') {
             // New page
             $actions['allParents'] = true;
-        } elseif ($status == 'update') {
+        } elseif ($status === 'update') {
             $updatedFieldNames = array_keys($updatedFields);
             foreach ($updatedFieldNames as $updatedFieldName) {
                 switch ($updatedFieldName) {
@@ -375,7 +365,7 @@ class TreelistCacheUpdateHooks
                         }
                         break;
                     default:
-                        if (in_array($updatedFieldName, $this->updateRequiringFields)) {
+                        if (in_array($updatedFieldName, $this->updateRequiringFields, true)) {
                             $actions['uidInTreelist'] = true;
                         }
                 }