Commit graph

476301 commits

Author SHA1 Message Date
Samuel Mehrbrodt
930ba0c1ec Fix comment
No need to mention gerrit URLs in code comments

Change-Id: I445f94211a3e660aa6e3482352f56ded7465e008
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135692
Tested-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
2022-06-13 10:32:56 +02:00
Miklos Vajna
cad76135ef sw: prefix members of WW8DupProperties, WW8SelBoxInfo, WW8_WrPc and ...
... WW8_WrtBookmarks

See tdf#94879 for motivation.

Change-Id: I4a3ede4c48220fb61ee8ccb4bbfb955a46cf56d0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135686
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
Tested-by: Jenkins
2022-06-13 10:22:05 +02:00
Samuel Mehrbrodt
2fded499df Add XFillBackgroundItem::dumpAsXml
Change-Id: I21aa5ded0c9c114199c9d503b48437ec1a6171eb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135691
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt <samuel.mehrbrodt@allotropia.de>
2022-06-13 10:18:49 +02:00
Adolfo Jayme Barrientos
3f817355a3 0.8 is too small on Windows; follow-up to 0ebbe1f00
Change-Id: Ie7b545acdb0648bccd39d452087c0b38c8c118e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135690
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2022-06-13 09:55:38 +02:00
Michael Weghorn
025477e244 vcl: Trigger IconView action on space key as well
Handle space key the same as the return key.
This e.g. allows to insert the selected element
into the formula this way in Math again after the
switch to IconView in

    commit d79c527c2a
    Date:   Wed Jun 1 11:20:12 2022 +0300

        Use IconView in SmElementsControl

gtk3 was already behaving that way without this
change in place.

Change-Id: I061b286723fedf023551afbb2184abc946b54b9f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135592
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
2022-06-13 09:55:23 +02:00
Adolfo Jayme Barrientos
f41b984255 TotD: Add a couple of missing words
The issue was flagged two years ago (!!) by Steen Rønnow.

Change-Id: I6d023bbe7c5392038f35b767acdf7f0ad0acd718
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135673
Tested-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2022-06-13 09:53:13 +02:00
Justin Luth
9a2e7bade3 tdf#145151 sw IsTableMode NumRule: unselected cells
...should not be affected when setting or deleting numbering.

This patch depends on follow-up patches to properly
detect if numbering is turned on or off in some cases.

This patch prevents numbering from "leaking" into the previous
cell if the cells were selected backwards.

Perhaps it would be better to fix the selection code itself
instead of handling all of these edge cases,
but doing that might have unintended consequences that
I wouldn't have any insight into. So this is safer.

Change-Id: I98e18d6056e93a4d89fdbe75b6237daca7832f41
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134885
Tested-by: Jenkins
Reviewed-by: Justin Luth <jluth@mail.com>
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2022-06-13 09:51:35 +02:00
Miklos Vajna
aa5824b2a6 tdf#149509 sw content controls: reject page break insertion
Similar to input fields, the intention is to keep the start and end of
(inline) content controls within the same text node, so just disable the
command in this context, as Word does.

Change-Id: Ib797ad164a3a36b4bbde4d686cde29adf6db96bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135599
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2022-06-13 08:09:51 +02:00
Miklos Vajna
c321498f91 sw content controls: reject typing inside checkbox or picture content controls
Content controls are editable by default (and not only editable, but
also capable of hosting rich text content), and Writer doesn't limit
the possibility to edit explicitly, either.

Certain content control types (checkbox and picture for now) limit the
hosted content though: checkbox overwrites the content on click and
picture is meant to host a single as-char anchored image. So far the
simple implementation Writer didn't enforce these limits, leading the
unexpected behavior when clicking on checkbox content controls (possibly
not only a checked/non-checked checkmark was toggled, but other content
was removed).

Fix the problem by making these content control types read-only: this is
what also Word does and this way you can't loose the content when you
can't enter it earlier.

We may want to also do this for dropdowns in the future, once combo
boxes will be supported.

Change-Id: I9d44206b3c719a64ec552f2fa0a076901094163e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135574
Tested-by: Jenkins
Reviewed-by: Miklos Vajna <vmiklos@collabora.com>
2022-06-13 08:07:02 +02:00
Caolán McNamara
37dde4ef13 crashtesting: assert seen on loading forum-mso-de-79405.docx
Change-Id: I91200a8e409eec76de4ead22d200bf45f02b88b0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135679
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-06-13 06:55:07 +02:00
Caolán McNamara
48de2ac373 crashtesting: assert seen on loading forum-mso-en-2633.pptx
probably since:

