BO: Fix method overrride with different number of parameters

[neoteknic]

Affect PHP 7.2 (crash), don't crash in 7.1but violates LSP and should be fixed.

Questions	Answers
Branch?	"develop"
Description?	Invalid parameter of overrided method (should have the same number of paramters).
Type?	bug fix
Category?	BO
BC breaks?	no
Deprecations?	no
How to test?	Just run the admin dashboard, affect nothing as the $isNewTheme is not used in the overrided method.

[prestonBot]

Hi!

These(s) commit(s) name(s) seems to be incomplete or malformed, regarding our guidelines:

`BO : Fix method overrride with different number of parameters

Affect PHP 7.2 (crash), don't crash in 7.1but violates LSP and should be fixed.` is malformed or incomplete.

A valid commit name can be, for instance:

BO: Shows company in BO search if B2B is enabled

Would you mind to amend your commits' names?

To do this, open a command line window and use git commit --amend for the commit's name. See GitHub's help page for more information.

Note: this must be done via the command line: you can't do this just by changing the title of the pull-request from the GitHub interface! :)

Thank you!

(note: this is an automated message, but answering it will reach a real human )

[Quetzacoalt91]

Hello @neoteknic,

Thanks for your pull-request.
I started the work on PHP 7.2 in order to fix all the crashed reported by the logs. But as it is not released as stable yet, I haven't merged anything yet. You can find the first changes here: #8246. You will see that other controllers need to be updated as well.

Because your suggested changes are already present in #8246, I have to close this one. Sorry about that. :/

Best regards