482499 Commits

Author SHA1 Message Date
Chenxiong Qi
be7f3df701 tdf#97362 Convert unotools tempfile tests to Python
Signed-off-by: Chenxiong Qi <qcxhome@gmail.com>
Change-Id: I43d0881bbd2a99e018e027e166dcb7b0bffa5ff5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144395
Tested-by: Jenkins
Reviewed-by: Hossein <hossein@libreoffice.org>
2023-01-17 06:20:58 +00:00
Michael Weghorn
bcce220531 tdf#153009 android: Include more doc layout related XML files
instdir/share/config/soffice.cfg/simpress/objectlist.xml
contains default positions for elements that are used
in new Impress presentations, but that file wasn't
included in the Android app/APK previously, so
couldn't be read from in `SdDrawDocument::InitObjectVector`.
As a result, the title and text were misplaced when creating a
new presentation in Android Viewer.

Include all of the related Impress XML files into the app to
make this work as expected.

(At least a failed attempt to read from `layoutlist.xml`
was also observed while debugging the issue mentioned
above.)

Change-Id: Iba1ac68c0b983d3e465515f07b3fc67162a0ee3f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145621
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2023-01-17 06:17:42 +00:00
Michael Weghorn
48916eb11b tdf#153009 Use recursive mutex in CDocumentBuilder
This was changed from `osl::Mutex` to `std::mutex`
in

    commit e66fdb597b30fc701bb068824d0ae4d89fecd55f
    Date:   Sat Jul 31 20:12:30 2021 +0200

        osl::Mutex->std::mutex in CDocumentBuilder

, but `DOM::CDocumentBuilder::parseURI` locks the mutex and
then calls `DOM::CDocumentBuilder::parse` for the

        // if we failed to parse the URI as a simple file, lets try via a ucb stream.
        // For Android file:///assets/ URLs which must go via the osl/ file API.

case, which would result in a deadlock because
`DOM::CDocumentBuilder::parse` wants to lock the mutex
again.

This happens once that code path is executed
on Android Viewer when creating a new Impress presentation
with Change-Id Iba1ac68c0b983d3e465515f07b3fc67162a0ee3f
("tdf#153009 android: Include more doc layout related XML files")
in place.

Fix the deadlock by using a "std::recursive_mutex` instead.

