Merge pull request #17 from punktDe/FixUndefinedArrayKeyWarning

[FIX] Check if array keys exist before accessing

Author: alex-boehm

Classes/Backend/PageLayoutHeader.php

@@ -120,12 +120,18 @@ protected function toolbarIsEnabledForUser(): bool
             $isEnabled = false;
         }
 
-        if ($this->backendUser->user['quickedit_disableToolbar']) {
+        if (
+            array_key_exists('quickedit_disableToolbar', $this->backendUser->user) &&
+            $this->backendUser->user['quickedit_disableToolbar']
+        ) {
             $isEnabled = false;
         }
 
         foreach ($this->backendUser->userGroups as $group) {
-            if ($group['quickedit_disableToolbar']) {
+            if (
+                array_key_exists('quickedit_disableToolbar', $group) &&
+                $group['quickedit_disableToolbar']
+            ) {
                 $isEnabled = false;
             }
         }
@@ -168,7 +174,7 @@ protected function getFieldConfigForPage(): array
     {
         $configForPageType = $this->getConfigForCurrentPage();
 
-        if (is_array($configForPageType) && count($configForPageType) > 0) {
+        if (count($configForPageType) > 0) {
             foreach ($configForPageType as $key => &$singleConfig) {
                 $singleConfig['fields'] = $this->prepareFieldsList($singleConfig['fields']);
 
@@ -200,12 +206,21 @@ protected function getFieldConfigForPage(): array
     protected function getConfigForCurrentPage(): array
     {
         $pageTsConfig = BackendUtility::getPagesTSconfig($this->pageUid);
-        $quickeditConfig = $pageTsConfig['mod.']['web_layout.']['PageTypes.'];
         $configForPageType = [];
 
-        if (is_array($quickeditConfig) && array_key_exists($this->pageRecord['doktype'] . '.', $quickeditConfig)) {
-            $configForPageType = $quickeditConfig[$this->pageRecord['doktype'] . '.']['config.'];
-            ksort($configForPageType);
+        if (
+            array_key_exists('mod.', $pageTsConfig) &&
+            is_array($pageTsConfig['mod.']) &&
+            array_key_exists('web_layout.', $pageTsConfig['mod.']) &&
+            is_array($pageTsConfig['mod.']['web_layout.']) &&
+            array_key_exists('PageTypes.', $pageTsConfig['mod.']['web_layout.'])
+        ) {
+            $quickeditConfig = $pageTsConfig['mod.']['web_layout.']['PageTypes.'];
+
+            if (is_array($quickeditConfig) && array_key_exists($this->pageRecord['doktype'] . '.', $quickeditConfig)) {
+                $configForPageType = $quickeditConfig[$this->pageRecord['doktype'] . '.']['config.'];
+                ksort($configForPageType);
+            }
         }
 
         return $configForPageType;
@@ -229,6 +244,11 @@ protected function prepareFieldsList(string $fields): string
             $fieldsArray = array_map('trim', $fieldsArray);
 
             foreach ($fieldsArray as $index => $field) {
+                if ($this->isFieldDefined($field) === false) {
+                    unset($fieldsArray[$index]);
+                    continue;
+                }
+
                 if ($this->userHasAccessToField($field) === false
                     || $this->fieldIsAvailableForLanguage($field) === false) {
                     unset($fieldsArray[$index]);
@@ -340,4 +360,15 @@ protected function isVisible(): bool
 
         return $isVisible;
     }
+
+
+
+    /**
+     * @param string $field
+     * @return bool
+     */
+    protected function isFieldDefined(string $field): bool
+    {
+        return array_key_exists($field, $GLOBALS['TCA']['pages']['columns']);
+    }
 }

Documentation/Changelog/Index.rst

@@ -6,10 +6,23 @@
 Change log
 ==========
 
+Version 1.0.1
+-------------
+
+- Fix php warning by accessing non existing array keys (https://github.com/punktDe/quickedit/issues/15)
+- Further code improvements
+- Update README.md example code (https://github.com/punktDe/quickedit/issues/19)
+
 Version 1.0.0
 -------------
 
-- Update extension for TYPO3 11 LTS support
+- Update extension for TYPO3 11 LTS support (https://github.com/punktDe/quickedit/issues/6)
+
+Version 0.2.2
+-------------
+
+- Fix php warning by accessing non existing array keys (https://github.com/punktDe/quickedit/issues/15)
+- Further code improvements
 
 Version 0.2.1
 -------------

Documentation/Settings.cfg

@@ -25,7 +25,7 @@ project     = Quickedit
 # ...   (recommended) version, displayed next to title (desktop) and in <meta name="book-version"
 # .................................................................................
 
-release     = 1.0.0
+release     = 1.0.1
 
 # .................................................................................
 # ...  (recommended) displayed in footer

README.md

@@ -45,7 +45,7 @@ mod {
                     }
                     2 {
                         label = Special
-                        fields = slug, suergroup, hidden
+                        fields = slug, hidden, fe_group
                         previewFields = *
                     }
                 }

ext_emconf.php

@@ -2,7 +2,7 @@
 $EM_CONF[$_EXTKEY] = [
     'title'            => 'Toolbar for editing page properties',
     'description'      => 'This extension provides a configurable toolbar for editing page properties.',
-    'version'          => '1.0.0',
+    'version'          => '1.0.1',
     'category'         => 'be',
     'constraints'      => [
         'depends'   => [