[TASK] Use strict comparison for strings 62/51462/6
authorGeorg Ringer <georg.ringer@gmail.com>
Mon, 30 Jan 2017 06:42:37 +0000 (07:42 +0100)
committerBenni Mack <benni@typo3.org>
Wed, 1 Feb 2017 06:20:23 +0000 (07:20 +0100)
Change string comparison for string checks. Checks which could lead
to false positives like comparing with '0' or '' are excluded.
RegEx used to find affected places: (?<=\s)((?:!|=)=)(?=[^=]'[^0-9'\n]+')

Resolves: #79540
Resolves: master
Change-Id: I9a04bdd9e0e78cc97036560709bb505097bf48f9
Reviewed-on: https://review.typo3.org/51462
Reviewed-by: Markus Klein <markus.klein@typo3.org>
Tested-by: Markus Klein <markus.klein@typo3.org>
Tested-by: TYPO3com <no-reply@typo3.com>
Reviewed-by: Stefan Neufeind <typo3.neufeind@speedpartner.de>
Reviewed-by: Benni Mack <benni@typo3.org>
Tested-by: Benni Mack <benni@typo3.org>
110 files changed:
typo3/sysext/backend/Classes/Backend/ToolbarItems/ShortcutToolbarItem.php
typo3/sysext/backend/Classes/Clipboard/Clipboard.php
typo3/sysext/backend/Classes/Configuration/TypoScript/ConditionMatching/ConditionMatcher.php
typo3/sysext/backend/Classes/Controller/ContentElement/ElementInformationController.php
typo3/sysext/backend/Classes/Controller/ContentElement/MoveElementController.php
typo3/sysext/backend/Classes/Controller/ContentElement/NewContentElementController.php
typo3/sysext/backend/Classes/Controller/EditDocumentController.php
typo3/sysext/backend/Classes/Controller/FormInlineAjaxController.php
typo3/sysext/backend/Classes/Form/Container/InlineRecordContainer.php
typo3/sysext/backend/Classes/Form/Element/SelectCheckBoxElement.php
typo3/sysext/backend/Classes/Form/FormDataProvider/AbstractItemProvider.php
typo3/sysext/backend/Classes/Form/FormDataProvider/TcaInline.php
typo3/sysext/backend/Classes/Form/Wizard/SuggestWizardDefaultReceiver.php
typo3/sysext/backend/Classes/History/RecordHistory.php
typo3/sysext/backend/Classes/Utility/BackendUtility.php
typo3/sysext/backend/Classes/View/PageLayoutView.php
typo3/sysext/beuser/Classes/Controller/BackendUserActionController.php
typo3/sysext/compatibility7/Classes/Controller/SearchFormController.php
typo3/sysext/compatibility7/Classes/Hooks/EnforceUrlSchemeHook.php
typo3/sysext/compatibility7/Classes/Hooks/PageLayoutActionHook.php
typo3/sysext/core/Classes/Authentication/AbstractUserAuthentication.php
typo3/sysext/core/Classes/Authentication/BackendUserAuthentication.php
typo3/sysext/core/Classes/Cache/Backend/SimpleFileBackend.php
typo3/sysext/core/Classes/Configuration/FlexForm/FlexFormTools.php
typo3/sysext/core/Classes/Configuration/TypoScript/ConditionMatching/AbstractConditionMatcher.php
typo3/sysext/core/Classes/Controller/CommandLineController.php
typo3/sysext/core/Classes/DataHandling/DataHandler.php
typo3/sysext/core/Classes/Database/QueryGenerator.php
typo3/sysext/core/Classes/Database/QueryView.php
typo3/sysext/core/Classes/Database/RelationHandler.php
typo3/sysext/core/Classes/FrontendEditing/FrontendEditingController.php
typo3/sysext/core/Classes/Html/HtmlParser.php
typo3/sysext/core/Classes/Html/RteHtmlParser.php
typo3/sysext/core/Classes/Http/ServerRequestFactory.php
typo3/sysext/core/Classes/Imaging/GraphicalFunctions.php
typo3/sysext/core/Classes/Integrity/DatabaseIntegrityCheck.php
typo3/sysext/core/Classes/Mail/Rfc822AddressesParser.php
typo3/sysext/core/Classes/Resource/Driver/LocalDriver.php
typo3/sysext/core/Classes/Resource/ResourceFactory.php
typo3/sysext/core/Classes/TimeTracker/TimeTracker.php
typo3/sysext/core/Classes/TypoScript/ExtendedTemplateService.php
typo3/sysext/core/Classes/TypoScript/TemplateService.php
typo3/sysext/core/Classes/Utility/ClientUtility.php
typo3/sysext/core/Classes/Utility/ExtensionManagementUtility.php
typo3/sysext/core/Classes/Utility/GeneralUtility.php
typo3/sysext/core/Classes/Utility/MathUtility.php
typo3/sysext/core/Tests/Unit/Resource/Driver/Fixtures/LocalDriverFilenameFilter.php
typo3/sysext/core/Tests/Unit/Resource/Driver/LocalDriverTest.php
typo3/sysext/css_styled_content/Classes/Controller/CssStyledContentController.php
typo3/sysext/extbase/Classes/Persistence/Generic/Storage/Typo3DbBackend.php
typo3/sysext/extbase/Tests/Unit/Persistence/Generic/Storage/Typo3DbQueryParserTest.php
typo3/sysext/extensionmanager/Classes/Utility/Connection/TerUtility.php
typo3/sysext/extensionmanager/Classes/Utility/Parser/ExtensionXmlPullParser.php
typo3/sysext/extensionmanager/Classes/Utility/Parser/MirrorXmlPullParser.php
typo3/sysext/extensionmanager/Classes/ViewHelpers/DownloadExtensionViewHelper.php
typo3/sysext/feedit/Classes/FrontendEditPanel.php
typo3/sysext/felogin/Classes/Controller/FrontendLoginController.php
typo3/sysext/filelist/Classes/Controller/FileListController.php
typo3/sysext/filelist/Classes/FileList.php
typo3/sysext/frontend/Classes/Configuration/TypoScript/ConditionMatching/ConditionMatcher.php
typo3/sysext/frontend/Classes/ContentObject/ContentObjectRenderer.php
typo3/sysext/frontend/Classes/ContentObject/Menu/AbstractMenuContentObject.php
typo3/sysext/frontend/Classes/ContentObject/RecordsContentObject.php
typo3/sysext/frontend/Classes/Controller/TypoScriptFrontendController.php
typo3/sysext/frontend/Classes/Imaging/GifBuilder.php
typo3/sysext/frontend/Classes/Page/PageGenerator.php
typo3/sysext/frontend/Classes/Plugin/AbstractPlugin.php
typo3/sysext/frontend/Classes/View/AdminPanelView.php
typo3/sysext/impexp/Classes/Clickmenu.php
typo3/sysext/impexp/Classes/Controller/ImportExportController.php
typo3/sysext/impexp/Classes/Export.php
typo3/sysext/impexp/Classes/Import.php
typo3/sysext/impexp/Classes/ImportExport.php
typo3/sysext/impexp/Tests/Functional/Export/AbstractExportTestCase.php
typo3/sysext/indexed_search/Classes/Controller/SearchController.php
typo3/sysext/indexed_search/Classes/Domain/Repository/IndexSearchRepository.php
typo3/sysext/indexed_search/Classes/FileContentParser.php
typo3/sysext/indexed_search/Classes/Indexer.php
typo3/sysext/indexed_search/Classes/Lexer.php
typo3/sysext/indexed_search/Classes/Utility/DoubleMetaPhoneUtility.php
typo3/sysext/install/Classes/Controller/Action/Tool/CleanUp.php
typo3/sysext/install/Classes/Service/SqlSchemaMigrationService.php
typo3/sysext/lang/Classes/Service/TerService.php
typo3/sysext/linkvalidator/Classes/LinkAnalyzer.php
typo3/sysext/lowlevel/Classes/View/DatabaseIntegrityView.php
typo3/sysext/recordlist/Classes/RecordList.php
typo3/sysext/recordlist/Classes/RecordList/AbstractDatabaseRecordList.php
typo3/sysext/recordlist/Classes/RecordList/DatabaseRecordList.php
typo3/sysext/reports/Classes/Report/Status/ConfigurationStatus.php
typo3/sysext/reports/Classes/Task/SystemStatusUpdateTaskNotificationEmailField.php
typo3/sysext/rtehtmlarea/Classes/Controller/ParseHtmlController.php
typo3/sysext/rtehtmlarea/Classes/Controller/SpellCheckingController.php
typo3/sysext/rtehtmlarea/Classes/Controller/UserElementsController.php
typo3/sysext/rtehtmlarea/Classes/Extension/BlockElements.php
typo3/sysext/rtehtmlarea/Classes/Extension/InlineElements.php
typo3/sysext/rtehtmlarea/Classes/Extension/SelectFont.php
typo3/sysext/rtehtmlarea/Classes/RteHtmlAreaApi.php
typo3/sysext/scheduler/Classes/Controller/SchedulerModuleController.php
typo3/sysext/scheduler/Classes/Example/TestTask.php
typo3/sysext/setup/Classes/Controller/SetupModuleController.php
typo3/sysext/sys_action/Classes/ActionTask.php
typo3/sysext/taskcenter/Classes/Controller/TaskModuleController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateConstantEditorModuleFunctionController.php
typo3/sysext/tstemplate/Classes/Controller/TypoScriptTemplateObjectBrowserModuleFunctionController.php
typo3/sysext/version/Classes/Controller/VersionModuleController.php
typo3/sysext/version/Classes/Hook/DataHandlerHook.php
typo3/sysext/wizard_crpages/Classes/Controller/CreatePagesWizardModuleFunctionController.php
typo3/sysext/workspaces/Classes/Controller/Remote/RemoteServer.php
typo3/sysext/workspaces/Classes/Hook/PreviewHook.php
typo3/sysext/workspaces/Classes/Service/GridDataService.php

index e089707..5b258e9 100644 (file)
@@ -544,10 +544,10 @@ class ShortcutToolbarItem implements ToolbarItemInterface
                 $shortcut['table'] = key($queryParameters['edit']);
                 $shortcut['recordid'] = key($queryParameters['edit'][$shortcut['table']]);
                 $shortcut['pid'] = BackendUtility::getRecord($shortcut['table'], $shortcut['recordid'])['pid'];
