<div dir="ltr">Hello all, <div><br></div><div>I've been investigating on this issue and I've tried the patch from Julien and it still doesn't seem to fix the problem. There unit test always fails on the equality assertion even if we reverse that as well. </div>
<div><br></div><div>Here is what I've got: </div><div><br></div><div>When we check against sal_Int32(0xD99594) (i.e. 14259604 in decimal) for start color aGradient.StartColor <br></div><div><div>equality assertion failed</div>
<div>- Expected: 14259604</div><div>- Actual : 12603469</div></div><div><br></div><div>and When we check against sal_Int32(0xC0504D) (i.e. 12603469 in decimal) for start color aGradient.StartColor</div><div><div>equality assertion failed</div>
<div>- Expected: 12603469</div><div>- Actual : 14259604</div></div><div><br></div><div>That is, whenever we reverse the hex value (the expected value) for the unit test, the actual value coming out of the test is surprisingly reversed as well!! Which I failed to explain... </div>
<div><br></div><div>I have to assume that there are other filters that are tested against this unit test (which in this case needs to reverse the startColor/endColor as I have done in the ooxmlexport filter) or there are some hidden mechanism in this unit test that changes the actual value when we change the expected value... </div>
<div><br></div><div>My previous patch was tested (on OSX so without unit tests) during the Milano hackathon and it has solved an interoperability bug so presumbly we should keep this patch. Now I guess we need some qa experts to shed some light on the mechanism of this unit test ... </div>
<div><br></div><div><br></div><div>Cheers, </div><div>Siqi</div></div><div class="gmail_extra"><br><br><div class="gmail_quote">2013/10/7 Siqi Liu <span dir="ltr"><<a href="mailto:me@siqi.fr" target="_blank">me@siqi.fr</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hello all, <div><br></div><div>Sorry for the late response! This was a patch that I've submitted during the hackathon in Milan and it fixes the #fdo65295 on bugzilla.</div>
<div><br></div><div>Actually we've analyzed the content of the .docx exported by writer and it seems that the startColor and the endColor were accidentally reversed so that each time we import and export a docx with gradient background, we reverse the start and end color in the exported .docx file. </div>
<div><br></div><div>I would test the patch above to see if that solves the problem when I'm back home. </div><div><br></div><div>Sorry for the broken build :P</div><div><br></div><div>ATB,</div><div>Siqi</div><div><br>
</div></div><div class="gmail_extra"><div><div class="h5"><br><br><div class="gmail_quote">2013/10/6 julien2412 <span dir="ltr"><<a href="mailto:serval2412@yahoo.fr" target="_blank">serval2412@yahoo.fr</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi,<br>
<br>
If StartColor and EndColor have been reversed, perhaps it was the same for<br>
qa part? So what about this patch?<br>
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx<br>
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx<br>
index f771ef9..8fdb7fb 100644<br>
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx<br>
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx<br>
@@ -733,8 +733,8 @@ void Test::testTextframeGradient()<br>
uno::Reference<beans::XPropertySet> xFrame(xIndexAccess->getByIndex(0),<br>
uno::UNO_QUERY);<br>
CPPUNIT_ASSERT_EQUAL(drawing::FillStyle_GRADIENT,<br>
getProperty<drawing::FillStyle>(xFrame, "FillStyle"));<br>
awt::Gradient aGradient = getProperty<awt::Gradient>(xFrame,<br>
"FillGradient");<br>
- CPPUNIT_ASSERT_EQUAL(sal_Int32(0xC0504D), aGradient.StartColor);<br>
- CPPUNIT_ASSERT_EQUAL(sal_Int32(0xD99594), aGradient.EndColor);<br>
+ CPPUNIT_ASSERT_EQUAL(sal_Int32(0xD99594), aGradient.StartColor);<br>
+ CPPUNIT_ASSERT_EQUAL(sal_Int32(0xC0504D), aGradient.EndColor);<br>
CPPUNIT_ASSERT_EQUAL(awt::GradientStyle_AXIAL, aGradient.Style);<br>
<br>
Julien<br>
<br>
<br>
<br>
--<br>
View this message in context: <a href="http://nabble.documentfoundation.org/Test-failure-tp4076788p4076897.html" target="_blank">http://nabble.documentfoundation.org/Test-failure-tp4076788p4076897.html</a><br>
Sent from the Dev mailing list archive at Nabble.com.<br>
_______________________________________________<br>
LibreOffice mailing list<br>
<a href="mailto:LibreOffice@lists.freedesktop.org" target="_blank">LibreOffice@lists.freedesktop.org</a><br>
<a href="http://lists.freedesktop.org/mailman/listinfo/libreoffice" target="_blank">http://lists.freedesktop.org/mailman/listinfo/libreoffice</a><br>
</blockquote></div><br><br clear="all"><div><br></div></div></div><span class="HOEnZb"><font color="#888888">-- <br><div dir="ltr"><div>--------</div><div><br></div><div>Cordialement,</div>Siqi LIU<div><br><div>Étudiant Ingénieur, 1ère année</div>
<div>École Supérieur d'Électricité (Supélec)</div>
</div><div><br></div></div>
</font></span></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br><div dir="ltr"><div>--------</div><div><br></div><div>Cordialement,</div>Siqi LIU<div><br><div>Étudiant Ingénieur, 1ère année</div><div>École Supérieur d'Électricité (Supélec)</div>
</div><div><br></div></div>
</div>