entityViewModeAlter throws an exception in php 8.2

Here's a patch. Please review.

Patch #2 applied successfully and working fine . It looks good to me
Thank you

Patch at #2 works like a charm.
Since we are already two people confirming it is ok, I guess we can move to RTBC?

Thanks!

Got the same issue, Patch from #2 applied and solved the problem.

Patch #2 solved the issue in PHP 8.1.17 in a D10.0.11 local environment
BUT I was not able to apply the patch from Issue#3345129 🐛 Override not working when view mode to override is not selected in the available view mode list Needs review at the same time as they BOTH alter ParagraphsViewmodeBehavior.php.

Patch #2 applied successfully in PHP 8.1 and D10.1.6. It looks good to me
Thank you

Hi guys. This issue affects https://www.drupal.org/project/paragraphs_viewmode/issues/3345129 🐛 Override not working when view mode to override is not selected in the available view mode list Needs review so i made patch that is compatible with both problems. Solves on my side!

Confirming, patches work.

+1

Hi there!

Currently on Drupal 10.3.6 and Paragraphs 8.x-1.18.

I tried to apply patch #8 → but its currently failing.

The Error:

TypeError: Drupal\paragraphs_viewmode\Plugin\paragraphs\Behavior\ParagraphsViewmodeBehavior::entityViewModeAlter(): Argument #3 ($context) must be of type array, null given, called in /var/www/html/web/modules/contrib/paragraphs_viewmode/paragraphs_viewmode.module on line 39 in Drupal\paragraphs_viewmode\Plugin\paragraphs\Behavior\ParagraphsViewmodeBehavior->entityViewModeAlter() (line 175 of modules/contrib/paragraphs_viewmode/src/Plugin/paragraphs/Behavior/ParagraphsViewmodeBehavior.php).

Could not figure out why patch is not working. Many thanks!

Chris

Looks (to me) like the patch is already merged in Paragraphs 8.x-1.18 ?But not sure. Maybe i am missing something ...

Looks like the context parameter is still there, but not sure if just removing that will savely remove the issue. But this at least prevents throwing the error.

Automatically closed - issue fixed for 2 weeks with no activity.