← Back to team overview

clearcorp team mailing list archive

[Merge] lp:~dr.clearcorp/openerp-costa-rica/6.1-fix_lafise_import_bank into lp:openerp-costa-rica/6.1

 

Diana Rodríguez Martínez has proposed merging lp:~dr.clearcorp/openerp-costa-rica/6.1-fix_lafise_import_bank into lp:openerp-costa-rica/6.1.

Requested reviews:
  CLEARCORP drivers (clearcorp-drivers)

For more details, see:
https://code.launchpad.net/~dr.clearcorp/openerp-costa-rica/6.1-fix_lafise_import_bank/+merge/163779

[FIX] Fix the way to build the line id for each statement
-- 
https://code.launchpad.net/~dr.clearcorp/openerp-costa-rica/6.1-fix_lafise_import_bank/+merge/163779
Your team CLEARCORP development team is subscribed to branch lp:openerp-costa-rica/6.1.
=== modified file 'l10n_cr_account_banking_cr_lafise/lafise_parser.py'
--- l10n_cr_account_banking_cr_lafise/lafise_parser.py	2013-05-06 01:38:09 +0000
+++ l10n_cr_account_banking_cr_lafise/lafise_parser.py	2013-05-14 17:13:28 +0000
@@ -137,6 +137,8 @@
     '''     
     def statement_lines (self, records, rows):
         parser = LafiseParser()
+        line_id = 1 #To distinct the lines, in the case of someone are repeat. 
+        
         mapping = {
             'execution_date' : '',
             'effective_date' : '',
@@ -171,7 +173,7 @@
             #execution_date
             mapping['execution_date'] = date #fecha_movimiento                       
            
-            str_line = self.clean_special_characters(line[2])
+            str_line = self.clean_special_characters(line[2]) + ' ID line: ' + str(line_id)
             
             mapping['local_currency'] = currencycode
             mapping['transfer_type'] = 'NTRF'
@@ -191,6 +193,7 @@
                 mapping['transferred_amount'] =  -debit
             
             statements.append(copy(mapping))
+            line_id += 1
 
         return statements    
     


Follow ups