Fix PHPCQ

Author: zonky2

src/CoreBundle/EventListener/DcGeneral/Breadcrumb/BreadcrumbDcaSettingConditionListener.php

@@ -111,7 +111,7 @@ private function getConditionAttribute($settingId)
         } else {
             $title = StringUtil::deserialize($setting->legendtitle, true);
             // @deprecated usage of TL_LANGUAGE - remove for Contao 5.0.
-            return ($title[LocaleUtil::formatAsLocale($GLOBALS['TL_LANGUAGE'])] ?? current($title));
+            return ($title[LocaleUtil::formatAsLocale($GLOBALS['TL_LANGUAGE'])] ?? (string) current($title));
         }
 
         return 'unknown ' . $setting->dcatype;

src/FrontendIntegration/FrontendFilter.php

@@ -634,11 +634,7 @@ private function getBaseParameters(array $values): array
             return [];
         }
 
-        if (null === ($filterSetting = $this->objFilterConfig->getFilterCollection())) {
-            throw new RuntimeException(
-                'Error: no filter object defined, call setFilterSettings() before setFilterParameters().'
-            );
-        }
+        $filterSetting = $this->objFilterConfig->getFilterCollection();
 
         $presetNames     = $filterSetting->getParameters();
         $filterParamKeys = \array_keys($filterSetting->getParameterFilterNames());

src/FrontendIntegration/MetaModelHybrid.php

@@ -41,12 +41,13 @@
 /**
  * Base implementation of a MetaModel Hybrid element.
  *
- * @property string $id                       The id of the element.
- * @property string $name                     The module name to use (if type is module).
- * @property string $metamodel                The id of the MetaModel to use.
- * @property string $metamodel_filtering      The id of the MetaModel filter setting to use.
- * @property string $metamodel_rendersettings The id of the MetaModel render setting to use.
- * @property bool   $metamodel_sort_override  The flag to override sorting.
+ * @property string       $id                       The id of the element.
+ * @property string       $name                     The module name to use (if type is module).
+ * @property string       $metamodel                The id of the MetaModel to use.
+ * @property string       $metamodel_filtering      The id of the MetaModel filter setting to use.
+ * @property string       $metamodel_rendersettings The id of the MetaModel render setting to use.
+ * @property bool         $metamodel_sort_override  The flag to override sorting.
+ * @property list<string> $cssID                    The CSS id and classes.
  *
  * @psalm-type TDatabaseResult=object{
  *   cssID: string,
@@ -155,7 +156,7 @@ protected function getConnection()
      * Create a new instance.
      *
      * @param Result|TDatabaseResult $objElement The object from the database.
-     * @param string                             $strColumn  The column the element is displayed within.
+     * @param string                 $strColumn  The column the element is displayed within.
      */
     public function __construct($objElement, $strColumn = 'main')
     {
@@ -165,6 +166,7 @@ public function __construct($objElement, $strColumn = 'main')
         $this->arrData = \method_exists($objElement, 'row') ? $objElement->row() : (array) $objElement;
 
         // Get CSS ID and headline from the parent element (!).
+        /** @psalm-suppress PropertyTypeCoercion */
         $this->cssID      = StringUtil::deserialize($objElement->cssID, true);
         $this->typePrefix = $objElement->typePrefix ?? '';
         $this->strKey     = $objElement->type;

src/Widgets/SubDcaWidget.php

@@ -97,7 +97,7 @@ class SubDcaWidget extends Widget
     /**
      * The prepared widgets.
      *
-     * @var array
+     * @var list<list<Widget>>
      */
     protected $arrWidgets = [];
 
@@ -567,9 +567,9 @@ protected function buildOptions()
         $options = [];
         foreach ($this->arrWidgets as $widgetRow) {
             $columns = [];
-            foreach ((array) $widgetRow as $widget) {
-                /** @var Widget $widget */
-                $rawValign = (string) ($widget->valign ?? '');
+            foreach ($widgetRow as $widget) {
+                /** @psalm-suppress UndefinedMagicPropertyFetch */
+                $rawValign = ((string) $widget->valign) ?: '';
                 $valign    = ($rawValign !== '' ? ' valign="' . $rawValign . '"' : '');
                 $rawClass = $widget->class;
                 $class    = ($rawClass !== '' ? ' class="' . $rawClass . '"' : '');