summaryrefslogtreecommitdiff
path: root/writerfilter
AgeCommit message (Collapse)Author
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-10-08tdf#94049 RTF import: add missing check for empty shape when popping stateMiklos Vajna
Change-Id: I3c9a5f89df721618bb60b8932b69a371bb6f3ff1
2015-10-07tdf#90810: skip breaks outside of the requested footnoteOliver Specht
the footnote stream is parsed completely at each footnote insertion (line) breaks outside of the requested footnote need to be skipped test include Change-Id: I78de7774b7daf44c2b8980115870082fc56924bd Reviewed-on: https://gerrit.libreoffice.org/19228 Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk> Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-10-07Revert "tdf#90810: skip all elements outside of the requested footnote"Stephan Bergmann
This reverts commit 3eff65e78a3a90b07c7a01ff26736fd25996e476, causes assert at writerfilter/source/dmapper/DomainMapper_Impl.cxx:5330 to fire during CppunitTest_sw_ooxmlexport2
2015-10-06tdf#90810: skip all elements outside of the requested footnoteOliver Specht
the footnote stream is parsed completely at each footnote insertion all elements outside of the requested footnote need to be skipped Change-Id: I8909145411003ce6f924e712c2da00a4dc03a974 Reviewed-on: https://gerrit.libreoffice.org/19192 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2015-10-06tdf#91417: import hyperlink with history flag correctlyOliver Specht
ignore '\h' flag in hyperlinks imported from docx including import test Change-Id: I512e811050704f118bf9b2c786378f99e5d04378 Reviewed-on: https://gerrit.libreoffice.org/19154 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2015-10-05tdf#94435 RTF import: \ltrpar should not override \qcMiklos Vajna
Change-Id: I342191cd8cf8b3998089236aea5743ea7cdd26f1
2015-10-02loplugin:unusedmethodsNoel Grandin
Change-Id: I3d6f1300f4fae2af9e580d1d3b2c2c80fa9e9268 Reviewed-on: https://gerrit.libreoffice.org/19075 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-09-29Fix typosAndrea Gelmini
Change-Id: I4525b43fa823d743b86bf4136c56af35274085e6 Reviewed-on: https://gerrit.libreoffice.org/18995 Reviewed-by: Joren De Cuyper <jorendc@libreoffice.org> Tested-by: Joren De Cuyper <jorendc@libreoffice.org>
2015-09-29Fix typosAndrea Gelmini
Change-Id: If2e4adb2ce93c02ee14d831eedf4ef31da04b92d Reviewed-on: https://gerrit.libreoffice.org/18996 Reviewed-by: Joren De Cuyper <jorendc@libreoffice.org> Tested-by: Joren De Cuyper <jorendc@libreoffice.org>
2015-09-28bnc#939996 tdf#93919 DOCX import: fix left-from-style and first-from-directMiklos Vajna
With this, <w:ind w:hanging="..."/> as direct paragraph formatting and <w:ind w:left="..." w:hanging="..."/> as a numbering level formatting is properly merged, i.e. w:left is not lost, defaulting to 0. Change-Id: If5534fbd9ee6d41139b0ed3a3df9d0cc5aad3239
2015-09-28tdf#94456 RTF import: fix list vs paragraph style interaction wrt para indentsMiklos Vajna
In case we set the paragraph left/right/first margin just to mimic the list vs paragraph style interaction as Word wants it, never overwrite properties, as they are always meant to be just callbacks in case there is no real hard formatting. Change-Id: Ibdb2834c693d43cf4114453e42628e8f64c0a856
2015-09-28tdf#90153 DOCX import: fix default sw TextFrame roundtripMiklos Vajna
The AnchorType of the shape was at-paragraph, which does not allow line-level VertOrientRelation (which is correct, it would be undefined, what line of the paragraph should be the used). Fix this by changing the AnchorType to at-character in the line-level case, which brings the filter in sync with the DOC one. With this, import of a DOCX file that was created by inserting a TextFrame into an empty document is roundtripped without shifting the shape up considerably. Change-Id: I6d85c38be859d6e730584f2349c857b87496a1d4
2015-09-23tdf#92124 DOCX import: don't add a dummy Suffix for an empty LabelFollowedByMiklos Vajna
Change-Id: I0c4366ad0a2f81a198860869f670767343a392be
2015-09-22tdf#92454 DOCX import: allow overriding para prop from num style in para styleMiklos Vajna
Word has a feature like this: a paragraph style can refer to a numbering style, and both can specify paragraph margins. If that's the case, then the ones from the paragraph style has priority. In Writer, the numbering style has priority, so the only chance for correct import result is to set the margin directly on the paragraph in this case. Change-Id: Iff3b03bcc56e0db3a48452c293acf41c91b8f159
2015-09-11tdf#94045: negative margins in OOXML reenabledOliver Specht
OOXMLUniversalMeasureValue is an integer value and needs to be converted to a signed integer. unit test included Change-Id: I7fe3fdf4b8b12df55e9cea2a968b9537676687eb Reviewed-on: https://gerrit.libreoffice.org/18496 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2015-09-01cid#1321599 missing breakMiklos Vajna
Change-Id: Idaf322c53cb8955bb88e18a6d79fd800dc48621d
2015-08-28Make sure nEnd-nIndex-1 is valid arg to OUString::copyStephan Bergmann
Change-Id: Ie646ae781bcd54be81173db42fe10f61b31ac628
2015-08-27loplugin:defaultparamsStephan Bergmann
Change-Id: Ifd4c92d2013b5698d9657f599db8af345437b079
2015-08-27Improving Asian phonetic guide for docx and rtf files.Mark Hung
RTF import, export, and ooxml export for ruby text are implemented. tdf#49073 - FILEOPEN: Furigana (ruby text) and characters with them are missing in opened .docx files. tdf#50786 - [TASK, METABUG] FILEOPEN, FILESAVE, FORMATTING : Japanese ruby-character handling is broken Change-Id: I4a5c30bad180241e3344e9da7efe7da4369fb325 Reviewed-on: https://gerrit.libreoffice.org/17241 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2015-08-21loplugin: defaultparamsNoel Grandin
Change-Id: If1d183f32079548645b4974a16161dc997c026b7
2015-08-18this debugging code can definitely use oslCaolán McNamara
Change-Id: I420e7f242868a25a2f9a473c23c67dfd9a285b7c
2015-08-10cppcheck: zerodivCaolán McNamara
Change-Id: Iaf327051d59c39cbd61e1a64bdf1ee5af5cc4f3b
2015-08-05loplugin:unusedmethodsNoel Grandin
Change-Id: I6801618efb5a66d24156fa429e026acb6ca03aba Reviewed-on: https://gerrit.libreoffice.org/17506 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-08-04-Werror,-Wunused-private-fieldStephan Bergmann
Change-Id: Ied02019c6e11287ba574bb861ed7b2a8da1f8083
2015-08-04writerfilter: use std::unique_ptr<> in SettingsTableMiklos Vajna
Change-Id: I0202b18301b4b4a7c82c840df1397a5cbdd695b1
2015-08-03inline some use-once typedefsNoel Grandin
Change-Id: I02cbbba56a2ad83e0ac3d806265a7e0d6a29594d Reviewed-on: https://gerrit.libreoffice.org/17495 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-08-03inline some use-once typedefsNoel Grandin
Change-Id: I55cc82c8e180cce371c996690608090b1bfdfda4 Reviewed-on: https://gerrit.libreoffice.org/17494 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-08-03new loplugin: refcountingNoel Grandin
This was a feature requested by mmeeks, as a result of tdf#92611. It validates that things that extend XInterface are not directly heap/stack-allocated, but have their lifecycle managed via css::uno::Reference or rtl::Reference. Change-Id: I28e3b8b236f6a4a56d0a6d6f26ad54e44b36e692 Reviewed-on: https://gerrit.libreoffice.org/16924 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-31writerfilter: use std::unique_ptr<> in TblStylePrHandlerMiklos Vajna
Change-Id: Ia98b351a3ef14826ab638382fa312734edeb751d
2015-07-30Cleaned up PropertyMap::GetPropertyValues in writerfilter::dmapper.Matthew Pottage
Removed unused variable. Used comphelper::makePropertyValue. Change-Id: I39ec3e51da484a1792bb980b166d5a10c597fe7b Reviewed-on: https://gerrit.libreoffice.org/17383 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2015-07-30writerfilter: manual memory management -> boost::optional in PropertyMapMiklos Vajna
Change-Id: Ie8f7ce11ab03d47835cfe224c775324dd79a5e7a
2015-07-30loplugin:unusedmethodsNoel Grandin
Change-Id: Ib4d77ee01e7362f5951f81fceeca3c489872d971 Reviewed-on: https://gerrit.libreoffice.org/17378 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-29writerfilter: clean up remaining boost::scoped_ptr<> usageMiklos Vajna
Change-Id: I270c91bec75ff96ac77b550e6adef8d03b07f2b5
2015-07-29writerfilter: use std::unique_ptr<> in StyleSheetTableMiklos Vajna
Change-Id: I38eebe098c87cceb69b9ff4c48e18d3434ad1557
2015-07-28tdf#91594 misinterprets letters from Symbol font in docx filesMark Hung
Fix the issue caused by wrong assumption about symbol chracter and symbol font attributes order in writerfilter. Also allow symbols to be displayed if user's language is not Western. Change-Id: I602d9fbfa79c33c90f655dbf5ee22738b6391ae6 Reviewed-on: https://gerrit.libreoffice.org/16543 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2015-07-28tdf#49893: final fix for blank rectangle problemVasily Melenchuk
removed artifacts from invalid shape: nested shape instruction destination now is not drawn on its own. instead it is merged with outer shape instruction. this helps to avoid drawing semi-ready shapes. Change-Id: I45589cc1b742d878e5914dd3f66a8184ecfd044c Reviewed-on: https://gerrit.libreoffice.org/14584 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-07-28writerfilter: use std::unique_ptr<> in DomainMapperTableManagerMiklos Vajna
Change-Id: Ib5910d3198824d625a96b53a30ad67c69c622642
2015-07-27writerfilter: use std::unique_ptr<> in DomainMapperMiklos Vajna
Change-Id: I569add9792d533bd3bcda5d5cb4023b37bd3f1ed
2015-07-24tdf#90046: RTF import: ensure that a run is started before \footnoteMichael Stahl
Avoids crashing with empty context stacks. Change-Id: I0ee8b457fdbb19b55f5c15876b7253680cde6e23
2015-07-24writerfilter: use std::shared_ptr<> in ThemeTableMiklos Vajna
Change-Id: I9d632ed036b1d94c29adfc79df4c920791b0c0fa
2015-07-23writerfilter: use std::unique_ptr<> in FontTableMiklos Vajna
Change-Id: I8dce4e71c53526605d83710012133784a5cbd7ba
2015-07-22tdf#86374 DOCX import: fix btLr text table direction without <w:cantSplit/>Miklos Vajna
Commit 0208ead70a9412ccd554fcef3e9308f8ca17037b (DOCX import: improve btLr table cell support, 2013-02-22) made any table row that has at least one btLr cell fixed height. This causes problems in case a table has a minimal height with lots of content, where the fixed height gives wrong layout, but the minimal height is correct. Fix the problem by only making the row fixed height if <w:cantSplit/> is set (as seen in the old bugdoc), and revert to setting the height type to minimal in any other case. Change-Id: Ibaf91f542e64e5caa7904df97eb6eb52618e0023
2015-07-21loplugin:unusedmethods writerfilterNoel Grandin
Change-Id: I52441115e14da6c1bcd7ba4926a0f3cc0620366e Reviewed-on: https://gerrit.libreoffice.org/17236 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-21Use comphelper::containerToSequence()Miklos Vajna
Change-Id: Ia7cc48f6f1bc041b95164d551e2cbc9ea2dbf2e8
2015-07-21tdf#89702 DOCX import: fix too large bullet characterMiklos Vajna
Commit c1f8437dbed0e8b989e41a345ef7e658a6e8a4cd (fdo#83465 RTF import: handle font of numbering, 2014-09-25), changed the "get the me character style of the current numbering's current level" member function to be successfull even in case we're inside a DOCX run, not when we're inside a DOCX paragraph, but outside runs. While this is necessary for RTF, the side effect of this was that unwanted run properties started to affect the above mentioned character style in case of DOCX. Fix the problem by enabling the "in paragraph and run" looking for RTF only. Change-Id: I610bfce6cec15b918fe547402360f5a894401f7e
2015-07-16Removed singleton PropertyNameSupplier and replaced it with single function.Matthew Pottage
Measurements showed that the "optimisation" of caching PropertyIds and their string equivalent leads to an increase of approx. 6 times in the total overall time spent in PropertyNameSupplier::getName(eId), when running the unit tests. PropertyNameSupplier was the only PropertyNameSupplier (no derived classes). This means that getPropertyName can easily provide the same functionality. Change-Id: I933b67c11d4cc35395a0c70e15f1c24ac9842ab0 Reviewed-on: https://gerrit.libreoffice.org/16665 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-07-15tdf#87539 map MSO washout filter to LO's watermark colormodeJoren De Cuyper
Change-Id: Id28e301a74a729245b20ba6cd2adecf203633f0e Reviewed-on: https://gerrit.libreoffice.org/16048 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
2015-07-10cleanup some unnecessary direct usage of the OString/OUString C API.Noel Grandin
Change-Id: I6edee95aae1c169ce70063562a0c1b287e0c4735 Reviewed-on: https://gerrit.libreoffice.org/16861 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>