[BUGFIX] Database query error for non-workspaces tables
[Packages/TYPO3.CMS.git] / typo3 / sysext / frontend / Classes / Page / PageRepository.php
index 63c302c..6c0c62d 100644 (file)
@@ -15,7 +15,7 @@ namespace TYPO3\CMS\Frontend\Page;
  *
  *  The GNU General Public License can be found at
  *  http://www.gnu.org/copyleft/gpl.html.
  *
  *  The GNU General Public License can be found at
  *  http://www.gnu.org/copyleft/gpl.html.
- *  A copy is found in the textfile GPL.txt and important notices to the license
+ *  A copy is found in the text file GPL.txt and important notices to the license
  *  from the author is found in LICENSE.txt distributed with these scripts.
  *
  *
  *  from the author is found in LICENSE.txt distributed with these scripts.
  *
  *
@@ -28,11 +28,14 @@ namespace TYPO3\CMS\Frontend\Page;
  ***************************************************************/
 
 use TYPO3\CMS\Core\Utility\GeneralUtility;
  ***************************************************************/
 
 use TYPO3\CMS\Core\Utility\GeneralUtility;
+use TYPO3\CMS\Core\Versioning\VersionState;
 
 /**
  * Page functions, a lot of sql/pages-related functions
 
 /**
  * Page functions, a lot of sql/pages-related functions
- * Mainly used in the frontend but also in some cases in the backend.
- * It's important to set the right $where_hid_del in the object so that the functions operate properly
+ *
+ * Mainly used in the frontend but also in some cases in the backend. It's
+ * important to set the right $where_hid_del in the object so that the
+ * functions operate properly
  *
  * @author Kasper Skårhøj <kasperYYYY@typo3.com>
  * @see \TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController::fetch_the_id()
  *
  * @author Kasper Skårhøj <kasperYYYY@typo3.com>
  * @see \TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController::fetch_the_id()
@@ -40,70 +43,115 @@ use TYPO3\CMS\Core\Utility\GeneralUtility;
 class PageRepository {
 
        /**
 class PageRepository {
 
        /**
+        * @var array
         * @todo Define visibility
         */
        public $urltypes = array('', 'http://', 'ftp://', 'mailto:', 'https://');
 
         * @todo Define visibility
         */
        public $urltypes = array('', 'http://', 'ftp://', 'mailto:', 'https://');
 
-       // This is not the final clauses. There will normally be conditions for the hidden,starttime and endtime fields as well. You MUST initialize the object by the init() function
        /**
        /**
+        * This is not the final clauses. There will normally be conditions for the
+        * hidden, starttime and endtime fields as well. You MUST initialize the object
+        * by the init() function
+        *
+        * @var string
         * @todo Define visibility
         */
        public $where_hid_del = ' AND pages.deleted=0';
 
         * @todo Define visibility
         */
        public $where_hid_del = ' AND pages.deleted=0';
 
-       // Clause for fe_group access
        /**
        /**
+        * Clause for fe_group access
+        *
+        * @var string
         * @todo Define visibility
         */
        public $where_groupAccess = '';
 
        /**
         * @todo Define visibility
         */
        public $where_groupAccess = '';
 
        /**
+        * @var int
         * @todo Define visibility
         */
        public $sys_language_uid = 0;
 
         * @todo Define visibility
         */
        public $sys_language_uid = 0;
 
-       // Versioning preview related:
-       // If TRUE, versioning preview of other record versions is allowed. THIS MUST ONLY BE SET IF the page is not cached and truely previewed by a backend user!!!
+       // Versioning preview related
+
        /**
        /**
+        * If TRUE, versioning preview of other record versions is allowed. THIS MUST
+        * ONLY BE SET IF the page is not cached and truely previewed by a backend
+        * user!!!
+        *
+        * @var bool
         * @todo Define visibility
         */
        public $versioningPreview = FALSE;
 
         * @todo Define visibility
         */
        public $versioningPreview = FALSE;
 
-       // Workspace ID for preview
        /**
        /**
+        * Workspace ID for preview
+        *
+        * @var int
         * @todo Define visibility
         */
        public $versioningWorkspaceId = 0;
 
        /**
         * @todo Define visibility
         */
        public $versioningWorkspaceId = 0;
 
        /**
+        * @var array
         * @todo Define visibility
         */
        public $workspaceCache = array();
 
         * @todo Define visibility
         */
        public $workspaceCache = array();
 
-       // Internal, dynamic:
-       // Error string set by getRootLine()
+       // Internal, dynamic
+
        /**
        /**
+        * Error string set by getRootLine()
+        *
+        * @var string
         * @todo Define visibility
         */
        public $error_getRootLine = '';
 
         * @todo Define visibility
         */
        public $error_getRootLine = '';
 
-       // Error uid set by getRootLine()
        /**
        /**
+        * Error uid set by getRootLine()
+        *
+        * @var int
         * @todo Define visibility
         */
        public $error_getRootLine_failPid = 0;
 
        // Internal caching
         * @todo Define visibility
         */
        public $error_getRootLine_failPid = 0;
 
        // Internal caching
+
+       /**
+        * @var array
+        */
        protected $cache_getRootLine = array();
 
        protected $cache_getRootLine = array();
 
+       /**
+        * @var array
+        */
        protected $cache_getPage = array();
 
        protected $cache_getPage = array();
 
+       /**
+        * @var array
+        */
        protected $cache_getPage_noCheck = array();
 
        protected $cache_getPage_noCheck = array();
 
+       /**
+        * @var array
+        */
        protected $cache_getPageIdFromAlias = array();
 
        protected $cache_getPageIdFromAlias = array();
 
+       /**
+        * @var array
+        */
        protected $cache_getMountPointInfo = array();
 
        /**
        protected $cache_getMountPointInfo = array();
 
        /**
+        * @var array
+        */
+       protected $tableNamesAllowedOnRootLevel = array(
+               'sys_file_metadata',
+               'sys_category',
+       );
+
+       /**
         * Named constants for "magic numbers" of the field doktype
         */
        const DOKTYPE_DEFAULT = 1;
         * Named constants for "magic numbers" of the field doktype
         */
        const DOKTYPE_DEFAULT = 1;
