[Libreoffice-commits] core.git: Changes to 'refs/changes/46/9146/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/67/7767/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/00/7000/7'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/00/7000/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/67/7767/3'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/00/7000/5'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/00/7000/3'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/00/7000/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/39/8539/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/92/9392/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/39/8539/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/74/7074/3'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/74/7074/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/67/7767/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/92/9392/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/92/9392/4'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/77/8677/3'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/31/10531/1'

2014-09-29 Thread Rajashri Bhat Udhoji

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/77/8677/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/39/7839/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/77/8677/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/14/9914/3'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/14/9914/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/14/9914/5'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/39/7839/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/48/8848/2'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/48/8848/3'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: Changes to 'refs/changes/48/8848/1'

2014-09-29 Thread Rajashri

___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa sw/source

2014-05-21 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/testWordArtWithinDraingtool.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx   |   19 
 sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx|2 
 sw/source/filter/ww8/docxattributeoutput.cxx   |   41 
++
 sw/source/filter/ww8/docxattributeoutput.hxx   |3 
 sw/source/filter/ww8/docxsdrexport.cxx |   10 --
 6 files changed, 67 insertions(+), 8 deletions(-)

New commits:
commit b349d2483e1fe64316d87b55d0b3b4c8f2293e2e
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Sat May 17 18:53:52 2014 +0530

fdo#78663 : Corrupt: 2007 WordArt whithin a Drawing tool

A file contains a 2007 word art within a drawing tool.
In Original file, the xml structure is as below:
sample xml:
p r ac drawing txbx txbxContent .. pict /pict 
/txbxContent/txbx /drawing /ac /r /p
LO was preserving this as :
p r ac drawing txbx txbxContent .. drawing txbx 
txbxContent ..
/txbxContent/txbx /drawing .. /txbxContent/txbx /drawing 
/ac /r /p
Now after this fix, If there is a custom shape within a text box, then 
we should postpone it and write it in
a separate run.
This approach is followed on the fact that LO writes nested TextBoxes 
in separate run.

NOTE1 :Due to this fix, xpath related to the below UT needs to be 
updated.
testFdo76249 in ooxmlsdrexport.cxx

NOTE2: In docxsdrexport.cxx : 'm_bIsInDMLTextFrame' is no more required.
For adding LockedCanvas, we only need to check if the property exists 
in the property set.
Therefoew removing the above bool variable.

Reviewed on:
https://gerrit.libreoffice.org/9392

Change-Id: I00e94712e912ad1977fcb65a945fefb927795d77

diff --git a/sw/qa/extras/ooxmlexport/data/testWordArtWithinDraingtool.docx 
b/sw/qa/extras/ooxmlexport/data/testWordArtWithinDraingtool.docx
new file mode 100644
index 000..c860828
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/testWordArtWithinDraingtool.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index 22b144e..7a69e67 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -3437,6 +3437,25 @@ DECLARE_OOXMLEXPORT_TEST(testFdo78651, fdo78651.docx)
 // ensure that there are only two tables
 assertXPath(pXmlDoc, //w:tbl, 2);
 }
+DECLARE_OOXMLEXPORT_TEST(testWordArtWithinDraingtool, 
testWordArtWithinDraingtool.docx)
+{
+/*   * Within a file, there is a 2007 wordArt enclosed in a drawing tool
+ * LO was exporting it as below:
+ * Sample XML as in Original file:
+ * p r ac drawing txbx txbxContent ..  pict /pict 
/txbxContent/txbx /drawing /ac /r /p
+ *  After RT :
+ * p r ac drawing txbx txbxContent ..  drawing txbx 
txbxContent ..  /txbxContent/txbx /drawing .. /txbxContent/txbx 
/drawing /ac /r /p
+ *  Expected : As there is nesting of a 2007 Word Art within a draing 
tool, then can be separated in two different runs.
+ * */
+
+xmlDocPtr pXmlDoc = parseExport(word/document.xml);
+if (!pXmlDoc)
+   return;
+
assertXPath(pXmlDoc,/w:document[1]/w:body[1]/w:p[1]/w:r[2]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:anchor[1]/a:graphic[1]/a:graphicData[1]/wps:wsp[1]/wps:txbx[1]/w:txbxContent[1],1);
+
assertXPath(pXmlDoc,/w:document[1]/w:body[1]/w:p[1]/w:r[2]/mc:AlternateContent[1]/mc:Fallback[1]/w:pict[1]/v:rect[1]/v:textbox[1]/w:txbxContent[1]/w:p[1]/w:r[1]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:inline[1]/a:graphic[1]/a:graphicData[1]/wps:wsp[1]/wps:txbx[1]/w:txbxContent[1],1);
+
assertXPath(pXmlDoc,/w:document[1]/w:body[1]/w:p[1]/w:r[3]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:anchor[1]/a:graphic[1]/a:graphicData[1]/wps:wsp[1]/wps:txbx[1],1);
+
assertXPath(pXmlDoc,/w:document[1]/w:body[1]/w:p[1]/w:r[4]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:inline[1]/a:graphic[1]/a:graphicData[1]/wps:wsp[1]/wps:txbx[1]/w:txbxContent[1],1);
+}
 
 #endif
 
