Browse Source

[FIX] TypeError: unlink() got multiple values for keyword argument 'context'

pull/22/head
Lorenzo Battistini 10 years ago
parent
commit
d2b123c6c2
  1. 27
      base_location_geonames_import/test/import.yml
  2. 2
      base_location_geonames_import/wizard/geonames_import.py

27
base_location_geonames_import/test/import.yml

@ -25,3 +25,30 @@
('country_id', '=', ref('base.it')) ('country_id', '=', ref('base.it'))
], context=context) ], context=context)
assert len(zip_ids) == 1, "There must be 1 'Isola Del Cantone'" assert len(zip_ids) == 1, "There must be 1 'Isola Del Cantone'"
-
I create the wizard again
-
!record {model: better.zip.geonames.import, id: import_wizard_2, view: better_zip_geonames_import_form}:
country_id: base.it
-
I run the import again
-
!python {model: better.zip.geonames.import}: |
self.run_import(cr, uid, [ref('import_wizard_2')], context=context)
-
I check the data
-
!python {model: res.better.zip}: |
state_obj = self.pool['res.country.state']
state_ids = state_obj.search(cr, uid, [
('code', '=', 'LG'),
('country_id', '=', ref('base.it')),
], context=context)
assert len(state_ids) == 1, "There must be 1 LG"
zip_ids = self.search(cr, uid, [
('name', '=', '16017'),
('city', '=', 'Isola Del Cantone'),
('state_id', '=', state_ids[0]),
('country_id', '=', ref('base.it'))
], context=context)
assert len(zip_ids) == 1, "There must be 1 'Isola Del Cantone'"

2
base_location_geonames_import/wizard/geonames_import.py

@ -106,7 +106,7 @@ class better_zip_geonames_import(models.TransientModel):
% (res_request.status_code, url)) % (res_request.status_code, url))
bzip_ids_to_delete = bzip_obj.search([('country_id', '=', country_id)]) bzip_ids_to_delete = bzip_obj.search([('country_id', '=', country_id)])
if bzip_ids_to_delete: if bzip_ids_to_delete:
bzip_obj.unlink(bzip_ids_to_delete)
bzip_ids_to_delete.unlink()
logger.info( logger.info(
'%d better zip entries deleted for country %s' '%d better zip entries deleted for country %s'
% (len(bzip_ids_to_delete), self.country_id.name)) % (len(bzip_ids_to_delete), self.country_id.name))

Loading…
Cancel
Save