fix: Suppression des arguments superflus dans les appels de méthodes (PHPStan lvl 1)

[kwizer15]

Description

Suppression des arguments passés en trop lors d'appels de méthodes. Ces paramètres n'existent pas dans la signature des fonctions appelées et sont donc ignorés silencieusement par PHP. Aucun changement de comportement.

Exemples :

• cache::set() appelé avec 4 args au lieu de 3
• history::getHistoryFromCalcul() avec un 6e argument inexistant
• interactQuery::byInteractDefId() avec un 2e argument inexistant
• listener::all(), queue::all(), jeeObject::getUISelectList() avec des arguments en trop
• jeedom::update() avec un 2e argument inexistant
• repo_market::backup_restore() avec un 2e argument inexistant
• update::getLastAvailableVersion() avec un argument inexistant
• ajax::success($return) remplacé par ajax::success() quand $return n'est pas défini
• user::removeBanIp() : retour void utilisé comme argument de ajax::success()

Suggested changelog entry

Nettoyage des arguments superflus dans les appels de méthodes (PHPStan)

Related issues/external references

Découpage de #3122

Types of changes

• Bug fix (non-breaking change which fixes)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)
  • This change is only breaking for integrators, not for external standards or end-users.
• Documentation improvement

PR checklist

• I have checked there is no other PR open for the same change.
• I have read the Contribution guidelines for this project.
• I grant the project the right to include and distribute the code under the GNU.
• I have added tests to cover my changes.
• I have verified that the code complies with the projects coding standards.
• [Required for new sniffs] I have added MD documentation for the sniff.

[Mips2648]

ok, this was not an issue as such, both will work same way I think but it is more explicit and clear now