- Issue created by @heddn
can you please share more details about how to reproduce this error...
- First commit to issue fork.
- 🇷🇴Romania amateescu
The documentation for `\Drupal\group\Entity\GroupInterface::addMember()` says
Leave blank to make the user just a member.
so that's what I did in
\Drupal\Tests\group_content_menu\Functional\GroupContentMenuTest::testGroupPermissions()
.The change in
\Drupal\group_content_menu\Plugin\Block\GroupMenuBlock::blockForm()
was needed for testing with Drupal 11 testing: https://git.drupalcode.org/issue/group_content_menu-3481817/-/jobs/3124325 What I think is now the failure of PHP unit test are because of the use of get_class() without an argument as the use of get_class() without an argument is deprecated in PHP 8. and changing that with
$this
can solve this, what do you say ??- 🇷🇴Romania amateescu
@dhruv.mittal, that problem is already fixed by using
static::class
instead ofget_class()
. - 🇷🇴Romania amateescu
And if you mean that https://git.drupalcode.org/issue/group_content_menu-3481818/-/jobs/3124610 is still failing, that's because 📌 Drupal 11 support Active is not merged yet.
- heddn Nicaragua
Posted a small question on the MR. But this is looking in good shape.
-
heddn →
committed 7b540e29 on 3.0.x authored by
amateescu →
Issue #3481818 by amateescu, heddn: Fix HEAD failures
-
heddn →
committed 7b540e29 on 3.0.x authored by
amateescu →
-
heddn →
committed fd594e74 on 2.0.x authored by
amateescu →
Issue #3481818 by amateescu, heddn: Fix HEAD failures
-
heddn →
committed fd594e74 on 2.0.x authored by
amateescu →
Automatically closed - issue fixed for 2 weeks with no activity.