From f226726fbf1abc471c78a8d5ea0f89f7211e4765 Mon Sep 17 00:00:00 2001 From: mleiseca Date: Wed, 10 Jul 2024 09:57:33 -0500 Subject: [PATCH 1/2] Fixing https://github.com/pyexcel/pyexcel-ods3/issues/36 ``` File "pyexcel_ods3/odsr.py", line 46, in cell_value ret = str(cell_value) + " " + cell.currency ~~~~~~~~~~~~~~~~~~~~~~^~~~~~~~~~~~~~~ TypeError: can only concatenate str (not "NoneType") to str ``` --- pyexcel_ods3/odsr.py | 5 ++++- tests/fixtures/currency_without_currency.ods | Bin 0 -> 12323 bytes tests/test_bug_fixes.py | 10 ++++++++++ 3 files changed, 14 insertions(+), 1 deletion(-) create mode 100644 tests/fixtures/currency_without_currency.ods diff --git a/pyexcel_ods3/odsr.py b/pyexcel_ods3/odsr.py index 262d1ce..e0577cc 100644 --- a/pyexcel_ods3/odsr.py +++ b/pyexcel_ods3/odsr.py @@ -43,7 +43,10 @@ def cell_value(self, row, column): if service.has_no_digits_in_float(cell_value): cell_value = int(cell_value) - ret = str(cell_value) + " " + cell.currency + if cell.currency is None: + ret = str(cell_value) + else: + ret = str(cell_value) + " " + cell.currency elif cell_type in service.ODS_FORMAT_CONVERSION: value = cell.value n_value = service.VALUE_CONVERTERS[cell_type](value) diff --git a/tests/fixtures/currency_without_currency.ods b/tests/fixtures/currency_without_currency.ods new file mode 100644 index 0000000000000000000000000000000000000000..c0e0b65ba7add3052a7dd6be145801e66339e3e9 GIT binary patch literal 12323 zcmZ{r18^r#xA%Xs{fn{T#@yJpZQC|Bwz08o+qP|MlWc6f`@g8C?sMPUQ(fINRp(4i z^|$MsKJ^)SDKKzYz<)L(K(IAO1l`}8e>(KvZew9%;_P8>VqkA?ZDC~KY++|h?`msI zXJ_DK;Y4R=Z(?g~XXIjIV(U!jWbbHVVC-aWV&W|S*C!AF`5zvL0PIz#gcFbez%n@i z@Xwo^oIR{foao$btQR%4>^4||zTaxU2r0DYct=w%X`;6%hEiBJ29V*cB6%56veHx~J5(PCF0`q4*?HMEMRjMPJ9W7A&(+(Rjlog0 zbd*%uorFd}QoOm_^hYTD!a;VEfMwt!FIdkGumD{mWh2!gK zIF+|1xQ9`uWN0FY1)g9QUV?j_Aypf*EB9xV=sPV*0Qia{E`5G@@JxNb15t9<36oT)*c5{6MwDs zu%Oi}KRXxjb#~+6Xl!1)sxDN22P0x${qa!{T|qLc15JI(;Bib#F2WD%|6?*R$XHle zf5q|~v4dbC8f7IjaIdCWhNbPff<;PP5&xh7MH;nS1n+<5cc^Zw(K5fv1aI^q^y`PL8b;=F=DiphiV720aN+8wV5I zTW{Qyu>fBB@Zym*$V?~d#MY2@7;#e;WDe6xYA)=sl-Yc!Qar3NakNLQad<|Ldt$Hx zxXsHdVhnVsGS zOzMg_HnwGfd#3<94zo?FDU?zXlCN;~R;^*-K5$+F__q+<3Z~($G4^U2S&(lp!u+M7 z*VRvXd8_7~iC6t~d#Ctuer6Y`l~>22>pJuma08KW_$hB1Ti>w8&_toG`gElAvgR!L zyL;7^pF6E@WjsQZ1sYUhE`>|5)2-337|wFm0(o9?_SlQTuVLPp+cz8JHo|Rlm|x5v z$H>CBfb&<0B;R;rcuaDJ-ySWf>(%q zMpu?mpdT(9#dCJ5let;htXvFs@H*Tk`1el+`A@T zH5@Q#>~i@F>u?zHu(~bE(e4biorlx!eH)BSd?&7V$m3{TZ8i%>^vXJ~N4UopI|A)) zyp(A2u*>Nqw^uVvL6y|`+|#`9e_MNbnQ-vF@`~t}1fqes)F@zmTCY>}t(m3eq{2f| zJ+)J+CkWFJ^Wg;*j2$KMzeuM%Ak5L(gAaTW&-CNYUul4fgWr&g?k@6@gHE6o7iSa> zl?D}@YL=e=6sQDKbW-8>0Jfs)+d{XF2SP6Bs8i%UV)pLwR0Q8hbJG2K85P*rw-|tP zpb+B@j06ktSKrfTLKH&{31Kh;g(XN}UC&j>6udFiqObmqgCG&wm>@*Nr=Q91BCJy! z_MYG%4jTT$Xe$4(2}N--D;iB#;7Er~eIyt*Jf0Ia=U27iQu@$hLU&HnF@7Vv*OYq=wl#R_H@ORJka`7T(imiiOs;^V z)FwuEz{pM8IM?J92jGMhoav4noX}n8Dh2e!lDR8S`n1`y#om}E#^>9G-q+E<`>`ogvuyRr zK`571GgZt<1Nu94@5xjOdBt1{rS?Z{YSyIK#MrGdYkR})(aqTVcDhH8v!J&3zIV%8 zz}uRDpHCO+{*&7BjLPgza}8OKSDPnnx8sT7ZA%Si!AS0owI1At^yljo_bL6CwrnJu z)9ORV+ngn6hK%pz%$f1yh{c|3Q#&6Ar>D0!w|6DpH0Dn1plO50VW*ZlU+M3gJ2(GJ%~`==>QaGkvea~97gzKL&ys(po^bRxEPzG(M9j@fEy zj{<5jPz0x`gMY{c+3`C@vK7>+bT$t!9IZjQEoJ-?_!J<;gW;tuMOy`3y?K$wiZYdA z$K&hHsig19kj{~X`+Dp_x?A$=YMu6}vv+>nGw({78u8j`_w~82P2cImjEbCIu}wL6 zpJ_>|BU?9H(A}6C{jfb*MITvqKJRkk(_AfCXsa~;*1sS)(vHm|t+tK+=ycmvlYHUQ z{!RK#b#=zPd+AN_O&sG7W-cX0MR*g?_V@~sPc!jPi+yd+RLM#P%PP{0PXTmpP?yM3yM!Y%V3OLEMQiSz*nsEkx~Iyg{TUA^CT zL=X77>5p?+L26`A0C8$?KSStlJwp`jtYRD=5UAz+oQ#gQS`M1;42{j53%uNXLHpve zUtSRb;hv5UkO|u~*DUrrQiFPu)8GCy;Az-PUz}r*p1f6=S(@z{dtq|u&jer4cZRE} zx!c?ScEL^aa490^m1Zn)B9?<|c-gmYbdb0S&b7KJxgZtG$;G zbaU#$H941wd$#zjt^amuv1iJx)1Eu3F@5Xm%md}xt08Ky#pg$xe8rI}_wCAb#x%