-                if ($queryParameters['edit'][$shortcut['table']][$shortcut['recordid']] == 'edit') {
+                if ($queryParameters['edit'][$shortcut['table']][$shortcut['recordid']] === 'edit') {
                     $shortcut['type'] = 'edit';
                     $shortcutNamePrepend = htmlspecialchars($languageService->getLL('shortcut_edit'));
-                } elseif ($queryParameters['edit'][$shortcut['table']][$shortcut['recordid']] == 'new') {
+                } elseif ($queryParameters['edit'][$shortcut['table']][$shortcut['recordid']] === 'new') {
                     $shortcut['type'] = 'new';
                     $shortcutNamePrepend = htmlspecialchars($languageService->getLL('shortcut_create'));
                 }
@@ -874,7 +874,7 @@ class ShortcutToolbarItem implements ToolbarItemInterface
     protected function getShortcutIconTitle($shortcutLabel, $moduleName, $parentModuleName = '')
     {
         $languageService = $this->getLanguageService();
-        if (substr($moduleName, 0, 5) == 'xMOD_') {
+        if (substr($moduleName, 0, 5) === 'xMOD_') {
             $title = substr($moduleName, 5);
         } else {
             list($mainModule, $subModule) = explode('_', $moduleName);
index 086944e..00ff2ae 100644 (file)
@@ -161,7 +161,7 @@ class Clipboard
     {
         if (is_array($cmd['el'])) {
             foreach ($cmd['el'] as $k => $v) {
-                if ($this->current == 'normal') {
+                if ($this->current === 'normal') {
                     unset($this->clipData['normal']);
                 }
                 if ($v) {
@@ -206,7 +206,7 @@ class Clipboard
             if (isset($this->clipData[$padIdent])) {
                 $this->clipData['current'] = ($this->current = $padIdent);
             }
-            if ($this->current != 'normal' || !$this->isElements()) {
+            if ($this->current !== 'normal' || !$this->isElements()) {
                 $this->clipData[$this->current]['mode'] = '';
             }
             // Setting mode to default (move) if no items on it or if not 'normal'
@@ -348,7 +348,7 @@ class Clipboard
             'label' => 'labels.normal',
             'padding' => $this->padTitle('normal')
         ];
-        if ($this->current == 'normal') {
+        if ($this->current === 'normal') {
             $tabArray['normal']['content'] = $this->getContentFromTab('normal');
         }
         // Print header and content for the NUMERIC tabs:
@@ -523,7 +523,7 @@ class Clipboard
     {
         $el = count($this->elFromTable($this->fileMode ? '_FILE' : '', $pad));
         if ($el) {
-            return ' (' . ($pad == 'normal' ? ($this->clipData['normal']['mode'] == 'copy' ? $this->clLabel('copy', 'cm') : $this->clLabel('cut', 'cm')) : htmlspecialchars($el)) . ')';
+            return ' (' . ($pad === 'normal' ? ($this->clipData['normal']['mode'] === 'copy' ? $this->clLabel('copy', 'cm') : $this->clLabel('cut', 'cm')) : htmlspecialchars($el)) . ')';
         }
         return '';
     }
@@ -710,19 +710,19 @@ class Clipboard
     public function confirmMsgText($table, $rec, $type, $clElements, $columnLabel = '')
     {
         if ($this->getBackendUser()->jsConfirmation(JsConfirmation::COPY_MOVE_PASTE)) {
-            $labelKey = 'LLL:EXT:lang/Resources/Private/Language/locallang_core.xlf:mess.' . ($this->currentMode() == 'copy' ? 'copy' : 'move') . ($this->current == 'normal' ? '' : 'cb') . '_' . $type;
+            $labelKey = 'LLL:EXT:lang/Resources/Private/Language/locallang_core.xlf:mess.' . ($this->currentMode() === 'copy' ? 'copy' : 'move') . ($this->current === 'normal' ? '' : 'cb') . '_' . $type;
             $msg = $this->getLanguageService()->sL($labelKey . ($columnLabel ? '_colPos': ''));
-            if ($table == '_FILE') {
+            if ($table === '_FILE') {
                 $thisRecTitle = basename($rec);
-                if ($this->current == 'normal') {
+                if ($this->current === 'normal') {
                     $selItem = reset($clElements);
                     $selRecTitle = basename($selItem);
                 } else {
                     $selRecTitle = count($clElements);
                 }
             } else {
-                $thisRecTitle = $table == 'pages' && !is_array($rec) ? $GLOBALS['TYPO3_CONF_VARS']['SYS']['sitename'] : BackendUtility::getRecordTitle($table, $rec);
-                if ($this->current == 'normal') {
+                $thisRecTitle = $table === 'pages' && !is_array($rec) ? $GLOBALS['TYPO3_CONF_VARS']['SYS']['sitename'] : BackendUtility::getRecordTitle($table, $rec);
+                if ($this->current === 'normal') {
                     $selItem = $this->getSelectedRecord();
                     $selRecTitle = $selItem['_RECORD_TITLE'];
                 } else {
@@ -779,7 +779,7 @@ class Clipboard
                 if ($v) {
                     list($table, $uid) = explode('|', $k);
                     // Rendering files/directories on the clipboard
-                    if ($table == '_FILE') {
+                    if ($table === '_FILE') {
                         if (file_exists($v) && GeneralUtility::isAllowedAbsPath($v)) {
                             $params['tx_impexp'][is_dir($v) ? 'dir' : 'file'][] = $v;
                         }
@@ -832,7 +832,7 @@ class Clipboard
      */
     public function currentMode()
     {
-        return $this->clipData[$this->current]['mode'] == 'copy' ? 'copy' : 'cut';
+        return $this->clipData[$this->current]['mode'] === 'copy' ? 'copy' : 'cut';
     }
 
     /**
@@ -846,7 +846,7 @@ class Clipboard
         if (is_array($this->clipData[$this->current]['el'])) {
             foreach ($this->clipData[$this->current]['el'] as $k => $v) {
                 list($table, $uid) = explode('|', $k);
-                if ($table != '_FILE') {
+                if ($table !== '_FILE') {
                     if (!$v || !is_array(BackendUtility::getRecord($table, $uid, 'uid'))) {
                         unset($this->clipData[$this->current]['el'][$k]);
                         $this->changed = 1;
@@ -883,9 +883,9 @@ class Clipboard
             foreach ($this->clipData[$pad]['el'] as $k => $v) {
                 if ($v) {
                     list($table, $uid) = explode('|', $k);
-                    if ($table != '_FILE') {
+                    if ($table !== '_FILE') {
                         if ((!$matchTable || (string)$table == (string)$matchTable) && $GLOBALS['TCA'][$table]) {
-                            $list[$k] = $pad == 'normal' ? $v : $uid;
+                            $list[$k] = $pad === 'normal' ? $v : $uid;
                         }
                     } else {
                         if ((string)$table == (string)$matchTable) {
@@ -909,7 +909,7 @@ class Clipboard
     public function isSelected($table, $uid)
     {
         $k = $table . '|' . $uid;
-        return $this->clipData[$this->current]['el'][$k] ? ($this->current == 'normal' ? $this->currentMode() : 1) : '';
+        return $this->clipData[$this->current]['el'][$k] ? ($this->current === 'normal' ? $this->currentMode() : 1) : '';
     }
 
     /**
@@ -979,7 +979,7 @@ class Clipboard
             $elements = $this->elFromTable($pTable);
             // So the order is preserved.
             $elements = array_reverse($elements);
-            $mode = $this->currentMode() == 'copy' ? 'copy' : 'move';
+            $mode = $this->currentMode() === 'copy' ? 'copy' : 'move';
             // Traverse elements and make CMD array
             foreach ($elements as $tP => $value) {
                 list($table, $uid) = explode('|', $tP);
@@ -995,7 +995,7 @@ class Clipboard
                 } else {
                     $CMD[$table][$uid][$mode] = $pUid;
                 }
-                if ($mode == 'move') {
+                if ($mode === 'move') {
                     $this->removeElement($tP);
                 }
             }
@@ -1043,11 +1043,11 @@ class Clipboard
     {
         list($pTable, $pUid) = explode('|', $ref);
         $elements = $this->elFromTable('_FILE');
-        $mode = $this->currentMode() == 'copy' ? 'copy' : 'move';
+        $mode = $this->currentMode() === 'copy' ? 'copy' : 'move';
         // Traverse elements and make CMD array
         foreach ($elements as $tP => $path) {
             $FILE[$mode][] = ['data' => $path, 'target' => $pUid];
-            if ($mode == 'move') {
+            if ($mode === 'move') {
                 $this->removeElement($tP);
             }
         }
index 82f4985..b573f01 100644 (file)
@@ -79,7 +79,7 @@ class ConditionMatcher extends AbstractConditionMatcher
                 case 'PIDupinRootline':
                 case 'PIDinRootline':
                     $values = GeneralUtility::trimExplode(',', $value, true);
-                    if ($key == 'PIDinRootline' || !in_array($this->pageId, $values) || $this->isNewPageWithPageId($this->pageId)) {
+                    if ($key === 'PIDinRootline' || !in_array($this->pageId, $values) || $this->isNewPageWithPageId($this->pageId)) {
                         foreach ($values as $test) {
                             foreach ($this->rootline as $rl_dat) {
                                 if ($rl_dat['uid'] == $test) {
index f91960b..7641696 100644 (file)
@@ -144,7 +144,7 @@ class ElementInformationController
 
         // Check permissions and uid value:
         if ($this->uid && $this->getBackendUser()->check('tables_select', $this->table)) {
-            if ((string)$this->table == 'pages') {
+            if ((string)$this->table === 'pages') {
                 $this->pageInfo = BackendUtility::readPageAccess($this->uid, $this->permsClause);
                 $this->access = is_array($this->pageInfo) ? 1 : 0;
                 $this->row = $this->pageInfo;
index 5e28212..e3d7443 100644 (file)
@@ -149,7 +149,7 @@ class MoveElementController extends AbstractModule
             $assigns['makeCopyChecked'] = $this->makeCopy ? ' checked="checked"' : '';
             $assigns['makeCopyOnClick'] = $onClick;
             // IF the table is "pages":
-            if ((string)$this->table == 'pages') {
+            if ((string)$this->table === 'pages') {
                 // Get page record (if accessible):
                 $pageInfo = BackendUtility::readPageAccess($this->page_id, $this->perms_clause);
                 if (is_array($pageInfo) && $backendUser->isInWebMount($pageInfo['pid'], $this->perms_clause)) {
@@ -176,7 +176,7 @@ class MoveElementController extends AbstractModule
                 }
             }
             // IF the table is "tt_content":
-            if ((string)$this->table == 'tt_content') {
+            if ((string)$this->table === 'tt_content') {
                 // First, get the record:
                 $tt_content_rec = BackendUtility::getRecord('tt_content', $this->moveUid);
                 // ?
@@ -248,12 +248,12 @@ class MoveElementController extends AbstractModule
     {
         $buttonBar = $this->moduleTemplate->getDocHeaderComponent()->getButtonBar();
         if ($this->page_id) {
-            if ((string)$this->table == 'pages') {
+            if ((string)$this->table === 'pages') {
                 $cshButton = $buttonBar->makeHelpButton()
                     ->setModuleName('xMOD_csh_corebe')
                     ->setFieldName('move_el_pages');
                 $buttonBar->addButton($cshButton);
-            } elseif ((string)$this->table == 'tt_content') {
+            } elseif ((string)$this->table === 'tt_content') {
                 $cshButton = $buttonBar->makeHelpButton()
                     ->setModuleName('xMOD_csh_corebe')
                     ->setFieldName('move_el_cs');
index 57028ba..f2b17b6 100644 (file)
@@ -491,7 +491,7 @@ class NewContentElementController extends AbstractModule
                     if (is_array($GLOBALS['TCA']['tt_content']['columns'][$fN])) {
                         // Get information about if the field value is OK:
                         $config = &$GLOBALS['TCA']['tt_content']['columns'][$fN]['config'];
-                        $authModeDeny = $config['type'] == 'select' && $config['authMode']
+                        $authModeDeny = $config['type'] === 'select' && $config['authMode']
                             && !$backendUser->checkAuthMode('tt_content', $fN, $fV, $config['authMode']);
                         // explode TSconfig keys only as needed
                         if (!isset($removeItems[$fN])) {
index e543a01..a7b280d 100644 (file)
@@ -607,7 +607,7 @@ class EditDocumentController extends AbstractModule
                             }
                             // Traverse all new records and forge the content of ->editconf so we can continue to EDIT
                             // these records!
-                            if ($tableName == 'pages'
+                            if ($tableName === 'pages'
                                 && $this->retUrl != BackendUtility::getModuleUrl('dummy')
                                 && $this->returnNewPageId
                             ) {
@@ -649,7 +649,7 @@ class EditDocumentController extends AbstractModule
                 $nRec = BackendUtility::getRecord($nTable, $nUid, $recordFields);
                 // Determine insertion mode ('top' is self-explaining,
                 // otherwise new elements are inserted after one using a negative uid)
-                $insertRecordOnTop = ($this->getNewIconMode($nTable) == 'top');
+                $insertRecordOnTop = ($this->getNewIconMode($nTable) === 'top');
                 // Setting a blank editconf array for a new record:
                 $this->editconf = [];
                 // Determine related page ID for regular live context
@@ -1026,7 +1026,7 @@ class EditDocumentController extends AbstractModule
             if (is_array($conf) && $GLOBALS['TCA'][$table] && $beUser->check('tables_modify', $table)) {
                 // Traverse the keys/comments of each table (keys can be a commalist of uids)
                 foreach ($conf as $cKey => $command) {
-                    if ($command == 'edit' || $command == 'new') {
+                    if ($command === 'edit' || $command === 'new') {
                         // Get the ids:
                         $ids = GeneralUtility::trimExplode(',', $cKey, true);
                         // Traverse the ids:
@@ -1067,7 +1067,7 @@ class EditDocumentController extends AbstractModule
                                 ) {
                                     $this->viewId = $formData['parentPageRow']['uid'];
                                 } else {
-                                    if ($table == 'pages') {
+                                    if ($table === 'pages') {
                                         $this->viewId = $formData['databaseRow']['uid'];
                                     } elseif (!empty($formData['parentPageRow']['uid'])) {
                                         $this->viewId = $formData['parentPageRow']['uid'];
@@ -1154,7 +1154,7 @@ class EditDocumentController extends AbstractModule
                                 $this->formResultCompiler->mergeResult($formResult);
 
                                 // Seems the pid is set as hidden field (again) at end?!
-                                if ($command == 'new') {
+                                if ($command === 'new') {
                                     // @todo: looks ugly
                                     $html .= LF
                                         . '<input type="hidden"'
@@ -1784,7 +1784,7 @@ class EditDocumentController extends AbstractModule
                     // Traverse the keys/comments of each table (keys can be a commalist of uids)
                     $newConf = [];
                     foreach ($conf as $cKey => $cmd) {
-                        if ($cmd == 'edit') {
+                        if ($cmd === 'edit') {
                             // Traverse the ids:
                             $ids = GeneralUtility::trimExplode(',', $cKey, true);
                             foreach ($ids as $idKey => $theUid) {
index 12f6bc9..df048ed 100644 (file)
@@ -900,7 +900,7 @@ class FormInlineAjaxController extends AbstractFormEngineAjaxController
             $childConfig = $childConfig[$flexFormNode];
 
             // Skip to the field configuration of a sheet
-            if (isset($childConfig['ROOT']) && $childConfig['ROOT']['type'] == 'array') {
+            if (isset($childConfig['ROOT']) && $childConfig['ROOT']['type'] === 'array') {
                 $childConfig = $childConfig['ROOT']['el'];
             }
         }
index bb36e26..d201118 100644 (file)
@@ -384,7 +384,7 @@ class InlineRecordContainer extends AbstractContainer
         // Initialize:
         $cells = [];
         $additionalCells = [];
-        $isNewItem = substr($rec['uid'], 0, 3) == 'NEW';
+        $isNewItem = substr($rec['uid'], 0, 3) === 'NEW';
         $isParentExisting = MathUtility::canBeInterpretedAsInteger($data['inlineParentUid']);
         $tcaTableCtrl = $GLOBALS['TCA'][$foreignTable]['ctrl'];
         $tcaTableCols = $GLOBALS['TCA'][$foreignTable]['columns'];
index c20010c..64838ea 100644 (file)
@@ -76,7 +76,7 @@ class SelectCheckBoxElement extends AbstractFormElement
                     // Non-selectable element:
                     if ($p[1] === '--div--') {
                         $selIcon = '';
-                        if (isset($p[2]) && $p[2] != 'empty-empty') {
+                        if (isset($p[2]) && $p[2] !== 'empty-empty') {
                             $selIcon = FormEngineUtility::getIconHtml($p[2]);
                         }
                         $currentGroup++;
index 1a7d671..14f4ed0 100644 (file)
@@ -819,7 +819,7 @@ abstract class AbstractItemProvider
         $flexFormTools = GeneralUtility::makeInstance(FlexFormTools::class);
         $flexForms = [];
         foreach ($GLOBALS['TCA'][$table]['columns'] as $tableField => $fieldConf) {
-            if (!empty($fieldConf['config']['type']) && !empty($fieldConf['config']['ds']) && $fieldConf['config']['type'] == 'flex') {
+            if (!empty($fieldConf['config']['type']) && !empty($fieldConf['config']['ds']) && $fieldConf['config']['type'] === 'flex') {
                 $flexForms[$tableField] = [];
                 foreach (array_keys($fieldConf['config']['ds']) as $flexFormKey) {
                     // Get extension identifier (uses second value if it's not empty, "list" or "*", else first one)
index 3b85350..34d6c98 100644 (file)
@@ -97,7 +97,7 @@ class TcaInline extends AbstractDatabaseRecordProvider implements FormDataProvid
             $table = $result['tableName'];
             $row = $result['databaseRow'];
             // If the parent is a page, use the uid(!) of the (new?) page as pid for the child records:
-            if ($table == 'pages') {
+            if ($table === 'pages') {
                 $liveVersionId = BackendUtility::getLiveVersionIdOfRecord('pages', $row['uid']);
                 $pid = is_null($liveVersionId) ? $row['uid'] : $liveVersionId;
             } elseif ($row['pid'] < 0) {
index 03beae1..09b5069 100644 (file)
@@ -118,7 +118,7 @@ class SuggestWizardDefaultReceiver
         if (isset($config['maxItemsInResultList'])) {
             $this->maxItems = $config['maxItemsInResultList'];
         }
-        if ($this->table == 'pages') {
+        if ($this->table === 'pages') {
             $this->queryBuilder->andWhere(
                 QueryHelper::stripLogicalOperatorPrefix($GLOBALS['BE_USER']->getPagePermsClause(1))
             );
@@ -328,7 +328,7 @@ class SuggestWizardDefaultReceiver
     {
         $retValue = true;
         $table = $this->mmForeignTable ?: $this->table;
-        if ($table == 'pages') {
+        if ($table === 'pages') {
             if (!BackendUtility::readPageAccess($uid, $GLOBALS['BE_USER']->getPagePermsClause(1))) {
                 $retValue = false;
             }
@@ -369,7 +369,7 @@ class SuggestWizardDefaultReceiver
     protected function getRecordPath(&$row, $uid)
     {
         $titleLimit = max($this->config['maxPathTitleLength'], 0);
-        if (($this->mmForeignTable ? $this->mmForeignTable : $this->table) == 'pages') {
+        if (($this->mmForeignTable ? $this->mmForeignTable : $this->table) === 'pages') {
             $path = BackendUtility::getRecordPath($uid, '', $titleLimit);
             // For pages we only want the first (n-1) parts of the path,
             // because the n-th part is the page itself
index 3bc5879..6ec79da 100644 (file)
@@ -656,7 +656,7 @@ class RecordHistory
 
         $changeLog = $this->getHistoryData($elParts[0], $elParts[1]);
         // get history of tables of this page and merge it into changelog
-        if ($elParts[0] == 'pages' && $this->showSubElements && $this->hasPageAccess('pages', $elParts[1])) {
+        if ($elParts[0] === 'pages' && $this->showSubElements && $this->hasPageAccess('pages', $elParts[1])) {
             foreach ($GLOBALS['TCA'] as $tablename => $value) {
                 // check if there are records on the page
                 $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable($tablename);
index acbcc76..1b8dd36 100644 (file)
@@ -991,7 +991,7 @@ class BackendUtility
                 $srcPointer = 'default';
             }
             // Get Data Source: Detect if it's a file reference and in that case read the file and parse as XML. Otherwise the value is expected to be XML.
-            if (substr($ds_array[$srcPointer], 0, 5) == 'FILE:') {
+            if (substr($ds_array[$srcPointer], 0, 5) === 'FILE:') {
                 $file = GeneralUtility::getFileAbsFileName(substr($ds_array[$srcPointer], 5));
                 if ($file && @is_file($file)) {
                     $dataStructArray = GeneralUtility::xml2array(file_get_contents($file));
@@ -1727,7 +1727,7 @@ class BackendUtility
                     }
 
                     $fileExtension = $fileObject->getExtension();
-                    if ($fileExtension == 'ttf'
+                    if ($fileExtension === 'ttf'
                         || GeneralUtility::inList($GLOBALS['TYPO3_CONF_VARS']['GFX']['imagefile_ext'], $fileExtension)
                     ) {
                         $imageUrl = $fileObject->process(
@@ -1884,14 +1884,14 @@ class BackendUtility
      */
     public static function getRecordIconAltText($row, $table = 'pages')
     {
-        if ($table == 'pages') {
+        if ($table === 'pages') {
             $out = self::titleAttribForPages($row, '', 0);
         } else {
             $out = !empty(trim($GLOBALS['TCA'][$table]['ctrl']['descriptionColumn'])) ? $row[$GLOBALS['TCA'][$table]['ctrl']['descriptionColumn']] . ' ' : '';
             $ctrl = $GLOBALS['TCA'][$table]['ctrl']['enablecolumns'];
             // Uid is added
             $out .= 'id=' . $row['uid'];
-            if ($table == 'pages' && $row['alias']) {
+            if ($table === 'pages' && $row['alias']) {
                 $out .= ' / ' . $row['alias'];
             }
             if (static::isTableWorkspaceEnabled($table) && $row['pid'] < 0) {
@@ -2651,7 +2651,7 @@ class BackendUtility
             if (is_array($GLOBALS['TCA'][$table])) {
                 if ($fN == $GLOBALS['TCA'][$table]['ctrl']['tstamp'] || $fN == $GLOBALS['TCA'][$table]['ctrl']['crdate']) {
                     $fVnew = self::datetime($fV);
-                } elseif ($fN == 'pid') {
+                } elseif ($fN === 'pid') {
                     // Fetches the path with no regard to the users permissions to select pages.
                     $fVnew = self::getRecordPath($fV, '1=1', 20);
                 } else {
@@ -2740,7 +2740,7 @@ class BackendUtility
                         case 'string':
 
                         case 'short':
-                            $formEl = '<input type="text" name="' . $dataPrefix . '[' . $fname . ']" value="' . $params[$fname] . '"' . static::getDocumentTemplate()->formWidth(($config[0] == 'short' ? 24 : 48)) . ' />';
+                            $formEl = '<input type="text" name="' . $dataPrefix . '[' . $fname . ']" value="' . $params[$fname] . '"' . static::getDocumentTemplate()->formWidth(($config[0] === 'short' ? 24 : 48)) . ' />';
                             break;
                         case 'check':
                             $formEl = '<input type="hidden" name="' . $dataPrefix . '[' . $fname . ']" value="0" /><input type="checkbox" name="' . $dataPrefix . '[' . $fname . ']" value="1"' . ($params[$fname] ? ' checked="checked"' : '') . ' />';
@@ -4201,7 +4201,7 @@ class BackendUtility
                 );
 
             // Look up the path:
-            if ($table == '_FILE') {
+            if ($table === '_FILE') {
                 if (!GeneralUtility::isFirstPartOfStr($ref, PATH_site)) {
                     return '';
                 }
index 0afc3cc..57ebc9e 100644 (file)
@@ -360,7 +360,7 @@ class PageLayoutView extends \TYPO3\CMS\Recordlist\RecordList\AbstractDatabaseRe
             foreach ($this->fieldArray as $field) {
                 if ($editIdList
                     && isset($GLOBALS['TCA']['pages']['columns'][$field])
-                    && $field != 'uid'
+                    && $field !== 'uid'
                     && !$this->pages_noEditColumns
                 ) {
                     $iTitle = sprintf(
@@ -393,7 +393,7 @@ class PageLayoutView extends \TYPO3\CMS\Recordlist\RecordList\AbstractDatabaseRe
                         $theData[$field] = '&nbsp;<strong>ID:</strong>';
                         break;
                     default:
-                        if (substr($field, 0, 6) == 'table_') {
+                        if (substr($field, 0, 6) === 'table_') {
                             $f2 = substr($field, 6);
                             if ($GLOBALS['TCA'][$f2]) {
                                 $theData[$field] = '&nbsp;' .
@@ -1357,7 +1357,7 @@ class PageLayoutView extends \TYPO3\CMS\Recordlist\RecordList\AbstractDatabaseRe
                     }
                     break;
                 default:
-                    if (substr($field, 0, 6) == 'table_') {
+                    if (substr($field, 0, 6) === 'table_') {
                         $f2 = substr($field, 6);
                         if ($GLOBALS['TCA'][$f2]) {
                             $c = $this->numberOfRecords($f2, $row['uid']);
index 92978b6..eab02dc 100644 (file)
@@ -54,9 +54,9 @@ class BackendUserActionController extends ActionController
     {
         /** @var BackendTemplateView $view */
         parent::initializeView($view);
-        if ($this->actionMethodName == 'indexAction'
-            || $this->actionMethodName == 'onlineAction'
-            || $this->actionMethodName == 'compareAction') {
+        if ($this->actionMethodName === 'indexAction'
+            || $this->actionMethodName === 'onlineAction'
+            || $this->actionMethodName === 'compareAction') {
             $this->generateMenu();
             $this->registerDocheaderButtons();
             $this->view->getModuleTemplate()->setFlashMessageQueue($this->controllerContext->getFlashMessageQueue());
index 7deafb2..0b432a2 100644 (file)
@@ -863,7 +863,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
                 $this->internal['results_at_a_time'] = $this->piVars['results'];
                 $this->internal['maxPages'] = MathUtility::forceIntegerInRange($this->conf['search.']['page_links'], 1, 100, 10);
                 $resultSectionsCount = count($this->resultSections);
-                $addString = $resData['count'] && $this->piVars['group'] == 'sections' && $freeIndexUid <= 0 ? ' ' . sprintf($this->pi_getLL(($resultSectionsCount > 1 ? 'inNsections' : 'inNsection')), $resultSectionsCount) : '';
+                $addString = $resData['count'] && $this->piVars['group'] === 'sections' && $freeIndexUid <= 0 ? ' ' . sprintf($this->pi_getLL(($resultSectionsCount > 1 ? 'inNsections' : 'inNsection')), $resultSectionsCount) : '';
                 $browseBox1 = $this->renderPagination(1, $addString, $this->printResultSectionLinks(), $freeIndexUid);
                 $browseBox2 = $this->renderPagination(0, '', '', $freeIndexUid);
                 // Browsing nav, bottom.
@@ -876,7 +876,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
             $content .= '<p' . $this->pi_classParam('noresults') . '>' . htmlspecialchars($this->pi_getLL('noResults')) . '</p>';
         }
         // Print a message telling which words we searched for, and in which sections etc.
-        $what = $this->tellUsWhatIsSeachedFor($sWArr) . (substr($this->piVars['sections'], 0, 2) == 'rl' ? ' ' . htmlspecialchars($this->pi_getLL('inSection')) . ' "' . $this->getPathFromPageId(substr($this->piVars['sections'], 4)) . '"' : '');
+        $what = $this->tellUsWhatIsSeachedFor($sWArr) . (substr($this->piVars['sections'], 0, 2) === 'rl' ? ' ' . htmlspecialchars($this->pi_getLL('inSection')) . ' "' . $this->getPathFromPageId(substr($this->piVars['sections'], 4)) . '"' : '');
         $what = '<div' . $this->pi_classParam('whatis') . '>' . $this->cObj->stdWrap($what, $this->conf['whatis_stdWrap.']) . '</div>';
         $content = $what . $content;
         // Return content:
@@ -920,7 +920,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
         if ($freeIndexUid <= 0) {
             switch ($this->piVars['group']) {
                 case 'sections':
-                    $rl2flag = substr($this->piVars['sections'], 0, 2) == 'rl';
+                    $rl2flag = substr($this->piVars['sections'], 0, 2) === 'rl';
                     $sections = [];
                     foreach ($resultRows as $row) {
                         $id = $row['rl0'] . '-' . $row['rl1'] . ($rl2flag ? '-' . $row['rl2'] : '');
@@ -932,7 +932,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
                         $theId = $rlParts[2] ? $rlParts[2] : ($rlParts[1] ? $rlParts[1] : $rlParts[0]);
                         $theRLid = $rlParts[2] ? 'rl2_' . $rlParts[2] : ($rlParts[1] ? 'rl1_' . $rlParts[1] : '0');
                         $sectionName = $this->getPathFromPageId($theId);
-                        if ($sectionName[0] == '/') {
+                        if ($sectionName[0] === '/') {
                             $sectionName = substr($sectionName, 1);
                         }
                         if (!trim($sectionName)) {
@@ -1167,11 +1167,11 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
     {
         $out = $this->wholeSiteIdList < 0 ? '' : ' AND ISEC.rl0 IN (' . $this->wholeSiteIdList . ')';
         $match = '';
-        if (substr($this->piVars['sections'], 0, 4) == 'rl1_') {
+        if (substr($this->piVars['sections'], 0, 4) === 'rl1_') {
             $list = implode(',', GeneralUtility::intExplode(',', substr($this->piVars['sections'], 4)));
             $out .= ' AND ISEC.rl1 IN (' . $list . ')';
             $match = true;
-        } elseif (substr($this->piVars['sections'], 0, 4) == 'rl2_') {
+        } elseif (substr($this->piVars['sections'], 0, 4) === 'rl2_') {
             $list = implode(',', GeneralUtility::intExplode(',', substr($this->piVars['sections'], 4)));
             $out .= ' AND ISEC.rl2 IN (' . $list . ')';
             $match = true;
@@ -1327,7 +1327,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
         // Indexing configuration clause:
         $freeIndexUidClause = $this->freeIndexUidWhere($freeIndexUid);
         // If any of the ranking sortings are selected, we must make a join with the word/rel-table again, because we need to calculate ranking based on all search-words found.
-        if (substr($this->piVars['order'], 0, 5) == 'rank_') {
+        if (substr($this->piVars['order'], 0, 5) === 'rank_') {
             switch ($this->piVars['order']) {
                 case 'rank_flag':
                     // This gives priority to word-position (max-value) so that words in title, keywords, description counts more than in content.
@@ -2032,7 +2032,7 @@ class SearchFormController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
             } else {
                 // Default creation / finding of icon:
                 $icon = '';
-                if ($it === '0' || substr($it, 0, 2) == '0:') {
+                if ($it === '0' || substr($it, 0, 2) === '0:') {
                     if (is_array($specRowConf['pageIcon.'])) {
                         $this->iconFileNameCache[$it] = $this->cObj->cObjGetSingle('IMAGE', $specRowConf['pageIcon.']);
                     } else {
index c1c0b97..0e752d2 100644 (file)
@@ -71,9 +71,9 @@ class EnforceUrlSchemeHook implements TypolinkModifyLinkConfigForPageLinksHookIn
         if (isset($parentObject->page['url_scheme']) && $parentObject->page['url_scheme'] > 0) {
             $newUrl = '';
             $requestUrlScheme = parse_url(GeneralUtility::getIndpEnv('TYPO3_REQUEST_URL'), PHP_URL_SCHEME);
-            if ((int)$parentObject->page['url_scheme'] === HttpUtility::SCHEME_HTTP && $requestUrlScheme == 'https') {
+            if ((int)$parentObject->page['url_scheme'] === HttpUtility::SCHEME_HTTP && $requestUrlScheme === 'https') {
                 $newUrl = 'http://' . substr(GeneralUtility::getIndpEnv('TYPO3_REQUEST_URL'), 8);
-            } elseif ((int)$parentObject->page['url_scheme'] === HttpUtility::SCHEME_HTTPS && $requestUrlScheme == 'http') {
+            } elseif ((int)$parentObject->page['url_scheme'] === HttpUtility::SCHEME_HTTPS && $requestUrlScheme === 'http') {
                 $newUrl = 'https://' . substr(GeneralUtility::getIndpEnv('TYPO3_REQUEST_URL'), 7);
             }
             if ($newUrl !== '') {
index 6a2d055..f40ffc5 100644 (file)
@@ -146,8 +146,8 @@ class PageLayoutActionHook
                 ];
                 $moveButton = $buttonBar->makeLinkButton()
                     ->setHref(BackendUtility::getModuleUrl('move_element', $urlParameters))
-                    ->setTitle($lang->getLL('move_' . ($this->eRParts[0] == 'tt_content' ? 'record' : 'page')))
-                    ->setIcon($iconFactory->getIcon('actions-' . ($this->eRParts[0] == 'tt_content' ? 'document' : 'page') . '-move', Icon::SIZE_SMALL));
+                    ->setTitle($lang->getLL('move_' . ($this->eRParts[0] === 'tt_content' ? 'record' : 'page')))
+                    ->setIcon($iconFactory->getIcon('actions-' . ($this->eRParts[0] === 'tt_content' ? 'document' : 'page') . '-move', Icon::SIZE_SMALL));
                 $buttonBar->addButton($moveButton, ButtonBar::BUTTON_POSITION_LEFT, 2);
             }
         }
@@ -411,7 +411,7 @@ class PageLayoutActionHook
         // Set the edit_record value for internal use in this function:
         $edit_record = $this->controller->edit_record;
         // If a command to edit all records in a column is issue, then select all those elements, and redirect to FormEngine
-        if (substr($edit_record, 0, 9) == '_EDIT_COL') {
+        if (substr($edit_record, 0, 9) === '_EDIT_COL') {
             $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable('tt_content');
             if ($this->controller->MOD_SETTINGS['tt_content_showHidden']) {
                 $queryBuilder->getRestrictions()->removeAll()->add(GeneralUtility::makeInstance(DeletedRestriction::class));
@@ -585,10 +585,10 @@ class PageLayoutActionHook
                 }
 
                 // Add hidden fields:
-                if ($uidVal == 'new') {
+                if ($uidVal === 'new') {
                     $panel .= '<input type="hidden" name="data[' . $tableName . '][' . $row['uid'] . '][pid]" value="' . $row['pid'] . '" />';
                 }
-                $redirect = ($uidVal == 'new' ? BackendUtility::getModuleUrl(
+                $redirect = ($uidVal === 'new' ? BackendUtility::getModuleUrl(
                     $this->moduleName,
                     ['id' => $this->controller->id, 'new_unique_uid' => $new_unique_uid, 'returnUrl' => $this->returnUrl]
                 ) : $this->R_URI);
index 08c14b4..29882d6 100644 (file)
@@ -403,7 +403,7 @@ abstract class AbstractUserAuthentication
         // Internal var 'id' is set
         $this->id = $id;
         // If fallback to get mode....
-        if ($mode == 'get' && $this->getFallBack && $this->get_name) {
+        if ($mode === 'get' && $this->getFallBack && $this->get_name) {
             $this->get_URL_ID = '&' . $this->get_name . '=' . $id;
         }
         // Make certain that NO user is set initially
@@ -506,7 +506,7 @@ abstract class AbstractUserAuthentication
             $cookieDomain = $GLOBALS['TYPO3_CONF_VARS'][$this->loginType]['cookieDomain'];
         }
         if ($cookieDomain) {
-            if ($cookieDomain[0] == '/') {
+            if ($cookieDomain[0] === '/') {
                 $match = [];
                 $matchCnt = @preg_match($cookieDomain, GeneralUtility::getIndpEnv('TYPO3_HOST_ONLY'), $match);
                 if ($matchCnt === false) {
@@ -622,7 +622,7 @@ abstract class AbstractUserAuthentication
             }
             // Refuse login for _CLI users, if not processing a CLI request type
             // (although we shouldn't be here in case of a CLI request type)
-            if (strtoupper(substr($loginData['uname'], 0, 5)) == '_CLI_' && !(TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI)) {
+            if (strtoupper(substr($loginData['uname'], 0, 5)) === '_CLI_' && !(TYPO3_REQUESTTYPE & TYPO3_REQUESTTYPE_CLI)) {
                 throw new \RuntimeException('TYPO3 Fatal Error: You have tried to login using a CLI user. Access prohibited!', 1270853931);
             }
         }
@@ -1296,7 +1296,7 @@ abstract class AbstractUserAuthentication
      */
     public function getModuleData($module, $type = '')
     {
-        if ($type != 'ses' || (isset($this->uc['moduleSessionID'][$module]) && $this->uc['moduleSessionID'][$module] == $this->id)) {
+        if ($type !== 'ses' || (isset($this->uc['moduleSessionID'][$module]) && $this->uc['moduleSessionID'][$module] == $this->id)) {
             return $this->uc['moduleData'][$module];
         }
         return null;
index 5d91ca9..48d8d46 100644 (file)
@@ -846,7 +846,7 @@ class BackendUserAuthentication extends \TYPO3\CMS\Core\Authentication\AbstractU
     {
         if ($this->isAdmin()) {
             $result = true;
-        } elseif ($tableName == 'pages') {
+        } elseif ($tableName === 'pages') {
             switch ($actionType) {
                 case 'edit':
                     $result = ($compiledPermissions & Permission::PAGE_EDIT) !== 0;
index 9975aa0..508a452 100644 (file)
@@ -130,7 +130,7 @@ class SimpleFileBackend extends \TYPO3\CMS\Core\Cache\Backend\AbstractBackend im
                 }
             } else {
                 $delimiter = ':';
-                if ($cacheDirectory[0] != '/') {
+                if ($cacheDirectory[0] !== '/') {
                     // relative path to cache directory.
                     $cacheDirectory = PATH_site . $cacheDirectory;
                 }
@@ -158,7 +158,7 @@ class SimpleFileBackend extends \TYPO3\CMS\Core\Cache\Backend\AbstractBackend im
                 );
             }
         } else {
-            if ($cacheDirectory[0] == '/') {
+            if ($cacheDirectory[0] === '/') {
                 // Absolute path to cache directory.
                 $documentRoot = '';
             }
index ca25544..88056cf 100644 (file)
@@ -811,7 +811,7 @@ class FlexFormTools
             foreach ($dataStruct as $key => $value) {
                 // The value of each entry must be an array.
                 if (is_array($value)) {
-                    if ($value['type'] == 'array') {
+                    if ($value['type'] === 'array') {
                         // Array (Section) traversal
                         if ($value['section']) {
                             $cc = 0;
index e6207d7..ac360e4 100644 (file)
@@ -293,7 +293,7 @@ abstract class AbstractConditionMatcher
                 if ($this->isUserLoggedIn()) {
                     $values = GeneralUtility::trimExplode(',', $value, true);
                     foreach ($values as $test) {
-                        if ($test == '*' || (string)$this->getUserId() === (string)$test) {
+                        if ($test === '*' || (string)$this->getUserId() === (string)$test) {
                             return true;
                         }
                     }
index c5955c2..7dee4a1 100644 (file)
@@ -160,7 +160,7 @@ class CommandLineController
                 foreach ($argSplit as $i => $v) {
                     $ii = $i;
                     if ($i > 0) {
-                        if (!isset($cli_args_copy[$argSplit[0]][$i - 1]) && $v[0] != '[') {
+                        if (!isset($cli_args_copy[$argSplit[0]][$i - 1]) && $v[0] !== '[') {
                             // Using "[]" around a parameter makes it optional
                             echo 'ERROR: Option "' . $argSplit[0] . '" requires a value ("' . $v . '") on position ' . $i . LF;
                             die;
index d67b74a..13eefbf 100644 (file)
@@ -1234,7 +1234,7 @@ class DataHandler
                     $newVersion_placeholderFieldArray = $this->overrideFieldArray($table, $newVersion_placeholderFieldArray);
                 }
                 // Setting system fields
-                if ($status == 'new') {
+                if ($status === 'new') {
                     if ($GLOBALS['TCA'][$table]['ctrl']['crdate']) {
                         $fieldArray[$GLOBALS['TCA'][$table]['ctrl']['crdate']] = $GLOBALS['EXEC_TIME'];
                         if ($createNewVersion) {
@@ -1270,7 +1270,7 @@ class DataHandler
                 // Performing insert/update. If fieldArray has been unset by some userfunction (see hook above), don't do anything
                 // Kasper: Unsetting the fieldArray is dangerous; MM relations might be saved already and files could have been uploaded that are now "lost"
                 if (is_array($fieldArray)) {
-                    if ($status == 'new') {
+                    if ($status === 'new') {
                         if ($table === 'pages') {
                             // for new pages always a refresh is needed
                             $this->pagetreeNeedsRefresh = true;
@@ -1475,7 +1475,7 @@ class DataHandler
                 case 'perms_group':
                 case 'perms_everybody':
                     // Permissions can be edited by the owner or the administrator
-                    if ($table == 'pages' && ($this->admin || $status == 'new' || $this->pageInfo($id, 'perms_userid') == $this->userid)) {
+                    if ($table === 'pages' && ($this->admin || $status === 'new' || $this->pageInfo($id, 'perms_userid') == $this->userid)) {
                         $value = (int)$fieldValue;
                         switch ($field) {
                             case 'perms_userid':
@@ -1573,7 +1573,7 @@ class DataHandler
                 }
                 return $res;
             }
-            if ($status == 'update') {
+            if ($status === 'update') {
                 // This checks 1) if we should check for disallowed tables and 2) if there are records from disallowed tables on the current page
                 $onlyAllowedTables = isset($GLOBALS['PAGES_TYPES'][$value]['onlyAllowedTables']) ? $GLOBALS['PAGES_TYPES'][$value]['onlyAllowedTables'] : $GLOBALS['PAGES_TYPES']['default']['onlyAllowedTables'];
                 if ($onlyAllowedTables) {
@@ -1946,7 +1946,7 @@ class DataHandler
             $valueArray = array_unique($valueArray);
         }
         // If an exclusive key is found, discard all others:
-        if ($tcaFieldConf['type'] == 'select' && $tcaFieldConf['exclusiveKeys']) {
+        if ($tcaFieldConf['type'] === 'select' && $tcaFieldConf['exclusiveKeys']) {
             $exclusiveKeys = GeneralUtility::trimExplode(',', $tcaFieldConf['exclusiveKeys']);
             foreach ($valueArray as $index => $key) {
                 if (in_array($key, $exclusiveKeys, true)) {
@@ -1959,7 +1959,7 @@ class DataHandler
         // NOTE!!! Must check max-items of files before the later check because that check would just leave out file names if there are too many!!
         $valueArray = $this->applyFiltersToValues($tcaFieldConf, $valueArray);
         // Checking for select / authMode, removing elements from $valueArray if any of them is not allowed!
-        if ($tcaFieldConf['type'] == 'select' && $tcaFieldConf['authMode']) {
+        if ($tcaFieldConf['type'] === 'select' && $tcaFieldConf['authMode']) {
             $preCount = count($valueArray);
             foreach ($valueArray as $index => $key) {
                 if (!$this->BE_USER->checkAuthMode($table, $field, $key, $tcaFieldConf['authMode'])) {
@@ -1972,7 +1972,7 @@ class DataHandler
             }
         }
         // For group types:
-        if ($tcaFieldConf['type'] == 'group') {
+        if ($tcaFieldConf['type'] === 'group') {
             switch ($tcaFieldConf['internal_type']) {
                 case 'file_reference':
                 case 'file':
@@ -2079,7 +2079,7 @@ class DataHandler
             $this->fileFunc->setFileExtensionPermissions($tcaFieldConf['allowed'], $tcaFieldConf['disallowed'] ?: '*');
         }
         // If there is an upload folder defined:
-        if ($tcaFieldConf['uploadfolder'] && $tcaFieldConf['internal_type'] == 'file') {
+        if ($tcaFieldConf['uploadfolder'] && $tcaFieldConf['internal_type'] === 'file') {
             $currentFilesForHistory = null;
             // If filehandling should NOT be bypassed, do processing:
             if (!$this->bypassFileHandling) {
@@ -2088,7 +2088,7 @@ class DataHandler
                 // Get destrination path:
                 $dest = $this->destPathFromUploadFolder($tcaFieldConf['uploadfolder']);
                 // If we are updating:
-                if ($status == 'update') {
+                if ($status === 'update') {
                     // Traverse the input values and convert to absolute filenames in case the update happens to an autoVersionized record.
                     // Background: This is a horrible workaround! The problem is that when a record is auto-versionized the files of the record get copied and therefore get new names which is overridden with the names from the original record in the incoming data meaning both lost files and double-references!
                     // The only solution I could come up with (except removing support for managing files when autoversioning) was to convert all relative files to absolute names so they are copied again (and existing files deleted). This should keep references intact but means that some files are copied, then deleted after being copied _again_.
@@ -2225,7 +2225,7 @@ class DataHandler
                     // Explode files
                     $dbAnalysis->itemArray[]['id'] = $theFile;
                 }
-                if ($status == 'update') {
+                if ($status === 'update') {
                     $dbAnalysis->writeMM($tcaFieldConf['MM'], $id, 0);
                     $newFiles = implode(',', $dbAnalysis->getValueArray());
                     list(, , $recFieldName) = explode(':', $recFID);
@@ -2427,7 +2427,7 @@ class DataHandler
         }
 
         foreach ($actionCMDs as $key => $value) {
-            if ($key == '_ACTION') {
+            if ($key === '_ACTION') {
                 // First, check if there are "commands":
                 if (current($actionCMDs[$key]) === '') {
                     continue;
@@ -2438,7 +2438,7 @@ class DataHandler
                 foreach ($actionCMDs[$key] as $idx => $order) {
                     // Just one reflection here: It is clear that when removing elements from a flexform, then we will get lost
                     // files unless we act on this delete operation by traversing and deleting files that were referred to.
-                    if ($order != 'DELETE') {
+                    if ($order !== 'DELETE') {
                         $newValueArray[$idx] = $valueArray[$idx];
                     }
                     unset($valueArray[$idx]);
@@ -2848,7 +2848,7 @@ class DataHandler
         } else {
             $tables = $tcaFieldConf['foreign_table'];
         }
-        $prep = $type == 'group' ? $tcaFieldConf['prepend_tname'] : '';
+        $prep = $type === 'group' ? $tcaFieldConf['prepend_tname'] : '';
         $newRelations = implode(',', $valueArray);
         /** @var $dbAnalysis RelationHandler */
         $dbAnalysis = $this->createRelationHandlerInstance();
@@ -2858,7 +2858,7 @@ class DataHandler
             // convert submitted items to use version ids instead of live ids
             // (only required for MM relations in a workspace context)
             $dbAnalysis->convertItemArray();
-            if ($status == 'update') {
+            if ($status === 'update') {
                 /** @var $oldRelations_dbAnalysis RelationHandler */
                 $oldRelations_dbAnalysis = $this->createRelationHandlerInstance();
                 $oldRelations_dbAnalysis->registerNonTableValues = !empty($tcaFieldConf['allowNonIdValues']);
@@ -2962,7 +2962,7 @@ class DataHandler
         // For each DS element:
         foreach ($DSelements as $key => $dsConf) {
             // Array/Section:
-            if ($DSelements[$key]['type'] == 'array') {
+            if ($DSelements[$key]['type'] === 'array') {
                 if (!is_array($dataValues[$key]['el'])) {
                     continue;
                 }
@@ -3019,7 +3019,7 @@ class DataHandler
                     }
                     // Finally, check if new and old values are different (or no .vDEFbase value is found) and if so, we record the vDEF value for diff'ing.
                     // We do this after $dataValues has been updated since I expect that $dataValues_current holds evaluated values from database (so this must be the right value to compare with).
-                    if (substr($vKey, -9) != '.vDEFbase') {
+                    if (substr($vKey, -9) !== '.vDEFbase') {
                         if ($this->updateModeL10NdiffData && $GLOBALS['TYPO3_CONF_VARS']['BE']['flexFormXMLincludeDiffBase'] && $vKey !== 'vDEF' && ((string)$dataValues[$key][$vKey] !== (string)$dataValues_current[$key][$vKey] || !isset($dataValues_current[$key][$vKey . '.vDEFbase']) || $this->updateModeL10NdiffData === 'FORCE_FFUPD')) {
                             // Now, check if a vDEF value is submitted in the input data, if so we expect this has been processed prior to this operation (normally the case since those fields are higher in the form) and we can use that:
                             if (isset($dataValues[$key]['vDEF'])) {
@@ -3062,7 +3062,7 @@ class DataHandler
         $dbAnalysis->start(implode(',', $valueArray), $foreignTable, '', 0, $table, $tcaFieldConf);
         // If the localizationMode is set to 'keep', the children for the localized parent are kept as in the original untranslated record:
         $localizationMode = BackendUtility::getInlineLocalizationMode($table, $tcaFieldConf);
-        if ($localizationMode == 'keep' && $status == 'update') {
+        if ($localizationMode === 'keep' && $status === 'update') {
             // Fetch the current record and determine the original record:
             $row = BackendUtility::getRecordWSOL($table, $id);
             if (is_array($row)) {
@@ -3085,7 +3085,7 @@ class DataHandler
             $dbAnalysis->writeForeignField($tcaFieldConf, $id, 0, $skipSorting);
             $newValue = $keepTranslation ? 0 : $dbAnalysis->countItems(false);
         } else {
-            if ($this->getInlineFieldType($tcaFieldConf) == 'mm') {
+            if ($this->getInlineFieldType($tcaFieldConf) === 'mm') {
                 // In order to fully support all the MM stuff, directly call checkValue_group_select_processDBdata instead of repeating the needed code here
                 $valueArray = $this->checkValue_group_select_processDBdata($valueArray, $tcaFieldConf, $id, $status, 'select', $table, $field);
                 $newValue = $keepTranslation ? 0 : $valueArray[0];
@@ -3305,7 +3305,7 @@ class DataHandler
             return null;
         }
 
-        $fullLanguageCheckNeeded = $table != 'pages';
+        $fullLanguageCheckNeeded = $table !== 'pages';
         //Used to check language and general editing rights
         if (!$ignoreLocalization && ($language <= 0 || !$this->BE_USER->checkLanguageAccess($language)) && !$this->BE_USER->recordEditAccessInternals($table, $uid, false, false, $fullLanguageCheckNeeded)) {
             if ($this->enableLogging) {
@@ -3348,7 +3348,7 @@ class DataHandler
                 // "pid" is hardcoded of course:
                 // isset() won't work here, since values can be NULL in each of the arrays
                 // except setDefaultOnCopyArray, since we exploded that from a string
-                if ($field == 'pid') {
+                if ($field === 'pid') {
                     $value = $destPid;
                 } elseif (array_key_exists($field, $overrideValues)) {
                     // Override value...
@@ -3731,13 +3731,13 @@ class DataHandler
         // Get the localization mode for the current (parent) record (keep|select):
         $localizationMode = BackendUtility::getInlineLocalizationMode($table, $field);
         // Register if there are references to take care of or MM is used on an inline field (no change to value):
-        if ($this->isReferenceField($conf) || $inlineSubType == 'mm') {
+        if ($this->isReferenceField($conf) || $inlineSubType === 'mm') {
             $value = $this->copyRecord_processManyToMany($table, $uid, $field, $value, $conf, $language, $localizationMode, $inlineSubType);
         } elseif ($inlineSubType !== false) {
             $value = $this->copyRecord_processInline($table, $uid, $field, $value, $row, $conf, $realDestPid, $language, $workspaceOptions, $localizationMode, $inlineSubType);
         }
         // For "flex" fieldtypes we need to traverse the structure for two reasons: If there are file references they have to be prepended with absolute paths and if there are database reference they MIGHT need to be remapped (still done in remapListedDBRecords())
-        if ($conf['type'] == 'flex') {
+        if ($conf['type'] === 'flex') {
             // Get current value array:
             $flexFormTools = GeneralUtility::makeInstance(FlexFormTools::class);
             $dataStructureIdentifier = $flexFormTools->getDataStructureIdentifier(
@@ -3773,8 +3773,8 @@ class DataHandler
      */
     protected function copyRecord_processManyToMany($table, $uid, $field, $value, $conf, $language, $localizationMode, $inlineSubType)
     {
-        $allowedTables = $conf['type'] == 'group' ? $conf['allowed'] : $conf['foreign_table'];
-        $prependName = $conf['type'] == 'group' ? $conf['prepend_tname'] : '';
+        $allowedTables = $conf['type'] === 'group' ? $conf['allowed'] : $conf['foreign_table'];
+        $prependName = $conf['type'] === 'group' ? $conf['prepend_tname'] : '';
         $mmTable = isset($conf['MM']) && $conf['MM'] ? $conf['MM'] : '';
         $localizeForeignTable = isset($conf['foreign_table']) && BackendUtility::isTableLocalizable($conf['foreign_table']);
         $localizeReferences = $localizeForeignTable && isset($conf['localizeReferencesAtParentLocalization']) && $conf['localizeReferencesAtParentLocalization'];
@@ -3835,8 +3835,8 @@ class DataHandler
                                                 array $workspaceOptions, $localizationMode, $inlineSubType)
     {
         // Localization in mode 'keep', isn't a real localization, but keeps the children of the original parent record:
-        if ($language > 0 && $localizationMode == 'keep') {
-            $value = $inlineSubType == 'field' ? 0 : '';
+        if ($language > 0 && $localizationMode === 'keep') {
+            $value = $inlineSubType === 'field' ? 0 : '';
         } else {
             // Fetch the related child records using \TYPO3\CMS\Core\Database\RelationHandler
             /** @var $dbAnalysis RelationHandler */
@@ -3888,7 +3888,7 @@ class DataHandler
                     }
                 }
                 // If the current field is set on a page record, update the pid of related child records:
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $this->registerDBPids[$v['table']][$v['id']] = $uid;
                 } elseif (isset($this->registerDBPids[$table][$uid])) {
                     $this->registerDBPids[$v['table']][$v['id']] = $this->registerDBPids[$table][$uid];
@@ -3945,7 +3945,7 @@ class DataHandler
     public function copyRecord_procFilesRefs($conf, $uid, $value)
     {
         // Prepend absolute paths to files:
-        if ($conf['type'] != 'group' || ($conf['internal_type'] != 'file' && $conf['internal_type'] != 'file_reference')) {
+        if ($conf['type'] !== 'group' || ($conf['internal_type'] !== 'file' && $conf['internal_type'] !== 'file_reference')) {
             return $value;
         }
 
@@ -3964,7 +3964,7 @@ class DataHandler
             $theFileValues = GeneralUtility::trimExplode(',', $value, true);
         }
         // Traverse this array of files:
-        $uploadFolder = $conf['internal_type'] == 'file' ? $conf['uploadfolder'] : '';
+        $uploadFolder = $conf['internal_type'] === 'file' ? $conf['uploadfolder'] : '';
         $dest = $this->destPathFromUploadFolder($uploadFolder);
         $newValue = [];
         foreach ($theFileValues as $file) {
@@ -4221,21 +4221,21 @@ class DataHandler
         $resolvedPid = $this->resolvePid($table, $destPid);
         // Finding out, if the record may be moved from where it is. If the record is a non-page, then it depends on edit-permissions.
         // If the record is a page, then there are two options: If the page is moved within itself, (same pid) it's edit-perms of the pid. If moved to another place then its both delete-perms of the pid and new-page perms on the destination.
-        if ($table != 'pages' || $resolvedPid == $moveRec['pid']) {
+        if ($table !== 'pages' || $resolvedPid == $moveRec['pid']) {
             // Edit rights for the record...
             $mayMoveAccess = $this->checkRecordUpdateAccess($table, $uid);
         } else {
             $mayMoveAccess = $this->doesRecordExist($table, $uid, 'delete');
         }
         // Finding out, if the record may be moved TO another place. Here we check insert-rights (non-pages = edit, pages = new), unless the pages are moved on the same pid, then edit-rights are checked
-        if ($table != 'pages' || $resolvedPid != $moveRec['pid']) {
+        if ($table !== 'pages' || $resolvedPid != $moveRec['pid']) {
             // Insert rights for the record...
             $mayInsertAccess = $this->checkRecordInsertAccess($table, $resolvedPid, 4);
         } else {
             $mayInsertAccess = $this->checkRecordUpdateAccess($table, $uid);
         }
         // Checking if there is anything else disallowing moving the record by checking if editing is allowed
-        $fullLanguageCheckNeeded = $table != 'pages';
+        $fullLanguageCheckNeeded = $table !== 'pages';
         $mayEditAccess = $this->BE_USER->recordEditAccessInternals($table, $uid, false, false, $fullLanguageCheckNeeded);
         // If moving is allowed, begin the processing:
         if (!$mayEditAccess) {
@@ -4313,7 +4313,7 @@ class DataHandler
         }
         // Insert as first element on page (where uid = $destPid)
         if ($destPid >= 0) {
-            if ($table != 'pages' || $this->destNotInsideSelf($destPid, $uid)) {
+            if ($table !== 'pages' || $this->destNotInsideSelf($destPid, $uid)) {
                 // Clear cache before moving
                 list($parentUid) = BackendUtility::getTSCpid($table, $uid, '');
                 $this->registerRecordIdForPageCacheClearing($table, $uid, $parentUid);
@@ -4375,7 +4375,7 @@ class DataHandler
                 $destPid = $sortInfo['pid'];
                 // If not an array, there was an error (which is already logged)
                 if (is_array($sortInfo)) {
-                    if ($table != 'pages' || $this->destNotInsideSelf($destPid, $uid)) {
+                    if ($table !== 'pages' || $this->destNotInsideSelf($destPid, $uid)) {
                         // clear cache before moving
                         $this->registerRecordIdForPageCacheClearing($table, $uid);
                         // We now update the pid and sortnumber
@@ -4462,13 +4462,13 @@ class DataHandler
      */
     public function moveRecord_procBasedOnFieldType($table, $uid, $destPid, $field, $value, $conf)
     {
-        if ($conf['type'] == 'inline') {
+        if ($conf['type'] === 'inline') {
             $foreign_table = $conf['foreign_table'];
             $moveChildrenWithParent = !isset($conf['behaviour']['disableMovingChildrenWithParent']) || !$conf['behaviour']['disableMovingChildrenWithParent'];
             if ($foreign_table && $moveChildrenWithParent) {
                 $inlineType = $this->getInlineFieldType($conf);
-                if ($inlineType == 'list' || $inlineType == 'field') {
-                    if ($table == 'pages') {
+                if ($inlineType === 'list' || $inlineType === 'field') {
+                    if ($table === 'pages') {
                         // If the inline elements are related to a page record,
                         // make sure they reside at that page and not at its parent
                         $destPid = $uid;
@@ -4651,8 +4651,8 @@ class DataHandler
         foreach ($GLOBALS['TCA'][$Ttable]['columns'] as $fN => $fCfg) {
             $translateToMsg = '';
             // Check if we are just prefixing:
-            if ($fCfg['l10n_mode'] == 'prefixLangTitle') {
-                if (($fCfg['config']['type'] == 'text' || $fCfg['config']['type'] == 'input') && (string)$row[$fN] !== '') {
+            if ($fCfg['l10n_mode'] === 'prefixLangTitle') {
+                if (($fCfg['config']['type'] === 'text' || $fCfg['config']['type'] === 'input') && (string)$row[$fN] !== '') {
                     list($tscPID) = BackendUtility::getTSCpid($table, $uid, '');
                     $TSConfig = $this->getTCEMAIN_TSconfig($tscPID);
                     if (!empty($TSConfig['translateToMessage'])) {
@@ -4794,7 +4794,7 @@ class DataHandler
         }
 
         $removeArray = [];
-        $mmTable = $inlineSubType == 'mm' && isset($config['MM']) && $config['MM'] ? $config['MM'] : '';
+        $mmTable = $inlineSubType === 'mm' && isset($config['MM']) && $config['MM'] ? $config['MM'] : '';
         // Fetch children from original language parent:
         /** @var $dbAnalysisOriginal RelationHandler */
         $dbAnalysisOriginal = $this->createRelationHandlerInstance();
@@ -4854,12 +4854,12 @@ class DataHandler
         }
         $updateFields = [];
         // Handle, reorder and store relations:
-        if ($inlineSubType == 'list') {
+        if ($inlineSubType === 'list') {
             $updateFields = [$field => $value];
-        } elseif ($inlineSubType == 'field') {
+        } elseif ($inlineSubType === 'field') {
             $dbAnalysisCurrent->writeForeignField($config, $id);
             $updateFields = [$field => $dbAnalysisCurrent->countItems(false)];
-        } elseif ($inlineSubType == 'mm') {
+        } elseif ($inlineSubType === 'mm') {
             $dbAnalysisCurrent->writeMM($config['MM'], $id);
             $updateFields = [$field => $dbAnalysisCurrent->countItems(false)];
         }
@@ -4913,7 +4913,7 @@ class DataHandler
      */
     public function deleteEl($table, $uid, $noRecordCheck = false, $forceHardDelete = false)
     {
-        if ($table == 'pages') {
+        if ($table === 'pages') {
             $this->deletePages($uid, $noRecordCheck, $forceHardDelete);
         } else {
             $this->deleteVersionsForRecord($table, $uid, $forceHardDelete);
@@ -4935,7 +4935,7 @@ class DataHandler
         if (is_array($versions)) {
             foreach ($versions as $verRec) {
                 if (!$verRec['_CURRENT_VERSION']) {
-                    if ($table == 'pages') {
+                    if ($table === 'pages') {
                         $this->deletePages($verRec['uid'], true, $forceHardDelete);
                     } else {
                         $this->deleteRecord($table, $verRec['uid'], true, $forceHardDelete);
@@ -5207,7 +5207,7 @@ class DataHandler
         $uid = (int)$uid;
         if ($uid) {
             foreach ($GLOBALS['TCA'] as $table => $_) {
-                if ($table != 'pages') {
+                if ($table !== 'pages') {
                     $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)
                         ->getQueryBuilderForTable($table);
 
@@ -5412,11 +5412,11 @@ class DataHandler
      */
     public function deleteRecord_procBasedOnFieldType($table, $uid, $field, $value, $conf, $undeleteRecord = false)
     {
-        if ($conf['type'] == 'inline') {
+        if ($conf['type'] === 'inline') {
             $foreign_table = $conf['foreign_table'];
             if ($foreign_table) {
                 $inlineType = $this->getInlineFieldType($conf);
-                if ($inlineType == 'list' || $inlineType == 'field') {
+                if ($inlineType === 'list' || $inlineType === 'field') {
                     /** @var RelationHandler $dbAnalysis */
                     $dbAnalysis = $this->createRelationHandlerInstance();
                     $dbAnalysis->start($value, $conf['foreign_table'], '', $uid, $table, $conf);
@@ -5441,7 +5441,7 @@ class DataHandler
                 }
             }
         } elseif ($this->isReferenceField($conf)) {
-            $allowedTables = $conf['type'] == 'group' ? $conf['allowed'] : $conf['foreign_table'];
+            $allowedTables = $conf['type'] === 'group' ? $conf['allowed'] : $conf['foreign_table'];
             $dbAnalysis = $this->createRelationHandlerInstance();
             $dbAnalysis->start($value, $allowedTables, $conf['MM'], $uid, $table, $conf);
             foreach ($dbAnalysis->itemArray as $v) {
@@ -5631,8 +5631,8 @@ class DataHandler
         foreach ($GLOBALS['TCA'][$table]['columns'] as $field => $fConf) {
             $conf = $fConf['config'];
             if ($this->isReferenceField($conf)) {
-                $allowedTables = $conf['type'] == 'group' ? $conf['allowed'] : $conf['foreign_table'];
-                $prependName = $conf['type'] == 'group' ? $conf['prepend_tname'] : '';
+                $allowedTables = $conf['type'] === 'group' ? $conf['allowed'] : $conf['foreign_table'];
+                $prependName = $conf['type'] === 'group' ? $conf['prepend_tname'] : '';
                 if ($conf['MM']) {
                     /** @var $dbAnalysis RelationHandler */
                     $dbAnalysis = $this->createRelationHandlerInstance();
@@ -5647,7 +5647,7 @@ class DataHandler
                         $this->version_remapMMForVersionSwap_reg[$swapWith][$field] = [$dbAnalysis, $conf['MM'], $prependName];
                     }
                 }
-            } elseif ($conf['type'] == 'flex') {
+            } elseif ($conf['type'] === 'flex') {
                 // Current record
                 $dataStructureIdentifier = $flexFormTools->getDataStructureIdentifier(
                     $fConf,
@@ -5695,8 +5695,8 @@ class DataHandler
         // Extract parameters:
         list($table, $uid, $field) = $pParams;
         if ($this->isReferenceField($dsConf)) {
-            $allowedTables = $dsConf['type'] == 'group' ? $dsConf['allowed'] : $dsConf['foreign_table'];
-            $prependName = $dsConf['type'] == 'group' ? $dsConf['prepend_tname'] : '';
+            $allowedTables = $dsConf['type'] === 'group' ? $dsConf['allowed'] : $dsConf['foreign_table'];
+            $prependName = $dsConf['type'] === 'group' ? $dsConf['prepend_tname'] : '';
             if ($dsConf['MM']) {
                 /** @var $dbAnalysis RelationHandler */
                 $dbAnalysis = $this->createRelationHandlerInstance();
@@ -5787,7 +5787,7 @@ class DataHandler
                                 }
                                 break;
                             case 'flex':
-                                if ($value == 'FlexForm_reference') {
+                                if ($value === 'FlexForm_reference') {
                                     // This will fetch the new row for the element
                                     $origRecordRow = $this->recordInfo($table, $theUidToUpdate, '*');
                                     if (is_array($origRecordRow)) {
@@ -5868,14 +5868,14 @@ class DataHandler
         // Will be set TRUE if an upgrade should be done...
         $set = false;
         // Allowed tables for references.
-        $allowedTables = $conf['type'] == 'group' ? $conf['allowed'] : $conf['foreign_table'];
+        $allowedTables = $conf['type'] === 'group' ? $conf['allowed'] : $conf['foreign_table'];
         // Table name to prepend the UID
-        $prependName = $conf['type'] == 'group' ? $conf['prepend_tname'] : '';
+        $prependName = $conf['type'] === 'group' ? $conf['prepend_tname'] : '';
         // Which tables that should possibly not be remapped
         $dontRemapTables = GeneralUtility::trimExplode(',', $conf['dontRemapTablesOnCopy'], true);
         // Convert value to list of references:
         $dbAnalysis = $this->createRelationHandlerInstance();
-        $dbAnalysis->registerNonTableValues = $conf['type'] == 'select' && $conf['allowNonIdValues'];
+        $dbAnalysis->registerNonTableValues = $conf['type'] === 'select' && $conf['allowNonIdValues'];
         $dbAnalysis->start($value, $allowedTables, $conf['MM'], $MM_localUid, $table, $conf);
         // Traverse those references and map IDs:
         foreach ($dbAnalysis->itemArray as $k => $v) {
@@ -5933,7 +5933,7 @@ class DataHandler
         $theUidToUpdate = $this->copyMappingArray_merged[$table][$uid];
         if ($conf['foreign_table']) {
             $inlineType = $this->getInlineFieldType($conf);
-            if ($inlineType == 'mm') {
+            if ($inlineType === 'mm') {
                 $this->remapListedDBRecords_procDBRefs($conf, $value, $theUidToUpdate, $table);
             } elseif ($inlineType !== false) {
                 /** @var $dbAnalysis RelationHandler */
@@ -5950,12 +5950,12 @@ class DataHandler
                 }
 
                 // Update child records if using pointer fields ('foreign_field'):
-                if ($inlineType == 'field') {
+                if ($inlineType === 'field') {
                     $dbAnalysis->writeForeignField($conf, $uid, $theUidToUpdate);
                 }
                 $thePidToUpdate = null;
                 // If the current field is set on a page record, update the pid of related child records:
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $thePidToUpdate = $theUidToUpdate;
                 } elseif (isset($this->registerDBPids[$table][$uid])) {
                     $thePidToUpdate = $this->registerDBPids[$table][$uid];
@@ -6433,7 +6433,7 @@ class DataHandler
         }
         // Processing the incoming $perms (from possible string to integer that can be AND'ed)
         if (!MathUtility::canBeInterpretedAsInteger($perms)) {
-            if ($table != 'pages') {
+            if ($table !== 'pages') {
                 switch ($perms) {
                     case 'edit':
 
@@ -6459,7 +6459,7 @@ class DataHandler
         // For all tables: Check if record exists:
         $isWebMountRestrictionIgnored = BackendUtility::isWebMountRestrictionIgnored($table);
         if (is_array($GLOBALS['TCA'][$table]) && $id > 0 && ($isWebMountRestrictionIgnored || $this->isRecordInWebMount($table, $id) || $this->admin)) {
-            if ($table != 'pages') {
+            if ($table !== 'pages') {
                 // Find record without checking page
                 // @todo: Thist should probably check for editlock
                 $queryBuilder = GeneralUtility::makeInstance(ConnectionPool::class)->getQueryBuilderForTable($table);
@@ -6770,7 +6770,7 @@ class DataHandler
      */
     public function getRecordProperties($table, $id, $noWSOL = false)
     {
-        $row = $table == 'pages' && !$id ? ['title' => '[root-level]', 'uid' => 0, 'pid' => 0] : $this->recordInfo($table, $id, '*');
+        $row = $table === 'pages' && !$id ? ['title' => '[root-level]', 'uid' => 0, 'pid' => 0] : $this->recordInfo($table, $id, '*');
         if (!$noWSOL) {
             BackendUtility::workspaceOL($table, $row);
         }
@@ -6808,7 +6808,7 @@ class DataHandler
      */
     public function eventPid($table, $uid, $pid)
     {
-        return $table == 'pages' ? $uid : $pid;
+        return $table === 'pages' ? $uid : $pid;
     }
 
     /*********************************************
@@ -6859,7 +6859,7 @@ class DataHandler
                     // Clear cache for relevant pages:
                     $this->registerRecordIdForPageCacheClearing($table, $id);
                     // Unset the pageCache for the id if table was page.
-                    if ($table == 'pages') {
+                    if ($table === 'pages') {
                         unset($this->pageCache[$id]);
                     }
                 } elseif ($this->enableLogging) {
@@ -7858,7 +7858,7 @@ class DataHandler
         $listArr = [];
         if (isset($GLOBALS['TCA'][$table]['columns'])) {
             foreach ($GLOBALS['TCA'][$table]['columns'] as $field => $configArr) {
-                if ($configArr['config']['type'] == 'group' && ($configArr['config']['internal_type'] == 'file' || $configArr['config']['internal_type'] == 'file_reference')) {
+                if ($configArr['config']['type'] === 'group' && ($configArr['config']['internal_type'] === 'file' || $configArr['config']['internal_type'] === 'file_reference')) {
                     $listArr[] = $field;
                 }
             }
@@ -7903,7 +7903,7 @@ class DataHandler
      */
     public function isReferenceField($conf)
     {
-        return $conf['type'] == 'group' && $conf['internal_type'] == 'db' || $conf['type'] == 'select' && $conf['foreign_table'];
+        return $conf['type'] === 'group' && $conf['internal_type'] === 'db' || $conf['type'] === 'select' && $conf['foreign_table'];
     }
 
     /**
@@ -8044,7 +8044,7 @@ class DataHandler
     public function extFileFunctions($table, $field, $filelist, $func)
     {
         $uploadFolder = $GLOBALS['TCA'][$table]['columns'][$field]['config']['uploadfolder'];
-        if ($uploadFolder && trim($filelist) && $GLOBALS['TCA'][$table]['columns'][$field]['config']['internal_type'] == 'file') {
+        if ($uploadFolder && trim($filelist) && $GLOBALS['TCA'][$table]['columns'][$field]['config']['internal_type'] === 'file') {
             $uploadPath = $this->destPathFromUploadFolder($uploadFolder);
             $fileArray = explode(',', $filelist);
             foreach ($fileArray as $theFile) {
index a14ce35..74b258f 100644 (file)
@@ -492,7 +492,7 @@ class QueryGenerator
             // Do stuff:
             $tempEl = $workArr[$ssArr[$i]];
             if (is_array($tempEl)) {
-                if ($tempEl['type'] != 'newlevel') {
+                if ($tempEl['type'] !== 'newlevel') {
                     $workArr[$ssArr[$i]] = [
                         'type' => 'newlevel',
                         'operator' => $tempEl['operator'],
@@ -599,7 +599,7 @@ class QueryGenerator
             $fieldName = '';
             $subscript = $parent . '[' . $key . ']';
             $lineHTML = [];
-            $lineHTML[] = $this->mkOperatorSelect($this->name . $subscript, $conf['operator'], $c, $conf['type'] != 'FIELD_');
+            $lineHTML[] = $this->mkOperatorSelect($this->name . $subscript, $conf['operator'], $c, $conf['type'] !== 'FIELD_');
             if (substr($conf['type'], 0, 6) === 'FIELD_') {
                 $fieldName = substr($conf['type'], 6);
                 $this->fieldName = $fieldName;
@@ -718,7 +718,7 @@ class QueryGenerator
                 if ($c != 0) {
                     $lineHTML[] = '<button class="btn btn-default" title="Move up" name="qG_up' . htmlspecialchars($subscript) . '"><i class="fa fa-chevron-up fa-fw"></i></button>';
                 }
-                if ($c != 0 && $fieldType != 'newlevel') {
+                if ($c != 0 && $fieldType !== 'newlevel') {
                     $lineHTML[] = '<button class="btn btn-default" title="New level" name="qG_nl' . htmlspecialchars($subscript) . '"><i class="fa fa-chevron-right fa-fw"></i></button>';
                 }
                 if ($fieldType === 'newlevel') {
index d3f07c8..c0dbcba 100644 (file)
@@ -683,8 +683,8 @@ class QueryView
         foreach ($row as $fieldName => $fieldValue) {
             if (GeneralUtility::inList($SET['queryFields'], $fieldName)
                 || !$SET['queryFields']
-                && $fieldName != 'pid'
-                && $fieldName != 'deleted'
+                && $fieldName !== 'pid'
+                && $fieldName !== 'deleted'
             ) {
                 if ($SET['search_result_labels']) {
                     $fVnew = $this->getProcessedValueExtra($table, $fieldName, $fieldValue, $conf, '<br />');
@@ -812,7 +812,7 @@ class QueryView
                         break;
                     case 'group':
                         $fields['type'] = 'files';
-                        if ($fields['internal_type'] == 'db') {
+                        if ($fields['internal_type'] === 'db') {
                             $fields['type'] = 'relation';
                         }
                         break;
@@ -852,7 +852,7 @@ class QueryView
                 break;
             case 'time':
                 if ($fieldValue != -1) {
-                    if ($splitString == '<br />') {
+                    if ($splitString === '<br />') {
                         $out = strftime('%H:%M' . $splitString . '%e-%m-%Y', $fieldValue);
                     } else {
                         $out = strftime('%H:%M %e-%m-%Y', $fieldValue);
@@ -933,10 +933,10 @@ class QueryView
     {
         $fieldSetup = $conf;
         $out = '';
-        if ($fieldSetup['type'] == 'files') {
+        if ($fieldSetup['type'] === 'files') {
             $d = dir(PATH_site . $fieldSetup['uploadfolder']);
             while (false !== ($entry = $d->read())) {
-                if ($entry == '.' || $entry == '..') {
+                if ($entry === '.' || $entry === '..') {
                     continue;
                 }
                 $fileArray[] = $entry;
@@ -953,9 +953,9 @@ class QueryView
                 }
             }
         }
-        if ($fieldSetup['type'] == 'multiple') {
+        if ($fieldSetup['type'] === 'multiple') {
             foreach ($fieldSetup['items'] as $key => $val) {
-                if (substr($val[0], 0, 4) == 'LLL:') {
+                if (substr($val[0], 0, 4) === 'LLL:') {
                     $value = $this->languageService->sL($val[0]);
                 } else {
                     $value = $val[0];
@@ -969,9 +969,9 @@ class QueryView
                 }
             }
         }
-        if ($fieldSetup['type'] == 'binary') {
+        if ($fieldSetup['type'] === 'binary') {
             foreach ($fieldSetup['items'] as $Key => $val) {
-                if (substr($val[0], 0, 4) == 'LLL:') {
+                if (substr($val[0], 0, 4) === 'LLL:') {
                     $value = $this->languageService->sL($val[0]);
                 } else {
                     $value = $val[0];
@@ -983,12 +983,12 @@ class QueryView
                 }
             }
         }
-        if ($fieldSetup['type'] == 'relation') {
+        if ($fieldSetup['type'] === 'relation') {
             $dontPrefixFirstTable = 0;
             $useTablePrefix = 0;
             if ($fieldSetup['items']) {
                 foreach ($fieldSetup['items'] as $key => $val) {
-                    if (substr($val[0], 0, 4) == 'LLL:') {
+                    if (substr($val[0], 0, 4) === 'LLL:') {
                         $value = $this->languageService->sL($val[0]);
                     } else {
                         $value = $val[0];
@@ -1050,7 +1050,7 @@ class QueryView
                     if ($GLOBALS['TCA'][$from_table]['columns'][$labelField]['config']['items']) {
                         $items = $GLOBALS['TCA'][$from_table]['columns'][$labelField]['config']['items'];
                         foreach ($items as $labelArray) {
-                            if (substr($labelArray[0], 0, 4) == 'LLL:') {
+                            if (substr($labelArray[0], 0, 4) === 'LLL:') {
                                 $labelFieldSelect[$labelArray[1]] = $this->languageService->sL($labelArray[0]);
                             } else {
                                 $labelFieldSelect[$labelArray[1]] = $labelArray[0];
@@ -1061,7 +1061,7 @@ class QueryView
                     if ($GLOBALS['TCA'][$from_table]['columns'][$altLabelField]['config']['items']) {
                         $items = $GLOBALS['TCA'][$from_table]['columns'][$altLabelField]['config']['items'];
                         foreach ($items as $altLabelArray) {
-                            if (substr($altLabelArray[0], 0, 4) == 'LLL:') {
+                            if (substr($altLabelArray[0], 0, 4) === 'LLL:') {
                                 $altLabelFieldSelect[$altLabelArray[1]] = $this->languageService->sL($altLabelArray[0]);
                             } else {
                                 $altLabelFieldSelect[$altLabelArray[1]] = $altLabelArray[0];
@@ -1128,7 +1128,7 @@ class QueryView
                                 ? 'on'
                                 : $GLOBALS['SOBE']->MOD_SETTINGS['labels_noprefix'];
                         $prefixString =
-                            $GLOBALS['SOBE']->MOD_SETTINGS['labels_noprefix'] == 'on'
+                            $GLOBALS['SOBE']->MOD_SETTINGS['labels_noprefix'] === 'on'
                                 ? ''
                                 : ' [' . $tablePrefix . $val['uid'] . '] ';
                         if (GeneralUtility::inList($fieldValue, $tablePrefix . $val['uid'])
@@ -1190,8 +1190,8 @@ class QueryView
         foreach ($row as $fieldName => $fieldValue) {
             if (GeneralUtility::inList($SET['queryFields'], $fieldName)
                 || !$SET['queryFields']
-                && $fieldName != 'pid'
-                && $fieldName != 'deleted'
+                && $fieldName !== 'pid'
+                && $fieldName !== 'deleted'
             ) {
                 if ($GLOBALS['SOBE']->MOD_SETTINGS['search_result_labels']) {
                     $title = htmlspecialchars($this->languageService->sL($conf['columns'][$fieldName]['label']
@@ -1224,7 +1224,7 @@ class QueryView
         $SET = $GLOBALS['SOBE']->MOD_SETTINGS;
         foreach ($row as $fieldName => $fieldValue) {
             if (GeneralUtility::inList($SET['queryFields'], $fieldName)
-                || !$SET['queryFields'] && $fieldName != 'pid') {
+                || !$SET['queryFields'] && $fieldName !== 'pid') {
                 if (!$out) {
                     if ($GLOBALS['SOBE']->MOD_SETTINGS['search_result_labels']) {
                         $out = htmlspecialchars($this->languageService->sL($conf['columns'][$fieldName]['label']
index 91fcbc8..4c1c78f 100644 (file)
@@ -472,7 +472,7 @@ class RelationHandler
     public function sortList($sortby)
     {
         // Sort directly without fetching addional data
-        if ($sortby == 'uid') {
+        if ($sortby === 'uid') {
             usort(
                 $this->itemArray,
                 function ($a, $b) {
@@ -589,7 +589,7 @@ class RelationHandler
                 // If tablesnames columns exists and contain a name, then this value is the table, else it's the firstTable...
                 $theTable = $row['tablenames'] ?: $this->firstTable;
             }
-            if (($row[$uidForeign_field] || $theTable == 'pages') && $theTable && isset($this->tableArray[$theTable])) {
+            if (($row[$uidForeign_field] || $theTable === 'pages') && $theTable && isset($this->tableArray[$theTable])) {
                 $this->itemArray[$key]['id'] = $row[$uidForeign_field];
                 $this->itemArray[$key]['table'] = $theTable;
                 $this->tableArray[$theTable][] = $row[$uidForeign_field];
@@ -694,7 +694,7 @@ class RelationHandler
             // For each item, insert it:
             foreach ($this->itemArray as $val) {
                 $c++;
-                if ($prep || $val['table'] == '_NO_TABLE') {
+                if ($prep || $val['table'] === '_NO_TABLE') {
                     // Insert current table if needed
                     if ($this->MM_is_foreign) {
                         $tablename = $this->currentTable;
@@ -1193,7 +1193,7 @@ class RelationHandler
             $prep = $tableC > 1 || $prependTableName;
             // Traverse the array of items:
             foreach ($this->itemArray as $val) {
-                $valueArray[] = ($prep && $val['table'] != '_NO_TABLE' ? $val['table'] . '_' : '') . $val['id'];
+                $valueArray[] = ($prep && $val['table'] !== '_NO_TABLE' ? $val['table'] . '_' : '') . $val['id'];
             }
         }
         // Return the array
index 3105238..d684788 100644 (file)
@@ -81,7 +81,7 @@ class FrontendEditingController
         // Page ID for new records, 0 if not specified
         $newRecordPid = (int)$conf['newRecordInPid'];
         if (!$conf['onlyCurrentPid'] || $dataArray['pid'] == $GLOBALS['TSFE']->id) {
-            if ($table == 'pages') {
+            if ($table === 'pages') {
                 $newUid = $uid;
             } else {
                 if ($conf['newRecordFromTable']) {
@@ -166,7 +166,7 @@ class FrontendEditingController
                 unset($this->TSFE_EDIT['cmd']);
             } else {
                 $cmd = (string)$this->TSFE_EDIT['cmd'];
-                if (($cmd != 'edit' || is_array($this->TSFE_EDIT['data']) && ($this->TSFE_EDIT['doSave'] || $this->TSFE_EDIT['update'] || $this->TSFE_EDIT['update_close'])) && $cmd != 'new') {
+                if (($cmd !== 'edit' || is_array($this->TSFE_EDIT['data']) && ($this->TSFE_EDIT['doSave'] || $this->TSFE_EDIT['update'] || $this->TSFE_EDIT['update_close'])) && $cmd !== 'new') {
                     // $cmd can be a command like "hide" or "move". If $cmd is "edit" or "new" it's an indication to show the formfields. But if data is sent with update-flag then $cmd = edit is accepted because edit may be sent because of .keepGoing flag.
                     return true;
                 }
@@ -186,7 +186,7 @@ class FrontendEditingController
     {
         if (is_array($this->TSFE_EDIT)) {
             $cmd = (string)$this->TSFE_EDIT['cmd'];
-            if ($cmd == 'edit' || $cmd == 'new') {
+            if ($cmd === 'edit' || $cmd === 'new') {
                 return true;
             }
         }
@@ -210,7 +210,7 @@ class FrontendEditingController
         if (($this->TSFE_EDIT['doSave'] || $this->TSFE_EDIT['update'] || $this->TSFE_EDIT['update_close']) && is_array($this->TSFE_EDIT['data'])) {
             $cmd = 'save';
         }
-        if ($cmd == 'save' || $cmd && $table && $uid && isset($GLOBALS['TCA'][$table])) {
+        if ($cmd === 'save' || $cmd && $table && $uid && isset($GLOBALS['TCA'][$table])) {
             // Hook for defining custom editing actions. Naming is incorrect, but preserves compatibility.
             if (is_array($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_tsfebeuserauth.php']['extEditAction'])) {
                 $_params = [];
@@ -536,7 +536,7 @@ class FrontendEditingController
             $editAccessInternals = true;
         }
         if ($editAccessInternals) {
-            if ($table == 'pages') {
+            if ($table === 'pages') {
                 // 2 = permission to edit the page
                 if ($GLOBALS['BE_USER']->isAdmin() || $GLOBALS['BE_USER']->doesUserHaveAccess($dataArray, 2)) {
                     $mayEdit = true;
@@ -552,7 +552,7 @@ class FrontendEditingController
                 $types = GeneralUtility::trimExplode(',', strtolower($conf['allow']), true);
                 $allow = array_flip($types);
                 $perms = $GLOBALS['BE_USER']->calcPerms($GLOBALS['TSFE']->page);
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $allow = $this->getAllowedEditActions($table, $conf, $dataArray['pid'], $allow);
                     // Can only display editbox if there are options in the menu
                     if (!empty($allow)) {
@@ -586,7 +586,7 @@ class FrontendEditingController
             $types = GeneralUtility::trimExplode(',', strtolower($conf['allow']), true);
             $allow = array_flip($types);
             $perms = $GLOBALS['BE_USER']->calcPerms($GLOBALS['TSFE']->page);
-            if ($table == 'pages') {
+            if ($table === 'pages') {
                 // Rootpage
                 if (count($GLOBALS['TSFE']->config['rootLine']) === 1) {
                     unset($allow['move']);
index 43cf87d..aab7392 100644 (file)
@@ -64,7 +64,7 @@ class HtmlParser
         // We skip the first element in foreach loop
         $partsSliced = array_slice($parts, 1, null, true);
         foreach ($partsSliced as $v) {
-            $isEndTag = substr($content, $pointer, 2) == '</' ? 1 : 0;
+            $isEndTag = substr($content, $pointer, 2) === '</' ? 1 : 0;
             $tagLen = strcspn(substr($content, $pointer), '>') + 1;
             // We meet a start-tag:
             if (!$isEndTag) {
@@ -256,7 +256,7 @@ class HtmlParser
         if (is_array($components)) {
             foreach ($components as $key => $val) {
                 // Only if $name is set (if there is an attribute, that waits for a value), that valuemode is enabled. This ensures that the attribute is assigned it's value
-                if ($val != '=') {
+                if ($val !== '=') {
                     if ($valuemode) {
                         if ($name) {
                             $attributes[$name] = $deHSC ? htmlspecialchars_decode($val) : $val;
@@ -307,7 +307,7 @@ class HtmlParser
         if (preg_match_all('/("[^"]*"|\'[^\']*\'|[^\\s"\'\\=]+|\\=)/s', $tag_tmp, $matches) > 0) {
             foreach ($matches[1] as $part) {
                 $firstChar = $part[0];
-                if ($firstChar == '"' || $firstChar == '\'') {
+                if ($firstChar === '"' || $firstChar === '\'') {
                     $metaValue[] = $firstChar;
                     $value[] = substr($part, 1, -1);
                 } else {
@@ -396,7 +396,7 @@ class HtmlParser
                 $tok = substr($tok, $eocPos + 10);
                 $inCdata = false;
                 $skipTag = true;
-            } elseif (substr($tok, 0, 3) == '!--') {
+            } elseif (substr($tok, 0, 3) === '!--') {
                 if (($eocPos = strpos($tok, '-->')) === false) {
                     // Comment started in this token but it does end in the same token. Set a flag to skip till the end of comment
                     $newContent[$c++] = '<' . $tok;
@@ -425,7 +425,7 @@ class HtmlParser
                 $tagEnd = strpos($tok, '>');
                 // If there is and end-bracket...      tagEnd can't be 0 as the first character can't be a >
                 if ($tagEnd) {
-                    $endTag = $firstChar == '/' ? 1 : 0;
+                    $endTag = $firstChar === '/' ? 1 : 0;
                     $tagContent = substr($tok, $endTag, $tagEnd - $endTag);
                     $tagParts = preg_split('/\\s+/s', $tagContent, 2);
                     $tagName = strtolower($tagParts[0]);
@@ -500,7 +500,7 @@ class HtmlParser
                                             if (is_array($params['list'])) {
                                                 // For the class attribute, remove from the attribute value any class not in the list
                                                 // Classes are case sensitive
-                                                if ($attr == 'class') {
+                                                if ($attr === 'class') {
                                                     $newClasses = [];
                                                     $classes = GeneralUtility::trimExplode(' ', $tagAttrib[0][$attr], true);
                                                     foreach ($classes as $class) {
@@ -519,7 +519,7 @@ class HtmlParser
                                                     }
                                                 }
                                             }
-                                            if ($params['removeIfFalse'] && $params['removeIfFalse'] != 'blank' && !$tagAttrib[0][$attr] || $params['removeIfFalse'] == 'blank' && (string)$tagAttrib[0][$attr] === '') {
+                                            if ($params['removeIfFalse'] && $params['removeIfFalse'] !== 'blank' && !$tagAttrib[0][$attr] || $params['removeIfFalse'] === 'blank' && (string)$tagAttrib[0][$attr] === '') {
                                                 unset($tagAttrib[0][$attr]);
                                             }
                                             if ((string)$params['removeIfEquals'] !== '' && $this->caseShift($tagAttrib[0][$attr], $params['casesensitiveComp']) === $this->caseShift($params['removeIfEquals'], $params['casesensitiveComp'])) {
@@ -585,7 +585,7 @@ class HtmlParser
                                     }
                                     if ($endTag) {
                                         $correctTag = 1;
-                                        if ($tags[$tagName]['nesting'] == 'global') {
+                                        if ($tags[$tagName]['nesting'] === 'global') {
                                             $lastEl = end($tagStack);
                                             if ($tagName !== $lastEl) {
                                                 if (in_array($tagName, $tagStack)) {
@@ -606,13 +606,13 @@ class HtmlParser
                                             $setTag = 0;
                                         } else {
                                             array_pop($tagRegister[$tagName]);
-                                            if ($tags[$tagName]['nesting'] == 'global') {
+                                            if ($tags[$tagName]['nesting'] === 'global') {
                                                 array_pop($tagStack);
                                             }
                                         }
                                     } else {
                                         array_push($tagRegister[$tagName], $c);
-                                        if ($tags[$tagName]['nesting'] == 'global') {
+                                        if ($tags[$tagName]['nesting'] === 'global') {
                                             array_push($tagStack, $tagName);
                                         }
                                     }
@@ -694,7 +694,7 @@ class HtmlParser
             if ($k % 2) {
                 $params = $this->get_tag_attributes($v);
                 // Detect tag-ending so that it is re-applied correctly.
-                $tagEnd = substr($v, -2) == '/>' ? ' />' : '>';
+                $tagEnd = substr($v, -2) === '/>' ? ' />' : '>';
                 // The 'name' of the first tag
                 $firstTagName = $this->getFirstTagName($v);
                 $somethingDone = 0;
index 793505b..ef9fd96 100644 (file)
@@ -200,7 +200,7 @@ class RteHtmlParser extends HtmlParser
 
         // Traverse modes
         foreach ($modes as $cmd) {
-            if ($direction == 'db') {
+            if ($direction === 'db') {
                 // Checking for user defined transformation:
                 if ($_classRef = $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_parsehtml_proc.php']['transformation'][$cmd]) {
                     $_procObj = GeneralUtility::getUserObj($_classRef);
@@ -230,7 +230,7 @@ class RteHtmlParser extends HtmlParser
                             // Do nothing
                     }
                 }
-            } elseif ($direction == 'rte') {
+            } elseif ($direction === 'rte') {
                 // Checking for user defined transformation:
                 if ($_classRef = $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_parsehtml_proc.php']['transformation'][$cmd]) {
                     $_procObj = GeneralUtility::getUserObj($_classRef);
@@ -960,8 +960,8 @@ class RteHtmlParser extends HtmlParser
             // Wrapping the line in <p> tags if not already wrapped and does not contain an hr tag
             if (!preg_match('/<(hr)(\\s[^>\\/]*)?[[:space:]]*\\/?>/i', $parts[$k])) {
                 $testStr = strtolower(trim($parts[$k]));
-                if (substr($testStr, 0, 4) != '<div' || substr($testStr, -6) != '</div>') {
-                    if (substr($testStr, 0, 2) != '<p' || substr($testStr, -4) != '</p>') {
+                if (substr($testStr, 0, 4) !== '<div' || substr($testStr, -6) !== '</div>') {
+                    if (substr($testStr, 0, 2) !== '<p' || substr($testStr, -4) !== '</p>') {
                         // Only set p-tags if there is not already div or p tags:
                         $parts[$k] = '<p>' . $parts[$k] . '</p>';
                     }
@@ -1072,7 +1072,7 @@ class RteHtmlParser extends HtmlParser
     {
         $info = [];
         $url = trim($url);
-        if (substr(strtolower($url), 0, 7) == 'mailto:') {
+        if (substr(strtolower($url), 0, 7) === 'mailto:') {
             $info['url'] = trim(substr($url, 7));
             $info['type'] = 'email';
         } elseif (strpos($url, '?file:') !== false) {
index 15d1ffc..ecef077 100644 (file)
@@ -90,7 +90,7 @@ class ServerRequestFactory
                     $headers[$name] = $value;
                 } elseif (strpos($key, 'CONTENT_') === 0) {
                     $name = substr($key, 8); // Content-
-                    $name = 'Content-' . (($name == 'MD5') ? $name : ucfirst(strtolower($name)));
+                    $name = 'Content-' . (($name === 'MD5') ? $name : ucfirst(strtolower($name)));
                     $name = strtolower($name);
                     $headers[$name] = $value;
                 }
index da00f11..56e0956 100644 (file)
@@ -815,7 +815,7 @@ class GraphicalFunctions
                 foreach ($utf8Chars as $char) {
                     $charInf = $this->ImageTTFBBoxWrapper($conf['fontSize'], $conf['angle'], $conf['fontFile'], $char, $conf['splitRendering.'], $sF);
                     $charW = $charInf[2] - $charInf[0];
-                    $x += $charW + ($char == ' ' ? $wordSpacing : $spacing);
+                    $x += $charW + ($char === ' ' ? $wordSpacing : $spacing);
                 }
             }
         } elseif (isset($conf['breakWidth']) && $conf['breakWidth'] && $this->getRenderedTextWidth($conf['text'], $conf) > $conf['breakWidth']) {
@@ -933,7 +933,7 @@ class GraphicalFunctions
                 $charInf = $this->ImageTTFBBoxWrapper($fontSize, $angle, $fontFile, $char, $splitRenderingConf, $sF);
                 $charW = $charInf[2] - $charInf[0];
                 $this->ImageTTFTextWrapper($im, $fontSize, $angle, $x, $y, $Fcolor, $fontFile, $char, $splitRenderingConf, $sF);
-                $x += $charW + ($char == ' ' ? $wordSpacing : $spacing);
+                $x += $charW + ($char === ' ' ? $wordSpacing : $spacing);
             }
         }
     }
@@ -2679,7 +2679,7 @@ class GraphicalFunctions
         }
 
         $ext = strtolower(substr($theFile, -4, 4));
-        if ((string)$ext == '.png' && $output_png || (string)$ext == '.gif' && !$output_png) {
+        if ((string)$ext === '.png' && $output_png || (string)$ext === '.gif' && !$output_png) {
             return $theFile;
         }
 
@@ -2762,7 +2762,7 @@ class GraphicalFunctions
      */
     public function gif_or_jpg($type, $w, $h)
     {
-        if ($type == 'ai' || $w * $h < $this->pixelLimitGif) {
+        if ($type === 'ai' || $w * $h < $this->pixelLimitGif) {
             return $this->gifExtension;
         } else {
             return 'jpg';
index b8c8e85..82be6f7 100644 (file)
@@ -144,7 +144,7 @@ class DatabaseIntegrityCheck
             // If all records should be shown, do so:
             if ($this->genTree_includeRecords) {
                 foreach ($GLOBALS['TCA'] as $tableName => $cfg) {
-                    if ($tableName != 'pages') {
+                    if ($tableName !== 'pages') {
                         $this->genTree_records($newID, '', $tableName);
                     }
                 }
@@ -245,7 +245,7 @@ class DatabaseIntegrityCheck
                     ];
                     $lostIdList[] = $row['uid'];
                 }
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $this->lostPagesList = implode(',', $lostIdList);
                 }
             }
@@ -349,12 +349,12 @@ class DatabaseIntegrityCheck
         foreach ($GLOBALS['TCA'] as $table => $tableConf) {
             $cols = $GLOBALS['TCA'][$table]['columns'];
             foreach ($cols as $field => $config) {
-                if ($config['config']['type'] == 'group') {
-                    if ((!$mode || $mode == 'file') && $config['config']['internal_type'] == 'file' || (!$mode || $mode == 'db') && $config['config']['internal_type'] == 'db') {
+                if ($config['config']['type'] === 'group') {
+                    if ((!$mode || $mode === 'file') && $config['config']['internal_type'] === 'file' || (!$mode || $mode === 'db') && $config['config']['internal_type'] === 'db') {
                         $result[$table][] = $field;
                     }
                 }
-                if ((!$mode || $mode == 'db') && $config['config']['type'] == 'select' && $config['config']['foreign_table']) {
+                if ((!$mode || $mode === 'db') && $config['config']['type'] === 'select' && $config['config']['foreign_table']) {
                     $result[$table][] = $field;
                 }
             }
@@ -377,7 +377,7 @@ class DatabaseIntegrityCheck
         foreach ($GLOBALS['TCA'] as $table => $tableConf) {
             $cols = $GLOBALS['TCA'][$table]['columns'];
             foreach ($cols as $field => $config) {
-                if ($config['config']['type'] == 'group' && $config['config']['internal_type'] == 'file' && $config['config']['uploadfolder'] == $uploadfolder) {
+                if ($config['config']['type'] === 'group' && $config['config']['internal_type'] === 'file' && $config['config']['uploadfolder'] == $uploadfolder) {
                     $result[] = [$table, $field];
                 }
             }
@@ -397,11 +397,11 @@ class DatabaseIntegrityCheck
         foreach ($GLOBALS['TCA'] as $table => $tableConf) {
             $cols = $GLOBALS['TCA'][$table]['columns'];
             foreach ($cols as $field => $config) {
-                if ($config['config']['type'] == 'group' && $config['config']['internal_type'] == 'db') {
-                    if (trim($config['config']['allowed']) == '*' || strstr($config['config']['allowed'], $theSearchTable)) {
+                if ($config['config']['type'] === 'group' && $config['config']['internal_type'] === 'db') {
+                    if (trim($config['config']['allowed']) === '*' || strstr($config['config']['allowed'], $theSearchTable)) {
                         $result[] = [$table, $field];
                     }
-                } elseif ($config['config']['type'] == 'select' && $config['config']['foreign_table'] == $theSearchTable) {
+                } elseif ($config['config']['type'] === 'select' && $config['config']['foreign_table'] == $theSearchTable) {
                     $result[] = [$table, $field];
                 }
             }
@@ -464,8 +464,8 @@ class DatabaseIntegrityCheck
                         foreach ($fields as $field) {
                             if (trim($row[$field])) {
                                 $fieldConf = $GLOBALS['TCA'][$table]['columns'][$field]['config'];
-                                if ($fieldConf['type'] == 'group') {
-                                    if ($fieldConf['internal_type'] == 'file') {
+                                if ($fieldConf['type'] === 'group') {
+                                    if ($fieldConf['internal_type'] === 'file') {
                                         // Files...
                                         if ($fieldConf['MM']) {
                                             $tempArr = [];
@@ -486,7 +486,7 @@ class DatabaseIntegrityCheck
                                             }
                                         }
                                     }
-                                    if ($fieldConf['internal_type'] == 'db') {
+                                    if ($fieldConf['internal_type'] === 'db') {
                                         $dbAnalysis = GeneralUtility::makeInstance(RelationHandler::class);
                                         $dbAnalysis->start(
                                             $row[$field],
@@ -501,7 +501,7 @@ class DatabaseIntegrityCheck
                                         }
                                     }
                                 }
-                                if ($fieldConf['type'] == 'select' && $fieldConf['foreign_table']) {
+                                if ($fieldConf['type'] === 'select' && $fieldConf['foreign_table']) {
                                     $dbAnalysis = GeneralUtility::makeInstance(RelationHandler::class);
                                     $dbAnalysis->start(
                                         $row[$field],
@@ -679,7 +679,7 @@ class DatabaseIntegrityCheck
                 // Now this is the field, where the reference COULD come from.
                 // But we're not guaranteed, so we must carefully examine the data.
                 $fieldConf = $GLOBALS['TCA'][$table]['columns'][$field]['config'];
-                $allowedTables = $fieldConf['type'] == 'group' ? $fieldConf['allowed'] : $fieldConf['foreign_table'];
+                $allowedTables = $fieldConf['type'] === 'group' ? $fieldConf['allowed'] : $fieldConf['foreign_table'];
                 $dbAnalysis = GeneralUtility::makeInstance(RelationHandler::class);
                 $dbAnalysis->start($row[$field], $allowedTables, $fieldConf['MM'], $row['uid'], $table, $fieldConf);
                 foreach ($dbAnalysis->itemArray as $tempArr) {
index bfc5683..85b5c78 100644 (file)
@@ -298,7 +298,7 @@ class Rfc822AddressesParser
         $string = $parts[0];
         $partsCounter = count($parts);
         for ($i = 0; $i < $partsCounter; $i++) {
-            if ($this->_hasUnclosedQuotes($string) || $this->_hasUnclosedBrackets($string, '<>') || $this->_hasUnclosedBrackets($string, '[]') || $this->_hasUnclosedBrackets($string, '()') || substr($string, -1) == '\\') {
+            if ($this->_hasUnclosedQuotes($string) || $this->_hasUnclosedBrackets($string, '<>') || $this->_hasUnclosedBrackets($string, '[]') || $this->_hasUnclosedBrackets($string, '()') || substr($string, -1) === '\\') {
                 if (isset($parts[$i + 1])) {
                     $string = $string . $char . $parts[$i + 1];
                 } else {
@@ -379,7 +379,7 @@ class Rfc822AddressesParser
         $parts = explode($char, $string);
         $partsCounter = count($parts);
         for ($i = 0; $i < $partsCounter; $i++) {
-            if (substr($parts[$i], -1) == '\\' || $this->_hasUnclosedQuotes($parts[$i])) {
+            if (substr($parts[$i], -1) === '\\' || $this->_hasUnclosedQuotes($parts[$i])) {
                 $num--;
             }
             if (isset($parts[$i + 1])) {
@@ -743,7 +743,7 @@ class Rfc822AddressesParser
      */
     protected function _validateDliteral($dliteral)
     {
-        return !preg_match('/(.)[][\\x0D\\\\]/', $dliteral, $matches) && $matches[1] != '\\';
+        return !preg_match('/(.)[][\\x0D\\\\]/', $dliteral, $matches) && $matches[1] !== '\\';
     }
 
     /**
index 75b1b0a..cc41f4d 100644 (file)
@@ -982,7 +982,7 @@ class LocalDriver extends AbstractHierarchicalFilesystemDriver
         $identifierMap = [];
         $identifierMap[$sourceFolderIdentifier] = $targetFolderIdentifier;
         foreach ($filesAndFolders as $oldItem) {
-            if ($oldItem['type'] == 'dir') {
+            if ($oldItem['type'] === 'dir') {
                 $oldIdentifier = $oldItem['identifier'];
                 $newIdentifier = $this->canonicalizeAndCheckFolderIdentifier(
                     str_replace($sourceFolderIdentifier, $targetFolderIdentifier, $oldItem['identifier'])
index 7530d82..c4d3b69 100644 (file)
@@ -479,7 +479,7 @@ class ResourceFactory implements ResourceFactoryInterface, \TYPO3\CMS\Core\Singl
             if (MathUtility::canBeInterpretedAsInteger($prefix)) {
                 // path or folder in a valid storageUID
                 return $this->getObjectFromCombinedIdentifier($input);
-            } elseif ($prefix == 'EXT') {
+            } elseif ($prefix === 'EXT') {
                 $input = GeneralUtility::getFileAbsFileName($input);
                 if (empty($input)) {
                     return null;
index 67aea8d..b016595 100644 (file)
@@ -509,7 +509,7 @@ class TimeTracker implements SingletonInterface
      */
     protected function fixCLen($c, $v)
     {
-        $len = $v == 'FILE' ? $this->printConf['contentLength_FILE'] : $this->printConf['contentLength'];
+        $len = $v === 'FILE' ? $this->printConf['contentLength_FILE'] : $this->printConf['contentLength'];
         if (strlen($c) > $len) {
             $c = '<span style="color:green;">' . htmlspecialchars(GeneralUtility::fixed_lgd_cs($c, $len)) . '</span>';
         } else {
index 5beec4d..888ece5 100644 (file)
@@ -414,9 +414,9 @@ class ExtendedTemplateService extends TemplateService
         $keyArr_alpha = [];
         foreach ($arr as $key => $value) {
             // Don't do anything with comments / linenumber registrations...
-            if (substr($key, -2) != '..') {
+            if (substr($key, -2) !== '..') {
                 $key = preg_replace('/\\.$/', '', $key);
-                if (substr($key, -1) != '.') {
+                if (substr($key, -1) !== '.') {
                     if (MathUtility::canBeInterpretedAsInteger($key)) {
                         $keyArr_num[$key] = $arr[$key];
                     } else {
@@ -463,7 +463,7 @@ class ExtendedTemplateService extends TemplateService
                             $urlParameters['breakPointLN'] = GeneralUtility::_GP('breakPointLN');
                         }
                         $aHref = BackendUtility::getModuleUrl('web_ts', $urlParameters);
-                        if ($this->bType != 'const') {
+                        if ($this->bType !== 'const') {
                             $ln = is_array($arr[$key . '.ln..']) ? 'Defined in: ' . $this->lineNumberToScript($arr[$key . '.ln..']) : 'N/A';
                         } else {
                             $ln = '';
@@ -573,7 +573,7 @@ class ExtendedTemplateService extends TemplateService
         $keyArr = [];
         foreach ($arr as $key => $value) {
             $key = preg_replace('/\\.$/', '', $key);
-            if (substr($key, -1) != '.') {
+            if (substr($key, -1) !== '.') {
                 $keyArr[$key] = 1;
             }
         }
@@ -657,7 +657,7 @@ class ExtendedTemplateService extends TemplateService
         $keyArr = [];
         foreach ($arr as $key => $value) {
             $key = preg_replace('/\\.$/', '', $key);
-            if (substr($key, -1) != '.') {
+            if (substr($key, -1) !== '.') {
                 $keyArr[$key] = 1;
             }
         }
@@ -781,7 +781,7 @@ class ExtendedTemplateService extends TemplateService
     {
         if ($chars >= 4) {
             if (strlen($string) > $chars) {
-                if (strlen($string) > 24 && substr($string, 0, 12) == '##' . $this->Cmarker . '_B##') {
+                if (strlen($string) > 24 && substr($string, 0, 12) === '##' . $this->Cmarker . '_B##') {
                     return '##' . $this->Cmarker . '_B##' . GeneralUtility::fixed_lgd_cs(substr($string, 12, -12), ($chars - 3))
                         . '##' . $this->Cmarker . '_E##';
                 } else {
@@ -800,7 +800,7 @@ class ExtendedTemplateService extends TemplateService
     public function ext_lnBreakPointWrap($lineNumber, $str)
     {
         return '<a href="#" id="line-' . $lineNumber . '" onClick="return brPoint(' . $lineNumber . ','
-            . ($this->ext_lineNumberOffset_mode == 'setup' ? 1 : 0) . ');">' . $str . '</a>';
+            . ($this->ext_lineNumberOffset_mode === 'setup' ? 1 : 0) . ');">' . $str . '</a>';
     }
 
     /**
@@ -826,9 +826,9 @@ class ExtendedTemplateService extends TemplateService
             }
             $cArr[$k] = $lineNum . str_replace(' ', '&nbsp;', $v);
             $firstChar = substr(trim($v), 0, 1);
-            if ($firstChar == '[') {
+            if ($firstChar === '[') {
                 $cArr[$k] = '<strong class="text-success">' . $cArr[$k] . '</strong>';
-            } elseif ($firstChar == '/' || $firstChar == '#') {
+            } elseif ($firstChar === '/' || $firstChar === '#') {
                 if ($comments) {
                     $cArr[$k] = '<span class="text-muted">' . $cArr[$k] . '</span>';
                 } else {
index 56dcae6..8081ba7 100644 (file)
@@ -902,7 +902,7 @@ class TemplateService
         }
         // If "Default (include before if root flag is set)" is set OR
         // "Always include before this template record" AND root-flag are set
-        if ($row['static_file_mode'] == 1 || $row['static_file_mode'] == 0 && substr($templateID, 0, 4) == 'sys_' && $row['root']) {
+        if ($row['static_file_mode'] == 1 || $row['static_file_mode'] == 0 && substr($templateID, 0, 4) === 'sys_' && $row['root']) {
             $this->addExtensionStatics($idList, $templateID, $pid, $row);
         }
         // Include Static Template Records after all other TypoScript has been included.
@@ -1122,7 +1122,7 @@ class TemplateService
         unset($this->setup['types']);
         if (is_array($this->setup)) {
             foreach ($this->setup as $key => $value) {
-                if ($value == 'PAGE') {
+                if ($value === 'PAGE') {
                     // Set the typeNum of the current page object:
                     if (isset($this->setup[$key . '.']['typeNum'])) {
                         $typeNum = $this->setup[$key . '.']['typeNum'];
@@ -1496,7 +1496,7 @@ class TemplateService
     public function removeQueryString($url)
     {
         GeneralUtility::logDeprecatedFunction();
-        if (substr($url, -1) == '?') {
+        if (substr($url, -1) === '?') {
             return substr($url, 0, -1);
         } else {
             return $url;
@@ -1656,7 +1656,7 @@ class TemplateService
             // Traverse rootpoints:
             foreach ($rootPoints as $p) {
                 $initMParray = [];
-                if ($p == 'root') {
+                if ($p === 'root') {
                     $p = $this->rootLine[0]['uid'];
                     if ($this->rootLine[0]['_MOUNT_OL'] && $this->rootLine[0]['_MP_PARAM']) {
                         $initMParray[] = $this->rootLine[0]['_MP_PARAM'];
index 0e57ad3..a64ff2b 100644 (file)
@@ -215,7 +215,7 @@ class ClientUtility
         // wap
         $browser = substr($userAgent, 0, 4);
         $wapviwer = substr(stristr($userAgent, 'wap'), 0, 3);
-        if ($wapviwer == 'wap' || $browser == 'noki' || $browser == 'eric' || $browser == 'r380' || $browser == 'up.b' || $browser == 'winw' || $browser == 'wapa') {
+        if ($wapviwer === 'wap' || $browser === 'noki' || $browser === 'eric' || $browser == 'r380' || $browser === 'up.b' || $browser === 'winw' || $browser === 'wapa') {
             $deviceType = 'wap';
         }
         // grabber
index 81b7910..dad5530 100644 (file)
@@ -1199,7 +1199,7 @@ class ExtensionManagementUtility
                 }
                 // Select a subtype randomly
                 // Useful to start a service by service key without knowing his subtypes - for testing purposes
-                if ($serviceSubType == '*') {
+                if ($serviceSubType === '*') {
                     $serviceSubType = key($info['serviceSubTypes']);
                 }
                 // This matches empty subtype too
index 1ff33de..9d8ffb4 100644 (file)
@@ -1443,7 +1443,7 @@ class GeneralUtility
         $attributes = [];
         foreach ($components as $key => $val) {
             // Only if $name is set (if there is an attribute, that waits for a value), that valuemode is enabled. This ensures that the attribute is assigned it's value
-            if ($val != '=') {
+            if ($val !== '=') {
                 if ($valuemode) {
                     if ($name) {
                         $attributes[$name] = $val;
@@ -1591,7 +1591,7 @@ class GeneralUtility
         foreach ($vals as $key => $val) {
             $type = $val['type'];
             // open tag:
-            if ($type == 'open' || $type == 'complete') {
+            if ($type === 'open' || $type === 'complete') {
                 $stack[$stacktop++] = $tagi;
                 if ($depth == $stacktop) {
                     $startPoint = $key;
@@ -1605,7 +1605,7 @@ class GeneralUtility
                 }
             }
             // finish tag:
-            if ($type == 'complete' || $type == 'close') {
+            if ($type === 'complete' || $type === 'close') {
                 $oldtagi = $tagi;
                 $tagi = $stack[--$stacktop];
                 $oldtag = $oldtagi['tag'];
@@ -1622,7 +1622,7 @@ class GeneralUtility
                 unset($oldtagi);
             }
             // cdata
-            if ($type == 'cdata') {
+            if ($type === 'cdata') {
                 $tagi['values'][] = $val['value'];
             }
         }
@@ -1753,7 +1753,7 @@ class GeneralUtility
                     // Otherwise, just htmlspecialchar the stuff:
                     $content = htmlspecialchars($v);
                     $dType = gettype($v);
-                    if ($dType == 'string') {
+                    if ($dType === 'string') {
                         if ($options['useCDATA'] && $content != $v) {
                             $content = '<![CDATA[' . $v . ']]>';
                         }
@@ -1925,14 +1925,14 @@ class GeneralUtility
         foreach ($vals as $val) {
             $type = $val['type'];
             // Open tag:
-            if ($type == 'open' || $type == 'complete') {
+            if ($type === 'open' || $type === 'complete') {
                 $XMLcontent .= '<' . $val['tag'];
                 if (isset($val['attributes'])) {
                     foreach ($val['attributes'] as $k => $v) {
                         $XMLcontent .= ' ' . $k . '="' . htmlspecialchars($v) . '"';
                     }
                 }
-                if ($type == 'complete') {
+                if ($type === 'complete') {
                     if (isset($val['value'])) {
                         $XMLcontent .= '>' . htmlspecialchars($val['value']) . '</' . $val['tag'] . '>';
                     } else {
@@ -1941,16 +1941,16 @@ class GeneralUtility
                 } else {
                     $XMLcontent .= '>';
                 }
-                if ($type == 'open' && isset($val['value'])) {
+                if ($type === 'open' && isset($val['value'])) {
                     $XMLcontent .= htmlspecialchars($val['value']);
                 }
             }
             // Finish tag:
-            if ($type == 'close') {
+            if ($type === 'close') {
                 $XMLcontent .= '</' . $val['tag'] . '>';
             }
             // Cdata
-            if ($type == 'cdata') {
+            if ($type === 'cdata') {
                 $XMLcontent .= htmlspecialchars($val['value']);
             }
         }
@@ -2345,7 +2345,7 @@ class GeneralUtility
             if (!is_link($path) && is_dir($path)) {
                 if ($removeNonEmpty == true && ($handle = @opendir($path))) {
                     while ($OK && false !== ($file = readdir($handle))) {
-                        if ($file == '.' || $file == '..') {
+                        if ($file === '.' || $file === '..') {
                             continue;
                         }
                         $OK = static::rmdir($path . '/' . $file, $removeNonEmpty);
@@ -2418,7 +2418,7 @@ class GeneralUtility
                 $dir = scandir($path);
                 $dirs = [];
                 foreach ($dir as $entry) {
-                    if (is_dir($path . '/' . $entry) && $entry != '..' && $entry != '.') {
+                    if (is_dir($path . '/' . $entry) && $entry !== '..' && $entry !== '.') {
                         $dirs[] = $entry;
                     }
                 }
@@ -2676,7 +2676,7 @@ class GeneralUtility
         $path = self::resolveBackPath(self::dirname(PATH_thisScript) . '/' . $lookupFile[0]);
 
         $doNothing = false;
-        if (TYPO3_MODE == 'FE') {
+        if (TYPO3_MODE === 'FE') {
             $mode = strtolower($GLOBALS['TYPO3_CONF_VARS'][TYPO3_MODE]['versionNumberInFilename']);
             if ($mode === 'embed') {
                 $mode = true;
@@ -3004,7 +3004,7 @@ class GeneralUtility
                 } elseif (defined('PATH_thisScript') && defined('PATH_site')) {
                     $lPath = PathUtility::stripPathSitePrefix(dirname(PATH_thisScript)) . '/';
                     $siteUrl = substr($url, 0, -strlen($lPath));
-                    if (substr($siteUrl, -1) != '/') {
+                    if (substr($siteUrl, -1) !== '/') {
                         $siteUrl .= '/';
                     }
                     $retVal = $siteUrl;
@@ -3018,7 +3018,7 @@ class GeneralUtility
                 break;
             case 'TYPO3_SSL':
                 $proxySSL = trim($GLOBALS['TYPO3_CONF_VARS']['SYS']['reverseProxySSL']);
-                if ($proxySSL == '*') {
+                if ($proxySSL === '*') {
                     $proxySSL = $GLOBALS['TYPO3_CONF_VARS']['SYS']['reverseProxyIP'];
                 }
                 if (self::cmpIP($_SERVER['REMOTE_ADDR'], $proxySSL)) {
@@ -4275,8 +4275,8 @@ class GeneralUtility
         // Init TYPO3 logging
         foreach (explode(';', $GLOBALS['TYPO3_CONF_VARS']['SYS']['systemLog'], 2) as $log) {
             list($type, $destination) = explode(',', $log, 3);
-            if ($type == 'syslog') {
-                if (TYPO3_OS == 'WIN') {
+            if ($type === 'syslog') {
+                if (TYPO3_OS === 'WIN') {
                     $facility = LOG_USER;
                 } else {
                     $facility = constant('LOG_' . strtoupper($destination));
@@ -4333,14 +4333,14 @@ class GeneralUtility
             }
             $msgLine = ' - ' . $extKey . ': ' . $msg;
             // Write message to a file
-            if ($type == 'file') {
+            if ($type === 'file') {
                 $file = fopen($destination, 'a');
                 if ($file) {
                     fwrite($file, date(($dateFormat . ' ' . $timeFormat)) . $msgLine . LF);
                     fclose($file);
                     self::fixPermissions($destination);
                 }
-            } elseif ($type == 'mail') {
+            } elseif ($type === 'mail') {
                 list($to, $from) = explode('/', $destination);
                 if (!self::validEmail($from)) {
                     $from = MailUtility::getSystemFrom();
@@ -4349,9 +4349,9 @@ class GeneralUtility
                 $mail = self::makeInstance(\TYPO3\CMS\Core\Mail\MailMessage::class);
                 $mail->setTo($to)->setFrom($from)->setSubject('Warning - error in TYPO3 installation')->setBody('Host: ' . $GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_div.php']['systemLogHost'] . LF . 'Extension: ' . $extKey . LF . 'Severity: ' . $severity . LF . LF . $msg);
                 $mail->send();
-            } elseif ($type == 'error_log') {
+            } elseif ($type === 'error_log') {
                 error_log($GLOBALS['TYPO3_CONF_VARS']['SC_OPTIONS']['t3lib/class.t3lib_div.php']['systemLogHost'] . $msgLine, 0);
-            } elseif ($type == 'syslog') {
+            } elseif ($type === 'syslog') {
                 $priority = [LOG_INFO, LOG_NOTICE, LOG_WARNING, LOG_ERR, LOG_CRIT];
                 syslog($priority[(int)$severity], $msgLine);
             }
index 855ada3..3635876 100644 (file)
@@ -129,34 +129,34 @@ class MathUtility
         foreach ($regSliced as $k => $v) {
             $v = (float)$v;
             $sign = $reg[1][$k];
-            if ($sign == '+' || $sign == '-') {
-                $Msign == '-' ? ($number -= $buffer) : ($number += $buffer);
+            if ($sign === '+' || $sign === '-') {
+                $Msign === '-' ? ($number -= $buffer) : ($number += $buffer);
                 $Msign = $sign;
                 $buffer = $v;
             } else {
-                if ($sign == '/') {
+                if ($sign === '/') {
                     if ($v) {
                         $buffer /= $v;
                     } else {
                         $err = 'dividing by zero';
                     }
                 }
-                if ($sign == '%') {
+                if ($sign === '%') {
                     if ($v) {
                         $buffer %= $v;
                     } else {
                         $err = 'dividing by zero';
                     }
                 }
-                if ($sign == '*') {
+                if ($sign === '*') {
                     $buffer *= $v;
                 }
-                if ($sign == '^') {
+                if ($sign === '^') {
                     $buffer = pow($buffer, $v);
                 }
             }
         }
-        $number = $Msign == '-' ? ($number -= $buffer) : ($number += $buffer);
+        $number = $Msign === '-' ? ($number -= $buffer) : ($number += $buffer);
         return $err ? 'ERROR: ' . $err : $number;
     }
 
index 91fa80d..0feedcf 100644 (file)
@@ -31,7 +31,7 @@ class LocalDriverFilenameFilter
      */
     public static function filterFilename($itemName, $itemIdentifier, $parentIdentifier, array $additionalInformation, \TYPO3\CMS\Core\Resource\Driver\AbstractDriver $driverInstance)
     {
-        if ($itemName == 'fileA' || $itemName == 'folderA/') {
+        if ($itemName === 'fileA' || $itemName === 'folderA/') {
             return -1;
         } else {
             return true;
index 2c0c486..c59afc6 100644 (file)
@@ -606,7 +606,7 @@ class LocalDriverTest extends \TYPO3\CMS\Core\Tests\Unit\Resource\BaseTestCase
      */
     public function createFileFixesPermissionsOnCreatedFile()
     {
-        if (TYPO3_OS == 'WIN') {
+        if (TYPO3_OS === 'WIN') {
             $this->markTestSkipped('createdFilesHaveCorrectRights() tests not available on Windows');
         }
 
index af0ed9f..5bb8dfc 100644 (file)
@@ -93,7 +93,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                 $quotedInput = '';
             }
             // Generate id prefix for accessible header
-            $headerScope = $headerPos == 'top' ? 'col' : 'row';
+            $headerScope = $headerPos === 'top' ? 'col' : 'row';
             $headerIdPrefix = $headerScope . $this->cObj->data['uid'] . '-';
             // Split into single lines (will become table-rows):
             $rows = GeneralUtility::trimExplode(LF, $content);
@@ -115,16 +115,16 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                     }
                     $cells[$a] = preg_replace('|<br */?>|i', LF, $cells[$a]);
                     $cellAttribs = $noStyles ? '' : ($a > 0 && $cols - 1 == $a ? ' class="td-last td-' . $a . '"' : ' class="td-' . $a . '"');
-                    if ($headerPos == 'top' && !$k || $headerPos == 'left' && !$a) {
+                    if ($headerPos === 'top' && !$k || $headerPos === 'left' && !$a) {
                         $scope = ' scope="' . $headerScope . '"';
-                        $scope .= ' id="' . $headerIdPrefix . ($headerScope == 'col' ? $a : $k) . '"';
+                        $scope .= ' id="' . $headerIdPrefix . ($headerScope === 'col' ? $a : $k) . '"';
                         $newCells[$a] = '
                                                        <th' . $cellAttribs . $scope . '>' . $this->cObj->stdWrap($cells[$a], $conf['innerStdWrap.']) . '</th>';
                     } else {
                         if (empty($headerPos)) {
                             $accessibleHeader = '';
                         } else {
-                            $accessibleHeader = ' headers="' . $headerIdPrefix . ($headerScope == 'col' ? $a : $k) . '"';
+                            $accessibleHeader = ' headers="' . $headerIdPrefix . ($headerScope === 'col' ? $a : $k) . '"';
                         }
                         $newCells[$a] = '
                                                        <td' . $cellAttribs . $accessibleHeader . '>' . $this->cObj->stdWrap($cells[$a], $conf['innerStdWrap.']) . '</td>';
@@ -144,7 +144,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                 $tableContents .= '
                                        <caption>' . $caption . '</caption>';
             }
-            if ($headerPos == 'top' && $rows[0]) {
+            if ($headerPos === 'top' && $rows[0]) {
                 $tableContents .= '<thead>' . $rows[0] . '
                                        </thead>';
                 unset($rows[0]);
@@ -273,7 +273,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
         }
         $renderMethod = $this->cObj->stdWrap($conf['renderMethod'], $conf['renderMethod.']);
         // Render using the default IMGTEXT code (table-based)
-        if (!$renderMethod || $renderMethod == 'table') {
+        if (!$renderMethod || $renderMethod === 'table') {
             return $this->cObj->cObjGetSingle('IMGTEXT', $conf);
         }
         $restoreRegisters = false;
@@ -534,7 +534,7 @@ class CssStyledContentController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlug
                         $imgConf['file.']['m.'] = $conf['image_frames.'][$image_frames . '.'];
                     }
                 }
-                if ($image_compression && $imgConf['file'] != 'GIFBUILDER') {
+                if ($image_compression && $imgConf['file'] !== 'GIFBUILDER') {
                     if ($image_compression == 1) {
                         $tempImport = $imgConf['file.']['import'];
                         $tempImport_dot = $imgConf['file.']['import.'];
index 78a262b..d039292 100644 (file)
@@ -586,7 +586,7 @@ class Typo3DbBackend implements BackendInterface, SingletonInterface
                 }
             }
             $pageRepository->versionOL($tableName, $row, true);
-            if ($tableName == 'pages') {
+            if ($tableName === 'pages') {
                 $row = $pageRepository->getPageOverlay($row, $querySettings->getLanguageUid());
             } elseif (isset($GLOBALS['TCA'][$tableName]['ctrl']['languageField'])
                       && $GLOBALS['TCA'][$tableName]['ctrl']['languageField'] !== ''
index 661dba9..4bd5928 100644 (file)
@@ -662,7 +662,7 @@ class Typo3DbQueryParserTest extends \TYPO3\CMS\Components\TestingFramework\Core
         $mockEnvironmentService = $this->getMockBuilder(\TYPO3\CMS\Extbase\Service\EnvironmentService::class)
             ->setMethods(['isEnvironmentInFrontendMode'])
             ->getMock();
-        $mockEnvironmentService->expects($this->any())->method('isEnvironmentInFrontendMode')->will($this->returnValue($mode == 'FE'));
+        $mockEnvironmentService->expects($this->any())->method('isEnvironmentInFrontendMode')->will($this->returnValue($mode === 'FE'));
 
         $mockTypo3DbQueryParser = $this->getAccessibleMock(Typo3DbQueryParser::class, ['dummy'], [], '', false);
         $mockTypo3DbQueryParser->_set('environmentService', $mockEnvironmentService);
@@ -724,7 +724,7 @@ class Typo3DbQueryParserTest extends \TYPO3\CMS\Components\TestingFramework\Core
         $mockEnvironmentService = $this->getMockBuilder(\TYPO3\CMS\Extbase\Service\EnvironmentService::class)
             ->setMethods(['isEnvironmentInFrontendMode'])
             ->getMock();
-        $mockEnvironmentService->expects($this->any())->method('isEnvironmentInFrontendMode')->will($this->returnValue($mode == 'FE'));
+        $mockEnvironmentService->expects($this->any())->method('isEnvironmentInFrontendMode')->will($this->returnValue($mode === 'FE'));
 
         $mockTypo3DbQueryParser = $this->getAccessibleMock(Typo3DbQueryParser::class, ['dummy'], [], '', false);
         $mockTypo3DbQueryParser->_set('environmentService', $mockEnvironmentService);
index bc1f3c6..d15d62f 100644 (file)
@@ -95,7 +95,7 @@ class TerUtility
         gzuncompress($dat);
         // compare hashes ignoring any leading whitespace. See bug #0000365.
         if (ltrim($parts[0]) == md5($dat)) {
-            if ($parts[1] == 'gzcompress') {
+            if ($parts[1] === 'gzcompress') {
                 if (function_exists('gzuncompress')) {
                     $dat = gzuncompress($dat);
                 } else {
@@ -123,7 +123,7 @@ class TerUtility
     public function decodeExchangeData($stream)
     {
         $parts = explode(':', $stream, 3);
-        if ($parts[1] == 'gzcompress') {
+        if ($parts[1] === 'gzcompress') {
             if (function_exists('gzuncompress')) {
                 $parts[2] = gzuncompress($parts[2]);
             } else {
index 1903abf..97c6c4c 100644 (file)
@@ -50,7 +50,7 @@ class ExtensionXmlPullParser extends AbstractExtensionXmlParser
     public function parseXml($file)
     {
         $this->createParser();
-        if (!(is_object($this->objXml) && get_class($this->objXml) == 'XMLReader')) {
+        if (!(is_object($this->objXml) && get_class($this->objXml) === 'XMLReader')) {
             throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Unable to create XML parser.', 1342640540);
         }
         if ($this->objXml->open($file, 'utf-8') === false) {
index e49f9d7..2778583 100644 (file)
@@ -52,7 +52,7 @@ class MirrorXmlPullParser extends AbstractMirrorXmlParser
     public function parseXml($file)
     {
         $this->createParser();
-        if (!(is_object($this->objXml) && get_class($this->objXml) == 'XMLReader')) {
+        if (!(is_object($this->objXml) && get_class($this->objXml) === 'XMLReader')) {
             throw new \TYPO3\CMS\Extensionmanager\Exception\ExtensionManagerException('Unable to create XML parser.', 1342640820);
         }
         if ($this->objXml->open($file, 'utf-8') === false) {
index 4edbe04..1bc68c6 100644 (file)
@@ -85,7 +85,7 @@ class DownloadExtensionViewHelper extends \TYPO3\CMS\Fluid\ViewHelpers\Form\Abst
         $pathSelector = '<ul class="is-hidden">';
         foreach ($installPaths as $installPathType => $installPath) {
             $pathSelector .= '<li>
-                               <input type="radio" id="' . htmlspecialchars($extension->getExtensionKey()) . '-downloadPath-' . htmlspecialchars($installPathType) . '" name="' . htmlspecialchars($this->getFieldNamePrefix()) . '[downloadPath]" class="downloadPath" value="' . htmlspecialchars($installPathType) . '" ' . ($installPathType == 'Local' ? 'checked="checked"' : '') . ' />
+                               <input type="radio" id="' . htmlspecialchars($extension->getExtensionKey()) . '-downloadPath-' . htmlspecialchars($installPathType) . '" name="' . htmlspecialchars($this->getFieldNamePrefix()) . '[downloadPath]" class="downloadPath" value="' . htmlspecialchars($installPathType) . '" ' . ($installPathType === 'Local' ? 'checked="checked"' : '') . ' />
                                <label for="' . htmlspecialchars($extension->getExtensionKey()) . '-downloadPath-' . htmlspecialchars($installPathType) . '">' . htmlspecialchars($installPathType) . '</label>
                        </li>';
         }
index 9cda050..f41b232 100644 (file)
@@ -233,7 +233,7 @@ class FrontendEditPanel
         } elseif ($conf['beforeLastTag'] > 0) {
             $cBuf = rtrim($content);
             $secureCount = 30;
-            while ($secureCount && substr($cBuf, -1) == '>' && substr($cBuf, -4) != '</a>') {
+            while ($secureCount && substr($cBuf, -1) === '>' && substr($cBuf, -4) !== '</a>') {
                 $cBuf = rtrim(preg_replace('/<[^<]*>$/', '', $cBuf));
                 $secureCount--;
             }
@@ -259,12 +259,12 @@ class FrontendEditPanel
     protected function editPanelLinkWrap($string, $formName, $cmd, $currentRecord = '', $confirm = '', $nPid = '')
     {
         $nV = GeneralUtility::_GP('ADMCMD_view') ? 1 : 0;
-        if ($cmd == 'edit') {
+        if ($cmd === 'edit') {
             $rParts = explode(':', $currentRecord);
             $out = $this->editPanelLinkWrap_doWrap($string, BackendUtility::getModuleUrl('record_edit', ['edit[' . $rParts[0] . '][' . $rParts[1] . ']' => 'edit', 'noView' => $nV, 'feEdit' => 1]), $currentRecord);
-        } elseif ($cmd == 'new') {
+        } elseif ($cmd === 'new') {
             $rParts = explode(':', $currentRecord);
-            if ($rParts[0] == 'pages') {
+            if ($rParts[0] === 'pages') {
                 $out = $this->editPanelLinkWrap_doWrap($string, BackendUtility::getModuleUrl('db_new', ['id' => $rParts[1], 'pagesOnly' => 1]), $currentRecord);
             } else {
                 if (!(int)$nPid) {
index e163a68..9295ee0 100644 (file)
@@ -792,20 +792,20 @@ class FrontendLoginController extends \TYPO3\CMS\Frontend\Plugin\AbstractPlugin
                             }
                             break;
                     }
-                } elseif ($this->logintype == '' && $redirMethod == 'login' && $this->conf['redirectPageLogin']) {
+                } elseif ($this->logintype == '' && $redirMethod === 'login' && $this->conf['redirectPageLogin']) {
                     // If login and page not accessible
                     $this->cObj->typoLink('', [
                         'parameter' => $this->conf['redirectPageLogin'],
                         'linkAccessRestrictedPages' => true
                     ]);
                     $redirect_url[] = $this->cObj->lastTypoLinkUrl;
-                } elseif ($this->logintype == '' && $redirMethod == 'logout' && $this->conf['redirectPageLogout'] && $this->frontendController->loginUser) {
+                } elseif ($this->logintype == '' && $redirMethod === 'logout' && $this->conf['redirectPageLogout'] && $this->frontendController->loginUser) {
                     // If logout and page not accessible
                     $redirect_url[] = $this->pi_getPageLink((int)$this->conf['redirectPageLogout']);
                 } elseif ($this->logintype === 'logout') {
                     // after logout
                     // Hook for general actions after after logout has been confirmed
-                    if ($this->logintype === 'logout' && $GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['felogin']['logout_confirmed']) {
+                    if ($GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['felogin']['logout_confirmed']) {
                         $_params = [];
                         foreach ($GLOBALS['TYPO3_CONF_VARS']['EXTCONF']['felogin']['logout_confirmed'] as $_funcRef) {
                             if ($_funcRef) {
index d7bb35d..e113da5 100644 (file)
@@ -349,7 +349,7 @@ class FileListController extends ActionController
             $this->filelist->clipObj->fileMode = 1;
             $this->filelist->clipObj->initializeClipboard();
             $CB = GeneralUtility::_GET('CB');
-            if ($this->cmd == 'setCB') {
+            if ($this->cmd === 'setCB') {
                 $CB['el'] = $this->filelist->clipObj->cleanUpCBC(array_merge(GeneralUtility::_POST('CBH'),
                     (array)GeneralUtility::_POST('CBC')), '_FILE');
             }
@@ -361,7 +361,7 @@ class FileListController extends ActionController
             // Saves
             $this->filelist->clipObj->endClipboard();
             // If the "cmd" was to delete files from the list (clipboard thing), do that:
-            if ($this->cmd == 'delete') {
+            if ($this->cmd === 'delete') {
                 $items = $this->filelist->clipObj->cleanUpCBC(GeneralUtility::_POST('CBC'), '_FILE', 1);
                 if (!empty($items)) {
                     // Make command array:
index 3eee1d2..dd9f9b7 100644 (file)
@@ -373,7 +373,7 @@ class FileList extends AbstractRecordList
             // Header line is drawn
             $theData = [];
             foreach ($this->fieldArray as $v) {
-                if ($v == '_CLIPBOARD_' && $this->clipBoard) {
+                if ($v === '_CLIPBOARD_' && $this->clipBoard) {
                     $cells = [];
                     $table = '_FILE';
                     $elFromTable = $this->clipObj->elFromTable($table);
index f2dd60b..3793313 100644 (file)
@@ -47,10 +47,10 @@ class ConditionMatcher extends AbstractConditionMatcher
                 case 'usergroup':
                     $groupList = $this->getGroupList();
                     // '0,-1' is the default usergroups when not logged in!
-                    if ($groupList != '0,-1') {
+                    if ($groupList !== '0,-1') {
                         $values = GeneralUtility::trimExplode(',', $value, true);
                         foreach ($values as $test) {
-                            if ($test == '*' || GeneralUtility::inList($groupList, $test)) {
+                            if ($test === '*' || GeneralUtility::inList($groupList, $test)) {
                                 return true;
                             }
                         }
@@ -68,7 +68,7 @@ class ConditionMatcher extends AbstractConditionMatcher
                 case 'PIDupinRootline':
                 case 'PIDinRootline':
                     $values = GeneralUtility::trimExplode(',', $value, true);
-                    if ($key == 'PIDinRootline' || !in_array($this->pageId, $values)) {
+                    if ($key === 'PIDinRootline' || !in_array($this->pageId, $values)) {
                         foreach ($values as $test) {
                             foreach ($this->rootline as $rlDat) {
                                 if ($rlDat['uid'] == $test) {
index 80251ee..1b2726c 100644 (file)
@@ -3658,7 +3658,7 @@ class ContentObjectRenderer
         if (is_object($d)) {
             $count = 0;
             while ($entry = $d->read()) {
-                if ($entry != '.' && $entry != '..') {
+                if ($entry !== '.' && $entry !== '..') {
                     // Because of odd PHP-error where <br />-tag is sometimes placed after a filename!!
                     $wholePath = $path . '/' . $entry;
                     if (file_exists($wholePath) && filetype($wholePath) === 'file') {
@@ -3693,7 +3693,7 @@ class ContentObjectRenderer
         }
         // Sort if required
         if (!empty($items['sorting'])) {
-            if (strtolower(trim($data_arr[3])) != 'r') {
+            if (strtolower(trim($data_arr[3])) !== 'r') {
                 asort($items['sorting']);
             } else {
                 arsort($items['sorting']);
@@ -4838,7 +4838,7 @@ class ContentObjectRenderer
                 } else {
                     // If a tag was not a typo tag, then it is just added to the content
                     $stripNL = false;
-                    if (GeneralUtility::inList($allowTags, $tag[0]) || $denyTags != '*' && !GeneralUtility::inList($denyTags, $tag[0])) {
+                    if (GeneralUtility::inList($allowTags, $tag[0]) || $denyTags !== '*' && !GeneralUtility::inList($denyTags, $tag[0])) {
                         $contentAccum[$contentAccumP] .= $data;
                     } else {
                         $contentAccum[$contentAccumP] .= htmlspecialchars($data);
index 1c5ac0a..c801fec 100644 (file)
@@ -1292,7 +1292,7 @@ abstract class AbstractMenuContentObject
                     // Checking if "&L" should be modified so links to non-accessible pages will not happen.
                     if ($this->conf['protectLvar']) {
                         $languageUid = (int)$tsfe->config['config']['sys_language_uid'];
-                        if ($languageUid && ($this->conf['protectLvar'] == 'all' || GeneralUtility::hideIfNotTranslated($data['l18n_cfg']))) {
+                        if ($languageUid && ($this->conf['protectLvar'] === 'all' || GeneralUtility::hideIfNotTranslated($data['l18n_cfg']))) {
                             $olRec = $tsfe->sys_page->getPageOverlay($data['uid'], $languageUid);
                             if (empty($olRec)) {
                                 // If no pages_language_overlay record then page can NOT be accessed in
index 1da4950..1e9ec67 100644 (file)
@@ -64,7 +64,7 @@ class RecordsContentObject extends AbstractContentObject
             // Add tables which have a configuration (note that this may create duplicate entries)
             if (is_array($conf['conf.'])) {
                 foreach ($conf['conf.'] as $key => $value) {
-                    if (substr($key, -1) != '.' && !in_array($key, $tablesArray)) {
+                    if (substr($key, -1) !== '.' && !in_array($key, $tablesArray)) {
                         $tablesArray[] = $key;
                     }
                 }
index 61f13f2..a41075a 100644 (file)
@@ -1316,7 +1316,7 @@ class TypoScriptFrontendController
         if (!$this->loginAllowedInBranch) {
             // Only if there is a login will we run this...
             if ($this->isUserOrGroupSet()) {
-                if ($this->loginAllowedInBranch_mode == 'all') {
+                if ($this->loginAllowedInBranch_mode === 'all') {
                     // Clear out user and group:
                     $this->fe_user->hideActiveLogin();
                     $this->gr_list = '0,-1';
@@ -2707,7 +2707,7 @@ class TypoScriptFrontendController
             if (!empty($this->config['config']['sys_language_isocode_default'])) {
                 $this->sys_language_isocode = $this->config['config']['sys_language_isocode_default'];
             } else {
-                $this->sys_language_isocode = $this->lang != 'default' ? $this->lang : 'en';
+                $this->sys_language_isocode = $this->lang !== 'default' ? $this->lang : 'en';
             }
         }
 
@@ -2754,7 +2754,7 @@ class TypoScriptFrontendController
                 // As str_* methods are locale aware and turkish has no upper case I
                 // Class autoloading and other checks depending on case changing break with turkish locale LC_CTYPE
                 // @see http://bugs.php.net/bug.php?id=35050
-                if (substr($this->config['config']['locale_all'], 0, 2) != 'tr') {
+                if (substr($this->config['config']['locale_all'], 0, 2) !== 'tr') {
                     setlocale(LC_CTYPE, ...$availableLocales);
                 }
                 setlocale(LC_MONETARY, ...$availableLocales);
index be78d32..9f91fa4 100644 (file)
@@ -465,17 +465,17 @@ class GifBuilder extends GraphicalFunctions
                             }
                             break;
                         case 'OUTLINE':
-                            if ($this->setup[$conf['textObjNum']] == 'TEXT' && ($txtConf = $this->checkTextObj($this->setup[$conf['textObjNum'] . '.']))) {
+                            if ($this->setup[$conf['textObjNum']] === 'TEXT' && ($txtConf = $this->checkTextObj($this->setup[$conf['textObjNum'] . '.']))) {
                                 $this->makeOutline($this->im, $conf, $this->workArea, $txtConf);
                             }
                             break;
                         case 'EMBOSS':
-                            if ($this->setup[$conf['textObjNum']] == 'TEXT' && ($txtConf = $this->checkTextObj($this->setup[$conf['textObjNum'] . '.']))) {
+                            if ($this->setup[$conf['textObjNum']] === 'TEXT' && ($txtConf = $this->checkTextObj($this->setup[$conf['textObjNum'] . '.']))) {
                                 $this->makeEmboss($this->im, $conf, $this->workArea, $txtConf);
                             }
                             break;
                         case 'SHADOW':
-                            if ($this->setup[$conf['textObjNum']] == 'TEXT' && ($txtConf = $this->checkTextObj($this->setup[$conf['textObjNum'] . '.']))) {
+                            if ($this->setup[$conf['textObjNum']] === 'TEXT' && ($txtConf = $this->checkTextObj($this->setup[$conf['textObjNum'] . '.']))) {
                                 $this->makeShadow($this->im, $conf, $this->workArea, $txtConf);
                             }
                             break;
@@ -752,11 +752,11 @@ class GifBuilder extends GraphicalFunctions
                 $objParts = explode('.', substr($theVal, 1, -1));
                 $theVal = 0;
                 if (isset($this->objBB[$objParts[0]])) {
-                    if ($objParts[1] == 'w') {
+                    if ($objParts[1] === 'w') {
                         $theVal = $this->objBB[$objParts[0]][0];
-                    } elseif ($objParts[1] == 'h') {
+                    } elseif ($objParts[1] === 'h') {
                         $theVal = $this->objBB[$objParts[0]][1];
-                    } elseif ($objParts[1] == 'lineHeight') {
+                    } elseif ($objParts[1] === 'lineHeight') {
                         $theVal = $this->objBB[$objParts[0]][2]['lineHeight'];
                     }
                     $theVal = (int)$theVal;
@@ -766,15 +766,15 @@ class GifBuilder extends GraphicalFunctions
             } else {
                 $theVal = 0;
             }
-            if ($sign == '-') {
+            if ($sign === '-') {
                 $calculatedValue -= $theVal;
-            } elseif ($sign == '+') {
+            } elseif ($sign === '+') {
                 $calculatedValue += $theVal;
-            } elseif ($sign == '/' && $theVal) {
+            } elseif ($sign === '/' && $theVal) {
                 $calculatedValue = $calculatedValue / $theVal;
-            } elseif ($sign == '*') {
+            } elseif ($sign === '*') {
                 $calculatedValue = $calculatedValue * $theVal;
-            } elseif ($sign == '%' && $theVal) {
+            } elseif ($sign === '%' && $theVal) {
                 $calculatedValue %= $theVal;
             }
         }
index 4c5a2c9..bf6c61a 100644 (file)
@@ -980,7 +980,7 @@ class PageGenerator
     {
         $OK = false;
         // Integer
-        if ($needle == 'int' || $needle == 'integer') {
+        if ($needle === 'int' || $needle === 'integer') {
             if (MathUtility::canBeInterpretedAsInteger($haystack)) {
                 $OK = true;
             }
index 7327b13..72b1207 100644 (file)
@@ -565,7 +565,7 @@ class AbstractPlugin
         // If this has a value the "previous" button is always visible (will be forced if "showFirstLast" is set)
         $alwaysPrev = $showFirstLast ? 1 : $this->pi_alwaysPrev;
         if (isset($this->internal['pagefloat'])) {
-            if (strtoupper($this->internal['pagefloat']) == 'CENTER') {
+            if (strtoupper($this->internal['pagefloat']) === 'CENTER') {
                 $pagefloat = ceil(($maxPages - 1) / 2);
             } else {
                 // pagefloat set as integer. 0 = left, value >= $this->internal['maxPages'] = right
index b9fb41e..e2da71f 100644 (file)
@@ -293,7 +293,7 @@ class AdminPanelView
         // which must change the form value with JavaScript (see "onsubmit" attribute of the "form" element")
         $query .= '<input type="hidden" name="TSFE_ADMIN_PANEL[DUMMY]" value="" />';
         foreach (GeneralUtility::_GET() as $key => $value) {
-            if ($key != 'TSFE_ADMIN_PANEL') {
+            if ($key !== 'TSFE_ADMIN_PANEL') {
                 if (is_array($value)) {
                     $query .= $this->getHiddenFields($key, $value);
                 } else {
index 1958242..7227608 100644 (file)
@@ -53,15 +53,15 @@ class Clickmenu
     {
         $localItems = [];
         // Show import/export on second level menu OR root level.
-        if ($backRef->cmLevel && GeneralUtility::_GP('subname') == 'moreoptions' || $table === 'pages' && $uid == 0) {
+        if ($backRef->cmLevel && GeneralUtility::_GP('subname') === 'moreoptions' || $table === 'pages' && $uid == 0) {
             $LL = $this->includeLL();
             $urlParameters = [
                 'tx_impexp' => [
                     'action' => 'export'
                 ],
-                'id' => ($table == 'pages' ? $uid : $backRef->rec['pid'])
+                'id' => ($table === 'pages' ? $uid : $backRef->rec['pid'])
             ];
-            if ($table == 'pages') {
+            if ($table === 'pages') {
                 $urlParameters['tx_impexp']['pagetree']['id'] = $uid;
                 $urlParameters['tx_impexp']['pagetree']['levels'] = 0;
                 $urlParameters['tx_impexp']['pagetree']['tables'][] = '_ALL';
index 185e3e5..04076ae 100644 (file)
@@ -314,7 +314,7 @@ class ImportExportController extends BaseScriptClass
         }
         // Input data grabbed:
         $inData = GeneralUtility::_GP('tx_impexp');
-        if ((string)$inData['action'] == 'import') {
+        if ((string)$inData['action'] === 'import') {
             if ($this->id && is_array($this->pageinfo) || $this->getBackendUser()->user['admin'] && !$this->id) {
                 if (is_array($this->pageinfo) && $this->pageinfo['uid']) {
                     // View
@@ -357,7 +357,7 @@ class ImportExportController extends BaseScriptClass
         $inData['maxFileSize'] = MathUtility::forceIntegerInRange($inData['maxFileSize'], 1, 1000000, 1000);
         $inData['filename'] = trim(preg_replace('/[^[:alnum:]._-]*/', '', preg_replace('/\\.(t3d|xml)$/', '', $inData['filename'])));
         if (strlen($inData['filename'])) {
-            $inData['filename'] .= $inData['filetype'] == 'xml' ? '.xml' : '.t3d';
+            $inData['filename'] .= $inData['filetype'] === 'xml' ? '.xml' : '.t3d';
         }
         // Set exclude fields in export object:
         if (!is_array($inData['exclude'])) {
@@ -597,7 +597,7 @@ class ImportExportController extends BaseScriptClass
             return;
         }
         foreach ($GLOBALS['TCA'] as $table => $value) {
-            if ($table != 'pages' && (in_array($table, $tables) || in_array('_ALL', $tables))) {
+            if ($table !== 'pages' && (in_array($table, $tables) || in_array('_ALL', $tables))) {
                 if ($this->getBackendUser()->check('tables_select', $table) && !$GLOBALS['TCA'][$table]['ctrl']['is_static']) {
                     $statement = $this->exec_listQueryPid($table, $k, MathUtility::forceIntegerInRange($maxNumber, 1));
                     while ($subTrow = $statement->fetch()) {
index b8f1a6b..ec89b2c 100644 (file)
@@ -457,13 +457,13 @@ class Export extends ImportExport
             }
             foreach ($this->dat['records'][$k]['rels'] as $fieldname => $vR) {
                 // For all DB types of relations:
-                if ($vR['type'] == 'db') {
+                if ($vR['type'] === 'db') {
                     foreach ($vR['itemArray'] as $fI) {
                         $this->export_addDBRelations_registerRelation($fI, $addR);
                     }
                 }
                 // For all flex/db types of relations:
-                if ($vR['type'] == 'flex') {
+                if ($vR['type'] === 'flex') {
                     // DB relations in flex form fields:
                     if (is_array($vR['flexFormRels']['db'])) {
                         foreach ($vR['flexFormRels']['db'] as $subList) {
@@ -575,7 +575,7 @@ class Export extends ImportExport
             }
             foreach ($this->dat['records'][$k]['rels'] as $fieldname => $vR) {
                 // For all file type relations:
-                if ($vR['type'] == 'file') {
+                if ($vR['type'] === 'file') {
                     foreach ($vR['newValueFiles'] as $key => $fI) {
                         $this->export_addFile($fI, $k, $fieldname);
                         // Remove the absolute reference to the file so it doesn't expose absolute paths from source server:
@@ -583,7 +583,7 @@ class Export extends ImportExport
                     }
                 }
                 // For all flex type relations:
-                if ($vR['type'] == 'flex') {
+                if ($vR['type'] === 'flex') {
                     if (is_array($vR['flexFormRels']['file'])) {
                         foreach ($vR['flexFormRels']['file'] as $key => $subList) {
                             foreach ($subList as $subKey => $fI) {
@@ -882,12 +882,12 @@ class Export extends ImportExport
     {
         $list = [];
         foreach ($dbrels as $dat) {
-            if ($dat['type'] == 'db') {
+            if ($dat['type'] === 'db') {
                 foreach ($dat['itemArray'] as $i) {
                     $list[$i['table'] . ':' . $i['id']] = $i;
                 }
             }
-            if ($dat['type'] == 'flex' && is_array($dat['flexFormRels']['db'])) {
+            if ($dat['type'] === 'flex' && is_array($dat['flexFormRels']['db'])) {
                 foreach ($dat['flexFormRels']['db'] as $subList) {
                     foreach ($subList as $i) {
                         $list[$i['table'] . ':' . $i['id']] = $i;
@@ -923,7 +923,7 @@ class Export extends ImportExport
                     }
                 }
             }
-            if ($dat['type'] == 'flex' && is_array($dat['flexFormRels']['softrefs'])) {
+            if ($dat['type'] === 'flex' && is_array($dat['flexFormRels']['softrefs'])) {
                 foreach ($dat['flexFormRels']['softrefs'] as $structurePath => $subSoftrefs) {
                     if (is_array($subSoftrefs['keys'])) {
                         foreach ($subSoftrefs['keys'] as $spKey => $elements) {
@@ -983,7 +983,7 @@ class Export extends ImportExport
      */
     public function compileMemoryToFileContent($type = '')
     {
-        if ($type == 'xml') {
+        if ($type === 'xml') {
             $out = $this->createXML();
         } else {
             $compress = $this->doOutputCompress();
index 1e0af8d..5c9428a 100644 (file)
@@ -716,7 +716,7 @@ class Import extends ImportExport
         if (is_array($this->dat['header']['records'])) {
             foreach ($this->dat['header']['records'] as $table => $recs) {
                 $this->addGeneralErrorsByTable($table);
-                if ($table != 'pages') {
+                if ($table !== 'pages') {
                     foreach ($recs as $uid => $thisRec) {
                         // PID: Set the main $pid, unless a NEW-id is found
                         $setPid = isset($this->import_mapId['pages'][$thisRec['pid']])
@@ -786,7 +786,7 @@ class Import extends ImportExport
                     foreach ($recList as $tableName => $uidList) {
                         // If $mainPid===$newPid then we are on root level and we can consider to move pages as well!
                         // (they will not be in the page tree!)
-                        if (($tableName != 'pages' || !$pagesFromTree[$pid]) && is_array($uidList)) {
+                        if (($tableName !== 'pages' || !$pagesFromTree[$pid]) && is_array($uidList)) {
                             $uidList = array_reverse(array_keys($uidList));
                             foreach ($uidList as $uid) {
                                 if ($this->dontIgnorePid($tableName, $uid)) {
@@ -872,7 +872,7 @@ class Import extends ImportExport
                 $ID = StringUtility::getUniqueId('NEW');
             }
             $this->import_newId[$table . ':' . $ID] = ['table' => $table, 'uid' => $uid];
-            if ($table == 'pages') {
+            if ($table === 'pages') {
                 $this->import_newId_pids[$uid] = $ID;
             }
             // Set main record data:
@@ -1708,7 +1708,7 @@ class Import extends ImportExport
                 $this->error('External import files for the given import source is currently not supported.');
             }
         }
-        if (strtolower($fI['extension']) == 'xml') {
+        if (strtolower($fI['extension']) === 'xml') {
             // XML:
             $xmlContent = file_get_contents($filename);
             if (strlen($xmlContent)) {
index 5fec200..b7e7504 100644 (file)
@@ -372,7 +372,7 @@ abstract class ImportExport
             // Subrecords:
             if (is_array($this->dat['header']['pid_lookup'][$k])) {
                 foreach ($this->dat['header']['pid_lookup'][$k] as $t => $recUidArr) {
-                    if ($t != 'pages') {
+                    if ($t !== 'pages') {
                         foreach ($recUidArr as $ruid => $value) {
                             $this->singleRecordLines($t, $ruid, $lines, $preCode . '&nbsp;&nbsp;&nbsp;&nbsp;');
                         }
@@ -418,7 +418,7 @@ abstract class ImportExport
         // Subrecords
         if (is_array($this->dat['header']['pid_lookup'][$pageUid])) {
             foreach ($this->dat['header']['pid_lookup'][$pageUid] as $table => $recordData) {
-                if ($table != 'pages') {
+                if ($table !== 'pages') {
                     foreach (array_keys($recordData) as $uid) {
                         unset($this->remainHeader['records'][$table][$uid]);
                     }
@@ -448,7 +448,7 @@ abstract class ImportExport
             // Subrecords:
             if (is_array($this->dat['header']['pid_lookup'][$k])) {
                 foreach ($this->dat['header']['pid_lookup'][$k] as $t => $recUidArr) {
-                    if ($t != 'pages') {
+                    if ($t !== 'pages') {
                         foreach ($recUidArr as $ruid => $value) {
                             $this->singleRecordLines($t, $ruid, $lines, '&nbsp;&nbsp;&nbsp;&nbsp;');
                         }
@@ -470,7 +470,7 @@ abstract class ImportExport
     {
         foreach ($pT as $t => $recUidArr) {
             $this->addGeneralErrorsByTable($t);
-            if ($t != 'pages') {
+            if ($t !== 'pages') {
                 $preCode = '';
                 foreach ($recUidArr as $ruid => $value) {
                     $this->singleRecordLines($t, $ruid, $lines, $preCode, 1);
@@ -637,7 +637,7 @@ abstract class ImportExport
                     if (strlen($info['subst']['description'])) {
                         $pInfo['title'] .= '<br/>' . $preCode_B . '<strong>' . htmlspecialchars($lang->getLL('impexpcore_singlereco_descr')) . '</strong> ' . htmlspecialchars(GeneralUtility::fixed_lgd_cs($info['subst']['description'], 60));
                     }
-                    $pInfo['title'] .= '<br/>' . $preCode_B . ($info['subst']['type'] == 'file' ? htmlspecialchars($lang->getLL('impexpcore_singlereco_filename')) . ' <strong>' . $info['subst']['relFileName'] . '</strong>' : '') . ($info['subst']['type'] == 'string' ? htmlspecialchars($lang->getLL('impexpcore_singlereco_value')) . ' <strong>' . $info['subst']['tokenValue'] . '</strong>' : '') . ($info['subst']['type'] == 'db' ? htmlspecialchars($lang->getLL('impexpcore_softrefsel_record')) . ' <strong>' . $info['subst']['recordRef'] . '</strong>' : '');
+                    $pInfo['title'] .= '<br/>' . $preCode_B . ($info['subst']['type'] === 'file' ? htmlspecialchars($lang->getLL('impexpcore_singlereco_filename')) . ' <strong>' . $info['subst']['relFileName'] . '</strong>' : '') . ($info['subst']['type'] === 'string' ? htmlspecialchars($lang->getLL('impexpcore_singlereco_value')) . ' <strong>' . $info['subst']['tokenValue'] . '</strong>' : '') . ($info['subst']['type'] === 'db' ? htmlspecialchars($lang->getLL('impexpcore_softrefsel_record')) . ' <strong>' . $info['subst']['recordRef'] . '</strong>' : '');
                 }
                 $pInfo['ref'] = 'SOFTREF';
                 $pInfo['size'] = '';
@@ -650,12 +650,12 @@ abstract class ImportExport
                 }
                 $lines[] = $pInfo;
                 // Add relations:
-                if ($info['subst']['type'] == 'db') {
+                if ($info['subst']['type'] === 'db') {
                     list($tempTable, $tempUid) = explode(':', $info['subst']['recordRef']);
                     $this->addRelations([['table' => $tempTable, 'id' => $tempUid, 'tokenID' => $info['subst']['tokenID']]], $lines, $preCode_B, [], '');
                 }
                 // Add files:
-                if ($info['subst']['type'] == 'file') {
+                if ($info['subst']['type'] === 'file') {
                     $this->addFiles([$info['file_ID']], $lines, $preCode_B, '', $info['subst']['tokenID']);
                 }
             }
@@ -865,12 +865,12 @@ abstract class ImportExport
             if ($r['type'] === 'record') {
                 return '<input type="checkbox" class="t3js-exclude-checkbox" name="tx_impexp[exclude][' . $r['ref'] . ']" id="checkExclude' . $r['ref'] . '" value="1" /> <label for="checkExclude' . $r['ref'] . '">' . htmlspecialchars($this->getLanguageService()->getLL('impexpcore_singlereco_exclude')) . '</label>';
             } else {
-                return  $r['type'] == 'softref' ? $this->softrefSelector($r['_softRefInfo']) : '';
+                return  $r['type'] === 'softref' ? $this->softrefSelector($r['_softRefInfo']) : '';
             }
         } else {
             // During import
             // For softreferences with editable fields:
-            if ($r['type'] == 'softref' && is_array($r['_softRefInfo']['subst']) && $r['_softRefInfo']['subst']['tokenID']) {
+            if ($r['type'] === 'softref' && is_array($r['_softRefInfo']['subst']) && $r['_softRefInfo']['subst']['tokenID']) {
                 $tokenID = $r['_softRefInfo']['subst']['tokenID'];
                 $cfg = $this->softrefCfg[$tokenID];
                 if ($cfg['mode'] === 'editable') {
@@ -1150,7 +1150,7 @@ abstract class ImportExport
         if (is_array($databaseRecord) && is_array($importRecord)) {
             // Traverse based on database record
             foreach ($databaseRecord as $fN => $value) {
-                if (is_array($GLOBALS['TCA'][$table]['columns'][$fN]) && $GLOBALS['TCA'][$table]['columns'][$fN]['config']['type'] != 'passthrough') {
+                if (is_array($GLOBALS['TCA'][$table]['columns'][$fN]) && $GLOBALS['TCA'][$table]['columns'][$fN]['config']['type'] !== 'passthrough') {
                     if (isset($importRecord[$fN])) {
                         if (trim($databaseRecord[$fN]) !== trim($importRecord[$fN])) {
                             // Create diff-result:
index c446972..b9b8312 100644 (file)
@@ -91,7 +91,7 @@ abstract class AbstractExportTestCase extends \TYPO3\CMS\Components\TestingFrame
     protected function addRecordsForPid($pid, array $tables)
     {
         foreach ($GLOBALS['TCA'] as $table => $value) {
-            if ($table != 'pages' && (in_array($table, $tables) || in_array('_ALL', $tables))) {
+            if ($table !== 'pages' && (in_array($table, $tables) || in_array('_ALL', $tables))) {
                 if ($GLOBALS['BE_USER']->check('tables_select', $table) && !$GLOBALS['TCA'][$table]['ctrl']['is_static']) {
                     $orderBy = $GLOBALS['TCA'][$table]['ctrl']['sortby'] ?: $GLOBALS['TCA'][$table]['ctrl']['default_sortby'];
 
index bde541f..8664d3d 100644 (file)
@@ -334,7 +334,7 @@ class SearchController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionControlle
             // Browsing box
             if ($resultData['count']) {
                 // could we get this in the view?
-                if ($this->searchData['group'] == 'sections' && $freeIndexUid <= 0) {
+                if ($this->searchData['group'] === 'sections' && $freeIndexUid <= 0) {
                     $resultSectionsCount = count($this->resultSections);
                     $result['sectionText'] = sprintf(LocalizationUtility::translate('result.' . ($resultSectionsCount > 1 ? 'inNsections' : 'inNsection'), 'IndexedSearch'), $resultSectionsCount);
                 }
@@ -382,8 +382,8 @@ class SearchController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionControlle
         }
         $resultRows = $newResultRows;
         $this->resultSections = [];
-        if ($freeIndexUid <= 0 && $this->searchData['group'] == 'sections') {
-            $rl2flag = substr($this->searchData['sections'], 0, 2) == 'rl';
+        if ($freeIndexUid <= 0 && $this->searchData['group'] === 'sections') {
+            $rl2flag = substr($this->searchData['sections'], 0, 2) === 'rl';
             $sections = [];
             foreach ($resultRows as $row) {
                 $id = $row['rl0'] . '-' . $row['rl1'] . ($rl2flag ? '-' . $row['rl2'] : '');
@@ -657,7 +657,7 @@ class SearchController extends \TYPO3\CMS\Extbase\Mvc\Controller\ActionControlle
             } else {
                 // Default creation / finding of icon:
                 $icon = '';
-                if ($imageType === '0' || substr($imageType, 0, 2) == '0:') {
+                if ($imageType === '0' || substr($imageType, 0, 2) === '0:') {
                     if (is_array($specRowConf['pageIcon'])) {
                         $this->iconFileNameCache[$imageType] = $GLOBALS['TSFE']->cObj->cObjGetSingle('IMAGE', $specRowConf['pageIcon']);
                     } else {
index ea6dc5b..91c56b1 100644 (file)
@@ -799,12 +799,12 @@ class IndexSearchRepository
                 $expressionBuilder->in('ISEC.rl0', GeneralUtility::intExplode(',', $this->searchRootPageIdList, true))
             );
         }
-        if (substr($this->sections, 0, 4) == 'rl1_') {
+        if (substr($this->sections, 0, 4) === 'rl1_') {
             $whereClause->add(
                 $expressionBuilder->in('ISEC.rl1', GeneralUtility::intExplode(',', substr($this->sections, 4)))
             );
             $match = true;
-        } elseif (substr($this->sections, 0, 4) == 'rl2_') {
+        } elseif (substr($this->sections, 0, 4) === 'rl2_') {
             $whereClause->add(
                 $expressionBuilder->in('ISEC.rl2', GeneralUtility::intExplode(',', substr($this->sections, 4)))
             );
index e8c5ba7..5a222a9 100644 (file)
@@ -78,7 +78,7 @@ class FileContentParser
         // Then read indexer-config and set if appropriate:
         $indexerConfig = unserialize($GLOBALS['TYPO3_CONF_VARS']['EXT']['extConf']['indexed_search'], ['allowed_classes' => false]);
         // If windows, apply extension to tool name:
-        $exe = TYPO3_OS == 'WIN' ? '.exe' : '';
+        $exe = TYPO3_OS === 'WIN' ? '.exe' : '';
         // lg
         $extOK = false;
         $mainExtension = '';
index 1ccd36a..203942b 100644 (file)
@@ -740,10 +740,10 @@ class Indexer
             $body = '';
             foreach ($expBody as $val) {
                 $part = explode('-->', $val, 2);
-                if (trim($part[0]) == 'begin') {
+                if (trim($part[0]) === 'begin') {
                     $body .= $part[1];
                     $prev = '';
-                } elseif (trim($part[0]) == 'end') {
+                } elseif (trim($part[0]) === 'end') {
                     $body .= $prev;
                 } else {
                     $prev = $val;
@@ -849,7 +849,7 @@ class Indexer
                 $tagAttributes = $htmlParser->get_tag_attributes($tagData, true);
                 $firstTagName = $htmlParser->getFirstTagName($tagData);
                 if (strtolower($firstTagName) === 'a') {
-                    if ($tagAttributes[0]['href'] && $tagAttributes[0]['href'][0] != '#') {
+                    if ($tagAttributes[0]['href'] && $tagAttributes[0]['href'][0] !== '#') {
                         $hyperLinksData[] = [
                             'tag' => $tagData,
                             'href' => $tagAttributes[0]['href'],
@@ -1049,7 +1049,7 @@ class Indexer
     protected function createLocalPathFromAbsoluteURL($sourcePath)
     {
         $localPath = '';
-        if ($sourcePath[0] == '/') {
+        if ($sourcePath[0] === '/') {
             $sourcePath = substr($sourcePath, 1);
             $localPath = PATH_site . $sourcePath;
             if (!self::isAllowedLocalFile($localPath)) {
@@ -1086,7 +1086,7 @@ class Indexer
     protected static function isRelativeURL($url)
     {
         $urlParts = @parse_url($url);
-        return $urlParts['scheme'] == '' && $urlParts['path'][0] != '/';
+        return $urlParts['scheme'] == '' && $urlParts['path'][0] !== '/';
     }
 
     /**
index d9175ac..786247b 100644 (file)
@@ -134,7 +134,7 @@ class Lexer
         in the same manner, and since the set of characters is huge so the
         extra matches are not significant.(Hint taken from ZOPEs chinese user group)[Kasper: As far as I can see this will only work well with or-searches!]
          */
-        if ($cType == 'cjk') {
+        if ($cType === 'cjk') {
             // Find total string length:
             $strlen = mb_strlen($theWord, 'utf-8');
             // Traverse string length and add words as pairs of two chars:
@@ -207,8 +207,8 @@ class Lexer
                     // We are in a sequence of words
                     if (
                         !$cType
-                        || $cType_prev == 'cjk' && ($cType === 'num' || $cType === 'alpha')
-                        || $cType == 'cjk' && ($cType_prev === 'num' || $cType_prev === 'alpha')
+                        || $cType_prev === 'cjk' && ($cType === 'num' || $cType === 'alpha')
+                        || $cType === 'cjk' && ($cType_prev === 'num' || $cType_prev === 'alpha')
                     ) {
                         // Check if the non-letter char is NOT a print-join char because then it signifies the end of the word.
                         if (!in_array($cp, $this->lexerConf['printjoins'])) {
index 4ec1714..e6fa919 100644 (file)
@@ -123,7 +123,7 @@ class DoubleMetaPhoneUtility
                     // '-mb', e.g. "dumb", already skipped over ...
                     $this->primary .= 'P';
                     $this->secondary .= 'P';
-                    if (substr($this->original, $this->current + 1, 1) == 'B') {
+                    if (substr($this->original, $this->current + 1, 1) === 'B') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -136,7 +136,7 @@ class DoubleMetaPhoneUtility
                     break;
                 case 'C':
                     // various gremanic
-                    if ($this->current > 1 && !$this->IsVowel($this->original, ($this->current - 2)) && $this->StringAt($this->original, $this->current - 1, 3, ['ACH']) && (substr($this->original, $this->current + 2, 1) != 'I' && (substr($this->original, $this->current + 2, 1) != 'E' || $this->StringAt($this->original, $this->current - 2, 6, ['BACHER', 'MACHER'])))) {
+                    if ($this->current > 1 && !$this->IsVowel($this->original, ($this->current - 2)) && $this->StringAt($this->original, $this->current - 1, 3, ['ACH']) && (substr($this->original, $this->current + 2, 1) !== 'I' && (substr($this->original, $this->current + 2, 1) !== 'E' || $this->StringAt($this->original, $this->current - 2, 6, ['BACHER', 'MACHER'])))) {
                         $this->primary .= 'K';
                         $this->secondary .= 'K';
                         $this->current += 2;
@@ -212,7 +212,7 @@ class DoubleMetaPhoneUtility
                         // 'bellocchio' but not 'bacchus'
                         if ($this->StringAt($this->original, $this->current + 2, 1, ['I', 'E', 'H']) && !$this->StringAt($this->original, ($this->current + 2), 2, ['HU'])) {
                             // 'accident', 'accede', 'succeed'
-                            if ($this->current == 1 && substr($this->original, $this->current - 1, 1) == 'A' || $this->StringAt($this->original, $this->current - 1, 5, ['UCCEE', 'UCCES'])) {
+                            if ($this->current == 1 && substr($this->original, $this->current - 1, 1) === 'A' || $this->StringAt($this->original, $this->current - 1, 5, ['UCCEE', 'UCCES'])) {
                                 $this->primary .= 'KS';
                                 $this->secondary .= 'KS';
                             } else {
@@ -289,7 +289,7 @@ class DoubleMetaPhoneUtility
                     $this->current += 1;
                     break;
                 case 'F':
-                    if (substr($this->original, $this->current + 1, 1) == 'F') {
+                    if (substr($this->original, $this->current + 1, 1) === 'F') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -298,7 +298,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'F';
                     break;
                 case 'G':
-                    if (substr($this->original, $this->current + 1, 1) == 'H') {
+                    if (substr($this->original, $this->current + 1, 1) === 'H') {
                         if ($this->current > 0 && !$this->IsVowel($this->original, ($this->current - 1))) {
                             $this->primary .= 'K';
                             $this->secondary .= 'K';
@@ -308,7 +308,7 @@ class DoubleMetaPhoneUtility
                         if ($this->current < 3) {
                             // 'ghislane', 'ghiradelli'
                             if ($this->current == 0) {
-                                if (substr($this->original, $this->current + 2, 1) == 'I') {
+                                if (substr($this->original, $this->current + 2, 1) === 'I') {
                                     $this->primary .= 'J';
                                     $this->secondary .= 'J';
                                 } else {
@@ -325,10 +325,10 @@ class DoubleMetaPhoneUtility
                             break;
                         } else {
                             // e.g. 'laugh', 'McLaughlin', 'cough', 'gough', 'rough', 'tough'
-                            if ($this->current > 2 && substr($this->original, $this->current - 1, 1) == 'U' && $this->StringAt($this->original, $this->current - 3, 1, ['C', 'G', 'L', 'R', 'T'])) {
+                            if ($this->current > 2 && substr($this->original, $this->current - 1, 1) === 'U' && $this->StringAt($this->original, $this->current - 3, 1, ['C', 'G', 'L', 'R', 'T'])) {
                                 $this->primary .= 'F';
                                 $this->secondary .= 'F';
-                            } elseif ($this->current > 0 && substr($this->original, $this->current - 1, 1) != 'I') {
+                            } elseif ($this->current > 0 && substr($this->original, $this->current - 1, 1) !== 'I') {
                                 $this->primary .= 'K';
                                 $this->secondary .= 'K';
                             }
@@ -336,13 +336,13 @@ class DoubleMetaPhoneUtility
                             break;
                         }
                     }
-                    if (substr($this->original, $this->current + 1, 1) == 'N') {
+                    if (substr($this->original, $this->current + 1, 1) === 'N') {
                         if ($this->current == 1 && $this->IsVowel($this->original, 0) && !$this->SlavoGermanic($this->original)) {
                             $this->primary .= 'KN';
                             $this->secondary .= 'N';
                         } else {
                             // not e.g. 'cagney'
-                            if (!$this->StringAt($this->original, ($this->current + 2), 2, ['EY']) && substr($this->original, $this->current + 1) != 'Y' && !$this->SlavoGermanic($this->original)) {
+                            if (!$this->StringAt($this->original, ($this->current + 2), 2, ['EY']) && substr($this->original, $this->current + 1) !== 'Y' && !$this->SlavoGermanic($this->original)) {
                                 $this->primary .= 'N';
                                 $this->secondary .= 'KN';
                             } else {
@@ -361,7 +361,7 @@ class DoubleMetaPhoneUtility
                         break;
                     }
                     // -ges-, -gep-, -gel- at beginning
-                    if ($this->current == 0 && (substr($this->original, $this->current + 1, 1) == 'Y' || $this->StringAt($this->original, $this->current + 1, 2, [
+                    if ($this->current == 0 && (substr($this->original, $this->current + 1, 1) === 'Y' || $this->StringAt($this->original, $this->current + 1, 2, [
                         'ES',
                         'EP',
                         'EB',
@@ -380,7 +380,7 @@ class DoubleMetaPhoneUtility
                         break;
                     }
                     // -ger-, -gy-
-                    if (($this->StringAt($this->original, $this->current + 1, 2, ['ER']) || substr($this->original, $this->current + 1, 1) == 'Y') && !$this->StringAt($this->original, 0, 6, ['DANGER', 'RANGER', 'MANGER']) && !$this->StringAt($this->original, ($this->current - 1), 1, ['E', 'I']) && !$this->StringAt($this->original, ($this->current - 1), 3, ['RGY', 'OGY'])) {
+                    if (($this->StringAt($this->original, $this->current + 1, 2, ['ER']) || substr($this->original, $this->current + 1, 1) === 'Y') && !$this->StringAt($this->original, 0, 6, ['DANGER', 'RANGER', 'MANGER']) && !$this->StringAt($this->original, ($this->current - 1), 1, ['E', 'I']) && !$this->StringAt($this->original, ($this->current - 1), 3, ['RGY', 'OGY'])) {
                         $this->primary .= 'K';
                         $this->secondary .= 'J';
                         $this->current += 2;
@@ -405,7 +405,7 @@ class DoubleMetaPhoneUtility
                         $this->current += 2;
                         break;
                     }
-                    if (substr($this->original, $this->current + 1, 1) == 'G') {
+                    if (substr($this->original, $this->current + 1, 1) === 'G') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -426,7 +426,7 @@ class DoubleMetaPhoneUtility
                 case 'J':
                     // obvious spanish, 'jose', 'san jacinto'
                     if ($this->StringAt($this->original, $this->current, 4, ['JOSE']) || $this->StringAt($this->original, 0, 4, ['SAN '])) {
-                        if ($this->current == 0 && substr($this->original, $this->current + 4, 1) == ' ' || $this->StringAt($this->original, 0, 4, ['SAN '])) {
+                        if ($this->current == 0 && substr($this->original, $this->current + 4, 1) === ' ' || $this->StringAt($this->original, 0, 4, ['SAN '])) {
                             $this->primary .= 'H';
                             $this->secondary .= 'H';
                         } else {
@@ -442,7 +442,7 @@ class DoubleMetaPhoneUtility
                         $this->secondary .= 'A';
                     } else {
                         // spanish pron. of .e.g. 'bajador'
-                        if ($this->IsVowel($this->original, $this->current - 1) && !$this->SlavoGermanic($this->original) && (substr($this->original, $this->current + 1, 1) == 'A' || substr($this->original, $this->current + 1, 1) == 'O')) {
+                        if ($this->IsVowel($this->original, $this->current - 1) && !$this->SlavoGermanic($this->original) && (substr($this->original, $this->current + 1, 1) === 'A' || substr($this->original, $this->current + 1, 1) === 'O')) {
                             $this->primary .= 'J';
                             $this->secondary .= 'H';
                         } else {
@@ -457,7 +457,7 @@ class DoubleMetaPhoneUtility
                             }
                         }
                     }
-                    if (substr($this->original, $this->current + 1, 1) == 'J') {
+                    if (substr($this->original, $this->current + 1, 1) === 'J') {
                         // it could happen
                         $this->current += 2;
                     } else {
@@ -465,7 +465,7 @@ class DoubleMetaPhoneUtility
                     }
                     break;
                 case 'K':
-                    if (substr($this->original, $this->current + 1, 1) == 'K') {
+                    if (substr($this->original, $this->current + 1, 1) === 'K') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -474,7 +474,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'K';
                     break;
                 case 'L':
-                    if (substr($this->original, $this->current + 1, 1) == 'L') {
+                    if (substr($this->original, $this->current + 1, 1) === 'L') {
                         // spanish e.g. 'cabrillo', 'gallegos'
                         if ($this->current == $this->length - 3 && $this->StringAt($this->original, $this->current - 1, 4, ['ILLO', 'ILLA', 'ALLE']) || ($this->StringAt($this->original, $this->last - 1, 2, ['AS', 'OS']) || $this->StringAt($this->original, $this->last, 1, ['A', 'O'])) && $this->StringAt($this->original, $this->current - 1, 4, ['ALLE'])) {
                             $this->primary .= 'L';
@@ -490,7 +490,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'L';
                     break;
                 case 'M':
-                    if ($this->StringAt($this->original, $this->current - 1, 3, ['UMB']) && ($this->current + 1 == $this->last || $this->StringAt($this->original, $this->current + 2, 2, ['ER'])) || substr($this->original, $this->current + 1, 1) == 'M') {
+                    if ($this->StringAt($this->original, $this->current - 1, 3, ['UMB']) && ($this->current + 1 == $this->last || $this->StringAt($this->original, $this->current + 2, 2, ['ER'])) || substr($this->original, $this->current + 1, 1) === 'M') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -499,7 +499,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'M';
                     break;
                 case 'N':
-                    if (substr($this->original, $this->current + 1, 1) == 'N') {
+                    if (substr($this->original, $this->current + 1, 1) === 'N') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -513,7 +513,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'N';
                     break;
                 case 'P':
-                    if (substr($this->original, $this->current + 1, 1) == 'H') {
+                    if (substr($this->original, $this->current + 1, 1) === 'H') {
                         $this->current += 2;
                         $this->primary .= 'F';
                         $this->secondary .= 'F';
@@ -529,7 +529,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'P';
                     break;
                 case 'Q':
-                    if (substr($this->original, $this->current + 1, 1) == 'Q') {
+                    if (substr($this->original, $this->current + 1, 1) === 'Q') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -546,7 +546,7 @@ class DoubleMetaPhoneUtility
                         $this->primary .= 'R';
                         $this->secondary .= 'R';
                     }
-                    if (substr($this->original, $this->current + 1, 1) == 'R') {
+                    if (substr($this->original, $this->current + 1, 1) === 'R') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -603,7 +603,7 @@ class DoubleMetaPhoneUtility
                     }
                     if ($this->StringAt($this->original, $this->current, 2, ['SC'])) {
                         // Schlesinger's rule
-                        if (substr($this->original, $this->current + 2, 1) == 'H') {
+                        if (substr($this->original, $this->current + 2, 1) === 'H') {
                             // dutch origin, e.g. 'school', 'schooner'
                             if ($this->StringAt($this->original, $this->current + 3, 2, ['OO', 'ER', 'EN', 'UY', 'ED', 'EM'])) {
                                 // 'schermerhorn', 'schenker'
@@ -617,7 +617,7 @@ class DoubleMetaPhoneUtility
                                 $this->current += 3;
                                 break;
                             } else {
-                                if ($this->current == 0 && !$this->IsVowel($this->original, 3) && substr($this->original, $this->current + 3, 1) != 'W') {
+                                if ($this->current == 0 && !$this->IsVowel($this->original, 3) && substr($this->original, $this->current + 3, 1) !== 'W') {
                                     $this->primary .= 'X';
                                     $this->secondary .= 'S';
                                 } else {
@@ -688,7 +688,7 @@ class DoubleMetaPhoneUtility
                     $this->secondary .= 'T';
                     break;
                 case 'V':
-                    if (substr($this->original, $this->current + 1, 1) == 'V') {
+                    if (substr($this->original, $this->current + 1, 1) === 'V') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
@@ -746,19 +746,19 @@ class DoubleMetaPhoneUtility
                     break;
                 case 'Z':
                     // chinese pinyin e.g. 'zhao'
-                    if (substr($this->original, $this->current + 1, 1) == 'H') {
+                    if (substr($this->original, $this->current + 1, 1) === 'H') {
                         $this->primary .= 'J';
                         $this->secondary .= 'J';
                         $this->current += 2;
                         break;
-                    } elseif ($this->StringAt($this->original, $this->current + 1, 2, ['ZO', 'ZI', 'ZA']) || $this->SlavoGermanic($this->original) && ($this->current > 0 && substr($this->original, $this->current - 1, 1) != 'T')) {
+                    } elseif ($this->StringAt($this->original, $this->current + 1, 2, ['ZO', 'ZI', 'ZA']) || $this->SlavoGermanic($this->original) && ($this->current > 0 && substr($this->original, $this->current - 1, 1) !== 'T')) {
                         $this->primary .= 'S';
                         $this->secondary .= 'TS';
                     } else {
                         $this->primary .= 'S';
                         $this->secondary .= 'S';
                     }
-                    if (substr($this->original, $this->current + 1, 1) == 'Z') {
+                    if (substr($this->original, $this->current + 1, 1) === 'Z') {
                         $this->current += 2;
                     } else {
                         $this->current += 1;
index f2fbc3c..bfbcc99 100644 (file)
@@ -278,7 +278,7 @@ class CleanUp extends Action\AbstractAction
         $directories = dir($pathTypo3Temp);
         if (is_object($directories)) {
             while ($entry = $directories->read()) {
-                if (is_dir($pathTypo3Temp . $entry) && $entry != '..' && $entry != '.') {
+                if (is_dir($pathTypo3Temp . $entry) && $entry !== '..' && $entry !== '.') {
                     $data['subDirectories'][$entry]['name'] = $entry;
                     $data['subDirectories'][$entry]['filesNumber'] = count(GeneralUtility::getFilesInDir($pathTypo3Temp . $entry));
                     $data['subDirectories'][$entry]['selected'] = false;
index 6da1582..a44a281 100644 (file)
@@ -80,7 +80,7 @@ class SqlSchemaMigrationService
                 if (strtoupper($parts[0]) === 'CREATE' && strtoupper($parts[1]) === 'TABLE') {
                     $table = str_replace('`', '', $parts[2]);
                     // tablenames are always lowercase on windows!
-                    if (TYPO3_OS == 'WIN') {
+                    if (TYPO3_OS === 'WIN') {
                         $table = strtolower($table);
                     }
                 }
@@ -137,7 +137,7 @@ class SqlSchemaMigrationService
                             $parts[1] = preg_replace('/^(KEY|INDEX) /', '', $parts[1]);
                         }
                         $newParts = explode(' ', $parts[1], 2);
-                        $key = $parts[0] == 'PRIMARY' ? $parts[0] : $newParts[0];
+                        $key = $parts[0] === 'PRIMARY' ? $parts[0] : $newParts[0];
                         $total[$table]['keys'][$key] = $lineV;
                         // This is a protection against doing something stupid: Only allow clearing of cache_* and index_* tables.
                         if (preg_match('/^(cache|index)_/', $table)) {
@@ -214,7 +214,7 @@ class SqlSchemaMigrationService
                     $colName .= '(' . $keyRow['Sub_part'] . ')';
                 }
                 $tempKeys[$tableName][$keyName][$keyRow['Seq_in_index']] = $colName;
-                if ($keyName == 'PRIMARY') {
+                if ($keyName === 'PRIMARY') {
                     $prefix = 'PRIMARY KEY';
                 } else {
                     if ($keyRow['Index_type'] === 'FULLTEXT') {
@@ -282,7 +282,7 @@ class SqlSchemaMigrationService
                             if (is_array($info[$theKey])) {
                                 foreach ($info[$theKey] as $fieldN => $fieldC) {
                                     $fieldN = str_replace('`', '', $fieldN);
-                                    if ($fieldN == 'COLLATE') {
+                                    if ($fieldN === 'COLLATE') {
                                         // @todo collation support is currently disabled (needs more testing)
                                         continue;
                                     }
@@ -341,7 +341,7 @@ class SqlSchemaMigrationService
         $deletedPrefixKey = $this->deletedPrefixKey;
         $deletedPrefixLength = strlen($deletedPrefixKey);
         $remove = 0;
-        if ($keyList == 'remove') {
+        if ($keyList === 'remove') {
             $remove = 1;
             $keyList = 'extra';
         }
@@ -376,7 +376,7 @@ class SqlSchemaMigrationService
                                         $info['extra']['CLEAR'] = 2;
                                     }
                                 }
-                                if ($theKey == 'extra') {
+                                if ($theKey === 'extra') {
                                     if ($remove) {
                                         if (substr($fN, 0, $deletedPrefixLength) !== $deletedPrefixKey) {
                                             // we've to make sure we don't exceed the maximal length
@@ -391,7 +391,7 @@ class SqlSchemaMigrationService
                                         $statement = 'ALTER TABLE ' . $table . ' ADD ' . $fN . ' ' . $fV . ';';
                                         $statements['add'][md5($statement)] = $statement;
                                     }
-                                } elseif ($theKey == 'diff') {
+                                } elseif ($theKey === 'diff') {
                                     $statement = 'ALTER TABLE ' . $table . ' CHANGE ' . $fN . ' ' . $fN . ' ' . $fV . ';';
                                     $statements['change'][md5($statement)] = $statement;
                                     $statements['change_currentValue'][md5($statement)] = $diffArr['diff_currentValues'][$table]['fields'][$fN];
@@ -404,13 +404,13 @@ class SqlSchemaMigrationService
                             if ($info['whole_table']) {
                                 $whole_table[] = $fV;
                             } else {
-                                if ($theKey == 'extra') {
+                                if ($theKey === 'extra') {
                                     if (!$remove) {
                                         $statement = 'ALTER TABLE ' . $table . ' ADD ' . $fV . ';';
                                         $statements['add'][md5($statement)] = $statement;
                                     }
-                                } elseif ($theKey == 'diff') {
-                                    $statement = 'ALTER TABLE ' . $table . ($fN == 'PRIMARY' ? ' DROP PRIMARY KEY' : ' DROP KEY ' . $fN) . ';';
+                                } elseif ($theKey === 'diff') {
+                                    $statement = 'ALTER TABLE ' . $table . ($fN === 'PRIMARY' ? ' DROP PRIMARY KEY' : ' DROP KEY ' . $fN) . ';';
                                     $statements['change'][md5($statement)] = $statement;
                                     $statement = 'ALTER TABLE ' . $table . ' ADD ' . $fV . ';';
                                     $statements['change'][md5($statement)] = $statement;
@@ -427,7 +427,7 @@ class SqlSchemaMigrationService
                             if (!$remove) {
                                 if (!$info['whole_table']) {
                                     // If the whole table is created at once, we take care of this later by imploding all elements of $info['extra']
-                                    if ($fN == 'CLEAR') {
+                                    if ($fN === 'CLEAR') {
                                         // Truncate table must happen later, not now
                                         // Valid values for CLEAR: 1=only clear if keys are missing, 2=clear anyway (force)
                                         if (!empty($info['keys']) || $fV == 2) {
@@ -474,7 +474,7 @@ class SqlSchemaMigrationService
 )';
                             if ($info['extra']) {
                                 foreach ($info['extra'] as $k => $v) {
-                                    if ($k == 'COLLATE' || $k == 'CLEAR') {
+                                    if ($k === 'COLLATE' || $k === 'CLEAR') {
                                         // Skip these special statements.
                                         // @todo collation support is currently disabled (needs more testing)
                                         continue;
@@ -503,7 +503,7 @@ class SqlSchemaMigrationService
     public function assembleFieldDefinition($row)
     {
         $field = [$row['Type']];
-        if ($row['Null'] == 'NO') {
+        if ($row['Null'] === 'NO') {
             $field[] = 'NOT NULL';
         }
         if (!strstr($row['Type'], 'blob') && !strstr($row['Type'], 'text')) {
@@ -582,7 +582,7 @@ class SqlSchemaMigrationService
                 $table = trim($reg[1]);
                 if ($table) {
                     // Table names are always lowercase on Windows!
-                    if (TYPO3_OS == 'WIN') {
+                    if (TYPO3_OS === 'WIN') {
                         $table = strtolower($table);
                     }
                     $sqlLines = explode(LF, $lineContent);
index e1c1ba4..9d5a259 100644 (file)
@@ -84,7 +84,7 @@ class TerService extends TerUtility implements SingletonInterface
                 // Traverse the parsed XML structure:
             foreach ($values as $val) {
                 // First, process the tag-name (which is used in both cases, whether "complete" or "close")
-                $tagName = (string)($val['tag'] == 'languagepack' && $val['type'] == 'open') ? $val['attributes']['language'] : $val['tag'];
+                $tagName = (string)($val['tag'] === 'languagepack' && $val['type'] === 'open') ? $val['attributes']['language'] : $val['tag'];
                 if (!$documentTag) {
                     $documentTag = $tagName;
                 }
index 64915ce..b337353 100644 (file)
@@ -294,7 +294,7 @@ class LinkAnalyzer
                             // Do processing
                             $resultArray = $softRefObj->findRef($table, $field, $idRecord, $valueField, $spKey, $spParams);
                             if (!empty($resultArray['elements'])) {
-                                if ($spKey == 'typolink_tag') {
+                                if ($spKey === 'typolink_tag') {
                                     $this->analyseTypoLinks($resultArray, $results, $htmlParser, $record, $field, $table);
                                 } else {
                                     $this->analyseLinks($resultArray, $results, $record, $field, $table);
index a7ff17a..d2dcd48 100644 (file)
@@ -170,13 +170,13 @@ class DatabaseIntegrityView extends BaseScriptClass
         $addConditionCheck = GeneralUtility::_GP('qG_ins');
         $setLimitToStart = false;
         foreach ($OLD_MOD_SETTINGS as $key => $val) {
-            if (substr($key, 0, 5) == 'query' && $this->MOD_SETTINGS[$key] != $val && $key != 'queryLimit' && $key != 'use_listview') {
+            if (substr($key, 0, 5) === 'query' && $this->MOD_SETTINGS[$key] != $val && $key !== 'queryLimit' && $key !== 'use_listview') {
                 $setLimitToStart = true;
-                if ($key == 'queryTable' && !$addConditionCheck) {
+                if ($key === 'queryTable' && !$addConditionCheck) {
                     $this->MOD_SETTINGS['queryConfig'] = '';
                 }
             }
-            if ($key == 'queryTable' && $this->MOD_SETTINGS[$key] != $val) {
+            if ($key === 'queryTable' && $this->MOD_SETTINGS[$key] != $val) {
                 $this->MOD_SETTINGS['queryFields'] = '';
             }
         }
@@ -340,11 +340,11 @@ class DatabaseIntegrityView extends BaseScriptClass
         $fullsearch->setFormName($this->formName);
         $submenu = '<div class="form-inline form-inline-spaced">';
         $submenu .= BackendUtility::getDropdownMenu(0, 'SET[search]', $searchMode, $this->MOD_MENU['search']);
-        if ($this->MOD_SETTINGS['search'] == 'query') {
+        if ($this->MOD_SETTINGS['search'] === 'query') {
             $submenu .= BackendUtility::getDropdownMenu(0, 'SET[search_query_makeQuery]', $this->MOD_SETTINGS['search_query_makeQuery'], $this->MOD_MENU['search_query_makeQuery']) . '<br />';
         }
         $submenu .= '</div>';
-        if ($this->MOD_SETTINGS['search'] == 'query') {
+        if ($this->MOD_SETTINGS['search'] === 'query') {
             $submenu .= '<div class="checkbox"><label for="checkSearch_query_smallparts">' . BackendUtility::getFuncCheck($GLOBALS['SOBE']->id, 'SET[search_query_smallparts]', $this->MOD_SETTINGS['search_query_smallparts'], '', '', 'id="checkSearch_query_smallparts"') . $lang->getLL('showSQL') . '</label></div>';
             $submenu .= '<div class="checkbox"><label for="checkSearch_result_labels">' . BackendUtility::getFuncCheck($GLOBALS['SOBE']->id, 'SET[search_result_labels]', $this->MOD_SETTINGS['search_result_labels'], '', '', 'id="checkSearch_result_labels"') . $lang->getLL('useFormattedStrings') . '</label></div>';
             $submenu .= '<div class="checkbox"><label for="checkLabels_noprefix">' . BackendUtility::getFuncCheck($GLOBALS['SOBE']->id, 'SET[labels_noprefix]', $this->MOD_SETTINGS['labels_noprefix'], '', '', 'id="checkLabels_noprefix"') . $lang->getLL('dontUseOrigValues') . '</label></div>';
@@ -399,7 +399,7 @@ class DatabaseIntegrityView extends BaseScriptClass
         $doktype = $GLOBALS['TCA']['pages']['columns']['doktype']['config']['items'];
         if (is_array($doktype)) {
             foreach ($doktype as $setup) {
-                if ($setup[1] != '--div--') {
+                if ($setup[1] !== '--div--') {
                     $doktypes[] = [
                         'icon' => $this->iconFactory->getIconForRecord('pages', ['doktype' => $setup[1]], Icon::SIZE_SMALL)->render(),
                         'title' => $lang->sL($setup[0]) . ' (' . $setup[1] . ')',
index ea0085c..f1fa8fc 100644 (file)
@@ -352,7 +352,7 @@ class RecordList extends AbstractModule
         // Clipboard actions are handled:
         // CB is the clipboard command array
         $CB = GeneralUtility::_GET('CB');
-        if ($this->cmd == 'setCB') {
+        if ($this->cmd === 'setCB') {
             // CBH is all the fields selected for the clipboard, CBC is the checkbox fields which were checked.
             // By merging we get a full array of checked/unchecked elements
             // This is set to the 'el' array of the CB after being parsed so only the table in question is registered.
@@ -370,12 +370,12 @@ class RecordList extends AbstractModule
         $dblist->clipObj->endClipboard();
         // This flag will prevent the clipboard panel in being shown.
         // It is set, if the clickmenu-layer is active AND the extended view is not enabled.
-        $dblist->dontShowClipControlPanels = ($dblist->clipObj->current == 'normal' && !$this->modTSconfig['properties']['showClipControlPanelsDespiteOfCMlayers']);
+        $dblist->dontShowClipControlPanels = ($dblist->clipObj->current === 'normal' && !$this->modTSconfig['properties']['showClipControlPanelsDespiteOfCMlayers']);
         // If there is access to the page or root page is used for searching, then render the list contents and set up the document template object:
         if ($access || ($this->id === 0 && $this->search_levels !== 0 && $this->search_field !== '')) {
             // Deleting records...:
             // Has not to do with the clipboard but is simply the delete action. The clipboard object is used to clean up the submitted entries to only the selected table.
-            if ($this->cmd == 'delete') {
+            if ($this->cmd === 'delete') {
                 $items = $dblist->clipObj->cleanUpCBC(GeneralUtility::_POST('CBC'), $this->cmd_table, 1);
                 if (!empty($items)) {
                     $cmd = [];
index 56e6200..d46cc62 100644 (file)
@@ -693,7 +693,7 @@ class AbstractDatabaseRecordList extends AbstractRecordList
         // Set LIMIT:
         $limit = $this->iLimit ? ($this->firstElementNumber ? $this->firstElementNumber . ',' : '') . $this->iLimit : '';
         // Filtering on displayable pages (permissions):
-        $pC = $table == 'pages' && $this->perms_clause ? ' AND ' . $this->perms_clause : '';
+        $pC = $table === 'pages' && $this->perms_clause ? ' AND ' . $this->perms_clause : '';
         // Adding search constraints:
         $search = $this->makeSearchString($table, $id);
         // Compiling query array:
@@ -1068,7 +1068,7 @@ class AbstractDatabaseRecordList extends AbstractRecordList
         switch ((string)$this->clickTitleMode) {
             case 'edit':
                 // If the listed table is 'pages' we have to request the permission settings for each page:
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $localCalcPerms = $this->getBackendUserAuthentication()->calcPerms(BackendUtility::getRecord('pages', $row['uid']));
                     $permsEdit = $localCalcPerms & Permission::PAGE_EDIT;
                 } else {
@@ -1082,9 +1082,9 @@ class AbstractDatabaseRecordList extends AbstractRecordList
                 break;
             case 'show':
                 // "Show" link (only pages and tt_content elements)
-                if ($table == 'pages' || $table == 'tt_content') {
+                if ($table === 'pages' || $table === 'tt_content') {
                     $code = '<a href="#" onclick="' . htmlspecialchars(
-                        BackendUtility::viewOnClick(($table == 'tt_content' ? $this->id . '#' . $row['uid'] : $row['uid']))
+                        BackendUtility::viewOnClick(($table === 'tt_content' ? $this->id . '#' . $row['uid'] : $row['uid']))
                     ) . '" title="' . htmlspecialchars($lang->sL('LLL:EXT:lang/Resources/Private/Language/locallang_core.xlf:labels.showPage')) . '">' . $code . '</a>';
                 }
                 break;
@@ -1094,7 +1094,7 @@ class AbstractDatabaseRecordList extends AbstractRecordList
                 break;
             default:
                 // Output the label now:
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $code = '<a href="' . htmlspecialchars($this->listURL($uid, '', 'firstElementNumber')) . '" onclick="setHighlight(' . $uid . ')">' . $code . '</a>';
                 } else {
                     $code = $this->linkUrlMail($code, $origCode);
@@ -1216,7 +1216,7 @@ class AbstractDatabaseRecordList extends AbstractRecordList
             if (isset($GLOBALS['TCA'][$table]['columns']) && is_array($GLOBALS['TCA'][$table]['columns'])) {
                 // Traverse configured columns and add them to field array, if available for user.
                 foreach ($GLOBALS['TCA'][$table]['columns'] as $fN => $fieldValue) {
-                    if ($dontCheckUser || (!$fieldValue['exclude'] || $backendUser->check('non_exclude_fields', $table . ':' . $fN)) && $fieldValue['config']['type'] != 'passthrough') {
+                    if ($dontCheckUser || (!$fieldValue['exclude'] || $backendUser->check('non_exclude_fields', $table . ':' . $fN)) && $fieldValue['config']['type'] !== 'passthrough') {
                         $fieldListArr[] = $fN;
                     }
                 }
index 0d18ece..dcee9f5 100644 (file)
@@ -568,7 +568,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
         if ($thumbsCol) {
             $selectFields[] = $thumbsCol;
         }
-        if ($table == 'pages') {
+        if ($table === 'pages') {
             $selectFields[] = 'module';
             $selectFields[] = 'extendToSubpages';
             $selectFields[] = 'nav_hide';
@@ -956,21 +956,21 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                     // It's a translated record with a language parent
                     $localizationMarkerClass = ' localization';
                 }
-            } elseif ($fCol == 'pid') {
+            } elseif ($fCol === 'pid') {
                 $theData[$fCol] = $row[$fCol];
-            } elseif ($fCol == '_PATH_') {
+            } elseif ($fCol === '_PATH_') {
                 $theData[$fCol] = $this->recPath($row['pid']);
-            } elseif ($fCol == '_REF_') {
+            } elseif ($fCol === '_REF_') {
                 $theData[$fCol] = $this->createReferenceHtml($table, $row['uid']);
-            } elseif ($fCol == '_CONTROL_') {
+            } elseif ($fCol === '_CONTROL_') {
                 $theData[$fCol] = $this->makeControl($table, $row);
-            } elseif ($fCol == '_CLIPBOARD_') {
+            } elseif ($fCol === '_CLIPBOARD_') {
                 $theData[$fCol] = $this->makeClip($table, $row);
-            } elseif ($fCol == '_LOCALIZATION_') {
+            } elseif ($fCol === '_LOCALIZATION_') {
                 list($lC1, $lC2) = $this->makeLocalizationPanel($table, $row);
                 $theData[$fCol] = $lC1;
                 $theData[$fCol . 'b'] = '<div class="btn-group">' . $lC2 . '</div>';
-            } elseif ($fCol == '_LOCALIZATION_b') {
+            } elseif ($fCol === '_LOCALIZATION_b') {
                 // deliberately empty
             } else {
                 $pageId = $table === 'pages' ? $row['uid'] : $row['pid'];
@@ -1058,7 +1058,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
         // Traverse the fields:
         foreach ($this->fieldArray as $fCol) {
             // Calculate users permissions to edit records in the table:
-            $permsEdit = $this->calcPerms & ($table == 'pages' ? 2 : 16) && $this->overlayEditLockPermissions($table);
+            $permsEdit = $this->calcPerms & ($table === 'pages' ? 2 : 16) && $this->overlayEditLockPermissions($table);
             switch ((string)$fCol) {
                 case '_PATH_':
                     // Path
@@ -1098,7 +1098,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                             . '</a>';
                     }
                     // If the numeric clipboard pads are enabled, display the control icons for that:
-                    if ($this->clipObj->current != 'normal') {
+                    if ($this->clipObj->current !== 'normal') {
                         // The "select" link:
                         $spriteIcon = $this->iconFactory->getIcon('actions-edit-copy', Icon::SIZE_SMALL)->render();
                         $cells['copyMarked'] = $this->linkClipboardHeaderIcon($spriteIcon, $table, 'setCB', '', $lang->getLL('clip_selectMarked'));
@@ -1173,14 +1173,14 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                                 $onClick = 'return jumpExt(' . GeneralUtility::quoteJSvalue($newContentWizScriptPath) . ');';
                                 $icon = '<a class="btn btn-default" href="#" onclick="' . htmlspecialchars($onClick) . '" title="'
                                     . htmlspecialchars($lang->getLL('new')) . '">' . $spriteIcon->render() . '</a>';
-                            } elseif ($table == 'pages' && $this->newWizards) {
+                            } elseif ($table === 'pages' && $this->newWizards) {
                                 $parameters = ['id' => $this->id, 'pagesOnly' => 1, 'returnUrl' => GeneralUtility::getIndpEnv('REQUEST_URI')];
                                 $href = BackendUtility::getModuleUrl('db_new', $parameters);
                                 $icon = '<a class="btn btn-default" href="' . htmlspecialchars($href) . '" title="' . htmlspecialchars($lang->getLL('new')) . '">'
                                     . $spriteIcon->render() . '</a>';
                             } else {
                                 $params = '&edit[' . $table . '][' . $this->id . ']=new';
-                                if ($table == 'pages_language_overlay') {
+                                if ($table === 'pages_language_overlay') {
                                     $params .= '&overrideVals[pages_language_overlay][doktype]=' . (int)$this->pageRow['doktype'];
                                 }
                                 $icon = '<a class="btn btn-default" href="#" onclick="' . htmlspecialchars(BackendUtility::editOnClick($params, '', -1))
@@ -1417,7 +1417,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
         ];
         // If the listed table is 'pages' we have to request the permission settings for each page:
         $localCalcPerms = 0;
-        if ($table == 'pages') {
+        if ($table === 'pages') {
             $localCalcPerms = $this->getBackendUserAuthentication()->calcPerms(BackendUtility::getRecord('pages', $row['uid']));
         }
         $permsEdit = $table === 'pages'
@@ -1428,7 +1428,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                         && $this->getBackendUserAuthentication()->recordEditAccessInternals($table, $row);
         $permsEdit = $this->overlayEditLockPermissions($table, $row, $permsEdit);
         // "Show" link (only pages and tt_content elements)
-        if ($table == 'pages' || $table == 'tt_content') {
+        if ($table === 'pages' || $table === 'tt_content') {
             $viewAction = '<a class="btn btn-default" href="#" onclick="'
                 . htmlspecialchars(
                     BackendUtility::viewOnClick(
@@ -1464,7 +1464,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
         if ($permsEdit && ($table === 'tt_content' || $table === 'pages')) {
             $onClick = 'return jumpExt(' . GeneralUtility::quoteJSvalue(BackendUtility::getModuleUrl('move_element') . '&table=' . $table . '&uid=' . $row['uid']) . ');';
             $linkTitleLL = htmlspecialchars($this->getLanguageService()->getLL('move_' . ($table === 'tt_content' ? 'record' : 'page')));
-            $icon = ($table == 'pages' ? $this->iconFactory->getIcon('actions-page-move', Icon::SIZE_SMALL) : $this->iconFactory->getIcon('actions-document-move', Icon::SIZE_SMALL));
+            $icon = ($table === 'pages' ? $this->iconFactory->getIcon('actions-page-move', Icon::SIZE_SMALL) : $this->iconFactory->getIcon('actions-document-move', Icon::SIZE_SMALL));
             $moveAction = '<a class="btn btn-default" href="#" onclick="' . htmlspecialchars($onClick) . '" title="' . $linkTitleLL . '">' . $icon->render() . '</a>';
             $this->addActionToCellGroup($cells, $moveAction, 'move');
         }
@@ -1505,7 +1505,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                 if ($table !== 'pages' && $this->calcPerms & Permission::CONTENT_EDIT || $table === 'pages' && $this->calcPerms & Permission::PAGE_NEW) {
                     if ($this->showNewRecLink($table)) {
                         $params = '&edit[' . $table . '][' . -($row['_MOVE_PLH'] ? $row['_MOVE_PLH_uid'] : $row['uid']) . ']=new';
-                        $icon = ($table == 'pages' ? $this->iconFactory->getIcon('actions-page-new', Icon::SIZE_SMALL) : $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL));
+                        $icon = ($table === 'pages' ? $this->iconFactory->getIcon('actions-page-new', Icon::SIZE_SMALL) : $this->iconFactory->getIcon('actions-add', Icon::SIZE_SMALL));
                         $titleLabel = 'new';
                         if ($GLOBALS['TCA'][$table]['ctrl']['sortby']) {
                             $titleLabel .= ($table === 'pages' ? 'Page' : 'Record');
@@ -1554,8 +1554,8 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                 if (!$permsEdit || $this->isRecordCurrentBackendUser($table, $row)) {
                     $hideAction = $this->spaceIcon;
                 } else {
-                    $hideTitle = htmlspecialchars($this->getLanguageService()->getLL('hide' . ($table == 'pages' ? 'Page' : '')));
-                    $unhideTitle = htmlspecialchars($this->getLanguageService()->getLL('unHide' . ($table == 'pages' ? 'Page' : '')));
+                    $hideTitle = htmlspecialchars($this->getLanguageService()->getLL('hide' . ($table === 'pages' ? 'Page' : '')));
+                    $unhideTitle = htmlspecialchars($this->getLanguageService()->getLL('unHide' . ($table === 'pages' ? 'Page' : '')));
                     if ($row[$hiddenField]) {
                         $params = 'data[' . $table . '][' . $rowUid . '][' . $hiddenField . ']=0';
                         $hideAction = '<a class="btn btn-default t3js-record-hide" data-state="hidden" href="#"'
@@ -1613,7 +1613,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
             }
             $this->addActionToCellGroup($cells, $deleteAction, 'delete');
             // "Levels" links: Moving pages into new levels...
-            if ($permsEdit && $table == 'pages' && !$this->searchLevels) {
+            if ($permsEdit && $table === 'pages' && !$this->searchLevels) {
                 // Up (Paste as the page right after the current parent page)
                 if ($this->calcPerms & Permission::PAGE_NEW) {
                     $params = '&cmd[' . $table . '][' . $row['uid'] . '][move]=' . -$this->id;
@@ -1720,7 +1720,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
         $cells = [];
         $cells['pasteAfter'] = ($cells['pasteInto'] = $this->spaceIcon);
         //enables to hide the copy, cut and paste icons for localized records - doesn't make much sense to perform these options for them
-        $isL10nOverlay = $this->localizationView && $table != 'pages_language_overlay' && $row[$GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField']] != 0;
+        $isL10nOverlay = $this->localizationView && $table !== 'pages_language_overlay' && $row[$GLOBALS['TCA'][$table]['ctrl']['transOrigPointerField']] != 0;
         // Return blank, if disabled:
         // Whether a numeric clipboard pad is active or the normal pad we will see different content of the panel:
         // For the "Normal" pad:
@@ -1746,7 +1746,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                     . $copyIcon->render() . '</a>';
 
                 // Check permission to cut page or content
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     $localCalcPerms = $this->getBackendUserAuthentication()->calcPerms(BackendUtility::getRecord('pages', $row['uid']));
                     $permsEdit = $localCalcPerms & Permission::PAGE_EDIT;
                 } else {
@@ -1755,7 +1755,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
                 $permsEdit = $this->overlayEditLockPermissions($table, $row, $permsEdit);
 
                 // If the listed table is 'pages' we have to request the permission settings for each page:
-                if ($table == 'pages') {
+                if ($table === 'pages') {
                     if ($permsEdit) {
                         $cells['cut'] = '<a class="btn btn-default" href="#" onclick="'
                         . htmlspecialchars('return jumpSelf(' . GeneralUtility::quoteJSvalue($this->clipObj->selUrlDB($table, $row['uid'], 0, ($isSel === 'cut'), ['returnUrl' => ''])) . ');')
@@ -1812,7 +1812,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
         }
         // Now, looking for elements in general:
         $elFromTable = $this->clipObj->elFromTable('');
-        if ($table == 'pages' && !empty($elFromTable)) {
+        if ($table === 'pages' && !empty($elFromTable)) {
             $cells['pasteInto'] = '<a class="btn btn-default t3js-modal-trigger"'
                 . ' href="' . htmlspecialchars($this->clipObj->pasteUrl('', $row['uid'])) . '"'
                 . ' title="' . htmlspecialchars($this->getLanguageService()->getLL('clip_pasteInto')) . '"'
@@ -2055,7 +2055,7 @@ class DatabaseRecordList extends AbstractDatabaseRecordList
      */
     public function clipNumPane()
     {
-        return in_array('_CLIPBOARD_', $this->fieldArray) && $this->clipObj->current != 'normal';
+        return in_array('_CLIPBOARD_', $this->fieldArray) && $this->