commit 855a56fea4
Date:   Fri May 13 08:12:17 2022 +0200

    tdf#148965 PPTX import: fix internal hyperlinks on shapes

Change-Id: I920cbbb5544cbe9d76f37b3d524c9ccebd426132
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135682
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-06-13 06:54:39 +02:00
Caolán McNamara
67a661800e crashtesting: fix crash seen on loading forum-mso-de-98993.xlsx
which may explain the backtraces seen with BitmapFilterStackBlur::filter
and BitmapBasicMorphologyFilter::filter in crashreporting

Change-Id: Ib55cde1603d354b8ca0e336a08fe2d73b548f73f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135680
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-06-13 06:54:11 +02:00
Caolán McNamara
7fee3f6243 Related: tdf#149490 OLE Object dialog should be modal
Change-Id: I909f2afbf149c4d92c97e04c2d8f93648e1e3162
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135655
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-06-13 06:53:31 +02:00
Adolfo Jayme Barrientos
8a09aa3eec Update git submodules
* Update helpcontent2 from branch 'master'
  to 9d11c15431aa1ae77c54d22e8a717cc7bdf3831b
  - Remove hardcoded product name from a translatable string
    
    Change-Id: I7788158520149c3b63b395fca089f6ecdd7b8f78
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/135685
    Tested-by: Jenkins
    Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2022-06-13 05:04:22 +02:00
Julien Nabet
c0e66cfab1 Fix context menu on image which doesnt display
- Launch Writer
- Insert any image in it
- Right click on it
=> Nothing displays
+ console shows:
warn:legacy.osl:67182:67182:vcl/source/window/menu.cxx:2750: PopupMenu::Execute: need a non-NULL window!

Regression from f71606c920
new loplugin:moveit

Change-Id: I89f5bfe2a3cd6a935b419d55c2f3e884c7eed2c7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135681
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-06-12 20:20:24 +02:00
Caolán McNamara
90d4b93c5a crashtesting: forum-mso-de-92960 GetEnglishSearchFontName varying results
depending on if it's called once, or twice on the input string.

The first converts from Full-Width Characters to equivalent lower ASCII
range with non letters retained. And then the second iteration would
convert to lower case ASCII with non letters filtered out. Presumably
the intention is the Full-Width case should get directly to the same
results as multiple calls.

Change-Id: Idba4ebe04c907c160ee53abf6d5551550da032dd
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135678
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-06-12 18:31:47 +02:00
Caolán McNamara
55eeffe768 split out the code that extracts a lower-ascii version of a fontname
Change-Id: I77778ecd075712c41bd29324be090ab4a8351ab9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135677
Tested-by: Caolán McNamara <caolanm@redhat.com>
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
2022-06-12 18:31:22 +02:00
Olivier Hallot
ab37d64477 Add precisions to extended tip
Change-Id: I8d33d95dd1cb8a5b81ae837f78accad6060a94f3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135667
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-12 13:18:20 +02:00
Olivier Hallot
3f2b5da839 Fix dialog name
Change-Id: I19c9157347488cf79acde6cf5af68baddc01a220
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135661
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-12 13:17:23 +02:00
Olivier Hallot
8c91f21b57 Bump docking3deffects.ui to latest Glade
Change-Id: I94fdc2d594f217a727e10fe69f5d7fcc746107bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135664
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-12 13:16:36 +02:00
Olivier Hallot
20ee72b916 Improve extended tip contents
Change-Id: Ie95d20bb1f5c8e153b3a9f9a088c835d5254880f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135665
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-12 13:16:01 +02:00
Olivier Hallot
a599fe4ac2 Bump envformatpage.ui to latest Glade
Change-Id: I74066e2c37a310c242d7222fce48fdc6f23a8a0d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135658
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-12 13:14:30 +02:00
Maxim Monastirsky
881774e9e3 Related: tdf#125040 NB: rework print preview context
Current implementation broadcasts the context change before the
new controller was set on the frame (via XFrame::setComponent).
This isn't going to work for anything LayoutManager based, as
it responds to frame events by recreating all toolbars, so the
newly created toolbar won't see the context sent for the old
controller.

Solve that by delaying the context change to SfxShell::Activate,
like anywhere else in the codebase. And make NB listen to frame
events, so it could also switch listening to the new controller.

Also fixed a crash when switching to print preview in Writer with
the navigator in the sidebar being active. Ideally we shouldn't
even try to create sidebar panels in print preview, as the sidebar
as a whole is disabled. But left that for a future investigation.

