[BUGFIX] Streamline deprecation messages 54/57754/2
authorBenni Mack <benni@typo3.org>
Wed, 1 Aug 2018 12:36:23 +0000 (14:36 +0200)
committerWouter Wolters <typo3@wouterwolters.nl>
Wed, 1 Aug 2018 12:49:20 +0000 (14:49 +0200)
Some usages of trigger_error() used \E_USER_DEPRECATED or no 2nd parameter
(which falls back to E_USER_NOTICE) which resulted in some weird issues
with opcache fatal PHP errors.

Resolves: #85706
Releases: master
Change-Id: I0916cc000d6c5d5b7e436bf91285072ff66dd657
Reviewed-on: https://review.typo3.org/57754
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Daniel Sattler <sattler@b13.de>
Tested-by: Daniel Sattler <sattler@b13.de>
Reviewed-by: Wouter Wolters <typo3@wouterwolters.nl>
Tested-by: Wouter Wolters <typo3@wouterwolters.nl>
typo3/sysext/adminpanel/Classes/View/AdminPanelView.php
typo3/sysext/backend/Classes/Form/FormDataProvider/DatabaseRowInitializeNew.php
typo3/sysext/extbase/Classes/Service/ExtensionService.php
typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php
typo3/sysext/install/Classes/Service/CoreUpdateService.php
typo3/sysext/install/Classes/Service/CoreVersionService.php

index 4d31d6d..9323071 100644 (file)
@@ -236,7 +236,7 @@ class AdminPanelView
     {
         trigger_error(
             'Deprecated since TYPO3 v9, use fluid and backend uri builder to create a toolbar',
-            \E_USER_DEPRECATED
+            E_USER_DEPRECATED
         );
         $editToolbarService = GeneralUtility::makeInstance(EditToolbarService::class);
         return $editToolbarService->createToolbar();
index 5eeb7fd..91c5ad5 100644 (file)
@@ -160,7 +160,7 @@ class DatabaseRowInitializeNew implements FormDataProviderInterface
             trigger_error(
                 'Default values for new database rows should be set from controller context. Applying default values'
                 . ' via GET/POST parameters is deprecated since 9.2 and will be removed in version 10',
-                \E_USER_DEPRECATED
+                E_USER_DEPRECATED
             );
             $result['defaultValues'] = $defaultValues;
         }
index 9b148c7..70a0de0 100644 (file)
@@ -238,7 +238,7 @@ class ExtensionService implements \TYPO3\CMS\Core\SingletonInterface
         // Legacy location
         $settings = $this->configurationManager->getConfiguration(ConfigurationManagerInterface::CONFIGURATION_TYPE_SETTINGS, $extensionName);
         if (isset($settings['view']['formatToPageTypeMapping']) && is_array($settings['view']['formatToPageTypeMapping'])) {
-            trigger_error('Extension "' . $extensionName . '": Defining settings.view.formatToPageTypeMapping will be removed in TYPO3 10. Move definition to view.formatToPageTypeMapping.');
+            trigger_error('Extension "' . $extensionName . '": Defining settings.view.formatToPageTypeMapping will be removed in TYPO3 10. Move definition to view.formatToPageTypeMapping.', E_USER_DEPRECATED);
             $formatToPageTypeMapping = $settings['view']['formatToPageTypeMapping'];
         }
         // Default behaviour
