Author: dpsenner Date: Fri Nov 27 20:23:25 2015 New Revision: 1716913 URL: http://svn.apache.org/viewvc?rev=1716913&view=rev Log: LOG4NET-495: adjusted the way how the parameters are cleared so that the AdoNetAppender tests pass
Modified: logging/log4net/trunk/src/Appender/AdoNetAppender.cs Modified: logging/log4net/trunk/src/Appender/AdoNetAppender.cs URL: http://svn.apache.org/viewvc/logging/log4net/trunk/src/Appender/AdoNetAppender.cs?rev=1716913&r1=1716912&r2=1716913&view=diff ============================================================================== --- logging/log4net/trunk/src/Appender/AdoNetAppender.cs (original) +++ logging/log4net/trunk/src/Appender/AdoNetAppender.cs Fri Nov 27 20:23:25 2015 @@ -551,25 +551,22 @@ namespace log4net.Appender dbCmd.Transaction = dbTran; } // prepare the command, which is significantly faster - dbCmd.Prepare(); - // run for all events - foreach (LoggingEvent e in events) - { - // Set the parameter values - foreach (AdoNetAppenderParameter param in m_parameters) - { - param.Prepare(dbCmd); - param.FormatValue(dbCmd, e); - } - - // Execute the query - dbCmd.ExecuteNonQuery(); - - // clear parameters that have been set - if (events.Length > 0) - { - dbCmd.Parameters.Clear(); - } + dbCmd.Prepare(); + // run for all events + foreach (LoggingEvent e in events) + { + // clear parameters that have been set + dbCmd.Parameters.Clear(); + + // Set the parameter values + foreach (AdoNetAppenderParameter param in m_parameters) + { + param.Prepare(dbCmd); + param.FormatValue(dbCmd, e); + } + + // Execute the query + dbCmd.ExecuteNonQuery(); } } }