diff options
| author | Michael Weghorn <m.weghorn@posteo.de> | 2020-05-05 10:36:28 +0200 |
|---|---|---|
| committer | Michael Weghorn <m.weghorn@posteo.de> | 2020-05-05 16:21:19 +0200 |
| commit | 00ba164a6ef36c0adb3ff73c6f1c6e0c4191c1cb (patch) | |
| tree | fb1a1009816a35b048ffe2ab62532357586aa04f /sw | |
| parent | 7821cdcc9a08e23408531fcfaf68c02f0556c969 (diff) | |
SwMMResultEmailDialog::FillInEmailSettings: Turn check into assert
Since SwMMResultEmailDialog::FillInEmailSettings is only called
from the SwMMResultEmailDialog ctor, 'm_xMailToLB->get_count()'
should always return 0 here.
Change-Id: I156552c17f5a53f6049d0a25231ba91143fdfe62
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/93458
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Diffstat (limited to 'sw')
| -rw-r--r-- | sw/source/ui/dbui/mmresultdialogs.cxx | 7 |
1 files changed, 3 insertions, 4 deletions
diff --git a/sw/source/ui/dbui/mmresultdialogs.cxx b/sw/source/ui/dbui/mmresultdialogs.cxx index adbc284c5b97..3794f9f861c9 100644 --- a/sw/source/ui/dbui/mmresultdialogs.cxx +++ b/sw/source/ui/dbui/mmresultdialogs.cxx @@ -406,10 +406,6 @@ void SwMMResultEmailDialog::FillInEmailSettings() } - //fill mail address ListBox - if (m_xMailToLB->get_count()) - return; - //select first column uno::Reference< sdbcx::XColumnsSupplier > xColsSupp(xConfigItem->GetResultSet(), uno::UNO_QUERY); //get the name of the actual columns @@ -417,6 +413,9 @@ void SwMMResultEmailDialog::FillInEmailSettings() uno::Sequence< OUString > aFields; if (xColAccess.is()) aFields = xColAccess->getElementNames(); + + // fill mail address ListBox + assert(m_xMailToLB->get_count() == 0); for (const OUString& rField : std::as_const(aFields)) m_xMailToLB->append_text(rField); |