Apply to/reset only the paragraph properties node.

Change-Id: I4dd3973a758f4afcc30f905b7351e9687d050e33
(cherry picked from commit c2932c10)

Change-Id: I5131f5a3ff261a4d86e65b4afff3eb8cdd98cebf
(cherry picked from commit 68093f95)

Skip hidden nodes, apply to properties-node.

Change-Id: I9c511e87c431bd39dc6efdcddf2cad9bbf18c592
(cherry picked from commit 6bbec38a)

Change-Id: Iaf6101f02bd5dbeb9a07e796932186b5e2b1a563
(cherry picked from commit 3b3a81ce)

It's really quite annoying to have RES_BREAK applied to all nodes; so
apply the node-items only to the first-/properties-node and skip hidden
nodes.

Keep applying text/character items as before, except skip over hidden
nodes.

Change-Id: I085afd59af78caf1ca8dac4080c53bdbdb4e56a1
(cherry picked from commit c253dde8)

MergeValues would set everything to invalid.

Change-Id: I45795fe9bd113399e276ac42851cbf5a198c9694
(cherry picked from commit 2ad1767c)

Merge items from first-node and props-node, as they are used in the
text formatting.

Change-Id: I4cbab55bc3b0f8bc6dc5bd3577da53b3160018a3
(cherry picked from commit eb3481d9)

Change-Id: I1de6b544a4868f0d30077ba2c6d195bffd2e8b00
(cherry picked from commit 2b610430)

Change-Id: I78d32031ebc42d749dbc750b7127983810bbfb1d
(cherry picked from commit cb53a34d)

If there are hints, the aFormatSet.Differentiate(rSet) will cause the
pre-existing items in the parameter rSet to win, whereas without hints
the SwContentNode::GetAttr() calls rSet.Set() which clears everything.

It looks like WW8Export::WriteFootnoteBegin() was sort of relying on
this; presumably it wants the pCFormat to win over node's items but
hints on the position to win over pCFormat.  Well that's what it
currently gets anyway, so try to preserve that.

The other use in SwUnoCursorHelper::SetPropertyValues() looks like an
oversight.

Change-Id: I80e18e040413fcf26f6f77844ed731de31a464d2
(cherry picked from commit a91992e7)

... so handle them as well in SwTextFrame::SwClientNotify().

Change-Id: Ifa8bf8f44bcd08a64dc36939bad58efefbd7af55
(cherry picked from commit 52e8b41f)

see commit 2136dc24 and commit e721f958

Change-Id: I07ace74f93adc2568003314b5f05c41611d6f77c
Reviewed-on: https://gerrit.libreoffice.org/64671
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit a10dd637)

Change-Id: I9891a20d3bc3de51fec84fb54c7aae42f7142662
Reviewed-on: https://gerrit.libreoffice.org/64329
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <tietze.heiko@gmail.com>
Tested-by: Heiko Tietze <tietze.heiko@gmail.com>
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit 0a139d98)
Reviewed-on: https://gerrit.libreoffice.org/64747

Change-Id: I56c68b74daa701e0e27ca4b661ece12bf20a8776
Reviewed-on: https://gerrit.libreoffice.org/64729
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
(cherry picked from commit 7c3c2ef4)
Reviewed-on: https://gerrit.libreoffice.org/64748

Need to change parent of an SdrObject in the
according SdrObjList methods before calling
InsertedStateChange - that one relies on asking
for the parent of the SdrObject

Change-Id: I8c4d39190cafe889b27975941d72afca870e2ce5
Reviewed-on: https://gerrit.libreoffice.org/64608
Tested-by: Jenkins
Reviewed-by: Armin Le Grand <Armin.Le.Grand@cib.de>
(cherry picked from commit b38f856f)
Reviewed-on: https://gerrit.libreoffice.org/64706Tested-by: Xisco Faulí <xiscofauli@libreoffice.org>
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>

...ever since f1579d3d "tdf#117171 support
localized number name numbering styles"

Change-Id: I93dcfa9f1310b35e3068a7e0efdce258364a7d78
Reviewed-on: https://gerrit.libreoffice.org/64720Reviewed-by: László Németh <nemeth@numbertext.org>
Tested-by: László Németh <nemeth@numbertext.org>
Tested-by: Jenkins
(cherry picked from commit 82af7584)
Reviewed-on: https://gerrit.libreoffice.org/64743Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

756c9f23 "Merge upstream fixes into
solenv/flatpak-manifest.in" had introduced it (via <https://github.com/flathub/
org.libreoffice.LibreOffice/commit/7d69e505165d4bbf5fa0b34b5fb2cbff9dfaf9bd>
"For now, bundle gst-libav to be able to play mp4 videos in impress" to fix
<https://github.com/flathub/org.libreoffice.LibreOffice/issues/20> "Impress
don't insert a video" back when the LO flatpak was based on
org.gnome.Platform//3.26 which doesn't contain
/usr/lib/gstreamer-1.0/libgstlibav.so.  (And
/app/lib/gstreamer-1.0/libgstlibav.so is apparently the only file that the
gst-libav module contributes to the LO flatpak.)

