darkgoon83 / log4jdbc-remix

Automatically exported from code.google.com/p/log4jdbc-remix
0 stars 0 forks source link

bug in logging batch execute #5

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago

You have added following method into class Slf4jSpyLogDelegator
    public String sqlOccured(Spy spy, String methodCall, String[] sqls)

with bug that makes impossible to log SQL's

line 259
sqlOccured(spy, sql, methodCall);

should be

sqlOccured(spy, methodCall, sql);

It is quite serious so i'm waiting for 0.2.6

I belive that it was stupid mistake :)
By the way very good idea to deploy it into maven repository

Daniel Charczyński
danielcharczynski@o2.pl

Original issue reported on code.google.com by danielch...@gmail.com on 25 Nov 2011 at 5:49

GoogleCodeExporter commented 8 years ago

Original comment by tim.azzo...@gmail.com on 26 Nov 2011 at 1:36

GoogleCodeExporter commented 8 years ago
Hi Daniel

Thank you for finding the bug and suggesting the solution.

Its fixed in release 0.2.6.

Original comment by tim.azzo...@gmail.com on 26 Nov 2011 at 2:04

GoogleCodeExporter commented 8 years ago

Original comment by tim.azzo...@gmail.com on 26 Nov 2011 at 2:04

GoogleCodeExporter commented 8 years ago
Dnia sobota, 26 listopada 2011 o 03:04:27 napisałeś:

I think that you miss somethig when you were deploying 0.2.6 into releases 
repository - there is no jar file

Thanks for quick response

Daniel Charczyński

Original comment by danielch...@gmail.com on 26 Nov 2011 at 12:57

GoogleCodeExporter commented 8 years ago
Hmmm not sure what happened with the maven release, I will try again.

Original comment by tim.azzo...@gmail.com on 26 Nov 2011 at 1:16