Project

General

Profile

Anomalie #462

Tableau de bord/ Bouton 'rappel' en 728

Added by Michel Verdier almost 7 years ago. Updated over 6 years ago.

Status:
Fermé
Priority:
Haut
Category:
Core
Target version:
-
Start date:
12/20/2012
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

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...)


Related issues

Related to Galette - Commandes #368: Envoi automatique de mails de rappel d'échéance de cotisationFermé10/07/2012

Actions
Precedes Galette - Evolution #605: Limiter les envois mailingsNouveau05/05/2013

Actions

Associated revisions

Revision b689792f (diff)
Added by Johan Cwiklinski almost 7 years ago

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

History

#1

Updated by Johan Cwiklinski almost 7 years ago

  • Assignee set to Johan Cwiklinski
  • Category set to Core
  • Status changed from Nouveau to In Progress
  • Priority changed from Normal to 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

Updated by Johan Cwiklinski almost 7 years ago

  • Priority changed from Urgent to 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

Updated by Johan Cwiklinski almost 7 years ago

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

Updated by Michel Verdier almost 7 years ago

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

Updated by Johan Cwiklinski almost 7 years ago

  • % Done changed from 0 to 50
#6

Updated by Johan Cwiklinski over 6 years ago

  • % Done changed from 50 to 100
  • Status changed from In Progress to 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 :)

Also available in: Atom PDF