When 727bfa2c "Merge in Flatpak improvements"
switched the runtime to org.freedesktop.Platform//1.6 (via
<https://github.com/flathub/org.libreoffice.LibreOffice/commit/
7a2393f5fcab8c6938180c4718f2c61acd495528> "port to freedesktop.org 1.6 runtime")
it kept the gst-libav module even though org.freedesktop.Platform//1.6 contains
/usr/lib/gstreamer-1.0/libgstlibav.so.  Lets assume it was kept by accident and
can be removed.  (At least the scenario from <https://github.com/flathub/
org.libreoffice.LibreOffice/issues/20#issuecomment-368543622> "Impress don't
insert a video" still works with this change.)

Change-Id: I692d01bfe5882c05d63124db7cdfa6390540e69f
Reviewed-on: https://gerrit.libreoffice.org/64703
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
(cherry picked from commit 3c015ddf)
Reviewed-on: https://gerrit.libreoffice.org/64731

cause command isn't available any more

Change-Id: Ie73491e02979fe24f59e8c627b69f902a12b512a
Reviewed-on: https://gerrit.libreoffice.org/64711
Tested-by: Jenkins
Reviewed-by: andreas_kainz <kainz.a@gmail.com>
(cherry picked from commit 7dbd56d5)
Reviewed-on: https://gerrit.libreoffice.org/64737

Change-Id: Id3b338278e6d47b90268f0bdc77f1bdaa495edb8
Reviewed-on: https://gerrit.libreoffice.org/64724
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Adhere to what gtk3's file picker does as well.
Threading is involved in Gtk3KDE5FilePickerIpc::execute().

Change-Id: I4fa0a12f46ed13ef04211cb8577e77d1db5e8eae
Reviewed-on: https://gerrit.libreoffice.org/64502
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
(cherry picked from commit 8dbe0af7)
Reviewed-on: https://gerrit.libreoffice.org/64585Reviewed-by: Katarina Behrens <Katarina.Behrens@cib.de>

Change-Id: I6208cf21f53d1a2a0eb4f08962c7bddcc372aab5
Reviewed-on: https://gerrit.libreoffice.org/64545
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 581c7e4a)
Reviewed-on: https://gerrit.libreoffice.org/64612

since...

commit e194f597
Date:   Sun Sep 30 16:07:16 2018 +0200

    Change GlyphItem::nFallbackLevel to font instance

Change-Id: I14b40a63cd48ef527e76e15af8f421ce48cce769
Reviewed-on: https://gerrit.libreoffice.org/64699
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Id05ad8b72e0ec4f9bad2a9cfb780a44a27618d37
Reviewed-on: https://gerrit.libreoffice.org/64701Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: If93a9ea6e0265ba616b4a1fbe138d310ea3e8693
Reviewed-on: https://gerrit.libreoffice.org/64546
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 3156228b)
Reviewed-on: https://gerrit.libreoffice.org/64613

Change-Id: I15636a64ad869bf874fc6e2ada87842050da1a9a
Reviewed-on: https://gerrit.libreoffice.org/64544
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 866eb1c4)
Reviewed-on: https://gerrit.libreoffice.org/64611

Change-Id: I1f1945e7fd2721a320ab11196385e22a70d3da70
Reviewed-on: https://gerrit.libreoffice.org/64542
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 9881ac68)
Reviewed-on: https://gerrit.libreoffice.org/64610

Change-Id: Ibbf374f72e83609a602504d9ffde922896338c2c
Reviewed-on: https://gerrit.libreoffice.org/64543
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 79ed7bba)
Reviewed-on: https://gerrit.libreoffice.org/64609

Change-Id: I73bc93ebb5010f3a376b3b121e1ac873f1be137a
Reviewed-on: https://gerrit.libreoffice.org/64677
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

(cherry picked from commit d1a1c9fa, adapting
the pdfium block, which has only been upgraded to 3550 on libreoffice-6-2)

Change-Id: Idffd274a18df14a96e61bf61ba09f837fdbbf9c7
Reviewed-on: https://gerrit.libreoffice.org/64670
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

It had originally been read as 16 bit via >> in
d2000efb "#i106421#: move msfilter to filter",
then accidentally changed to be read as 32 bit via >> in
9830fd36 "date/time IDL datatypes incompatible
change" (and later changed to be read as 32 bit explicitly via ReadUInt32 with
15535e32 "convert SvStream::operator>> methods
to ReadXXX methods").

Change-Id: I5062e67a578d182a0df2726ab8d0bae465f154f3
Reviewed-on: https://gerrit.libreoffice.org/64604Reviewed-by: Lionel Elie Mamane <lionel@mamane.lu>
Tested-by: Jenkins
(cherry picked from commit f84f5b0a)
Reviewed-on: https://gerrit.libreoffice.org/64635Reviewed-by: Stephan Bergmann <sbergman@redhat.com>

