Fix Types for file picker contao 4.13 and 5+

Author: shoaib0300

src/Contao/Widgets/MultiColumnWizard.php

@@ -1071,16 +1071,21 @@ protected function initializeWidget(&$arrField, $intRow, $strKey, $varValue)
 
         // Add the help wizard
         if (isset($arrField['eval']['helpwizard']) && $arrField['eval']['helpwizard']) {
+            $label = (isset($arrField['label']) && is_array($arrField['label']) && isset($arrField['label'][0]))
+                ? $arrField['label'][0]
+                : 'Help';
+
+            // Construct the help wizard link
             $xlabel .= ' <a href="contao/help.php?table=' . $this->strTable . '&amp;field=' . $this->strField
-                       . '" title="' . StringUtil::specialchars($GLOBALS['TL_LANG']['MSC']['helpWizard'])
-                       . '" onclick="Backend.openModalIframe({\'width\':735,\'height\':405,\'title\':\''
-                       . StringUtil::specialchars(str_replace("'", "\\'", $arrField['label'][0]))
-                       . '\',\'url\':this.href});return false">'
-                       . Image::getHtml(
-                           'about.gif',
-                           $GLOBALS['TL_LANG']['MSC']['helpWizard'],
-                           'style="vertical-align:text-bottom"'
-                       ) . '</a>';
+                . '" title="' . StringUtil::specialchars($GLOBALS['TL_LANG']['MSC']['helpWizard'])
+                . '" onclick="Backend.openModalIframe({\'width\':735,\'height\':405,\'title\':\''
+                . StringUtil::specialchars(str_replace("'", "\\'", $label))
+                . '\',\'url\':this.href});return false">'
+                . Image::getHtml(
+                    'about.gif',
+                    $GLOBALS['TL_LANG']['MSC']['helpWizard'],
+                    'style="vertical-align:text-bottom"'
+                ) . '</a>';
         }
 
         // Add the popup file manager

src/EventListener/Mcw/TinyMce.php

@@ -46,9 +46,9 @@ public function executeEvent(GetTinyMceStringEvent $event)
         $table   = $event->getTableName();
         $fieldId = $event->getFieldId();
 
-        list($file, $type) = explode('|', $field['eval']['rte'], 2) + [null, null];
+        list ($file, $type) = explode('|', $field['eval']['rte'] ?? '') + [null, null];
 
-        $fileBrowserTypes = array();
+        $fileBrowserTypes = [];
         // Since we don't know if this is the right call for other versions of contao
         // we won't use dependencies injection.
         $pickerBuilder = System::getContainer()->get('contao.picker.builder');
@@ -59,11 +59,14 @@ public function executeEvent(GetTinyMceStringEvent $event)
             }
         }
 
+        // Convert fileBrowserTypes to a comma-separated string for TinyMCE compatibility.
+        $fileBrowserTypesString = implode(',', $fileBrowserTypes);
+
         /** @var BackendTemplate|object $objTemplate */
         $objTemplate                   = new BackendTemplate('be_' . $file);
         $objTemplate->selector         = 'ctrl_' . $fieldId;
         $objTemplate->type             = $type;
-        $objTemplate->fileBrowserTypes =  json_encode($fileBrowserTypes);
+        $objTemplate->fileBrowserTypes = $fileBrowserTypesString;
         $objTemplate->source           = $table . '.' . $fieldId;
 
         // Deprecated since Contao 4.0, to be removed in Contao 5.0