Part of the backtrace:

    DOM::CDocumentBuilder::parse(const com::sun::uno::Reference<…> &) documentbuilder.cxx:331
    DOM::CDocumentBuilder::parseURI(const rtl::OUString &) documentbuilder.cxx:387
    non-virtual thunk to DOM::CDocumentBuilder::parseURI(rtl::OUString const&) documentbuilder.cxx:0
    SdDrawDocument::InitLayoutVector() drawdoc.cxx:1130
    SdDrawDocument::SdDrawDocument(DocumentType, SfxObjectShell *) drawdoc.cxx:147
    sd::DrawDocShell::Construct(bool) docshell.cxx:94
    sd::DrawDocShell::DrawDocShell(SfxModelFlags, bool, DocumentType) docshell.cxx:140
    sd_PresentationDocument_get_implementation::$_1::operator()(SfxModelFlags) const unodoc.cxx:65
    std::__ndk1::__invoke<…>(sd_PresentationDocument_get_implementation::$_1 &, SfxModelFlags &&) type_traits:4353
    std::__ndk1::__invoke_void_return_wrapper::__call<…>(sd_PresentationDocument_get_implementation::$_1 &, SfxModelFlags &&) __functional_base:318
    std::__ndk1::__function::__alloc_func::operator()(SfxModelFlags &&) functional:1527
    std::__ndk1::__function::__func::operator()(SfxModelFlags &&) functional:1651
    std::__ndk1::__function::__value_func::operator()(SfxModelFlags &&) const functional:1799
    std::__ndk1::function::operator()(SfxModelFlags) const functional:2347
    sfx2::createSfxModelInstance(const com::sun::uno::Sequence<…> &, std::__ndk1::function<…>) sfxmodelfactory.cxx:82
    sd_PresentationDocument_get_implementation(com::sun::uno::XComponentContext *, const com::sun::uno::Sequence<…> &) unodoc.cxx:61
    std::__ndk1::__invoke<…>(com::sun::uno::XInterface *(*&)(com::sun::uno::XComponentContext *, const com::sun::uno::Sequence<…> &), com::sun::uno::XComponentContext *&&, const com::sun::uno::Sequence<…> &) type_traits:4353
    std::__ndk1::__invoke_void_return_wrapper::__call<…>(com::sun::uno::XInterface *(*&)(com::sun::uno::XComponentContext *, const com::sun::uno::Sequence<…> &), com::sun::uno::XComponentContext *&&, const com::sun::uno::Sequence<…> &) __functional_base:318
    std::__ndk1::__function::__alloc_func::operator()(com::sun::uno::XComponentContext *&&, const com::sun::uno::Sequence<…> &) functional:1527
    std::__ndk1::__function::__func::operator()(com::sun::uno::XComponentContext *&&, const com::sun::uno::Sequence<…> &) functional:1651
    std::__ndk1::__function::__value_func::operator()(com::sun::uno::XComponentContext *&&, const com::sun::uno::Sequence<…> &) const functional:1799
    std::__ndk1::function::operator()(com::sun::uno::XComponentContext *, const com::sun::uno::Sequence<…> &) const functional:2347
    cppuhelper::ServiceManager::Data::Implementation::doCreateInstance(const com::sun::uno::Reference<…> &) servicemanager.cxx:706
    cppuhelper::ServiceManager::Data::Implementation::createInstance(const com::sun::uno::Reference<…> &, bool) servicemanager.cxx:675
    cppuhelper::ServiceManager::createInstanceWithContext(const rtl::OUString &, const com::sun::uno::Reference<…> &) servicemanager.cxx:1006
    non-virtual thunk to cppuhelper::ServiceManager::createInstanceWithContext(rtl::OUString const&, com::sun::uno::Reference<com::sun::uno::XComponentContext> const&) servicemanager.cxx:0
    SfxFrameLoader_Impl::load(const com::sun::uno::Sequence<…> &, const com::sun::uno::Reference<…> &) frmload.cxx:707
    non-virtual thunk to (anonymous namespace)::SfxFrameLoader_Impl::load(com::sun::uno::Sequence<com::sun:🫘:PropertyValue> const&, com::sun::uno::Reference<com::sun::frame::XFrame> const&) frmload.cxx:0
    framework::LoadEnv::impl_loadContent() loadenv.cxx:1167
    framework::LoadEnv::start() loadenv.cxx:403
    framework::LoadEnv::startLoading(const rtl::OUString &, const com::sun::uno::Sequence<…> &, const com::sun::uno::Reference<…> &, const rtl::OUString &, long, LoadEnvFeatures) loadenv.cxx:308
    framework::LoadEnv::loadComponentFromURL(const com::sun::uno::Reference<…> &, const com::sun::uno::Reference<…> &, const rtl::OUString &, const rtl::OUString &, long, const com::sun::uno::Sequence<…> &) loadenv.cxx:168
    framework::Desktop::loadComponentFromURL(const rtl::OUString &, const rtl::OUString &, long, const com::sun::uno::Sequence<…> &) desktop.cxx:594
    non-virtual thunk to framework::Desktop::loadComponentFromURL(rtl::OUString const&, rtl::OUString const&, long, com::sun::uno::Sequence<com::sun:🫘:PropertyValue> const&) desktop.cxx:0
    lo_documentLoadWithOptions(_LibreOfficeKit *, const char *, const char *) init.cxx:2666
    lo_documentLoad(_LibreOfficeKit *, const char *) init.cxx:2532
    Java_org_libreoffice_kit_Office_documentLoadNative(JNIEnv *, jobject, jstring) lokandroid.cxx:134
    [...]

Change-Id: I63bc7b6f682dab2e94f4e932429c513c5129428e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145620
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2023-01-17 06:17:36 +00:00
Michael Weghorn
3cecdaeab1 tdf#153009 android: Set BRAND_SHARE_SUBDIR for more default structure
Set `BRAND_SHARE_SUBDIR=share` in the `fundamentalrc`
shipped with Android Viewer, as is the case for the
desktop variant as well. This implies that
the .ui files will be expected underneath the "config/share"
directory in the app's assets from now on instead of just
"config", since `BRAND_SHARE_SUBDIR` will be expanded accordingly
in the paths.
Therefore, put the .ui files there. Merge the block responsible
for copying UI files into into the existing block already copying to
that dir in the `copyTask` gradle task.