Change-Id: I07759c676d2a2eb6f752fe778b559b15d2d759ec
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135492
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky <momonasmon@gmail.com>
2022-06-12 10:38:27 +02:00
offtkp
932257c50c Add compression support for GZ through ZCodec
ZCodec uses zlib to deflate. That file has code
to allow for compressing to GZ but it's wrapped in defines.

There's places in code where there's need for zlib headers
(ie. svx/source/gallery2/codec.cxx:71) and there will be need
for gz header compression when EMZ/WMZ/SVGZ export support
is added.

Made compression functions care about mbGzLib bool. Also added
a SetCompressionMetadata function to set metadata which are otherwise
impossible to know (such as filename, which is needed in the header)

Change-Id: Ic01050262b0b15e0632564b139c66656afac4611
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135213
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl <quikee@gmail.com>
2022-06-12 09:29:53 +02:00
Julien Nabet
988285410f tdf#98743: TIFF export uses 96dpi by default
Create a brand new odg with A4 format with just a rectangle or a smiley on it and export it in tiff.
Since we export the whole page, the size is:
- in cm: 21x29.7 (definition of A4)
- in inches: about 8.2677x11.6929

Taking a look at the value of the generated tiff, we got:
ImageWidth: 794
ImageLength: 1123

Notice that 794/8.2677 like 1123/11.6929 is about 96.
So resolution isn't 300x300 (like the code suggests) but 96x96.
Then let's remove all the useless mechanism to try to find the X and Y resolutions.

TIFF Documentation indicates that "XResolution" and "YResolution" tags use "RATIONAL" type which is defined as:
"Two LONGs: the first represents the numerator of a fraction; the second, the denominator."
Since we got a non floating value, we can use "1" for denominator and "96" for nominator.
With this we change TIFFWriter::ImplWriteResolution implementation to put the nominator first which corresponds to the resolution
passed by argument to the method and the denominator afterwards.

Finally, let's fix the name of the second of ImplWriteResolution, it's the resolution value, not the resolution unit.

Next step would be to use the dialog box for compressing options (like for png or jpg).
But for this I think we should take benefit of external lib "libtiff" since we now use it now for import.

Change-Id: I7dbd04e506e98c344f97e455955cdf2c2f6d83c2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135631
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-06-11 21:35:14 +02:00
Eike Rathke
35edeb2d2c Add English (Mauritius) {en-MU} locale data
Change-Id: Ib384ceb8bdd3268d8d1e076ea2d2d4baeef1f475
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135628
Tested-by: Eike Rathke <erack@redhat.com>
Reviewed-by: Eike Rathke <erack@redhat.com>
2022-06-11 20:32:02 +02:00
Olivier Hallot
3d25248301 Update git submodules
* Update helpcontent2 from branch 'master'
  to 7701d69b3b526f438d9e11ec6c37884c8616422a
  - Add info on sparkline custom vertical max/min
    
    Change-Id: I5e87f7036eb34b1d5cbd0c635b319417981a16b3
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/135639
    Tested-by: Jenkins
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 20:10:55 +02:00
Adolfo Jayme Barrientos
0ebbe1f008 Related: tdf#119827 Add “scale” property for non-interactive labels in more places
Informative or secondary labels should be set apart in the UI’s typographical scale.
This is helpful for more verbose languages, and will contribute to an effort to
make dialogs more space-efficient.

Change-Id: I9eacbebe365a37fe3f973fa6818986d0a4c1fa48
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135656
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2022-06-11 19:40:26 +02:00
Olivier Hallot
ed8dc22564 Update git submodules
* Update helpcontent2 from branch 'master'
  to 8f4baba42e6068c031f51f9683cca53562fbe79e
  - Object anchor is only one.
    
    Change-Id: I7f7c0c6a50e86dc069421648a54d4a9d811588e3
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/135635
    Tested-by: Jenkins
    Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2022-06-11 17:39:56 +02:00
Dietrich Schulten
9638a1ff71 Update git submodules
* Update helpcontent2 from branch 'master'
  to 2f46574a53e1d22e302dc343218ea3ed1ba610b8
  - Relates tdf#36466 - Update help about copying visible cells only
    
    Change-Id: Ia33f942232b73aaad53bb34a5e02b4c457a0d8bc
    Reviewed-on: https://gerrit.libreoffice.org/c/help/+/135541
    Tested-by: Jenkins
    Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 14:09:39 +02:00
