[Libreoffice-commits] .: sw/source

[Caolán McNamara]

 sw/source/ui/dbui/mmoutputpage.cxx |    4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 20b3b54b246159baeac941fe0acdeb61cc5cd112
Author: Kohei Yoshida <kyoshida at novell.com>
Date:   Sun Jan 9 14:06:58 2011 +0000

    We need to move to the next record during mail merge. (fdo#32790)
    
    I believe this code change was unintentional.  We shouldn't be
    skipping MoveResultSet call or else all the emails would be sent to
    the first person unintentionally. ;-)

diff --git a/sw/source/ui/dbui/mmoutputpage.cxx b/sw/source/ui/dbui/mmoutputpage.cxx
index 0d9b4be..a99533d 100644
--- a/sw/source/ui/dbui/mmoutputpage.cxx
+++ b/sw/source/ui/dbui/mmoutputpage.cxx
@@ -1207,9 +1207,9 @@ IMPL_LINK(SwMailMergeOutputPage, SendDocumentsHdl_Impl, PushButton*, pButton)
 
 #ifdef DBG_UTIL
         sal_Int32 nTarget =
-                rConfigItem.MoveResultSet(rInfo.nDBRow);
-        OSL_ENSURE( nTarget == rInfo.nDBRow, "row of current document could not be selected");
 #endif
+            rConfigItem.MoveResultSet(rInfo.nDBRow);
+        OSL_ENSURE( nTarget == rInfo.nDBRow, "row of current document could not be selected");
         OSL_ENSURE( sEMailColumn.Len(), "No email column selected");
         ::rtl::OUString sEMail = lcl_GetColumnValueOf(sEMailColumn, xColAccess);
         SwMailDescriptor aDesc;