diff --git a/addons/payment_acquirer_paypal/controllers/main.py b/addons/payment_acquirer_paypal/controllers/main.py index 7b9d654b30b..5603f9cf88f 100644 --- a/addons/payment_acquirer_paypal/controllers/main.py +++ b/addons/payment_acquirer_paypal/controllers/main.py @@ -35,9 +35,11 @@ class PaypalController(http.Controller): _notify_url = '/payment/paypal/ipn/' _return_url = '/payment/paypal/dpn/' _cancel_url = '/payment/paypal/cancel/' - # _ipn_url2 = '/payment/paypal//ipn/' - @website.route('/payment/paypal//ipn/', type='http', auth='admin') + @website.route([ + '/payment/paypal/test/ipn/' + '/payment/paypal/ipn/', + ], type='http', auth='admin') def paypal_ipn(self, **post): print 'Entering paypal_ipn with post', post # step 1: return an empty HTTP 200 response -> will be done at the end by returning '' @@ -65,10 +67,22 @@ class PaypalController(http.Controller): @website.route([ '/payment/paypal/test/dpn', + '/payment/paypal/dpn', ], type='http', auth="public") - def paypal_test_success(self, **post): + def paypal_dpn(self, **post): """ TODO """ cr, uid, context = request.cr, request.uid, request.context - print post + print 'Entering paypal_dpn with post', post + return '' + + @website.route([ + '/payment/paypal/test/cancel', + '/payment/paypal/cancel', + ], type='http', auth="public") + def paypal_cancel(self, **post): + """ TODO + """ + cr, uid, context = request.cr, request.uid, request.context + print 'Entering paypal_cancel with post', post return '' diff --git a/addons/payment_acquirer_paypal/models/paypal.py b/addons/payment_acquirer_paypal/models/paypal.py index 42113994e22..c54eb698a19 100644 --- a/addons/payment_acquirer_paypal/models/paypal.py +++ b/addons/payment_acquirer_paypal/models/paypal.py @@ -64,16 +64,16 @@ class AcquirerPaypal(osv.Model): 'item_number': reference, 'amount': amount, 'currency_code': currency and currency.name or 'EUR', - 'address1': partner and ' '.join((partner.street, partner.street2)).strip() or ' '.join((partner_values.get('street', ''), partner_values.get('street2', ''))).strip(), + 'address1': partner and ' '.join((partner.street or '', partner.street2 or '')).strip() or ' '.join((partner_values.get('street', ''), partner_values.get('street2', ''))).strip(), 'city': partner and partner.city or partner_values.get('city', ''), 'country': partner and partner.country_id and partner.country_id.name or partner_values.get('country_name', ''), 'email': partner and partner.email or partner_values.get('email', ''), 'zip': partner and partner.zip or partner_values.get('zip', ''), 'first_name': partner and partner.name or partner_values.get('name', '').split()[-1:], 'last_name': partner and partner.name or partner_values.get('name', '').split()[:-1], - 'return': '%s/%s' % (base_url, PaypalController._return_url), - 'notify_url': '%s/%s' % (base_url, PaypalController._notify_url), - 'cancel_return': '%s/%s' % (base_url, PaypalController._cancel_url), + 'return': '%s' % urlparse.urljoin(base_url, PaypalController._return_url), + 'notify_url': '%s' % urlparse.urljoin(base_url, PaypalController._notify_url), + 'cancel_return': '%s' % urlparse.urljoin(base_url, PaypalController._cancel_url), } if tx_custom_values: tx_values.update(tx_custom_values) @@ -235,7 +235,3 @@ class TxPaypal(osv.Model): response.get("reason_code"))) ) return response - - def _transaction_feedback_paypal(self, **values): - print values - return True diff --git a/addons/payment_acquirer_paypal/static/src/img/paypal_logo.png b/addons/payment_acquirer_paypal/static/src/img/paypal_logo.png new file mode 100644 index 00000000000..4803ff5d1ef Binary files /dev/null and b/addons/payment_acquirer_paypal/static/src/img/paypal_logo.png differ diff --git a/addons/payment_acquirer_paypal/views/paypal.xml b/addons/payment_acquirer_paypal/views/paypal.xml index 105a8595057..0277c8d1d98 100644 --- a/addons/payment_acquirer_paypal/views/paypal.xml +++ b/addons/payment_acquirer_paypal/views/paypal.xml @@ -4,9 +4,9 @@