Olivier Hallot
5a614a2585 Add precision to extended tip
Change-Id: I9723896a84df297d245e78f582f23d33c8a6dc4b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135625
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 12:49:40 +02:00
Jan-Marek Glogowski
5b0c50544d WASM use 'em-config LLVM_ROOT' to get llvm-nm path
Change-Id: I3073330ed25676d5fbf9f2af008e42828466fc3e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135565
Tested-by: Jenkins
Reviewed-by: Jan-Marek Glogowski <glogow@fbihome.de>
2022-06-11 10:20:22 +02:00
Stephan Bergmann
7e2f0f6c08 Remove some more dead functionality
Change-Id: I653c85d247b2d52c75e12fc0c54701616a44abd7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134148
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-06-11 09:59:09 +02:00
Stephan Bergmann
25fc5f91ab [API CHANGE] Deprecate css.registry.XSimpleRegistry::mergeKey
...and let its implementations just throw a RuntimeException, now that regmerge
has been removed in "[API CHANGE] Remove deprecated idlc and regmerge from the
SDK".  Deprecate css.registry.MergeConflictException along with it (though
stoc/source/implementationregistration/ still uses it internally).  And remove
the previously unused include/registry/writer.hxx along with the newly unused
include/registry/writer.h, RegistryTypeWriter, and RegistryTypeReader.

Change-Id: I10fd286617975dc0ef9f74460a3a3eab8060939d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123615
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-06-11 09:58:49 +02:00
Stephan Bergmann
3ae546f97b Use idlc/test/parser/conversion.tests in CustomTarget_unoidl/unoidl-write_test
...now that idlc is gone since "[API CHANGE] Remove deprecated idlc and regmerge
from the SDK".  (idlc implemented some dubious conversions from floating-point
to integer types, which unoidl-write does not implement, so the original version
of idlc/test/parser/conversion.tests was not usable in
CustomTarget_unoidl/unoidl-write_test.)

Change-Id: I3b69920dda24c54c9313cd5c2cd0c0fdd9951ca7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/123614
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-06-11 09:58:17 +02:00
Rene Engelhard
69c964bbd5 fix 32bit build in new a11y cppunit tests
/<<PKGBUILDDIR>>/toolkit/qa/cppunit/a11y/XAccessibleComponentTester.cxx: In member function ‘void XAccessibleComponentTester::testBounds()’:
/<<PKGBUILDDIR>>/toolkit/qa/cppunit/a11y/XAccessibleComponentTester.cxx:51:5: error: no matching function for call to ‘assertGreaterEqual(int, sal_Int32&, CppUnit::SourceLine, const char [1])’
   51 |     CPPUNIT_ASSERT_GREATEREQUAL(0, bounds.X);
      |     ^
