EntityTranslationMenuTranslationTestCase::createPage() and EntityTranslationMenuTranslationTestCase::createTranslation() are incompatible with their parents which throws strict warning all over the place.

CommentFileSizeAuthor
#4 entity_translation-strict-warnings-1991452-4.patch3.83 KBtwistor
#1 entity_translation-strict-warnings-1991452-1.patch3.05 KBtwistor

Comments

twistor’s picture

twistor commented
Status: Active » Needs review
StatusFileSize
new entity_translation-strict-warnings-1991452-1.patch3.05 KB

Here is a simple approach, rename createPage() to createMenuPage() and createTranslation() to createMenuTranslation(). There might be a better, deeper re-factoring that could be done, I didn't really dig into the tests.

bforchhammer’s picture

bforchhammer commented

I'd rather keep the current method names and make them private or protected (that should work as well, no?); Hm, can we do that with the recommended PHP version for D7?

twistor’s picture

twistor commented

That won't help. It's not a visibility problem. The idea is that you should be able to use a class or its children interchangeably, so no method may have more required parameters than its parent method.

twistor’s picture

twistor commented
StatusFileSize
new entity_translation-strict-warnings-1991452-4.patch3.83 KB

This should work as well.

bforchhammer’s picture

bforchhammer commented
Status: Needs review » Fixed

Okay, works for me. Committed #4, thanks!

Status: Fixed » Closed (fixed)

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

  • Commit 274769d on 7.x-1.x, factory, et-fc, revisions authored by twistor, committed by bforchhammer: 
    Issue #1991452 by twistor: Fixed Strict warnings on testing page.

  • Commit 274769d on 7.x-1.x, factory, et-fc, revisions, workbench authored by twistor, committed by bforchhammer: 
    Issue #1991452 by twistor: Fixed Strict warnings on testing page.