Projet

Général

Profil

Bug #10030

le backend systempayv2 n'appelle pas le __init__ de la classe parente

Ajouté par Frédéric Péters il y a environ 8 ans. Mis à jour il y a environ 8 ans.

Statut:
Fermé
Priorité:
Normal
Assigné à:
-
Version cible:
Début:
18 février 2016
Echéance:
% réalisé:

0%

Temps estimé:
Patch proposed:
Oui
Planning:

Fichiers


Demandes liées

Lié à Combo - Bug #10029: erreur lors d'une tentative de paiementFermé18 février 2016

Actions

Révisions associées

Révision a7dcc7a5 (diff)
Ajouté par Frédéric Péters il y a environ 8 ans

general: give same signature to all Payment object constructors (#10030)

Historique

#1

Mis à jour par Frédéric Péters il y a environ 8 ans

#2

Mis à jour par Frédéric Péters il y a environ 8 ans

  • Lié à Bug #10029: erreur lors d'une tentative de paiement ajouté
#3

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-systempayv2-set-service_url-as-first-parameter.patch ajouté

Un patch en plus systempayv2 car sinon normal_return_url est écrasé dans le backend payzen par les lignes:

    service_url = 'https://secure.payzen.eu/vads-payment/'
    ...
    description = deepcopy(systempayv2.Payment.description)
    description['caption'] = 'PayZen'
    description['parameters'][0]['name'] = service_url

#4

Mis à jour par Frédéric Péters il y a environ 8 ans

Ça mérite un commentaire au niveau du paramètres pour dire "warning: service_url must be kept as first parameter as it's overridden in the payzen backend". (ou alors modifier le backend payzen pour chercher la clé sur base du nom).

#5

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

C'est trop fragile je préférerai qu'on cherche explicitement le bon paramètre ou bien qu'on en fasse pas un paramètre du tout, parce que ça ne devrait jamais changer (mais bon il ne faut pas dire fontaine...).

#6

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Fichier 0001-systempayv2-set-service_url-as-first-parameter.patch supprimé
#7

Mis à jour par Serghei Mihai il y a environ 8 ans

C'est mieux de patcher le backend payzen, dans lequel au passage on écrasait le nom du parametre au lieu de sa valeur

#8

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

Ack.

#9

Mis à jour par Serghei Mihai il y a environ 8 ans

  • Statut changé de En cours à Résolu (à déployer)

Poussé les commit de Fréderic et le mien:

commit eea419cca912c84584f0891c8ff4c55cb274ab28
Author: Serghei Mihai <smihai@entrouvert.com>
Date:   Thu Feb 18 17:25:12 2016 +0100

    payzen: update service_url default value

commit a7dcc7a5e50f599f4bb35afa7c82578f9189bc2f
Author: Frédéric Péters <fpeters@entrouvert.com>
Date:   Thu Feb 18 16:05:58 2016 +0100

    general: give same signature to all Payment object constructors (#10030)

#10

Mis à jour par Frédéric Péters il y a environ 8 ans

  • Statut changé de Résolu (à déployer) à Solution déployée
#11

Mis à jour par Benjamin Dauvergne il y a environ 8 ans

  • Version cible mis à 1.6

Formats disponibles : Atom PDF