Projet

Général

Profil

Anomalie #462

Tableau de bord/ Bouton 'rappel' en 728

Ajouté par Michel Verdier il y a presque 5 ans. Mis à jour il y a plus de 4 ans.

Statut:
Fermé
Priorité:
Haut
Assigné à:
Catégorie:
Core
Version cible:
-
Début:
20/12/2012
Echéance:
% réalisé:

100%

Version utilisée:

Description

re
avec le display à 1 (maiss j'avais bien testé avant la generation de pdf)

Fatal error: Allowed memory size of 33554432 bytes exhausted (tried to allocate 512 bytes) in /home/www/leclub404/www/galette/lib/Galette/Repository/Members.php on line 232
connu non ? (c'était déjà le cas en 727, il me semble...)


Demandes liées

Lié à Galette - Commandes #368: Envoi automatique de mails de rappel d'échéance de cotisation Fermé 07/10/2012
Précède Galette - Evolution #605: Limiter les envois mailings Nouveau 05/05/2013

Révisions associées

Révision b689792f
Ajouté par Johan Cwiklinski il y a plus de 4 ans

Reminders should only concern active accounts, other minor improvements; refs #462

Historique

#1 Mis à jour par Johan Cwiklinski il y a presque 5 ans

  • Assigné à mis à Johan Cwiklinski
  • Catégorie mis à Core
  • Statut changé de Nouveau à In Progress
  • Priorité changé de Normal à Urgent

Aie, aie, aie... J'ai du louper quelque chose de ce côté là ; même avec d'avantage de mémoire, ça plante ; je me retrouve proprement déconnecté si je le laisse faire...

#2 Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • Priorité changé de Urgent à Haut

Bon, non, je n'ai pas forcément loupé grand chose...

Avec la base que tu m'as transmise, le rappel essaie de s'envoyer à ~2500 adhérents (qu'ils aient une adresse email ou pas importe peu, puisque les étiquettes peuvent être générées aussi). Il est clair que le système de mailing dans son ensemble n'a pas été prévu pour de tels volumes, l'envoi de mails en masse est para ailleurs souvent considéré comme du spam de toutes façons...

Est-ce que ça a réellement du sens de pouvoir envoyer 2500 rappels d'un coup ? Je ne pense pas. Il s'agit dans le cas de ta base pour la majorité de faux positifs, toutes ces personnes ne sont pas en retard, si ? Pas mal de comptes devraient être désactivés pour ne plus « faire de bruit ».

Quoi qu'il en soit, l'application continue de planter lamentablement ; je pense limiter cette fonctionnalité à un nombre « raisonnable » d'adhérents sélectionnés je pense ; et afficher un avertissement lorsque cette limite est atteinte. Il faudrait dans ce cas passer par le filtrage/la recherche avancée dans la liste des adhérents pour affiner la recherche.

#3 Mis à jour par Johan Cwiklinski il y a plus de 4 ans

Je suis allé vérifier une petite chose... Les adhérents inactifs sont pris en compte dans la recherche des retardataires pour le rappel ; ça ne devrait pas être le cas ! :-)

#4 Mis à jour par Michel Verdier il y a plus de 4 ans

Tout à fait d'accord !!!!
EN pratique je trouve que le bouton devrait s'adresser :
- soit aux adhérents à échéance proche
- soit aux retardataires.
Ce serait un raccourci ....
d'un point de vue fonctionnel, notre base contient l'historique du club depuis quasiment 30 ans mais on y tient...
En fait 700 adhérents à jours,un noyau dur de la moitié et une petite vingtaine qui ont cotisé pour 10 ans...à 404 euros....!!! (mis en place pour rire en 2004 lors du passage à l'euro, le tresorier voulant pas de cotisation à 40,4...J'y ai perdu ma barbe...pas longtemps :):) ).
Indexer l'activité du compte ?

#5 Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • % réalisé changé de 0 à 50

#6 Mis à jour par Johan Cwiklinski il y a plus de 4 ans

  • % réalisé changé de 50 à 100
  • Statut changé de In Progress à Fermé

En l'état actuel, les rappels ne devraient être envoyés qu'aux vrais retardataires.

Il faudra quand même que je prévoie un garde fou, histoire que ça n'explose plus la mémoire, notamment si une base était mal configurée (ou si un bug survenait à nouveau) ; c'est cependant de l'ordre de l'amélioration, et non pas de l'anomalie :)

Formats disponibles : Atom PDF