Bug 81944

bibisect range:
24bd259451fae36c993977f52d62365fa63ac87c..8a89a732522f90d68f1b1c84c9c34c666f2eb882

regression from:
commit 0805b222f87bf99ec0c53ca678d1c670eb5293a2
Author:     Miklos Vajna <vmiklos@suse.cz>
AuthorDate: Tue Feb 26 15:36:09 2013 +0100

I should have previously mentioned that i've seen more items in the RTF not retain their font name in 4.3.1 than 4.2.5 and below. An example of this is step 3, which only showed up in 4.3.1.

Michael Stahl committed a patch related to this issue.
It has been pushed to "master":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=f8e8e476aa0a5576234dbb4daac9dc299e044e61

fdo#81944: writerfilter: RTF import: don't drop fonts in stylesheet



The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

funny thing is, it appears to work in current libreoffie-4-2 branch...

then this one broke it again:

commit 1be0a3fa9ebb22b607c54b47739d4467acfed259
Author:     Michael Stahl <mstahl@redhat.com>
AuthorDate: Tue Jun 17 18:40:04 2014 +0200

    n#825305: writerfilter RTF import: override style properties like Word
  

good thing we decided not to backport that earlier today...

so the problem appears to be that the default style's fonts
are not imported, or at least fixing that makes the right font
show up in the locations from the description:

on master:
para style Default Style
 Western: Times New Roman
 CJK: Courier New

Word:
para style Normal
 Western: Segoe UI
 CJK/Complex: Times New Roman

probably if this ever worked before it was by accident...

Created attachment 103872 [details]
How the first page Word RTF looks in LibO 4.2.5 VS 4.3.1

Everything after 3.6.7 has not worked flawlessly from the test documents i've been testing and things have shown to gotten worse between 4.2.5 and 4.3.1. I came across another file that was fine in 4.2.5 and then in 4.3.1, Times New Roman patches showed up here and there. You can see it by converting attachment  103815 [details] to RTF in Word 2013 and then opening it up in LibO.

Hi Michael,

When you do get a chance, can you check out bug 82033, as it is about RTF exporting attachment 103815 [details], which results in a 169mb file.

Michael Stahl committed a patch related to this issue.
It has been pushed to "libreoffice-4-3":

http://cgit.freedesktop.org/libreoffice/core/commit/?id=aed0f1e27be04ecd43b2edd3885c1f2eea0a3de9&h=libreoffice-4-3

fdo#81944: writerfilter: RTF import: don't drop fonts in stylesheet


It will be available in LibreOffice 4.3.1.

The patch should be included in the daily builds available at
http://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
http://wiki.documentfoundation.org/Testing_Daily_Builds
Affected users are encouraged to test the fix and report feedback.

Created attachment 104064 [details]
sample file

Hi Michael,

Thanks for the fix. It has worked perfectly on the file for this bug report.

Unfortunately, it has messed up another rtf that i've been testing these last few days. I have included the part that has gotten messed up in the last week. It is on page 2, all the text in the 2 frames used to be verdana and they are are now times new roman.

Also in the file on page 1, the numbering list numbers are supposed to be Verdana, but instead they are using Liberation Serif.

*** Bug 82074 has been marked as a duplicate of this bug. ***

Changing the status as the fix messed up another file.

Or should i create another bug report for the new file Michael?

Yes, please open a new bug. Thanks! :-)

*** Bug 82107 has been marked as a duplicate of this bug. ***

(In reply to comment #12)
> Yes, please open a new bug. Thanks! :-)

The new bug is bug 83464. :D