[Libreoffice-commits] core.git: Branch 'libreoffice-6-4' - sd/source

[Caolán McNamara (via logerrit)]

 sd/source/ui/view/DocumentRenderer.cxx |    1 +
 1 file changed, 1 insertion(+)

New commits:
commit f8e2fe46fa7cd2df90853b0aca31cfda49a06d3d
Author:     Caolán McNamara <caolanm at redhat.com>
AuthorDate: Tue Mar 24 14:51:09 2020 +0000
Commit:     Adolfo Jayme Barrientos <fitojb at ubuntu.com>
CommitDate: Mon Mar 30 21:56:54 2020 +0200

    selecting multiple slides and using "print..." doesn't set selection as range
    
    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/+/91169
    Tested-by: Jenkins
    Reviewed-by: Adolfo Jayme Barrientos <fitojb at ubuntu.com>

diff --git a/sd/source/ui/view/DocumentRenderer.cxx b/sd/source/ui/view/DocumentRenderer.cxx
index 30fe413e5d83..5faa56a82140 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;
                 }