Skip to content

Commit

Permalink
[FIX] Format code black and add newline at the end of readme files
Browse files Browse the repository at this point in the history
  • Loading branch information
kmmelcher authored and marcelsavegnago committed Jun 20, 2022
1 parent e63ecc9 commit 68eb9e3
Show file tree
Hide file tree
Showing 6 changed files with 22 additions and 21 deletions.
11 changes: 8 additions & 3 deletions payment_pagseguro/controllers/main.py
Original file line number Diff line number Diff line change
Expand Up @@ -18,11 +18,16 @@ class PagseguroController(http.Controller):
def pagseguro_s2s_create_json_3ds(self, verify_validity=False, **kwargs):
if not kwargs.get("partner_id"):
kwargs = dict(kwargs, partner_id=request.env.user.partner_id.id)
token = request.env["payment.acquirer"].browse(
int(kwargs.get("acquirer_id"))).s2s_process(kwargs)
token = (
request.env["payment.acquirer"]
.browse(int(kwargs.get("acquirer_id")))
.s2s_process(kwargs)
)

if not token:
res = {"result": False, }
res = {
"result": False,
}
else:
res = {
"result": True,
Expand Down
1 change: 0 additions & 1 deletion payment_pagseguro/data/payment_acquirer_data.xml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,6 @@
<field name="save_token">none</field>
<field name="payment_flow">s2s</field>
<field name="capture_manually">True</field>

<field
name="pre_msg"
><![CDATA[
Expand Down
18 changes: 9 additions & 9 deletions payment_pagseguro/models/payment_transaction.py
Original file line number Diff line number Diff line change
Expand Up @@ -105,17 +105,17 @@ def pagseguro_s2s_capture_transaction(self):
self.write(
{
"date": fields.datetime.now(),
"acquirer_reference": res['id'],
"state_message": res['payment_response']['message'],
"acquirer_reference": res["id"],
"state_message": res["payment_response"]["message"],
}
)
self._set_transaction_done()
self.execute_callback()
else:
self.sudo().write(
{
"state_message": res['error_messages'][0]['message'],
"acquirer_reference": res['error_messages'][0]['code'],
"state_message": res["error_messages"][0]["message"],
"acquirer_reference": res["error_messages"][0]["code"],
"date": fields.datetime.now(),
}
)
Expand All @@ -136,7 +136,7 @@ def pagseguro_s2s_void_transaction(self):
"charge_id": self.id,
"amount": {
"value": int(self.amount * 100),
}
},
}

r = requests.post(
Expand All @@ -158,16 +158,16 @@ def pagseguro_s2s_void_transaction(self):
self.write(
{
"date": fields.datetime.now(),
"acquirer_reference": res['id'],
"state_message": res['payment_response']['message'],
"acquirer_reference": res["id"],
"state_message": res["payment_response"]["message"],
}
)
self._set_transaction_cancel()
else:
self.sudo().write(
{
"state_message": res['error_messages'][0]['message'],
"acquirer_reference": res['error_messages'][0]['code'],
"state_message": res["error_messages"][0]["message"],
"acquirer_reference": res["error_messages"][0]["code"],
"date": fields.datetime.now(),
}
)
Expand Down
2 changes: 1 addition & 1 deletion payment_pagseguro/readme/DESCRIPTION.rst
Original file line number Diff line number Diff line change
Expand Up @@ -8,4 +8,4 @@ checkout transparent approach.
You can learn more about that on this `youtube video <https://www.youtube.com/watch?v=3DxKEKoSPkA>`_.

If you wish to use Pagseguro on production environment you need to get an approval
from pagseguro, more on that `here <https://dev.pagseguro.uol.com.br/docs/comecando-processo-de-integracao>`_.
from pagseguro, more on that `here <https://dev.pagseguro.uol.com.br/docs/comecando-processo-de-integracao>`_.
2 changes: 1 addition & 1 deletion payment_pagseguro/readme/INSTALL.rst
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
This module depends on:

* payment
* web_tour
* web_tour
9 changes: 3 additions & 6 deletions payment_pagseguro/tests/test_pagseguro.py
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,6 @@
# License AGPL-3.0 or later (http://www.gnu.org/licenses/agpl).

import time
import logging

import odoo

Expand All @@ -18,12 +17,10 @@ def test_buy_pagseguro(self):
timeout=20000,
)

tx = self.env['payment.transaction'].search([], limit=1, order='id desc')
tx = self.env["payment.transaction"].search([], limit=1, order="id desc")
tx.pagseguro_s2s_capture_transaction()
self.assertEqual(tx.state, 'done',
'transaction state should be authorized')
self.assertEqual(tx.state, "done", "transaction state should be authorized")

time.sleep(3)
tx.pagseguro_s2s_void_transaction()
self.assertEqual(tx.state, 'done',
'transaction state should be done')
self.assertEqual(tx.state, "done", "transaction state should be done")

0 comments on commit 68eb9e3

Please sign in to comment.