[Bug 64716] New: wmf display error

classic Classic list List threaded Threaded
17 messages Options
Reply | Threaded
Open this post in threaded view
|

[Bug 64716] New: wmf display error

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

            Bug ID: 64716
           Summary: wmf display error
           Product: POI
           Version: 4.1.2-FINAL
          Hardware: PC
                OS: Mac OS X 10.1
            Status: NEW
          Severity: normal
          Priority: P2
         Component: XSLF
          Assignee: [hidden email]
          Reporter: [hidden email]
  Target Milestone: ---

Created attachment 37430
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37430&action=edit
the original wmf and office correctly rendered png.

hi,
the attached wmf files can't display correctly.

thanks.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

Andreas Beeker <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |NEEDINFO

--- Comment #1 from Andreas Beeker <[hidden email]> ---
The rendered results looks good on the trunk on my machine. Actually I think
your 2.png contains an error - 2'80 is probably wrong and should be 2x80 (2*h
where h = 80).

Please follow the instruction link below and comment in this ticket, if it
works or not:

http://poi.apache.org/components/slideshow/ppt-wmf-emf-renderer.html#Instructions+to+run

If it doesn't work, you can dump the wmf with the "-dump <file>" option and
check the font records for the font typefaces - those should be available to
your Java environment.

While doing the above, I noticed that one font name seems to be erroneous read
by POI and I'll check how it's read correctly. Apart of that the fonts "Times
New Roman", "Symbol", "System" plus the font not correctly interpreted.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

Andreas Beeker <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|NEEDINFO                    |RESOLVED

--- Comment #2 from Andreas Beeker <[hidden email]> ---
Fixed the font typeface string loading via r1881567
The charset of the font is now used to read the typeface name.

Apart of that, the 宋体 font (i.e. the simsun font) was the missing font.
https://github.com/sonatype/maven-guide-zh/blob/master/content-zh/src/main/resources/fonts/simsun.ttc

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #3 from liming-coder <[hidden email]> ---
Created attachment 37455
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37455&action=edit
sample2.zip

thanks every much. i found some other wmf display error. see attachment
sample2.zip

thanks.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

liming-coder <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|FIXED                       |---
             Status|RESOLVED                    |REOPENED

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

liming-coder <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #37430|the original wmf and office |sample1.zip
        description|correctly rendered png.     |

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #4 from liming-coder <[hidden email]> ---
sample1.zip is fixed in the nightly trunk.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

Andreas Beeker <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
         Resolution|---                         |FIXED
             Status|REOPENED                    |RESOLVED

--- Comment #5 from Andreas Beeker <[hidden email]> ---
Thank you for the sample2 - fixed it via r1881750

the current location is now moved to the end of the rendered string.

you need to download the euclid symbol font, for the braces to be displayed
proper

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #6 from liming-coder <[hidden email]> ---
Created attachment 37456
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37456&action=edit
sample3_emf.zip

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

liming-coder <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--- Comment #7 from liming-coder <[hidden email]> ---
hi,

same2.zip now displays ok in the nightly trunk.

sample3_emf.zip contains an emf picture which throws exception with the
following error:
org.apache.poi.util.RecordFormatException: Undefined record of type: 2596720087
at 0
        at
org.apache.poi.hemf.record.emf.HemfRecordIterator._next(HemfRecordIterator.java:74)
        at
org.apache.poi.hemf.record.emf.HemfRecordIterator.<init>(HemfRecordIterator.java:38)
        at
org.apache.poi.hemf.usermodel.HemfPicture.getRecords(HemfPicture.java:81)
        at
org.apache.poi.hemf.usermodel.HemfPicture.getHeader(HemfPicture.java:72)
        at
org.apache.poi.hemf.usermodel.HemfPicture.getBoundsInPoints(HemfPicture.java:145)
        at
org.apache.poi.hemf.usermodel.HemfPicture.getSize(HemfPicture.java:154)

my code:
 HemfPicture emf = new HemfPicture(fis);
 Dimension2D dim = emf.getSize();
 ...

thanks.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

Andreas Beeker <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|REOPENED                    |RESOLVED
         Resolution|---                         |FIXED

--- Comment #8 from Andreas Beeker <[hidden email]> ---
(In reply to liming-coder from comment #7)
> sample3_emf.zip contains an emf picture which throws exception with the
> following error ...

Easy fix ... rename it to .wmf.
I've replaced the Renderer selection in PPTX2PNG based on the FileMagic via
r1881838

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #9 from liming-coder <[hidden email]> ---
Created attachment 37524
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37524&action=edit
sample4.zip

hi,
i've collected some wmf/emf images which also have some problems. see
sample4.zip,sample5.zip.

thanks.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #10 from liming-coder <[hidden email]> ---
Created attachment 37525
  --> https://bz.apache.org/bugzilla/attachment.cgi?id=37525&action=edit
sample5.zip

sample5.zip

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

liming-coder <[hidden email]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |REOPENED
         Resolution|FIXED                       |---

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #11 from Andreas Beeker <[hidden email]> ---
Applied via r1883051

EMF: workaround for invalid EMF header bounds
EMF: add option to PPTX2PNG / DrawableHint to fallback to force EMF header
bounds
EMF: use RGB instead of HSL gradiants Subversion

The offset issues in emf1 and emf2 really took me a while to figure out and now
its unfortunately just a compromise/workaround which often happens to be ok.

For EMFs which make use of inner window / viewport bounds to scale parts
completely different to the header bounds this doesn't work and hence I've
introduced another rendering hint DrawableHint.EMF_FORCE_HEADER_BOUNDS
(boolean) to force the usage of the header bounds, i.e. ignore the inner bounds
/ records.

Apart of that, I've replaced the HSL gradient calculation with RGB gradients.
Although HSL looks much better - in my point of view - it looks like Powerpoint
use RGB ...

There are still other issues, which I hope to fix soon ... you can always
test-drive the nightly for the current fix

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #12 from liming-coder <[hidden email]> ---
thanks Andreas. i will try this weekend.

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]

Reply | Threaded
Open this post in threaded view
|

[Bug 64716] wmf display error

Bugzilla from bugzilla@apache.org
In reply to this post by Bugzilla from bugzilla@apache.org
https://bz.apache.org/bugzilla/show_bug.cgi?id=64716

--- Comment #13 from Andreas Beeker <[hidden email]> ---
applied via r1883460

- close polygons
- reset current path location on emf modifyworldtransform
- mirror text if the AffineTransform of the graphics context is negative scaled

JDKs > Java 8 don't fallback to one of the builtin fonts, if the font is not
installed - *on my machine*. But I've tested on the Apache Jenkins with other
environments (Java8-15) and there it seems to be ok.

(to be continued ...)

--
You are receiving this mail because:
You are the assignee for the bug.
---------------------------------------------------------------------
To unsubscribe, e-mail: [hidden email]
For additional commands, e-mail: [hidden email]