From 0762dbb20636708e4532bb2fdad11a3733ea0b2d Mon Sep 17 00:00:00 2001
From: "Ronald Portier (Therp BV)" <rportier@therp.nl>
Date: Sat, 13 Jun 2015 01:20:18 +0200
Subject: [PATCH] Support zip files.

---
 .../__manifest__.py                           |   2 +-
 account_bank_statement_import_camt/camt.py    |   9 +++++-
 .../test_files/test-camt053.xml               |  30 +++++++++---------
 .../test_files/test-camt053.zip               | Bin 0 -> 3111 bytes
 .../tests/test_import_bank_statement.py       |  11 +++++--
 5 files changed, 33 insertions(+), 19 deletions(-)
 create mode 100644 account_bank_statement_import_camt/test_files/test-camt053.zip

diff --git a/account_bank_statement_import_camt/__manifest__.py b/account_bank_statement_import_camt/__manifest__.py
index a59c096..86ce0c5 100644
--- a/account_bank_statement_import_camt/__manifest__.py
+++ b/account_bank_statement_import_camt/__manifest__.py
@@ -19,7 +19,7 @@
 ##############################################################################
 {
     'name': 'CAMT Format Bank Statements Import',
-    'version': '8.0.0.3.0',
+    'version': '8.0.0.4.0',
     'license': 'AGPL-3',
     'author': 'Odoo Community Association (OCA), Therp BV',
     'website': 'https://github.com/OCA/bank-statement-import',
diff --git a/account_bank_statement_import_camt/camt.py b/account_bank_statement_import_camt/camt.py
index 96e7a51..aed01db 100644
--- a/account_bank_statement_import_camt/camt.py
+++ b/account_bank_statement_import_camt/camt.py
@@ -69,7 +69,7 @@ class CamtParser(object):
                 './ns:RmtInf/ns:Ustrd',
                 './ns:AddtlTxInf',
                 './ns:AddtlNtryInf',
-            ], transaction, 'message')
+            ], transaction, 'message', join_str='\n')
         # eref
         self.add_value_from_node(
             ns, node, [
@@ -137,6 +137,13 @@ class CamtParser(object):
         if not transaction.message:
             self.add_value_from_node(
                 ns, node, './ns:AddtlNtryInf', transaction, 'message')
+        if not transaction.eref:
+            self.add_value_from_node(
+                ns, node, [
+                    './ns:NtryDtls/ns:Btch/ns:PmtInfId',
+                ],
+                transaction, 'eref'
+            )
         transaction.data = etree.tostring(node)
         return transaction
 
diff --git a/account_bank_statement_import_camt/test_files/test-camt053.xml b/account_bank_statement_import_camt/test_files/test-camt053.xml
index 1d74d81..4a2a047 100644
--- a/account_bank_statement_import_camt/test_files/test-camt053.xml
+++ b/account_bank_statement_import_camt/test_files/test-camt053.xml
@@ -2,15 +2,15 @@
     <BkToCstmrStmt>
         <GrpHdr>
             <MsgId>TESTBANK/NL/1420561226673</MsgId>
-            <CreDtTm>2013-01-06T16:20:26.673Z</CreDtTm>
+            <CreDtTm>2014-01-06T16:20:26.673Z</CreDtTm>
         </GrpHdr>
         <Stmt>
             <Id>1234Test/1</Id>
             <LglSeqNb>2</LglSeqNb>
-            <CreDtTm>2013-01-06T16:20:26.673Z</CreDtTm>
+            <CreDtTm>2014-01-06T16:20:26.673Z</CreDtTm>
             <FrToDt>
-                <FrDtTm>2013-01-05T00:00:00.000Z</FrDtTm>
-                <ToDtTm>2013-01-05T23:59:59.999Z</ToDtTm>
+                <FrDtTm>2014-01-05T00:00:00.000Z</FrDtTm>
+                <ToDtTm>2014-01-05T23:59:59.999Z</ToDtTm>
             </FrToDt>
             <Acct>
                 <Id>
@@ -32,7 +32,7 @@
                 <Amt Ccy="EUR">15568.27</Amt>
                 <CdtDbtInd>CRDT</CdtDbtInd>
                 <Dt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </Dt>
             </Bal>
             <Bal>
@@ -44,7 +44,7 @@
                 <Amt Ccy="EUR">15121.12</Amt>
                 <CdtDbtInd>CRDT</CdtDbtInd>
                 <Dt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </Dt>
             </Bal>
             <Ntry>
@@ -52,10 +52,10 @@
                 <CdtDbtInd>DBIT</CdtDbtInd>
                 <Sts>BOOK</Sts>
                 <BookgDt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </BookgDt>
                 <ValDt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </ValDt>
                 <BkTxCd>
                     <Domn>
@@ -104,9 +104,9 @@
                             </CdtrAgt>
                         </RltdAgts>
                         <RmtInf>
-                            <Ustrd>Insurance policy 857239PERIOD 01.01.2013 - 31.12.2013</Ustrd>
+                            <Ustrd>Insurance policy 857239PERIOD 01.01.2014 - 31.12.2014</Ustrd>
                         </RmtInf>
-                        <AddtlTxInf>MKB Insurance 859239PERIOD 01.01.2013 - 31.12.2013</AddtlTxInf>
+                        <AddtlTxInf>MKB Insurance 859239PERIOD 01.01.2014 - 31.12.2014</AddtlTxInf>
                     </TxDtls>
                 </NtryDtls>
             </Ntry>
@@ -116,10 +116,10 @@
                 <RvslInd>true</RvslInd>
                 <Sts>BOOK</Sts>
                 <BookgDt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </BookgDt>
                 <ValDt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </ValDt>
                 <BkTxCd>
                     <Domn>
@@ -182,10 +182,10 @@
                 <CdtDbtInd>CRDT</CdtDbtInd>
                 <Sts>BOOK</Sts>
                 <BookgDt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </BookgDt>
                 <ValDt>
-                    <Dt>2013-01-05</Dt>
+                    <Dt>2014-01-05</Dt>
                 </ValDt>
                 <BkTxCd>
                     <Domn>
@@ -202,7 +202,7 @@
                 <NtryDtls>
                     <TxDtls>
                         <Refs>
-                            <InstrId>INNDNL2U20130105000217200000708</InstrId>
+                            <InstrId>INNDNL2U20140105000217200000708</InstrId>
                             <EndToEndId>115</EndToEndId>
                         </Refs>
                         <AmtDtls>
diff --git a/account_bank_statement_import_camt/test_files/test-camt053.zip b/account_bank_statement_import_camt/test_files/test-camt053.zip
new file mode 100644
index 0000000000000000000000000000000000000000..ccf5b3c2ba72ad03cc137edcf2ee55f67b7a2952
GIT binary patch
literal 3111
zcmeH}=T{S08pZ=5FccF7R764P14xk;5EP_{0RmY=5O4w{AVr!$C=oHlAWJhyReBE!
zAqJ!<ML=3G(xoXyK_C$fO&!PxF7ED`-JPBJ19s1xd!BpW_setcr{D7+tbu&u007_z
z0GPk;y!Wsd2MYoK4$A`oVt``++?5cVnmfiHr>U#$s;Q+O;qQmC7X$#CFp;S53XKp3
z@bIyK0Kng`kJ*<4=k&#UW|xqT(@Tov5YUyorl#QtnL>GL823Dkl9(M6fc0cI%(|Ni
z6Q^GLcu71wsM2}%?CvjH!8b!s<&9$V%#|&2<ZU+x4lkelsJ@AUmdg;7B*ccxhUN-5
zijby6yeWwKg10+0y4PY=?AGzegyYneT*WLqu|~D|1%fBU^|_yGn{dzewB`y3Xw16I
zsbi7NVKR=R@}@fQOEy8i2tN<66f;t(CB5_~Y;b&?$*>9fp7$*Z6H3|gs9r@Iab`}w
zn70^AGxt!43sw<D8nGjAhkqGr<@iHX3XH$e^=3FK!Vdi@`Dlxw0!g>21&ugEeu5!s
zB=aSvS|O$Bs(_1W`3N!7kW{?C+9uvd?a-AuvRv(>X!goeIdlV;b{mCeyP1vK$0=9p
zhvGziT|Owbw0q_obsli0rv4Sr^-c(wMC`04QcZhDwWKHp@XFHQlHEbhsWMKSBZZOw
zA}>C4#%A3<RyqHC|H{C84g2~7)+?~mq)y4Cdmmth4$b-^<%mXU%Z@!EmLZ0p`Y4D`
zCiV;*CS`>WrP-sj`HDO4lA-;!yrBVX&3%#eOMKMU@+IV(nv7lhYS%{;$3{6@D@-?8
zj%$0W-f~sSJ^sQi^P9;MO$$IZFznJn;)6b5Ft;O?XIp3Wvr?Mxgx4Co_F1d*^z6>W
z16A~WLf`!H5cIp~GA4ZCR<HxixmC8@9s0l}W$J@Uyl>{`G`WEw$zws*gec}$;&fZM
zo5o5(B`9uG=!A2Ina*WDMms&g&>^6!qHt5;<KPny{Q3tsB)WHBGwU*37mC@IBYi!E
zLkn#5C8<djf(2^~=*c31_a%f12huPtP3I)}RkuK~2~;ejF11*qTcPxE{7Lr=Z@!#4
zBC4e+Nb+Lrx!Cp?MuLa8Bx0hz)-z#Qpuazc%LuA;LteK^$NNS;7f0#g+M;y&uGw@b
zqtC<LiD^_`n2=|*w%K?AKGPR+_skJHwVze7Mqk4XKIH1(U(~55KW!X-U48`AKn1=R
zm~q>GqTVR1>*Ec+Z)I*wlOs-1v{7k(SD7cx&3TX<f17Z=oQAMO<2Q*_vAvs?Y5TO5
zdnUz3DAP62sX1D4&=d;r$S{^BfqD{I(G=vG&{u;I(|ZJecC)IfhR<1ZetNo2@Z!-n
zbscZPH@Efb-lA*$?4R91Mv2&H@Ca>>4UUU|z4aL*H}d*MEMlY`)`QpML{Lh>UZ>Y$
z#RRHb?<f$ZHdvy)BAhLfa=_8?hkke6oG};X-8z!G$D?Y;F%R+%@q)82y&tH+30NY1
z@~1LeV?{^FvTvb9=Bts?k0rCK<l^3O*d#@QIvnh3mG+qJG+63d!hMO~SQGVUrkBcL
z<K5ux-KyR-?X%YLqPCI2At5EA(MZ(0`;k5*L)6{=Tn$v!qMu5Ik@Yi=@hpbR5-!7h
zvI(&u(04GDF!)ncP6)-oE6YMSA(s^fOMwi=929H?RpGCF)w{a3ku9?02YT3DQ#~fz
zo=^uaYCQK!#+@T8c1V_F5=U-kxm*|*%++A&xlGRvbo`+EJ~^b=4%S|A!a6W$nM=&*
z84bJ+^J(2m9R}Y#<FNY-fscU2D^P2gFql%(41HIHnl4`_Wwd*7vC{ZJf)+vcQ+xG@
zDEIwIbI4}W+2SB^^`%vO7R#=iG>tB<n73Ze7>{p^?Uws=MQd|l>2$Th%nhp<L6f@v
zFRe>47tF`Rx7B!RoxbGo{<z9%;2S)@PzCrD9k#o6OsB5-t*Rs>x9Ijt)~YQ+V444%
zPzs&KG$xTJ)T2(<N6-wu4uhEyZ&rCfpWT5Ve#N!=9<EiO9E$rN;#&JxTyO26`n&$;
za1H+ruD3;fiKWCw^&vj^y+E7wUhHgB#8it@dQW1ZQar<I|AKuwe7$sj2)$zYt+C+@
zHesNE;CM7JJOiY<bYXy2%(C03$=NvEDMf0{Yxl8Uq)((W&~-qKq7k+UGw<AA_VC(S
zdni#WsOP}zOKrF>Qz^8|QRB}mXgGS2uVz8>+^ZoN75^~j6MqhVW51<$wUzWDyF|J1
zirME@jT#|^OHAAz3aV0H+~Jxv^`tk-VY6f-Rl*g=@Y-F-Ba{e9gU80nOv-XmG14&C
zV<gElSr<s}nDKZW?iPybi*uak&oqTCQ8Ex78q&tDBAzoAb{A27`{Tf7r)TsH1z$Za
z>(E)5%on5Y=I7h_Gab0duA42{ST<Q-`SuY*GRv@QLb@e9NkLXnbo&f;B@<GsFONp?
z73)D5Y{*gm_an{Y1vWHHL3{ZKclT;2;b->}8V<9on@o5adK;Lj4$TmL;iTZRlaNcE
zF1&6LTv;hka~iWF8};~H>r~tMthOl-2An0f&<V6dCmd9xy;0yHziJtQj>eCUPY+h!
z>c3ZOHlMoCRY`(}DnQqY19eBIViuA`HhKm_r5I4{dR+8rdB$SfS*>Oa_D7-8&b&W)
z{U@*guX)YGD-Hakf-SLEya527J(Uu_PrqBYLI0@RzhO7R`d`NX9R>ex{Ae#A{?quM
b!LVoiJE4HE2JwIY3-8{k*?V1uzi)p7q~E=A

literal 0
HcmV?d00001

diff --git a/account_bank_statement_import_camt/tests/test_import_bank_statement.py b/account_bank_statement_import_camt/tests/test_import_bank_statement.py
index 8a1694f..d14aca9 100644
--- a/account_bank_statement_import_camt/tests/test_import_bank_statement.py
+++ b/account_bank_statement_import_camt/tests/test_import_bank_statement.py
@@ -31,11 +31,10 @@ class TestImport(TestStatementFile):
             {
                 'remote_account': 'NL46ABNA0499998748',
                 'transferred_amount': -754.25,
-                'value_date': '2013-01-05',
+                'value_date': '2014-01-05',
                 'ref': '435005714488-ABNO33052620',
             },
         ]
-        # statement name is account number + '-' + date of last 62F line:
         self._test_statement_import(
             'account_bank_statement_import_camt', 'test-camt053.xml',
             '1234Test/1',
@@ -43,3 +42,11 @@ class TestImport(TestStatementFile):
             start_balance=15568.27, end_balance=15121.12,
             transactions=transactions
         )
+
+    def test_zip_import(self):
+        """Test import of multiple statements from zip file."""
+        self._test_statement_import(
+            'account_bank_statement_import_camt', 'test-camt053.zip',
+            '1234Test/2',  # Only name of first statement
+            local_account='NL77ABNA0574908765',
+        )