A minor issue, but the getInfo() methods in the LegalRegisterTestCase, LegalScrollBoxDisplayTestCase, LegalScrollBoxCSSDisplayTestCase, LegalHTMLTextDisplayTestCase and LegalLinkDisplayTestCase (class names added for Googlbing) classes should have a static declaration. call_user_func() on line 327 of simpletest.module calls the class/method statically.

The enclosed patch fixes this minor issue.

CommentFileSizeAuthor
getInfo_should_be_static-Unknown-1.patch1.92 KBsdrycroft

Comments

geek-merlin’s picture

geek-merlin
Primary language German
Location Freiburg, Germany
commented
Issue summary: View changes
Status: Needs review » Reviewed & tested by the community
Related issues: +#2499285: Fixed in Legal sandbox

Yes this makes sense.

  • Robert Castelo committed 800e9c7 on 7.x-1.x 
    Issue #1860018 by sdrycroft: getInfo() should be declared "static"
robert castelo’s picture

robert castelo commented
Status: Reviewed & tested by the community » Fixed

Thanks sdrycroft. Added to dev and will be part of next release (soon).

Status: Fixed » Closed (fixed)

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