Project

General

Profile

Bug #10030

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

Added by Frédéric Péters about 8 years ago. Updated almost 8 years ago.

Status:
Fermé
Priority:
Normal
Assignee:
-
Target version:
Start date:
18 February 2016
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:

Files


Related issues

Related to Combo - Bug #10029: erreur lors d'une tentative de paiementFermé18 February 2016

Actions

Associated revisions

Revision a7dcc7a5 (diff)
Added by Frédéric Péters about 8 years ago

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

History

#1

Updated by Frédéric Péters about 8 years ago

#2

Updated by Frédéric Péters about 8 years ago

  • Related to Bug #10029: erreur lors d'une tentative de paiement added
#3

Updated by Serghei Mihai about 8 years ago

  • File 0001-systempayv2-set-service_url-as-first-parameter.patch added

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

Updated by Frédéric Péters about 8 years ago

Ç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

Updated by Benjamin Dauvergne about 8 years ago

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

Updated by Serghei Mihai about 8 years ago

  • File deleted (0001-systempayv2-set-service_url-as-first-parameter.patch)
#7

Updated by Serghei Mihai about 8 years ago

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

#8

Updated by Benjamin Dauvergne about 8 years ago

Ack.

#9

Updated by Serghei Mihai about 8 years ago

  • Status changed from En cours to 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

Updated by Frédéric Péters almost 8 years ago

  • Status changed from Résolu (à déployer) to Solution déployée
#11

Updated by Benjamin Dauvergne almost 8 years ago

  • Target version set to 1.6

Also available in: Atom PDF