summaryrefslogtreecommitdiff
path: root/oox/source/ppt
AgeCommit message (Collapse)AuthorFilesLines
2016-12-13OSL_TRACE->SAL in framework..salNoel Grandin4-8/+6
Change-Id: I9a897af88aa9f6f7ca98ce521c69b5a4ee8462e9 Reviewed-on: https://gerrit.libreoffice.org/31903 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-12-09tdf#104445: PPTX import: Extra bullets are added to placeholder textTamás Zolnai1-1/+1
Change-Id: I25ce98ed391f70292bed6238645b121b9cf50d5e Reviewed-on: https://gerrit.libreoffice.org/31771 Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com> Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2016-11-23loplugin:unnecessaryoverride (dtors) in ooxStephan Bergmann2-32/+1
Change-Id: I55b82f4669d89e8492b456ff8b0b430f444674a3
2016-11-17tdf#103876: PPTX import: Title shape's character properties are wrongTamás Zolnai1-0/+11
Text properties are applied on a shape during text insertion, but if a placeholder shape has no text, then it has a placehodler text which should have the right text properties. Change-Id: I54175d52dd25915ee4d7153298e01ec07c6be1f6 Reviewed-on: https://gerrit.libreoffice.org/30881 Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com> Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2016-11-10tdf#103792: No placeholder text imported for empty title shapeTamás Zolnai1-1/+0
Change-Id: I95b4358f0d4311e8f427c8de18863049fb718d9b Reviewed-on: https://gerrit.libreoffice.org/30731 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2016-11-07google docs can generate a pptx with an empty cmLstCaolán McNamara1-9/+12
Change-Id: I2a3dcae5361929c6880b2da1b3107cb2c8141f32
2016-10-13loplugin:unnecessaryoverrideNoel Grandin1-5/+0
Change-Id: I08c55a3023ec2e8990098eeb60e91cd18556e7ae Reviewed-on: https://gerrit.libreoffice.org/29656 Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk> Tested-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-09-30loplugin:constantparamNoel Grandin2-4/+3
Change-Id: Idbe8c8e6b3d44cacce296ec8c79b2b244281057c Reviewed-on: https://gerrit.libreoffice.org/29321 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2016-09-13loplugin:override: No more need for the "MSVC dtor override" workaroundStephan Bergmann11-20/+20
The issue of 362d4f0cd4e50111edfae9d30c90602c37ed65a2 "Explicitly mark overriding destructors as 'virtual'" appears to no longer be a problem with MSVC 2013. (The little change in the rewriting code of compilerplugins/clang/override.cxx was necessary to prevent an endless loop when adding "override" to OOO_DLLPUBLIC_CHARTTOOLS virtual ~CloseableLifeTimeManager(); in chart2/source/inc/LifeTime.hxx, getting stuck in the leading OOO_DLLPUBLIC_CHARTTOOLS macro. Can't remember what that isAtEndOfImmediateMacroExpansion thing was originally necessary for, anyway.) Change-Id: I534c634504d7216b9bb632c2775c04eaf27e927e
2016-08-19Some clang-tidy misc-move-constructor-initStephan Bergmann4-6/+6
...by turning the relevant ctor parameters into "const &". Change-Id: Ia8d0aba5da10ad6b25f8689e2281e45b3d71c1fc
2016-06-09Revert "Slideshow: Add 'Oval' Shape Transition variant"jan iversen2-18/+0
The patch caused problems with the export filter tests, to check that you need to add --with-export-validation to your autopen.input see https://cgit.freedesktop.org/libreoffice/contrib/dev-tools/tree/export-validation for more info on how to set it up This reverts commit 248c5ea771255b54e64394458a321ccf829bbd02. Change-Id: Ib3b8fa7bf80630feeca1f24dfb1ceb5a945d7162 Reviewed-on: https://gerrit.libreoffice.org/26114 Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-06-09Slideshow: Add 'Oval' Shape Transition variantMayank Gupta2-0/+18
Change-Id: Ibc3d617d3bb94bdd0702bb4d60ce5fbe2eea8e24 Reviewed-on: https://gerrit.libreoffice.org/23661 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-05-26tdf#42949: clean up includes in include/oox/ppt with iwyuJorenz Paragas8-1/+9
Change-Id: Ib2cf76418e8406b925113ae7a9e3c5b07ffd2de2 Reviewed-on: https://gerrit.libreoffice.org/25011 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-05-18clang-tidy modernize-make-sharedNoel Grandin3-5/+5
Change-Id: I3fa866bfb3093fc876474a9d9db29fe05dc2af3a Reviewed-on: https://gerrit.libreoffice.org/25056 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-05-10tdf#42949: clean up includes in include/oox/helper with iwyuJorenz Paragas16-2/+32
...and fix the many compiler errors that occurred as a result. Change-Id: I497c326272b2f02737ca3765720d6815b735423b Reviewed-on: https://gerrit.libreoffice.org/24735 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-05-10Insert explicit "break" when falling through to empty next caseStephan Bergmann2-0/+2
...which itself only contains a "break" (or nothing at all at the end of the "switch"), as otherwise Clang -Wimplicit-fallthrough would warn about these. Change-Id: I25c1cf2ca74dfeba7ca0385ca8f1c1bf30bbf91b
2016-05-03tdf#42949: clean up includes in include/oox/drawingml with iwyuJorenz Paragas4-0/+4
Add includes in other files to fix compiler errors. Change-Id: I566e2e081094cb074d20afbe7ceee49f759c725b Reviewed-on: https://gerrit.libreoffice.org/24510 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-04-29tdf#99030: PPTX import: Fixed lost slide background colorMatus Uzak2-2/+4
Regression from commit f3d1ac7 Change-Id: I5cb9fe1bb6c753c34b49e72194a9fbe4c10c1654 Reviewed-on: https://gerrit.libreoffice.org/23930 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
2016-04-26update loplugin stylepolice to check local pointers varsNoel Grandin1-3/+3
are actually pointer vars. Also convert from regex to normal code, so we can enable this plugin all the time. Change-Id: Ie36a25ecba61c18f99c77c77646d6459a443cbd1 Reviewed-on: https://gerrit.libreoffice.org/24391 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-25tdf#42949: clean up includes in include/oox/core using iwyuJorenz Paragas16-1/+39
This allows the headers in that folder to stand on their own and not depend on any transitively included headers. All changes to other files are needed to fix compiler errors related to missing includes that occurred afterwards. Change-Id: I2083b30763f0b2c1aacdff00226e5f567ae52db4 Reviewed-on: https://gerrit.libreoffice.org/24355 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Michael Meeks <michael.meeks@collabora.com> Tested-by: Michael Meeks <michael.meeks@collabora.com>
2016-04-22Avoid reserved identifiersStephan Bergmann1-1/+1
Change-Id: Ie1664e06dc02f7070e4ef77155e6541c70b2f8e8
2016-04-20loplugin:salbool: Automatic rewrite of sal_False/TrueStephan Bergmann3-3/+3
Change-Id: I379e89e1e6b9fd8668ba7ac59af33f63487a4f66
2016-04-18clean-up: unused using declarations and includesJochen Nitschke1-1/+0
Searched source for using declarations. Checked if those symbols reappear in the source file, even in comments or dead code but not in #include statements. If they don't reappear, remove the declaration. Remove includes whose symbol got removed. Change-Id: Ibb77163f63c1120070e9518e3dc0a78c6c59fab0 Reviewed-on: https://gerrit.libreoffice.org/24148 Reviewed-by: Jochen Nitschke <j.nitschke+logerrit@ok.de> Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-12cleanup: remove unused com/sun/star/uno includesJochen Nitschke3-4/+0
Sequence.h(xx), Any.h(xx) and Type.h(xx) and remove unused using-declarations from these files. Add a few missing includes provided by them. Change-Id: I6b91b6d1fdf9d0496dd546c0aab9bdcc6831a5d4 Reviewed-on: https://gerrit.libreoffice.org/23805 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-04-11clang-tidy performance-unnecessary-value-param in ooxNoel Grandin1-1/+1
Change-Id: I1ac86906ea21c00f8d77834c94583be7d9e9974e
2016-03-23tdf#93868: PPTX import: Incorrect inheritance of shape fill propertiesMatus Uzak3-9/+6
DrawingML: The useBgFill attribute specifies that the shape fill should be set to that of the slide background. Change-Id: I8b568e730f00326d51e7b604579f4ff990b26f8a Reviewed-on: https://gerrit.libreoffice.org/23039 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>
2016-01-21oox: fix indentation of PPTShape::addShape()Michael Stahl1-195/+197
Change-Id: Ic878535d3bc2d57dc24cc1821d8ce8403482b253
2016-01-12loplugin:unusedmethods unused return value in include/ooxNoel Grandin1-2/+2
Change-Id: I22acb6b94774c5944d8276827c53a419f0803f83
2016-01-10Fix typosAndrea Gelmini2-2/+2
Change-Id: I9a5940027423ff0791fa7da0b79b617412ce6b86 Reviewed-on: https://gerrit.libreoffice.org/21209 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Ashod Nakashian <ashnakash@gmail.com>
2015-12-26oox: change instances of maName to more descriptive namesChris Sherlock1-1/+1
Change-Id: Iee410ec4d256eb5b663d88e048d5bf6200845256 Reviewed-on: https://gerrit.libreoffice.org/20959 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2015-12-22tdf#96051: FILESAVE: Roundtrip any PPTX file with Shapes creates invalid OOXMLZolnai Tamás1-3/+6
Revert "bnc#584721: Import subtitle block to master slides" This reverts commit b3d50feaa87b670baf68288974005ac26ad31736. It seems the imported subtitle block makes export corrupt the output. Better to have a valid output. Change-Id: I1a3d4f03e4fa0c4431a5394495682a1b9d7677bc
2015-12-22loplugin:unusedfields in oox,packageNoel Grandin4-6/+0
Change-Id: I8f8c3ff9ded455a09f3df81c501da17cca0b2310
2015-11-18pptx: import/export of honeycomb slide transitionTomaž Vajngerl2-0/+5
Change-Id: I3edb77d5b046691e64d600d61dcd44e892c31301
2015-11-13slideshow: add "glitter" slide transitionTomaž Vajngerl2-0/+5
Change-Id: Ie89b64c4399cd0092eee579660c9fe85f8ca8e73
2015-11-10loplugin:nullptr (automatic rewrite)Stephan Bergmann10-25/+25
Change-Id: I62a20b440064aca6f102e61cb3984bb95e739a4f
2015-11-09loplugin:passstuffbyrefStephan Bergmann1-1/+1
Change-Id: Ibcab688c042e32a81b936573503c4a9066244161
2015-11-09pptx: "fall" transition, which needed p15 namespaceTomaž Vajngerl2-0/+21
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-08pptx: import outside/insde turning cube transitionTomaž Vajngerl2-0/+16
Change-Id: I12f668aca158a8c868ecaeb37f6724852308a515
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-04use uno::Reference::set method instead of assignmentNoel Grandin1-1/+1
Change-Id: I11822c50fa66d038a3d6f38054ab35c2e613f077
2015-10-29Too much copy pasta is not good for youTor Lillqvist9-534/+465
Change-Id: Ie5a3cddd6fcf9d1a763284c1aea0fca579da4f8d
2015-10-23com::sun::star->css in lotuswordpro,mysqlc,ooxNoel Grandin14-260/+260
Change-Id: Id99ff87c4eb3f1b36702261ce63d5d2793ea35d7
2015-10-20loplugin:defaultparamsStephan Bergmann1-1/+1
Change-Id: I52308fced25173f6a4b1c56c0c494e89fb5c1648
2015-10-12Replace "SAL_OVERRIDE" with "override" in LIBO_INTERNAL_ONLY codeStephan Bergmann12-34/+34
Change-Id: I2ea407acd763ef2d7dae2d3b8f32525523ac8274
2015-09-30Fix typosAndrea Gelmini1-1/+1
Change-Id: I013414cf10e25acd5216643855aaa230258a1b5f Reviewed-on: https://gerrit.libreoffice.org/18962 Reviewed-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de> Tested-by: Samuel Mehrbrodt <Samuel.Mehrbrodt@cib.de>
2015-09-29Renamed wrongly prefixed boolean variablesStefan Heinemann1-3/+3
Fixed tdf#94269 Change-Id: I63109cc4e095bad680d7637a065080ea368860ae Reviewed-on: https://gerrit.libreoffice.org/18851 Reviewed-by: Michael Stahl <mstahl@redhat.com> Tested-by: Michael Stahl <mstahl@redhat.com>
2015-09-01Resolves: tdf#91060 don't crash on loading certain pptxCaolán McNamara1-1/+1
Change-Id: Ibf32ef644222a59c5fe6a9736ff64739ab24b74f
2015-08-31loplugin:stringconstant: OUStringBuffer: appendAscii -> appendStephan Bergmann1-1/+1
Change-Id: I21969536cec33a18be3f47a310789568a480ec2f