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

Files: 
CommentFileSizeAuthor
#4 entity_translation-strict-warnings-1991452-4.patch3.83 KBtwistor
PASSED: [[SimpleTest]]: [MySQL] 681 pass(es).
[ View ]
#1 entity_translation-strict-warnings-1991452-1.patch3.05 KBtwistor
PASSED: [[SimpleTest]]: [MySQL] 681 pass(es).
[ View ]

Comments

Status:Active» Needs review
StatusFileSize
new3.05 KB
PASSED: [[SimpleTest]]: [MySQL] 681 pass(es).
[ View ]

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.

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?

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.

StatusFileSize
new3.83 KB
PASSED: [[SimpleTest]]: [MySQL] 681 pass(es).
[ View ]

This should work as well.

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.