Galette

Bonjour,
la correction sur le chemin de fichier a bien fonctionné, maintenant les deux méthodes renvoient la même erreur :

PHP Notice: Undefined variable: container in /home/gullarverne/galette.linuxarverne.org-web/htdocs/includes/galette.inc.php on line 257
PHP Fatal error: Uncaught Error: Call to a member function get() on null in /home/gullarverne/galette.linuxarverne.org-web/htdocs/includes/galette.inc.php:257
Stack trace:
#0 /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php(44): require_once()
#1 {main}
thrown in /home/gullarverne/galette.linuxarverne.org-web/htdocs/includes/galette.inc.php on line 257

Merci aussi pour la précision sur les cron jobs (en lisant mieux la doc de tuxfamily je comprend qu'il s'agit juste de la restriction de cron pour ne proposer que la possibilité d'exécuter des fichiers).

En effet, voilà un second correctif :
https://github.com/galette/galette/pull/44/commits/7695512980402f0d469dc6ca61ad8d15807599e3

Ça ne fonctionne pas encore :

PHP Notice:  Undefined variable: container in /home/gullarverne/galette.linuxarverne.org-web/htdocs/includes/galette.inc.php on line 257
PHP Fatal error:  Uncaught Error: Call to a member function get() on null in /home/gullarverne/galette.linuxarverne.org-web/htdocs/includes/galette.inc.php:257
Stack trace:
#0 /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php(44): require_once()
#1 {main}
  thrown in /home/gullarverne/galette.linuxarverne.org-web/htdocs/includes/galette.inc.php on line 257

La ligne qui inclut le fichier dependencies.php est elle bien présente ?
Je ne reproduit pas du tout ; je penche pour un match non appliqué. Le mieux, c'est de récupérer le fichier complet ici https://raw.githubusercontent.com/galette/galette/7695512980402f0d469dc6ca61ad8d15807599e3/galette/cron/reminder.php

le problème n'a pas l'air de se trouver dans ce fichier, car même avec le fichier donné en lien, le même message d'erreur persiste.

Vérifie que tu as bien appliqué les deux patchs dans leur intégralité, et qu'il n'y a pas d'erreur de syntaxe dans les fichiers modifiés, ces erreurs sont fréquentes lors de modifications à la main de fichiers.

J'ai vérifié, je ne vois pas quelle erreur dans les copier-coller j'aurai pu faire.
Est-il possible de me donner les liens vers les fichiers complets includes/dependencies.php includes/main.inc.php et includes/galette.inc.php comme ça a été fait avec cron/reminder.php pour être sûr ?

Les fichiers sont tous sur cette branche :
https://github.com/galette/galette/tree/hotfix/0.9.3.1/galette

Puis tu vas sur le fichier voulu, et tu utilises le bouton "raw" Pour avoir le code source.

Après avoir remplacé les quatre fichiers par ceux de la branche ça avait changé de message :

localhost - 2020-01-26 20:14:44 - 4 - Lang fr_FR does not exist, switching to default.
PHP Notice:  Undefined index: fr_FR in /home/gullarverne/galette.linuxarverne.org-web/htdocs/lib/Galette/Core/I18n.php on line 151

En remettant les versions respectivement à v0.9.3 et 0.930 le message de mon commentaire #6 contenant "Undefined variable: container" revient à l'identique.

Ha OK ! En effet, la vérification de la version n'est pas effectuée correctement, et le script ne renvoie rien dans ce cas là (entre autres)...

J'en suis rendu de mon côté à des problèmes de requêtes SQL (instance sous PostgreSQL) pour le moment :/

Dans le fichier galette.inc.php, les lignes suivantes sont à déplacer :

if ($cron) {
    $container->get('login')->logCron(basename($argv[0], '.php'));
}

Il faut les remettre dans cron/reminder.php avant le if (!$container->get('login')->isCron()).

Avec la modification, c'est toujours en erreur, mais avec un message différent (toutefois les premières lignes sont celles de mon commentaire #12) :

localhost - 2020-01-27 07:44:44 - 4 - Lang fr_FR does not exist, switching to default.
PHP Notice:  Undefined index: fr_FR in /home/gullarverne/galette.linuxarverne.org-web/htdocs/lib/Galette/Core/I18n.php on line 151
PHP Fatal error:  Uncaught ArgumentCountError: Too few arguments to function Galette\Repository\Reminders::__construct(), 0 passed in /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php on line 68 and exactly 1 expected in /home/gullarverne/galette.linuxarverne.org-web/htdocs/lib/Galette/Repository/Reminders.php:73
Stack trace:
#0 /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php(68): Galette\Repository\Reminders->__construct()
#1 {main}
  thrown in /home/gullarverne/galette.linuxarverne.org-web/htdocs/lib/Galette/Repository/Reminders.php on line 73

Matthieu Coudert a écrit :

Avec la modification, c'est toujours en erreur, mais avec un message différent (toutefois les premières lignes sont celles de mon commentaire #12) :
[...]

En effet, j'avais corrigé cette erreur fatale en local :

diff --git a/galette/lib/Galette/Repository/Reminders.php b/galette/lib/Galette/Repository/Reminders.php
index 4024c092..20cda164 100644
--- a/galette/lib/Galette/Repository/Reminders.php
+++ b/galette/lib/Galette/Repository/Reminders.php
@@ -70,7 +70,7 @@ class Reminders
      *
      * @param array $selected Selected types for sending
      */
-    public function __construct($selected)
+    public function __construct($selected = null)
     {
         if (isset($selected) && is_array($selected)) {
             $this->selected = array_map('intval', $selected);

Nouvelle erreur :

localhost - 2020-01-27 13:00:53 - 4 - Lang fr_FR does not exist, switching to default.
PHP Notice:  Undefined index: fr_FR in /home/gullarverne/galette.linuxarverne.org-web/htdocs/lib/Galette/Core/I18n.php on line 151
PHP Fatal error:  Uncaught Slim\Exception\ContainerValueNotFoundException: Identifier "hist" is not defined. in /home/gullarverne/galette.linuxarverne.org-web/htdocs/vendor/slim/slim/Slim/Container.php:105
Stack trace:
#0 /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php(75): Slim\Container->get('hist')
#1 {main}
  thrown in /home/gullarverne/galette.linuxarverne.org-web/htdocs/vendor/slim/slim/Slim/Container.php on line 105

Un autre correctif :
https://github.com/galette/galette/commit/b404c7b9aba9905bd8cdfc4f98bf7cd94cf14e12

Le correctif en commentaire 15 est inclus.
Je n'ai pas encore trouvé pour la notice (mais ce ne devrait pas être bloquant).

Je n'ai pas d'environnement de test en mesure d'envoyer des courriels (j'ai même du ajouter une notion de timeout pour ne pas glander 5 minutes à chaque test :D) ; et sur le branche hotfix/0.9.3.1, l'envoi des reminders semble fonctionner en ligne de commande (PHP 7.3, comme chez TF).

Quant à au problème « Undefined index: fr_FR » ; je ne comprends pas trop qui se passe, et je ne l'ai pas en local. Mais j'ai aussi d'autres correctifs (tout ceux de la branche hotfix/0.9.3.1).
Pas de soucis sur la version web (il faut certainement passer en mode debug et consulter les logs pour en être sûr) ?

On en voit la fin !
Ça affichait encore des messages d'erreur :

PHP Notice:  Undefined variable: error_detected in /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php on line 90
PHP Warning:  count(): Parameter must be an array or an object that implements Countable in /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php on line 90
PHP Notice:  Undefined variable: error_detected in /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php on line 107
PHP Warning:  count(): Parameter must be an array or an object that implements Countable in /home/gullarverne/galette.linuxarverne.org-web/htdocs/cron/reminder.php on line 107

$error_detected = array();

Reste à voir pourquoi cette variable n'était pas définie.

Est-ce que la notice « Undefined index: fr_FR » est considérée comme une erreur par les cron jobs de tuxfamilly ? Si oui, dois-je rediriger vers /dev/null pour ne pas recevoir de mail à chaque exécution ?

Matthieu Coudert a écrit :

On en voit la fin !

Oui :D

Ça affichait encore des messages d'erreur :
[...]
J'ai tenté un ajout : [...] à la ligne 76 de cron/reminder.php

Bien vu.

Je viens de recevoir un mail "Votre adhésion arrive à terme" sur l'adhérent fictif que j'avais créé pour tester, donc ça a fonctionné \o/

Reste à voir pourquoi cette variable n'était pas définie.

Elles l'étaient dans une version précédente ; ça a été supprimé, mais pas remis d'équerre à cet endroit :/

Est-ce que la notice « Undefined index: fr_FR » est considérée comme une erreur par les cron jobs de tuxfamilly ? Si oui, dois-je rediriger vers /dev/null pour ne pas recevoir de mail à chaque exécution ?

Toute sortie du script cron enverra un mail. Pour contrer ce problème, oui, rediriger la sortie standdard sur /dev/null sera une solution. Le risque, c'est de louper de vraies qui seraient affichées en sortie standard (et non comme elles le devraient en sortie erreur) ; ça peut malheureusement arriver (j'en ai corrigé 2 pour le script des reminders :D).

Aurais-tu la possibilité de mener quelques tests sur cette notice ? Il faudrait voir si tu as ce problème en ayant configuré Galette et l'adhérent en anglais et en français ? (genre galette fr, adh : fr => KO - galette en, adh fr => OK, ...).

Je suppose que les fichiers de langue sont bien présents, et que ton interface est bien traduite lorsque tu changes ? Est-ce que tu peux vérifier que les droits semblent cohérents sur ces derniers.