From dfca33290a4284b4c51a4c04d27519ddab52f8ea Mon Sep 17 00:00:00 2001 From: "Crystal.Sea" Date: Sun, 26 Sep 2021 14:00:16 +0800 Subject: [PATCH] batchDelete fix --- mybatis-jpa-extra-2.5.jar | Bin 80904 -> 80935 bytes .../provider/SqlProviderDelete.java | 2 +- .../mybatis/jpa/test/MyBatisTestRunner.java | 7 +++++++ 3 files changed, 8 insertions(+), 1 deletion(-) diff --git a/mybatis-jpa-extra-2.5.jar b/mybatis-jpa-extra-2.5.jar index dd7986829dbee993908485681b38badd51b86b55..313de54c8d42d5fb5342248f4bd43cc695fbfd72 100644 GIT binary patch delta 3083 zcmZWr2{=^i8~4p1`#vVSjCC+3``E|+=Ti2pQHrroBw3m$WNcTH4nimmQW4p5%Unw& zdpB8P(p6@%~du5B6y2Z?W)E3Un!Nj=4QgAuk zvT)H9T&<=#oIT|-&M9yG(^3^B5pGYi<4iIUve-=8oxul}=MN@NSILxRXX0;+8BnRm z7P-RZhe$UcrB6;puD-)WzvRC~6}yHitfBIJE273e7ahT$I@s`I?3k5bjpCpgUf3Td z)8|T%XRc1?>@FAb8?+&--q0tIG=EGC=%AfGUT#mcEWe*3)n6u0b=JR5nJLqL+GF57{V3s(47A=fwUJdGqYL=0gK{yeyDADkPAx zqRZbb>_0aBTHQfrg63GHS5CAVHkS-gQM#3rR#IPIuCpMQ>?6%=qjqgrj8;#!^N0-} z{pfl8$FR!wT|jD2fPyd<#L2rEVItpy-lHdGYXE-4WBX|YH+B6D7&g|}$I zv4mrCGEn1A`AVycD-o{E7Uh3de|76=ox&56=!K4#&6{`v|2ygZF z=90BK4r~4ku~S%#8Jmwtg&*nBelghODjnfEWAplYPVexKtjjrlgyVeV#E-N(cAE9tb3jHcedqLj?pM1h=g&!UR-B}+HkvLMLTqrbTT&;R~<$~Z&oxR z{Y+ChC}sJB!t22ym*i{WulQQirV9)`i=2p3tJEl@R^9C*#TUX~e8lk1Vn1c`C-xck zea)~G;#E8-$hP}d{dIl}*62ndPSS;Vd1LwpIqQhMRRAFmF6BbxB97k;xz@|JObW}z zkNgg&zDO|bl+ALoefagS|GclaC@RV`bp&Z^S(|qpI#W%&?@``|_H`q&7vHf)1zjxm ze`;v7{AUK4QfQ6jzkwCza5aE*|RsQIAhjW4$2dt}kW5NWNwR z?bKfT;`Z<%_6?PWwId!DthXEbhXP3k1sVNWc6M1hisdW|_VH^KoYKAf13Y?+cT^=$C29vopkJd%b%!?Nn2^ko=oqC@Bz|j+{ zK3R^4&kQ=wZ5ZP2k+0AiW65MhDtxBn9548`4CBk!ChD!Vka7PkAP7cwRq`67en5DY zv{#&W?X+k$*}PnMp+9UHwc*W2YjkPtP(SN1#A&eRjdspRK0aB2m2UYUIUGd!{3 z*W=c$(u*a}vMV^;Pt=eJSQj4AeCdF7L43}`oqock-k-k_P&d8?3f z!p1OV_v}`udVIL=+|v3GtMN|kyt7W&Rn+ncr#B&a#Rf~Ok2Cs0hA-sy*b~ZJk<+Iq zO$XWva_$Yk(Vn!)3a6|Zw}o}GJkN@Zj~f*TJ5DPK-QteU+*CWcDM0$>L!*hr?o6w* zVot`e-$Hbt4<4so3ccOFy8CFBS)c6n@xgR^?^bsQ;zhq#if}ZGS8Hq1%e0H>ndjUh zT7s61%^KF5O?XMAAE~hPS1w069z!)bqd$=ZLYp^d+DZB^J>vLP6|B!s zHo^BLoC!@i_Om++&ooKjriHwJJZusRsD*4Mtm2}LT^D$%Pe&dl)=E^B&HMI--~5xx zK7`pu+@uL!{-9)#H8305VJpl#_jL&x}*S>5&me6Git7D zMM!?XK^W%1Pl)|+TGehYI@-^Td(BM@1dxn{o7PtE0wnQk!7lgi2x41i`Aj#@VrJfp zeUqp)nQgW3lbw_?E7H>J({a$J3m%h+DIuA0?Mn0Rb{r;h&N?#l_Ndm~lZVAdnoEa8 zm`8|B(H^V`^Zy~S`{#qSSfquR+?qQxa`8f&0MY51g%Uq+XN8S4-ovj_W$Uhh0QCeN2JIP(YJ0s5J7)0_wj;6{t-CoMvWBE|9W5 z5QUv%h6bg90hFW&(6BC6iXT>lvdyXltub&bHv>ajO28hLhJ^5FAVcX!@n7T4UrbH+UFwD^~b1WTeLRk#iT5Hd4LK)y; z5N=O4Rskj;0lKIHd_fO1qXGh8hb^H*6(C6ovE+fewE!2CqzWWKA5^MJPnBlPP>Mqb zF~ArmcXSU>e2$7j*%*3)7>|AJD2DzfCSLp6^IpOXEf?gh1|(sXe)PN|l(&9TP_-J} zWi((9a|en*99TMDB|wMOfgA-JgoLcs=w27Fbg$*meHIfYw9n2l*zyPls|YC{26`}! z$bFD~80f<^QW-#*5|_#YjbZ^GWigA7b8nWUKW-=y52J87pf$b53aN2azcHnPP2C8NdxC^2;qyaP}4fLT)3?zfvL%I;3BH$8YVP!PVlzuB7WMW!G0Xq6G DDZ(w( delta 3097 zcmYjT2{=^i8+XPojGeLX%M4~@8c*yw0iV>C9MLzccF z8v9HGPtW>W7}mzt@}+eJDVpAb*u&P2;WOHa{n2r&5~YPTpr8~woo(98wVrO;Tl?}l z71k2ht9YARKTq}D)qcn0QgGznanTJ^6`x6ikz?w0u@!C?mYWHAc)q9O28Nsq3~#;U zJ;sALljj(&Dcf*sW*DdVn&VcU{QS06+F?s7BYGEw9_|8W~szJ#Om2P%C!vn>Uq4msi13D8^Kli z)OBk!tn6S`x<)1yVb<%(S4g(@RJs>Y%BOhS+Pc@eIx0h>_}&AF{Jl=*$d^Awd_J`x zOrOayp4m54_ix;3*?H5r^I|Cf*I&Kfbl+wzft4;({l(2TNex%G1=z|0u`5A!ewNNP zmrsUq$YG=7RiWntw)W198-v9{QOt7Ac4PuBDlLGujjaIYXlU0mptzeRq>B%yidJx^ei;E=qN%DBmp$CsMQT*YLWNLAnqw{9C|(~}F$KevI{{P{R$ zKxwz-4w_-ZDBSclDA&D3bT6|SRAHsq_8{@7#g6X+MM@q%_!zVwA4iz&34hlBix`iV zd++5Qn<9)NN}BGCJh?;G45sRo-Ve-nONewCbU_R8voecbe)9N}!1j{0O2xRbT+@Zo zLbvI1A$~{n;*3hdMUsbK#f*6B`eQtc5HI#%%#v!SR`bJh^$5N+WPdp@e! zkr*Qx;s>4U4q0(cVk^WCnaqhH6Urhw8Na;?tUEaq_l^E_d}qL=iP3ANEDPOTL9+sH zjVX`D?UG#;Wa?`#o;Qur_3>-Vw2Rrf<9=@M+?^k}o@Z~}?Hd)YQGP;}eh_49vHV=; zeP~sVDQJE-1rzHU9;?$GPTsUD>1v+R}V@@m)2$B}x9XlRR4W`Ae5aFp1@J zI^n0fLI@VGcWhgtO!@U+-9(_vccq*%s_LfCcBKY5b)|0dver-QZrlFgL-+08Pdb9N z3~vA4F)40GI2%;Qd4y<{KKvcUxmdMvZ9mYZW}f6ylUmkr8?>9c6>{2Yx+p$Tn6ta# z@!aR#*d$zK)dj>&icnxhatcqu9J2DPD*w}&rlj+mX8cbR|N6!D)NBdyLCJ8=zm@m2 zv-bJ+afh+)c)K6MFK4$Cl0MFbTtxctb);ij+i&rFds&Q>o!>zce@5w7&b+@1lnf@Z zJ5PLAQm`j#Ggk$<8d7URgSKe zJxOT<@3g+PIFnQ9yxMX7O$!1&t9{j|^T@?=a+fqh%wKc5x9pmXMPrt8Y-YfD!7;zR zbzUlJQEnwA$RZ?^s3Dz~^6Xbm^Xi?}z5dN2){?*OmSign1ow?o6t$zrJGavtJ9Rlc z-N7k0M)Tk5Uw$P9b_i3I9MzLDJ3{X=KFK+!AfzUX@m0>uVP_h*)OwhK8Oh9ZYzc~q z=PAR>tLdwSxi+cXXu^DC%ZsS5&JDwabRN%2;G_1hhNha!6Wx_MC%z2`S&Wm(!;^~M zYiWiPj+IAaB$iGrA%_js92y&xs(%Q3Ek4@qzA%VX-?yLKBNtla6%N}YxK!})kowLl zt*=+woC3l#T8^Vee2TeEMPzy^DSTDi)jjhm_uVJ&bJ0zPd9js^YS~j z%T}}V#UwI>h%?$NPLcSnY_8y375HRJ<8Di8tJ({D%B)xuY*|{QwISa!U%f#LZeZ(A zrCUF7XFZ)?hQJ)EZ#*P3^YHW1`uvzkvnu7YaA?>DPItKD{A?@3$KY=2Qv>ILOQ&Xd zV!}r$PA4zH(LG|nexLJ&dHxpDfc@#{R;b9IL&xcSVdpQn?x6E@JI;$8J_JSSrmD`rD z!`u|EyNZ%F^$^r(;tb{GFa>dj&7!uqC&~svM^p^u54rSuRqz3fQ1oy`^2L|k$UKJ@ ziT0g6&VbPw;@0QcU+{l_&=nxBCcs-={gpru4IkjH44^?9KkCu4K)+(00~HwsDf}X z1qrZ&vr2$C5JN@+PB0Cr0}YHp0$Q*Yc^Z%cCy~H$m^$hJko-|RhpX&SfH=$ub7;v% zK^9ZZL$Hqm3}IP1hrmf0Fn~qsfq)8t1_zV@4A4tf0X$$wjcJrP$=jF{%v6EC^9FpS z0@%ZnOrT7~K&&dD2a~sg-nGC=Rj7iS4$!+Osmg(uhS@=WH1v7Hi~lDXNZ}WG4g~{k zfB_6aLmer1hExRUgR=zKhlWCodC)M(g#k{&*1bq=0dgetRYA~82jC*@`*N`#%zMz| zVf3o+fvV!iM^o8B10)oaNevJNNa}qx+IiywVOXgDpJopXsaSxCq(8?2p2k8UZted~8UXLWK`h`1yTJku zVF7V4iXBRSng!qi2cRthZ|XoX6xjdKL}@eiv0frT_gFLQvfAX*V%=VN4|wd@3RD+{ ArrayList idValues=(ArrayList)parametersMap.get("idList"); String keyValue=""; for(String value : idValues) { - if(value.trim().length() > 1) { + if(value.trim().length() > 0) { keyValue += ",'" + value + "'"; _logger.trace("batch delete by id " + value); } diff --git a/mybatis-jpa-extra-core/src/test/java/org/apache/mybatis/jpa/test/MyBatisTestRunner.java b/mybatis-jpa-extra-core/src/test/java/org/apache/mybatis/jpa/test/MyBatisTestRunner.java index 222a496..9fafcc1 100644 --- a/mybatis-jpa-extra-core/src/test/java/org/apache/mybatis/jpa/test/MyBatisTestRunner.java +++ b/mybatis-jpa-extra-core/src/test/java/org/apache/mybatis/jpa/test/MyBatisTestRunner.java @@ -136,6 +136,13 @@ public class MyBatisTestRunner { idList.add("dafd5ba4-d2e3-4656-bd42-178841e610fe"); service.batchDelete(idList); } + + @Test + public void batchDeleteByIds() throws Exception{ + _logger.info("batchDeleteByIds..."); + service.batchDelete("2"); + service.batchDelete("2,639178432667713536"); + } @Test public void queryPageResults() throws Exception{ -- GitLab