From e4443ee3cedf1ad36b926ee256bd147f62f5dafc Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Thibault=20Delavall=C3=A9e?= <tde@openerp.com>
Date: Tue, 19 Nov 2013 19:04:00 +0100
Subject: [PATCH] [IMP] payment_acquirer_transfer: WIP about makign this module
 actually work.

- improved form: using acquirer.message to display custom message, like the account number
- added controllers; form will directly send to a controller that will validate the
sale order, then back to the payment checkout process.

bzr revid: tde@openerp.com-20131119180400-m6okb7empnpnre2p
---
 addons/payment_acquirer_transfer/__init__.py  |  1 +
 .../controllers/__init__.py                   |  3 +++
 .../controllers/main.py                       | 22 +++++++++++++++++++
 .../data/transfer.xml                         |  2 +-
 .../models/payment_acquirer.py                |  3 +++
 .../views/transfer.xml                        |  9 +++++++-
 6 files changed, 38 insertions(+), 2 deletions(-)
 create mode 100644 addons/payment_acquirer_transfer/controllers/__init__.py
 create mode 100644 addons/payment_acquirer_transfer/controllers/main.py

diff --git a/addons/payment_acquirer_transfer/__init__.py b/addons/payment_acquirer_transfer/__init__.py
index d95d47860af..dde6f25c728 100644
--- a/addons/payment_acquirer_transfer/__init__.py
+++ b/addons/payment_acquirer_transfer/__init__.py
@@ -20,3 +20,4 @@
 ##############################################################################
 
 import models
+import controllers
diff --git a/addons/payment_acquirer_transfer/controllers/__init__.py b/addons/payment_acquirer_transfer/controllers/__init__.py
new file mode 100644
index 00000000000..bbd183e955b
--- /dev/null
+++ b/addons/payment_acquirer_transfer/controllers/__init__.py
@@ -0,0 +1,3 @@
+# -*- coding: utf-8 -*-
+
+import main
diff --git a/addons/payment_acquirer_transfer/controllers/main.py b/addons/payment_acquirer_transfer/controllers/main.py
new file mode 100644
index 00000000000..711dcec8de2
--- /dev/null
+++ b/addons/payment_acquirer_transfer/controllers/main.py
@@ -0,0 +1,22 @@
+# -*- coding: utf-8 -*-
+import logging
+import pprint
+
+from openerp.addons.web import http
+from openerp.addons.web.http import request
+from openerp.addons.website.models import website
+
+_logger = logging.getLogger(__name__)
+
+
+class OgoneController(http.Controller):
+    _accept_url = '/payment/transfer/feedback'
+
+    @website.route([
+        '/payment/transfer/feedback',
+    ], type='http', auth='admin')
+    def transfer_form_feedback(self, **post):
+        _logger.info('Transfer: entering form_feedback with post data %s', pprint.pformat(post))  # debug
+        cr, uid, context = request.cr, request.uid, request.context
+        request.registry['payment.transaction'].transfer_form_feedback(cr, uid, post, context)
+        return request.redirect(post.pop('return_url', '/'))
diff --git a/addons/payment_acquirer_transfer/data/transfer.xml b/addons/payment_acquirer_transfer/data/transfer.xml
index 441d39f1aab..ce6e56be57d 100644
--- a/addons/payment_acquirer_transfer/data/transfer.xml
+++ b/addons/payment_acquirer_transfer/data/transfer.xml
@@ -6,7 +6,7 @@
             <field name="name">transfer</field>
             <field name="view_template_id" ref="transfer_acquirer_button"/>
             <field name="env">test</field>
-            <field name="portal_published" eval="False"/>
+            <field name="portal_published" eval="True"/>
             <field name="message"><![CDATA[
                 <p>Please use the account 001-002-003 to make the payment.</p>]]>
             </field>
diff --git a/addons/payment_acquirer_transfer/models/payment_acquirer.py b/addons/payment_acquirer_transfer/models/payment_acquirer.py
index 283868aa8e9..cbcded835b5 100644
--- a/addons/payment_acquirer_transfer/models/payment_acquirer.py
+++ b/addons/payment_acquirer_transfer/models/payment_acquirer.py
@@ -13,3 +13,6 @@ class TransferPaymentAcquirer(osv.Model):
 
 class TransferPaymentTransaction(osv.Model):
     _inherit = 'payment.transaction'
+
+    def transfer_form_feedback(self, cr, uid, data, context=None):
+        return True
diff --git a/addons/payment_acquirer_transfer/views/transfer.xml b/addons/payment_acquirer_transfer/views/transfer.xml
index 8024bae85e8..31cf85ee5a8 100644
--- a/addons/payment_acquirer_transfer/views/transfer.xml
+++ b/addons/payment_acquirer_transfer/views/transfer.xml
@@ -5,7 +5,14 @@
         <!-- templates -->
         <template id="transfer_acquirer_button">
             <div>
-                <t t-field="acquirer.message"/>
+                <div t-field="acquirer.message"/>
+                <form t-if="acquirer" action="/payment/transfer/feedback" method="post" target="_self">
+                    <t t-if="tx_values.get('return_url')">
+                        <input type='hidden' name='return_url' t-att-value='tx_values["return_url"]'/>
+                    </t>
+                    <input type="image" name="submit" id="payment_submit"
+                        src="/payment_acquirer_ogone/static/src/img/ogone_logo_plain.gif"/>
+            </form>
             </div>
         </template>