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

Problem Description : For some documents containing redLine data there was already processing done for paragraph properties pPr
but when it comes to DocxAttributeOutput::EndParagraphProperties() instead of writing these pPr, LO was calling Redline( pRedlineData )
which has it's pPr inside w:pPrChange.
This was the reason when LO calls WriteCollectedParagraphProperties() after calling Redline() it was writing both paragraph properties
inside single w:shd element.Hence the RT document was getting Corrupt.

Added condition in DocxAttributeOutput::EndParagraphProperties() which is checking for RedlineData and if it's there call
WriteCollectedParagraphProperties() before calling Redline().

Added Export Test case.

Change-Id: I7000c27fe6ee372cac81d6f22e0d3ca9219a24d7
Reviewed-on: https://gerrit.libreoffice.org/9712Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>

  - File attached to issue contains footnote between a table and reference
    of footnote is in field.
  - In PopFieldContext code was throwing exception due to null value of
    footnotestartnode.

Reviewed on:
	https://gerrit.libreoffice.org/9697

Change-Id: I467e234a7daa6a5f724087c2cc959e4c9d76f4d2

Change-Id: Ic301459b751585a91aa4d6667fbe512d13bbb458

...there was a call site that passed undefined "null pointer reference"
(apparently in a case where the passed argument was actually unused)

Change-Id: I663d4264b7a84f44ca69c732f3bc502f614b2b2a

File contains SDT content, in which after export databinding have duplicate attribute.

Conflicts:
	sw/qa/extras/ooxmlexport/ooxmlexport.cxx
Reviewed on:
	https://gerrit.libreoffice.org/9693

Change-Id: Ibe828964c054bdd5a63b5c0c903bc7441d953d72

	Issue :
	- In file there is a WordArt inside Table.
	- In RT, LO exports <w:drawing> inside <w:drawing>.
	- The xml sequence was :
  	  <mc:AltrnateContent>
            <mc:Choice>
              <w:drawing>                // first start of drawing
                <w:txbxContent>
                  <w:tbl>
                    <w:drawing>   // second start of drawing. MSO does not allow.

	Implementation :
	- In DocxAttributeOutput::OutputFlyFrame_Impl() under "case sw::Frame::eDrawing",
	  if drawing is already open then Postpone the Inner Drawing.
	- Added Export Unit Test case to check two separate drawings are
          now written in document.xml of RT.

Change-Id: I01d06621576d60d26cd51489ee9718dd79eb9c72
Reviewed-on: https://gerrit.libreoffice.org/9653Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>

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

Change-Id: I78a388385d4629bda35af8c08445f21ac547e86b

Change-Id: Idff480084b22a5db6a0f00f90b7febc8488ba2af

Change-Id: I56ae09cb89da7d078f64906e15677de58296c119

7b72e505 removed the need for that.

Change-Id: I1fa635eab33bf259bcb4e229e9a0deac5fc987d6

Change-Id: I3d5ddbca7760a32231ef7fee19b45bca83667ca7

Change-Id: Ib92dc55827b69cf33a04fe4084cf296049efc9c6

Change-Id: Ie2c64d3254dba35ba3fcb1af2566de84b8b300e2

No idea why it crashes whatsoever....

Change-Id: Ic46d5a98f21cacb03adb3bc0d6da239bde5d8ce2

Change-Id: Iddd0299f2229383c0d8368d117e6175887931978

Change-Id: I8a7e1d0a42aacdc7dcaeda0f7e51db78727f4b54

Change-Id: If83bcfb0e39c0b41ea5323b0ad3832bd65185c6c

...so CrossRefBookmark::GetOtherMarkPos can legitimately return a null pointer

Change-Id: I88774cf0d139d017d67d6727ee01214997a73636

Change-Id: Ib585af0755bdc6f7aeebc89d5d109d3bb6084e22

Change-Id: I7b1252656a6ada4ab86151462629aaa952011b70

Change-Id: I52d9ae4daf8e4f904f18b150a4adaad9d2921e61

Change-Id: I8f9df1d92c86c087e37f77209f413ff8c1c6358f

Change-Id: I4b51abf6193c33367edebc82772054875b5b9bba

Change-Id: Ia08f67e359bbd26cefdba8661f0b0c4ae2147382

Otherwise parts of the image are missing.

Change-Id: If9018c32807359862a9be825b0ebdaacce2d3490

Change-Id: Ia3fe1ff0075434d0b6903bf978c16499e07d699d

Change-Id: I5863de6d8b1f0fe2419ea7419b1ff90087bb513d
Reviewed-on: https://gerrit.libreoffice.org/9715Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

Change-Id: I24ee2599ffe8029c3d3888de49cff8bfd0a368e0
Reviewed-on: https://gerrit.libreoffice.org/9723Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

The import mechanism of custom-dash (a:custDash) was wrong, and imported
wrong values, which causes that if you would import-export-import-export -
you would get inflated values, which might cause a corruption.

The attributes for custom-dash nodes (a:ds) are of type 'PositivePercentage'.
Office will read percentages formatted with a trailing percent sign or
formatted as 1000th of a percent without a trailing percent sign, but only
write percentages as 1000th's of a percent without a trailing percent sign.

During import - LO did not check if it was in '%' format or in
'1000th of a percent' format. So that was fixed. Also - when exporting -
it always exports now in '1000th of a percent' format.

Change-Id: I6bd74df26951974f85173227c832386c70034afb
Reviewed-on: https://gerrit.libreoffice.org/9681Reviewed-by: Miklos Vajna <vmiklos@collabora.co.uk>
Tested-by: Miklos Vajna <vmiklos@collabora.co.uk>

Change-Id: I14c653f8b3fe0dc9855aa51643cfc40cbccb763e
Reviewed-on: https://gerrit.libreoffice.org/9714Tested-by: Michael Stahl <mstahl@redhat.com>
Reviewed-by: Michael Stahl <mstahl@redhat.com>

Change-Id: Ia403e6c6c6747479adfc5d77b1306d9383b3e706

We don't actually need to check mbAnchorCtr to change
text spacing. This txXfrm workaround works only with rectangles,
because other shapes' text area can be smaller then the shape
size. So add some condition to avoid using it for
other shapes.

Plus fix typos cause regression introduced in:
53c376d3

Change-Id: I87917b8e0b2bb97ae1bba773e7dda7f81682736f

Change-Id: Id17c7d4cf9c9bcb81d0da486c9bce44abb04b6cd

Change-Id: Ie20e0ca939139c952f3524c7dae494e24bc7cab7

Change-Id: I5871ca0c3eea931e2144d7a469844cf3a69d0038

It's possible to write this tag in oox (so it represents the properties
of the shape) or in sw (so it represents the properties of the shape's
textbox). Do the previous, as the textbox is really just a container in
this use case, nothing more.

If we are at it, also fix the default value of <wps:bodyPr>'s l/r/t/bIns
attributes.

Change-Id: I0571b9d8ea7dc0acd5c61f3e28e18400d305eab3

to avoid problem with ucpp

(cherry picked from commit 6e370ddd)

Conflicts:
	idlc/source/options.cxx

Change-Id: Ia8c7dc3a9a7df68a483dd106cd47c8624952aeae

...there were a number of call sites that passed undefined "null pointer
references" (apparently in cases where the passed argument was actually unused)

Change-Id: I19799e90f0cd8e98367782441a5ea9df27b59830