[TASK] Drop XCLASS registration from DefaultConfiguration 05/45805/3
authorWouter Wolters <typo3@wouterwolters.nl>
Sat, 9 Jan 2016 19:05:27 +0000 (20:05 +0100)
committerAnja Leichsenring <aleichsenring@ab-softlab.de>
Sat, 9 Jan 2016 20:34:05 +0000 (21:34 +0100)
Resolves: #72616
Releases: master
Change-Id: I2814e00582ee3daae60744a6958bff82d138da04
Reviewed-on: https://review.typo3.org/45805
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Reviewed-by: Michael Oehlhof <typo3@oehlhof.de>
Tested-by: Michael Oehlhof <typo3@oehlhof.de>
Reviewed-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
Tested-by: Anja Leichsenring <aleichsenring@ab-softlab.de>
typo3/sysext/core/Configuration/DefaultConfiguration.php
typo3/sysext/install/Classes/Service/SilentConfigurationUpgradeService.php
typo3/sysext/reports/Classes/Report/Status/Typo3Status.php
typo3/sysext/reports/Resources/Private/Language/locallang_reports.xlf
typo3/sysext/reports/Tests/Unit/Report/Status/Typo3StatusTest.php

index b28f337..052c338 100644 (file)
@@ -1047,7 +1047,6 @@ return array(
                 'Headers' => array('clickJackingProtection' => 'X-Frame-Options: SAMEORIGIN')
             )
         ),
