diff --git a/0001-selecting-multiple-slides-and-using-print.-doesn-t-s.patch b/0001-selecting-multiple-slides-and-using-print.-doesn-t-s.patch new file mode 100644 index 0000000..cf36328 --- /dev/null +++ b/0001-selecting-multiple-slides-and-using-print.-doesn-t-s.patch @@ -0,0 +1,55 @@ +From c82b891dc9370d4bf72bc07752884603df7d05ac Mon Sep 17 00:00:00 2001 +From: =?UTF-8?q?Caol=C3=A1n=20McNamara?= +Date: Tue, 24 Mar 2020 14:51:09 +0000 +Subject: [PATCH] selecting multiple slides and using "print..." doesn't set + selection as range +MIME-Version: 1.0 +Content-Type: text/plain; charset=UTF-8 +Content-Transfer-Encoding: 8bit + +start impress, copy the default slide a few times, select more than +one slide and use "print..." + +"slides" is selected, but just the first selected slide is shown in the range + +since... + +commit 76e22667625ce8d16dfaa8617a199b774e987a70 +Date: Thu Jun 14 23:29:34 2018 -0300 + + Configures page range in print dialog + +where + +aPageRange = aBuf.getStr() + +was removed, but if that line is removed then the surrounding loop +of "for (auto pPage: *pPageSelection)" just accumulates text +in the OUStringBuffer to then discard it and no longer makes sense + +perhaps this was an oversight rather than a planned change. + +Change-Id: I39e76c70714ee39d189016179fb1d13a71ec74ae +Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90994 +Tested-by: Jenkins +Tested-by: Caolán McNamara +Reviewed-by: Caolán McNamara +--- + sd/source/ui/view/DocumentRenderer.cxx | 1 + + 1 file changed, 1 insertion(+) + +diff --git a/sd/source/ui/view/DocumentRenderer.cxx b/sd/source/ui/view/DocumentRenderer.cxx +index bb59ccda6b1b..b91771f623c4 100644 +--- a/sd/source/ui/view/DocumentRenderer.cxx ++++ b/sd/source/ui/view/DocumentRenderer.cxx +@@ -616,6 +616,7 @@ namespace { + else + aBuf.append(','); + aBuf.append(OUString::number(pPage->GetPageNum() / 2 + 1)); ++ aPageRange = aBuf.getStr(); + } + nPrintRange = 1; + } +-- +2.24.1 + diff --git a/libreoffice.spec b/libreoffice.spec index 12c71c2..f8861db 100644 --- a/libreoffice.spec +++ b/libreoffice.spec @@ -54,7 +54,7 @@ Summary: Free Software Productivity Suite Name: libreoffice Epoch: 1 Version: %{libo_version}.2 -Release: 2%{?libo_prerelease}%{?dist} +Release: 3%{?libo_prerelease}%{?dist} License: (MPLv1.1 or LGPLv3+) and LGPLv3 and LGPLv2+ and BSD and (MPLv1.1 or GPLv2 or LGPLv2 or Netscape) and Public Domain and ASL 2.0 and MPLv2.0 and CC0 URL: http://www.libreoffice.org/ @@ -244,6 +244,7 @@ Patch2: 0001-rhbz-1775767-null-deref.patch Patch3: 0001-Adapt-SAL_WARN-to-C-20-deleted-ostream-for-sal_Unico.patch Patch4: 0001-opengl-slide-transitions-not-working-with-glm-GLM-0..patch Patch5: 0001-rhbz-1810732-crash-assert-inserting-document-with-in.patch +Patch6: 0001-selecting-multiple-slides-and-using-print.-doesn-t-s.patch %if 0%{?rhel} # not upstreamed @@ -2147,6 +2148,9 @@ done %{_includedir}/LibreOfficeKit %changelog +* Fri Mar 27 2020 Caolán McNamara - 1:6.3.5.2-3 +- selecting multiple slides and using print dlg doesn't fill in range + * Mon Mar 23 2020 Stephan Bergmann - 1:6.3.5.2-2 - Rebuilt for Flatpak