/usr/include/cppunit/TestAssert.h:251:6: note: candidate: ‘template<class T> void CppUnit::assertGreaterEqual(const T&, const T&, CppUnit::SourceLine, const string&)’
  251 | void assertGreaterEqual( const T& expected,
      |      ^~~~~~~~~~~~~~~~~~
/usr/include/cppunit/TestAssert.h:251:6: note:   template argument deduction/substitution failed:
/<<PKGBUILDDIR>>/toolkit/qa/cppunit/a11y/XAccessibleComponentTester.cxx:51:5: note:   deduced conflicting types for parameter ‘const T’ (‘int’ and ‘sal_Int32’ {aka ‘long int’})
   51 |     CPPUNIT_ASSERT_GREATEREQUAL(0, bounds.X);
      |     ^

etc.

Regression from d2a5b4bc0b

Change-Id: I123894e5ed6d6f3bcf46b1c5b37f1f49cb5f9f99
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135630
Tested-by: Jenkins
Tested-by: René Engelhard <rene@debian.org>
Reviewed-by: René Engelhard <rene@debian.org>
2022-06-11 09:07:44 +02:00
Rene Engelhard
f18b06ca51 really fix bigendian builds
this time - since I missed to take over one rename
between trees. sigh. Continues 089c91b1ad

Change-Id: I726410ab1024b995937952fbce1214866ee8ba19
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135629
Tested-by: René Engelhard <rene@debian.org>
Reviewed-by: René Engelhard <rene@debian.org>
2022-06-11 08:44:10 +02:00
Olivier Hallot
da8d90854d Fix English in extended tip
Change-Id: Ifc72a7cacee66f78ae275ec2981c54142d9a6f19
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135619
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 08:28:26 +02:00
Olivier Hallot
df15a079e9 Change extended tip in dimensionlinestabpages.ui
Change-Id: Ibd51662c88d3d3ed0210f4af1f66369bf94c0d62
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135617
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 08:27:48 +02:00
Olivier Hallot
0a7bdf6eeb Bump hangulhanjaoptdialog.ui to latest Glade
Change-Id: Ie9bb451699b2ca9dcd0ace6dd9a341076862d1b8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135618
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 08:27:16 +02:00
Olivier Hallot
46927cdb83 Bump pdfviewpage.ui to latest Glade
Change-Id: Id8c5eca99cf1054587e8554581c32f4d5ca1c974
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135608
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 08:26:25 +02:00
Olivier Hallot
30e6a4748b Bump customslideshow.ui to latest Glade
Change-Id: I15bb8c7bfde9df109af3e9466763ff70c0e0a526
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135612
Tested-by: Jenkins
Reviewed-by: Olivier Hallot <olivier.hallot@libreoffice.org>
2022-06-11 08:25:47 +02:00
Luboš Luňák
7689bf4452 find last data row before searching for first data row
GetLastDataRow() is more efficient than iterating every cell
with HasDataAt(), so first reduce the row range from the end
(in case the whole range is empty cells it's just one call).

Change-Id: Idc494795f68492ed8b05b2cd575598a9c5868b7f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135605
Tested-by: Jenkins
Reviewed-by: Luboš Luňák <l.lunak@collabora.com>
2022-06-11 06:22:12 +02:00
Olivier Hallot
e0c5f8b4f6 Fix typo on xmlfiltertabpagegeneral.ui
Change-Id: I2bc3caa254d997c815bfd36f47504dbc70cadff8
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135611
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos <fitojb@ubuntu.com>
2022-06-11 00:31:41 +02:00
Stephan Bergmann
b32ab8e750 CppunitTest_sd_shape_import_export_tests also needs the ASan hack
...from e854abe076 "Avoid external processes
picking up instdir/program/libnspr4.so", to avoid

> Test name: ShapeImportExportTest::testTextDistancesOOXML_Export
> equality assertion failed
> - Expected: 0
> - Actual  : 32512
> - failed to execute: sh .../bin/officeotron.sh .../tmp/ShapeImportExportTest__testTextDistancesOOXML_Export_2l2moj.tmp > .../tmp/ShapeImportExportTest__testTextDistancesOOXML_Export_2l2mol.tmp 2>&1
> java: symbol lookup error: .../instdir/program/libnspr4.so: undefined symbol: __asan_init

Change-Id: If7f3f9de14c7122b4a08b5a4807b4d28dffb9e06
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135595
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann <sbergman@redhat.com>
2022-06-10 22:33:56 +02:00
Justin Luth
758945b077 tdf#75297 sw uno: override default num char style when NONE
The problem was during ODT import. When no text:style-name
is provided, or if it is an empty string, then the default
"Numbering symbols" character style remained active,
preventing the ability to round-trip a NONE char style.

Since this depends on the built-in name
(and that name can easily enough change)
there is not much point in creating a unit test.

Change-Id: Id942060abd3e024758f93f3d279ef8b561cfc5a9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/134888
Reviewed-by: Justin Luth <jluth@mail.com>
Tested-by: Jenkins
2022-06-10 22:27:37 +02:00
Noel Grandin
95d91cd41e loplugin:constantparam
Change-Id: I8134744b6c1279c497d4763eddf614bb840f7f3f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135602
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-06-10 21:14:20 +02:00
Noel Grandin
3b5c926777 loplugin:unnecessaryvirtual
Change-Id: Ica246a12d93f5417ad857b9484575d03c66f761a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135601
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-06-10 21:14:02 +02:00
Noel Grandin
230e461065 loplugin:unusedvarsglobal
Change-Id: I37a1772ffdd682d17571aa06e24089136d2d8547
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135600
Tested-by: Jenkins
Reviewed-by: Noel Grandin <noel.grandin@collabora.co.uk>
2022-06-10 21:13:43 +02:00
Julien Nabet
7a7afd3ae5 tdf#148978: Business Cards in Writer dont show database
+ remove "medium" tab since its goal is exactly what "label" tab does.
So now "label" tab is common for "File - New - Business Cards" and "File - New - Labels"

Change-Id: I9c7f4e74d7cbc0ffa8ee146a0ef39c87ff5261e3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135566
Tested-by: Jenkins
Reviewed-by: Julien Nabet <serval2412@yahoo.fr>
2022-06-10 18:32:31 +02:00