Blame SOURCES/0001-selecting-multiple-slides-and-using-print.-doesn-t-s.patch

697d25
From c82b891dc9370d4bf72bc07752884603df7d05ac Mon Sep 17 00:00:00 2001
697d25
From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= <caolanm@redhat.com>
697d25
Date: Tue, 24 Mar 2020 14:51:09 +0000
697d25
Subject: [PATCH] selecting multiple slides and using "print..." doesn't set
697d25
 selection as range
697d25
MIME-Version: 1.0
697d25
Content-Type: text/plain; charset=UTF-8
697d25
Content-Transfer-Encoding: 8bit
697d25
697d25
start impress, copy the default slide a few times, select more than
697d25
one slide and use "print..."
697d25
697d25
"slides" is selected, but just the first selected slide is shown in the range
697d25
697d25
since...
697d25
697d25
commit 76e22667625ce8d16dfaa8617a199b774e987a70
697d25
Date:   Thu Jun 14 23:29:34 2018 -0300
697d25
697d25
    Configures page range in print dialog
697d25
697d25
where
697d25
697d25
aPageRange = aBuf.getStr()
697d25
697d25
was removed, but if that line is removed then the surrounding loop
697d25
of "for (auto pPage: *pPageSelection)" just accumulates text
697d25
in the OUStringBuffer to then discard it and no longer makes sense
697d25
697d25
perhaps this was an oversight rather than a planned change.
697d25
697d25
Change-Id: I39e76c70714ee39d189016179fb1d13a71ec74ae
697d25
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90994
697d25
Tested-by: Jenkins
697d25
Tested-by: Caolán McNamara <caolanm@redhat.com>
697d25
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
697d25
---
697d25
 sd/source/ui/view/DocumentRenderer.cxx | 1 +
697d25
 1 file changed, 1 insertion(+)
697d25
697d25
diff --git a/sd/source/ui/view/DocumentRenderer.cxx b/sd/source/ui/view/DocumentRenderer.cxx
697d25
index bb59ccda6b1b..b91771f623c4 100644
697d25
--- a/sd/source/ui/view/DocumentRenderer.cxx
697d25
+++ b/sd/source/ui/view/DocumentRenderer.cxx
697d25
@@ -616,6 +616,7 @@ namespace {
697d25
                         else
697d25
                             aBuf.append(',');
697d25
                         aBuf.append(OUString::number(pPage->GetPageNum() / 2 + 1));
697d25
+                        aPageRange = aBuf.getStr();
697d25
                     }
697d25
                     nPrintRange = 1;
697d25
                 }
697d25
-- 
697d25
2.24.1
697d25