There seems to be no particular reason for omitting
the "share" subdirectory just for the .ui files in the Android
assets.

This is also in preparation of fixing tdf#153009:
Not having `BRAND_SHARE_SUBDIR` set on Android resulted
in

    "vnd.sun.star.expand:$BRAND_BASE_DIR/$BRAND_SHARE_SUBDIR/config/soffice.cfg/simpress/objectlist.xml"

previously getting expanded to

    file:///assets//config/soffice.cfg/simpress/objectlist.xml

in `SdDrawDocument::InitObjectVector`, and that would cause
failure to read object property settings (e.g. for the placeholder
text in a new Impress presentation) from that file, even
when `objectlist.xml` was put into the app's assets
(s. Change-Id Iba1ac68c0b983d3e465515f07b3fc67162a0ee3f,
"tdf#153009 android: Include more doc layout related XML files")
at "config/soffice.cfg/simpress/objectlist.xml" due to the
double-slash after "assets" not being treated the same way
as a single slash at that place.

Change-Id: Ia9d86e1f59daf8a41951cefb3a1e9f82f0e9f184
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145619
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2023-01-17 06:17:29 +00:00
Caolán McNamara
f1830bff71 ofz#55090 fuzzers need docmodel now
Change-Id: I89ffdca9a6d497cd57788e125cbfc7bb1cfde672
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145643
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 20:08:25 +00:00
Noel Grandin
45a2d738c4 XUnoTunnel->dynamic_cast in OContentHelper
Change-Id: I3788af96cc8664d8afda2bd5eeaf85e9cd0cf374
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145618
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 19:37:34 +00:00
Miklos Vajna
654f972a97 sw lok: make sure author name change doesn't mark the doc as modified
This was added in commit d4760b5d4b49fef1c832357059c7e184b3cc0bc4 (sw:
lok: use redline author for saved author fields, 2023-01-09), but it
makes online.git unit-wopi-save-on-exit fail.

Try to get up to date author names + passing tests at the same time by
still updating the fields but not marking the doc as modified right
after load.

Change-Id: Ic07fcc4464058e778f07d5a88b1d8479dbd8c144
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145623
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2023-01-16 18:47:20 +00:00
Noel Grandin
0045c49cf4 XUnoTunnel->dynamic_cast in ORowSetBase
Change-Id: I685d15982f56b336862da57ecfcbd9ac2d14838b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145616
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 18:41:37 +00:00
Noel Grandin
805a634ccc XUnoTunnel->dynamic_cast in OQueryDescriptor
Change-Id: Ide46f159674ce84f6ecd8b9b0045a66a3b4ddd98
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145615
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 18:41:31 +00:00
Noel Grandin
27d76f145a tdf#153039 submitting XML from XForms document
regression from
    commit 5fd2f0b93bd7d38a277823bfc251c71da7a6f490
    Author: Noel Grandin <noel.grandin@collabora.co.uk>
    Date:   Thu Jan 12 09:19:03 2023 +0200
    use concrete class in Submission

Change-Id: I99bc069b5243e994da71287960e29be6ab3338ac
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145567
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 15:33:24 +00:00
Andrea Gelmini
0857ba551b Fix typo
Change-Id: I01c9a76fd9abca9611748e4e9f575eeb5a45f0dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145570
Tested-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2023-01-16 13:34:19 +00:00
Miklos Vajna
d5c11b12db CppunitTest_sw_layoutwriter2: fix testTdf152952_compat with en-only hyph dict
The bugdoc had German text, but the test was only checking for an English
hyphenation dictionary, so adapt the bugdoc.

Also, now that the text is English, it's hyphenated a little bit
differently, so adapt the asserts. The point is that it's still
hyphenated.

Change-Id: Iaaf45843ad37ca7e7ee87c60ec59f41cc10efb6e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145565
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2023-01-16 13:33:26 +00:00
Ilmari Lauhakangas
531cfd9ebd readme.xrm: update for macOS
Change-Id: I93793b689271dbc15b1670979061e0c34128289d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145563
Tested-by: Jenkins
Reviewed-by: Ilmari Lauhakangas <ilmari.lauhakangas@libreoffice.org>
2023-01-16 12:49:55 +00:00
Mike Kaganski
15fe7346ad Simplify FastAttributeList
Change-Id: Id89edb25e35527e8603c32e44fb2940721aeda58
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145562
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2023-01-16 11:40:18 +00:00
Miklos Vajna
f41d424915 sw: disable AutoUpdate for many new recent field/book/refmark UNO commands
As noted in
<https://gerrit.libreoffice.org/c/core/+/145470/1#message-afe4d72912e02c6bacc63392305c3f910a354c6a>,
these only make sense for commands which have a GetState()
implementation (AutoUpdate will call GetState() automatically for you
after an exec), and none of these commands have an explicit GetState(),
so AutoUpdate=TRUE doesn't make sense for them.

Change-Id: I9899c297a7b0f8c447c66b50a228bc7e1896e146
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145557
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2023-01-16 11:34:30 +00:00
Caolán McNamara
1520708cd9 The third "depth argument is unused
as far as I can determine. We do query the 2nd arg in "gen"
when there is an opengl slide transition.

Change-Id: I180c91fa193ee6e7e3d5a415e4278aded9fbbba1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145544
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 11:32:13 +00:00
Noel Grandin
0dbc1bd6b7 XUnoTunnel->dynamic_cast in file::OResultSet
Change-Id: Icfb3760b05e80925833f2ecaa294a81c85b6c672
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145561
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 11:30:45 +00:00
Rafael Lima
d466df5bc2 tdf#106731 Warn user that a module/dialog is read-only in Basic IDE
As discussed in the ticket the Basic IDE does not make it clear that the currently opened module is read-only, which can cause confusion to the user, since they may be pressing keys and seeing no result.

With this patch the IDE clarifies that the module/dialog is read-only in two ways:
1) In the status bar the string "(Read-only)" is added after the module/dialog name
2) When a module/dialog is opened an infobar is shown warning that it is read-only.
3) In a module, if the user closed the infobar and tries to edit the code, the infobar is shown again.

Change-Id: I15197c4316bb2ed19e54be0c33f4fb61b5bbd862
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145307
Tested-by: Jenkins
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
2023-01-16 11:28:35 +00:00
Caolán McNamara
465ec8b275 getFormat() is always DeviceFormat::DEFAULT
Change-Id: I93139a6c2ce28f28f615b7399e37fcb06fac0997
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145543
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 11:13:10 +00:00
Caolán McNamara
17dc855c6f cid#1519051 silence Unchecked dynamic_cast
and

cid#1519050 Unchecked dynamic_cast
cid#1519049 Unchecked dynamic_cast
cid#1519047 Unchecked dynamic_cast

Change-Id: I3feb25cda71c459c549f0e64558935d1030ed309
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145541
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 11:13:02 +00:00
Caolán McNamara
be3b99f12f cid#1519048 silence Unchecked return value
Change-Id: Ia4763684d84b671e4fb2523b6d5bc222fb5e71e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145540
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 11:12:53 +00:00
Caolán McNamara
b03cc1dd65 cid#1474008 silence Unchecked return value from library
Change-Id: I23228ae5d61654e2c2d401453bc741a2a6074278
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145539
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 10:33:02 +00:00
Caolán McNamara
093a7f422c cid#1519046 Uninitialized scalar variable
bogus, but earlier return makes things easier anyway

Change-Id: Id6c0a72b0ec9bbae061cc1f883b383fbef88e771
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145538
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 10:32:55 +00:00
Caolán McNamara
8003a7b117 cid#1412875 Uninitialized pointer field
Change-Id: I551a18eddb5a2f436ecb947411faef09ac74d016
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145537
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2023-01-16 10:32:47 +00:00
Mike Kaganski
e5b86ec34d Use o3tl::toInt32 in oox/source/drawingml/textfield.cxx
rtl_ustr_toInt32 was incorrectly used, relying on null-terminated
content of string views.

Change-Id: Ia3e4bb04a029149e5945274f592252704c34f3eb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145549
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
Tested-by: Jenkins
2023-01-16 10:30:59 +00:00
Noel Grandin
f025dcc44c XUnoTunnel->dynamic_cast in connectivityy
Change-Id: Ibe5e87a1ef1bdc11451e8fe240d017f1ef60dfe6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145556
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 10:19:16 +00:00
Miklos Vajna
0250d6c643 sw, UpdateFieldContent: address a minor performance nit
See
<https://gerrit.libreoffice.org/c/core/+/145459/1#message-fa0ede3525e35014c1f6c99455a96a69bc2a3601>,
it's enough to start grouping undo/redo after we know that there is a
relevant refmark under the cursor.

Change-Id: I53e7e371768480d6b7e968c90bf8c5d9901013b9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145555
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2023-01-16 09:28:06 +00:00
Miklos Vajna
2cf59dee96 sw, UpdateBookmark: address some minor performance nits
See
<https://gerrit.libreoffice.org/c/core/+/145413/2#message-6fda3c30e7b8cdab1137e2397a5f3f00b97d5acc>:

- can start undo later, after making sure that we have a bookmark under
  the cursor

- can look up the bookmark text later, after we managed to insert the
  marks (which may fail in some protected cursor position)

Change-Id: I8ee183974fcc503a5c9a3964e86168888db13586
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145554
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2023-01-16 09:10:41 +00:00
Stephan Bergmann
5877c6b118 external/skia: Missing include
> In file included from workdir/UnpackedTarball/skia/third_party/vulkanmemoryallocator/GrVulkanMemoryAllocator.cpp:27:
> In file included from workdir/UnpackedTarball/skia/third_party/vulkanmemoryallocator/GrVulkanMemoryAllocator.h:35:
> workdir/UnpackedTarball/skia/third_party/vulkanmemoryallocator/include/vk_mem_alloc.h:2817:9: error: use of undeclared identifier 'snprintf'
>         snprintf(outStr, strLen, "%u", static_cast<unsigned int>(num));
>         ^

etc.

Change-Id: I4f13c8a137f23ef62d67c3b4dfa8f1c0ddfc5ec9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145552
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2023-01-16 08:48:45 +00:00
Noel Grandin
d21260708b XUnoTunnel->dynamic_cast in VCLXGraphics
Change-Id: Ia401542ec8249809ad16dadd79c51aa025044b42
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145534
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 08:41:00 +00:00
Noel Grandin
8081533f7c XUnoTunnel->dynamic_cast in VCLXBitmap
Change-Id: I2cf42ac720184bc2b8b89ced5564dbb95e7dc1ba
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145535
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 08:40:47 +00:00
Andreas Heinisch
b9411e5875 tdf#147122 - Return cell object when a simple selection is merged
Change-Id: I4ddd3b3a804f8300a5ec15526f4c9c77aaf45fc5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145378
Tested-by: Jenkins
Reviewed-by: Andreas Heinisch <andreas.heinisch@yahoo.de>
2023-01-16 08:28:49 +00:00
Miklos Vajna
1d6593dd79 sw: add a new .uno:DeleteFields UNO command
This is similar to 40753de837b9776dd8b33e830be0cceef83f024a (sw: add a
new .uno:DeleteBookmarks UNO command, 2023-01-13), but that was about
deleting bookmarks matching a given prefix with their name, and this one
is about reference marks (fields in general), matching a certain type &
prefix with their name.

Change-Id: Iec953034cd0e6875f173712b0fb10bfddf16ed3f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145551
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
2023-01-16 08:05:45 +00:00
Radhey Parekh
4114218584 tdf#112417 Insert Sheet called when clicking anywhere on the footer
Change-Id: I7524ac11729d1c80254c84c27211de3efaa622c9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145102
Reviewed-by: Heiko Tietze <heiko.tietze@documentfoundation.org>
Tested-by: Jenkins
2023-01-16 07:59:16 +00:00
Miklos Vajna
77e0c4b5d2 sw: prefix members of BookmarkCollectionHelper, ...
... HeadersFootersEnumWrapper, RowsEnumWrapper and UnderLineMapper

See tdf#94879 for motivation.

Change-Id: Id4c63d3ebf83dc358747143dabe3b66edea94749
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145550
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
2023-01-16 07:56:18 +00:00
Taichi Haradaguchi
67c6e66c5b external/python3: fix line offsets in internal-zlib.patch.0
"@@ -11607,13 +11607,13 @@" -> "@@ -12333,13 +12333,13 @@"