Z@AOb2%7Es)D^oPln$&;*h0cD6pexM|Vei zw@YUdMb9-cG%zf(PO|Kqfy>G9>y)exqf|8))i^6sH$eP>S0 z&tChMWgnW@$T8SjpBr0Wg!~^mK(!s+i6AkH&SJXd%;A=ohlLPZjqL%n4&80d<@1Zc zx21zQ=XVAd_2m6|&9$9YU}pMhQ1CzDzj?g+&OoBtGf@7DxpX@A4UihC zqSwK|T<&7g)L7}=U#3F1Zr`e?)@dGrpqm!bi42nY_M&F>jbU7EUF>b>{HOl9y=B2! zpo(^fe+2|L=bsEcZ;i1}uV@q$s`fkCgq0y z;tBC&TI6yO=egy|T=Ez^MMP3~rJ*ceR8$`7M4cv6K_4%2Og>WryOi$w)yUVxbjdw; z)mR@XPE@aJ@~mN3Mg1uxu(chR;QL-##d(`+L*b(|$5K3xf6j5){oW*qj%Ncqmu(aW zR<{a-XTaFI3cbOR{6hJSb8&aQvb2y#_<X%4K}WXtTqhnElZ#MX9_(Rori zmO(*`ln!KXHAg~)YUI2WkMGbk^ju#catIyq`+Ox|ctb2Y5^t(~>|)#^BOr;Ljrr*@ z4o6pSLYXvzZ;F5}ob6*(%6|L4%_5IZ?wSqFh~ZSqPF;qwFN#cLD@;Ae5)^X{<4`rZ zPdFEhwFagbCX{WU)>N>|($@Y=*8WADiCe#!$Si__NLs+gfS}WHz2}*SD%)eTmVpQA zHQY*&r*P(L4sE3*HXWX|v5(!!iKgO@r*!z{bHYpymE37@oNb_ty7;q_1m~QLtaL6+ zdd?^;dUfIeZ(J^|Dg!h5I;;pV4~8l7oi~&0I)B^fw@oGkb109DdD*<$S|IGlr(Suq z=28(Thw&=Wtduz=Baf{kG8BpC7pS>X`5e(!q^C0-6C3A@<^XyB!@WEY8MsFRF=%2-BO>$RAJG29BNyuCu1xHS0JMhViQb1Z&q3~EsC`cS( znpo!qO%7OXFs{{_DQkGh!UOG(c7_Zy)!};-9t{$f}SM*|;W`g;kj+p_LHebFEg^j5+

