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

[Lorenzo Battistini]

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

11. By Lorenzo Battistini on 2012-12-17

[FIX] removing fix contained by this https://code.launchpad.net/~elbati/enapps-csv-import-tool/fix-get_related_id/+merge/125226 yet

[Lorenzo Battistini (elbati)]

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

Unmerged revisions

11. By Lorenzo Battistini on 2012-12-17

[FIX] removing fix contained by this https://code.launchpad.net/~elbati/enapps-csv-import-tool/fix-get_related_id/+merge/125226 yet

10. By Lorenzo Battistini on 2012-12-17

[FIX] allowing regexp replace by empty string

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_line_regexp_field.py'
--- ea_import/ea_import_template_line_regexp_field.py	2012-05-28 12:49:58 +0000
+++ ea_import/ea_import_template_line_regexp_field.py	2012-12-17 17:40:29 +0000
@@ -49,14 +49,14 @@
                                 =================
                                 result - "ENAPPS LTD is an awesome and cool British company"
                               '''),
-        'replace_by': fields.char('Replace by', size=512, required=True, ),
+        'replace_by': fields.char('Replace by', size=512, required=False, ),
         }
 
     def replace_string(self, cr, uid, ids, target_string, context={}):
         for regexp_field in self.browse(cr, uid, ids, context=context):
             search_result = re.search(regexp_field.regexp, target_string)
             if search_result:
-                return re.sub(regexp_field.regexp, regexp_field.replace_by, target_string)
+                return re.sub(regexp_field.regexp, regexp_field.replace_by or '', target_string)
         return target_string