[kmymoney] [Bug 412366] Print to File (PDF) broken on KDE4 in 4.8.4

[Ralf Habacker]

https://bugs.kde.org/show_bug.cgi?id=412366

--- Comment #3 from Ralf Habacker <ralf.habacker at freenet.de> ---
(In reply to Erich from comment #0)
> What I believe is the cause of the problem:
> The net result of patches:
> 5f4f90e83565647f3579660e4c5d9ac447547a02 Fix 'Printer settings are not saved'
> 15ac7d472af41fc503dc5209643cfc8b392b0089 Add print support for report charts
> Changes the "print" command from
> m_part->view()->print()
> to
> q->m_part->view()->print(true);
> when KDE_IS_VERSION(4, 14, 65) is false (which is the case for me).
    /**
     * Prints the HTML document.
     * @param quick if true, fully automated printing, without print dialog
     */
    void print( bool quick = false );

> From what I can tell from reading the docs (I am not a KDE developer), this
> changes the "print" command from "print by asking the user how to print" to
> "print using the default settings".  Unfortunately, those "default settings"
> are not what KMyMoney asked about in the code immediately preceding the
> "print" command, they must be the system-wide defaults.  
Thanks for reporting this hidden issue.

> My minimally-invasive patch simply changes the print command back to what it
> used to be, with the net effect that the print dialog appears twice when
> printing a report.
This could be fixed with the patch from attachment 122884.

> I also developed a small patch which restores Print to File (PDF)
> functionality for me, but the print dialog appears twice (I did expect that
> to happen when I tried out the patch).  So it is not a proper patch.
which is required as fallback to avoid this issue, if the mentioned patch is
not applied to kdelibs4.

-- 
You are receiving this mail because:
You are the assignee for the bug.