summaryrefslogtreecommitdiff
path: root/sw
AgeCommit message (Collapse)AuthorFilesLines
2016-02-17Pass map by pointer (just like all the other SwAccessible... ctors do)Stephan Bergmann3-4/+4
Change-Id: Idc3daacd258b4033f81cee6b59dfa6a6c2e458da
2016-02-17gtk3: use native GtkPopover for calc formula prompt, etcCaolán McNamara1-3/+6
Change-Id: I9aa7e977228780b6273daa03be5e872342414437
2016-02-17use consistent #define checks for the Windows platformNoel Grandin25-36/+36
stage 2 of replacing usage of various checks for the windows platform with the compiler-defined '_WIN32' macro In this stage we focus on replacing usage of the WIN macro Change-Id: Ie8a4a63198a6de96bd158ecd707dadafb9c8ea84 Reviewed-on: https://gerrit.libreoffice.org/22393 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-17sw: indentation fixesMiklos Vajna3-10/+10
Change-Id: I2a33fa5df5afed804adcbf904d5b47a507924034
2016-02-17tdf#97526 - in copy/paste ignore unwanted page break on first paragraphJuergen Funk1-0/+15
In copy/paste ignore the page break attribute from the first paragraph of the document (see also tdf#39400) Change-Id: I838c21c14647d4692673bd30df320e6704de1455 Reviewed-on: https://gerrit.libreoffice.org/22270 Reviewed-by: Oliver Specht <oliver.specht@cib.de> Tested-by: Oliver Specht <oliver.specht@cib.de>
2016-02-16tdf#97429: Fix crash on print selected areaXisco Fauli1-3/+12
Regression from 9eb61489ee03486a7147acdf923eaf5e3f355183 Use DelFlyInRange to iterate over nodes' range Change-Id: I219c7ec83eaa69990340d423dc34546131bb372c Reviewed-on: https://gerrit.libreoffice.org/22129 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Xisco Faulí <anistenis@gmail.com>
2016-02-16cid#1352648 uninitialized membersMiklos Vajna1-0/+1
Change-Id: I21faf2034c6253226637bfb65deb869d0c4eda07
2016-02-16Related: tdf#59699 DOCX import: fix linked graphics with relative URLsMiklos Vajna3-0/+8
Because the writerfilter::ooxml::OOXMLFastContextHandlerShape constructor only sent the input stream (and not the full media descriptor) to oox::shape::ShapeContextHandler, it's startFastElement() tried to reconstruct a media descriptor from just the input stream. As a result, the base URL of the document got lost at the time the oox::drawingml::BlipContext constructor tried to call getAbsoluteUrl(), to convert the relative URL to an absolute one. Fix the problem by sending not only the input stream, but the full media descriptor to the shape importer. As a bonus that makes code simpler, too. Change-Id: I1f06c04d0745aa7e2e06d4848cf454790d5073ca
2016-02-15tdf#56321 - .doc import image flipped statusJustin Luth3-0/+38
Although LO was exporting whether an image was flipped horizontally or vertically, it wasn't reading it in during an import. Change-Id: Ia85e3459d00f40d852bbcd3fcfe15e7b53d518ba Reviewed-on: https://gerrit.libreoffice.org/22344 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Justin Luth <justin_luth@sil.org> Reviewed-by: Michael Stahl <mstahl@redhat.com>
2016-02-15drop outdated fileDavid Tardon1-3/+0
Change-Id: I268f0de42e0567d74dd9b9b84b19d6afbf4d44df
2016-02-15SwView::WriteUserDataSequence: use comphelper::makePropertyValue()Miklos Vajna1-43/+15
Change-Id: Ibdb99942ee6c022abad74df93ebfb99c4ce6567f
2016-02-14tdf#92157: allow both dimensions of a graphic to be 0Mike Kaganski3-4/+6
Commit ca80f73 made it possible for one of dimensions to be zero. This commit goes further, allowing opening real-life documents with graphics having both width and height equal to 0. Thanks to libreoffice@arbruijn.dds.nl for debugging and initial patch! Change-Id: I96a13b776adfd9fe46fc2c7691eb7904400c20a1 Reviewed-on: https://gerrit.libreoffice.org/21287 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-13tdf#76817: fix missing heading styles assigned to outline levels in ooxmlMark Hung5-19/+41
1. Export pStyle because writerfilter regard a list as an outline only if associated pStyle is set to "Heading N". 2. Export English style name to ensure style names created by LO with non-English UI is always recognized. Change-Id: I88c590e4bae6cf30e78ebc1011023461eac4e883 Reviewed-on: https://gerrit.libreoffice.org/22086 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
2016-02-13tdf#81129 Support reading non-BMP characters in HTML documents.Mark Hung2-0/+23
1. Allow character entity ( &#nnnn; ) to exceed 0xffff in HTMLParser::ScanText() 2. Return a character as sal_uInt32 ( utf32 ) instead of sal_Unicode ( utf16 ) from SvParser::GetNextChar(). Conflicts: sw/qa/extras/htmlexport/htmlexport.cxx Change-Id: Ida455040970fae800f0f11471b27f53461fb78e4 Reviewed-on: https://gerrit.libreoffice.org/21152 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
2016-02-13tdf#96392 disallow hanging punctuation if the frame is inside a table.Mark Hung1-0/+1
Change-Id: Ib9d8386b47f8fa7b8afabb94f94085c7fb2fc0bb Reviewed-on: https://gerrit.libreoffice.org/22158 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Mark Hung <marklh9@gmail.com>
2016-02-12sw: fix a STL assertion in lcl_MoveAllLowerObjs()Michael Stahl1-1/+4
For OLE objects MoveObjectIfActive() ends up calling SwSortedObjs::Update() which removes and re-inserts objects, thus invalidating the pSortedObj iterator. (regression from 04783fd91832fa01a5b096f395edd7ad4f9c0f6b) Change-Id: I2628f1b4ecd1c20ebbc4e9dda4e4befc9fad4644
2016-02-12sw,sfx2: assert that embedded object supports XEmbedPersistMichael Stahl1-5/+2
Change-Id: Ibdeec746e9664f8bfe4e2af9afb8e5ec0142bd08
2016-02-12sfx2: related tdf#56270: loss of embedded objects imported from DOCXMichael Stahl3-0/+31
After the import some of these are kept in RUNNING state. For Math objects imported from MathType3 OLEs in particular, first a new Math object is created and stored to the XStorage, only then is the MathType3 stream imported. This means the Math object is modified and contains data that must be stored. The problem is then that SfxObjectShell::ImportFrom() simply calls setModified(false), clearing the flag without storing the object. For Flat ODF export we lose all the objects that are cached in sw's SwOLELRUCache; for the bugdoc something more inexplicable happens for ODT export where we lose "Object 214" (which is the first one in the cache) but no other ones. (The main difference is that for ODF there is an optimization to copy the embedded object's storage without loading the object, but for Flat ODF every object must be loaded and exported.) (regression from 83777cd6e0f3f1a4458af896fd13344c696ecb1e) Change-Id: Id1474fba9f4da2d5247c7ff4dc6819ddb9829fe8
2016-02-12sw: ensure that configuration change in odfexporttest.cxx ...Michael Stahl5-37/+71
... is reverted in case the test function throws an exception, so that subsequent^Wfollowing tests don't inherit the changed configuration. Change-Id: I748f9edf15a7f860607ae4cce891450db254c73e
2016-02-12sw: unindent SwOLELRUCache::Load()Michael Stahl1-22/+20
Change-Id: If1a83fd4716b7a8ebfb09a7118e760fcca3a61b4
2016-02-12sw: don't crash if Office.Common/Cache/Writer/OLE_Objects set to 1Michael Stahl1-15/+20
(possibly regression from b717bda1f6484905aebc571c4538165a1fbfd2bb) Change-Id: I9113fe2e769cd6ba56bdccc629ac63241b238553
2016-02-13vcl: include vcl headers via triangular brackets instead of quotesChris Sherlock1-1/+1
Change-Id: I311f7db622ce341527fe12a92b516f800b602f92
2016-02-12sw: prefix members of ConstArcMiklos Vajna2-10/+10
Change-Id: Id717c768075abfc5652b60c7bb8b67dbd51548bf
2016-02-11tdf#96444 Replace chr(20) with '%20' in style uno commandsYousuf Philips1-10/+10
Change-Id: I88dec81db837f8839f1473371b558000d4bc9cf9 Reviewed-on: https://gerrit.libreoffice.org/22265 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
2016-02-11tdf#89505: replace tab character with space in table of contentsOliver Specht1-1/+1
the ToxWhitespaceStripper now replaces also tab characters so that they don't break the formatting Change-Id: If66aaddcbc0f8c65461f53f07ca7031f8f1d11b4 Reviewed-on: https://gerrit.libreoffice.org/22271 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Oliver Specht <oliver.specht@cib.de>
2016-02-10tdf#87034 RTF import: fix multiple superscripts after footnote ...Miklos Vajna2-0/+20
... in the same paragraph. The intention was to avoid replaying the supertext buffer and using its contents as a custom footnote mark at the same time. However, it's enough to check if the buffer is empty to do so, and that avoids the mis-import of the bugdoc as well. Change-Id: I1e3b0c7f0c6d8eb8250d8b1d0d7d196039c40e79
2016-02-09don't assert that no ODF import happens without a base URLMichael Stahl1-1/+3
Turns out that if you create a new document and then copy an embedded object from it to the clipboard, there really is no usable base URL. So don't assert and just log something, hopefully the important file import cases have been fixed already. Change-Id: Iee1027cbab4db10a273518739a5ab2c31627180e
2016-02-09crashtesting: assert on exporting tdf94913-1.docx to docCaolán McNamara1-1/+1
starts with "CITATION ", and not " CITATION ", but I think that's ok given the similar test at RES_AUTHORITY Change-Id: I3c429209e70bac905e87b6919af074e7143e45b0
2016-02-09Remove excess newlinesChris Sherlock21-44/+0
A ridiculously fast way of doing this is: for i in $(pcregrep -l -M -r --include='.*[hc]xx$' \ --exclude-dir=workdir --exclude-dir=instdir '^ {3,}' .) do perl -0777 -i -pe 's/^ {3,}/ /gm' $i done Change-Id: Iebb93eccbee9e4fc5c4380474ba595858a27ac2c Reviewed-on: https://gerrit.libreoffice.org/22224 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-09Formatting changes across all modulesChris Sherlock3-6/+6
+ Removed comment cruft + Tab formatting in number of files + Some commented out code removed + Tab characters replaced with spaces + Newline cleanup in quite a few files + Tweak header guard #endifs Change-Id: I3208ff2f047da890edcc49b73389aca22442f5fc Reviewed-on: https://gerrit.libreoffice.org/22221 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-08vcl: cmdevt.[hc]xx -> commandevent.[hc]xxChris Sherlock7-7/+7
Change-Id: I77b30f28ae5a6fad360d7cada9acfaa9c324408b Reviewed-on: https://gerrit.libreoffice.org/22216 Reviewed-by: Chris Sherlock <chris.sherlock79@gmail.com> Tested-by: Chris Sherlock <chris.sherlock79@gmail.com>
2016-02-08sw lok: For PgDown/PgUp, don't set the VisArea, instead use a dedicated var.Jan Holesovsky4-2/+24
The m_aVisArea may be changed by many other means (called internally from LibreOffice), so let's introduce a dedicated offset for PgUp / PgDown handling overwriting the computed value (if set). Change-Id: I7c869b1e3582145b58f0185f4df2882d07a81ddf
2016-02-08cppcheck: noExplicitConstructorCaolán McNamara1-1/+1
Change-Id: I74585aa5a50cd335f4fd0a3ba4fed230b98993c3
2016-02-08remove unused ModulePrefix in *.SDI filesNoel Grandin1-4/+2
Change-Id: I92bb724259b6fd59af3a14810744cf1537ae2428
2016-02-08remove unused HelpText attribute from *.sdi filesNoel Grandin1-1/+0
Change-Id: I21bcf66c552cd38eaae1bdc85626aa9bd1782ebd
2016-02-08remove unnecessary module GUIDs in *.sdi filesNoel Grandin1-2/+0
Change-Id: I78784abc1031027d69bbe31d150bc78c8bfbfcf4
2016-02-08sw - m_eLastSearchCommand initializationMarco Cecchetti1-0/+1
Change-Id: Iafd72a05a745207dfff7663eeeb7bc697b5b8fd1
2016-02-08tdf#83890 sw html export: fix missing start numbering attributeMarc Bessières4-4/+15
In the sequence 1 Title 1 1.1 Subtitle 1.1 Not numbered item 1.2 Subitle 1.2 2 Title 2 The numbering element for the Subitle 1.2 was missing the html start attribute when it was followed by a numbering element of lower level. So let's force it, when it comes after a not numbered item. Adding a test case that validate that the Subtitle now contains the right attribute Removed a reference to an openoffice bug that brings no value too Change-Id: I928592a667f59128a4aa079d6b042ffb14244181 Reviewed-on: https://gerrit.libreoffice.org/22191 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
2016-02-08svx: SdrModel::mbTiledRendering is never readMiklos Vajna1-2/+0
Change-Id: I8571032b5c43a47872cb3364613ffb936624aa2d
2016-02-08Fix typosAndrea Gelmini1-1/+1
Change-Id: I4586168d3af81f047a4ded59fc6d257f17554885 Reviewed-on: https://gerrit.libreoffice.org/22194 Tested-by: Jenkins <ci@libreoffice.org> Reviewed-by: jan iversen <jani@documentfoundation.org> Tested-by: jan iversen <jani@documentfoundation.org>
2016-02-08remove some unnecessary declarations from *.sdi fileNoel Grandin1-2/+0
found with a bunch of grepping Change-Id: I920609fb7df5a5e55ecbf5c2bf4880ff972cfbed
2016-02-08loplugin:unusedmethodsNoel Grandin6-71/+1
using an idea from dtardon: <dtardon> noelgrandin, hi. could you try to run the unusedmethods clang plugin with "make build-nocheck"? that would catch functions that are only used in tests. e.g., i just removed the whole o3tl::range class, which has not been used in many years, but htere was a test for it... <noelgrandin> dtardon, interesting idea! Sure, I can do that. Change-Id: I5653953a426a2186a1e43017212d87ffce520387 Reviewed-on: https://gerrit.libreoffice.org/22041 Reviewed-by: Noel Grandin <noelgrandin@gmail.com> Tested-by: Noel Grandin <noelgrandin@gmail.com>
2016-02-07sw - single search after find all selects the 2nd occurrence - fixedMarco Cecchetti2-0/+15
Now, after a search all action we place the cursor at the beginning of the document so that the single search selects the first matching occurrence in the document instead of the second. Change-Id: I8c295bcd316c6197154c68ae97eb424ee6cc9904 Reviewed-on: https://gerrit.libreoffice.org/22175 Reviewed-by: Marco Cecchetti <mrcekets@gmail.com> Tested-by: Marco Cecchetti <mrcekets@gmail.com>
2016-02-06lok: Search result should contain info if it is a 'search all' result.Jan Holesovsky1-3/+4
Change-Id: Ia3ee81ced4f74c0d029a478bd59eff44d72ef327
2016-02-06vcl: bmpacc.hxx -> bitmapaccess.hxxChris Sherlock2-2/+2
Change-Id: I4bb19d6103c4a6a902d86b62a857e3478493924c
2016-02-06vcl/apptypes.hxx is goneStephan Bergmann1-1/+0
Change-Id: I6c3f7ffa8d7c1cac360ae28c1a52ef3b55da3a3d
2016-02-06vcl: split exception types from input typesChris Sherlock3-24/+31
Having them in the file apptypes.hxx isn't necessary helpful, IMO so I've split the types into inputtypes.hxx and exceptiontypes.hxx Change-Id: I89a1ff168c3ae276b2f5486669d4ec2dda062d57
2016-02-05interface to new XTextSearch2 with SearchOptions2, tdf#72196Eike Rathke2-4/+5
Places that had utl::TextSearch::UpgradeToSearchOptions2() introduced are worth an inspection if the new SearchAlgorithms2::WILDCARD search should be supported or at least use SearchOptions2 instead of SearchOptions to eliminate the small performance penalty that conversion involves. Change-Id: I565f73af2b551ae9ad0f488e672823dc6c5c1109
2016-02-05sw: move all SwXTextTable implementation details to pImplMichael Stahl2-69/+73
... and drop some obviously unnecessary SolarMutexGuards. Change-Id: Ie61198a264d52d1ebf7bd7c7de12a83845f4beb1
2016-02-05sw: remove redundant SwXTextTable::Impl::m_isDescriptorMichael Stahl1-9/+11
Change-Id: I51139c1c7658e6b71ebb704a8d02d1a8366a32d5