Merge lp:~domsense/domsense-agilebg-addons/ea_import_upper_selection_name into lp:~domsense/domsense-agilebg-addons/ea_import

[Alex Comba - Agile BG]

[IMP] get_selection_from_name adding uppercase string comparison

Added upper string comparison before to get selection.

[Lorenzo Battistini (elbati)]

I think the right way to import selection fields is to use the key string instead of the value string

review: Disapprove

[lp:~domsense/domsense-agilebg-addons/ea_import_upper_selection_name]

28. By Alex Comba - Agile BG on 2013-03-21

[MERGE]

Unmerged revisions

28. By Alex Comba - Agile BG on 2013-03-21

[MERGE]

27. By Alex Comba - Agile BG on 2013-02-18

[IMP] get_selection_from_name adding uppercase string comparison

=== modified file 'ea_import/ea_import_template_line.py'
--- ea_import/ea_import_template_line.py	2013-01-25 14:51:56 +0000
+++ ea_import/ea_import_template_line.py	2013-03-21 08:40:51 +0000
@@ -249,8 +249,8 @@
         for template_line in self.browse(cr, uid, ids, context=context):
             target_model_pool = self.pool.get(template_line.target_model_id.model)
             selection_dict = dict(target_model_pool._columns[template_line.target_field.name].selection)
-            revrese_selection_dict = dict((v, k) for k, v in selection_dict.iteritems())
-            return revrese_selection_dict.get(target_string)
+            reverse_selection_dict = dict((v.upper(), k) for k, v in selection_dict.iteritems())
+            return reverse_selection_dict.get(target_string.upper())
 
 ea_import_template_line()
 # vim:expandtab:smartindent:tabstop=4:softtabstop=4:shiftwidth=4: