Apache OpenOffice (AOO) Bugzilla – Issue 119535
[From Symphony]Trendline of two Data Series can't be displayed but after edit it the trendline will be showed correctly
Last modified: 2012-12-26 08:30:05 UTC
Created attachment 77743 [details] sample file build info:AOO3.4_1327774 steps: 1.open the sample file in AOO3.4 2.in chart object the two trend lines of two data series are lost. Only after double clicking it to change to edit mode,then the trend line will be shown correctly.
confirmed,change its status
Looking at this issue...
Root cause: Line brush command was not parsed in emf picture filter Resolution: Resolve it Limitation: Line style in sample file is not supported by aoo. So its line style will be imported as SOLID instead.
Created attachment 78871 [details] Patch for this issue
The patch looks good and reviewed.
"lijiany" committed SVN revision 1373702 into trunk: #119535# Fix WMF image with type pattern brushReported by: Du JingPatch by: J...
UT is made. The patch looks good.
@Jianyuan Li: Reopening because the change breaks the build (Windows7, non-pro) in main/svtools/source/filter/wmf/enhwmf.cxx. There are two problems: 1. On line 1318 a new variable aColor is introduced. This is inside a switch/case statement. Adding curly braces around the whole case will probably fix this. 2. There are now two case statements for EMR_CREATEDIBPATTERNBRUSHPT. One in the new code, the other inside the following #ifdef WIN_MTF_ASSERT block. Removing the second case statement will probably fix this.
@Andre: Sorry for the break. I have a try in Win7 with cygwin with this patch. No break happened. So can you tell me what you do when build(configure parameter...)?
Created attachment 78991 [details] patch for win7 build break
The possible issues has been patched.
@Jianyuan Li: Don't worry about the build break. If you don't cause a build break now and then, the you don't work hard enough :-) Your patch works and looks good. svtools is compilable again. Please go ahead and commit it. I can reproduce both problems on Windows 7 and cygwin with non-pro build.
Comitted in Revision: 1374181
Win7 build break issue has been fixed
Verified pass with AOO trunk r1374181.