diff --git a/addons/payment_acquirer_adyen/models/adyen.py b/addons/payment_acquirer_adyen/models/adyen.py index 4ba04b1bfe4..d76419c30cb 100644 --- a/addons/payment_acquirer_adyen/models/adyen.py +++ b/addons/payment_acquirer_adyen/models/adyen.py @@ -21,26 +21,24 @@ _logger = logging.getLogger(__name__) class AcquirerAdyen(osv.Model): _inherit = 'payment.acquirer' - def _get_adyen_urls(self, cr, uid, ids, name, args, context=None): + def _get_adyen_urls(self, cr, uid, env, context=None): """ Adyen URLs - yhpp: hosted payment page: pay.shtml for single, select.shtml for multiple """ - res = {} - for acquirer in self.browse(cr, uid, ids, context=context): - qualif = acquirer.env - res[acquirer.id] = { - 'adyen_form_url': 'https://%s.adyen.com/hpp/pay.shtml' % qualif, + if env == 'prod': + return { + 'adyen_form_url': 'https://prod.adyen.com/hpp/pay.shtml', + } + else: + return { + 'adyen_form_url': 'https://test.adyen.com/hpp/pay.shtml', } - return res _columns = { 'adyen_merchant_account': fields.char('Merchant Account', required_if_provider='adyen'), 'adyen_skin_code': fields.char('Skin Code', required_if_provider='adyen'), 'adyen_skin_hmac_key': fields.char('Skin HMAC Key', required_if_provider='adyen'), - 'adyen_form_url': fields.function( - _get_adyen_urls, multi='_get_adyen_urls', - type='char', string='Transaction URL', required_if_provider='adyen'), } def _adyen_generate_merchant_sig(self, acquirer, inout, values): @@ -99,7 +97,7 @@ class AcquirerAdyen(osv.Model): def adyen_get_form_action_url(self, cr, uid, id, context=None): acquirer = self.browse(cr, uid, id, context=context) - return acquirer.adyen_form_url + return self._get_adyen_urls(cr, uid, acquirer.env, context=context)['adyen_form_url'] class TxAdyen(osv.Model): diff --git a/addons/payment_acquirer_adyen/static/description/icon.png b/addons/payment_acquirer_adyen/static/description/icon.png index ccbc55b8c5d..e1889808855 100644 Binary files a/addons/payment_acquirer_adyen/static/description/icon.png and b/addons/payment_acquirer_adyen/static/description/icon.png differ diff --git a/addons/payment_acquirer_adyen/static/src/img/adyen_icon.png b/addons/payment_acquirer_adyen/static/src/img/adyen_icon.png new file mode 100644 index 00000000000..90d492c3fe9 Binary files /dev/null and b/addons/payment_acquirer_adyen/static/src/img/adyen_icon.png differ diff --git a/addons/payment_acquirer_adyen/static/src/img/logo.jpg b/addons/payment_acquirer_adyen/static/src/img/logo.jpg deleted file mode 100644 index 63a58006324..00000000000 Binary files a/addons/payment_acquirer_adyen/static/src/img/logo.jpg and /dev/null differ diff --git a/addons/payment_acquirer_adyen/static/src/img/logo.png b/addons/payment_acquirer_adyen/static/src/img/logo.png new file mode 100644 index 00000000000..e1889808855 Binary files /dev/null and b/addons/payment_acquirer_adyen/static/src/img/logo.png differ diff --git a/addons/payment_acquirer_adyen/views/adyen.xml b/addons/payment_acquirer_adyen/views/adyen.xml index aeaa5260b11..55a5fd4b5b0 100644 --- a/addons/payment_acquirer_adyen/views/adyen.xml +++ b/addons/payment_acquirer_adyen/views/adyen.xml @@ -3,7 +3,7 @@ diff --git a/addons/payment_acquirer_adyen/views/payment_acquirer.xml b/addons/payment_acquirer_adyen/views/payment_acquirer.xml index 9b75828f315..82cc70c9972 100644 --- a/addons/payment_acquirer_adyen/views/payment_acquirer.xml +++ b/addons/payment_acquirer_adyen/views/payment_acquirer.xml @@ -7,10 +7,12 @@ payment.acquirer - - + + + + + diff --git a/addons/payment_acquirer_ogone/models/ogone.py b/addons/payment_acquirer_ogone/models/ogone.py index 7decd389e95..eed9c2cc347 100644 --- a/addons/payment_acquirer_ogone/models/ogone.py +++ b/addons/payment_acquirer_ogone/models/ogone.py @@ -25,23 +25,19 @@ _logger = logging.getLogger(__name__) class PaymentAcquirerOgone(osv.Model): _inherit = 'payment.acquirer' - def _get_ogone_urls(self, cr, uid, ids, name, args, context=None): + def _get_ogone_urls(self, cr, uid, env, context=None): """ Ogone URLS: - standard order: POST address for form-based @TDETODO: complete me """ - res = {} - for acquirer in self.browse(cr, uid, ids, context=context): - qualif = acquirer.env - res[acquirer.id] = { - 'ogone_standard_order_url': 'https://secure.ogone.com/ncol/%s/orderstandard.asp' % qualif, - 'ogone_direct_order_url': 'https://secure.ogone.com/ncol/%s/orderdirect.asp' % qualif, - 'ogone_direct_query_url': 'https://secure.ogone.com/ncol/%s/querydirect.asp' % qualif, - 'ogone_afu_agree_url': 'https://secure.ogone.com/ncol/%s/AFU_agree.asp' % qualif, - } - return res + return { + 'ogone_standard_order_url': 'https://secure.ogone.com/ncol/%s/orderstandard.asp' % env, + 'ogone_direct_order_url': 'https://secure.ogone.com/ncol/%s/orderdirect.asp' % env, + 'ogone_direct_query_url': 'https://secure.ogone.com/ncol/%s/querydirect.asp' % env, + 'ogone_afu_agree_url': 'https://secure.ogone.com/ncol/%s/AFU_agree.asp' % env, + } _columns = { 'ogone_pspid': fields.char( @@ -54,19 +50,6 @@ class PaymentAcquirerOgone(osv.Model): 'SHA Key IN', size=32, required_if_provider='ogone'), 'ogone_shakey_out': fields.char( 'SHA Key OUT', size=32, required_if_provider='ogone'), - # store ogone contact URLs -> not necessary IMHO - 'ogone_standard_order_url': fields.function( - _get_ogone_urls, type='char', multi='_get_ogone_urls', - string='Stanrd Order URL (form)'), - 'ogone_direct_order_url': fields.function( - _get_ogone_urls, type='char', multi='_get_ogone_urls', - string='Direct Order URL (2)'), - 'ogone_direct_query_url': fields.function( - _get_ogone_urls, type='char', multi='_get_ogone_urls', - string='Direct Query URL'), - 'ogone_afu_agree_url': fields.function( - _get_ogone_urls, type='char', multi='_get_ogone_urls', - string='AFU Agree URL'), } def _ogone_generate_shasign(self, acquirer, inout, values): @@ -128,7 +111,7 @@ class PaymentAcquirerOgone(osv.Model): def ogone_get_form_action_url(self, cr, uid, id, context=None): acquirer = self.browse(cr, uid, id, context=context) - return acquirer.ogone_standard_order_url + return self._get_ogone_urls(cr, uid, acquirer.env, context=context)['ogone_standard_order_url'] class PaymentTxOgone(osv.Model): @@ -140,10 +123,8 @@ class PaymentTxOgone(osv.Model): _ogone_cancel_tx_status = [1] _columns = { - 'ogone_3ds': fields.dummy('3ds Activated'), + 'ogone_3ds': fields.boolean('3DS Activated'), 'ogone_3ds_html': fields.html('3DS HTML'), - 'ogone_feedback_model': fields.char(), - 'ogone_feedback_eval': fields.char(), 'ogone_complus': fields.char('Complus'), 'ogone_payid': fields.char('PayID', help='Payment ID, generated by Ogone') } @@ -243,6 +224,7 @@ class PaymentTxOgone(osv.Model): 'PROCESS_MODE': 'CHECKANDPROCESS', } + # TODO: fix URL computation request = urllib2.Request(tx.acquirer_id.ogone_afu_agree_url, urlencode(tx_data)) result = urllib2.urlopen(request).read() diff --git a/addons/payment_acquirer_ogone/static/src/img/ogone_icon.png b/addons/payment_acquirer_ogone/static/src/img/ogone_icon.png new file mode 100644 index 00000000000..f8274724529 Binary files /dev/null and b/addons/payment_acquirer_ogone/static/src/img/ogone_icon.png differ diff --git a/addons/payment_acquirer_ogone/static/src/img/ogone_logo.png b/addons/payment_acquirer_ogone/static/src/img/ogone_logo.png deleted file mode 100644 index 89e0ef6816f..00000000000 Binary files a/addons/payment_acquirer_ogone/static/src/img/ogone_logo.png and /dev/null differ diff --git a/addons/payment_acquirer_ogone/static/src/img/ogone_logo_plain.gif b/addons/payment_acquirer_ogone/static/src/img/ogone_logo_plain.gif deleted file mode 100644 index e7833ec3b6d..00000000000 Binary files a/addons/payment_acquirer_ogone/static/src/img/ogone_logo_plain.gif and /dev/null differ diff --git a/addons/payment_acquirer_ogone/views/ogone.xml b/addons/payment_acquirer_ogone/views/ogone.xml index f71f92197f0..6cb06507ff1 100644 --- a/addons/payment_acquirer_ogone/views/ogone.xml +++ b/addons/payment_acquirer_ogone/views/ogone.xml @@ -3,7 +3,7 @@ diff --git a/addons/payment_acquirer_ogone/views/payment_acquirer.xml b/addons/payment_acquirer_ogone/views/payment_acquirer.xml index 8ec958a5351..8a64f801dce 100644 --- a/addons/payment_acquirer_ogone/views/payment_acquirer.xml +++ b/addons/payment_acquirer_ogone/views/payment_acquirer.xml @@ -7,9 +7,8 @@ payment.acquirer - - + + @@ -29,8 +28,10 @@ - + + + + diff --git a/addons/payment_acquirer_paypal/data/paypal.xml b/addons/payment_acquirer_paypal/data/paypal.xml index 117a32dd39f..15a5c4dfcf7 100644 --- a/addons/payment_acquirer_paypal/data/paypal.xml +++ b/addons/payment_acquirer_paypal/data/paypal.xml @@ -6,6 +6,8 @@ paypal test + You will be redirected to the Paypal website after cliking on the payment button.

]]>
https://www.sandbox.paypal.com/cgi-bin/webscr dummy dummy diff --git a/addons/payment_acquirer_paypal/models/paypal.py b/addons/payment_acquirer_paypal/models/paypal.py index e6b886a941b..371b4423eda 100644 --- a/addons/payment_acquirer_paypal/models/paypal.py +++ b/addons/payment_acquirer_paypal/models/paypal.py @@ -21,10 +21,22 @@ _logger = logging.getLogger(__name__) class AcquirerPaypal(osv.Model): _inherit = 'payment.acquirer' + def _get_paypal_urls(self, cr, uid, env, context=None): + """ Paypal URLS """ + if env == 'prod': + return { + 'paypal_form_url': 'https://www.sandbox.paypal.com/cgi-bin/webscr', + 'paypal_rest_url': 'https://api.sandbox.paypal.com/v1/oauth2/token', + } + else: + return { + 'paypal_form_url': 'https://www.sandbox.paypal.com/cgi-bin/webscr', + 'paypal_rest_url': 'https://api.sandbox.paypal.com/v1/oauth2/token', + } + _columns = { 'paypal_email_id': fields.char('Email ID', required_if_provider='paypal'), 'paypal_username': fields.char('Username', required_if_provider='paypal'), - 'paypal_tx_url': fields.char('Transaction URL', required_if_provider='paypal'), 'paypal_use_ipn': fields.boolean('Use IPN'), # Server 2 server 'paypal_api_enabled': fields.boolean('Use Rest API'), @@ -35,7 +47,6 @@ class AcquirerPaypal(osv.Model): } _defaults = { - 'paypal_tx_url': 'https://www.sandbox.paypal.com/cgi-bin/webscr', 'paypal_use_ipn': True, 'fees_active': False, 'fees_dom_fixed': 0.35, @@ -95,7 +106,7 @@ class AcquirerPaypal(osv.Model): def paypal_get_form_action_url(self, cr, uid, id, context=None): acquirer = self.browse(cr, uid, id, context=context) - return acquirer.paypal_tx_url + return self._get_paypal_urls(cr, uid, acquirer.env, context=context)['paypal_form_url'] def _paypal_s2s_get_access_token(self, cr, uid, ids, context=None): """ diff --git a/addons/payment_acquirer_paypal/static/src/img/paypal_icon.png b/addons/payment_acquirer_paypal/static/src/img/paypal_icon.png new file mode 100644 index 00000000000..9d0534b3fa8 Binary files /dev/null and b/addons/payment_acquirer_paypal/static/src/img/paypal_icon.png differ diff --git a/addons/payment_acquirer_paypal/views/payment_acquirer.xml b/addons/payment_acquirer_paypal/views/payment_acquirer.xml index 6734b2fdcc5..1ccd18e29b9 100644 --- a/addons/payment_acquirer_paypal/views/payment_acquirer.xml +++ b/addons/payment_acquirer_paypal/views/payment_acquirer.xml @@ -10,14 +10,16 @@ - - - - - - + + + + + + + + diff --git a/addons/payment_acquirer_paypal/views/paypal.xml b/addons/payment_acquirer_paypal/views/paypal.xml index a6f401f6c3c..186d45fc452 100644 --- a/addons/payment_acquirer_paypal/views/paypal.xml +++ b/addons/payment_acquirer_paypal/views/paypal.xml @@ -3,16 +3,14 @@ diff --git a/addons/payment_acquirer_transfer/static/src/img/transfer_icon.png b/addons/payment_acquirer_transfer/static/src/img/transfer_icon.png new file mode 100644 index 00000000000..63b10e383ec Binary files /dev/null and b/addons/payment_acquirer_transfer/static/src/img/transfer_icon.png differ diff --git a/addons/payment_acquirer_transfer/views/transfer.xml b/addons/payment_acquirer_transfer/views/transfer.xml index af5c6fcef0f..75912a30f42 100644 --- a/addons/payment_acquirer_transfer/views/transfer.xml +++ b/addons/payment_acquirer_transfer/views/transfer.xml @@ -3,16 +3,19 @@