diff --git a/sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx
index 21eff8e..fc5c8d7 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlsdrexport.cxx
@@ -1017,7 +1017,7 @@ DECLARE_OOXMLEXPORT_TEST(testFdo76249, fdo76249.docx)
 xmlDocPtr pXmlDoc = parseExport(word/document.xml);
 if (!pXmlDoc)
return;
-assertXPath(pXmlDoc, 
//mc:Choice/w:drawing//w:txbxContent//w:drawing//lc:lockedCanvas, 1);
+assertXPath(pXmlDoc, 
//mc:Choice[2]/w:drawing[1]/wp:inline[1]/a:graphic[1]/a:graphicData[1]/lc:lockedCanvas[1],1);
 }
 
 DECLARE_OOXMLEXPORT_TEST(testFdo76979, fdo76979.docx)
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index 9e3bcd7..96602c3 100644

[Libreoffice-commits] core.git: sw/qa sw/source

2014-04-28 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/SdtContent.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx  |8 
 sw/source/filter/ww8/docxattributeoutput.cxx  |6 +++---
 3 files changed, 11 insertions(+), 3 deletions(-)

New commits:
commit d9de57dd2acc4719f846e908284ed81f09856098
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Thu Apr 24 14:44:33 2014 +0530

fdo#77713 : File Corruption - Header.xml

Document containing track changes within a SDT content
is getting corrupted.
Reason for corruption: In export side,parent tag [SDT] is
getting closed before child tag ['del' tag].
The corruption is resolved now.

Change-Id: I42451495f80e30ddfebab8b81bcf1e309b0c2ca2
Reviewed-on: https://gerrit.libreoffice.org/9146
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/qa/extras/ooxmlexport/data/SdtContent.docx 
b/sw/qa/extras/ooxmlexport/data/SdtContent.docx
new file mode 100644
index 000..91e0ef0
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/SdtContent.docx 
differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index f33771f..5dc3bcf 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -2947,6 +2947,14 @@ DECLARE_OOXMLEXPORT_TEST(testlvlPicBulletId, 
lvlPicBulletId.docx)
 assertXPath(pXmlDoc, 
/w:numbering[1]/w:abstractNum[1]/w:lvl[1]/w:lvlPicBulletId[1], 0);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testSdtContent, SdtContent.docx)
+{
+xmlDocPtr pXmlDoc = parseExport(word/header1.xml);
+if (!pXmlDoc)
+   return;
+assertXPath(pXmlDoc, /w:hdr[1]/w:p[1]/w:sdt[1]/w:sdtContent[1]/w:del[1]);
+}
+
 DECLARE_OOXMLEXPORT_TEST(testCitation,FDO74775.docx)
 {
 xmlDocPtr pXmlDoc = parseExport();
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index 27835b3..3321999 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -836,6 +836,9 @@ void DocxAttributeOutput::EndRun()
 // append the actual run end
 m_pSerializer-endElementNS( XML_w, XML_r );
 
+// if there is some redlining in the document, output it
+EndRedline( m_pRedlineData );
+
 // enclose in a sdt block, if necessary
 WriteSdtBlock( m_nRunSdtPrToken, m_pRunSdtPrTokenChildren, 
m_pRunSdtPrDataBindingAttrs );
 m_pSerializer-mergeTopMarks();
@@ -848,9 +851,6 @@ void DocxAttributeOutput::EndRun()
 
 WritePendingPlaceholder();
 
-// if there is some redlining in the document, output it
-EndRedline( m_pRedlineData );
-
 m_pRedlineData = NULL;
 
 if ( m_closeHyperlinkInThisRun )
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa sw/source

2014-04-18 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/NumberedList.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx|   15 +++
 sw/source/filter/ww8/docxattributeoutput.cxx|2 +-
 3 files changed, 16 insertions(+), 1 deletion(-)

New commits:
commit a7f68538f5f27093b8165e2e8fce3f7fade3aaf9
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Fri Jan 31 15:08:11 2014 +0530

fdo#74150:Numbered list getting display after RT -table-within-table-1.docx

Description:
For pStyle = 'NumberedList1', iLvl and numId was not preserved.
These tags needs to be preserved under numPr in document.xml
These tags are preserved  now.

Conflicts:

sw/qa/extras/ooxmlexport/ooxmlexport.cxx

Change-Id: Ib6734c9ae55bd72d4fbc0f6d6cae85764ac1a244
Reviewed-on: https://gerrit.libreoffice.org/7767
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/qa/extras/ooxmlexport/data/NumberedList.docx 
b/sw/qa/extras/ooxmlexport/data/NumberedList.docx
new file mode 100644
index 000..53f4490
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/NumberedList.docx 
differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index e7f57e6..a7ad9b7 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -3129,6 +3129,21 @@ DECLARE_OOXMLEXPORT_TEST(test77219, test77219.docx)
 assertXPath(pXmlDoc, 
/w:document[1]/w:body[1]/w:p[6]/w:r[1]/mc:AlternateContent[1]/mc:Choice[1]/w:drawing[1]/wp:anchor[1],
 behindDoc, 1);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testNumberedList,NumberedList.docx)
