diff options
author | Michael Weghorn <m.weghorn@posteo.de> | 2019-07-19 08:16:54 +0200 |
---|---|---|
committer | Michael Weghorn <m.weghorn@posteo.de> | 2019-07-19 19:08:43 +0200 |
commit | 0f0adc34889ff68da584cea97a9de649174b0333 (patch) | |
tree | bd1991e818e9daf9fbc4f6535605747b3da0942b | |
parent | 3bc1f99d3a792cdba037bcc757e8611d7e187b73 (diff) |
tdf#126471 Apply selected MM printer on dialog open
Calling 'PrinterChangeHdl_Impl' in
'SwMMResultPrintDialog::FillInPrinterSettings' makes
sure that a preselected printer is actually taken
into account (i.e. 'm_pTempPrinter' set accordingly
etc.) as soon as the dialog is created, not only when
explicitly selecting another printer.
Since 'PrinterChangeHdl_Impl' is now called initially
and every time another printer is selected, there's no
more need to do so in 'PrinterSetupHdl_Impl' any more.
This fixes the issue that printout would always
happen on default printer instead of the preselected one
when SwMMResultPrintDialog was opened a second time.
Change-Id: I5c930ed56d1025a7e94cc98fb70c01612bda0d22
Reviewed-on: https://gerrit.libreoffice.org/75933
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
-rw-r--r-- | sw/source/ui/dbui/mmresultdialogs.cxx | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/sw/source/ui/dbui/mmresultdialogs.cxx b/sw/source/ui/dbui/mmresultdialogs.cxx index 741cd9baab49..8a4201d904c8 100644 --- a/sw/source/ui/dbui/mmresultdialogs.cxx +++ b/sw/source/ui/dbui/mmresultdialogs.cxx @@ -342,6 +342,7 @@ void SwMMResultPrintDialog::FillInPrinterSettings() { m_xPrinterLB->set_active_text(xConfigItem->GetSelectedPrinter()); } + PrinterChangeHdl_Impl(*m_xPrinterLB); sal_Int32 count = xConfigItem->GetMergedDocumentCount(); m_xToNF->set_value(count); @@ -782,8 +783,6 @@ IMPL_LINK_NOARG(SwMMResultPrintDialog, PrintHdl_Impl, weld::Button&, void) IMPL_LINK_NOARG(SwMMResultPrintDialog, PrinterSetupHdl_Impl, weld::Button&, void) { - if (!m_pTempPrinter) - PrinterChangeHdl_Impl(*m_xPrinterLB); if (m_pTempPrinter) m_pTempPrinter->Setup(m_xDialog.get()); } |