- Status changed to Needs work
over 1 year ago 10:36pm 14 May 2023 - 🇦🇹Austria paper boy
The patch #8 creates
/src/Tests/ContentModerationTranslationTest.php
.ContentModerationTranslationTest
extendsInlineEntityFormTestBase
which is not defined in this namespace.This breaks automatic tests on the 2.0.x branch:
PHP Fatal error: Uncaught Error: Class "Drupal\inline_entity_form\Tests\InlineEntityFormTestBase" not found in /var/www/html/modules/contrib/inline_entity_form/src/Tests/ContentModerationTranslationTest.php:12
See this result for example: https://www.drupal.org/pift-ci-job/2666453 →
- 🇦🇹Austria paper boy
The test in
ContentModerationTranslationTest.php
andInlineEntityFormTestBase
require some refactoring, asInlineEntityFormTestBase
extendsWebDriverTestBase
.ContentModerationTranslationTest.php
requires ause Drupal\Tests\inline_entity_form\FunctionalJavascript\InlineEntityFormTestBase;
and the deprecated calls todrupalPostAjaxForm(..)
need to replaced withdrupalPostForm
.See https://www.drupal.org/docs/automated-testing/converting-simpletests-to-... → .
- First commit to issue fork.
- @shivam_tiwari opened merge request.
- Status changed to Needs review
over 1 year ago 12:03pm 16 May 2023 - Status changed to Needs work
over 1 year ago 12:20pm 16 May 2023 - 🇺🇦Ukraine podarok Ukraine
https://www.drupal.org/pift-ci-job/2668244 →
> Drupal\Core\Composer\Composer::upgradePHPUnit PHP Fatal error: Declaration of Drupal\inline_entity_form\Tests\ContentModerationTranslationTest::setUp() must be compatible with Drupal\Tests\inline_entity_form\FunctionalJavascript\InlineEntityFormTestBase::setUp(): void in /var/www/html/modules/contrib/inline_entity_form/src/Tests/ContentModerationTranslationTest.php on line 30
- 🇦🇹Austria paper boy
setUp()
must be defined asprotected function setUp(): void {
I also see some other errors when I open
ContentModerationTranslationTest.php
in Vscode:Undefined method 'drupalPostAjaxForm'
Undefined method 'drupalPostAjaxForm'
Undefined method 'assertTrue'
- @shivam_tiwari opened merge request.
- Status changed to Needs review
over 1 year ago 1:32pm 16 May 2023 - 🇦🇹Austria paper boy
Thank you @shivam_tiwari. Did not run a test but code looks good to me now.
@podarok, could you please merge and run a test?
- Status changed to Fixed
over 1 year ago 3:25pm 16 May 2023 Automatically closed - issue fixed for 2 weeks with no activity.
- Status changed to Needs work
11 months ago 6:06pm 9 December 2023 - 🇧🇪Belgium falc0
For me some fields where not hidden when they should've been hidden. Updated the patch from #11 to fix that.
At first, thanks for the patch, works great so far :) One thing still not working is that if the field references a media element containing an image file upload, in the translation record I can't delete the referenced item but only edit it.
Not sure if this only applies to referenced media entities but also for other ones.
Does anyone else have this behavior?