Problem/Motivation

Warning: Declaration of MenuNodeAPIBasicTestCase::setUp() should be compatible with MenuNodeAPITestCase::setUp($modules = Array) in include_once() (line 3517 of /var/www/html/public/includes/bootstrap.inc).
Warning: Declaration of MenuNodeAPIHookTestCase::setUp() should be compatible with MenuNodeAPITestCase::setUp($modules = Array) in include_once() (line 3517 of /var/www/html/public/includes/bootstrap.inc).

Steps to reproduce

Enable simpletest and go to the listing page with error messages enabled.

Proposed resolution

Match the declaration

API changes

Changed the visibility to protected to match base class.

Data model changes

CommentFileSizeAuthor
#2 3243877-2-test-declaration-setup-match.patch1.08 KBjoelpittet

Comments

joelpittet created an issue. See original summary.

joelpittet’s picture

joelpittet
he/him
Primary language English
Location Vancouver
commented
StatusFileSize
new 3243877-2-test-declaration-setup-match.patch1.08 KB
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Needs review » Reviewed & tested by the community

  • vuil committed 84a221f on 7.x-1.x authored by joelpittet 
    Issue #3243877 by joelpittet, vuil: Warning: Declaration test class ::...
vuil’s picture

vuil
Location Bulgaria 🇧🇬 🇪🇺 🌍
commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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