Problem/Motivation

2 bugs found during beta5 tests

1 A little typing error which can be fixed by this:

++ b/src/Element/ComponentElementAlter.php
@@ -56,7 +56,7 @@ class ComponentElementAlter implements TrustedCallbackInterface {
 
       // Because SDC validator is sometimes confused by an empty slot.
       // We check the current slot render element.
-      if (self::isSlotEmpty($slot)) {
+      if (is_array($slot) && self::isSlotEmpty($slot)) {
         self::mergeSlotBubbleableMetadata($element, $slot, 1);
         unset($element['#slots'][$slot_id]);
         continue;

2. A fatal error I didn't find any solution yet:
Error: Typed property Drupal\Core\Plugin\DefaultPluginManager::$moduleExtensionList must not be accessed before initialization in Drupal\ui_patterns\ComponentPluginManager->getExtensionLabel() (line 360 of modules/custom/ui_patterns/src/ComponentPluginManager.php).

They may be related to your MR of yesterday. Can you have a look?

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

Comments

pdureau created an issue. See original summary.

pdureau’s picture

Title: [2.0.0-beta5] Final beta wrap-up » [2.0.0-beta5] DefaultPluginManager::$moduleExtensionList must not be accessed before initialization

christian.wiedemann made their first commit to this issue’s fork.

christian.wiedemann’s picture

Assigned: christian.wiedemann » pdureau
Status: Active » Needs review

pdureau’s picture

Assigned: pdureau » Unassigned
Status: Needs review » Fixed
pdureau’s picture

Status: Fixed » Closed (fixed)
christian.wiedemann’s picture