From 5ac9ccc9e580e46a5d32931d161bbd3aa937804a Mon Sep 17 00:00:00 2001 From: Kohei Yoshida Date: Sat, 8 Jan 2011 18:11:12 -0500 Subject: [PATCH] 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. ;-) --- sw/source/ui/dbui/mmoutputpage.cxx | 4 ++-- 1 files changed, 2 insertions(+), 2 deletions(-) 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; -- 1.7.3.1