summaryrefslogtreecommitdiff
path: root/xmloff
AgeCommit message (Collapse)AuthorFilesLines
2015-05-27tdf#88056: Implement ODF import/export of page title fieldKatarina Behrens6-0/+73
to/from loext namespace, since new tag's needed for the new field and it isn't approved by ODF-TC yet Change-Id: I70ceb4e16e1199663520dbdf3beae31423bc81d0 Reviewed-on: https://gerrit.libreoffice.org/15485 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de> Tested-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-05-25tdf#91140: tweak fix a bit, turns out xmloff was also passing empty URLMichael Stahl1-1/+2
Change-Id: Ia6e1fbe18e72c9c06915e1b437076a1f56a6c206 (cherry picked from commit 12f907da9535ae9fb28fb7ef1b05240eabf51e82) Reviewed-on: https://gerrit.libreoffice.org/15867 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-05-09it works if we first set the size and then the position, tdf#86624Markus Mohrhard1-3/+3
Change-Id: I8ed98981ef3041839ab73b749a04febe369f66fe Reviewed-on: https://gerrit.libreoffice.org/15600 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-04-21tdf#90640: xmloff: ODF export: fix style:background-image attributesMichael Stahl2-26/+30
... for Writer frames, which were missing the draw:opacity; turns out this is because now the the default valued properties are not exported any more, which causes them to be missing in the property array so the hard-coded indexes in XMLTextExportPropertySetMapper::handleElementItem() do not find them. (regression from c3e49660e782816d1203cb936156ef6d6a1ad572) Change-Id: I84b6c09b23a2275161135290f1b0ac9f52546a80 (cherry picked from commit c13b80e202a07b374531babe3af5a8529483f060) Reviewed-on: https://gerrit.libreoffice.org/15465 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-04-17xmloff: accept character borders in the extension namespaceMiklos Vajna1-0/+32
(partially cherry-pick of commit f1f6b6db730ae67a427c7974b59a5e19ab571984) Conflicts: sw/qa/extras/odfexport/odfexport.cxx Change-Id: Ia112cf626126149ea9cf09c5d6ff812d5d5ffec5 Reviewed-on: https://gerrit.libreoffice.org/15352 Reviewed-by: Markus Mohrhard <markus.mohrhard@googlemail.com> Tested-by: Markus Mohrhard <markus.mohrhard@googlemail.com>
2015-04-14xmloff: add back the presumably accidentally removed "FootnoteLineStyle"Michael Stahl1-0/+1
(regression from 7d9bb549d498d6beed2c4050c402d09643febdfa) Change-Id: I67ec4516ff431efd47451ff07fc261ba32e80385 (cherry picked from commit ea8db4ad8e1c75c3ab2f31fea4e061600231f9d1) Reviewed-on: https://gerrit.libreoffice.org/15306 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-04-09tdf#89802: xmloff: fix ODF import of frame with empty style name refMichael Stahl1-0/+14
draw:fill-gradient-name="" in the graphic properties now causes BaseFrameProperties_Impl::FillBaseProperties() to fail and prevent insertion of the SwXTextFrame. (regression from 6e61ecd09679a66060f932835622821d39e92f01) (cherry picked from commit db1d278dcc308c73eb5edebc20481c96e7f479d8) Change-Id: I9b4f3ddb0f0e1b3a80e86bbdc1af2dade1c8edf7 Reviewed-on: https://gerrit.libreoffice.org/15192 Tested-by: Caolán McNamara <caolanm@redhat.com> Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2015-03-19Related tdf#50133: Wrong underlining in ReportsJulien Nabet1-1/+3
By running some tests, I noticed double wave didn't appear when opening report in non edit mode. First you must know that to indicate "double wave", xml contains these 2 information: style:text-underline-style="wave" style:text-underline-type="double" The problem is the reading order of the parser doesn't seem the same each time. So if it reads style then type, it's ok thanks to this part: 132 case awt::FontUnderline::DOUBLE: 133 // A double line style has priority over a bold line style, 134 // but not over the line style itself. 135 switch( eUnderline ) 136 { 137 case awt::FontUnderline::SINGLE: 138 case awt::FontUnderline::BOLD: 139 break; 140 case awt::FontUnderline::WAVE: 141 case awt::FontUnderline::BOLDWAVE: 142 eNewUnderline = awt::FontUnderline::DOUBLEWAVE; 143 break; but if the parser read type in the first place then style, the case FontUnderline "DOUBLEWAVE" wasn't managed Change-Id: Ie090507898e537827eebaae157dfac248baa43f1 Reviewed-on: https://gerrit.libreoffice.org/14801 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com> (cherry picked from commit 61a3ace9f42a37640b1ec18fad776ba7c1f61714) Reviewed-on: https://gerrit.libreoffice.org/14914
2015-03-08tdf#50133: Wrong underlining in ReportsJulien Nabet1-1/+1
Change-Id: Ia1d37d150c2186f014733cd87179445722b54346 (cherry picked from commit 27bc6714df6638c6fd9d2ee5c26870ae5eb2f63e) Reviewed-on: https://gerrit.libreoffice.org/14791 Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu> Tested-by: Lionel Elie Mamane <lionel@mamane.lu>
2015-02-05Revert "ODF export: don't write invalid "group-name" attribute"Lionel Elie Mamane2-4/+6
This reverts commit 2d4b87f0c1bfd97185a89c18d5b7680d11a958d6. The reverted commit leads to the following regressions: - Basic dialogs (which were not targeted, but impacted, by the reverted commit) with several RadioButtons sharing a group-name (as they will have a tendency to do) cannot be loaded anymore, since the implementation assumes (and checks) that names are unique. - Even in forms, where a RadioButton had both a form:name and a form:group-name attribute, the form:name attribute wins and thus RadioButtons that has the same group-name but different form:name (as they will tend to do) will not anymore be mutually exclusive, which defeats their point. Additionally, since it did not change the UI parts (property editor window), the user was still presented with two different editable properties "Name" and "Group Name", where "Group Name" was empty... Change-Id: I1bff532a5a7336cf2eb0579bcd4e2d16be6480fe Reviewed-on: https://gerrit.libreoffice.org/14338 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-02-05Revert "xmloff: dead code gives MSVC fits"Lionel Elie Mamane1-0/+30
This reverts commit 29e1b2f1ca6e2dcbf9a04c63a3ac1d554cfdcb52. in preparation for reverting 2d4b87f0c1bfd97185a89c18d5b7680d11a958d6 Change-Id: I9a262bf136b239c9f737f1d5523543ff425885ff Reviewed-on: https://gerrit.libreoffice.org/14337 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-01-19fdo#84714 ODT import: read <loext:table> inside <draw:custom-shape>Miklos Vajna1-0/+1
A later version of ODF will hopefully allow <table:table> here as well, but read <loext:table> in the meantime. Change-Id: I42a461e0a6e9eff9387379acbab9660a155ecefe (cherry picked from commit 62391c28fae5099dd1f67c322867933fcb05bc9f) Reviewed-on: https://gerrit.libreoffice.org/13857 Tested-by: Michael Stahl <mstahl@redhat.com> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2015-01-13LeakSanitizer: leak detected in ooo56669-1.odtCaolán McNamara1-2/+2
Change-Id: Ifdd98068762762316dbfd50fb465ed3c970f53d9 (cherry picked from commit 5e6a4591d31619f9bf32a9891c59a6dcc3af80b5) Reviewed-on: https://gerrit.libreoffice.org/13862 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-01-07teach XML import/export about unsigned integer typesLionel Elie Mamane2-2/+15
resolves a shitload of warnings like warn:legacy.osl:941:1:xmloff/source/forms/elementimport.cxx:426: OElementImport::implImportGenericProperties: unsupported value type! warn:legacy.osl:941:1:xmloff/source/forms/elementimport.cxx:443: OElementImport::EndElement: could not set the property "ObjIDinMSO"! warn:legacy.osl:941:1:xmloff/source/forms/elementimport.cxx:444: caught an exception! in function:void xmloff::OElementImport::implApplyGenericProperties() type: com.sun.star.lang.IllegalArgumentException Change-Id: Ia598d12e7d9429fe4ad3b1e7173e11e75060a613 Reviewed-on: https://gerrit.libreoffice.org/13553 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-01-06export format of embedded font tooDavid Tardon1-0/+4
Change-Id: Ife7dbb298861a71987501f3847d3b931c7e83715 (cherry picked from commit 2862ef7e65e4994ff3cf173a36ec59b47445455e) Reviewed-on: https://gerrit.libreoffice.org/13752 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2015-01-06read embedded fonts from Flat ODFDavid Tardon2-3/+32
Change-Id: Ic94608f4afac9d8ac05ec3140d195b0526e9420a (cherry picked from commit 0a0496858905622374f9880b21b35ac1943c7a19) Reviewed-on: https://gerrit.libreoffice.org/13751 Reviewed-by: Caolán McNamara <caolanm@redhat.com> Tested-by: Caolán McNamara <caolanm@redhat.com>
2014-12-31Revert "Resolves: #i124452# correct svg:viewBox for EnhancedCustomShape..."Andras Timar1-42/+1
This reverts commit c6e316f52021cc26d4c5ec9a7b87a07fbf595b62. The bug #i124452# was in AOO only, no need to fix it in LibreOffice. Especially when the "fix" causes regression. The related fdo#76334 should be fixed one day, but that is a different issue. Conflicts: xmloff/source/draw/ximpcustomshape.cxx Change-Id: Ice0bf378f97e2caf0ee8386d0b5c0b8abcbcd1c2 (cherry picked from commit 4cd048d98ff258abc6ce036cb715d2c577128fb0) Reviewed-on: https://gerrit.libreoffice.org/13683 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2014-12-25survive "TextBox" having no valueLionel Elie Mamane1-1/+1
Change-Id: Iebed41cfcc5899387e3defb8d8a0792276349098 Reviewed-on: https://gerrit.libreoffice.org/13560 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2014-12-10xmloff: make the export of the horrible field marks aware of auto-stylesMichael Stahl1-96/+105
When exporting the .DOC bugdoc of fdo#87110 to ODF, the auto-styles export will iterate over the fieldmarks in the page header and add the attributes for them but not export the elements, so the first auto-style gets a bunch of duplicate attributes. Change-Id: I3fcf39f03e3d9ae5fca661efa7eb4bbb3eab9f5c (cherry picked from commit 06f85d41d02ebef76487b230f35f2ec638c46c8b)
2014-12-10fdo#87110: xmloff: don't catch IllegalArgumentException hereMichael Stahl2-7/+11
There are at least 2 callers of InsertTextContent() that have non-trivial catch handlers for this exception, which aren't called now. (regression from bebf8ccfba37f77d6a43c7874249b31736467b17) Change-Id: I085b710dfd5877e9b7e71610951543eddf6a6e46 (cherry picked from commit b86f5530161a417d31e28e75408ee80352fadad7)
2014-12-10don't write extension elements into ODF namespaces, related fdo#45266Markus Mohrhard1-6/+6
We will still import the old elements but only write them into the lo-ext namespace. Also it will only be exported for ODF 1.2 extended. (cherry picked from commit 88d3931d92bc89519acf95f0510f6f2a6ff7c72c) Signed-off-by: Michael Stahl <mstahl@redhat.com> Backpoting the export part of this too since the attributes were added in LO 4.3 so the change has very limited impact. Conflicts: xmloff/source/chart/PropertyMap.hxx Change-Id: I1c41716b8b16e186ec84767b4b3a636d083162b8
2014-12-09Resolves: fdo#87108 crash on saving fodgCaolán McNamara1-4/+7
Change-Id: Ib88f0e9b0a3ba229c9e9f6cf20831c16051e3e29 (cherry picked from commit bfd63516c9fd4ec366576f9a0e3c456bc3d530a3)
2014-12-08iimport part for: "use correct xml element name, related fdo#45266"Markus Mohrhard1-0/+3
Change-Id: I8ec31d026f5c6fdc4b7fc573060e7837a337d50b
2014-12-08import part for: " use correct xml element name, related fdo#45266"Markus Mohrhard1-0/+1
Change-Id: Ied71383fc837c7e667455f84937801e8d45f2989
2014-12-08import part for: "export to extension namespace instead of odf ..."Markus Mohrhard1-0/+5
Change-Id: I0b61e05e3b153ba7efe351b18cb705ea6348f546
2014-12-08the xlink:type attribute is mandatory, related fdo#79449Markus Mohrhard1-0/+1
Change-Id: I03c4217f786bc10aa915780dc10ea52d94019b6e
2014-12-08remove invalid attribute that is not read anywayMarkus Mohrhard1-1/+0
Change-Id: Id7770ffda49ba7bc0ebc780c67c56b81152213f7
2014-12-05fdo#86795: xmloff: ODF import: do not import checkbox fieldmark ...Michael Stahl1-15/+34
... if it has start/end, because Writer will become grumpy and crash. Change-Id: I6024051249eeac6ed9e43856fa77db969287f888 (cherry picked from commit 25fd11e78279aef5a6b7656347758e5c67a9c45a)
2014-12-01fdo#85496: add some asserts to detect this sort of problemMichael Stahl1-0/+14
Change-Id: Iff787c8d2a71bc3082192cc98e3d916badee65dd (cherry picked from commit 7a242b463132d67a4a2d6e69319e0da367145cc0)
2014-11-27Resolves: fdo#84043 don't create duplicate Mirrored[X|Y] propertiesCaolán McNamara1-8/+24
regression from 13ef16423e78d3ea825172594f08c47d2f9bfd09 commit 13ef16423e78d3ea825172594f08c47d2f9bfd09 Author: Armin Le Grand <alg@apache.org> Date: Wed Nov 21 13:23:01 2012 +0000 For backward compatibility take mirrorings in setTransformation into account Also found an error in SdrObjCustomShape::TRGetBaseGeometry when MirrorY was used (cherry picked from commit 4116c33b12d3787c406f0348f89efcb1cf409507) Change-Id: I7bfb5dea32b8ab8498e3d92975c49b830c81e6fb (cherry picked from commit 751e5b32c5c361995bf0ba3295f773341fd92c23) Reviewed-on: https://gerrit.libreoffice.org/13149 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-11-18cppuhelper: clean up public headers with include-what-you-useMichael Stahl15-0/+15
Change-Id: I41ba46831f24b2960a1fe982b74a2b623e682e0b
2014-11-18cppu: clean up public headers with include-what-you-useMichael Stahl1-0/+1
Unfortunately iwyu gets quite confused by the weird cyclic dependencies between various foo.h/foo.hxx and cppumaker generated headers, so it's not obvious if any improvement here is realistic... Change-Id: I0bc66f98b146712e28cabc18d56c11c08418c721
2014-11-17sal: clean up public headers with include-what-you-useMichael Stahl79-0/+89
Sadly cannot forward declare "struct {...} TimeValue;". rtl/(u)?string.hxx still include sal/log.hxx but removing osl/diagnose.h was painful enough for now... Change-Id: Id41e17f3870c4f24c53ce7b11f2c40a3d14d1f05
2014-11-16Remove now useless setTextEmbeddedGraphicURL()Zolnai Tamás1-7/+0
Since this commit: 286e2f5c6ec829bc0987b1be7016699f7ef03e5e it's not necessary to update the package URL. Change-Id: I25c829e9bc0c666838baf19cd60f19938ebb430c
2014-11-14more catch by referenceCaolán McNamara1-1/+1
Change-Id: I2db9f94a517f814ef0854190e6f1194501070409
2014-11-14fdo#86023 - O[U]String needs a 'clear' methodBrij Mohan Lal Srivastava13-15/+15
Added clear() method to OString and OUString class, Updated appropriate call-sites. Change-Id: I0ba97fa6dc7af3e31b605953089a4e8e9c3e61ac Signed-off-by: Stephan Bergmann <sbergman@redhat.com>
2014-11-12Fix common typos. No automatic tools. Handmade…Andrea Gelmini10-11/+11
Change-Id: I1ab4e23b0539f8d39974787f226e57a21f96e959 Reviewed-on: https://gerrit.libreoffice.org/12164 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2014-11-11xmloff: XML_DROPDOWN should be XML_DROP_DOWNMichael Stahl4-4/+4
Change-Id: I47247289bb4367faa9a07f3568270718b5423353
2014-11-08xmloff: [loplugin:unusedvariablecheck]Michael Stahl1-1/+0
Change-Id: I5b141d35ab05e1a3f225a980aad1280102a80ee0
2014-11-08xmloff: remove unused parameter of SvxXMLListStyleContext::FillUnoNumRule()Michael Stahl3-15/+9
Change-Id: I359f37ce778d55e6868bd1c78c0ff0d452f36088
2014-11-07Removed #includes including themselvesAndrea Gelmini2-4/+0
Change-Id: I3fe3227967e07b6b0c82dccf3c9400bfe6e1d729 Reviewed-on: https://gerrit.libreoffice.org/12292 Reviewed-by: Riccardo Magliocchetti <riccardo.magliocchetti@gmail.com> Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-11-05markup with event type not checker typeCaolán McNamara2-1/+3
Change-Id: I14c0c5d90b67000cb4fe9e6be647854abfe784da
2014-11-05fdo#38835 strip out OUString globalsNoel Grandin11-56/+55
they are largely unnecessary these days, since our OUString infrastructure gained optimised handling for static char constants. Change-Id: I07f73484f82d0582252cb4324d4107c998432c37
2014-11-03coverity#705391: xmloff: XMLMergeElemTransformerContext misuse of enumMichael Stahl4-15/+15
The enum types XMLElemTransformerAction and XMLAttrTransformerAction use the same integer values so a mixture of values from these 2 in one switch is very suspicious. In the various *ActionTable arrays in OOo2Oasis/Oasis2OOo there is just one such array that mixes values from the 2 enums, and that is just the values that coverity complains about. The XML_ETACTION_EXTRACT_CHARACTERS was added at a later date in commits 8d374c039e7d349141befe3c9ef97f82f42bfa7 2685b93a6a23867baa4df265bb0cc35abaf76c0e 7d2405a54d13f68a35dfc5f619ad4bdb0b761856 c276550e07f8078fe9345985f70e8384d930a83f ...but it turns out that the XMLAttrTransformerAction ones are handled in StartElement() and ProcessAttrs() methods but the XMLElemTransformerAction ones are handled in CreateChildContext() methods so probably the XML_ATACTION_MOVE_FROM_ELEM* values are in the wrong enum. Change-Id: Ib1791f6eafac4fb1e99334f41c17a90cfb076359
2014-11-03coverity#1250700 Uninitialized scalar fieldCaolán McNamara1-0/+1
Change-Id: I5db51e7864017f0d0e474ed3e253657288245850
2014-10-31Removed duplicated includesAndrea Gelmini2-2/+0
Change-Id: I5362d997bfa086c9fb1726efcb15132a966684f6 Reviewed-on: https://gerrit.libreoffice.org/12160 Tested-by: LibreOffice gerrit bot <gerrit@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2014-10-31coverity#705348 Missing break in switchEike Rathke1-0/+1
Change-Id: I23e03e45ba4f6d259128bc5e4c2fd5952be05c2f
2014-10-31fdo#80403: Import baseclasses implement FastParser interfacesDaniel Sikeler2-0/+124
SvXMLImportContext implements XFastContextHandler SvXMLImport implements XFastDocumentHandler Change-Id: Id400260af112f4a448fe469c9580f0ebacec4ab6
2014-10-30Fixed typos. No automatic tools (sed, and so on).Andrea Gelmini2-2/+2
Change-Id: Ia43976d84eede6f699381bc4f3daf89b95e4cb4f Reviewed-on: https://gerrit.libreoffice.org/12150 Reviewed-by: Bryan Quigley <gquigs@gmail.com> Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2014-10-29coverity#735663 Division or modulo by zeroCaolán McNamara1-0/+3
Change-Id: I69410ebe3dcded9951bfa9e83844644147f4416a