Created attachment 138877 [details] sample Steps to reproduce: 1. Open attached document 2. Check the chart. it's an ellipse instead. See the image attached Reproduced in Version: 6.1.0.0.alpha0+ Build ID: 2bf1cc7372088ec31ac5f0fb60de57feda59d3b7 CPU threads: 4; OS: Linux 4.10; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); Calc: group threaded [Bug found by office-interoperability-tools]
Created attachment 138878 [details] comparison MSO 2010 and LibreOffice 6.1
Regression introduced by: author Tomaž Vajngerl <tomaz.vajngerl@collabora.co.uk> 2017-12-13 22:03:12 +0900 committer Tomaž Vajngerl <quikee@gmail.com> 2017-12-18 12:37:41 +0100 commit e1697600253361a26d77a1ef61f8bde16af0ed2c (patch) tree 533a2baeea5a71b6977d90d63f71d986c8394d73 parent beb091666aa325376b4ad6d1ff68c7c3ae82fcd4 (diff) chart2: Batch changes to properties when creating 3D objects MultiPropertySet doesn't broadcast ItemSets as often as setting each property separately. This can help when we create a lot of 3D objects. Bisected with: bibisect-linux64-6.1 Adding Cc: to Tomaž Vajngerl
attachment 94196 [details] from bug 74111 is also affected.
*** Bug 117705 has been marked as a duplicate of this bug. ***
An ODT example from bug 117705: attachment 142205 [details] The change to impl_createCube(...) is what results in the different behavior, but the actual cause must be elsewhere, perhaps in how MultyPropertySet is handled for the shape vs PropertySet.
The Shape UNO API implementation for the XMultiPropertySet::setPropertyValues implementation was a bit problematic. Instead of only changing the properties for the few properties that were actually set, the implementation was always setting every single property again. This led to some slightly different calls than by calls to XPropertySet::setPropertyValue. I have a fix that actually changes the behavior of the XMultiPropertySet::setPropertyValues to only set the changed properties.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "master": https://git.libreoffice.org/core/+/80cf278d365a2d357c70b8c28947c91fa97e7a99%5E%21 tdf#114836, only set changed SfxItemSet properties It will be available in 6.3.0. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "libreoffice-6-1": https://git.libreoffice.org/core/+/b3af23568f9b9b444ae09f2a1ab4a61d212b422a%5E%21 tdf#114836, only set changed SfxItemSet properties It will be available in 6.1.4. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Markus Mohrhard committed a patch related to this issue. It has been pushed to "libreoffice-6-2": https://git.libreoffice.org/core/+/d4190685ac208677bc77f66976287dda0360c42d%5E%21 tdf#114836, only set changed SfxItemSet properties It will be available in 6.2.0.1. The patch should be included in the daily builds available at https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More information about daily builds can be found at: https://wiki.documentfoundation.org/Testing_Daily_Builds Affected users are encouraged to test the fix and report feedback.
Verified in Version: 6.2.0.0.beta1+ Build ID: a63cd8bbe7cf881daa8dc7a7f32f3e5ac384e902 CPU threads: 4; OS: Linux 4.15; UI render: default; VCL: gtk3; Locale: ca-ES (ca_ES.UTF-8); UI-Language: en-US Calc: threaded @Markus Mohrhard, thanks for fixing this!!