- Issue created by @EthanT
- Status changed to Needs work
5 months ago 11:48pm 27 June 2024 - 🇺🇸United States smustgrave
So with issues like this need to some research/backtracing to figure out why it's not assumed to be a string. Putting a check may make the warning go away but could be hiding a larger issue.
MR should be against 11.x as the current development branch
- 🇺🇸United States EthanT Sarasota, Florida
Yeah, very likely. That said, I think it’s worthwhile to open up an issue / create an initial solution when a fatal error is being thrown.
- 🇺🇸United States smustgrave
Fatal error will definitely need test coverage to prevent in the future
- Status changed to Postponed: needs info
5 months ago 10:19pm 30 June 2024 - 🇳🇿New Zealand danielveza Brisbane, AU
This may be resolved by 🐛 Add missing category to Drupal\layout_builder\Plugin\Layout\BlankLayout and let modules and themes alter the list of layouts Fixed .
Since this needs a contrib module to be able to replicate this, I think it would be good to understand why this is happening with LB plus so we can define if the issue exists in that module or something that should be fixed specifically in core. Just casting things to strings feels fishy to me.
- 🇵🇹Portugal adaragao
Could this be also related with 3408206 🐛 Deprecated function: strnatcasecmp(): Passing null to parameter #2 ($string2) of type string is deprecated in Drupal\Core\Language\Language::Drupal\Core\Language\{closure}() (line 161 of /app/docroot/core/lib/Drupal/Core/Language/Language.php) Postponed: needs info ?