6@_no!eOn*>&|ilJ5~r!O(NCI?q6c;_wAVwQE1 zp@9l?`hb>7w$UDW!pVmVJ67zNcUbb@ zW_`d&0@W?$bjDL(c^BxH(*RmgN<2%z4(_M%5RC#RG09!{gVK1~ltF^}B+(}RM$@?Z zgiByHl_6s|x#pDi&)`;$>4p&bs=B+%Q-88-3b{QZrh@=xk^QDxl;wUy^C09^X)(z8 zgC58+3Csc+$rDQM-_1^5GN(mvHU0iXZs1`E@Wy^4>4U$@NJp$>Jc-tlhv{ zzwtW{X{#aSExpI*QDu+&X^wWSj2EbvMHn72v7`^G@Mt7Ax~loR^J~qMnuD>+L(Xy; z>!4Xvl*4v`ZaCJlDa(jpr8?Se5z6m+VASiE+HP(Q>Xo|P z>xuuO+S>rPOoDXb@og6DoD5ecoQ)<>qZORBqe^L`Q^^(ut?UEcQ(i`au3)Kp_KeZ)F92>V8Gdp`ZTxMOtcxbMQ71e`Hl1^*nEa?308W{n3-V{K`<9Yh!sMTltXf z@EWwp3)LwwC;JedMkY--#4rtpc`#1+VhlS>kqpr{MJi%FVNR`a5AaMYq=t@gN&^F3 zO#t5q7?wIc4_QtqLa0c-&$k)B(T!z@v>SyclAv_ds&l|7w6q`55LR(+&>8dA>h$9!awv;#@F_$6;-AE)^GY)TBgSc}4dshmdjsCo5D z;Ky-dmuQdp>jy!(!NQEA&tRzmxXpU_CJ4$7vDAiX!bLLIBDhH_2~&Bn=6`mzaZ<;F ziPPqP3>&QrGAI*|rL2NcDaZYIj3{-{yRIm46Xs5MHikKxk_gYp7+d9h5r8i#vV=rD zDXofJn5XBn9SA6@i5GJq#|u7YEHtd1wi>uSsp$dB%WWaD5)Iwknprl{;hu0!M1D@y z@tJw<45^<7B8Sv_5+~5?gB2LMn=-b#lvJkJV*Qn5=7qRsBq@seGtogZX`@R)LFRAK z8CfqY%3?~~^;uxMLx2N>Kw>qm@u(j@ zwEkn3##(~q@5t)KPw*;cemRB*8YixO<{PL5iRV_|3z2%vuu@7wV z(a0$32&);HSFTIx^wKv%0Mmkz<-Vk&RDKyzpSxx)s@*1Z13$ZN{(>!KdC-QD8z=mI zqzb9Zv>3T^Mf^}QyO%Wj!F}UcSoxF@xHk~gTO2L*h>1FbeVKIXLNN8V!BCrDR&ygp zUDp+T)SizGcqxb>jdgfcC)YX8q>e9XNk!`<$21iDo;V*+L|`SuAXC)SqYoR-6bA^A z;Ur@jMN!k0u0$0P{XrOu9W_es!~XN$^eHFC*P-lpR$*ddoh=av=RV7~cs$9d8d9X9 z3AJvvk+g5)Yo~~(KkGyYB*Erq=(JNyoG-#a;KKu<3$9@pvS*X&WIEpU!R{iWgF;af z92FN9C)j-}MeKVF+>|8}hz6V1HtDLTOpW)`rE~+hBgPoRVpZ!Ti~d*foY9~@$&or z%k_Ba{_2w@a60Cc<=ddjONkf8J=KL+D8ojsfO~TZkxUokF~+#EwC^(=8JLb|6Da4* zGRi5Vd)nJzzZ_J}M1)KYU`L_eR-ME0hsA6EkUI;L>Zm6eR!>wqeYy22eQLhCh4zcU zi@q6`4-Y2ijw@?7b=Hjo1>$r|9mN=-_@mN8hy^m}!x{vxZLZH?aej`;*Hw(El_N^1 z6^|s{0xR?>Rm%dXivZe`{Pq~z%%Uk*^sizn;CR&x(WT_QjmpTc8)llsnHi}QFnmE4 zXOcXp_!eq@)kD+?@pii{gu)%Pl@YscD`gl?I{^|J4^F@VLgGPWv~Q*VYf z)RG@N6-e^R88c<@zCvQGU22O6n=S}IT*=pd_ zsE&_4O^Cso>Rbx7t zS(&e~O+~R}__J4DkP`+O_7NLk7(UXpewyJdh3RD*u6%ja+=VBW#0m?AE3haR2&0#V zf!-xcOvKq-Eyz1l^qgVEX@Cf2C3QQ};krss%{n(7bb~wfy_FkZJ4be`)wkAORn3jJ z^U{+3bRir}x>K)w^K;s|R%B}}7YM2AMm-H_Hq>>1Fv!Z)&>vGo-W)LCB_jNVL(G|h zrn0Si;VZTMqs4O`B3Ggm_yXFixB&j=E3=TQI{PXwPp7x{)u9q!hLRT@!(&xXO4u5L zC5VowfW8Sf*1w(R&QO;QJR=EBUa?{I7;Ws6jI5kM=h2RCO_}&{Zw-DU#5*asch<`v z7sXaMQ$W8X_Dix9WsGYJzXk;c#0>|;QXf|uwVFYqYQ!q@magn31_zp`>e9SgHyo^!h=A`4Ua!{SCh;6=~D4eWtKDrTT zj{>LUk6rPRg<cyyU5IgHs^d1|3pAx4o*`{yuG6*Kij5#i75p{X4|3Bm z7~O&+XK4OxX^6{rw2$8xn1yezs;?QLFx>^&7ZO%V@1J?aKe`D-88??)9poNM0V(m zU97}`dl1f+gsv*H7u(Zo^$doLEW{Oj&n)qP6KP6IRfRmE>gNMTy zX+WJ=W^zxzrVnMbHFz z4Z?{sF?UlXV4S~MR*sGk8Gl}q)aIHxxYd{GyzG$|FJa9BKt-;QE`&%{pTv42Z5}^d zlqbV@^WBeEZJ?=1UKIaLfcud@uL%bb(;p^;!c&CTS1;NQl}iEHv&;A|Hrk|k<8YYRv{X(vLGW&^nhRx#*664$Dr0rV`gr+Jai zGDMtKOm=7u&2$!T9*$u{cl@zoEZ+P8C8~QBHbq zQuQOq4~fpCt;QUaAar>ylos7{xo?(mO&l&T#hW&vP5Xey68BspHu6O_zqE_@EN+qv zA<5|(SKL7eC%vA?I|XWBX59X|WA^JF`?fu#D~Xv_Za)PLl1zkl_;G~?BCt(BaFXSF zHt`D1-(qu>ZeS2}$%x#Bk(sG<(31wID7qn{V^uzzax{aFAO5%~Za`ncdcA>-8l#E8 zFWxpR)ly)FG+bwVVR^*Va)L3fW|EHYfh6<8NVRL}kQ6kUC5a61h*yrbn8T6Ng>mnh zmS-4gLh*@_8>JpW`w3Q z2fF%i^}pFw!DxkSKC)6~O3M85->eN0@|R)_edo2VY_E?61J7RA+igq;h{UG3U~6j) z=v1FzlIh8!S~$x}1hU#yQc#=Uf0U$EN)*0w1A--A&xRa^i6<}dZm8_fZkA6(kh}pqmGW4 z*CV<$QSu@WP$=o|UbyyKM&O6Wtg4FQ<@gU7w!C{iH25|loJz_5w)w(TYT1b52;dZ% zFX!u8QPGi@hMexn4V&iRXQpy1L#I*^(s8M;8Bw%0%6UPrSUC} znx;+Vt)ze&PnpfB&DiONkWXkxpZ^k_(TTttBSfA8%ZC;C=!q4B#K{1n?_|+is*%KQ zjC4zhG*apQSxpH9V!3-Ehx8$x?Kst+_2iMX0dcEM2h&L#@>>2!Q({y$OuI~upu!|Z zvuJ1az5Ll}V%lXHxrn056ar(xVhH37x_Vj}>0WWt2rU>!6A2Y`2=TW(i!brlWa}S2 zYvgPr=LKj9{qFE|{jHr!yMeQdhNI3|xh1rJaJXd(jpE=5<7FU`hk3#r4vZ#a;?1(F z9HsweJGLma}4mfdd{5i+{@kkA(R|A@>3+lmj2vKRLoY3KMh zE~Q@k{htPk-NwRumqCp&2}&q)$e#RvZ7QvmrRn#9(U(9}5~T1$O~vt6S7gv;vcmtmMU&Fk^tEaccJkRu*A@jl7@QGmSXixJ=Z*DdX|Ri zx+imCXF%xC2i5FKbofQJfY(IBl~N|bRR$q`S|O4?i|G_=CjI*m;-xSgT;P(|f*33L zr{ONt-Nh&wa{;yBl+R$o0wEh)Vx7xvp!kmwW0pr*vA<=JMr!Q=peiY~;TXw9-s|WP zfx*n*B9pLqdRUxxdu7n&C@}&m=}0M_0h9$zznrxf>41chOnTuQp-!rI2^frHS#QrT zQlx&?-k8PnE*4zyXiy@Xj0`(zq#L*M^)syXO4D{ldiUcjPGJ>zhq(d98BishM1ADCBo~A?C4A)_D^v* z?=Mlhp)>NBa)4DvE#i3h=t2%!Z%iWj8i?-=wug7y=~D1bBNtl+ykL2^<7sek3e(%U zTo)3eBh^m2Us(7vNn1uE8K}X1jt;RsV_sp=UgKxCNq5u2j&dAPuF+kmo?iae!8Y$y z8Zc~|N}p|p#UTO}*nVXf;HCf8@=nnHT_+z`LOz z{X!TN0P9=jLK<;V&1H=cY~t@7*D5QKeIgDNhAyyPu7o>6#*LveA?n(}nb&V%U#CF-$+fV0>Bd?DA+5UKIQ^@;x}CF>hbarLM-jrkzW&TG&ojlhU^J9E~Aw-n7}nIJ&HSq zm1&LFv{#gr{Mviiffz869=XEX4m>U21-4>ymr4Ju-X^oW+^y^!5&V=A`8(kY0yF=P z@HNiFMk@?+=7=6vDab&Jrpf+7s%E?`h`vKh8vMHrgUYhgS*Bd{_TON@WM zRFbm{TeP@Vn*Q5onea%x(av8nKSDBh?Jps|hU7PyD6NRchHAxEdT^`DHvMT#PRnH7 zi~u($htdfrHkxMy%49xEKh;eYqQjtOh1U{^{?%kHv7X68;->~GWW9m-cW^U0c#Y+t znR$MU40WPpzZeo~Q#KFzS&!g9dBvEaIr1q0dm<)CuWNf4=u7AWY(tD_h*nHTX24o*MyXeSr~No z?*7a$onh|bteFK<|FNPu!&UxnkD(^_v}{3S&V<32=_=6N<9g&1`r}DhF1)G&qZ~*X z)XayNdl{RUd+9XhjJB?;a6~V-5ywJ4&GPKL^sA{kA67?4rG`6rCQ!RCK)cE5F3w^| zRe9t`zNY@8KFy!k?re;3OYl-UyP%mt19}Ay@*Vu(FOg?A1XCZ1dVM@tFtld`gwp60 z`ZiTU-egg)_j^*Ar}0`P!Bc(b2{W<^)(>`C9AXw$@?X~EeUwFzL&Cs;JWBlb{9JZn zWyx?Z>|NtipE6pIjkhO%V_j@POjBu&p|Y7Tbkb4{mGy+-E~K5>g`dt!meW;XD&5bw z%|q8)VyiiuoljKa(O*P{+4*wC4BvPWS&Ha!kxRZRu`nZw#nZPEBtb2EXTiOgB(<)@ zLq?#~Qkg*4qNhtB5---VeTsWRRD<${K?G-Gi^Qy}uRE*o8;b3OW z7B<%C3APZ(aa4u^lq0NhQyf|Pnz^v<)D=q0v0$}g(0d6;+#7E)6F)W_+_@z+K$2@t zLUWE0)8~`RdpPf`rJ7sm&1Y&U*B7k7S4#7UF6GDLiCn1lqia&ibbYvcSB2zl#N{Z;7bdW9CMX-(U7}Oqm>0xR-LPe8v+y@?KzMPS$;=C? zh!yM`P*g~1*yi0}BfCRAI!MdwF4PoG9907=)4DwNdEVD zSI-^0xGSuy9x?7vPfEp7W>GdKFjll9kZV{s*nn@-9AW{5j}m_M`j|yYlnG!FVQ-)d z^B+}#V%k*cDq4@0#RDFIU^WMljN20^Kz7jz8n@)MJ;Q6=*Fg3|G|Wid|8T%YTTN#1ANza|CoJm zXftt>PGJo$5*0O>z%w74P^bo*S17hCVO*3hr-PotCXyY*f61%Ym46rv@0ncuOAB^= zz|ev=JGGuOPtvdXLL8W4IH7iRM#O9tT|8S~Idr@H%8J?^D`$;6cw54d=0jg8W%AC9 zoLA;`ux-rx#+W8s^#zo^l(&cSqKTcRa#1jzD7+>|Jq$m+y2sti)j;vXv}_Vps2oT& zG}|6mwEx7;!NBW3GUo|`X#6DuDA!I{AnM+jrU^9oR$q;v1VzY;rEtPGI#0i z%bvaC@K*QVC9;b#Uv15ul*D8gVj4WF?##9nZ_RwOwX!u`9jxq_y74s@PEKZn zPImi>qkU|FG1J{K!+x#5mIh7hG;CD$G#$*gO-(DG*Q>N9zezlhq?AWHR!Jxc!geGRqUdgg&kF zc~-oZ?X6-L+*NRN1MLuhrp<_5Rx&Yv=sEZI zs!*vH=rQDW5R}bgYrBKWCj|i$8s|5cg-LhHj&AO$x$pjSmmLk-zGXWWmT#SXXDR~U zG;ts3r4@^fsNWZP>is4n^(SCsWW_-wNYx`=CvwINCWIS?gp0&*{q?QUTf4f#Q7N&* zRLRN4WwBrX%%IWzGI4FPYs8Z5rQZt^LlAkb<*LMADvQ-2xOkX^F}PP!*nA@2I29um zoooEra)8gycarW~c;HC>-y>Z=#Ks)0nUyYL1Vk3MtZvEI4U%WuaTF~iTcrdf)s!`o zx^AKWZ1p(Dn9c`@hRYeGMfT(G7)DK%oj<~Ao!{>aO3>(Pj_hJ~+CKh5c>lR-AfUAW z0ipk0M!^5Z-T!&|ulC>YyS&stR{uB9{x1vVztex=?f-7~?^NKw?8Kn{2P63JhX20P w{$)r*^MCBSe|Pck3jLQ0Hrl_K|9|OKUJBwbs1E=@{JjYO>iP1Y{{R604`*a0j{pDw literal 0 HcmV?d00001 diff --git a/tests/test_bug_fixes.py b/tests/test_bug_fixes.py index aa69420..07ba468 100644 --- a/tests/test_bug_fixes.py +++ b/tests/test_bug_fixes.py @@ -109,5 +109,15 @@ def test_issue_30_precision_loss(): sheet.save_as(test_file) +def test_issue_36(): + from pyexcel_ods3 import get_data + + test_file = "currency_without_currency.ods" + data = get_data(get_fixtures(test_file)) + eq_(data["Sheet1"][6][0], '1.75"') + eq_(data["Sheet1"][6][5], "95") + eq_(data["Sheet1"][6][6], 25) + + def get_fixtures(filename): return os.path.join("tests", "fixtures", filename) From 2ea121ff913d3740c2c9c2c400ad6ea36d01ae7c Mon Sep 17 00:00:00 2001 From: mleiseca Date: Wed, 10 Jul 2024 10:02:56 -0500 Subject: [PATCH 2/2] Adding to changelog.yml --- changelog.yml | 6 ++++++ 1 file changed, 6 insertions(+) diff --git a/changelog.yml b/changelog.yml index 09c536a..fc7568c 100644 --- a/changelog.yml +++ b/changelog.yml @@ -1,6 +1,12 @@ name: pyexcel-ods3 organisation: pyexcel releases: +- changes: + - action: added + details: + - '`#36`: fixing get_data for a currency cell with no currency defined' + date: 10.7.2024 + version: 0.6.2 - changes: - action: added details: