bump product version to 3.5.7-rc1, release number to 1

Change-Id: If3e938bf8c4850929050abff92907011e92f54f7

Branch libreoffice-3-5-7

This is 'libreoffice-3-5-7' - the stable branch for the 3.5.7 release.
Only very safe changes, reviewed by three people are allowed.

If you want to commit more complicated fix for the next 3.5.x release,
please use the 'libreoffice-3-5' branch.

If you want to build something cool, unstable, and risky, use master.

Change-Id: I1a6089f9a57ca07530aaa14d8d9d2170bcc0107f

fdo#54232 set encoding of Romanian MSI tables to UTF-8

Change-Id: I0604998e5019757c47050e4f94b1feb1b847cfca

Signed-off-by: Fridrich Štrba <fridrich.strba@bluewin.ch>

fdo#37529 clear DataColumns read-only information when we clear DataColumns

Change-Id: I6f3b4f1646df1244d97ae98f56ba054bc26e3314
(cherry picked from commit 522b4c65dcc90719288b4f7aa7eb565c15b64e86)

Signed-off-by: Miklos Vajna <vmiklos@suse.cz>

Bin SAL_UNUSED_PARAMETER

Change-Id: Ieee86e5d024c650065e855a9213e2f7421493d17
Signed-off-by: Andras Timar <atimar@suse.com>

fdo#45700: sw ODF export: workaround corrupted table model:

do not export invalid table:number-columns-spanned="0"

(cherry picked from commit c45c64e3de51d9f56c9d9789729b6f7952547a61)

Conflicts:
sw/source/filter/xml/xmltble.cxx

Change-Id: I7e286a9fcb6ed3df7222c83fe73e870e69561649
Reviewed-on: https://gerrit.libreoffice.org/613
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>

fdo#44721 protect against negative calculated height value

Change-Id: I4faedd535421eaf1e3e425f4731b3bb8831e7360
Reviewed-on: https://gerrit.libreoffice.org/623
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>

fdo#49677: ODF export: always add xlink:type attribute

The xlink:type was optional on text:a and draw:a elements in ODF 1.1 but
is mandatory in ODF 1.2

Change-Id: Ifc7301ed1b0120d31a156751888dea1a55e952fe
(cherry picked from commit a27a6b68b7e4d2a4674b4de59fc487b17c7c9907)
Reviewed-on: https://gerrit.libreoffice.org/615
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>

fdo#49876: ODF export: do not export font-size="0pt"

This is invalid, font size must be positive.

Change-Id: I2425d7c596ca9c7c80c0d24c44cdd9ec1ac1dd80
(cherry picked from commit c9699f82edf67a55f4cfdd885fa779b13d816c87)
Reviewed-on: https://gerrit.libreoffice.org/620
Reviewed-by: Fridrich Strba <fridrich@documentfoundation.org>
Tested-by: Fridrich Strba <fridrich@documentfoundation.org>

fdo#49678: ODF export: add svg:viewBox to draw:connector

This attribute is mandatory; for the given bugdoc various
svg:viewBox="0 0 2500 1" are now written.

Change-Id: I7ea2294e0796546dcbb245993c01c2dd2b2a755a
(cherry picked from commit a2959aba6affa06dc0c6fb19d0c2b73958e6b221)
Reviewed-on: https://gerrit.libreoffice.org/618
Reviewed-by: Thorsten Behrens <tbehrens@suse.com>
Tested-by: Thorsten Behrens <tbehrens@suse.com>

fix for fdo#49255

This patch disables row height calculations ( and uses the hard heights read
from the xml instead ) The bug here partially exisits because the height
calculation doesn't work correctly because the factor from
ScDocShell::GetOutputFactor() has not been calculated yet ( and has a default
value of 1 ) This number is used in setting up the correct nPPTX and because it
isn't setup correctly the height calculated for the text is incorrect. We use
EnableAdjustHeight to inhibit auto height calculations that result from various
changes ( like style changes etc. ) from the start of the load and re-enabled
it at the end.

Change-Id: Ib6715c39b7cb5793458f15b1f0b3c9ecd16412f9

Signed-off-by: Fridrich Štrba <fridrich.strba@bluewin.ch>

fdo#53872: reportdesign: fix section drawpage crash:

In order to wrap the SdrPage's UNO object completely, set its mxUnoPage
member to the OSection wrapper instance in OSection::init; only OSection
should have access to it.
Also initialize m_xDrawPage_Tunnel (thanks Lionel for the hint).
(regression from 05218c101df486302bf4cfe8be23ad840daa3f73)

(cherry picked from commit 1146b889325fd01f24c6aacf557d3940dd147936)

Conflicts:
svx/inc/svx/svdpage.hxx

Change-Id: I048ddafc31e946853e56e6a403ddc9487cfbcf0e
Reviewed-on: https://gerrit.libreoffice.org/559
Reviewed-by: Miklos Vajna <vmiklos@suse.cz>
Tested-by: Miklos Vajna <vmiklos@suse.cz>

fdo#42450: fix crash in SwXTextSection::attach:

Inserting the section can fail if the given text range is not valid.

(cherry picked from commit 5bc6c7b2e170a35914d7cd07347c77a9c9d23664
 and commit 2940a697dce2292e9c5b118feb8ec4159e3606fa)

Change-Id: Ib6ba3b02dd581dce08b646b841354073caf894b2
Reviewed-on: https://gerrit.libreoffice.org/604
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Reviewed-by: Miklos Vajna <vmiklos@suse.cz>
Tested-by: Miklos Vajna <vmiklos@suse.cz>

fix linking error with boost 1.50

(cherry picked from commit 43a20bf6fc3213c38a77a2941a4999313bdfefbe)

Conflicts:
mdds/makefile.mk

Change-Id: Ib0892a4ff55fc8036feedad3b3fd093c86d0cb1a
(cherry picked from commit 547e2d66f590a076eed0941d6ae47c5727a6ed40)

Signed-off-by: Michael Stahl <mstahl@redhat.com>

MinGW: snprintf might be present.

Signed-off-by: Luboš Luňák <l.lunak@suse.cz>

fdo#38913: Prevent invalid parameter handler crashes

It appears that on Windows at least some jvm.dll versions can cause calls to
_fileno(NULL), which leads to a call of the invalid parameter handler (see
<http://msdn.microsoft.com/en-us/library/ksazx244%28v=vs.80%29.aspx> "Parameter
Validation: Visual Studio 2005").  The default handler causes the application to
crash, so install a "harmless" one instead.

Change-Id: Id6a3ffb63f70b0c65546bc933e994c8dbf35203c
(cherry picked from commit a82e532ce006c54b2740de74d1da5d11307da7c1)
Reviewed-on: https://gerrit.libreoffice.org/565
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

missing part of fix for fdo#54721

commit id cea414bfe8fe356793778bdf09f5ca3b3a42daa0 for
"fix vba not tracking currently selected doc correctly" was missing necessary
change in sfx2

Conflicts:

sfx2/source/doc/objxtor.cxx

Change-Id: I1697c005374b3d26dc63c78702a46c6d9879adaa

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#54721 fix vba not tracking currently selected doc correctly

Change-Id: I43b17dfc621d0aba76cdd33ebf97c457fa1da912

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#54718 fix opcode detection in basic resulting in failed/unregcognized code

fix is followup to bf5b493104d2dd4ab964f1fcb845200eaefbbcd5 which didn't adjust the runtime to
cater for the new enum layout

Change-Id: I6613fb8aacd8a70947c4fff556fb3e2d33c1113e
Signed-off-by: Petr Mladek <pmladek@suse.cz>

don't use unitialized global variable, fdo#53713

Change-Id: I9e5150fb4c5681abe9c786f4c2d80f5a10e8a66b
(cherry picked from commit 2ac26e6b5248f2f3fde8dc4341b01ef15b6c3a29)

Signed-off-by: David Tardon <dtardon@redhat.com>
(cherry picked from commit 8a861b99ee0b53d03b6f93a3bd6d256a27172ace)

fdo#53557: use metadata's idea of column count to allocate a row

m_pColumnNames contains only the columns that come straight from a table, not calculated columns

Change-Id: I2706790e8c1447d1c4e144bbdef773deeca3e2e8
Reviewed-on: https://gerrit.libreoffice.org/522
Reviewed-by: Michael Meeks <michael.meeks@suse.com>
Tested-by: Michael Meeks <michael.meeks@suse.com>
Signed-off-by: Michael Meeks <michael.meeks@suse.com>

fdo#50584 bundle MS Visual C++ 2010 Runtime so LO can work with JRE7

(cherry picked from master commit 8b749cb16dde14b199a61456741b586ddd88a584 plus
238eba337cedda17a9ed1ce7e91755c66446195a "make the vc2010 redist optional" to
libreoffice-3-6 as 6a59a378f6d492aad09ae254a024983eb9ee8060, cherry-picked to
libreoffice-3-5 from there)
Conflicts:
oowintool

Change-Id: If22bed6728121ec66ecd89ad4f869e980aeb79b7
Signed-off-by: Stephan Bergmann <sbergman@redhat.com>

multiline tooltip for very long (eg. footnote) texts #i42424#

Change-Id: Ib6383220c3cdd6fef55dee82fcfdc3472da45163
Signed-off-by: Andras Timar <atimar@suse.com>

fdo#53872: reportdesign: fix section drawpage crash:

In order to wrap the SdrPage's UNO object completely, set its mxUnoPage
member to the OSection wrapper instance in OSection::init; only OSection
should have access to it.
Also initialize m_xDrawPage_Tunnel (thanks Lionel for the hint).
(regression from 05218c101df486302bf4cfe8be23ad840daa3f73)

Change-Id: I048ddafc31e946853e56e6a403ddc9487cfbcf0e
Signed-off-by: Lionel Elie Mamane <lionel@mamane.lu>
(cherry picked from commit 0e6054f8f5e2d41e50a50645defa5861599fe375)

Signed-off-by: Lionel Elie Mamane <lionel@mamane.lu>

SwTokenWindow::InsertAtSelection: fix STL assertion:

 error: attempt to copy-construct an iterator from a singular iterator.
(regression from 39b8a5f87f55abe53488c9c3c35b65fb0df84cde)

Change-Id: Iab29f4c356ea1cb5ca0f687bcfc5e54f185fbba3
(cherry picked from commit c47505a4525c342694ba4196544715467c2bdb8e)
Reviewed-on: https://gerrit.libreoffice.org/460
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>

SwHeaderFooterWin: memory leak

Change-Id: Ie8322b93745c48970766c2033a23cca2c710a516
(cherry picked from commit 442d0766eb9575a45572e29f2018bed2831c177f)
Reviewed-on: https://gerrit.libreoffice.org/467
Reviewed-by: Bosdonnat Cedric <cedric.bosdonnat@free.fr>
Tested-by: Bosdonnat Cedric <cedric.bosdonnat@free.fr>
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>

SwPageBreakWin: memory leak

Change-Id: Ibb543b8da8c89c7fa3799fada03c787dabe2b3cb
(cherry picked from commit a251863970e0512a1dc2bd857913a3adf7148bb3)
Reviewed-on: https://gerrit.libreoffice.org/473
Reviewed-by: Norbert Thiebaud <nthiebaud@gmail.com>
Tested-by: Norbert Thiebaud <nthiebaud@gmail.com>

fdo#36149: Do not display error dialogs while showing a menu

...it apparently leads to crashes, but is probably not good from a usability
perspective anyway (as the menu closes again when the dialog appears/is operated
on by the user).

For now, just disable the Java specific interaction handler here; might make
sense to address this more generally though (there's framework::QuietInteraction
btw).

(cherry picked from commit 9cfc64c66ff9e0f0251a006f039bf4e26a63147a)

Change-Id: I6ae303c0084549b5339d219e158cdb89e5a6b331
Reviewed-on: https://gerrit.libreoffice.org/505
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

fdo#49901: SwView::ExecSearch: fix Replace All cursor:

Because the "fix" for i#8288 only set the point of the cursor and not
the mark, it can happen that after a Replace All weird things are
selected; fix that (and i#8288 properly) by using the SwCrsrShell stack,
and don't do it when searching in the selection.
This cannot be done in SwCrsrShell itself because ExecSearch sets the
cursor to the beginning/end of the document already.
(regression from cda75d6f358d438f7ed36ed7fa296ac23e3c4a04)

Change-Id: Ie9f8d7da2fa26000a0237c24f8742e47ffa29213
(cherry picked from commit b3233579ccb2528912835deedde37c1d48417d6c)
Reviewed-on: https://gerrit.libreoffice.org/482
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

fdo#51514: SwXBookmarks: only consider real bookmarks:

Since CWS swrefactormarks2 the SwXBookmarks collection handles not only
bookmarks but at least cross-ref marks as well, which then bother users
when they show up in the Insert->Hyperlink dialog; remove non-bookmarks
again.
(regression from df6d312ca537402463e4eb0530f22b956600fc02)

Change-Id: I6a64ba8a43468dd3ce1569e944371d3ef71f8824
(cherry picked from commit 45be3ac8151d63ccb61879f876696704542a4ce7)
Reviewed-on: https://gerrit.libreoffice.org/461
Reviewed-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>
Tested-by: Björn Michaelsen <bjoern.michaelsen@canonical.com>

fix math export/import in docx/rtf

For some reason older gcc versions don't manage to dynamic_cast
to the necessary cast. I'm not quite sure why, forcing sal/osl/unx/module.cxx
to always use RTLD_GLOBAL does not seem to help. Most probably
compiler bug. Changing the cast to two simpler ones helps.

Change-Id: I870d51fff00ee4dfc74c4f872ad8eeac28b374d2
Signed-off-by: Miklos Vajna <vmiklos@suse.cz>

fdo#46249: UCBContentHelper::GetTitle must not fail for void property

This was a regression introduced with 2af9040d38af7c7353855415dbea0134585058f3
"Cleaned up utl::UCBContentHelper."  GetSize exhibited the same problem and has
also been fixed.  IsYounger exhibits a similar problem, but has not been
addressed, as it is unclear what to return in case no dates can be compared; it
is only used in one place (handling SID_EDITDOC in
SfxViewFrame::ExecReload_Impl, sfx2/source/view/viewfrm.cxx) where, it appears,
only contents for which DateModified /does/ yield a non-void value are relevant.

(cherry picked from commit 27c7682e5e5859eb353443d6091143694a56bee8)

Conflicts:
unotools/source/ucbhelper/ucbhelper.cxx

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

fdo#47434: Zero rect. size causing wrong line positions.

Thanks to Korrawit Pruegsanusak <detective.conan.1412@gmail.com>
for the indepth analysis and debugging.

Signed-off-by: Petr Mladek <pmladek@suse.cz>

sync cell context menu & edit menu 'PasteSpecial' entries for calc fdo#46250

calc application specifically handles PasteSpecial so no need to set the state
of the menu entry to enabled always ( for calc at least )

Change-Id: Iaa4b3713f55bff9e129f8f9ee7105650122b7145
Signed-off-by: Kohei Yoshida <kohei.yoshida@gmail.com>

fix for fdo#53229, import group shapes position and size correctly

a) need to properly calculate the topleftmost position for the combined shapes
   in the group.
b) when you have groups within groups then the offsets need to be applied to
   the topmost parent.

Change-Id: Ie1967f3625030fcd0201f2ab43a0740153cd4bd7
Signed-off-by: Kohei Yoshida <kohei.yoshida@gmail.com>

fdo#47311 don't crash when pasting into more than 1 sheet

Change-Id: I61c2c988eedd91f44a0b9c2553f69a692f47dbfb

ensure placeholder substitution on XclImpDrawObjBase::ReadObj8 failure

Change-Id: I7a72153d971aaccea937955acc77cdba128985d7
Signed-off-by: Michael Stahl <mstahl@redhat.com>

validate polypolygon point counts

Change-Id: Ibf6bdf48e5855583f14cd2be36f1e4896a396d32
Signed-off-by: Michael Stahl <mstahl@redhat.com>

fdo#43433: Binary URP works gracefully with old Java URP

...which did not support protocol properties yet.

Change-Id: Ic5bb346764fb039856e16169d5ae96d01422e7e0
Signed-off-by: Michael Stahl <mstahl@redhat.com>

Fix fdo#47495 backport

On master, the fix was fine, but on -3-6 this broke the fdo#47036 unit
test, fix this.

Change-Id: I02449a41801646c7eb2408e60f9051e1269d6693

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#47495 fix RTF import of multiple shapes inside textframes

(cherry-picked from commit ad72a47dbd16f65316432df71dffe3b9b835fcc7)

Change-Id: Iee8541ff9e88139648d45f08968476f2b1d4e1b3

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#53210 SwDoc::UpdateExpFlds don't crash when hiding all sections

(cherry picked from commit bb6bd1ff9cd3eecec7eb2cd7bd0a4dcef584c903)

Change-Id: I9b7a61f18f987214708195a89f6e346c865c7f9f

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#48446 implement RTF_CPG

(cherry picked from commit f6a24ace5ad12e79f0cc90709a290a30e3758781)

Change-Id: I2fb2004f2f755ce337ddd1419acf2510f17622f3

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#50654: update virtual device width on modifying WB_EXTRAFIELD flag

Writer sets this flag dynamically via Ruler::SetStyle depending on
the text direction, the flag is set by default and gets unset for the
vertical ruler, because the initial text direction is horizontal,
see SwView::StateTabWin.

Ruler::SetStyle calls Ruler::ImplInitExtraField, which modifies mnVirOff,
however mnVirWidth depends on mnVirOff, but gets updated only in Resize.

This patch copies the code from Resize to ImplInitExtraField, we cannot
just call Resize from ImplInitExtraField due to possible infinite recursion.

Change-Id: Ic7bb897059295aebe86c11977c37419017b55787
Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#53154: fix report design wizard crash:

As described by Stephan in the bug report, the XAggregation usage in
OSection of SvxFmDrawPage is broken; replace it by explicit forwarding
of the XDrawPage, XShapeGrouper, XFormsSupplier2 methods; it seems
to fix the crash here.  Also get rid of horrid refcount manipulation
etc. while at it.

(cherry picked from commit 8ec791316c1d90b25e85c94ad105e9cc62e568e2)

Change-Id: Ib08830d2092bd2026db5b57c1c2e165b946889b3
Signed-off-by: Miklos Vajna <vmiklos@suse.cz>

n#759210: Certain smartart drawings weren't imported.

SmartArt info was reset, when there was a chart (e.g.)
in the same slide.

This also fixes fdo#50907.

Conflicts:

oox/source/drawingml/diagram/diagram.cxx

(cherry picked from commit 11c5699dad06fb0d7fc0e458550a1dac82f8ee5f)

Signed-off-by: Petr Mladek <pmladek@suse.cz>

Fixes crashing while loading manipulated pptx files.

Signed-off-by: Petr Mladek <pmladek@suse.cz>

Fix windows build where min and max are macros

Change-Id: I265e0fddc12a40cff0b6a6172bfca96eda0842ec
Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#36138: sd: fix undo in text objects

Change-Id: I44e9b5154939f3b150d3115085204361248e9b6d
Signed-off-by: Petr Mladek <pmladek@suse.cz>

Fix windows build where min and max are macros

Change-Id: I59186db4003564b0b9c202456b7fdf91eaa67751
Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#51239 refresh row lazily (when data is requested)

This avoids fetching data that will not be requested when the "cursor"
is only moved and no data requested. That is typically what
RowSetCache does when its own cursor is moved within its window.

This basically makes the whole {next,previous,absolute,...}_checked
story obsolete, by basically always being as fast as the
i_bFetchRow==false case, but in a safer way.

Change-Id: I89eaf277069736b3077bde8b45325929db290f2d

Signed-off-by: Petr Mladek <pmladek@suse.cz>

i#102625 avoid fetching same row twice in different queries

We do a "SELECT * FROM table" just to fetch the primary key columns;
so reuse the same XResultSet to fetch all columns.
Else, we immediately issue a "SELECT * FROM table WHERE
primary_key=current_value" to read the other columns, which is
wasteful and particularly silly.

Commit 1ae17f5b03cc14844fb600ca3573a96deb37ab3b already tried
to do that, but was essentially reverted piecewise because
it caused fdo#47520, fdo#48345, fdo#50372.

Commit c08067d6da94743d53217cbc26cffae00a22dc3a thought it did that,
but actually reverted commit 1ae17f5b03cc14844fb600ca3573a96deb37ab3b.

This implementation fetches the whole current row and caches it in memory;
only one row is cached: when the current row changes, the cache contains
the new current row.

This could be problematic (wrt to memory consumption) if the current
row is big (e.g. with BLOBs) and nobody is interested in the data
anyway (as would often be the case with BLOBs). Note that because of
our "SELECT *", the driver most probably has it in memory already
anyway, so we don't make the situation that much worse.

This could be incrementally improved with a heuristic of not
preemptively caching binary data (and also not LONGVARCHAR / TEXT /
MEMO / ...); a getFOO on these columns would issue a specific "SELECT
column FROM table WHERE primary_key=current_value" each time.

The *real* complete fix to all these issues would be to not do "SELECT
*" at all. Use "SELECT pkey_col1, pkey_col2, ..." when we are only
interested in the key columns. As to data, somehow figure out which
columns were ar interested in and "SELECT" only these (and maybe only
those with "small datatype"?). Interesting columns could be determined
by our caller (creator) as an argument to our constructor, or some
heuristic (no binary data, no "big" unbound data).
Also be extra smart and use *(m_aKeyIter) when getFOO is called
on a column included in it (and don't include it in any subsequent
SELECT).

However, there are several pitfalls.

One is buggy drivers that give use column names of columns that we
cannot fetch :-| Using "SELECT *" works around that because the driver
there *obviously* gives us only fetchable columns in the result.

Another one is the very restrictive nature of some database access
technologies. Take for example ODBC:

 - Data can be fetched only *once* (with the SQLGetData interface;
   bound columns offer a way around that, but that's viable only for
   constant-length data, not variable-length data).

   This could be addressed by an intelligent & lazy cache.

 - Data must be fetched in increasing order of column number
   (again, this is about SQLGetData).

   This is a harder issue. The current solution has the nice advantage
   of completely isolating the rest of LibO from these restrictions.

   I don't currently see how to cleanly avoid (potentially
   unnecessarily) caching column 4 if we are asked for column 3 then
   column 5, just in case we are asked for column 4 later on, unless
   we issue a specific "SELECT column4" later. But the latter would be
   quite expensive in terms of app-to-database roudtripe times :-( and
   thus creates another performance issue.

Change-Id: I999b3f8f0b8a215acb390ffefc839235346e8353

Squashed commits:

 Change-Id: Ie28f6a55cd8715a7180f5d88fe23c5b310440744
 Change-Id: Ia8d12d02829087309e248506a7d3b0f94b5a425e

Signed-off-by: Petr Mladek <pmladek@suse.cz>

OKeySet: tryRefetch and refreshRow share most of their code

Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#52610: Fixed several docx export bugs

 + hyperlink starting before a field and ending in its results isn't
   valid.
 + when footnotes have hyperlinks we need to add the proper XML
   namespace and output the relations to a different file.

Change-Id: I1c6cb8acba21c35e10bb0497eeaf96bbd73730d1
(cherry picked from commit 0e4846ec20c1aefc554037e8f08255019c08da47)

Signed-off-by: Miklos Vajna <vmiklos@suse.cz>
Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#46020: fixed missing footnote docx/doc/rtf export

Don't skip a next position when parsing a paragraph into runs to export.

Change-Id: If7e887ea84ad24256b37aa96187bc30ea154632b
Signed-off-by: Miklos Vajna <vmiklos@suse.cz>
Signed-off-by: Petr Mladek <pmladek@suse.cz>

fdo#38056: fix XMLTextImportPropertyMapper regression:

The invalidation introduced in 7f9928bfa561ccb6ed4e2baacc7d6960bc1ce231
is unfortunately in the wrong place; the pAllFoo actually point at
vector elements, and so they must not be dereferenced after new elements
are added to the vector, so move the invalidation up a bit.

Change-Id: I244d1d5cfd40469ff6d2c081e057355460d34cf4
(cherry picked from commit 9e74e9f3c0931f34fb5421056851a06b83501ce9)

Signed-off-by: Ivan Timofeev <timofeev.i.s@gmail.com>

fdo#51034 fix docx import of HYPERLINK field, l param

Change-Id: Ic6896f7c795d7d303bf7589a74afb937d4a8e8b5
(cherry picked from commit 68cb050f0fcd37a6b1e78c147402a521d24c25c1)

fdo#49892 import RTF_SHPB{X,Y}PAGE

(cherry picked from commit 2111c6e6dc18f3ffd11de176791ecc04bab41afb)

Conflicts:

writerfilter/source/rtftok/rtfdocumentimpl.cxx

Change-Id: I8fe2b28f44ee61fc0f1438de3681fd86b16d29ae

some tweaks in RTF filter

Signed-off-by: Petr Mladek <pmladek@suse.cz>

bump product version to 3.5.7.0, build id to 0

Change-Id: I04f6411143d5921ccdee0609c612d4c0a4b82f11

fdo#45724 improve WW8 textbox export

If there is no default value, but there is a current one, export that
instead.

Change-Id: I01af5cfa641b061b0aeb959e579bca66340781e9

fdo#45724 fix WW8 export of textbox default background color

regression from 5609c512cd4d2899300b432d88d93cff05a05c87

Change-Id: I5f3ce73943b2805b7de1ae41699e605af9086b15

configure: remove check for empty $COMPATH

It would be extremely suprising if we could get to line 11075 in the
configure script if there isn't actually a C++ compiler available, and
the test does fail on Fedora 17 without ccache installed because $CC is
/bin/gcc.

Change-Id: I727ba171a3d647e2ba974d196794943b6450ce6e
Signed-off-by: Petr Mladek <pmladek@suse.cz>

xpdf: fix some warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

redland: fix some warnings in raptor

Signed-off-by: Petr Mladek <pmladek@suse.cz>

neon: fix some warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

mythes: fix some warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

hunspell: fix warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

basic, cui, extensions, filter, vcl: fix some warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

sd, sfx2, writerfilter, xmloff: fix some warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

sw: fix some warnings

Signed-off-by: Petr Mladek <pmladek@suse.cz>

sot: prevent some null pointer crashes

rhbz#842552 always create text content

The following line in oox::drawingml::table::TableCell::pushToXCell
suggests that it is assumed that the cell always contains text:

Reference< text::XText > xText( rxCell, UNO_QUERY_THROW );

So TableCell should always hold a valid TextBody too.

Change-Id: Ic2db7b535c98dd5f2b18941846709a781df4585c

fdo#48033 fix RTF import of pictures inside table cells

Change-Id: I2d9ef3b346d4b10b99b67d0934d63d59f6119f97

warning: statement has no effect

Change-Id: I06a26ebd1703e1d753e35daeed610671fc451a8f
(cherry picked from commit 114ccd28e09f0fd13c8ff4e61e82e3f13198febf)

Signed-off-by: Eike Rathke <erack@redhat.com>

added back a missing else

c75e2ed27d5da71891ed7a148ff9603c58eaa58e (merge of OOo filter rework)
omitted an else of a condition.

Change-Id: I6f342a48938aaf4090fc738afc6cf551a083b4fc
(cherry picked from commit b06d679eb3d8190a470f7640740571f388106605)

Signed-off-by: Michael Stahl <mstahl@redhat.com>

Lotus WK: fix warnings

StarCalc 1.0: fix memory leaks

QuattroPro warnings

hwpfilter: improve text import and error handling

mhu24: add comphelper::newArray_{null,ex} helpers

Signed-off-by: Petr Mladek <pmladek@suse.cz>

better page descriptor handling in RTF

Signed-off-by: Petr Mladek <pmladek@suse.cz>

do not crash when opening rtf file with unclosed field group

It still fails to load the file (General input/output error), because
RTFDocumentImpl::resolve() throws io::UnexpectedEOFException, but that
is a bit better than a crash .-)

Example of such a file can be found in fdo#49666 or rhbz#819304 .

Change-Id: I I1e575bf8e3194afe063297a4daf3e98eaaa510cc
Signed-off-by: Petr Mladek <pmladek@suse.cz>

discard broken embedded object previews

Change-Id: Idbce67d16ed3e72395e130b1c4aa1d6977acd74b

Signed-off-by: Petr Mladek <pmladek@suse.cz>

Uploading libvisio 0.0.19, fixes fdo#50990

Change-Id: Iafd39d5aa3f3922d84518a3d808aac25dbae0e06
Signed-off-by: Thorsten Behrens <tbehrens@suse.com>
Conflicts:
libvisio/makefile.mk

fdo#47368 - fix one potential re-enterancy hazard around even emission

Change-Id: I8be50fb82d1333731dae825b1785e981fe69adbd
Signed-off-by: Norbert Thiebaud <nthiebaud@gmail.com>

fdo#39812: Writer: fix collapsing merged table border painting:

Create a table with a merged cell like in the screenshot in the
bug, with a SAL_DEBUG in SwTabFrmPainter::PaintLines the following
lines are painted:

debug: paint start
1 debug: start: 2749,1488 end: 12387,1488
2 debug: start: 2749,1945 end: 7567,1945
3 debug: start: 7567,1945 end: 12387,1945
4 debug: start: 2749,2015 end: 12387,2015
5 debug: start: 2749,2542 end: 7567,2542
6 debug: start: 7567,2542 end: 12387,2542
7 debug: start: 2749,1488 end: 2749,1945
8 debug: start: 2749,1945 end: 2749,2015
9 debug: start: 2749,2015 end: 2749,2542
A debug: start: 7567,1945 end: 7567,2542
B debug: start: 12387,1488 end: 12387,1945
C debug: start: 12387,1945 end: 12387,2015
D debug: start: 12387,2015 end: 12387,2542
debug: paint end

*11111*11111*
7           B
7           B
*22222*33333*
8     A     C
*44444*44444*
9     A     D
9     A     D
*55555*66666*

The problem is obviously that the Y coordinates of the lines 2, 3
and 4 differ; they should be on the same Y position.
The problem here is that logically horizontal lines must be painted
not centered but "below" the line, and It turns out that
SwTabFrmPainter::Insert cannot correct the positions properly to
do that, because it only looks at borders in a single cell.

When using the UI to set the borders, we get (for innner table borders)
only a bottom border in the cells, but no top borders, so the
top position of the logically vertical borders needs to be corrected
with the width of the bottom border of the cell _above_; a symmetric
correction of the bottom position to the top border of the cell below
is also necessary.

Fortunately if we just leave the positons alone in Insert then
TabFrmPainter will eliminate duplicate lines with equal positions
and so it's only necessary to correct the positions when actually
painting the line in wTabFrmPainter::PaintLines,
where we have the neighboring lines available.

(cherry picked from commit 02e80d2e431a57ad775a674eb3cfcd6cec53e09f)

Change-Id: Ia8519f6673db0f3a1ecaa68038896cac39609129
Signed-off-by: Miklos Vajna <vmiklos@suse.cz>

fdo#44534 invalidate bound rect after drag&drop

Change-Id: Ie3319870ae26e9e2cac32641a33be62e4e020c02
(cherry picked from commit b480f91968a3df16a655585fba494425aea725dc)

Signed-off-by: Caolán McNamara <caolanm@redhat.com>

Resolves: fdo#50975 rsvg_handle_new_from_data calls rsvg_handle_close

rsvg_handle_new_from_data calls rsvg_handle_fill_with_data which itself calls
rsvg_handle_close on success, so we don't have to.

And older versions of librsvg (e.g. 2.16.1) don't protect against
rsvg_handle_close getting called twice, although new versions do,
so we crash on e.g. RHEL-5

Change-Id: I451075e50c9c2c7b07289356d7a92eb55db82f9f
(cherry picked from commit 682cb83ed59233decbeeaff68c4d871442b58bb1)

Signed-off-by: Michael Stahl <mstahl@redhat.com>

fdo#50800 populate composedCommandNames ASAP, not at call of getFromClause