@@ -114,6 +162,7 @@ class PageRepository {
        const DOKTYPE_SPACER = 199;
        const DOKTYPE_SYSFOLDER = 254;
        const DOKTYPE_RECYCLER = 255;
        const DOKTYPE_SPACER = 199;
        const DOKTYPE_SYSFOLDER = 254;
        const DOKTYPE_RECYCLER = 255;
+
        /**
         * Named constants for "magic numbers" of the field shortcut_mode
         */
        /**
         * Named constants for "magic numbers" of the field shortcut_mode
         */
@@ -121,9 +170,12 @@ class PageRepository {
        const SHORTCUT_MODE_FIRST_SUBPAGE = 1;
        const SHORTCUT_MODE_RANDOM_SUBPAGE = 2;
        const SHORTCUT_MODE_PARENT_PAGE = 3;
        const SHORTCUT_MODE_FIRST_SUBPAGE = 1;
        const SHORTCUT_MODE_RANDOM_SUBPAGE = 2;
        const SHORTCUT_MODE_PARENT_PAGE = 3;
+
        /**
         * init() MUST be run directly after creating a new template-object
        /**
         * init() MUST be run directly after creating a new template-object
-        * This sets the internal variable $this->where_hid_del to the correct where clause for page records taking deleted/hidden/starttime/endtime/t3ver_state into account
+        * This sets the internal variable $this->where_hid_del to the correct where
+        * clause for page records taking deleted/hidden/starttime/endtime/t3ver_state
+        * into account
         *
         * @param boolean $show_hidden If $show_hidden is TRUE, the hidden-field is ignored!! Normally this should be FALSE. Is used for previewing.
         * @return void
         *
         * @param boolean $show_hidden If $show_hidden is TRUE, the hidden-field is ignored!! Normally this should be FALSE. Is used for previewing.
         * @return void
@@ -137,23 +189,29 @@ class PageRepository {
                        $this->where_hid_del .= 'AND pages.hidden=0 ';
                }
                $this->where_hid_del .= 'AND pages.starttime<=' . $GLOBALS['SIM_ACCESS_TIME'] . ' AND (pages.endtime=0 OR pages.endtime>' . $GLOBALS['SIM_ACCESS_TIME'] . ') ';
                        $this->where_hid_del .= 'AND pages.hidden=0 ';
                }
                $this->where_hid_del .= 'AND pages.starttime<=' . $GLOBALS['SIM_ACCESS_TIME'] . ' AND (pages.endtime=0 OR pages.endtime>' . $GLOBALS['SIM_ACCESS_TIME'] . ') ';
-               // Filter out new/deleted place-holder pages in case we are NOT in a versioning preview (that means we are online!)
+               // Filter out new/deleted place-holder pages in case we are NOT in a
+               // versioning preview (that means we are online!)
                if (!$this->versioningPreview) {
                if (!$this->versioningPreview) {
-                       $this->where_hid_del .= ' AND NOT pages.t3ver_state>0';
+                       $this->where_hid_del .= ' AND NOT pages.t3ver_state>' . new VersionState(VersionState::DEFAULT_STATE);
                } else {
                } else {
-                       // For version previewing, make sure that enable-fields are not de-selecting hidden pages - we need versionOL() to unset them only if the overlay record instructs us to.
+                       // For version previewing, make sure that enable-fields are not
+                       // de-selecting hidden pages - we need versionOL() to unset them only
+                       // if the overlay record instructs us to.
                        // Copy where_hid_del to other variable (used in relation to versionOL())
                        $this->versioningPreview_where_hid_del = $this->where_hid_del;
                        // Clear where_hid_del
                        $this->where_hid_del = ' AND pages.deleted=0 ';
                        // Copy where_hid_del to other variable (used in relation to versionOL())
                        $this->versioningPreview_where_hid_del = $this->where_hid_del;
                        // Clear where_hid_del
                        $this->where_hid_del = ' AND pages.deleted=0 ';
+                       // Restrict to live and current workspaces
+                       $this->where_hid_del .= ' AND (pages.t3ver_wsid=0 OR pages.t3ver_wsid=' . (int)$this->versioningWorkspaceId . ')';
                }
        }
 
                }
        }
 
-       /*******************************************
+       /**************************
         *
         * Selecting page records
         *
         *
         * Selecting page records
         *
-        ******************************************/
+        **************************/
+
        /**
         * Returns the $row for the page with uid = $uid (observing ->where_hid_del)
         * Any pages_language_overlay will be applied before the result is returned.
        /**
         * Returns the $row for the page with uid = $uid (observing ->where_hid_del)
         * Any pages_language_overlay will be applied before the result is returned.
@@ -161,6 +219,7 @@ class PageRepository {
         *
         * @param integer $uid The page id to look up.
         * @param boolean $disableGroupAccessCheck If set, the check for group access is disabled. VERY rarely used
         *
         * @param integer $uid The page id to look up.
         * @param boolean $disableGroupAccessCheck If set, the check for group access is disabled. VERY rarely used
+        * @throws \UnexpectedValueException
         * @return array The page row with overlayed localized fields. Empty it no page.
         * @see getPage_noCheck()
         * @todo Define visibility
         * @return array The page row with overlayed localized fields. Empty it no page.
         * @see getPage_noCheck()
         * @todo Define visibility
@@ -182,7 +241,7 @@ class PageRepository {
                        return $this->cache_getPage[$uid][$cacheKey];
                }
                $result = array();
                        return $this->cache_getPage[$uid][$cacheKey];
                }
                $result = array();
-               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages', 'uid=' . intval($uid) . $this->where_hid_del . $accessCheck);
+               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages', 'uid=' . (int)$uid . $this->where_hid_del . $accessCheck);
                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                if ($row) {
                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                if ($row) {
@@ -196,7 +255,8 @@ class PageRepository {
        }
 
        /**
        }
 
        /**
-        * Return the $row for the page with uid = $uid WITHOUT checking for ->where_hid_del (start- and endtime or hidden). Only "deleted" is checked!
+        * Return the $row for the page with uid = $uid WITHOUT checking for
+        * ->where_hid_del (start- and endtime or hidden). Only "deleted" is checked!
         *
         * @param integer $uid The page id to look up
         * @return array The page row with overlayed localized fields. Empty array if no page.
         *
         * @param integer $uid The page id to look up
         * @return array The page row with overlayed localized fields. Empty array if no page.
@@ -207,7 +267,7 @@ class PageRepository {
                if ($this->cache_getPage_noCheck[$uid]) {
                        return $this->cache_getPage_noCheck[$uid];
                }
                if ($this->cache_getPage_noCheck[$uid]) {
                        return $this->cache_getPage_noCheck[$uid];
                }
-               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages', 'uid=' . intval($uid) . $this->deleteClause('pages'));
+               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages', 'uid=' . (int)$uid . $this->deleteClause('pages'));
                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                $result = array();
                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                $result = array();
@@ -231,7 +291,7 @@ class PageRepository {
         */
        public function getFirstWebPage($uid) {
                $output = '';
         */
        public function getFirstWebPage($uid) {
                $output = '';
-               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages', 'pid=' . intval($uid) . $this->where_hid_del . $this->where_groupAccess, '', 'sorting', '1');
+               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', 'pages', 'pid=' . (int)$uid . $this->where_hid_del . $this->where_groupAccess, '', 'sorting', '1');
                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                if ($row) {
                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                if ($row) {
@@ -273,6 +333,7 @@ class PageRepository {
         *
         * @param mixed $pageInput If $pageInput is an integer, it's the pid of the pageOverlay record and thus the page overlay record is returned. If $pageInput is an array, it's a page-record and based on this page record the language record is found and OVERLAYED before the page record is returned.
         * @param integer $lUid Language UID if you want to set an alternative value to $this->sys_language_uid which is default. Should be >=0
         *
         * @param mixed $pageInput If $pageInput is an integer, it's the pid of the pageOverlay record and thus the page overlay record is returned. If $pageInput is an array, it's a page-record and based on this page record the language record is found and OVERLAYED before the page record is returned.
         * @param integer $lUid Language UID if you want to set an alternative value to $this->sys_language_uid which is default. Should be >=0
+        * @throws \UnexpectedValueException
         * @return array Page row which is overlayed with language_overlay record (or the overlay record alone)
         * @todo Define visibility
         */
         * @return array Page row which is overlayed with language_overlay record (or the overlay record alone)
         * @todo Define visibility
         */
@@ -305,11 +366,14 @@ class PageRepository {
                        }
                        if (count($fieldArr)) {
                                // NOTE to enabledFields('pages_language_overlay'):
                        }
                        if (count($fieldArr)) {
                                // NOTE to enabledFields('pages_language_overlay'):
-                               // Currently the showHiddenRecords of TSFE set will allow pages_language_overlay records to be selected as they are child-records of a page.
-                               // However you may argue that the showHiddenField flag should determine this. But that's not how it's done right now.
+                               // Currently the showHiddenRecords of TSFE set will allow
+                               // pages_language_overlay records to be selected as they are
+                               // child-records of a page.
+                               // However you may argue that the showHiddenField flag should
+                               // determine this. But that's not how it's done right now.
                                // Selecting overlay record:
                                // Selecting overlay record:
-                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery(implode(',', $fieldArr), 'pages_language_overlay', 'pid=' . intval($page_id) . '
-                                                               AND sys_language_uid=' . intval($lUid) . $this->enableFields('pages_language_overlay'), '', '', '1');
+                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery(implode(',', $fieldArr), 'pages_language_overlay', 'pid=' . (int)$page_id . '
+                                                               AND sys_language_uid=' . (int)$lUid . $this->enableFields('pages_language_overlay'), '', '', '1');
                                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                $this->versionOL('pages_language_overlay', $row);
                                $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                $this->versionOL('pages_language_overlay', $row);
@@ -325,8 +389,17 @@ class PageRepository {
                }
                // Create output:
                if (is_array($pageInput)) {
                }
                // Create output:
                if (is_array($pageInput)) {
-                       // If the input was an array, simply overlay the newfound array and return...
-                       return is_array($row) ? array_merge($pageInput, $row) : $pageInput;
+                       if (is_array($row)) {
+                               // Overwrite the original field with the overlay
+                               foreach ($row as $fieldName => $fieldValue) {
+                                       if ($fieldName !== 'uid' && $fieldName !== 'pid') {
+                                               if ($this->shouldFieldBeOverlaid('pages_language_overlay', $fieldName, $fieldValue)) {
+                                                       $pageInput[$fieldName] = $fieldValue;
+                                               }
+                                       }
+                               }
+                       }
+                       return $pageInput;
                } else {
                        // Always an array in return
                        return is_array($row) ? $row : array();
                } else {
                        // Always an array in return
                        return is_array($row) ? $row : array();
@@ -334,13 +407,15 @@ class PageRepository {
        }
 
        /**
        }
 
        /**
-        * Creates language-overlay for records in general (where translation is found in records from the same table)
+        * Creates language-overlay for records in general (where translation is found
+        * in records from the same table)
         *
         * @param string $table Table name
         * @param array $row Record to overlay. Must containt uid, pid and $table]['ctrl']['languageField']
         * @param integer $sys_language_content Pointer to the sys_language uid for content on the site.
         *
         * @param string $table Table name
         * @param array $row Record to overlay. Must containt uid, pid and $table]['ctrl']['languageField']
         * @param integer $sys_language_content Pointer to the sys_language uid for content on the site.
-        * @param string $OLmode Overlay mode. If "hideNonTranslated" then records without translation will not be returned un-translated but unset (and return value is FALSE)
-        * @return mixed Returns the input record, possibly overlaid with a translation. But if $OLmode is "hideNonTranslated" then it will return FALSE if no translation is found.
+        * @param string $OLmode Overlay mode. If "hideNonTranslated" then records without translation will not be returned  un-translated but unset (and return value is FALSE)
+        * @throws \UnexpectedValueException
+        * @return mixed Returns the input record, possibly overlaid with a translation.  But if $OLmode is "hideNonTranslated" then it will return FALSE if no translation is found.
         * @todo Define visibility
         */
        public function getRecordOverlay($table, $row, $sys_language_content, $OLmode = '') {
         * @todo Define visibility
         */
        public function getRecordOverlay($table, $row, $sys_language_content, $OLmode = '') {
@@ -353,19 +428,20 @@ class PageRepository {
                                $hookObject->getRecordOverlay_preProcess($table, $row, $sys_language_content, $OLmode, $this);
                        }
                }
                                $hookObject->getRecordOverlay_preProcess($table, $row, $sys_language_content, $OLmode, $this);
                        }
                }
-               if ($row['uid'] > 0 && $row['pid'] > 0) {
+               if ($row['uid'] > 0 && ($row['pid'] > 0 || in_array($table, $this->tableNamesAllowedOnRootLevel))) {
                        if ($GLOBALS['TCA'][$table] && $GLOBALS['TCA'][$table]['ctrl']['languageField'] && $GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField']) {
                                if (!$GLOBALS['TCA'][$table]['ctrl']['transOrigPointerTable']) {
                        if ($GLOBALS['TCA'][$table] && $GLOBALS['TCA'][$table]['ctrl']['languageField'] && $GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField']) {
                                if (!$GLOBALS['TCA'][$table]['ctrl']['transOrigPointerTable']) {
-                                       // Will not be able to work with other tables (Just didn't implement it yet; Requires a scan
-                                       // over all tables [ctrl] part for first FIND the table that carries localization information for
-                                       // this table (which could even be more than a single table) and then use that. Could be
-                                       // implemented, but obviously takes a little more....)
-                                       // Will try to overlay a record only if the sys_language_content value is larger than zero.
+                                       // Will not be able to work with other tables (Just didn't implement it yet;
+                                       // Requires a scan over all tables [ctrl] part for first FIND the table that
+                                       // carries localization information for this table (which could even be more
+                                       // than a single table) and then use that. Could be implemented, but obviously
+                                       // takes a little more....) Will try to overlay a record only if the
+                                       // sys_language_content value is larger than zero.
                                        if ($sys_language_content > 0) {
                                                // Must be default language or [All], otherwise no overlaying:
                                                if ($row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] <= 0) {
                                                        // Select overlay record:
                                        if ($sys_language_content > 0) {
                                                // Must be default language or [All], otherwise no overlaying:
                                                if ($row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] <= 0) {
                                                        // Select overlay record:
-                                                       $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', $table, 'pid=' . intval($row['pid']) . ' AND ' . $GLOBALS['TCA'][$table]['ctrl']['languageField'] . '=' . intval($sys_language_content) . ' AND ' . $GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField'] . '=' . intval($row['uid']) . $this->enableFields($table), '', '', '1');
+                                                       $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', $table, 'pid=' . (int)$row['pid'] . ' AND ' . $GLOBALS['TCA'][$table]['ctrl']['languageField'] . '=' . (int)$sys_language_content . ' AND ' . $GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField'] . '=' . (int)$row['uid'] . $this->enableFields($table), '', '', '1');
                                                        $olrow = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                                        $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                                        $this->versionOL($table, $olrow);
                                                        $olrow = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                                        $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                                        $this->versionOL($table, $olrow);
@@ -379,10 +455,7 @@ class PageRepository {
                                                                }
                                                                foreach ($row as $fN => $fV) {
                                                                        if ($fN != 'uid' && $fN != 'pid' && isset($olrow[$fN])) {
                                                                }
                                                                foreach ($row as $fN => $fV) {
                                                                        if ($fN != 'uid' && $fN != 'pid' && isset($olrow[$fN])) {
-                                                                               if (
-                                                                                       $GLOBALS['TCA'][$table]['columns'][$fN]['l10n_mode'] != 'exclude'
-                                                                                       && ($GLOBALS['TCA'][$table]['columns'][$fN]['l10n_mode'] != 'mergeIfNotBlank' || strcmp(trim($olrow[$fN]), ''))
-                                                                               ) {
+                                                                               if ($this->shouldFieldBeOverlaid($table, $fN, $olrow[$fN])) {
                                                                                        $row[$fN] = $olrow[$fN];
                                                                                }
                                                                        } elseif ($fN == 'uid') {
                                                                                        $row[$fN] = $olrow[$fN];
                                                                                }
                                                                        } elseif ($fN == 'uid') {
@@ -390,15 +463,16 @@ class PageRepository {
                                                                        }
                                                                }
                                                        } elseif ($OLmode === 'hideNonTranslated' && $row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] == 0) {
                                                                        }
                                                                }
                                                        } elseif ($OLmode === 'hideNonTranslated' && $row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] == 0) {
-                                                               // Unset, if non-translated records should be hidden. ONLY done if the source record
-                                                               // really is default language and not [All] in which case it is allowed.
+                                                               // Unset, if non-translated records should be hidden. ONLY done if the source
+                                                               // record really is default language and not [All] in which case it is allowed.
                                                                unset($row);
                                                        }
                                                } elseif ($sys_language_content != $row[$GLOBALS['TCA'][$table]['ctrl']['languageField']]) {
                                                        unset($row);
                                                }
                                        } else {
                                                                unset($row);
                                                        }
                                                } elseif ($sys_language_content != $row[$GLOBALS['TCA'][$table]['ctrl']['languageField']]) {
                                                        unset($row);
                                                }
                                        } else {
-                                               // When default language is displayed, we never want to return a record carrying another language!
+                                               // When default language is displayed, we never want to return a record carrying
+                                               // another language!
                                                if ($row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] > 0) {
                                                        unset($row);
                                                }
                                                if ($row[$GLOBALS['TCA'][$table]['ctrl']['languageField']] > 0) {
                                                        unset($row);
                                                }
@@ -418,15 +492,19 @@ class PageRepository {
                return $row;
        }
 
                return $row;
        }
 
-       /*******************************************
+       /************************************************
         *
         * Page related: Menu, Domain record, Root line
         *
         *
         * Page related: Menu, Domain record, Root line
         *
-        ******************************************/
+        ************************************************/
+
        /**
        /**
-        * Returns an array with pagerows for subpages with pid=$uid (which is pid here!). This is used for menus.
-        * If there are mount points in overlay mode the _MP_PARAM field is set to the corret MPvar.
-        * If the $uid being input does in itself require MPvars to define a correct rootline these must be handled externally to this function.
+        * Returns an array with pagerows for subpages with pid=$uid (which is pid
+        * here!). This is used for menus. If there are mount points in overlay mode
+        * the _MP_PARAM field is set to the corret MPvar.
+        *
+        * If the $uid being input does in itself require MPvars to define a correct
+        * rootline these must be handled externally to this function.
         *
         * @param integer $uid The page id for which to fetch subpages (PID)
         * @param string $fields List of fields to select. Default is "*" = all
         *
         * @param integer $uid The page id for which to fetch subpages (PID)
         * @param string $fields List of fields to select. Default is "*" = all
@@ -440,17 +518,19 @@ class PageRepository {
         */
        public function getMenu($uid, $fields = '*', $sortField = 'sorting', $addWhere = '', $checkShortcuts = TRUE) {
                $output = array();
         */
        public function getMenu($uid, $fields = '*', $sortField = 'sorting', $addWhere = '', $checkShortcuts = TRUE) {
                $output = array();
-               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery($fields, 'pages', 'pid=' . intval($uid) . $this->where_hid_del . $this->where_groupAccess . ' ' . $addWhere, '', $sortField);
+               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery($fields, 'pages', 'pid=' . (int)$uid . $this->where_hid_del . $this->where_groupAccess . ' ' . $addWhere, '', $sortField);
                while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
                        $this->versionOL('pages', $row, TRUE);
                        if (is_array($row)) {
                                // Keep mount point:
                                $origUid = $row['uid'];
                while ($row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res)) {
                        $this->versionOL('pages', $row, TRUE);
                        if (is_array($row)) {
                                // Keep mount point:
                                $origUid = $row['uid'];
-                               // $row MUST have "uid", "pid", "doktype", "mount_pid", "mount_pid_ol" fields in it
+                               // $row MUST have "uid", "pid", "doktype", "mount_pid", "mount_pid_ol" fields
+                               // in it
                                $mount_info = $this->getMountPointInfo($origUid, $row);
                                // There is a valid mount point.
                                if (is_array($mount_info) && $mount_info['overlay']) {
                                $mount_info = $this->getMountPointInfo($origUid, $row);
                                // There is a valid mount point.
                                if (is_array($mount_info) && $mount_info['overlay']) {
-                                       // Using "getPage" is OK since we need the check for enableFields AND for type 2 of mount pids we DO require a doktype < 200!
+                                       // Using "getPage" is OK since we need the check for enableFields AND for type 2
+                                       // of mount pids we DO require a doktype < 200!
                                        $mp_row = $this->getPage($mount_info['mount_pid']);
                                        if (count($mp_row)) {
                                                $row = $mp_row;
                                        $mp_row = $this->getPage($mount_info['mount_pid']);
                                        if (count($mp_row)) {
                                                $row = $mp_row;
@@ -464,12 +544,13 @@ class PageRepository {
                                        if ($row['shortcut_mode'] == self::SHORTCUT_MODE_NONE) {
                                                // No shortcut_mode set, so target is directly set in $row['shortcut']
                                                $searchField = 'uid';
                                        if ($row['shortcut_mode'] == self::SHORTCUT_MODE_NONE) {
                                                // No shortcut_mode set, so target is directly set in $row['shortcut']
                                                $searchField = 'uid';
-                                               $searchUid = intval($row['shortcut']);
+                                               $searchUid = (int)$row['shortcut'];
                                        } elseif ($row['shortcut_mode'] == self::SHORTCUT_MODE_FIRST_SUBPAGE || $row['shortcut_mode'] == self::SHORTCUT_MODE_RANDOM_SUBPAGE) {
                                                // Check subpages - first subpage or random subpage
                                                $searchField = 'pid';
                                        } elseif ($row['shortcut_mode'] == self::SHORTCUT_MODE_FIRST_SUBPAGE || $row['shortcut_mode'] == self::SHORTCUT_MODE_RANDOM_SUBPAGE) {
                                                // Check subpages - first subpage or random subpage
                                                $searchField = 'pid';
-                                               // If a shortcut mode is set and no valid page is given to select subpags from use the actual page.
-                                               $searchUid = intval($row['shortcut']) ? intval($row['shortcut']) : $row['uid'];
+                                               // If a shortcut mode is set and no valid page is given to select subpags
+                                               // from use the actual page.
+                                               $searchUid = (int)$row['shortcut'] ?: $row['uid'];
                                        } elseif ($row['shortcut_mode'] == self::SHORTCUT_MODE_PARENT_PAGE) {
                                                // Shortcut to parent page
                                                $searchField = 'uid';
                                        } elseif ($row['shortcut_mode'] == self::SHORTCUT_MODE_PARENT_PAGE) {
                                                // Shortcut to parent page
                                                $searchField = 'uid';
@@ -495,7 +576,8 @@ class PageRepository {
 
        /**
         * Will find the page carrying the domain record matching the input domain.
 
        /**
         * Will find the page carrying the domain record matching the input domain.
-        * Might exit after sending a redirect-header IF a found domain record instructs to do so.
+        * Might exit after sending a redirect-header IF a found domain record
+        * instructs to do so.
         *
         * @param string $domain Domain name to search for. Eg. "www.typo3.com". Typical the HTTP_HOST value.
         * @param string $path Path for the current script in domain. Eg. "/somedir/subdir". Typ. supplied by \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('SCRIPT_NAME')
         *
         * @param string $domain Domain name to search for. Eg. "www.typo3.com". Typical the HTTP_HOST value.
         * @param string $path Path for the current script in domain. Eg. "/somedir/subdir". Typ. supplied by \TYPO3\CMS\Core\Utility\GeneralUtility::getIndpEnv('SCRIPT_NAME')
@@ -525,11 +607,11 @@ class PageRepository {
                                        $prependStr = ltrim(substr($request_uri, strlen($path)), '/');
                                        $redirectUrl .= '/' . $prependStr;
                                }
                                        $prependStr = ltrim(substr($request_uri, strlen($path)), '/');
                                        $redirectUrl .= '/' . $prependStr;
                                }
-                               $statusCode = intval($row['redirectHttpStatusCode']);
+                               $statusCode = (int)$row['redirectHttpStatusCode'];
                                if ($statusCode && defined('TYPO3\\CMS\\Core\\Utility\\HttpUtility::HTTP_STATUS_' . $statusCode)) {
                                        \TYPO3\CMS\Core\Utility\HttpUtility::redirect($redirectUrl, constant('TYPO3\\CMS\\Core\\Utility\\HttpUtility::HTTP_STATUS_' . $statusCode));
                                } else {
                                if ($statusCode && defined('TYPO3\\CMS\\Core\\Utility\\HttpUtility::HTTP_STATUS_' . $statusCode)) {
                                        \TYPO3\CMS\Core\Utility\HttpUtility::redirect($redirectUrl, constant('TYPO3\\CMS\\Core\\Utility\\HttpUtility::HTTP_STATUS_' . $statusCode));
                                } else {
-                                       \TYPO3\CMS\Core\Utility\HttpUtility::redirect($redirectUrl, 'TYPO3\\CMS\\Core\\Utility\\HttpUtility::HTTP_STATUS_301');
+                                       \TYPO3\CMS\Core\Utility\HttpUtility::redirect($redirectUrl, \TYPO3\CMS\Core\Utility\HttpUtility::HTTP_STATUS_301);
                                }
                                die;
                        } else {
                                }
                                die;
                        } else {
@@ -540,13 +622,21 @@ class PageRepository {
 
        /**
         * Returns array with fields of the pages from here ($uid) and back to the root
 
        /**
         * Returns array with fields of the pages from here ($uid) and back to the root
-        * NOTICE: This function only takes deleted pages into account! So hidden, starttime and endtime restricted pages are included no matter what.
-        * Further: If any "recycler" page is found (doktype=255) then it will also block for the rootline)
-        * If you want more fields in the rootline records than default such can be added by listing them in $GLOBALS['TYPO3_CONF_VARS']['FE']['addRootLineFields']
+        *
+        * NOTICE: This function only takes deleted pages into account! So hidden,
+        * starttime and endtime restricted pages are included no matter what.
+        *
+        * Further: If any "recycler" page is found (doktype=255) then it will also block
+        * for the rootline)
+        *
+        * If you want more fields in the rootline records than default such can be added
+        * by listing them in $GLOBALS['TYPO3_CONF_VARS']['FE']['addRootLineFields']
         *
         * @param integer $uid The page uid for which to seek back to the page tree root.
         * @param string $MP Commalist of MountPoint parameters, eg. "1-2,3-4" etc. Normally this value comes from the GET var, MP
         * @param boolean $ignoreMPerrors If set, some errors related to Mount Points in root line are ignored.
         *
         * @param integer $uid The page uid for which to seek back to the page tree root.
         * @param string $MP Commalist of MountPoint parameters, eg. "1-2,3-4" etc. Normally this value comes from the GET var, MP
         * @param boolean $ignoreMPerrors If set, some errors related to Mount Points in root line are ignored.
+        * @throws \Exception
+        * @throws \RuntimeException
         * @return array Array with page records from the root line as values. The array is ordered with the outer records first and root record in the bottom. The keys are numeric but in reverse order. So if you traverse/sort the array by the numeric keys order you will get the order from root and out. If an error is found (like eternal looping or invalid mountpoint) it will return an empty array.
         * @see \TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController::getPageAndRootline()
         * @todo Define visibility
         * @return array Array with page records from the root line as values. The array is ordered with the outer records first and root record in the bottom. The keys are numeric but in reverse order. So if you traverse/sort the array by the numeric keys order you will get the order from root and out. If an error is found (like eternal looping or invalid mountpoint) it will return an empty array.
         * @see \TYPO3\CMS\Frontend\Controller\TypoScriptFrontendController::getPageAndRootline()
         * @todo Define visibility
@@ -594,7 +684,8 @@ class PageRepository {
        }
 
        /**
        }
 
        /**
-        * Returns the URL type for the input page row IF the doktype is 3 and not disabled.
+        * Returns the URL type for the input page row IF the doktype is 3 and not
+        * disabled.
         *
         * @param array $pagerow The page row to return URL type for
         * @param boolean $disable A flag to simply disable any output from here.
         *
         * @param array $pagerow The page row to return URL type for
         * @param boolean $disable A flag to simply disable any output from here.
@@ -608,7 +699,7 @@ class PageRepository {
                        // If relative path, prefix Site URL:
                        $uI = parse_url($redirectTo);
                        // Relative path assumed now.
                        // If relative path, prefix Site URL:
                        $uI = parse_url($redirectTo);
                        // Relative path assumed now.
-                       if (!$uI['scheme'] && substr($redirectTo, 0, 1) != '/') {
+                       if (!$uI['scheme'] && $redirectTo[0] !== '/') {
                                $redirectTo = GeneralUtility::getIndpEnv('TYPO3_SITE_URL') . $redirectTo;
                        }
                        return $redirectTo;
                                $redirectTo = GeneralUtility::getIndpEnv('TYPO3_SITE_URL') . $redirectTo;
                        }
                        return $redirectTo;
@@ -617,7 +708,9 @@ class PageRepository {
 
        /**
         * Returns MountPoint id for page
 
        /**
         * Returns MountPoint id for page
-        * Does a recursive search if the mounted page should be a mount page itself. It has a run-away break so it can't go into infinite loops.
+        *
+        * Does a recursive search if the mounted page should be a mount page itself. It
+        * has a run-away break so it can't go into infinite loops.
         *
         * @param integer $pageId Page id for which to look for a mount pid. Will be returned only if mount pages are enabled, the correct doktype (7) is set for page and there IS a mount_pid (which has a valid record that is not deleted...)
         * @param array $pageRec Optional page record for the page id. If not supplied it will be looked up by the system. Must contain at least uid,pid,doktype,mount_pid,mount_pid_ol
         *
         * @param integer $pageId Page id for which to look for a mount pid. Will be returned only if mount pages are enabled, the correct doktype (7) is set for page and there IS a mount_pid (which has a valid record that is not deleted...)
         * @param array $pageRec Optional page record for the page id. If not supplied it will be looked up by the system. Must contain at least uid,pid,doktype,mount_pid,mount_pid_ol
@@ -635,10 +728,11 @@ class PageRepository {
                        }
                        // Get pageRec if not supplied:
                        if (!is_array($pageRec)) {
                        }
                        // Get pageRec if not supplied:
                        if (!is_array($pageRec)) {
-                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid,pid,doktype,mount_pid,mount_pid_ol,t3ver_state', 'pages', 'uid=' . intval($pageId) . ' AND pages.deleted=0 AND pages.doktype<>255');
+                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid,pid,doktype,mount_pid,mount_pid_ol,t3ver_state', 'pages', 'uid=' . (int)$pageId . ' AND pages.deleted=0 AND pages.doktype<>255');
                                $pageRec = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                $pageRec = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
-                               // Only look for version overlay if page record is not supplied; This assumes that the input record is overlaid with preview version, if any!
+                               // Only look for version overlay if page record is not supplied; This assumes
+                               // that the input record is overlaid with preview version, if any!
                                $this->versionOL('pages', $pageRec);
                        }
                        // Set first Page uid:
                                $this->versionOL('pages', $pageRec);
                        }
                        // Set first Page uid:
@@ -646,7 +740,7 @@ class PageRepository {
                                $firstPageUid = $pageRec['uid'];
                        }
                        // Look for mount pid value plus other required circumstances:
                                $firstPageUid = $pageRec['uid'];
                        }
                        // Look for mount pid value plus other required circumstances:
-                       $mount_pid = intval($pageRec['mount_pid']);
+                       $mount_pid = (int)$pageRec['mount_pid'];
                        if (is_array($pageRec) && $pageRec['doktype'] == self::DOKTYPE_MOUNTPOINT && $mount_pid > 0 && !in_array($mount_pid, $prevMountPids)) {
                                // Get the mount point record (to verify its general existence):
                                $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid,pid,doktype,mount_pid,mount_pid_ol,t3ver_state', 'pages', 'uid=' . $mount_pid . ' AND pages.deleted=0 AND pages.doktype<>255');
                        if (is_array($pageRec) && $pageRec['doktype'] == self::DOKTYPE_MOUNTPOINT && $mount_pid > 0 && !in_array($mount_pid, $prevMountPids)) {
                                // Get the mount point record (to verify its general existence):
                                $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid,pid,doktype,mount_pid,mount_pid_ol,t3ver_state', 'pages', 'uid=' . $mount_pid . ' AND pages.deleted=0 AND pages.doktype<>255');
@@ -658,7 +752,7 @@ class PageRepository {
                                        $prevMountPids[] = $mount_pid;
                                        $recursiveMountPid = $this->getMountPointInfo($mount_pid, $mountRec, $prevMountPids, $firstPageUid);
                                        // Return mount point information:
                                        $prevMountPids[] = $mount_pid;
                                        $recursiveMountPid = $this->getMountPointInfo($mount_pid, $mountRec, $prevMountPids, $firstPageUid);
                                        // Return mount point information:
-                                       $result = $recursiveMountPid ? $recursiveMountPid : array(
+                                       $result = $recursiveMountPid ?: array(
                                                'mount_pid' => $mount_pid,
                                                'overlay' => $pageRec['mount_pid_ol'],
                                                'MPvar' => $mount_pid . '-' . $firstPageUid,
                                                'mount_pid' => $mount_pid,
                                                'overlay' => $pageRec['mount_pid_ol'],
                                                'MPvar' => $mount_pid . '-' . $firstPageUid,
@@ -675,11 +769,12 @@ class PageRepository {
                return $result;
        }
 
                return $result;
        }
 
-       /*********************************
+       /********************************
         *
         * Selecting records in general
         *
         *
         * Selecting records in general
         *
-        **********************************/
+        ********************************/
+
        /**
         * Checks if a record exists and is accessible.
         * The row is returned if everything's OK.
        /**
         * Checks if a record exists and is accessible.
         * The row is returned if everything's OK.
@@ -691,7 +786,7 @@ class PageRepository {
         * @todo Define visibility
         */
        public function checkRecord($table, $uid, $checkPage = 0) {
         * @todo Define visibility
         */
        public function checkRecord($table, $uid, $checkPage = 0) {
-               $uid = intval($uid);
+               $uid = (int)$uid;
                if (is_array($GLOBALS['TCA'][$table]) && $uid > 0) {
                        $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', $table, 'uid = ' . $uid . $this->enableFields($table));
                        $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                if (is_array($GLOBALS['TCA'][$table]) && $uid > 0) {
                        $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('*', $table, 'uid = ' . $uid . $this->enableFields($table));
                        $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
@@ -700,7 +795,7 @@ class PageRepository {
                                $this->versionOL($table, $row);
                                if (is_array($row)) {
                                        if ($checkPage) {
                                $this->versionOL($table, $row);
                                if (is_array($row)) {
                                        if ($checkPage) {
-                                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid', 'pages', 'uid=' . intval($row['pid']) . $this->enableFields('pages'));
+                                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery('uid', 'pages', 'uid=' . (int)$row['pid'] . $this->enableFields('pages'));
                                                $numRows = $GLOBALS['TYPO3_DB']->sql_num_rows($res);
                                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                                if ($numRows > 0) {
                                                $numRows = $GLOBALS['TYPO3_DB']->sql_num_rows($res);
                                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                                if ($numRows > 0) {
@@ -728,8 +823,9 @@ class PageRepository {
         * @todo Define visibility
         */
        public function getRawRecord($table, $uid, $fields = '*', $noWSOL = FALSE) {
         * @todo Define visibility
         */
        public function getRawRecord($table, $uid, $fields = '*', $noWSOL = FALSE) {
-               $uid = intval($uid);
-               // Excluding pages here so we can ask the function BEFORE TCA gets initialized. Support for this is followed up in deleteClause()...
+               $uid = (int)$uid;
+               // Excluding pages here so we can ask the function BEFORE TCA gets initialized.
+               // Support for this is followed up in deleteClause()...
                if ((is_array($GLOBALS['TCA'][$table]) || $table == 'pages') && $uid > 0) {
                        $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery($fields, $table, 'uid = ' . $uid . $this->deleteClause($table));
                        $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                if ((is_array($GLOBALS['TCA'][$table]) || $table == 'pages') && $uid > 0) {
                        $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery($fields, $table, 'uid = ' . $uid . $this->deleteClause($table));
                        $row = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
@@ -752,9 +848,9 @@ class PageRepository {
         * @param string $theField The fieldname to match, eg. "uid" or "alias
         * @param string $theValue The value that fieldname must match, eg. "123" or "frontpage
         * @param string $whereClause Optional additional WHERE clauses put in the end of the query. DO NOT PUT IN GROUP BY, ORDER BY or LIMIT!
         * @param string $theField The fieldname to match, eg. "uid" or "alias
         * @param string $theValue The value that fieldname must match, eg. "123" or "frontpage
         * @param string $whereClause Optional additional WHERE clauses put in the end of the query. DO NOT PUT IN GROUP BY, ORDER BY or LIMIT!
-        * @param string $groupBy Optional GROUP BY field(s), if none, supply blank string.
-        * @param string $orderBy Optional ORDER BY field(s), if none, supply blank string.
-        * @param string $limit Optional LIMIT value ([begin,]max), if none, supply blank string.
+        * @param string $groupBy Optional GROUP BY field(s). If none, supply blank string.
+        * @param string $orderBy Optional ORDER BY field(s). If none, supply blank string.
+        * @param string $limit Optional LIMIT value ([begin,]max). If none, supply blank string.
         * @return mixed Returns array (the record) if found, otherwise nothing (void)
         * @todo Define visibility
         */
         * @return mixed Returns array (the record) if found, otherwise nothing (void)
         * @todo Define visibility
         */
@@ -774,55 +870,55 @@ class PageRepository {
                }
        }
 
                }
        }
 
-       /*********************************
+       /********************************
         *
         * Caching and standard clauses
         *
         *
         * Caching and standard clauses
         *
-        **********************************/
+        ********************************/
+
        /**
        /**
-        * Returns string value stored for the hash string in the cache "cache_hash"
-        * Can be used to retrieved a cached value
+        * Returns data stored for the hash string in the cache "cache_hash"
+        * Can be used to retrieved a cached value, array or object
         * Can be used from your frontend plugins if you like. It is also used to
         * store the parsed TypoScript template structures. You can call it directly
         * like \TYPO3\CMS\Frontend\Page\PageRepository::getHash()
         *
         * @param string $hash The hash-string which was used to store the data value
         * @param integer The expiration time (not used anymore)
         * Can be used from your frontend plugins if you like. It is also used to
         * store the parsed TypoScript template structures. You can call it directly
         * like \TYPO3\CMS\Frontend\Page\PageRepository::getHash()
         *
         * @param string $hash The hash-string which was used to store the data value
         * @param integer The expiration time (not used anymore)
-        * @return string The "content" field of the "cache_hash" cache entry.
+        * @return mixed The "data" from the cache
         * @see tslib_TStemplate::start(), storeHash()
         */
        static public function getHash($hash, $expTime = 0) {
                $hashContent = NULL;
         * @see tslib_TStemplate::start(), storeHash()
         */
        static public function getHash($hash, $expTime = 0) {
                $hashContent = NULL;
-               if (is_object($GLOBALS['typo3CacheManager'])) {
-                       $contentHashCache = $GLOBALS['typo3CacheManager']->getCache('cache_hash');
-                       $cacheEntry = $contentHashCache->get($hash);
-                       if ($cacheEntry) {
-                               $hashContent = $cacheEntry;
-                       }
+               $contentHashCache = GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Cache\\CacheManager')->getCache('cache_hash');
+               $cacheEntry = $contentHashCache->get($hash);
+               if ($cacheEntry) {
+                       $hashContent = $cacheEntry;
                }
                return $hashContent;
        }
 
        /**
                }
                return $hashContent;
        }
 
        /**
-        * Stores a string value in the cache_hash cache identified by $hash.
+        * Stores $data in the 'cache_hash' cache with the hash key, $hash
+        * and visual/symbolic identification, $ident
+        *
         * Can be used from your frontend plugins if you like. You can call it
         * directly like \TYPO3\CMS\Frontend\Page\PageRepository::storeHash()
         *
         * @param string $hash 32 bit hash string (eg. a md5 hash of a serialized array identifying the data being stored)
         * Can be used from your frontend plugins if you like. You can call it
         * directly like \TYPO3\CMS\Frontend\Page\PageRepository::storeHash()
         *
         * @param string $hash 32 bit hash string (eg. a md5 hash of a serialized array identifying the data being stored)
-        * @param string $data The data string. If you want to store an array, then just serialize it first.
+        * @param mixed $data The data to store
         * @param string $ident Is just a textual identification in order to inform about the content!
         * @param integer $lifetime The lifetime for the cache entry in seconds
         * @return void
         * @see tslib_TStemplate::start(), getHash()
         */
        static public function storeHash($hash, $data, $ident, $lifetime = 0) {
         * @param string $ident Is just a textual identification in order to inform about the content!
         * @param integer $lifetime The lifetime for the cache entry in seconds
         * @return void
         * @see tslib_TStemplate::start(), getHash()
         */
        static public function storeHash($hash, $data, $ident, $lifetime = 0) {
-               if (is_object($GLOBALS['typo3CacheManager'])) {
-                       $GLOBALS['typo3CacheManager']->getCache('cache_hash')->set($hash, $data, array('ident_' . $ident), intval($lifetime));
-               }
+               GeneralUtility::makeInstance('TYPO3\\CMS\\Core\\Cache\\CacheManager')->getCache('cache_hash')->set($hash, $data, array('ident_' . $ident), (int)$lifetime);
        }
 
        /**
        }
 
        /**
-        * Returns the "AND NOT deleted" clause for the tablename given IF $GLOBALS['TCA'] configuration points to such a field.
+        * Returns the "AND NOT deleted" clause for the tablename given IF
+        * $GLOBALS['TCA'] configuration points to such a field.
         *
         * @param string $table Tablename
         * @return string
         *
         * @param string $table Tablename
         * @return string
@@ -830,8 +926,9 @@ class PageRepository {
         * @todo Define visibility
         */
        public function deleteClause($table) {
         * @todo Define visibility
         */
        public function deleteClause($table) {
-               // Hardcode for pages because TCA might not be loaded yet (early frontend initialization)
-               if (!strcmp($table, 'pages')) {
+               // Hardcode for pages because TCA might not be loaded yet (early frontend
+               // initialization)
+               if ($table === 'pages') {
                        return ' AND pages.deleted=0';
                } else {
                        return $GLOBALS['TCA'][$table]['ctrl']['delete'] ? ' AND ' . $table . '.' . $GLOBALS['TCA'][$table]['ctrl']['delete'] . '=0' : '';
                        return ' AND pages.deleted=0';
                } else {
                        return $GLOBALS['TCA'][$table]['ctrl']['delete'] ? ' AND ' . $table . '.' . $GLOBALS['TCA'][$table]['ctrl']['delete'] . '=0' : '';
@@ -839,20 +936,27 @@ class PageRepository {
        }
 
        /**
        }
 
        /**
-        * Returns a part of a WHERE clause which will filter out records with start/end times or hidden/fe_groups fields set to values that should de-select them according to the current time, preview settings or user login. Definitely a frontend function.
-        * Is using the $GLOBALS['TCA'] arrays "ctrl" part where the key "enablefields" determines for each table which of these features applies to that table.
+        * Returns a part of a WHERE clause which will filter out records with start/end
+        * times or hidden/fe_groups fields set to values that should de-select them
+        * according to the current time, preview settings or user login. Definitely a
+        * frontend function.
+        *
+        * Is using the $GLOBALS['TCA'] arrays "ctrl" part where the key "enablefields"
+        * determines for each table which of these features applies to that table.
         *
         * @param string $table Table name found in the $GLOBALS['TCA'] array
         * @param integer $show_hidden If $show_hidden is set (0/1), any hidden-fields in records are ignored. NOTICE: If you call this function, consider what to do with the show_hidden parameter. Maybe it should be set? See ContentObjectRenderer->enableFields where it's implemented correctly.
         * @param array $ignore_array Array you can pass where keys can be "disabled", "starttime", "endtime", "fe_group" (keys from "enablefields" in TCA) and if set they will make sure that part of the clause is not added. Thus disables the specific part of the clause. For previewing etc.
         * @param boolean $noVersionPreview If set, enableFields will be applied regardless of any versioning preview settings which might otherwise disable enableFields
         *
         * @param string $table Table name found in the $GLOBALS['TCA'] array
         * @param integer $show_hidden If $show_hidden is set (0/1), any hidden-fields in records are ignored. NOTICE: If you call this function, consider what to do with the show_hidden parameter. Maybe it should be set? See ContentObjectRenderer->enableFields where it's implemented correctly.
         * @param array $ignore_array Array you can pass where keys can be "disabled", "starttime", "endtime", "fe_group" (keys from "enablefields" in TCA) and if set they will make sure that part of the clause is not added. Thus disables the specific part of the clause. For previewing etc.
         * @param boolean $noVersionPreview If set, enableFields will be applied regardless of any versioning preview settings which might otherwise disable enableFields
+        * @throws \InvalidArgumentException
         * @return string The clause starting like " AND ...=... AND ...=...
         * @see \TYPO3\CMS\Frontend\ContentObject\ContentObjectRenderer::enableFields(), deleteClause()
         * @todo Define visibility
         */
        public function enableFields($table, $show_hidden = -1, $ignore_array = array(), $noVersionPreview = FALSE) {
                if ($show_hidden == -1 && is_object($GLOBALS['TSFE'])) {
         * @return string The clause starting like " AND ...=... AND ...=...
         * @see \TYPO3\CMS\Frontend\ContentObject\ContentObjectRenderer::enableFields(), deleteClause()
         * @todo Define visibility
         */
        public function enableFields($table, $show_hidden = -1, $ignore_array = array(), $noVersionPreview = FALSE) {
                if ($show_hidden == -1 && is_object($GLOBALS['TSFE'])) {
-                       // If show_hidden was not set from outside and if TSFE is an object, set it based on showHiddenPage and showHiddenRecords from TSFE
+                       // If show_hidden was not set from outside and if TSFE is an object, set it
+                       // based on showHiddenPage and showHiddenRecords from TSFE
                        $show_hidden = $table == 'pages' ? $GLOBALS['TSFE']->showHiddenPage : $GLOBALS['TSFE']->showHiddenRecords;
                }
                if ($show_hidden == -1) {
                        $show_hidden = $table == 'pages' ? $GLOBALS['TSFE']->showHiddenPage : $GLOBALS['TSFE']->showHiddenRecords;
                }
                if ($show_hidden == -1) {
@@ -866,14 +970,32 @@ class PageRepository {
                        if ($ctrl['delete']) {
                                $query .= ' AND ' . $table . '.' . $ctrl['delete'] . '=0';
                        }
                        if ($ctrl['delete']) {
                                $query .= ' AND ' . $table . '.' . $ctrl['delete'] . '=0';
                        }
-                       // Filter out new place-holder records in case we are NOT in a versioning preview (that means we are online!)
-                       if ($ctrl['versioningWS'] && !$this->versioningPreview) {
-                               // Shadow state for new items MUST be ignored!
-                               $query .= ' AND ' . $table . '.t3ver_state<=0 AND ' . $table . '.pid<>-1';
+                       if ($ctrl['versioningWS']) {
+                               if (!$this->versioningPreview) {
+                                       // Filter out placeholder records (new/moved/deleted items)
+                                       // in case we are NOT in a versioning preview (that means we are online!)
+                                       $query .= ' AND ' . $table . '.t3ver_state<=' . new VersionState(VersionState::DEFAULT_STATE);
+                               } else {
+                                       if ($table !== 'pages') {
+                                               // show only records of live and of the current workspace
+                                               // in case we are in a versioning preview
+                                               $query .= ' AND (' .
+                                                                       $table . '.t3ver_wsid=0 OR ' .
+                                                                       $table . '.t3ver_wsid=' . (int)$this->versioningWorkspaceId .
+                                                                       ')';
+                                       }
+                               }
+
+                               // Filter out versioned records
+                               if (!$noVersionPreview && empty($ignore_array['pid'])) {
+                                       $query .= ' AND ' . $table . '.pid<>-1';
+                               }
                        }
                        }
+
                        // Enable fields:
                        if (is_array($ctrl['enablecolumns'])) {
                        // Enable fields:
                        if (is_array($ctrl['enablecolumns'])) {
-                               // In case of versioning-preview, enableFields are ignored (checked in versionOL())
+                               // In case of versioning-preview, enableFields are ignored (checked in
+                               // versionOL())
                                if (!$this->versioningPreview || !$ctrl['versioningWS'] || $noVersionPreview) {
                                        if ($ctrl['enablecolumns']['disabled'] && !$show_hidden && !$ignore_array['disabled']) {
                                                $field = $table . '.' . $ctrl['enablecolumns']['disabled'];
                                if (!$this->versioningPreview || !$ctrl['versioningWS'] || $noVersionPreview) {
                                        if ($ctrl['enablecolumns']['disabled'] && !$show_hidden && !$ignore_array['disabled']) {
                                                $field = $table . '.' . $ctrl['enablecolumns']['disabled'];
@@ -892,7 +1014,8 @@ class PageRepository {
                                                $query .= $this->getMultipleGroupsWhereClause($field, $table);
                                        }
                                        // Call hook functions for additional enableColumns
                                                $query .= $this->getMultipleGroupsWhereClause($field, $table);
                                        }
                                        // Call hook functions for additional enableColumns
-                                       // It is used by the extension ingmar_accessctrl which enables assigning more than one usergroup to content and page records
+                                       // It is used by the extension ingmar_accessctrl which enables assigning more
+                                       // than one usergroup to content and page records
                                        if (is_array($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_page.php']['addEnableColumns'])) {
                                                $_params = array(
                                                        'table' => $table,
                                        if (is_array($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_page.php']['addEnableColumns'])) {
                                                $_params = array(
                                                        'table' => $table,
@@ -913,7 +1036,8 @@ class PageRepository {
        }
 
        /**
        }
 
        /**
-        * Creating where-clause for checking group access to elements in enableFields function
+        * Creating where-clause for checking group access to elements in enableFields
+        * function
         *
         * @param string $field Field with group list
         * @param string $table Table name
         *
         * @param string $field Field with group list
         * @param string $table Table name
@@ -936,16 +1060,24 @@ class PageRepository {
                return ' AND (' . implode(' OR ', $orChecks) . ')';
        }
 
                return ' AND (' . implode(' OR ', $orChecks) . ')';
        }
 
-       /*********************************
+       /**********************
         *
         * Versioning Preview
         *
         *
         * Versioning Preview
         *
-        **********************************/
+        **********************/
+
        /**
         * Finding online PID for offline version record
        /**
         * Finding online PID for offline version record
-        * ONLY active when backend user is previewing records. MUST NEVER affect a site served which is not previewed by backend users!!!
-        * Will look if the "pid" value of the input record is -1 (it is an offline version) and if the table supports versioning; if so, it will translate the -1 PID into the PID of the original record
+        *
+        * ONLY active when backend user is previewing records. MUST NEVER affect a site
+        * served which is not previewed by backend users!!!
+        *
+        * Will look if the "pid" value of the input record is -1 (it is an offline
+        * version) and if the table supports versioning; if so, it will translate the -1
+        * PID into the PID of the original record.
+        *
         * Used whenever you are tracking something back, like making the root line.
         * Used whenever you are tracking something back, like making the root line.
+        *
         * Principle; Record offline! => Find online?
         *
         * @param string $table Table name
         * Principle; Record offline! => Find online?
         *
         * @param string $table Table name
@@ -963,19 +1095,24 @@ class PageRepository {
                                $oid = $rr['t3ver_oid'];
                                $wsid = $rr['t3ver_wsid'];
                        } else {
                                $oid = $rr['t3ver_oid'];
                                $wsid = $rr['t3ver_wsid'];
                        } else {
-                               // Otherwise we have to expect "uid" to be in the record and look up based on this:
+                               // Otherwise we have to expect "uid" to be in the record and look up based
+                               // on this:
                                $newPidRec = $this->getRawRecord($table, $rr['uid'], 't3ver_oid,t3ver_wsid', TRUE);
                                if (is_array($newPidRec)) {
                                        $oid = $newPidRec['t3ver_oid'];
                                        $wsid = $newPidRec['t3ver_wsid'];
                                }
                        }
                                $newPidRec = $this->getRawRecord($table, $rr['uid'], 't3ver_oid,t3ver_wsid', TRUE);
                                if (is_array($newPidRec)) {
                                        $oid = $newPidRec['t3ver_oid'];
                                        $wsid = $newPidRec['t3ver_wsid'];
                                }
                        }
-                       // If workspace ids matches and ID of current online version is found, look up the PID value of that:
-                       if ($oid && ($this->versioningWorkspaceId == 0 && $this->checkWorkspaceAccess($wsid) || !strcmp((int) $wsid, $this->versioningWorkspaceId))) {
+                       // If workspace ids matches and ID of current online version is found, look up
+                       // the PID value of that:
+                       if ($oid && ((int)$this->versioningWorkspaceId === 0 && $this->checkWorkspaceAccess($wsid) || (int)$wsid === (int)$this->versioningWorkspaceId)) {
                                $oidRec = $this->getRawRecord($table, $oid, 'pid', TRUE);
                                if (is_array($oidRec)) {
                                $oidRec = $this->getRawRecord($table, $oid, 'pid', TRUE);
                                if (is_array($oidRec)) {
-                                       // SWAP uid as well? Well no, because when fixing a versioning PID happens it is assumed that this is a "branch" type page and therefore the uid should be kept (like in versionOL()).
-                                       // However if the page is NOT a branch version it should not happen - but then again, direct access to that uid should not happen!
+                                       // SWAP uid as well? Well no, because when fixing a versioning PID happens it is
+                                       // assumed that this is a "branch" type page and therefore the uid should be
+                                       // kept (like in versionOL()). However if the page is NOT a branch version it
+                                       // should not happen - but then again, direct access to that uid should not
+                                       // happen!
                                        $rr['_ORIG_pid'] = $rr['pid'];
                                        $rr['pid'] = $oidRec['pid'];
                                }
                                        $rr['_ORIG_pid'] = $rr['pid'];
                                        $rr['pid'] = $oidRec['pid'];
                                }
@@ -989,8 +1126,15 @@ class PageRepository {
 
        /**
         * Versioning Preview Overlay
 
        /**
         * Versioning Preview Overlay
-        * ONLY active when backend user is previewing records. MUST NEVER affect a site served which is not previewed by backend users!!!
-        * Generally ALWAYS used when records are selected based on uid or pid. If records are selected on other fields than uid or pid (eg. "email = ....") then usage might produce undesired results and that should be evaluated on individual basis.
+        *
+        * ONLY active when backend user is previewing records. MUST NEVER affect a site
+        * served which is not previewed by backend users!!!
+        *
+        * Generally ALWAYS used when records are selected based on uid or pid. If
+        * records are selected on other fields than uid or pid (eg. "email = ....") then
+        * usage might produce undesired results and that should be evaluated on
+        * individual basis.
+        *
         * Principle; Record online! => Find offline?
         *
         * @param string $table Table name
         * Principle; Record online! => Find offline?
         *
         * @param string $table Table name
@@ -1003,18 +1147,23 @@ class PageRepository {
         */
        public function versionOL($table, &$row, $unsetMovePointers = FALSE, $bypassEnableFieldsCheck = FALSE) {
                if ($this->versioningPreview && is_array($row)) {
         */
        public function versionOL($table, &$row, $unsetMovePointers = FALSE, $bypassEnableFieldsCheck = FALSE) {
                if ($this->versioningPreview && is_array($row)) {
-                       // will overlay any movePlhOL found with the real record, which in turn will be overlaid with its workspace version if any.
+                       // will overlay any movePlhOL found with the real record, which in turn
+                       // will be overlaid with its workspace version if any.
                        $movePldSwap = $this->movePlhOL($table, $row);
                        $movePldSwap = $this->movePlhOL($table, $row);
-                       // implode(',',array_keys($row)) = Using fields from original record to make sure no additional fields are selected. This is best for eg. getPageOverlay()
+                       // implode(',',array_keys($row)) = Using fields from original record to make
+                       // sure no additional fields are selected. This is best for eg. getPageOverlay()
                        if ($wsAlt = $this->getWorkspaceVersionOfRecord($this->versioningWorkspaceId, $table, $row['uid'], implode(',', array_keys($row)), $bypassEnableFieldsCheck)) {
                                if (is_array($wsAlt)) {
                        if ($wsAlt = $this->getWorkspaceVersionOfRecord($this->versioningWorkspaceId, $table, $row['uid'], implode(',', array_keys($row)), $bypassEnableFieldsCheck)) {
                                if (is_array($wsAlt)) {
-                                       // Always fix PID (like in fixVersioningPid() above). [This is usually not the important factor for versioning OL]
+                                       // Always fix PID (like in fixVersioningPid() above). [This is usually not
+                                       // the important factor for versioning OL]
                                        // Keep the old (-1) - indicates it was a version...
                                        $wsAlt['_ORIG_pid'] = $wsAlt['pid'];
                                        // Set in the online versions PID.
                                        $wsAlt['pid'] = $row['pid'];
                                        // Keep the old (-1) - indicates it was a version...
                                        $wsAlt['_ORIG_pid'] = $wsAlt['pid'];
                                        // Set in the online versions PID.
                                        $wsAlt['pid'] = $row['pid'];
-                                       // For versions of single elements or page+content, preserve online UID and PID (this will produce true "overlay" of element _content_, not any references)
-                                       // For page+content the "_ORIG_uid" should actually be used as PID for selection of tables with "versioning_followPages" enabled.
+                                       // For versions of single elements or page+content, preserve online UID and PID
+                                       // (this will produce true "overlay" of element _content_, not any references)
+                                       // For page+content the "_ORIG_uid" should actually be used as PID for selection
+                                       // of tables with "versioning_followPages" enabled.
                                        $wsAlt['_ORIG_uid'] = $wsAlt['uid'];
                                        $wsAlt['uid'] = $row['uid'];
                                        // Translate page alias as well so links are pointing to the _online_ page:
                                        $wsAlt['_ORIG_uid'] = $wsAlt['uid'];
                                        $wsAlt['uid'] = $row['uid'];
                                        // Translate page alias as well so links are pointing to the _online_ page:
@@ -1024,20 +1173,33 @@ class PageRepository {
                                        // Changing input record to the workspace version alternative:
                                        $row = $wsAlt;
                                        // Check if it is deleted/new
                                        // Changing input record to the workspace version alternative:
                                        $row = $wsAlt;
                                        // Check if it is deleted/new
-                                       if ((int) $row['t3ver_state'] === 1 || (int) $row['t3ver_state'] === 2) {
+                                       $rowVersionState = VersionState::cast($row['t3ver_state']);
+                                       if (
+                                               $rowVersionState->equals(VersionState::NEW_PLACEHOLDER)
+                                               || $rowVersionState->equals(VersionState::DELETE_PLACEHOLDER)
+                                       ) {
                                                // Unset record if it turned out to be deleted in workspace
                                                $row = FALSE;
                                        }
                                                // Unset record if it turned out to be deleted in workspace
                                                $row = FALSE;
                                        }
-                                       // Check if move-pointer in workspace (unless if a move-placeholder is the reason why it appears!):
-                                       // You have to specifically set $unsetMovePointers in order to clear these because it is normally a display issue if it should be shown or not.
-                                       if (((int) $row['t3ver_state'] === 4 && !$movePldSwap) && $unsetMovePointers) {
+                                       // Check if move-pointer in workspace (unless if a move-placeholder is the
+                                       // reason why it appears!):
+                                       // You have to specifically set $unsetMovePointers in order to clear these
+                                       // because it is normally a display issue if it should be shown or not.
+                                       if (
+                                               ($rowVersionState->equals(VersionState::MOVE_POINTER)
+                                                       && !$movePldSwap
+                                               ) && $unsetMovePointers
+                                       ) {
                                                // Unset record if it turned out to be deleted in workspace
                                                $row = FALSE;
                                        }
                                } else {
                                                // Unset record if it turned out to be deleted in workspace
                                                $row = FALSE;
                                        }
                                } else {
-                                       // No version found, then check if t3ver_state =1 (online version is dummy-representation)
-                                       // Notice, that unless $bypassEnableFieldsCheck is TRUE, the $row is unset if enablefields for BOTH the version AND the online record deselects it. See note for $bypassEnableFieldsCheck
-                                       if ($wsAlt <= -1 || (int) $row['t3ver_state'] > 0) {
+                                       // No version found, then check if t3ver_state = VersionState::NEW_PLACEHOLDER
+                                       // (online version is dummy-representation)
+                                       // Notice, that unless $bypassEnableFieldsCheck is TRUE, the $row is unset if
+                                       // enablefields for BOTH the version AND the online record deselects it. See
+                                       // note for $bypassEnableFieldsCheck
+                                       if ($wsAlt <= -1 || VersionState::cast($row['t3ver_state'])->indicatesPlaceholder()) {
                                                // Unset record if it turned out to be "hidden"
                                                $row = FALSE;
                                        }
                                                // Unset record if it turned out to be "hidden"
                                                $row = FALSE;
                                        }
@@ -1047,8 +1209,9 @@ class PageRepository {
        }
 
        /**
        }
 
        /**
-        * Checks if record is a move-placeholder (t3ver_state==3) and if so it will set $row to be the pointed-to live record (and return TRUE)
-        * Used from versionOL
+        * Checks if record is a move-placeholder
+        * (t3ver_state==VersionState::MOVE_PLACEHOLDER) and if so it will set $row to be
+        * the pointed-to live record (and return TRUE) Used from versionOL
         *
         * @param string $table Table name
         * @param array $row Row (passed by reference) - only online records...
         *
         * @param string $table Table name
         * @param array $row Row (passed by reference) - only online records...
@@ -1057,9 +1220,13 @@ class PageRepository {
         * @todo Define visibility
         */
        public function movePlhOL($table, &$row) {
         * @todo Define visibility
         */
        public function movePlhOL($table, &$row) {
-               if (($table == 'pages' || (int) $GLOBALS['TCA'][$table]['ctrl']['versioningWS'] >= 2) && (int) $row['t3ver_state'] === 3) {
+               if (
+                       ($table == 'pages'
+                               || (int)$GLOBALS['TCA'][$table]['ctrl']['versioningWS'] >= 2
+                       ) && (int)VersionState::cast($row['t3ver_state'])->equals(VersionState::MOVE_PLACEHOLDER)
+               ) {
                        // Only for WS ver 2... (moving)
                        // Only for WS ver 2... (moving)
-                       // If t3ver_move_id is not found, then find it... (but we like best if it is here...)
+                       // If t3ver_move_id is not found, then find it (but we like best if it is here)
                        if (!isset($row['t3ver_move_id'])) {
                                $moveIDRec = $this->getRawRecord($table, $row['uid'], 't3ver_move_id', TRUE);
                                $moveID = $moveIDRec['t3ver_move_id'];
                        if (!isset($row['t3ver_move_id'])) {
                                $moveIDRec = $this->getRawRecord($table, $row['uid'], 't3ver_move_id', TRUE);
                                $moveID = $moveIDRec['t3ver_move_id'];
@@ -1068,7 +1235,7 @@ class PageRepository {
                        }
                        // Find pointed-to record.
                        if ($moveID) {
                        }
                        // Find pointed-to record.
                        if ($moveID) {
-                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery(implode(',', array_keys($row)), $table, 'uid=' . intval($moveID) . $this->enableFields($table));
+                               $res = $GLOBALS['TYPO3_DB']->exec_SELECTquery(implode(',', array_keys($row)), $table, 'uid=' . (int)$moveID . $this->enableFields($table));
                                $origRow = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                if ($origRow) {
                                $origRow = $GLOBALS['TYPO3_DB']->sql_fetch_assoc($res);
                                $GLOBALS['TYPO3_DB']->sql_free_result($res);
                                if ($origRow) {
@@ -1092,13 +1259,13 @@ class PageRepository {
         */
        public function getMovePlaceholder($table, $uid, $fields = '*') {
                if ($this->versioningPreview) {
         */
        public function getMovePlaceholder($table, $uid, $fields = '*') {
                if ($this->versioningPreview) {
-                       $workspace = (int) $this->versioningWorkspaceId;
-                       if (($table == 'pages' || (int) $GLOBALS['TCA'][$table]['ctrl']['versioningWS'] >= 2) && $workspace !== 0) {
+                       $workspace = (int)$this->versioningWorkspaceId;
+                       if (($table == 'pages' || (int)$GLOBALS['TCA'][$table]['ctrl']['versioningWS'] >= 2) && $workspace !== 0) {
                                // Select workspace version of record:
                                $row = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow($fields, $table, 'pid<>-1 AND
                                // Select workspace version of record:
                                $row = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow($fields, $table, 'pid<>-1 AND
-                                               t3ver_state=3 AND
-                                               t3ver_move_id=' . intval($uid) . ' AND
-                                               t3ver_wsid=' . intval($workspace) . $this->deleteClause($table));
+                                               t3ver_state=' . new VersionState(VersionState::MOVE_PLACEHOLDER) . ' AND
+                                               t3ver_move_id=' . (int)$uid . ' AND
+                                               t3ver_wsid=' . (int)$workspace . $this->deleteClause($table));
                                if (is_array($row)) {
                                        return $row;
                                }
                                if (is_array($row)) {
                                        return $row;
                                }
@@ -1120,7 +1287,9 @@ class PageRepository {
         * @todo Define visibility
         */
        public function getWorkspaceVersionOfRecord($workspace, $table, $uid, $fields = '*', $bypassEnableFieldsCheck = FALSE) {
         * @todo Define visibility
         */
        public function getWorkspaceVersionOfRecord($workspace, $table, $uid, $fields = '*', $bypassEnableFieldsCheck = FALSE) {
-               if ($workspace !== 0) {
+               if ($workspace !== 0 && !empty($GLOBALS['TCA'][$table]['ctrl']['versioningWS'])) {
+                       $workspace = (int)$workspace;
+                       $uid = (int)$uid;
                        // Have to hardcode it for "pages" table since TCA is not loaded at this moment!
                        // Setting up enableFields for version record:
                        if ($table == 'pages') {
                        // Have to hardcode it for "pages" table since TCA is not loaded at this moment!
                        // Setting up enableFields for version record:
                        if ($table == 'pages') {
@@ -1130,13 +1299,14 @@ class PageRepository {
                        }
                        // Select workspace version of record, only testing for deleted.
                        $newrow = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow($fields, $table, 'pid=-1 AND
                        }
                        // Select workspace version of record, only testing for deleted.
                        $newrow = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow($fields, $table, 'pid=-1 AND
-                                       t3ver_oid=' . intval($uid) . ' AND
-                                       t3ver_wsid=' . intval($workspace) . $this->deleteClause($table));
-                       // If version found, check if it could have been selected with enableFields on as well:
+                                       t3ver_oid=' . $uid . ' AND
+                                       t3ver_wsid=' . $workspace . $this->deleteClause($table));
+                       // If version found, check if it could have been selected with enableFields on
+                       // as well:
                        if (is_array($newrow)) {
                                if ($bypassEnableFieldsCheck || $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('uid', $table, 'pid=-1 AND
                        if (is_array($newrow)) {
                                if ($bypassEnableFieldsCheck || $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('uid', $table, 'pid=-1 AND
-                                               t3ver_oid=' . intval($uid) . ' AND
-                                               t3ver_wsid=' . intval($workspace) . $enFields)) {
+                                               t3ver_oid=' . $uid . ' AND
+                                               t3ver_wsid=' . $workspace . $enFields)) {
                                        // Return offline version, tested for its enableFields.
                                        return $newrow;
                                } else {
                                        // Return offline version, tested for its enableFields.
                                        return $newrow;
                                } else {
@@ -1144,8 +1314,9 @@ class PageRepository {
                                        return -1;
                                }
                        } else {
                                        return -1;
                                }
                        } else {
-                               // OK, so no workspace version was found. Then check if online version can be selected with full enable fields and if so, return 1:
-                               if ($bypassEnableFieldsCheck || $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('uid', $table, 'uid=' . intval($uid) . $enFields)) {
+                               // OK, so no workspace version was found. Then check if online version can be
+                               // selected with full enable fields and if so, return 1:
+                               if ($bypassEnableFieldsCheck || $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('uid', $table, 'uid=' . $uid . $enFields)) {
                                        // Means search was done, but no version found.
                                        return 1;
                                } else {
                                        // Means search was done, but no version found.
                                        return 1;
                                } else {
@@ -1154,7 +1325,8 @@ class PageRepository {
                                }
                        }
                }
                                }
                        }
                }
-               // No look up in database because versioning not enabled / or workspace not offline
+               // No look up in database because versioning not enabled / or workspace not
+               // offline
                return FALSE;
        }
 
                return FALSE;
        }
 
@@ -1174,7 +1346,7 @@ class PageRepository {
                } else {
                        if ($wsid > 0) {
                                // No $GLOBALS['TCA'] yet!
                } else {
                        if ($wsid > 0) {
                                // No $GLOBALS['TCA'] yet!
-                               $ws = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('*', 'sys_workspace', 'uid=' . intval($wsid) . ' AND deleted=0');
+                               $ws = $GLOBALS['TYPO3_DB']->exec_SELECTgetSingleRow('*', 'sys_workspace', 'uid=' . (int)$wsid . ' AND deleted=0');
                                if (!is_array($ws)) {
                                        return FALSE;
                                }
                                if (!is_array($ws)) {
                                        return FALSE;
                                }
@@ -1187,7 +1359,36 @@ class PageRepository {
                return $ws['_ACCESS'] != '';
        }
 
                return $ws['_ACCESS'] != '';
        }
 
-}
+       /**
+        * Determine if a field needs an overlay
+        *
+        * @param string $table TCA tablename
+        * @param string $field TCA fieldname
+        * @param mixed $value Current value of the field
+        * @return boolean Returns TRUE if a given record field needs to be overlaid
+        */
+       protected function shouldFieldBeOverlaid($table, $field, $value) {
+               $l10n_mode = isset($GLOBALS['TCA'][$table]['columns'][$field]['l10n_mode'])
+                       ? $GLOBALS['TCA'][$table]['columns'][$field]['l10n_mode']
+                       : '';
 
 
+               $shouldFieldBeOverlaid = TRUE;
 
 
-?>
\ No newline at end of file
+               if ($l10n_mode === 'exclude') {
+                       $shouldFieldBeOverlaid = FALSE;
+               } elseif ($l10n_mode === 'mergeIfNotBlank') {
+                       $checkValue = $value;
+
+                       // 0 values are considered blank when coming from a group field
+                       if (empty($value) && $GLOBALS['TCA'][$table]['columns'][$field]['config']['type'] === 'group') {
+                               $checkValue = '';
+                       }
+
+                       if (trim($checkValue) === '') {
+                               $shouldFieldBeOverlaid = FALSE;
+                       }
+               }
+
+               return $shouldFieldBeOverlaid;
+       }
+}