summaryrefslogtreecommitdiff
path: root/oox
AgeCommit message (Collapse)AuthorFilesLines
2015-11-09Clean up Sequence constructionStephan Bergmann1-66/+44
...to avoid silly errors like the one fixed with 37039fde1187c2e71bdde6e7fafd23c6ae8a871e "Missing comma." Change-Id: Iab4d25f44fd8867ee1ef5e15c209ada5c6ebb0c8 Reviewed-on: https://gerrit.libreoffice.org/19871 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2015-11-09Missing commaStephan Bergmann1-1/+1
Change-Id: Ia2497902771b78130b6263885bdb3362d1e6c76c
2015-11-09loplugin:passstuffbyrefStephan Bergmann1-1/+1
Change-Id: Ibcab688c042e32a81b936573503c4a9066244161
2015-11-09pptx: "fall" transition, which needed p15 namespaceTomaž Vajngerl8-0/+34
Added p15 namespace which is present in MSO 2013+ documents. It adds a "prstTrans" element with a string "prst" attribute which is a name of the transition to be performed. This was added to support "fall" transition which translates to "fallOver" preset transition. Change-Id: I429bb106a12c1b99cce756502508e917720daef3
2015-11-09pptx: import "Vortex" and "Ripple" transition + testTomaž Vajngerl2-0/+10
Change-Id: I29d71e9c07c4964275c13772cdb6042624550acb
2015-11-09new loplugin: oncevarNoel Grandin3-54/+15
Change-Id: If57390510dde4d166be3141b9f658a7453755d3f Reviewed-on: https://gerrit.libreoffice.org/19815 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-11-08loplugin:stringconstantStephan Bergmann1-1/+1
Change-Id: I33876585b5a406231954013051bc7f61045226b0
2015-11-08oox: check for namespace in MCE is flawed, use a namespace listTomaž Vajngerl1-7/+13
In "AlternateContent" nodes we have to check if we support the namespace which is provided in "Requires" attribute of "Choice". Currently we tried to resolve the namespace with a call to the xml filter, however this doesn't work as the filter is already gone. In writerfilter we also have to handle a similar situation but there we just compare it to a list of predefined namespace alias ("wps" and "wpg"). This commit adds a list of supported namespace aliases to fragmenthandler2 instead of the namespace checking to support the "p14" namespace alias correctly. Change-Id: I25c430b97336c9e140bb5641a76a60895734b91f
2015-11-08add p14 namespace to xmlfilterbaseTomaž Vajngerl1-2/+4
Change-Id: I34d253e9f5bdfff14f1940544ae5bbb019bf506b
2015-11-08pptx: import outside/insde turning cube transitionTomaž Vajngerl2-0/+16
Change-Id: I12f668aca158a8c868ecaeb37f6724852308a515
2015-11-08oox: define p14 namespace in strict, define P14_TOKENTomaž Vajngerl3-1/+6
Change-Id: I9c92b9a637f43fd425cd0741e480b72ee5cff919
2015-11-08oox: p14 namespace (MSO 2010) + tokens for slide transitionsTomaž Vajngerl2-0/+23
Change-Id: I72d1901ceb2e43e459efb197cb72bd701c7d8f63
2015-11-08pptx: import/export of "cut through black" transitionTomaž Vajngerl1-3/+1
"Cut through black" is not supported in ppt so we detectit only in pptx specific export code. This needed some changes in the pptx code for exporting transitions. Change-Id: Ibc7361311017b3ffadd289db4e8ae233e1101ea8
2015-11-08pptx: fix export of newsflash transitionTomaž Vajngerl1-3/+2
Change-Id: I0e34e1b8860b9142d3171246d41bb528e710d10c
2015-11-06loplugin:stringconstant: elide explicit ctor usage (manually due to macros)Stephan Bergmann2-5/+5
Change-Id: Ic8c87b17d0c2c1b18b01a44ababee0daccf1fb28
2015-11-06loplugin:stringconstant: elide explicit ctor usage (automatic rewrite)Stephan Bergmann3-10/+10
Change-Id: Ia944d081842753fcf82f77bda82defe1d5566b37
2015-11-04use uno::Reference::set method instead of assignmentNoel Grandin3-4/+4
Change-Id: I11822c50fa66d038a3d6f38054ab35c2e613f077
2015-11-04yyyyyNoel Grandin4-62/+36
Change-Id: I9a947beefd2dfe21da8239e841ea3fb416bd1548
2015-11-02tdf#89088 DOCX import: fix missing text due to throwing ShapeContextHandlerMiklos Vajna1-1/+1
Regression from commit 866a4436d3cfac1ff42d7996250bf96fb703aeaa (oox: handle textboxes in ShapeContextHandler::endFastElement(), 2014-06-04), the problem was that code in ShapeContextHandler::endFastElement() assumed that all the XShape implementation it may see provides a TextBox UNO property (which is specific to Writer) of type bool, but this is not true. Fix the problem by assuming false in case it has an other type. Testcase can be a simple "does it load" one, as in the meantime exceptions such as trying to read a bool from an uno::Any make the DOCX XFilter::filter() implementation return false, unlike in the originally reported LibreOffice 4.4 version that just silently discarded the remaining content of the document. Change-Id: I81781a7ec28d085ceee50d386b8eda023f83c29f
2015-10-30UNO: no need to use OUString constructor when calling createInstanceNoel Grandin1-1/+1
Change-Id: I37da28539b94706574116d0fff5f008aabfb5526 Reviewed-on: https://gerrit.libreoffice.org/19682 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-29Too much copy pasta is not good for youTor Lillqvist9-534/+465
Change-Id: Ie5a3cddd6fcf9d1a763284c1aea0fca579da4f8d
2015-10-27oox: replace boost::ptr_vector with std::vector<std::unique_ptr>Michael Stahl4-25/+43
With one additional m_Effects.clear() in EffectProperties::assignUsed() to make it idempotent like it was before and avoid duplicating elements. Change-Id: I105535a2a250e682a5d6976e0c7f74374b1f31ac
2015-10-27Reduce scope of #include <tools/stream.hxx>Matteo Casalin1-0/+1
Change-Id: I0752cbb0cb2e0f89607d81d9eda7ce7091449fd6
2015-10-27sal_uIntPtr to sal_uInt64, sal_uLong to sal_Size for streamsMatteo Casalin1-14/+14
Change-Id: I062f1f6c5b20ca47734a9a3cd1a229d51763a206
2015-10-23Revert "oox: replace boost::ptr_vector with std::vector<std::unique_ptr>"Stephan Bergmann4-42/+25
This reverts commit d29e614ff9cd91c4e4a1bada6a21884e33323f8d, it broke CppunitTest_sd_export_tests. Change-Id: I908cc46cdf4df706ad64d04777b8e1401e0800c1
2015-10-23com::sun::star->css in lotuswordpro,mysqlc,ooxNoel Grandin67-518/+521
Change-Id: Id99ff87c4eb3f1b36702261ce63d5d2793ea35d7
2015-10-23oox: replace boost::ptr_vector with std::vector<std::unique_ptr>Michael Stahl4-25/+42
Change-Id: I105535a2a250e682a5d6976e0c7f74374b1f31ac
2015-10-21chart series support hatches, tdf#94227Markus Mohrhard1-0/+1
Change-Id: Ib3a1ccda738091466d5e2866823f64b254ed5bab
2015-10-21more removal of com::sun::star typedefsNoel Grandin1-3/+2
Change-Id: Ia73f1b4f1dcfa3f0936359e744afe76e02dcd2eb
2015-10-20loplugin:defaultparamsStephan Bergmann2-2/+2
Change-Id: I52308fced25173f6a4b1c56c0c494e89fb5c1648
2015-10-20Turns out this was a non-debug buildStephan Bergmann1-1/+1
Change-Id: Ica14ca0cb2320f9787d48e93a2fb81910601ce70
2015-10-19Work around what looks like MSVC 2013 miscompilationStephan Bergmann1-1/+6
Change-Id: Ic2ed1b00d9d29f004f1e3cd60af863d6aafc7188
2015-10-19loplugin:defaultparamsStephan Bergmann1-6/+6
Change-Id: Id2c0a8dbfe7110f816360ca9eb2bbca8e2f55a30
2015-10-17coverity#1327457 Uninitialized scalar fieldCaolán McNamara1-0/+1
Change-Id: Idbad4c6d09fd2eb9a044d8193fc7101de63c02f5
2015-10-13Use nRadius value at least to decide between "rect" and "roundRect" presetsStephan Bergmann1-1/+3
(found by clang-analyzer-deadcode.DeadStores) Change-Id: I27ff1222295a151a7616ed380439b87d27866899
2015-10-13Reduce scope of variableStephan Bergmann1-2/+1
Change-Id: I07745fc47f5d7908c35045e83cb230d4e91a813a
2015-10-13clang-analyzer-deadcode.DeadStoresStephan Bergmann1-1/+0
ever since 68e756de2ec7f9aeb1b6a6a28468745eca114715 "n#657896 - Gradient Transparencies and Muticolor Gradient fills" moved the use of nDmlANgle (to set aGradient.Angle) from after the if/else to inside the else branch. According to vmiklos: "i guess it's just an oversight that the 'nDmlAngle = nShapeRotation;' is not removed in the 'if' branch." Change-Id: I63e449e01a83415d988d6616388d724449997f39
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann88-324/+324
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-10-12Replace "SAL_DELETED_FUNCTION" with "= delete" in LIBO_INTERNAL_ONLY codeStephan Bergmann1-2/+2
Change-Id: I328ac7a95ccc87732efae48b567a0556865928f3
2015-10-12cppcheck:variableScopeNoel Grandin1-1/+1
Change-Id: I037feb335499629300309851dcda3bb661f03d4f Reviewed-on: https://gerrit.libreoffice.org/19316 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2015-10-09remove redundant codeRosemary1-54/+40
- Use the existing createHexStringFromDigit() to export hex digits as strings - The parameters offset and stream name are not required Change-Id: If9aab7816efb81655c40a11743e8951ab0b26e1d
2015-10-09correct creation of hex string from digitRosemary1-1/+1
Change-Id: I891a2a0b3e2194892bd4629c22b823902d18adc2
2015-10-09Convert hex string to upper caseRosemary1-1/+1
Change-Id: I5ccdfc62a8fd3521b6c4ee0029e70b41aaf4542e
2015-10-08blind fix for OSXMarkus Mohrhard1-0/+1
2015-10-08don't hardcode the project idMarkus Mohrhard1-11/+8
Now that we have the encryption we no longer need to use the hardcoded project id. Change-Id: I014aa35344960eb960d9b05b76a11f8d8d7e0de6
2015-10-08Generate random numbers for seedRosemary1-2/+8
Change-Id: I7527b0526a74c4d13036d58d904f2b7293c3463f
2015-10-08oox: loplugin:unreffunMiklos Vajna1-0/+4
Change-Id: I8ee7d5690495bb089ab57da0d7ad6e31bb2154b7
2015-10-08oox: loplugin:loopvartoosmallMiklos Vajna1-1/+1
Change-Id: Id2f2275f1f1bdb94b032bcf3cd34e486c9620940
2015-10-08The correct size of length is 4Rosemary1-1/+1
Change-Id: I8735e68e1094e40f989ebc0a8a3926c9e2f06fd4
2015-10-08add test for projkey generationMarkus Mohrhard2-15/+29
Change-Id: I42957abbdcf396830713d7ca4eb7539e6c110e11