Change-Id: I03542ad3171814f6869bccab4dde77673eaa90f7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/144982
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens <thorsten.behrens@allotropia.de>
2023-01-16 07:37:22 +00:00
Noel Grandin
0160109aae XUnoTunnel->dynamic_cast in GridColumn
And use the concrete type in DefaultGridColumnModel, which already makes
such an assumption

Change-Id: I4c91d46f1889ebe22b6c031a8aabe4e8957addec
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145531
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 06:42:12 +00:00
Noel Grandin
8a1108a0c7 XUnoTunnel->dynamic_cast in OToolBoxWindowItem
Change-Id: I65e535d9f9a81a3712b14b50cc4906b534533e86
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145536
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 06:42:00 +00:00
Noel Grandin
96cd662366 XUnoTunnel->dynamic_cast in VCLXPointer
Change-Id: I322115c214de5d476f84650eb53dae8452d5d1a7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145533
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 06:41:53 +00:00
Noel Grandin
e5092fdf24 XUnoTunnel->dynamic_cast in VCLXRegion
Change-Id: I75956a2c713d52c01dcfd47fc0b8506ba526523e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145532
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-16 06:41:46 +00:00
Noel Grandin
d04c197b43 XUnoTunnel->dynamic_cast in CNode
Change-Id: Ifcedcb6fc4e20da6351e4ccf035a3fe06d607961
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145507
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-15 18:36:44 +00:00
Winfried Donkers
bf432958c1 tdf#152774 Fix incorrect result with HLOOKUP and VLOOKUP.
Use case now complies with ODF 6.9.5 (HLOOKUP) and 6.9.12 (VLOOKUP).
Added use case to unit test.

Change-Id: I19df718b6446098f614136f462662c753a515036
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145514
Reviewed-by: Eike Rathke <erack@redhat.com>
Tested-by: Jenkins
2023-01-15 16:02:49 +00:00
Mike Kaganski
f2f008c52a Merge SvXMLAttributeList to comphelper::AttributeList
And simplify the latter, to always use "CDATA" type (as the former did).
"CDATA" was used in all cases but one, where an empty string was used.

Change-Id: I1b3bfae40e29628e4094d9a6e58a69a66865874c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145526
Tested-by: Jenkins
Reviewed-by: Mike Kaganski <mike.kaganski@collabora.com>
2023-01-15 12:32:29 +00:00
Julien Nabet
9d2355b674 tdf#153004: error when copying content to a table without autovalue primary key
In fact, hsql is well managed without extra code.

Change-Id: I93e2b02f3ec78e263cd0f9fab6e33b708dbd3f11
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145525
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Tested-by: Jenkins
2023-01-15 10:36:20 +00:00
Noel Grandin
4f0012b73c XUnoTunnel->dynamic_cast in UnoBinaryDataContainer
Change-Id: Ifdef69448558f4c5d6902188208d3eea1a080334
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145506
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-15 10:14:16 +00:00
Noel Grandin
e5e40c0bf5 XUnoTunnel->dynamic_cast in StyleMap
Change-Id: Ib4070fbff0c919cdc2be12f5a2bed1b1ac9e03c3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145505
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-15 10:14:05 +00:00
Noel Grandin
68c6133e4d XUnoTunnel->dynamic_cast in XMLMutableAttributeList
Change-Id: Ia36576da1f287c9b687042eba5d429366b3db0a3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145504
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2023-01-15 10:13:54 +00:00
Andrea Gelmini
92e6cd75a3 Typo in "code"
Not sure about this...

Change-Id: I8aebfdf48dc7dc2bdbce180fa2f1bd3305489a24
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145516
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2023-01-15 09:27:42 +00:00
Tomaž Vajngerl
28edd79792 vcl: add more device independent checks for bitmaps in svmtest
Adds writing of "contentchecksum" to MetafileXmlDump, which sums
the content of all the colors in an device independent way, by
reading the each pixel color from the bitmap and summing each
component always in RGBA order.

In addition add some others bitmap attributes not checked before,
like bitmap width, height and pixel format.

Change-Id: I3bb5f7f6342766df235af71a4682a5d3ce17ab44
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145500
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2023-01-15 08:07:41 +00:00
Justin Luth
5b92a7f211 tdf#117266 sc oox: macro button unit test
Change-Id: I50b8b5a2d29a5892490a7dbd4cc428786147d500
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/145519
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
2023-01-15 04:15:14 +00:00