Merge pull request 'agregada direccion de envio y facturación al enviar las compras a tryton #14' (#15) from add_shipment_address_to_tryton_sale_#14 into main
Reviewed-on: #15
This commit is contained in:
commit
477405a094
@ -397,7 +397,7 @@ class CustomersFromTrytonView(APIView):
|
||||
)
|
||||
|
||||
def __get_party_datails(self, party_ids, tryton_client, context):
|
||||
tryton_fields = ['id', 'name']
|
||||
tryton_fields = ['id', 'name', 'addresses']
|
||||
method = 'model.party.party.read'
|
||||
params = (party_ids, tryton_fields, context)
|
||||
response = tryton_client.call(method, params)
|
||||
@ -406,15 +406,22 @@ class CustomersFromTrytonView(APIView):
|
||||
def __need_update(self, customer, tryton_party):
|
||||
if not customer.name == tryton_party.get('name'):
|
||||
return True
|
||||
if tryton_party.get('addresses') and tryton_party.get('addresses')[0]:
|
||||
if not customer.address_external_id == str(tryton_party.get('addresses')[0]):
|
||||
return True
|
||||
|
||||
def __create_customer(self, tryton_party):
|
||||
customer = Customer()
|
||||
customer.name = tryton_party.get('name')
|
||||
customer.external_id = tryton_party.get('id')
|
||||
if tryton_party.get('addresses') and tryton_party.get('addresses')[0]:
|
||||
customer.address_external_id = tryton_party.get('addresses')[0]
|
||||
customer.save()
|
||||
return customer
|
||||
|
||||
def __update_customer(self, customer, tryton_party):
|
||||
customer.name = tryton_party.get('name')
|
||||
customer.external_id = tryton_party.get('id')
|
||||
if tryton_party.get('addresses') and tryton_party.get('addresses')[0]:
|
||||
customer.address_external_id = tryton_party.get('addresses')[0]
|
||||
customer.save()
|
||||
|
@ -30,13 +30,13 @@ class TestCustomersFromTryton(TestCase):
|
||||
return [5, 6, 7, 8]
|
||||
|
||||
party_read = 'model.party.party.read'
|
||||
read_args = ([5, 6, 7, 8], ['id', 'name'], {'company': 1})
|
||||
read_args = ([5, 6, 7, 8], ['id', 'name', 'addresses'], {'company': 1})
|
||||
if (args == (party_read, read_args)):
|
||||
return [
|
||||
{'id': 5, 'name': 'Carlos'},
|
||||
{'id': 6, 'name': 'Cristian'},
|
||||
{'id': 7, 'name': 'Ana'},
|
||||
{'id': 8, 'name': 'José'},
|
||||
{'id': 5, 'name': 'Carlos', 'addresses': [303]},
|
||||
{'id': 6, 'name': 'Cristian', 'addresses': []},
|
||||
{'id': 7, 'name': 'Ana', 'addresses': [302]},
|
||||
{'id': 8, 'name': 'José', 'addresses': []},
|
||||
]
|
||||
|
||||
raise Exception(f"Sorry, args non expected on this test: {args}")
|
||||
@ -60,7 +60,9 @@ class TestCustomersFromTryton(TestCase):
|
||||
created_customer = Customer.objects.get(id=3)
|
||||
self.assertEqual(created_customer.external_id, str(7))
|
||||
self.assertEqual(created_customer.name, 'Ana')
|
||||
self.assertEqual(created_customer.address_external_id, str(302))
|
||||
|
||||
updated_customer = Customer.objects.get(id=1)
|
||||
self.assertEqual(updated_customer.external_id, str(5))
|
||||
self.assertEqual(updated_customer.name, 'Carlos')
|
||||
self.assertIn(updated_customer.address_external_id, str(303))
|
||||
|
@ -19,7 +19,7 @@ class TestExportarVentasParaTryton(TestCase):
|
||||
self.customer = Customer.objects.create(
|
||||
name='Camilo',
|
||||
external_id=1,
|
||||
address_external_id=1
|
||||
address_external_id=307,
|
||||
)
|
||||
self.sale = Sale.objects.create(
|
||||
customer=self.customer,
|
||||
@ -100,4 +100,4 @@ class TestExportarVentasParaTryton(TestCase):
|
||||
self.assertEqual(updated_sale.external_id, external_id)
|
||||
mock_connect.assert_called_once()
|
||||
mock_call.assert_called_once()
|
||||
mock_call.assert_called_with('model.sale.sale.create', [[{'company': 1, 'shipment_address': '1', 'invoice_address': '1', 'currency': 31, 'description': '', 'party': '1', 'reference': 'don_confiao 1', 'sale_date': {'__class__': 'date', 'year': 2024, 'month': 9, 'day': 2}, 'lines': [['create', [{'product': '1', 'quantity': {'__class__': 'Decimal', 'decimal': '2.00'}, 'type': 'line', 'unit': '1', 'unit_price': {'__class__': 'Decimal', 'decimal': '3000.00'}}, {'product': '1', 'quantity': {'__class__': 'Decimal', 'decimal': '3.00'}, 'type': 'line', 'unit': '1', 'unit_price': {'__class__': 'Decimal', 'decimal': '5000.00'}}]]]}], {'company': 1, 'shops': [1]}])
|
||||
mock_call.assert_called_with('model.sale.sale.create', [[{'company': 1, 'shipment_address': '307', 'invoice_address': '307', 'currency': 31, 'description': '', 'party': '1', 'reference': 'don_confiao 1', 'sale_date': {'__class__': 'date', 'year': 2024, 'month': 9, 'day': 2}, 'lines': [['create', [{'product': '1', 'quantity': {'__class__': 'Decimal', 'decimal': '2.00'}, 'type': 'line', 'unit': '1', 'unit_price': {'__class__': 'Decimal', 'decimal': '3000.00'}}, {'product': '1', 'quantity': {'__class__': 'Decimal', 'decimal': '3.00'}, 'type': 'line', 'unit': '1', 'unit_price': {'__class__': 'Decimal', 'decimal': '5000.00'}}]]]}], {'company': 1, 'shops': [1]}])
|
||||
|
Loading…
Reference in New Issue
Block a user