Project

General

Profile

Anomalie #1496

Ajout des contributions; faire le commit avant d'appeler callPostContributionScript()

Added by Manuel Her about 1 month ago. Updated about 1 month ago.

Status:
Fermé
Priority:
Normal
Category:
Core
Target version:
Start date:
10/22/2020
Due date:
% Done:

100%

Estimated time:
Version utilisée:
Vote:

Description

Bonjour,

J'utilise un script de postcontribution.
Il récupére l'Id de la nouvelle contribution via les paramètres passé en JSON et charge celle ci avec Contribution::load()

le problème est que l'appel de
$emitter->emit('contribution.add', $this);
se fait avant le commit (ligne 681):
$this->zdb->connection->commit();

Au moment de l'appel du script, je ne suis donc pas en mesure de charger la nouvelle contribution.

Cette partie fonctionnait avant l'arrivée des Controllers.

Manuel

#1

Updated by Johan Cwiklinski about 1 month ago

  • Category set to Core
  • Target version deleted (0.9.5)

Si ça fonctionnait avant l'implémentation des contrôleurs, c'était un coup de chance a priori.

De la même manière, les champs dynamiques sont enregistrés après que l'événement ait été déclenché, et ne sont pas accessibles non plus. Et c'est valable sur l'ensemble des évènements déclenchés.

#2

Updated by Johan Cwiklinski about 1 month ago

  • Assignee set to Johan Cwiklinski
  • Target version set to 0.9.4.1
#3

Updated by Johan Cwiklinski about 1 month ago

  • Target version changed from 0.9.4.1 to 0.9.4.2
#4

Updated by Johan Cwiklinski about 1 month ago

Voici une proposition de correctif ; je n'ai pas encore eu le temps de tester :
https://github.com/galette/galette/pull/76

#5

Updated by Johan Cwiklinski about 1 month ago

  • Status changed from Nouveau to In Progress
#6

Updated by Manuel Her about 1 month ago

Merci pour le patch. ça marche :)
Manuel

#7

Updated by Johan Cwiklinski about 1 month ago

  • Status changed from In Progress to Résolu
  • % Done changed from 0 to 100
#8

Updated by Johan Cwiklinski about 1 month ago

  • Status changed from Résolu to Fermé

Also available in: Atom PDF