index 3558dd8..de9a44e 100644 (file)
@@ -3092,7 +3092,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
      */
     public function checkPageForMountpointRedirect()
     {
-        trigger_error('Method ' . __FUNCTION__ . 'is deprecated.', \E_USER_DEPRECATED);
+        trigger_error('Method ' . __FUNCTION__ . 'is deprecated.', E_USER_DEPRECATED);
         if (!empty($this->originalMountPointPage) && $this->originalMountPointPage['doktype'] == PageRepository::DOKTYPE_MOUNTPOINT) {
             $this->redirectToCurrentPage();
         }
@@ -3123,7 +3123,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
      */
     public function checkPageForShortcutRedirect()
     {
-        trigger_error('Method ' . __FUNCTION__ . 'is deprecated.', \E_USER_DEPRECATED);
+        trigger_error('Method ' . __FUNCTION__ . 'is deprecated.', E_USER_DEPRECATED);
         if (!empty($this->originalShortcutPage) && (int)$this->originalShortcutPage['doktype'] === PageRepository::DOKTYPE_SHORTCUT) {
             $this->redirectToCurrentPage();
         }
@@ -3136,7 +3136,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
      */
     protected function redirectToCurrentPage()
     {
-        trigger_error('Method ' . __FUNCTION__ . 'is deprecated.', \E_USER_DEPRECATED);
+        trigger_error('Method ' . __FUNCTION__ . 'is deprecated.', E_USER_DEPRECATED);
         $redirectUrl = $this->getUriToCurrentPageForRedirect();
         // Prevent redirection loop
         if (!empty($redirectUrl) && GeneralUtility::getIndpEnv('REQUEST_URI') !== '/' . $redirectUrl) {
@@ -4953,31 +4953,31 @@ class TypoScriptFrontendController implements LoggerAwareInterface
     {
         switch ($propertyName) {
             case 'sys_language_uid':
-                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'sys_language_content':
-                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'sys_language_contentOL':
-                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'sys_language_mode':
-                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'loginUser':
-                trigger_error('Property $TSFE->loginUser is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->loginUser is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'gr_list':
-                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'beUserLogin':
-                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.');
+                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'showHiddenPage':
-                trigger_error('Property $TSFE->showHiddenPage is not in use anymore as this information is now stored within the visibility aspect.');
+                trigger_error('Property $TSFE->showHiddenPage is not in use anymore as this information is now stored within the visibility aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
             case 'showHiddenRecords':
-                trigger_error('Property $TSFE->showHiddenRecords is not in use anymore as this information is now stored within the visibility aspect.');
+                trigger_error('Property $TSFE->showHiddenRecords is not in use anymore as this information is now stored within the visibility aspect.', E_USER_DEPRECATED);
                 return isset($this->$propertyName);
         }
         return false;
@@ -4996,32 +4996,32 @@ class TypoScriptFrontendController implements LoggerAwareInterface
     {
         switch ($propertyName) {
             case 'sys_language_uid':
-                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('language', 'id', 0);
             case 'sys_language_content':
-                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('language', 'contentId', 0);
             case 'sys_language_contentOL':
-                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('language', 'legacyOverlayType', '0');
                 break;
             case 'sys_language_mode':
-                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('language', 'legacyLanguageMode', '');
             case 'loginUser':
-                trigger_error('Property $TSFE->loginUser is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->loginUser is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('frontend.user', 'isLoggedIn', false);
             case 'gr_list':
-                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 return implode(',', $this->context->getPropertyFromAspect('frontend.user', 'groupIds', [0, -1]));
             case 'beUserLogin':
-                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.');
+                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('backend.user', 'isLoggedIn', false);
             case 'showHiddenPage':
-                trigger_error('Property $TSFE->showHiddenPage is not in use anymore as this information is now stored within the visibility aspect.');
+                trigger_error('Property $TSFE->showHiddenPage is not in use anymore as this information is now stored within the visibility aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('visibility', 'includeHiddenPages', false);
             case 'showHiddenRecords':
-                trigger_error('Property $TSFE->showHiddenRecords is not in use anymore as this information is now stored within the visibility aspect.');
+                trigger_error('Property $TSFE->showHiddenRecords is not in use anymore as this information is now stored within the visibility aspect.', E_USER_DEPRECATED);
                 return $this->context->getPropertyFromAspect('visibility', 'includeHiddenContent', false);
         }
         return $this->$propertyName;
@@ -5042,19 +5042,19 @@ class TypoScriptFrontendController implements LoggerAwareInterface
     {
         switch ($propertyName) {
             case 'sys_language_uid':
-                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, (int)$propertyValue, $aspect->getContentId(), $aspect->getOverlayType(), $aspect->getFallbackChain()));
                 break;
             case 'sys_language_content':
-                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, $aspect->getId(), (int)$propertyValue, $aspect->getOverlayType(), $aspect->getFallbackChain()));
                 break;
             case 'sys_language_contentOL':
-                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 switch ((string)$propertyValue) {
@@ -5070,7 +5070,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, $aspect->getId(), $aspect->getContentId(), $overlayType, $aspect->getFallbackChain()));
                 break;
             case 'sys_language_mode':
-                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 switch ((string)$propertyValue) {
@@ -5102,7 +5102,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, $aspect->getId(), $aspect->getContentId(), $aspect->getOverlayType(), $fallBackOrder));
                 break;
             case 'loginUser':
-                trigger_error('Property $TSFE->loginUser is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->loginUser is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 /** @var UserAspect $aspect */
                 $aspect = $this->context->getAspect('frontend.user');
                 if ($propertyValue) {
@@ -5113,11 +5113,11 @@ class TypoScriptFrontendController implements LoggerAwareInterface
                 $this->context->setAspect('frontend.user', $aspect);
                 break;
             case 'gr_list':
-                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 $this->context->setAspect('frontend.user', GeneralUtility::makeInstance(UserAspect::class, $this->fe_user ?: null, GeneralUtility::intExplode(',', $propertyValue)));
                 break;
             case 'beUserLogin':
-                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.');
+                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.', E_USER_DEPRECATED);
                 if ($propertyValue) {
                     $aspect = GeneralUtility::makeInstance(UserAspect::class, $GLOBALS['BE_USER']);
                 } else {
@@ -5127,7 +5127,7 @@ class TypoScriptFrontendController implements LoggerAwareInterface
                 break;
             case 'showHiddenPage':
             case 'showHiddenRecords':
-                trigger_error('Property $TSFE->' . $propertyName . ' is not in use anymore as this information is now stored within the visibility aspect.');
+                trigger_error('Property $TSFE->' . $propertyName . ' is not in use anymore as this information is now stored within the visibility aspect.', E_USER_DEPRECATED);
                 /** @var VisibilityAspect $aspect */
                 $aspect = $this->context->getAspect('visibility');
                 if ($propertyName === 'showHiddenPage') {
@@ -5150,23 +5150,23 @@ class TypoScriptFrontendController implements LoggerAwareInterface
     {
         switch ($propertyName) {
             case 'sys_language_uid':
-                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_uid is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class));
                 break;
             case 'sys_language_content':
-                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_content is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, $aspect->getId(), 0, $aspect->getOverlayType()));
                 break;
             case 'sys_language_contentOL':
-                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_contentOL is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, $aspect->getId(), $aspect->getContentId(), LanguageAspect::OVERLAYS_OFF));
                 break;
             case 'sys_language_mode':
-                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.');
+                trigger_error('Property $TSFE->sys_language_mode is not in use anymore as this information is now stored within the language aspect.', E_USER_DEPRECATED);
                 /** @var LanguageAspect $aspect */
                 $aspect = $this->context->getAspect('language');
                 $this->context->setAspect('language', GeneralUtility::makeInstance(LanguageAspect::class, $aspect->getId(), $aspect->getContentId(), $aspect->getOverlayType(), ['off']));
@@ -5177,16 +5177,16 @@ class TypoScriptFrontendController implements LoggerAwareInterface
                 $this->context->setAspect('frontend.user', GeneralUtility::makeInstance(UserAspect::class, null, $aspect->getGroupIds()));
                 break;
             case 'gr_list':
-                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.');
+                trigger_error('Property $TSFE->gr_list is not in use anymore as this information is now stored within the frontend.user aspect.', E_USER_DEPRECATED);
                 $this->context->setAspect('frontend.user', GeneralUtility::makeInstance(UserAspect::class, $this->fe_user ?: null, []));
                 break;
             case 'beUserLogin':
-                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.');
+                trigger_error('Property $TSFE->beUserLogin is not in use anymore as this information is now stored within the backend.user aspect.', E_USER_DEPRECATED);
                 $this->context->setAspect('backend.user', GeneralUtility::makeInstance(UserAspect::class));
                 break;
             case 'showHiddenPage':
             case 'showHiddenRecords':
-                trigger_error('Property $TSFE->' . $propertyName . ' is not in use anymore as this information is now stored within the visibility aspect.');
+                trigger_error('Property $TSFE->' . $propertyName . ' is not in use anymore as this information is now stored within the visibility aspect.', E_USER_DEPRECATED);
                 /** @var VisibilityAspect $aspect */
                 $aspect = $this->context->getAspect('visibility');
                 if ($propertyName === 'showHiddenPage') {
index 3ef6809..21f1a8d 100644 (file)
@@ -134,7 +134,7 @@ class CoreUpdateService
     {
         trigger_error(
             'The method updateVersionMatrix() is deprecated since v9 and will be removed in v10, use the REST api directly (see https://get.typo3.org/v1/api/doc).',
-            \E_USER_DEPRECATED
+            E_USER_DEPRECATED
         );
         $success = true;
         try {
index 9fa7586..e395126 100644 (file)
@@ -197,7 +197,7 @@ class CoreVersionService
      */
     public function updateVersionMatrix(): void
     {
-        trigger_error('Method updateVersionMatrix is deprecated, use new REST API directly (see https://get.typo3.org/v1/api/doc).', \E_USER_DEPRECATED);
+        trigger_error('Method updateVersionMatrix is deprecated, use new REST API directly (see https://get.typo3.org/v1/api/doc).', E_USER_DEPRECATED);
         $url = 'https://get.typo3.org/json';
         $versionJson = GeneralUtility::getUrl($url);
         if (!$versionJson) {
@@ -226,7 +226,7 @@ class CoreVersionService
     {
         trigger_error(
             'Method getYoungestPatchDevelopmentRelease() is deprecated since v9 and will be removed in v10, use getYoungestPatchRelease() instead.',
-            \E_USER_DEPRECATED
+            E_USER_DEPRECATED
         );
         return $this->getYoungestPatchRelease();
     }
@@ -242,7 +242,7 @@ class CoreVersionService
     {
         trigger_error(
             'Method isYoungerPatchDevelopmentReleaseAvailable() is deprecated since v9 and will be removed in v10, use isYoungerPatchReleaseAvailable() instead.',
-            \E_USER_DEPRECATED
+            E_USER_DEPRECATED
         );
         return $this->isYoungerPatchReleaseAvailable();
     }
@@ -255,7 +255,7 @@ class CoreVersionService
     {
         trigger_error(
             'Method getDownloadBaseUrl() is deprecated since v9 and will be removed in v10, use https://get.typo3.org directly.',
-            \E_USER_DEPRECATED
+            E_USER_DEPRECATED
         );
         return $this->apiBaseUrl;
     }