-        'XCLASS' => array()
     ),
     'FE' => array( // Configuration for the TypoScript frontend (FE). Nothing here relates to the administration backend!
         'addAllowedPaths' => '',        // Additional relative paths (comma-list) to allow TypoScript resources be in. Should be prepended with '/'. If not, then any path where the first part is like this path will match. That is: 'myfolder/ , myarchive' will match eg. 'myfolder/', 'myarchive/', 'myarchive_one/', 'myarchive_2/' ... No check is done to see if this directory actually exists in the root of the site. Paths are matched by simply checking if these strings equals the first part of any TypoScript resource filepath. (See class template, function init() in \TYPO3\CMS\Core\TypoScript\Parser\TypoScriptParser)
@@ -1096,7 +1095,6 @@ return array(
         'versionNumberInFilename' => 'querystring',        // String: embed,querystring,''. Allows to automatically include a version number (timestamp of the file) to referred CSS and JS filenames on the rendered page. This will make browsers and proxies reload the files if they change (thus avoiding caching issues). Set to 'embed' will have the timestamp embedded in the filename, ie. filename.1269312081.js. IMPORTANT: 'embed' requires extra .htaccess rules to work (please refer to _.htaccess or the _.htaccess file from the dummy package)<p>Set to 'querystring' (default setting) to append the version number as a query parameter (doesn't require mod_rewrite). Set to '' will turn this functionality off (behaves like TYPO3 &lt; v4.4).</p>
         'contentRenderingTemplates' => array(),    // Array to define the TypoScript parts that define the main content rendering. Extensions like "css_styled_content" provide content rendering templates. Other extensions like "felogin" or "indexed search" extend these templates and their TypoScript parts are added directly after the content templates. See EXT:css_styled_content/ext_localconf.php and EXT:frontend/Classes/TypoScript/TemplateService.php
         'ContentObjects' => array(),    // Array to register ContentObject (cObjects) like TEXT or HMENU within ext_localconf.php, see EXT:frontend/ext_localconf.php
-        'XCLASS' => array(),        // See 'Inside TYPO3' document for more information.
     ),
     'MAIL' => array( // Mail configurations to tune how \TYPO3\CMS\Core\Mail\ classes will send their mails.
         'transport' => 'mail',        // <p>String:</p><dl><dt>mail</dt><dd>Sends messages by delegating to PHP's internal mail() function. No further settings required. This is the most unreliable option. If you are serious about sending mails, consider using "smtp" or "sendmail".</dd><dt>smtp</dt><dd>Sends messages over the (standardized) Simple Message Transfer Protocol. It can deal with encryption and authentication. Most flexible option, requires a mail server and configurations in transport_smtp_* settings below. Works the same on Windows, Unix and MacOS.</dd><dt>sendmail</dt><dd>Sends messages by communicating with a locally installed MTA - such as sendmail. See setting transport_sendmail_command bellow.<dd><dt>mbox</dt><dd>This doesn't send any mail out, but instead will write every outgoing mail to a file adhering to the RFC 4155 mbox format, which is a simple text file where the mails are concatenated. Useful for debugging the mail sending process and on development machines which cannot send mails to the outside. Configure the file to write to in the 'transport_mbox_file' setting below</dd><dt>&lt;classname&gt;</dt><dd>Custom class which implements Swift_Transport. The constructor receives all settings from the MAIL section to make it possible to add custom settings.</dd></dl>
index c73a6cc..d68a0cb 100755 (executable)
@@ -85,6 +85,9 @@ class SilentConfigurationUpgradeService
         'BE/unzip_path',
         // #72615
         'BE/notificationPrefix',
+        // #72616
+        'BE/XCLASS',
+        'FE/XCLASS',
     );
 
     /**
index 45c6e38..753d460 100644 (file)
@@ -32,7 +32,6 @@ class Typo3Status implements StatusProviderInterface
     public function getStatus()
     {
         $statuses = array(
-            'oldXclassStatus' => $this->getOldXclassUsageStatus(),
             'registeredXclass' => $this->getRegisteredXclassStatus(),
             'compatibility6' => $this->getCompatibility6Status(),
         );
@@ -40,39 +39,6 @@ class Typo3Status implements StatusProviderInterface
     }
 
     /**
-     * Check for usage of old way of implementing XCLASSes
-     *
-     * @return \TYPO3\CMS\Reports\Status
-     */
-    protected function getOldXclassUsageStatus()
-    {
-        $message = '';
-        $value = $this->getLanguageService()->getLL('status_none');
-        $severity = ReportStatus::OK;
-
-        $xclasses = array_merge(
-            (array)$GLOBALS['TYPO3_CONF_VARS']['BE']['XCLASS'],
-            (array)$GLOBALS['TYPO3_CONF_VARS']['FE']['XCLASS']
-        );
-
-        $numberOfXclasses = count($xclasses);
-        if ($numberOfXclasses > 0) {
-            $value = sprintf($this->getLanguageService()->getLL('status_oldXclassUsageFound'), $numberOfXclasses);
-            $message = $this->getLanguageService()->getLL('status_oldXclassUsageFound_message') . '<br />';
-            $message .= '<ol><li>' . implode('</li><li>', $xclasses) . '</li></ol>';
-            $severity = ReportStatus::NOTICE;
-        }
-
-        return GeneralUtility::makeInstance(
-            ReportStatus::class,
-            $this->getLanguageService()->getLL('status_oldXclassUsage'),
-            $value,
-            $message,
-            $severity
-        );
-    }
-
-    /**
      * List any Xclasses registered in the system
      *
      * @return \TYPO3\CMS\Reports\Status
index d574adc..7e4d073 100644 (file)
@@ -264,15 +264,6 @@ You can increase the size to 8MB (default on unix) by adding to the httpd.conf:
                        <trans-unit id="status_updateTask_email_issues">
                                <source>Issues</source>
                        </trans-unit>
-                       <trans-unit id="status_oldXclassUsage">
-                               <source>Old XCLASS method used</source>
-                       </trans-unit>
-                       <trans-unit id="status_oldXclassUsageFound">
-                               <source>Old XCLASS method is used in %d cases:</source>
-                       </trans-unit>
-                       <trans-unit id="status_oldXclassUsageFound_message">
-                               <source>The old XCLASS registration does not work since TYPO3 CMS version 6.0. This is not necessarily a bad thing, the registration might be done by an extension author to keep compatibility with older TYPO3 CMS versions. Please check the affected extension for TYPO3 CMS 6.0 compatibility to verify whether this is a problem for you.</source>
-                       </trans-unit>
                        <trans-unit id="status_xclassUsage">
                                <source>XCLASS used</source>
                        </trans-unit>
index 299b470..6ef237d 100644 (file)
@@ -30,31 +30,6 @@ class Typo3StatusTest extends \TYPO3\CMS\Core\Tests\UnitTestCase
     /**
      * @test
      */
-    public function getStatusReturnsOldXclassStatusObjectWithSeverityOkIfNoOldXclassExists()
-    {
-        $GLOBALS['TYPO3_CONF_VARS']['BE']['XCLASS'] = array();
-        $GLOBALS['TYPO3_CONF_VARS']['FE']['XCLASS'] = array();
-        $fixture = new \TYPO3\CMS\Reports\Report\Status\Typo3Status;
-        $result = $fixture->getStatus();
-        $statusObject = $result['oldXclassStatus'];
-        $this->assertSame(\TYPO3\CMS\Reports\Status::OK, $statusObject->getSeverity());
-    }
-
-    /**
-     * @test
-     */
-    public function getStatusReturnsOldXclassStatusObjectWithSeverityNoticeIfOldXclassExists()
-    {
-        $GLOBALS['TYPO3_CONF_VARS']['BE']['XCLASS'] = array('foo' => 'bar');
-        $fixture = new \TYPO3\CMS\Reports\Report\Status\Typo3Status;
-        $result = $fixture->getStatus();
-        $statusObject = $result['oldXclassStatus'];
-        $this->assertSame(\TYPO3\CMS\Reports\Status::NOTICE, $statusObject->getSeverity());
-    }
-
-    /**
-     * @test
-     */
     public function getStatusReturnsXclassStatusObjectWithSeverityOkIfNoXclassExists()
     {
         $GLOBALS['TYPO3_CONF_VARS']['SYS']['Objects'] = array();