Merge lp:~elbati/enapps-csv-import-tool/fix-get_related_id into lp:~enapps-london-uk/enapps-csv-import-tool/old_trunk

[Lorenzo Battistini]

See http://bazaar.launchpad.net/~elbati/enapps-csv-import-tool/fix-get_related_id/revision/8

[Enapps Ltd OpenERP Partner London UK (enapps-london-uk)]

Nice :)

review: Approve

[Lorenzo Battistini (elbati)]

So, any plan to merge this?
And this https://code.launchpad.net/~domsense/enapps-csv-import-tool/ea_import_fix_1069819_lep/+merge/131044 ?

Thanks

[lp:~elbati/enapps-csv-import-tool/fix-get_related_id]

10. By Lorenzo Battistini on 2012-12-18

[FIX] many2one_rel_type == 'template' too

[Lorenzo Battistini (elbati)]

You can find here
https://code.launchpad.net/~domsense/domsense-agilebg-addons/ea_import
our last updates

Unmerged revisions

10. By Lorenzo Battistini on 2012-12-18

[FIX] many2one_rel_type == 'template' too

9. By Lorenzo Battistini on 2012-09-19

[FIX] also, when csv field is empty, we should not check the 'update_current'

8. By Lorenzo Battistini on 2012-09-19

[FIX] when importing m2o by template,
if csv field is empty, the get_related_id method should not return the list of every record for current model.
This happens when key=[], so
    self.low_level_search(cr, uid, [template.id], key, context=context)
returns the list of all records

=== modified file 'ea_import/ea_import_template.py'
--- ea_import/ea_import_template.py	2012-09-06 08:51:17 +0000
+++ ea_import/ea_import_template.py	2012-12-18 08:14:20 +0000
@@ -82,6 +82,8 @@
                         value = template_line.get_field(input_list, testing=True)
                     if value:
                         key.append((field_name, '=', value))
+            if not key:
+                return [None]
             if template.update_current:
                 key.append(('create_date', '>', context['time_of_start']))
             result = self.low_level_search(cr, uid, [template.id], key, context=context)

=== modified file 'ea_import/ea_import_template_line.py'
--- ea_import/ea_import_template_line.py	2012-09-06 09:17:30 +0000
+++ ea_import/ea_import_template_line.py	2012-12-18 08:14:20 +0000
@@ -222,6 +222,8 @@
                         result = template_line.related_template_id.get_related_id(input_list)
                         if testing:
                             test_many2one_result(result, template_line, target_string)
+                        if not result:
+                            return None
                         return result[0]
             else:
                 continue