Fix/deprecated notices

[pifou25]

Description

il n'est pas acceptable de masquer les notices / deprecation avec un niveau moins élevé d'affichage des erreurs. J'ai donc affiché toutes les notices possible et je les corrige au fil de l'eau... ça pourrait nous éviter de mauvaises surprises pour plus tard.

Note pour plus tard : pourquoi la fonction fromHumanReadable est présente 3 fois à l'identique (dans eqLogic.class.php jeeObject.class.php et scenario.class.php)

E_WARNING: Undefined array key "HTTP_IF_MODIFIED_SINCE" in /var/www/html/core/php/getResource.php on line 45

E_DEPRECATED: strtotime(): Passing null to parameter #1 ($datetime) of type string is deprecated in /var/www/html/core/php/getResource.php on line 45

E_DEPRECATED: strpos(): Passing null to parameter #1 ($haystack) of type string is deprecated in /var/www/html/core/class/utils.class.php on line 235

E_DEPRECATED: preg_match_all(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/html/core/class/eqLogic.class.php on line 528

E_DEPRECATED: preg_match_all(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/html/core/class/scenario.class.php on line 695

E_DEPRECATED: preg_match_all(): Passing null to parameter #2 ($subject) of type string is deprecated in /var/www/html/core/class/jeeObject.class.php on line 181

Suggested changelog entry

fix deprecated notices

Related issues/external references

Fixes #

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 [La ligne directrice pour contribuer à ce projet / 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.

[pifou25]

Je n'arrive pas à faire 2 PR simultanées sans que ça se cumule, il y a donc aussi l'autre ici :
#2654

[Hotfirenet]

@pifou25 tu peux mettre a jour l'alpha dans ta PR, il y avait une erreur sur la classe historique. 🙏

[zoic21]

BONJOUR A TOI AUSSI !!!!!!!!!

Il n'est pas acceptable de commencer un message comme tu le fais sans avoir au moins une formule de politesse !!!!

Pour en revenir au sujet si c'est normal de masquer les notice, seul les dev doivent les avoir sinon ca fait plein de message pas vraiment utile que les utilisateurs ne comprennent pas. Un notice n'est vraiment pas important et je ne connais pas un seul programme qui les affiches.

Pour la fonction fromHumanReadable oui elle est a plusieurs endroit mais ne fait pas la meme chose, tout comme la fonction save, remove et j'en passe.

[pifou25]

BONJOUR A TOI AUSSI !!!!!!!!!

Je tiens à m'excuser pour le ton involontairement offensant, je me suis pas relu et c'est bien sûr pas ma pensée, en tout cas merci pour ta réponse :)
il est tout à fait légitime de masquer les erreurs / notices / autres dans une appli distribuée à des millions d'utilisateurs, je voulais juste dire que, en tant que dev, sur la branche alpha, on devrait faire l'effort de les corriger si on les trouve, et j'essaye de le faire à mon niveau :)
Comme cette PR est mixée avec l'autre, je la clos, pour séparer 2 sujets. J'ai un peu du mal avec github desktop mais je progresse...