In particular com/sun/star/wizards/ui/FilterComponent calls getSelectClause before calling getFromClause, and then all hell breaks loose: composedCommandNames is empty, thus cannot find the proper Alias column name, thus the column names in the select list were not properly escaped, ...
We have just made getFromClause quadratic instead of linear, but:
1) I do not think this would be a problem (small datastructures)
2) If it is, rather use a hashmap or something like that, wich will also make getSelectClause faster

Also make the fallback case of "unknown table" more robust: escape the table name (if any) and column name!

Change-Id: I474adc51fc6378d836bd5865d9eb9505983dcbc5
Sgined-off-by: Miklos Vajna <vmiklos@suse.cz>

fdo#38056: ODF import: fix page style attributes:

PageMasterImportPropMapper: in the case of a single "fo:border"
and "style:border-line-width" attribute, it is possible that the border
is imported wrongly, like this:

1. pAllBorderProperty is set from the imported value, with name "TopBorder"
2. individual pNewBorder[i] are created as copies from pAllBorderProperty,
   one of which also with name "TopBorder"
3. pNewBorder[i] is updated with widths from pBorderWidths[i]
4. the individual pNewBorder[i] are added to the property vector
5. the property vector is sorted by property name
6. the properites are applied in order; if the pNewBorder[TOP]
   happens to precede the pAllBorderProperty (which is indeterminate
   as they both have name "TopBorder"), then the pAllBorderProperty
   will overwrite the border widths computed in step 3.

Thus, nerf the various pAllFoo properties so they do not override
the individual Foo properties later on.

(cherry picked from commit 7f9928bfa561ccb6ed4e2baacc7d6960bc1ce231)

Change-Id: I87755f1184d59da2aa72ac053e6f77d7295d6958
Signed-off-by: Miklos Vajna <vmiklos@suse.cz>

fdo#48335 two RTF export fixes

1) don't try to end not started runs

(cherry picked from commit 5d505e5b1edee7f709e4baff70a971cb3fe851c2)

Conflicts:

sw/source/filter/ww8/rtfattributeoutput.cxx

2) avoid fake page breaks on page style changes

The problem was that a page break has been always exported when the page
style changed -- but in case the page style changes just because of
"first page"-like styles, we don't need that.

(cherry picked from commit a03895986308206cc13a6f5ae25138d4b4ad5d43)

Change-Id: I05940e7ff649051ecae4a72ae73617a47ffca885

Signed-off-by: Luboš Luňák <l.lunak@suse.cz>

fdo#46966 dmapper: fix headery/footery default value

The docx spec doesn't say what is the default value, the rtf spec says
it's 720, not 1440.

Change-Id: Icb331591d4f2f96a7786f808d99af5974e645f8e

Signed-off-by: Luboš Luňák <l.lunak@suse.cz>

fdo#52066 fix RTF import of rectangle shape without text in it

We used to always add a paragraph on shapes, which breaks import of
abused rectangle shapes with minimal height, used as lines.

Change-Id: Ice240bad68bc030e7889c46f72c45646307f17e5

Signed-off-by: Luboš Luňák <l.lunak@suse.cz>

fix build

Change-Id: I0802bd56a9fe7ea2674e6627c2239850967f3646
Sigend-off-by: Andras Timar <atimar@suse.com>

fdo#50801 fix cross-reference text when Caption order is Numbering first

Change-Id: I7306f99c18d0f9cfb3b0ce147ecc200662d23b3d
Sigend-off-by: Andras Timar <atimar@suse.com>

fdo#52143 use Unicode functions for QuickStarter tooltip

Change-Id: Ic9c7ed84925ea5b4ce324a98a1089102dcda24b8
Signed-off-by: Fridrich Štrba <fridrich.strba@bluewin.ch>

fdo#47947 Data Form dialog layout fix

Change-Id: Ifa1689ff9be7afa14f5694825c506f7c8c3eff43
(cherry picked from commit c61e3c941d46fe1ae6066f31e01786100e39da56)

Signed-off-by: Caolán McNamara <caolanm@redhat.com>

prevent double delete in calc's html parser, fdo#52107

Change-Id: I0a203a39697d3b12304e4b9dc99714cd7b2a008c
(cherry picked from commit 0d21d9ddbe0561f45419ae4fc376c312565b413b)

Signed-off-by: Caolán McNamara <caolanm@redhat.com>

fdo#47071: Undocked toolbars do not show all icons in special ratio

 restore the old behaviour (pre-1703501a100cd6d52578baeb4e8097218b285ffb)
 for floating toolbars.

Change-Id: I8435bd94a44744c5af7a3abdbeb99e8f7cf0af24
Signed-off-by: Jan Holesovsky <kendy@suse.cz>