From 461e4ef6bd1898cb6f58bcc91ccac3fcbe2571e3 Mon Sep 17 00:00:00 2001 From: Harmen Date: Wed, 21 Apr 2021 08:13:59 +0200 Subject: [PATCH] fix sharedStringsValue.Reset() (#37) --- shared_strings.go | 2 +- shared_strings_test.go | 8 ++++++++ test/test-bold.xlsx | Bin 0 -> 4948 bytes 3 files changed, 9 insertions(+), 1 deletion(-) create mode 100644 test/test-bold.xlsx diff --git a/shared_strings.go b/shared_strings.go index d792772..cf4d121 100644 --- a/shared_strings.go +++ b/shared_strings.go @@ -38,7 +38,7 @@ func (sv *sharedStringsValue) String() string { // Reset zeroes data inside struct. func (sv *sharedStringsValue) Reset() { sv.Text = "" - sv.RichText = sv.RichText[0:] + sv.RichText = sv.RichText[:0] } // Sentinel error to indicate that no shared strings file can be found diff --git a/shared_strings_test.go b/shared_strings_test.go index 7c4c163..8c3ce20 100644 --- a/shared_strings_test.go +++ b/shared_strings_test.go @@ -42,4 +42,12 @@ func TestLoadingSharedStrings(t *testing.T) { require.Equal(t, []string{"rec_id", "culture", "sex"}, actual.sharedStrings) }) } + + t.Run("bold", func(t *testing.T) { + actual, err := OpenFile("./test/test-bold.xlsx") + require.NoError(t, err) + defer actual.Close() + + require.Equal(t, []string{"Contact ID", "Phone Number"}, actual.sharedStrings) + }) } diff --git a/test/test-bold.xlsx b/test/test-bold.xlsx new file mode 100644 index 0000000000000000000000000000000000000000..6c9ba4a026d4a9884797c88ae9329ae78ecfaa69 GIT binary patch literal 4948 zcmZ`-2{=@57auWbl65R8TM@FBHM`21eHpT3SC%pMU1JxrZ&@>zN|q5)wu(eTXzXK+ z5eivT4By=P^5_43{hsIEJ9FQ2f6sHy`JMMY@2hj2m;?#}fyhCgw^H;-BF;Dj6N5lI z<_Vwuy5+*K z?w=&7gL;g7@ijY@@5Jw$p3>9cxnL?*ttQaXRb7eAZ>eoee%)~gUn|HYbJZ?+ny5bm zBPEsIpyp!nzC}7T@t%>0{8#WxGQ;;GF!-&#b64OHDCMxablb||5xGt>W5 z!ufUKxiVUo=%?l2k6KYwwIGAm7{E1`aRbGS@-2c%jydnDoJ))0}#)j4enCZYT8m{zp$*UxG;f`ZA9{-*Kb|1dm1jE1z11+8>^Pki zreZXOR}@m#IinG2HO5Az{4#^J=o3!jG!>G6$OrfHRh#|PR2bhkp5?e1N!A4J3dni5 z!V`M^xefV3h6mxypL8d*g*dnemI_Vul*}EPPiq_FoSsZJxW+_mUXA_P#3N^M-{6se zFxAy6FPkYzDq45vb=A>`z$@)%agN2a2|0Ag z$FFu(gT@6Y!Fa#T@ceXh!$O}Y%wBIZ@2R};?M3Pjtgu`4v$ekI^QCPpcisp|VMFc& z7qQ6w2*mH=zAK61q4=c4#!hIDIBR^1rEp_gjbWcyCdN>9Q?J4dTV-zcT)<%hbZh;S z4lIM!`Rbr;8cCm8W!?(yvV{{hPtSyu$k!Bzpza{1!1)}$3nObAA9pA7h0b{msu;9! z2xI?!F=_YBYZ73yf(NAqvCk230pns;N0HbZGVV6U3$?2B=pdKuNLNh;HUaeJT^4Wo z;WcqHD~mc->~#Mz>@oKbY36y)-$Z*oZqii5;0xZrT)JPy!mU){)7WoA$E#mwTO~Np zXZtuYzfM+`ycrx>?a{085@Si8vn0iuWhPv9dv2u*t*fM%d1XAYBTQIEWWqpO`Z4P_ z?Fh=L8I$V$*YET6#;MGk?CBH^T|tGuaB!U zqm!PY{aBuwxP|Q6okQvzRJ4HuO!rcIt3d~$zH5m{Ag*HVQ9Rogt`CW z?bxspPn9D4>%K2%HTgH!lifF9Hgd^YnRrDq``B~kS#yyERG~VYjSLe-X9qt)Q2UZtEkvwINnKE$ec9dMbqsnTL`REv63mTj`}uuUhcw-qu7PorUMP=bTK*lt>F?2<`n z@|EQ2xcOQN<#wZM`4O17%-8=cVep-~lAMeSl)@Wd5Ue!TWHPN`?s#gllC*p4oJ6I=LUVePYaRbX zrA%Ho%_X`BW~H=jIHCVsWo@p>?6DwR7{zL z38EA}^Jsd}mbQnvSvt;4F1yzua&m)@^eiI>ii;u2o@WQ@hZMJvkJx|Ld8x?WR%vMR z0@Oo<6YkQ#n)x}i|C-4-PinIEGPg>~e%Ke9es-MH{JtYPVY;_U`1)k_ZKPzaRVI`) zIKZ&PN`Fs=%A*_7GU^~(8aY~OrSK;pEdPO4`U{R`~8I;V!* zH+{NC+b+IWEQwV!d@#nEI#B`|i~eMq^WmEXtENIO?@^j^_X2wCcbmyv?#amIWyQQt z@{iHVaVu``a~1e7-+mL`3|sAs`6{2)Nx6hvbQks2&70gDn3XuHqB(BW`KJVN@DoHr zO;C|7!bZm(;p!n|}4CP=*RaBYpA;7AV2i{fT{s8t&Wv@8V-j6fvlPlOx|;JXIdmvj=W4XjChx>djM9elGomkGNhsuf zn(xL0G+a_H)Y+WYnLg*H>Bvvl(UAF}~gmZFZBe#!?rM*_>}SWwU>(7*R=|lWeh><~X>-f4wTOIzV|s?^=hCMC|fW-!b;S zFSiN)R1^H8IqbU)!rktV^S~egz!1QJ^YFxG^;Ti($d#a&!~khEzO$$$)>K(uau^iB zk-J2iEy~$ffk2cs7(-=zf`=w`Kxn z^$*^xk1xGhbF6^h{dl1WuB;kVb0XzE*5RvK^4*mgtv-FPAPn5Wv?>{wgR>i1Rd+ov zxPAR35?tcFJrs}~bnIvhb$x2)VhLdxAY61uV(8`JiGcr!A}|a9Fa>ZB#oQ0gkT79t zlHLXj8uy`LR&|;}#)UdEY(r}tX_oW5ijR9i`Vf3DYm^l$W=ngYTAI_HsYb@&8I+ph zgCM`jMB8=0^dO^}p>P^nY4#+tkQ%d0n?h(N{q8lximNXC=jGkv9dvJ(iwNhB$dJI= z>*aW>X$pmA5>hPCBR?$MmDl}LzOT|(KdS$9@~C6q@_VMM2)UOabcIyEr&pknv_>GD z{x}5;0RYSZ95@x1VvfKN>SASw?_Sc2LbofKwaK2xYNO4A&%)a5aBbYtV(pZ*)n7la zNS2@m5~nW6p@Z_)Gk4cY^k-?xQj{4*%`d4d7%8icLpR#D=9Vrujq$Jb4M#Oyq#}9X zWYjyjb0?Rx!Vxv&;^5!wtA85nmhccxJ7hZ5@(lD+U#)HU#z(i31r1~DzSaHubb7N) z$9aXFQ`U-Z2_5yYt#}uP@Y!buuJRcQP@g>$RjPt~KK0a7)7L7m?mHwx;(R`zD(oP+ zJCK%6zRb~NSsL2b>FOqA<=HIde@3*0Pl(N`^ZT^ysbU8EUET=9iL7#_o$?@}ks8+1 zIaSrxeu1zK}y)aHyzN701!GA~0_x^Lz4vvDuvgUu8K#O@;I*`ifG zhBO4|)tgR$(c>_GFv^_X$mb%YFk9jk>5P0Ug*~5kyzzd12F&r*&fU^)jN4yUh9uB8 zd$q#r&TN#richZ(kcY1~pH_E$%+N0OY{qrZnUrPpj1)4lLcuS7HT~XOSX`x0R8#Vc ztMwda)9I;JlS%rQCE{q2L_B_UQzJY4+c>qHUHesvqq&D&3qFM*Bvj}a2*mt5_ZIGU z&K|$E5`PpR2)qFr55Nil@CASa8I&hAYrPSs0A`4HK3U!=)K@fp6jd9lRzbRp;sc|c zRVT3%dF&*fwB(f1N_)A1d$Xe17?~0yPF5RXoeX*&3PEohCJLDqHpH?ehM}8I3${kT zWn-N-Ng_U_^s#C1tIUV=&ilf%T5Jxr$=8`ZtClUv`b(eG57gKynDTgq7-Bzw;0sVXupL4C;o#rJZ+aEs;%VpNX<^{!YUg48 zr%r%j0Dvige*%=M>Cz%h9q~Pb1X@VW3fjWZ}3+zk!$CQmry4+P?&8>8Zuq6181b$|x{wijr8p||( z!bTCA@26T$9F{ib-+k`RPc9F$vTfR7n^*g-;bJrz8rDmSDzaCqj?V^~E3d(kMBHh;WwR2J1mjz$&+P{%%W^8&Vdx;Z7GiN7Z-`#n3A>8IVHu-iMf6T_$)b`} zW|7C(7cY6;M5R1Xyti$MBEDCmWg{(ukShL6S9s^s4~F%nQx7Cv(42|9-&ez&JZ*<% zHwVGz2RTe~y+6xR!Qr$~c~ga<1U@bW23~Thq?swx8H?EUS}C zR3nA#ocxIzr&wqCfBJi^_1Zn@)^?{aw*8l($0RN;axd+g46hX^8-v`DO$WgTo z8mbK_2N?y`F?mNimFV{O*pyC^G^TN@c20TZo^$b-mN)bPleKn5@|VWMW4Y%s4YkQD z?L!0AV}m#RUhBHWv{roAEn;Tm8@af{82nhe=EB@;UvWlcs9{b#3XI;!Hwo9&l~LU8 zPV_q8O2N1FFxd*hS}b|TuMw=RiE$cBlDpx#tHRj(=$);RT+!ld*)|d*lu|_z2R^+* zC{yMsm)SG_7vqaRdo7dFbjuX#YerqAi&% z-xNj7Jk{^x`=oYAb*>W~V#Yt*Z^uS3ITrp{g46t`;g&Z2Mho^^uO%?yTv*F ziygQXg?uC|G5>GFbC8#RQytwE9Hv?%`Av1ONBAqp(Vp-yhcFrFe>%j!(j0BC4%6@x z4rdM;%|Ff7UwMwU7>9XW34i|KIcPZk%5-#1Kg^^CImqIl`TegfM^}f#EJ_r=SpKqB x{FU(NG