This reverts commit 85818da3 which
caused bug 121110. Only reverting from 6.2 because I hope to fix
this for the current master 6.3, but I won't backport the fix
because it isn't trivial.

Change-Id: Ie91b707fbb05fbc5c6806aef7d1760ab69abb277
Reviewed-on: https://gerrit.libreoffice.org/64660
Tested-by: Jenkins
Reviewed-by: Justin Luth <justin_luth@sil.org>

is that how it should work, or should there be an additional TogglePause
created at import time to get the apparent intended pause effect

Change-Id: I0573bd8d9534271f2b36d7ab3b8cdc20e0f985a1
Reviewed-on: https://gerrit.libreoffice.org/64548
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <Thorsten.Behrens@CIB.de>
(cherry picked from commit 1aa10fcb)
Reviewed-on: https://gerrit.libreoffice.org/64620

Basically this was always wrong.

Original code didn't treat the r attribute as optional and
resulted in an invalid -1 row for such cases, hence the RowModel
wasn't applied later.

Then

commit af8e6f4f
CommitDate: Wed May 29 22:05:02 2013 +0200

    r is an optional attribute in c and r, fdo#65059

introduced a 0-based SheetDataContext::mnRow but assigned that to
a 1-based RowModel::mnRow as well if r attribute was missing, and
did not update SheetDataContext::mnRow if r attribute was given.

commit ff56553e
CommitDate: Thu Mar 13 18:39:22 2014 -0400

    fdo#76032: This row index is 1-based whereas our own mnRow is 0-based.

then correctly assigned the decremented r attribute to
SheetDataContext::mnRow but did not touch the assignment to
RowModel::mnRow if r was not given. Probably confused by the
++mnRow already there which is correct but not sufficient for the
assignment.

Which we now finally fix herewith..

Change-Id: Iae1563ed01b95e7852ddad4fd59374c07186b493
Reviewed-on: https://gerrit.libreoffice.org/64580
Tested-by: Jenkins
Reviewed-by: Eike Rathke <erack@redhat.com>
(cherry picked from commit 76fbe494)
Reviewed-on: https://gerrit.libreoffice.org/64638

it's still used on Mac. Revert it until XOL is removed

This reverts 25830187

This also includes https://gerrit.libreoffice.org/#/c/64596/
Thanks Stephan

Change-Id: I06548a590f370618ad640724a1b9c59a3faceec2
Reviewed-on: https://gerrit.libreoffice.org/64582
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
(cherry picked from commit be504e8c)
Reviewed-on: https://gerrit.libreoffice.org/64631Reviewed-by: Xisco Faulí <xiscofauli@libreoffice.org>

This restores compatibility for API users who called getByName() on the
bitmap table and expected that the result can be set as a value for the
GraphicURL property.

The case is similar to the Writer images, which was handled in commit
587803ba (sw TextGraphicObject: handle
XBitmap for GraphicURL, 2018-08-08) already.

Change-Id: I3740a68989b306425d0d6c4d4e350ac163cb4170
Reviewed-on: https://gerrit.libreoffice.org/64069
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
(cherry picked from commit e30f3e76)
Reviewed-on: https://gerrit.libreoffice.org/64427

Make sure that our normal SAL_WARN() reporting is registered during the
xmlSecCheckVersionExt() invocation, just like it's registered during
other libxmlsec API calls.

(cherry picked from commit 099ab643)

Change-Id: Ia19368b55dbae47974f48954f86b50e36d1462ff
Reviewed-on: https://gerrit.libreoffice.org/64605
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

Change-Id: Ie171eea334de65b993554146c11adb95e0dbb85e
Reviewed-on: https://gerrit.libreoffice.org/64245
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 9a446225)
Reviewed-on: https://gerrit.libreoffice.org/64535

It's silly to just return NaN just because the function can't handle
something. If nothing else, at least a proper error should be reported
(or in this case, the fallback to the core function will take care of it).

Change-Id: I9c971082f4c5c9836318cf63d15fa7c278274273
Reviewed-on: https://gerrit.libreoffice.org/64244
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit c0c03aad)
Reviewed-on: https://gerrit.libreoffice.org/64534

Change-Id: I885a1b70556311f5a0862fec24a3596dbeada5d6
Reviewed-on: https://gerrit.libreoffice.org/64243
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 93054cd0)
Reviewed-on: https://gerrit.libreoffice.org/64533

sc_subsequent_filters_test in ScFiltersTest::testFunctionsODS() also
checks "SERIESSUM(2;3;2;3)", which the opencl code does not handle.
At least bail out gracefully instead of returning 0.

Change-Id: I154dca8cc437a6225b4eb98012232d80683f0114
Reviewed-on: https://gerrit.libreoffice.org/64242
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
(cherry picked from commit 6a2241e7)
Reviewed-on: https://gerrit.libreoffice.org/64532