+{
+//fdo74150:In document.xml, for pStyle = NumberedList1, iLvl and numId 
was not preserved
+xmlDocPtr pXmlDoc = parseExport(word/document.xml);
+if (!pXmlDoc)
+return;
+assertXPath(pXmlDoc, 
/w:document/w:body/w:tbl/w:tr[1]/w:tc[1]/w:p[1]/w:pPr[1]/w:pStyle, val, 
NumberedList1);
+assertXPath(pXmlDoc, 
/w:document/w:body/w:tbl/w:tr[1]/w:tc[1]/w:p[1]/w:pPr[1]/w:numPr/w:ilvl,val,
 0);
+assertXPath(pXmlDoc, 
/w:document/w:body/w:tbl/w:tr[1]/w:tc[1]/w:p[1]/w:pPr[1]/w:numPr/w:numId,val,
 0);
+
+assertXPath(pXmlDoc, 
/w:document/w:body/w:tbl[1]/w:tr[1]/w:tc[1]/w:p[3]/w:pPr[1]/w:pStyle,val, 
NumberedList1);
+assertXPath(pXmlDoc, 
/w:document/w:body/w:tbl[1]/w:tr[1]/w:tc[1]/w:p[3]/w:pPr[1]/w:numPr/w:ilvl,val,
 0);
+assertXPath(pXmlDoc, 
/w:document/w:body/w:tbl[1]/w:tr[1]/w:tc[1]/w:p[3]/w:pPr[1]/w:numPr/w:numId,val,
 0);
+}
+
 DECLARE_OOXMLEXPORT_TEST(testFDO76597, fdo76597.docx)
 {
 // check XML
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index 3de5f8b..a4679f7 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -6106,7 +6106,7 @@ void DocxAttributeOutput::ParaHyphenZone( const 
SvxHyphenZoneItem rHyphenZone )
 
 void DocxAttributeOutput::ParaNumRule_Impl( const SwTxtNode* /*pTxtNd*/, 
sal_Int32 nLvl, sal_Int32 nNumId )
 {
-if ( USHRT_MAX != nNumId  0 != nNumId )
+if ( USHRT_MAX != nNumId )
 {
 m_pSerializer-startElementNS( XML_w, XML_numPr, FSEND );
 m_pSerializer-singleElementNS( XML_w, XML_ilvl, FSNS( XML_w, XML_val 
), OString::number( nLvl).getStr(), FSEND );
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa writerfilter/source

2014-04-11 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/testTcBorders.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx |   19 +++
 writerfilter/source/dmapper/DomainMapperTableHandler.cxx |   14 +++
 3 files changed, 28 insertions(+), 5 deletions(-)

New commits:
commit 07f90c89ef78484a422ca4536ba587217147e80a
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Fri Apr 4 09:00:27 2014 +0530

fdo#76635 : Table borders not preserved after RT

The document containing a custom table is getting crashed.
In the document the user has created his own costume table styles and
in the created custom table style the user has set  repeat first row as 
header property.
The corresponding tag for this property is 'tblHeader'
LO was adding this property as a part of cellProperties in 
DomainMapperTableHandler.cxx.
But it is a part of a Row Property. That is why there was a crash @ open.

In DomainMapperTableHandler.cxx : 'endTableGetCellProperties' tblHeader 
property is removed from
style as it is not allowed for a cell.

Change-Id: I83cd369aa4497ec3f4b567174f23ddee9be2864e
Reviewed-on: https://gerrit.libreoffice.org/8848
Tested-by: David Tardon dtar...@redhat.com
Reviewed-by: David Tardon dtar...@redhat.com

diff --git a/sw/qa/extras/ooxmlexport/data/testTcBorders.docx 
b/sw/qa/extras/ooxmlexport/data/testTcBorders.docx
new file mode 100644
index 000..b7fb81a
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/testTcBorders.docx 
differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index cafd9ea..d13bb70 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -1344,6 +1344,21 @@ DECLARE_OOXMLEXPORT_TEST(testCalendar2, calendar2.docx)
 assertXPath(pXmlStyles, 
/w:styles/w:style[@w:styleId='Calendar2']/w:tblPr/w:tblBorders/w:insideV, 
themeTint, 99);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testTcBorders, testTcBorders.docx)
+{
+//fdo#76635 : Table borders are not getting preserved.
+
+xmlDocPtr pXmlDocument = parseExport(word/document.xml);
+if (!pXmlDocument)
+return;
+
+   assertXPath(pXmlDocument, 
/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tcPr[1]/w:tcBorders[1]/w:bottom[1][@w:val
 = 'single'],1);
+   assertXPath(pXmlDocument, 
/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tcPr[1]/w:tcBorders[1]/w:bottom[1][@w:sz
 = 4], 1);
+   assertXPath(pXmlDocument, 
/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tcPr[1]/w:tcBorders[1]/w:bottom[1][@w:space
 = 0], 1);
+   assertXPath(pXmlDocument, 
/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tcPr[1]/w:tcBorders[1]/w:bottom[1][@w:color
 = 808080], 1);
+
+}
+
 DECLARE_OOXMLEXPORT_TEST(testQuicktables, quicktables.docx)
 {
 xmlDocPtr pXmlStyles = parseExport(word/styles.xml);
@@ -1384,6 +1399,10 @@ DECLARE_OOXMLEXPORT_TEST(testFdo71302, fdo71302.docx)
 assertXPath(pXmlStyles, /w:styles/w:style[@w:styleId='Strong'], 1);
 }
 
+
+
+
+
 DECLARE_OOXMLEXPORT_TEST(testSmartart, smartart.docx)
 {
 uno::Referencetext::XTextDocument xTextDocument(mxComponent, 
uno::UNO_QUERY);
diff --git a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx 
b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
index 08d1919..336514b 100644
--- a/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
+++ b/writerfilter/source/dmapper/DomainMapperTableHandler.cxx
@@ -545,7 +545,6 @@ TableStyleSheetEntry * 
DomainMapperTableHandler::endTableGetTableStyle(TableInfo
 sal_Int32 nHoriOrient = text::HoriOrientation::LEFT_AND_WIDTH;
 m_aTableProperties-getValue( TablePropertyMap::HORI_ORIENT, 
nHoriOrient ) ;
 m_aTableProperties-Insert( PROP_HORI_ORIENT, uno::makeAny( 
sal_Int16(nHoriOrient) ) );
-
 //fill default value - if not available
 const PropertyMap::const_iterator aRepeatIter =
 m_aTableProperties-find(PROP_HEADER_ROW_COUNT);
@@ -695,12 +694,17 @@ CellPropertyValuesSeq_t 
DomainMapperTableHandler::endTableGetCellProperties(Tabl
 }
 
 // Remove properties from style/row that aren't allowed in 
cells
-const PropertyMap::iterator aDefaultRepeatIt = 
pAllCellProps-find(PROP_HEADER_ROW_COUNT);
+PropertyMap::iterator aDefaultRepeatIt = 
pAllCellProps-find(PROP_HEADER_ROW_COUNT);
+if ( aDefaultRepeatIt != pAllCellProps-end( ) )
+pAllCellProps-erase( aDefaultRepeatIt );
+
+aDefaultRepeatIt = pAllCellProps-find(PROP_PARA_LINE_SPACING);
 if ( aDefaultRepeatIt != pAllCellProps-end( ) )
 pAllCellProps-erase( aDefaultRepeatIt );
-const PropertyMap::iterator aDefaultRepeatIt2 = 
pAllCellProps-find(PROP_PARA_LINE_SPACING);
-if ( aDefaultRepeatIt2 != pAllCellProps-end( ) )
-pAllCellProps-erase

[Libreoffice-commits] core.git: sw/qa sw/source

2014-03-21 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/TCTagMisMatch.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx |   12 
 sw/source/filter/ww8/docxattributeoutput.cxx |   13 +
 3 files changed, 21 insertions(+), 4 deletions(-)

New commits:
commit 5ff0e274bb146bbaede31e9e349cba301137d4f2
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Thu Mar 20 13:27:17 2014 +0530

fdo#76361 : LO crash: LO crashes while saving the document to docx

The document contains an empty table with borders.
LO was treating this table as a nested table, which lead to
a TC tag mismatch.
In docxattributeoutput.cxx, inside 'switchHeaderFooter'
added a check against 'm_oldTableReference-m_pOldTablepInner'
For some documents, even if the above value was null, still
it was getting assigned to 'm_tableReference' . Thus in such situations
it leads to a crash.

Change-Id: I883267585cb804f961989797de57c4b843b96821
Reviewed-on: https://gerrit.libreoffice.org/8677
Reviewed-by: Miklos Vajna vmik...@collabora.co.uk
Tested-by: Miklos Vajna vmik...@collabora.co.uk

diff --git a/sw/qa/extras/ooxmlexport/data/TCTagMisMatch.docx 
b/sw/qa/extras/ooxmlexport/data/TCTagMisMatch.docx
new file mode 100644
index 000..6d890ac
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/TCTagMisMatch.docx 
differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index a1542d9..966e52c 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -2887,6 +2887,18 @@ DECLARE_OOXMLEXPORT_TEST(test76108, test76108.docx)
 assertXPath(pXmlDoc, 
/w:document[1]/w:body[1]/w:p[1]/w:r[1]/w:fldChar[1], fldCharType, begin);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testTCTagMisMatch, TCTagMisMatch.docx)
+{
+   // TCTagMisMatch.docx : This document contains a empty table with borders.
+   // there was a TC tag mismatch which resulted into a crash.
+
+   xmlDocPtr pXmlDoc = parseExport(word/document.xml);
+   if(!pXmlDoc)
+  return;
+   
assertXPath(pXmlDoc,/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1]/w:tbl[1]/w:tr[1]/w:tc[1],0);
+   assertXPath(pXmlDoc,/w:document[1]/w:body[1]/w:tbl[1]/w:tr[1]/w:tc[1], 1);
+}
+
 DECLARE_OOXMLEXPORT_TEST(testHyperLinkTagEnded, fdo76316.docx)
 {
 /* XML tag w:hyperlink was not getting closed when its inside another
diff --git a/sw/source/filter/ww8/docxattributeoutput.cxx 
b/sw/source/filter/ww8/docxattributeoutput.cxx
index a088367..e287f71 100644
--- a/sw/source/filter/ww8/docxattributeoutput.cxx
+++ b/sw/source/filter/ww8/docxattributeoutput.cxx
@@ -2489,10 +2489,15 @@ void DocxAttributeOutput::switchHeaderFooter(bool 
isHeaderFooter, sal_Int32 inde
 }
 else if( index == -1)
 {
-*m_tableReference = *m_oldTableReference;
-//Reset the oldReference, after copying it back to the original.
-m_oldTableReference-m_bTableCellOpen = false ;
-m_oldTableReference-m_nTableDepth = 0;
+   if (m_oldTableReference-m_pOldTablepInner)
+   {
+   *m_tableReference = *m_oldTableReference;
+
+//Reset the oldReference, after copying it back to the original.
+m_oldTableReference-m_bTableCellOpen = false ;
+m_oldTableReference-m_nTableDepth = 0;
+   }
+
 }
 else
 {
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa sw/source

2014-03-13 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/lvlPicBulletId.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx  |9 +
 sw/source/filter/ww8/wrtww8.cxx   |4 +++-
 3 files changed, 12 insertions(+), 1 deletion(-)

New commits:
commit 3075b062261435d568d0680c3405ca61a7a40f39
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Wed Mar 12 10:44:57 2014 +0530

fdo#74772 : File Corruption - Issue related to List Bullet Numbering

- A File containing a graphic shape bullet is captured as a numPictBullet in
  MS office[in numbering.xml].
- Currently LO has support for graphic bullet as Image. Graphic bullet as 
Shape
needs to be preserved in LO.
- Since the numPictBullet is not exported in LO, but in abstractNum
lvlPicBulletId was exported which lead to corruption.
- Fixed the corruption caused due to lvlPicBulletId.
- However the shape is not getting retained. Need to add implementation
for gprahic bullet as a shape.

Conflicts:
sw/qa/extras/ooxmlexport/ooxmlexport.cxx

Change-Id: I2fc2b1ab4b75e8a050e66837a64578b235059473

diff --git a/sw/qa/extras/ooxmlexport/data/lvlPicBulletId.docx 
b/sw/qa/extras/ooxmlexport/data/lvlPicBulletId.docx
new file mode 100644
index 000..28e656b
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/lvlPicBulletId.docx 
differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index b7fdef7..045d41e 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -2744,6 +2744,15 @@ DECLARE_OOXMLEXPORT_TEST(testOLEObjectinHeader, 
2129393649.docx)
 
assertXPath(pXmlDoc,/rels:Relationships/rels:Relationship[1],Id,rId1);
 }
 
+DECLARE_OOXMLEXPORT_TEST(testlvlPicBulletId, lvlPicBulletId.docx)
+{
+xmlDocPtr pXmlDoc = parseExport(word/numbering.xml);
+if (!pXmlDoc)
+   return;
+assertXPath(pXmlDoc, 
/w:numbering[1]/w:abstractNum[1]/w:lvl[1]/w:lvlPicBulletId[1], 0);
+
+}
+
 #endif
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/sw/source/filter/ww8/wrtww8.cxx b/sw/source/filter/ww8/wrtww8.cxx
index 82b0fbd..9832e18 100644
--- a/sw/source/filter/ww8/wrtww8.cxx
+++ b/sw/source/filter/ww8/wrtww8.cxx
@@ -1423,7 +1423,9 @@ int MSWordExportBase::CollectGrfsOfBullets()
 }
 if (!bHas)
 {
-m_vecBulletPic.push_back(pGraf);
+Size aSize(pGraf-GetPrefSize());
+if (0 != aSize.Height()  0 != aSize.Width())
+   m_vecBulletPic.push_back(pGraf);
 }
 }
 }
___
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits


[Libreoffice-commits] core.git: sw/qa sw/source

2014-02-14 Thread Rajashri
 sw/qa/extras/ooxmlexport/data/EquationAsScientificNumbering.docx|binary
 sw/qa/extras/ooxmlexport/data/EquationWithAboveAndBelowCaption.docx |binary
 sw/qa/extras/ooxmlexport/data/FigureAsLabelPicture.docx |binary
 sw/qa/extras/ooxmlexport/data/TableWithAboveCaptions.docx   |binary
 sw/qa/extras/ooxmlexport/ooxmlexport.cxx|   56 
++
 sw/source/filter/ww8/docxattributeoutput.cxx|2 
 sw/source/filter/ww8/ww8atr.cxx |   12 +-
 7 files changed, 63 insertions(+), 7 deletions(-)

New commits:
commit d738b063134e97c780205f8baf138f291016ded4
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Mon Feb 3 19:29:16 2014 +0530

fdo#74431 : Captions' are preserved but remove a space

For the below captions :
ALPHABETIC
alphabetic
ROMAN
roman
ARABIC
in document.xml, captions were added as
w:instrText xml:space=preserve SEQ scientific *ROMAN /w:instrText
-no space after *
-Double quotes added for scientific
-For Caption ARABIC, it was comming as
w:instrText xml:space=preserve SEQ scientific *Arabic /w:instrText
it should be
w:instrText xml:space=preserve SEQ scientific * ARABIC /w:instrText
- After adding the above code, anchor value for a hyperlink tag was getting
corrupted. So aaded a fix for that.
UT which was failing due to above code changes :testBnc834035
equality assertion failed
- Expected: _Toc363553908
- Actual  : Figure!1|sequence

Conflicts:
sw/qa/extras/ooxmlexport/ooxmlexport.cxx
Reviewed on:
https://gerrit.libreoffice.org/7839

Change-Id: I9124eefa0ae63baf7ee3eaa2148f3c739d6fa699

diff --git a/sw/qa/extras/ooxmlexport/data/EquationAsScientificNumbering.docx 
b/sw/qa/extras/ooxmlexport/data/EquationAsScientificNumbering.docx
new file mode 100644
index 000..e6d6bdf
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/EquationAsScientificNumbering.docx differ
diff --git 
a/sw/qa/extras/ooxmlexport/data/EquationWithAboveAndBelowCaption.docx 
b/sw/qa/extras/ooxmlexport/data/EquationWithAboveAndBelowCaption.docx
new file mode 100644
index 000..7657d05
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/EquationWithAboveAndBelowCaption.docx differ
diff --git a/sw/qa/extras/ooxmlexport/data/FigureAsLabelPicture.docx 
b/sw/qa/extras/ooxmlexport/data/FigureAsLabelPicture.docx
new file mode 100644
index 000..50415bc
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/FigureAsLabelPicture.docx differ
diff --git a/sw/qa/extras/ooxmlexport/data/TableWithAboveCaptions.docx 
b/sw/qa/extras/ooxmlexport/data/TableWithAboveCaptions.docx
new file mode 100644
index 000..f445ab6
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/TableWithAboveCaptions.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
index 79e0bac..5a0b817 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport.cxx
@@ -3126,6 +3126,62 @@ DECLARE_OOXMLEXPORT_TEST(testIndentation, 
test_indentation.docx)
 assertXPath(pXmlDoc, /w:document/w:body/w:p/w:pPr/w:ind, end, );
 }
 
+DECLARE_OOXMLEXPORT_TEST(testCaption1, EquationAsScientificNumbering.docx)
+{
+// fdo#74431 : This test case is to verify the Captions are comming 
properly
+//earlier it was comming as SEQ scientific\*ROMAN now it is SEQ 
scientific\* ROMAN
+
+xmlDocPtr pXmlDoc = parseExport();
+if (!pXmlDoc)
+return;
+xmlNodeSetPtr pXmlNodes = 
getXPathNode(pXmlDoc,/w:document/w:body/w:p/w:r[3]/w:instrText);
+xmlNodePtr pXmlNode = pXmlNodes-nodeTab[0];
+OUString contents = OUString::createFromAscii((const 
char*)((pXmlNode-children[0]).content));
+CPPUNIT_ASSERT(contents.match( SEQ scientific \\* ROMAN));
+}
+
+
+
+DECLARE_OOXMLEXPORT_TEST(testCaption2, EquationWithAboveAndBelowCaption.docx)
+{
+// fdo#72563 : There was a problem that in case of TOC,PAGEREF field tag 
was not preserved during Roundtrip
+// This test case is to verify that PAGEREF tag is coming with proper 
values inside hyperlink tag.
+xmlDocPtr pXmlDoc = parseExport();
+if (!pXmlDoc)
+return;
+xmlNodeSetPtr pXmlNodes = 
getXPathNode(pXmlDoc,/w:document/w:body/w:p[5]/w:r[3]/w:instrText);
+xmlNodePtr pXmlNode = pXmlNodes-nodeTab[0];
+OUString contents = OUString::createFromAscii((const 
char*)((pXmlNode-children[0]).content));
+CPPUNIT_ASSERT(contents.match( SEQ Equation \\* ARABIC));
+}
+
+
+DECLARE_OOXMLEXPORT_TEST(testCaption3, FigureAsLabelPicture.docx)
+{
+// fdo#72563 : There was a problem that in case of TOC,PAGEREF field tag 
was not preserved during Roundtrip
+// This test case is to verify that PAGEREF tag is coming with proper 
values inside hyperlink tag.
+xmlDocPtr pXmlDoc = parseExport();
+if (!pXmlDoc)
+return

[Libreoffice-commits] core.git: chart2/qa oox/source

2014-01-16 Thread Rajashri
 chart2/qa/extras/chart2export.cxx |   12 +++
 chart2/qa/extras/data/docx/testAreaChartLoad.docx |binary
 oox/source/export/chartexport.cxx |   73 +++---
 3 files changed, 63 insertions(+), 22 deletions(-)

New commits:
commit c7cfe0876d7ca3d861cc6d8c1f84970f9f71def9
Author: Rajashri rajashri.udh...@synerzip.com
Date:   Fri Dec 6 16:20:54 2013 +0530

fdo#72217 : Fix for corruption area chart with data labels after Round Trip.

For labels under one series, there were child tags created for every
data label individually.
for example, if under one series there are five labels, then under
dLbls there were 5 dLbl tags created separately.
This issue is resolved now.

Conflicts:
chart2/qa/extras/chart2export.cxx

Change-Id: I7ae214f413bc27728df8a3d4cb8f4d703cba2f77

diff --git a/chart2/qa/extras/chart2export.cxx 
b/chart2/qa/extras/chart2export.cxx
index 6cef73b..7b1629a 100644
--- a/chart2/qa/extras/chart2export.cxx
+++ b/chart2/qa/extras/chart2export.cxx
@@ -39,6 +39,7 @@ public:
 void testChartDataTable();
 void testChartExternalData();
 void testEmbeddingsGrabBag();
+void testAreaChartLoad();
 
 CPPUNIT_TEST_SUITE(Chart2ExportTest);
 CPPUNIT_TEST(test);
@@ -50,6 +51,7 @@ public:
 CPPUNIT_TEST(testChartDataTable);
 CPPUNIT_TEST(testChartExternalData);
 CPPUNIT_TEST(testEmbeddingsGrabBag);
+CPPUNIT_TEST(testAreaChartLoad);
 CPPUNIT_TEST_SUITE_END();
 
 protected:
@@ -440,6 +442,7 @@ void Chart2ExportTest::testCrosses()
 
 assertXPath(pXmlDoc, /c:chartSpace/c:chart/c:plotArea/c:catAx/c:crosses, 
val, autoZero);
 }
+
 void Chart2ExportTest::testChartDataTable()
 {
 load(/chart2/qa/extras/data/docx/, testChartDataTable.docx);
@@ -500,6 +503,15 @@ void Chart2ExportTest::testEmbeddingsGrabBag()
CPPUNIT_ASSERT(bEmbeddings); // Grab Bag has all the expected elements
 }
 
+void Chart2ExportTest::testAreaChartLoad()
+{
+load (/chart2/qa/extras/data/docx/, testAreaChartLoad.docx);
+xmlDocPtr pXmlDoc = parseExport(word/charts/chart,Office Open XML 
Text);
+CPPUNIT_ASSERT(pXmlDoc);
+assertXPath(pXmlDoc, 
/c:chartSpace/c:chart/c:plotArea/c:areaChart/c:ser/c:dLbls/c:showVal, val, 
1);
+assertXPath(pXmlDoc, 
/c:chartSpace/c:chart/c:plotArea/c:areaChart/c:ser/c:dLbls/c:dLbl, 0);
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Chart2ExportTest);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git a/chart2/qa/extras/data/docx/testAreaChartLoad.docx 
b/chart2/qa/extras/data/docx/testAreaChartLoad.docx
new file mode 100644
index 000..9383f75
Binary files /dev/null and b/chart2/qa/extras/data/docx/testAreaChartLoad.docx 
differ
diff --git a/oox/source/export/chartexport.cxx 
b/oox/source/export/chartexport.cxx
index 8bbf222..a795aac 100644
--- a/oox/source/export/chartexport.cxx
+++ b/oox/source/export/chartexport.cxx
@@ -2410,15 +2410,38 @@ void ChartExport::exportDataLabels(
 bool showCategoryName = false;
 bool showNumberInPercent = false;
 
-sal_Int32 nElem;
-for( nElem = 0; nElem  nSeriesLength; ++nElem)
+sal_Int32 nElem = 0;
+
+
+uno::Reference beans::XPropertySet  xPropSet;
+if(nSeriesLength != 0)
 {
-uno::Reference beans::XPropertySet  xPropSet;
+   try
+   {
+   xPropSet = SchXMLSeriesHelper::createOldAPIDataPointPropertySet(
+  xSeries, nElem, getModel() );
+   }
+   catch( const uno::Exception  rEx )
+   {
+SAL_WARN(oox, Exception caught during Export of data label: 
  rEx.Message );
+   }
+}
 
+namespace cssc2 = ::com::sun::star::chart2;
+cssc2::DataPointLabel aTempLabel;
+if( xPropSet.is() )
+{
+if (GetProperty( xPropSet, Label))
+   mAny = aTempLabel;
+
+
+
+for( nElem = 1; nElem  nSeriesLength; ++nElem)
+{
 try
 {
 xPropSet = 
SchXMLSeriesHelper::createOldAPIDataPointPropertySet(
-xSeries, nElem, getModel() );
+   xSeries, nElem, getModel() );
 }
 catch( const uno::Exception  rEx )
 {
@@ -2454,39 +2477,45 @@ void ChartExport::exportDataLabels(
case csscd::AVOID_OVERLAP: aPlacement = bestFit;  
break;
}
 
-   if(aLabel.ShowLegendSymbol || aLabel.ShowNumber || 
aLabel.ShowCategoryName || aLabel.ShowNumberInPercent)
-   {
+   if (aLabel.ShowLegendSymbol)
+   showLegendSymbol = true;
+   if(aLabel.ShowNumber)
+   showNumber = true;
+   if(aLabel.ShowCategoryName)
+   showCategoryName =  true;
+   if(aLabel.ShowNumberInPercent)
+   showNumberInPercent = true