Project

General

Profile

Actions

Anomalie #1496

closed

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

Added by Manuel Her almost 4 years ago. Updated almost 4 years ago.

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

100%

Estimated time:
Version utilisée:

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

Actions #1

Updated by Johan Cwiklinski almost 4 years 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.

Actions #2

Updated by Johan Cwiklinski almost 4 years ago

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

Updated by Johan Cwiklinski almost 4 years ago

  • Target version changed from 0.9.4.1 to 0.9.4.2
Actions #4

Updated by Johan Cwiklinski almost 4 years ago

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

Actions #5

Updated by Johan Cwiklinski almost 4 years ago

  • Status changed from Nouveau to In Progress
Actions #6

Updated by Manuel Her almost 4 years ago

Merci pour le patch. ça marche :)
Manuel

Actions #7

Updated by Johan Cwiklinski almost 4 years ago

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

Updated by Johan Cwiklinski almost 4 years ago

  • Status changed from Résolu to Fermé
Actions

Also available in: Atom PDF