[Libreoffice-bugs] [Bug 130076] New: Flip from file format ignores angles in section, arc and cut shape

[bugzilla-daemon at bugs.documentfoundation.org]

https://bugs.documentfoundation.org/show_bug.cgi?id=130076

            Bug ID: 130076
           Summary: Flip from file format ignores angles in section, arc
                    and cut shape
           Product: LibreOffice
           Version: 6.5.0.0.alpha0+ Master
          Hardware: x86-64 (AMD64)
                OS: Windows (All)
            Status: UNCONFIRMED
          Keywords: easyHack
          Severity: normal
          Priority: medium
         Component: Draw
          Assignee: libreoffice-bugs at lists.freedesktop.org
          Reporter: rb.henschel at t-online.de

Created attachment 157246
  --> https://bugs.documentfoundation.org/attachment.cgi?id=157246&action=edit
File to reproduce the error

The file format has a draw:transform attribute, which can express horizontal
and vertical flip. It can be done with values scale() or matrix(), see attached
file.
In case an ellipse or a circle of kind "arc", "cut" or "section" is used, the
start- and the end-angle needs to be flipped too. That works without problems,
if flipping is done via UI. But is fails, if flipping is part of the file.

In the attached file, the right, green shape should be a horizontal flip of the
red shape; and the lower, blue shape should be a vertical flip of the red
shape.

The error is in file
 https://opengrok.libreoffice.org/xref/core/xmloff/source/draw/ximpshap.cxx
The transformation is applied before the kind of circle or ellipse is set.
Therefore the methods of SdrCircObj get the default kind SdrCircKind::Full and
skip all calculating on angles.

A unit test can go to /core/svx/qa/unit/classicshapes.cxx.

Re-Saving will change the draw:transform attribute. LibreOffice does not write
flipping as draw:transform attribute, but writes the changed angles. So be
careful to use a file similar to the attached file for testing.

The attached file contains only the kind "section", but you can change it in
file to "cut" or "arc" for the other cases.

I propose it as easyHack. In case you think it is not suitable, remove the
keyword.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.freedesktop.org/archives/libreoffice-bugs/attachments/20200118/bb43124b/attachment.htm>