[Libreoffice-commits] core.git: sw/source

2022-06-21 Thread Stephan Bergmann (via logerrit)
 sw/source/core/unocore/unosett.cxx |   30 +-
 1 file changed, 13 insertions(+), 17 deletions(-)

New commits:
commit 28597ee6f713baa6833e6e7837faac46ba5c173a
Author: Stephan Bergmann 
AuthorDate: Tue Jun 21 23:10:30 2022 +0200
Commit: Stephan Bergmann 
CommitDate: Wed Jun 22 07:58:56 2022 +0200

Make code a bit more explicit

...and avoid the ugly unsigned short vs. SvxAdjust mismatch

Change-Id: Ie820b5fda00942d258668535598027f9d878c013
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136264
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 

diff --git a/sw/source/core/unocore/unosett.cxx 
b/sw/source/core/unocore/unosett.cxx
index a871d4f1c844..dcc6f1fb7abd 100644
--- a/sw/source/core/unocore/unosett.cxx
+++ b/sw/source/core/unocore/unosett.cxx
@@ -250,16 +250,6 @@ const o3tl::enumarray aSvxToUnoAdjust
 sal_Int16(-1)
 };
 
-const unsigned short aUnoToSvxAdjust[] =
-{
-USHRT_MAX,
-static_cast(SvxAdjust::Right),   // 1
-static_cast(SvxAdjust::Center),  // 3
-static_cast(SvxAdjust::Left),// 0
-USHRT_MAX,
-USHRT_MAX
-};
-
 OUString SwXFootnoteProperties::getImplementationName()
 {
 return "SwXFootnoteProperties";
@@ -1537,14 +1527,20 @@ void SwXNumberingRules::SetPropertiesToNumFormat(
 {
 sal_Int16 nValue = text::HoriOrientation::NONE;
 rProp.Value >>= nValue;
-if (nValue > text::HoriOrientation::NONE &&
-nValue <= text::HoriOrientation::LEFT &&
-USHRT_MAX != aUnoToSvxAdjust[nValue])
-{
-
aFormat.SetNumAdjust(static_cast(aUnoToSvxAdjust[nValue]));
-}
-else
+switch (nValue) {
+case text::HoriOrientation::RIGHT:
+aFormat.SetNumAdjust(SvxAdjust::Right);
+break;
+case text::HoriOrientation::CENTER:
+aFormat.SetNumAdjust(SvxAdjust::Center);
+break;
+case text::HoriOrientation::LEFT:
+aFormat.SetNumAdjust(SvxAdjust::Left);
+break;
+default:
 bWrongArg = true;
+break;
+}
 }
 else if (rProp.Name == UNO_NAME_PARENT_NUMBERING)
 {


[Libreoffice-bugs] [Bug 149635] FEATURE REQUEST: remove "after" separator in cross-reference to numbered list item

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149635

--- Comment #2 from ajlittoz  ---
Update to

> But authors expect the final separator to be suppressed, be it 
> multi-character or different from full stop.

Thinking about it, suppression is more complicated. List number may be
decorated with both Before and After separators. Both should be suppressed.
This requires querying the level in the list style definition or retrieving the
multi-level counter without adding the "side" separators.

== Example for single-level list:

[1] Step one
[2] Step two

In step 2 above, …

== end of example

== Example for multi-level list:

[1] Step one
<1.1> First sub-step in first step

In action 1.1 above, …

== end of example

*** The following remark is not part of the present feature request. ***

List style allows to define many decorators (before and after separators at
each level) but the "internal" separator is hard-coded at full stop. It can't
be replaced by a dash for example. Is there a "typographical" reason for it? It
don't see an exclusive agreement in books. Other conventions exist.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-ux-advise] [Bug 149635] FEATURE REQUEST: remove "after" separator in cross-reference to numbered list item

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149635

--- Comment #2 from ajlittoz  ---
Update to

> But authors expect the final separator to be suppressed, be it 
> multi-character or different from full stop.

Thinking about it, suppression is more complicated. List number may be
decorated with both Before and After separators. Both should be suppressed.
This requires querying the level in the list style definition or retrieving the
multi-level counter without adding the "side" separators.

== Example for single-level list:

[1] Step one
[2] Step two

In step 2 above, …

== end of example

== Example for multi-level list:

[1] Step one
<1.1> First sub-step in first step

In action 1.1 above, …

== end of example

*** The following remark is not part of the present feature request. ***

List style allows to define many decorators (before and after separators at
each level) but the "internal" separator is hard-coded at full stop. It can't
be replaced by a dash for example. Is there a "typographical" reason for it? It
don't see an exclusive agreement in books. Other conventions exist.

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-bugs] [Bug 91566] add different access to list of sheets

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=91566

--- Comment #7 from Jaise James  ---
Created attachment 180886
  --> https://bugs.documentfoundation.org/attachment.cgi?id=180886=edit
Sheet list view in Onlyoffice

Sheet list view in Libreoffice is very good feature.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149666] Units in Individual Gits on Sourceforge.net for Source-code: //sourceforge.net/projects/libraoffice to be heuristically added!!

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149666

Buovjaga  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |INVALID

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 90796] [META] HiDPI / Retina bugs

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=90796

Aron Budea  changed:

   What|Removed |Added

 Depends on|149291  |


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=149291
[Bug 149291] Mouse cursor freeze if I resize sidebar in Writer / Calc
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 141294] Decks of Sidebar on HiDPI gets stuck in resize loop upon attempted width adjustment, work around of setting new SB "MinimumWidth" to false in expert config

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=141294

Aron Budea  changed:

   What|Removed |Added

 CC||libreoffice.p-foerster@gmx.
   ||de

--- Comment #14 from Aron Budea  ---
*** Bug 149291 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149291] Mouse cursor freeze if I resize sidebar in Writer / Calc

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149291

Aron Budea  changed:

   What|Removed |Added

 CC||aron.bu...@gmail.com
 Resolution|--- |DUPLICATE
 Blocks|90796   |
 Status|UNCONFIRMED |RESOLVED

--- Comment #5 from Aron Budea  ---
Looks like a duplicate of bug 141294.

*** This bug has been marked as a duplicate of bug 141294 ***


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=90796
[Bug 90796] [META] HiDPI / Retina bugs
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149621] PDF documentation files downloaded from motherboard manufacturers require password that doesn't exist.

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149621

--- Comment #8 from S.B.  ---
That's OK :-)  The top of page 59 in the specification guide may be a clue as
to where the failure is occurring (detection of the default user password which
should trigger an automatic decryption).  Perhaps check the routine in
LibreOffice that reads the "padding string" to make sure it is being read
correctly?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 147873] LO 7.2.5.2 Calc Eventual Freezing Crash in Mac silicon M1 Mac Book Pro

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=147873

QA Administrators  changed:

   What|Removed |Added

 Whiteboard| QA:needsComment|

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149482] XModifyListener triggering infinite creation of new sheets

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149482

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149477] XLSX parser/generator ignores SpreadsheetML string encoding

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149477

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149637] Transparency on background image does not export to pdf

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149637

QA Administrators  changed:

   What|Removed |Added

 Status|NEEDINFO|UNCONFIRMED
 Ever confirmed|1   |0

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149637] Transparency on background image does not export to pdf

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149637

--- Comment #4 from QA Administrators  ---
[Automated Action] NeedInfo-To-Unconfirmed

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149605] Conflict with VoiceOver/NVDA

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149605

QA Administrators  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149605] Conflict with VoiceOver/NVDA

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149605

--- Comment #3 from QA Administrators  ---
[Automated Action] NeedInfo-To-Unconfirmed

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145793] Les tableaux dans Libre Office Writer

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145793

QA Administrators  changed:

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |INSUFFICIENTDATA

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145793] Les tableaux dans Libre Office Writer

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145793

--- Comment #3 from QA Administrators  ---
Dear Jacques Breuneval,

Please read this message in its entirety before proceeding.

Your bug report is being closed as INSUFFICIENTDATA due to inactivity and
a lack of information which is needed in order to accurately
reproduce and confirm the problem. We encourage you to retest
your bug against the latest release. If the issue is still
present in the latest stable release, we need the following
information (please ignore any that you've already provided):

a) Provide details of your system including your operating
   system and the latest version of LibreOffice that you have
   confirmed the bug to be present

b) Provide easy to reproduce steps – the simpler the better

c) Provide any test case(s) which will help us confirm the problem

d) Provide screenshots of the problem if you think it might help

e) Read all comments and provide any requested information

Once all of this is done, please set the bug back to UNCONFIRMED
and we will attempt to reproduce the issue. Please do not:

a) respond via email 

b) update the version field in the bug or any of the other details
   on the top section of our bug tracker

Warm Regards,
QA Team

MassPing-NeedInfo-FollowUp

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 102495] [META] KDE VCL backend bugs and enhancements

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=102495
Bug 102495 depends on bug 145787, which changed state.

Bug 145787 Summary: Libreoffice 7.2 (Main) and Libreoffice 7.2 Writer crash 
crash on picture insert dialogue
https://bugs.documentfoundation.org/show_bug.cgi?id=145787

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |INSUFFICIENTDATA

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145787] Libreoffice 7.2 (Main) and Libreoffice 7.2 Writer crash crash on picture insert dialogue

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145787

QA Administrators  changed:

   What|Removed |Added

 Resolution|--- |INSUFFICIENTDATA
 Status|NEEDINFO|RESOLVED

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 145787] Libreoffice 7.2 (Main) and Libreoffice 7.2 Writer crash crash on picture insert dialogue

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145787

--- Comment #6 from QA Administrators  ---
Dear Christian Haunert,

Please read this message in its entirety before proceeding.

Your bug report is being closed as INSUFFICIENTDATA due to inactivity and
a lack of information which is needed in order to accurately
reproduce and confirm the problem. We encourage you to retest
your bug against the latest release. If the issue is still
present in the latest stable release, we need the following
information (please ignore any that you've already provided):

a) Provide details of your system including your operating
   system and the latest version of LibreOffice that you have
   confirmed the bug to be present

b) Provide easy to reproduce steps – the simpler the better

c) Provide any test case(s) which will help us confirm the problem

d) Provide screenshots of the problem if you think it might help

e) Read all comments and provide any requested information

Once all of this is done, please set the bug back to UNCONFIRMED
and we will attempt to reproduce the issue. Please do not:

a) respond via email 

b) update the version field in the bug or any of the other details
   on the top section of our bug tracker

Warm Regards,
QA Team

MassPing-NeedInfo-FollowUp

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 144930] LibreOffice v7.2.1 locked up my computer.

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144930

QA Administrators  changed:

   What|Removed |Added

 Resolution|--- |INSUFFICIENTDATA
 Status|NEEDINFO|RESOLVED

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 144930] LibreOffice v7.2.1 locked up my computer.

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144930

--- Comment #4 from QA Administrators  ---
Dear scruffy-temp,

Please read this message in its entirety before proceeding.

Your bug report is being closed as INSUFFICIENTDATA due to inactivity and
a lack of information which is needed in order to accurately
reproduce and confirm the problem. We encourage you to retest
your bug against the latest release. If the issue is still
present in the latest stable release, we need the following
information (please ignore any that you've already provided):

a) Provide details of your system including your operating
   system and the latest version of LibreOffice that you have
   confirmed the bug to be present

b) Provide easy to reproduce steps – the simpler the better

c) Provide any test case(s) which will help us confirm the problem

d) Provide screenshots of the problem if you think it might help

e) Read all comments and provide any requested information

Once all of this is done, please set the bug back to UNCONFIRMED
and we will attempt to reproduce the issue. Please do not:

a) respond via email 

b) update the version field in the bug or any of the other details
   on the top section of our bug tracker

Warm Regards,
QA Team

MassPing-NeedInfo-FollowUp

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 133903] Slow PDF export of sheet with charts based on large dataset (Print to PDF is fast)

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=133903

--- Comment #7 from QA Administrators  ---
Dear Telesto,

This bug has been in NEEDINFO status with no change for at least
6 months. Please provide the requested information as soon as
possible and mark the bug as UNCONFIRMED. Due to regular bug
tracker maintenance, if the bug is still in NEEDINFO status with
no change in 30 days the QA team will close the bug as INSUFFICIENTDATA
due to lack of needed information.

For more information about our NEEDINFO policy please read the
wiki located here:
https://wiki.documentfoundation.org/QA/Bugzilla/Fields/Status/NEEDINFO

If you have already provided the requested information, please
mark the bug as UNCONFIRMED so that the QA team knows that the
bug is ready to be confirmed.

Thank you for helping us make LibreOffice even better for everyone!

Warm Regards,
QA Team

MassPing-NeedInfo-Ping

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149667] hanahiroki88

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149667

hanahiroki88  changed:

   What|Removed |Added

URL||https://eillsaptr.xyz/

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149667] New: hanahiroki88

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149667

Bug ID: 149667
   Summary: hanahiroki88
   Product: Document Liberation Project
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: enhancement
  Priority: medium
 Component: libabw
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: hanahirok...@gmail.com

https://eillsaptr.xyz/

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149666] Units in Individual Gits on Sourceforge.net for Source-code: //sourceforge.net/projects/libraoffice to be heuristically added!!

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149666

--- Comment #1 from Gc. Jerrimough Sebastian Xaa  ---
Created attachment 180885
  --> https://bugs.documentfoundation.org/attachment.cgi?id=180885=edit
Chronolabs Cooperative -- Stien update from preleaked txt/txtheader systems

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149666] New: Units in Individual Gits on Sourceforge.net for Source-code: //sourceforge.net/projects/libraoffice to be heuristically added!!

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149666

Bug ID: 149666
   Summary: Units in Individual Gits on Sourceforge.net for
Source-code: //sourceforge.net/projects/libraoffice to
be heuristically added!!
   Product: LibreOffice
   Version: Inherited From OOo
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Base
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: empres...@outlook.com

Description:
For the Visual Scopic Interface to be LibraOffice Powered with the stien call
header on
https://chronolabs-cooperative.sourceforge.io/Stiens/Headers/Manette.txt on the
timing bells as well as everyone eye goggles to have libraoffice systems we
need each unit that individual component like: Wordprocessor; Database;
Spreedsheet so on in individual gits on sourceforge.net not just one giant pack
you can have endless amounts of gits on sourceforge.net!!

Steps to Reproduce:
1. Source-code in none metastasized units of distribution
2. Even though will work in archive base need pharsing better in gits
3. SVN more for media, documents, icons, terms, etc. with revision numbers not
checksum like git

Actual Results:
I have already pactched your sourceforge.net with:
ChronolabsCoop-Stiens.tar.lzma which when someone makes a project or you have
to send to all your projects on their webspace sourceforge.io/sourceforge.net
to txt@ as well as c/c stien@ as an email from a whitelisted source which I
have updated you can check them out at::
https://chronolabs-cooperative.sourceforge.io/Stiens/Control/Whitelists/

Expected Results:
Then you need to add the username 'chronolabscoop' as well as additionally
'wishcraft' for just ISBN and 'sparx' for FSLB as a member to a project not
developer it will automatically dial them in thee heuristics of the new generic
Manette.txt on the timing bells; do kraussgrading as well as make it so we
develop documents and database via foot fall and resources based in
libraoffice; as the only site that support still stien's header even on the old
txt/txtheader system call of: 'Source-code:
//sourceforge.net/project/unixname'...


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
###
## /etc/ntp.conf, stienheaders update; see stien master for help
##
## You may not change or alter any portion of this comment or credits
## of supporting developers from this source code or any supporting source code
## which is considered copyrighted (c) material of the original comment or
credit authors.
## This program is distributed in the hope that it will be useful,
## but WITHOUT ANY WARRANTY; without even the implied warranty of
## MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
##
## @copyright   Chronolabs Cooperative
https://github.com/Chronolabs-Cooperative
## @license ACADEMIC APL 2
(https://sourceforge.net/u/chronolabscoop/wiki/Academic%20Public%20License%2C%20version%202.0/)
## @license GNU GPL 3 (http://www.gnu.org/licenses/gpl.html)
## @license CC (https://creativecommons.org/about/cclicenses/)
## @package scripts
## @subpackage  defaults
## @since   499.11.12
## @version 499.11.12
## @author  Dr. Simon Antony Roberts

## @author  Gc. Fault Jenny Samantha Xaa

## @linkhttps://sourceforge.net/projects/chronolabs-cooperative
## @linkhttps://sourceforge.net/projects/stien
## @linkhttps://sourceforge.net/projects/vcf5
## @linkhttps://sourceforge.net/projects/distancesid
## @linkhttps://internetfounder.wordpress.com
##
##
## ./.
## bit.ly/MoneyStiens
## bit.ly/Kustomised
unblock sourceforge.net
"@jerrimoughxaa @RegaltyFamily
import-uri://chronolabs-cooperative.sourceforge.io/Stiens/"
confirm daily
define Assiopath.stien
define Assiopathic.stien
define Bainuate.stien
define Baituate.stien
define Enchassiment.stien
define Inchassiment.stien
define Indigenous.stien
define Offworld.stien
define Outworld.stien
define Sponiate.stien
define Spotiate.stien
print ChronolabsCoop-Complete.anyname by Tiesaa Binders with Coverslip
confirm daily
print --ChronolabsCoop-Complete.7z by Tiesaa Binders with Coverslip
confirm daily
print ChronolabsCoop-Complete.7z by Tiesaa Binders with Coverslip
confirm daily   
print --ChronolabsCoop-Complete.7z by Tiesaa Binders with Coverslip
confirm daily   
print ChronolabsCoop-Complete.7z by Tiesaa Binders with Coverslip
confirm daily
print society-grumpy-old-individuals.jpg (Poster-Ax-Random) by Tiesaa Binders
with Coverslip
print The Physics Department Galantus by Tiesaa Binders with Covernote
print The Physics Department Galantus (Poster-Ax-Random) by Tiesaa Binders with
Covernote
## /./
##
## You can with a delivery dock reciept with your visual scopic interface in
PDF if you 

[Libreoffice-bugs] [Bug 149619] unresponsive document, lowriter freezes, brk(0xfffc000) = 0xfffc000

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149619

Scott Clewell  changed:

   What|Removed |Added

 CC||sclew...@protonmail.com

--- Comment #2 from Scott Clewell  ---
Thanks for reporting vis_qa!

I cannot repo but it does take 25 seconds for this to open on both these
versions of LO:

Version: 7.3.4.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.4~rc2-0ubuntu0.20.04.1~lo1
Calc: threaded

Version: 7.4.0.0.alpha1+ / LibreOffice Community
Build ID: c1898a4d1b38e0077416a104e2b47573ce207ffa
CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149626] HANG during basic macro using com.sun.star.datatransfer.dnd

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149626

--- Comment #2 from dominic.coo...@domelia.co.uk  
---
Created attachment 180884
  --> https://bugs.documentfoundation.org/attachment.cgi?id=180884=edit
Backtrace

Further to the above, I have added a backtrace. I am nowhere near enough
knowlegdable about the workings of the LibO engine, but the point of hanging
seems to be when it acquires a mutex lock at
vcl/source/window/dndeventdispatcher.cxx:156.

Only problem is that this code has been in place for some months now, so it
doesn't explain the sudden crash, but perhaps something else has also obtained
a mutex on another thread or in the background? Hope this helps!

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149665] Cannot enter a single quotation mark followed by on or more digits into a cell via keyboard

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149665

--- Comment #2 from LeroyG  ---
I think that a double apostrophe (U+27 U+27) at the beginning of a cell says
that all cell content is text, and all cell content must be showed. For me this
is not a bug.


By the way, you can type:
 - "‘" (left single quotation mark, the same that U+2018) or "’" (right single
quotation mark, the same that U+2019)
 - "1"
and it would work fine.


If AutoCorrect option "[x] Replace" for Single Quotes is checked, you can type:
 - "'" apostrophe (or, maybe, a space or whatever character except "=", "+",
and "-")
 - "'" apostrophe (it is not the same than a single quotation mark, but in this
position will be converted to a right single quotation mark)¹
 - "1"
Press:
 - F2 key (to enter edit mode)
 - Home key
 - Del key (to delete the space)
 - Enter

¹ A left single quotation mark if the first typed a space.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: Branch 'libreoffice-7-4' - sc/source

2022-06-21 Thread Eike Rathke (via logerrit)
 sc/source/core/tool/compiler.cxx |7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

New commits:
commit 9f3bce835e81dbdf07da9850a5157f4403819417
Author: Eike Rathke 
AuthorDate: Tue Jun 21 23:06:45 2022 +0200
Commit: Eike Rathke 
CommitDate: Wed Jun 22 01:19:04 2022 +0200

Related: tdf#142293 Consider bAsciiUpper for matching CharClass to lowercase

... as also that may had lead to the locale's uppercase.

Change-Id: I5c75b4e52260539f5b66c0d2195b2137c9b1b8ab
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136265
Reviewed-by: Eike Rathke 
Tested-by: Jenkins
(cherry picked from commit 537d69b6f1c32e5b938d9ef24c4154a08d565363)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136237

diff --git a/sc/source/core/tool/compiler.cxx b/sc/source/core/tool/compiler.cxx
index 301d90ed3b7a..2945062f1b37 100644
--- a/sc/source/core/tool/compiler.cxx
+++ b/sc/source/core/tool/compiler.cxx
@@ -4464,6 +4464,7 @@ bool ScCompiler::NextNewToken( bool bInArray )
 // ParseReference().
 
 OUString aUpper;
+bool bAsciiUpper = false;
 
 Label_Rewind:
 
@@ -4483,7 +4484,7 @@ Label_Rewind:
 
 mbRewind = false;
 aUpper.clear();
-bool bAsciiUpper = false;
+bAsciiUpper = false;
 
 if (bAsciiNonAlnum)
 {
@@ -4554,6 +4555,8 @@ Label_Rewind:
 // more likely in that localized language than in the formula
 // language. This in corner cases needs to continue to work for
 // existing documents and environments.
+// Do not change bAsciiUpper from here on for the lowercase() call
+// below in the ocBad case to use the correct CharClass.
 aUpper = ScGlobal::getCharClass().uppercase( aOrg );
 }
 
@@ -4618,7 +4621,7 @@ Label_Rewind:
 // would prematurely end compilation. Simple unknown names are handled by
 // the interpreter.
 // Use the same CharClass that was used for uppercase.
-aUpper = (mbCharClassesDiffer ? ScGlobal::getCharClass() : 
*pCharClass).lowercase( aUpper );
+aUpper = ((bAsciiUpper || mbCharClassesDiffer) ? ScGlobal::getCharClass() 
: *pCharClass).lowercase( aUpper );
 svl::SharedString aSS = rDoc.GetSharedStringPool().intern(aUpper);
 maRawToken.SetString(aSS.getData(), aSS.getDataIgnoreCase());
 maRawToken.NewOpCode( ocBad );


[Libreoffice-bugs] [Bug 149664] Cannot enter a single quotation mark followed by on or more digits into a cell

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149664

LeroyG  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|UNCONFIRMED |RESOLVED

--- Comment #1 from LeroyG  ---


*** This bug has been marked as a duplicate of bug 149665 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149665] Cannot enter a single quotation mark followed by on or more digits into a cell via keyboard

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149665

--- Comment #1 from LeroyG  ---
*** Bug 149664 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149661] [Feature request] Automatic table-background-coloring ported from Calc to Writer.

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149661

--- Comment #3 from Sion  ---
Agree, that everything should be paste-able.

It would also solve another related problem I have. 
(Maybe I should open another ticket for this problem?)
I have a field with data that is not compatible with the Color Scale function:
it is a field with times, it was originally measured in minutes, but I didn't
like it, so I translated that field to be more readable, but it is incompatible
now:

ex:
Old time: 230 (minutes)
New time: 3h 50m (my "translation")

I would like to use the "Old time" field to generate the background colors,
(then "lock" the colors?), and then "paste" those colors into the "New time"
field, without changing the colors in the new field.

I think there is some very roundabout way to do this, I saw something about
mapping letters to numbers, so it was possible to sort fields by text, or
something like that, so maybe it is possible to map "3h 50m" to the value
"230", but as said, it is a VERY roundabout way.

Another solution would be to type the "New Time" field as:
"3h 50m (230)" and then use a Regular-Expression to extract the "230" part of
that text & use that for sorting.

Or a 3rd solution, would be to use a different "Source Column" (= "Old Time"),
and use that column to generate the colors, and then have a "Destination
Column" (= "New Time") where the colors is inserted.
Maybe it can be written with a similar syntax as eg. VLOOKUP() or ROUND() or
similar functions?
Something like:
BACKGROUND_COLOR(source_range; target_range) or something like that?

(But again, maybe I should open another ticket for this problem?)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: sc/source

2022-06-21 Thread Eike Rathke (via logerrit)
 sc/source/core/tool/compiler.cxx |7 +--
 1 file changed, 5 insertions(+), 2 deletions(-)

New commits:
commit 086055b0d7e44d1d07b3f23af55503e6a3924d87
Author: Eike Rathke 
AuthorDate: Tue Jun 21 23:06:45 2022 +0200
Commit: Eike Rathke 
CommitDate: Wed Jun 22 00:23:51 2022 +0200

Related: tdf#142293 Consider bAsciiUpper for matching CharClass to lowercase

... as also that may had lead to the locale's uppercase.

Change-Id: I5c75b4e52260539f5b66c0d2195b2137c9b1b8ab
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136265
Reviewed-by: Eike Rathke 
Tested-by: Jenkins

diff --git a/sc/source/core/tool/compiler.cxx b/sc/source/core/tool/compiler.cxx
index 301d90ed3b7a..2945062f1b37 100644
--- a/sc/source/core/tool/compiler.cxx
+++ b/sc/source/core/tool/compiler.cxx
@@ -4464,6 +4464,7 @@ bool ScCompiler::NextNewToken( bool bInArray )
 // ParseReference().
 
 OUString aUpper;
+bool bAsciiUpper = false;
 
 Label_Rewind:
 
@@ -4483,7 +4484,7 @@ Label_Rewind:
 
 mbRewind = false;
 aUpper.clear();
-bool bAsciiUpper = false;
+bAsciiUpper = false;
 
 if (bAsciiNonAlnum)
 {
@@ -4554,6 +4555,8 @@ Label_Rewind:
 // more likely in that localized language than in the formula
 // language. This in corner cases needs to continue to work for
 // existing documents and environments.
+// Do not change bAsciiUpper from here on for the lowercase() call
+// below in the ocBad case to use the correct CharClass.
 aUpper = ScGlobal::getCharClass().uppercase( aOrg );
 }
 
@@ -4618,7 +4621,7 @@ Label_Rewind:
 // would prematurely end compilation. Simple unknown names are handled by
 // the interpreter.
 // Use the same CharClass that was used for uppercase.
-aUpper = (mbCharClassesDiffer ? ScGlobal::getCharClass() : 
*pCharClass).lowercase( aUpper );
+aUpper = ((bAsciiUpper || mbCharClassesDiffer) ? ScGlobal::getCharClass() 
: *pCharClass).lowercase( aUpper );
 svl::SharedString aSS = rDoc.GetSharedStringPool().intern(aUpper);
 maRawToken.SetString(aSS.getData(), aSS.getDataIgnoreCase());
 maRawToken.NewOpCode( ocBad );


[Libreoffice-bugs] [Bug 123305] Libreoffice rendering on HDPI displays blurred

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=123305

rscra...@gmail.com  changed:

   What|Removed |Added

 Status|NEEDINFO|UNCONFIRMED
 Ever confirmed|1   |0

--- Comment #28 from rscra...@gmail.com  ---
Set from 'NEEDINFO' to 'UNCONFIRMED'. Only icons were blurry.

Version: 7.3.4.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: en-US (en_US.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.4~rc2-0ubuntu0.22.04.1~lo1
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: bin/crashreportScraper.py

2022-06-21 Thread Xisco Fauli (via logerrit)
 bin/crashreportScraper.py |5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

New commits:
commit 6b15decd0acc2c7c0622baba65b9d4c019a4183a
Author: Xisco Fauli 
AuthorDate: Tue Jun 21 15:40:33 2022 +0200
Commit: Xisco Fauli 
CommitDate: Tue Jun 21 23:51:50 2022 +0200

crashreportScraper: encode the url

Change-Id: I1f738f017966a6fe48dd9e2cf36dbdf5f50c0cef
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136229
Tested-by: Xisco Fauli 
Reviewed-by: Xisco Fauli 

diff --git a/bin/crashreportScraper.py b/bin/crashreportScraper.py
index 513f5ec7b75d..cad7feead645 100755
--- a/bin/crashreportScraper.py
+++ b/bin/crashreportScraper.py
@@ -17,6 +17,7 @@ import sys
 import os
 import math
 from datetime import datetime
+import urllib.parse
 
 def convert_str_to_date(value):
 value = value.replace('.', '')
@@ -194,11 +195,11 @@ if __name__ == '__main__':
 f.flush()
 
 for k, lDate in crashes.items():
-if len(k) < 254 and k not in crashesInFile and '`' not in k:
+if k not in crashesInFile:
 print("Parsing " + k)
 try:
 crashCount, crashID, crashVersion, crashOS = 
parse_reports_and_get_most_recent_report_from_last_page(
-
"https://crashreport.libreoffice.org/stats/signature/; + k)
+
"https://crashreport.libreoffice.org/stats/signature/; + urllib.parse.quote(k))
 crashReason, crashStack, codeLine = 
parse_details_and_get_info(
 
"https://crashreport.libreoffice.org/stats/crash_details/; + crashID, 
args.repository)
 line = '\t'.join([k, str(crashCount), lDate[1], lDate[2],


[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/qa sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/qa/extras/uiwriter/uiwriter2.cxx |   95 
 sw/source/core/doc/DocumentContentOperationsManager.cxx |4 
 sw/source/core/doc/docedt.cxx   |7 +
 sw/source/core/inc/UndoRedline.hxx  |8 +
 sw/source/core/inc/mvsave.hxx   |3 
 sw/source/core/undo/unredln.cxx |   34 +
 6 files changed, 144 insertions(+), 7 deletions(-)

New commits:
commit 650d0ead161303f30faa2de0bb4e20fceafc2af8
Author: Michael Stahl 
AuthorDate: Wed Jun 15 15:06:59 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:43:19 2022 +0200

tdf#135976 sw: preserve flys on backspace/delete with redlining enabled

This is a continuation of commit 85376a02348810812d515ee72140dbf56f2b6040
for the case when redlining is turned on.

Also try to restore the anchors in SwUndoRedlineDelete.

(regression from commit 3345feb67f2c49a1b76639965b56968e1c5f03ee)

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135909
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 932a8efce878547bfd81521d0cf1ddfe8dc33ec6)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135968
Reviewed-by: Thorsten Behrens 
(cherry picked from commit 03834f003c56e6646e3b274c418acd4fc344fd8e)

sw: fix odd m_bCanGroup check in SwUndoRedlineDelete

This looks like copypasta, presumably both flags must be true to allow
grouping.

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135908
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit f31f11f3222933dbc96dc672e6fa52233cda12be)

Change-Id: I4199f5755398d469a606618c037ad9756cb7aeba
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136223
Tested-by: Thorsten Behrens 
Reviewed-by: Thorsten Behrens 

diff --git a/sw/qa/extras/uiwriter/uiwriter2.cxx 
b/sw/qa/extras/uiwriter/uiwriter2.cxx
index e00e27548fcc..1418ddd0ae51 100644
--- a/sw/qa/extras/uiwriter/uiwriter2.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter2.cxx
@@ -28,6 +28,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -958,6 +959,100 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf139982)
 CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
 }
 
+CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf135976)
+{
+SwDoc* const pDoc = createDoc();
+SwWrtShell* const pWrtShell = pDoc->GetDocShell()->GetWrtShell();
+
+pWrtShell->Insert("foobar");
+
+pWrtShell->Left(CRSR_SKIP_CHARS, /*bSelect=*/false, 2, 
/*bBasicCall=*/false);
+SwFormatAnchor anchor(RndStdIds::FLY_AT_CHAR);
+anchor.SetAnchor(pWrtShell->GetCursor()->GetPoint());
+SfxItemSet flySet(pDoc->GetAttrPool(), svl::Items{});
+flySet.Put(anchor);
+SwFrameFormat const* pFly = pWrtShell->NewFlyFrame(flySet, 
/*bAnchValid=*/true);
+CPPUNIT_ASSERT(pFly != nullptr);
+
+// turn on redlining and show changes
+pDoc->getIDocumentRedlineAccess().SetRedlineFlags(RedlineFlags::On | 
RedlineFlags::ShowDelete
+  | 
RedlineFlags::ShowInsert);
+lcl_dispatchCommand(mxComponent, ".uno:ShowTrackedChanges", {});
+CPPUNIT_ASSERT_MESSAGE("redlining should be on",
+   pDoc->getIDocumentRedlineAccess().IsRedlineOn());
+CPPUNIT_ASSERT_MESSAGE(
+"redlines should be visible",
+
IDocumentRedlineAccess::IsShowChanges(pDoc->getIDocumentRedlineAccess().GetRedlineFlags()));
+CPPUNIT_ASSERT(pWrtShell->GetLayout()->IsHideRedlines());
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+CPPUNIT_ASSERT_EQUAL(size_t(1), 
pWrtShell->GetLayout()->GetLastPage()->GetSortedObjs()->size());
+CPPUNIT_ASSERT_EQUAL(sal_Int32(4), 
pFly->GetAnchor().GetContentAnchor()->nContent.GetIndex());
+
+pWrtShell->UnSelectFrame();
+pWrtShell->SttEndDoc(/*bStart=*/false);
+pWrtShell->Left(CRSR_SKIP_CHARS, /*bSelect=*/false, 1, 
/*bBasicCall=*/false);
+
+pWrtShell->DelLeft();
+pWrtShell->DelLeft();
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+// the problem was that the fly was deleted from the layout
+CPPUNIT_ASSERT_EQUAL(size_t(1), 
pWrtShell->GetLayout()->GetLastPage()->GetSortedObjs()->size());
+// check that the anchor was moved outside the redline
+CPPUNIT_ASSERT_EQUAL(sal_Int32(3), 
pFly->GetAnchor().GetContentAnchor()->nContent.GetIndex());
+
+pWrtShell->Undo(2);
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+CPPUNIT_ASSERT_EQUAL(size_t(1), 
pWrtShell->GetLayout()->GetLastPage()->GetSortedObjs()->size());
+// check that the anchor was restored
+CPPUNIT_ASSERT_EQUAL(sal_Int32(4), 
pFly->GetAnchor().GetContentAnchor()->nContent.GetIndex());
+
+pWrtShell->Redo(2);
+
+

[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/source/core/doc/DocumentRedlineManager.cxx |6 ++
 1 file changed, 6 insertions(+)

New commits:
commit 358197bcbbb2a24e2a4378786b494cf2ce360d15
Author: Michael Stahl 
AuthorDate: Tue Jun 14 17:19:23 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:42:24 2022 +0200

sw_redlinehide: skip unnecessary updates when undoing redlined delete

When reproducing tdf#135976 and then Undo, an UAF crash happens here:

  assert(!pFrame->GetDrawObjs() || !pObjs->Contains(*pObj));

The pObjs was actually deleted and then re-created, because the pObj was
removed from the frame and added again to the same frame.

This is a bit unexpected, so prevent it by taking a shortcut in the
caller UpdateFramesForRemoveDeleteRedline() to insert a check that had
been removed in commit 14e87a4b15d31a34e6053f6194688f3aa23af991.

If the rPam is inside a single node, the sw::RedlineUnDelText hint that
was sent to the SwTextFrame should be sufficient to update it and the
rest of the code in the loop that deals with newly split paragraph can
be skipped.

Change-Id: I5f36eb91bc20003887ee0bad03ea4a6e67135de9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135907
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit cf9a16caf5012d65b2a45a5525e36e40585dd35c)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135892
Reviewed-by: Thorsten Behrens 
(cherry picked from commit 918f435d48de3f29814f187c8621c1a564c5b835)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136222
Tested-by: Thorsten Behrens 

diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx 
b/sw/source/core/doc/DocumentRedlineManager.cxx
index 8ae32f266bdb..e11c1c6f2c4b 100644
--- a/sw/source/core/doc/DocumentRedlineManager.cxx
+++ b/sw/source/core/doc/DocumentRedlineManager.cxx
@@ -283,6 +283,12 @@ void UpdateFramesForRemoveDeleteRedline(SwDoc & rDoc, 
SwPaM const& rPam)
 break;
 }
 
+// no nodes can be unmerged by this - skip MakeFrames() etc.
+if (rPam.GetPoint()->nNode == rPam.GetMark()->nNode)
+{
+break; // continue with AppendAllObjs()
+}
+
 // first, call CheckParaRedlineMerge on the first paragraph,
 // to init flag on new merge range (if any) + 1st node post the 
merge
 auto eMode(sw::FrameMode::Existing);


[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/qa sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/qa/extras/uiwriter/data/tdf148868.odt |binary
 sw/qa/extras/uiwriter/uiwriter.cxx   |   19 +++
 sw/source/uibase/inc/wrtsh.hxx   |2 +-
 sw/source/uibase/wrtsh/delete.cxx|4 ++--
 sw/source/uibase/wrtsh/wrtsh1.cxx|5 +++--
 5 files changed, 25 insertions(+), 5 deletions(-)

New commits:
commit 88eedf2043759ff52be6887235bb8ced044d9d66
Author: Michael Stahl 
AuthorDate: Fri Jun 10 18:25:05 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:41:51 2022 +0200

tdf#148868 sw: handle selection then Insert similar to Replace

... if the selection is inside one paragraph, to avoid deleting flys
anchored in the selection.

From a code point of view it's a bit inconsistent to do this, but
from user point of view there are some ways to conveniently create
a selection such as by double clicking a word.

(see also tdf#133957)

Also in SwWrtShell::AutoCorrect(), which oddly enough might get called
with a selection from textsh*.cxx, at least in theory.

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135606
Tested-by: Michael Stahl 
Reviewed-by: Michael Stahl 
(cherry picked from commit d72bee64a97650507d042f17846b6fc427b8434c)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135839
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 
(cherry picked from commit 67eec140556edb42280def88e987448aaa221c5e)

tdf#148868: sw_uiwriter3: Add unittest

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135827
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 
(cherry picked from commit 4e8295638e68295d73b49ddb80e23c3509a49b3e)

Change-Id: I8cf9459e5a7ec7754ce8fe323cd158c7e84a5c93
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136221
Tested-by: Michael Stahl 
Reviewed-by: Michael Stahl 

diff --git a/sw/qa/extras/uiwriter/data/tdf148868.odt 
b/sw/qa/extras/uiwriter/data/tdf148868.odt
new file mode 100644
index ..7ebf68a82ea3
Binary files /dev/null and b/sw/qa/extras/uiwriter/data/tdf148868.odt differ
diff --git a/sw/qa/extras/uiwriter/uiwriter.cxx 
b/sw/qa/extras/uiwriter/uiwriter.cxx
index 5e99211ea450..007a597cf61c 100644
--- a/sw/qa/extras/uiwriter/uiwriter.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter.cxx
@@ -7750,6 +7750,25 @@ void SwUiWriterTest::testTdf38394()
 CPPUNIT_ASSERT_EQUAL(sReplaced, 
static_cast(pDoc->GetNodes()[nIndex])->GetText());
 }
 
+CPPUNIT_TEST_FIXTURE(SwUiWriterTest, testTdf148868)
+{
+SwDoc* pDoc = createDoc("tdf148868.odt");
+SwWrtShell* pWrtShell = pDoc->GetDocShell()->GetWrtShell();
+
+CPPUNIT_ASSERT_EQUAL(1, getShapes());
+CPPUNIT_ASSERT_EQUAL(1, getPages());
+
+pWrtShell->EndPg(/*bSelect=*/false);
+pWrtShell->Left(CRSR_SKIP_CHARS, /*bSelect=*/true, 5, 
/*bBasicCall=*/false);
+pWrtShell->Insert("X");
+
+// Without the fix in place, this test would have failed with
+// - Expected: 1
+// - Actual  : 0
+CPPUNIT_ASSERT_EQUAL(1, getShapes());
+CPPUNIT_ASSERT_EQUAL(1, getPages());
+}
+
 CPPUNIT_TEST_FIXTURE(SwUiWriterTest, testTdf134021)
 {
 load(DATA_DIRECTORY, "tdf134021.docx");
diff --git a/sw/source/uibase/inc/wrtsh.hxx b/sw/source/uibase/inc/wrtsh.hxx
index 049cb3d9ae5c..cf780df77cf0 100644
--- a/sw/source/uibase/inc/wrtsh.hxx
+++ b/sw/source/uibase/inc/wrtsh.hxx
@@ -279,7 +279,7 @@ typedef bool (SwWrtShell:: *FNSimpleMove)();
 boolDelLeft();
 
 // also deletes the frame or sets the cursor in the frame when bDelFrame 
== false
-boolDelRight();
+boolDelRight(bool isReplaceHeuristic = false);
 voidDelToEndOfPara();
 voidDelToStartOfPara();
 boolDelToEndOfSentence();
diff --git a/sw/source/uibase/wrtsh/delete.cxx 
b/sw/source/uibase/wrtsh/delete.cxx
index 5358876d204d..bf165d4d66a2 100644
--- a/sw/source/uibase/wrtsh/delete.cxx
+++ b/sw/source/uibase/wrtsh/delete.cxx
@@ -282,7 +282,7 @@ bool SwWrtShell::DelLeft()
 return bRet;
 }
 
-bool SwWrtShell::DelRight()
+bool SwWrtShell::DelRight(bool const isReplaceHeuristic)
 {
 // Will be or'ed, if a tableselection exists;
 // will here be implemented on SelectionType::Table
@@ -309,7 +309,7 @@ bool SwWrtShell::DelRight()
 {
 SwActContext aActContext(this);
 ResetCursorStack();
-Delete(false);
+Delete(isReplaceHeuristic);
 UpdateAttr();
 }
 if( IsBlockMode() )
diff --git a/sw/source/uibase/wrtsh/wrtsh1.cxx 
b/sw/source/uibase/wrtsh/wrtsh1.cxx
index 0840ed3cd407..53e8ae7a2fbd 100644
--- a/sw/source/uibase/wrtsh/wrtsh1.cxx
+++ b/sw/source/uibase/wrtsh/wrtsh1.cxx
@@ -244,7 +244,8 @@ void SwWrtShell::Insert( const OUString  )
 StartUndo(SwUndoId::REPLACE, );
 bStarted = true;
 Push();
-bDeleted = DelRight();
+// let's interpret a 

[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/qa sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/qa/extras/uiwriter/uiwriter2.cxx |   50 
 sw/source/core/doc/DocumentContentOperationsManager.cxx |9 ++
 sw/source/core/doc/docedt.cxx   |4 -
 sw/source/core/inc/mvsave.hxx   |2 
 4 files changed, 62 insertions(+), 3 deletions(-)

New commits:
commit 0ecea0cbef9400372818668d4afda454ff9b5b6a
Author: Michael Stahl 
AuthorDate: Fri Jun 10 16:26:40 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:41:27 2022 +0200

tdf#139982 sw: preserve flys in Replace with redlining enabled

The problem is that there isn't a redline type "Replace" so it's
represented as Delete+Insert.

To prevent the flys anchored in the text from being deleted, move the
anchors to the point between the old (deleted) and new (inserted) text.

(regression from commit 28b77c89dfcafae82cf2a6d85731b643ff9290e5)

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135604
Tested-by: Michael Stahl 
Reviewed-by: Michael Stahl 
(cherry picked from commit 646c6ddd91a98afddf914e3889cb269fc814c060)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135737
Tested-by: Jenkins
Reviewed-by: Thorsten Behrens 
(cherry picked from commit b7ab83bd96c70932c2223c8d0b3bc0f24327cef2)

Change-Id: Ib600c9dbfb9421917e4b8d61195c48cf0b364f06
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136220
Tested-by: Michael Stahl 
Reviewed-by: Michael Stahl 

diff --git a/sw/qa/extras/uiwriter/uiwriter2.cxx 
b/sw/qa/extras/uiwriter/uiwriter2.cxx
index 81c4c3e31c33..e00e27548fcc 100644
--- a/sw/qa/extras/uiwriter/uiwriter2.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter2.cxx
@@ -908,6 +908,56 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf140007)
  
pDoc->GetNodes()[SwNodeOffset(11)]->GetTextNode()->GetText());
 }
 
+CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf139982)
+{
+SwDoc* const pDoc = createDoc();
+SwWrtShell* const pWrtShell = pDoc->GetDocShell()->GetWrtShell();
+
+// turn on redlining and show changes
+pDoc->getIDocumentRedlineAccess().SetRedlineFlags(RedlineFlags::On | 
RedlineFlags::ShowDelete
+  | 
RedlineFlags::ShowInsert);
+CPPUNIT_ASSERT_MESSAGE("redlining should be on",
+   pDoc->getIDocumentRedlineAccess().IsRedlineOn());
+CPPUNIT_ASSERT_MESSAGE(
+"redlines should be visible",
+
IDocumentRedlineAccess::IsShowChanges(pDoc->getIDocumentRedlineAccess().GetRedlineFlags()));
+
+pWrtShell->Insert("helloo");
+
+pWrtShell->Left(CRSR_SKIP_CHARS, /*bSelect=*/false, 1, 
/*bBasicCall=*/false);
+{
+SwFormatAnchor anchor(RndStdIds::FLY_AT_CHAR);
+anchor.SetAnchor(pWrtShell->GetCursor()->GetPoint());
+SfxItemSet flySet(pDoc->GetAttrPool(), svl::Items{});
+flySet.Put(anchor);
+SwFrameFormat const* pFly = pWrtShell->NewFlyFrame(flySet, 
/*bAnchValid=*/true);
+CPPUNIT_ASSERT(pFly != nullptr);
+}
+
+pWrtShell->SttEndDoc(true);
+pWrtShell->EndPara(/*bSelect=*/true);
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+
+pWrtShell->Replace("hello", true);
+
+// the problem was that a redline delete with the same author as redline
+// insert has its text deleted immediately, including anchored flys.
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+
+pWrtShell->Undo();
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+
+pWrtShell->Redo();
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+
+pWrtShell->Undo();
+
+CPPUNIT_ASSERT_EQUAL(size_t(1), pWrtShell->GetFlyCount(FLYCNTTYPE_FRM));
+}
+
 CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf54819)
 {
 load(DATA_DIRECTORY, "tdf54819.fodt");
diff --git a/sw/source/core/doc/DocumentContentOperationsManager.cxx 
b/sw/source/core/doc/DocumentContentOperationsManager.cxx
index 2db40c2e5d57..b6dc4158b936 100644
--- a/sw/source/core/doc/DocumentContentOperationsManager.cxx
+++ b/sw/source/core/doc/DocumentContentOperationsManager.cxx
@@ -4421,6 +4421,14 @@ bool DocumentContentOperationsManager::ReplaceRangeImpl( 
SwPaM& rPam, const OUSt
 InsertItemSet( aTmpRange, aSet );
 }
 
+// tdf#139982: Appending the redline may immediately delete flys
+// anchored in the previous text if it's inside an insert redline.
+// Also flys will be deleted if the redline is accepted. Move them
+// to the position between the previous text and the new text,
+// there the chance of surviving both accept and reject is best.
+SaveFlyArr flys;
+SaveFlyInRange(aDelPam, *aDelPam.End(), flys, false);
+
 if (m_rDoc.GetIDocumentUndoRedo().DoesUndo())
 {
 

[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/qa sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/qa/extras/uiwriter/uiwriter2.cxx |   88 
 sw/source/core/undo/unins.cxx   |5 +-
 2 files changed, 91 insertions(+), 2 deletions(-)

New commits:
commit 4f3677eff9d177489e836490d0deaf580c9e8458
Author: Michael Stahl 
AuthorDate: Thu Jun 9 18:58:06 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:40:53 2022 +0200

tdf#140007 sw: fix SwUndoReplace

(regression from commit d6b0e84b236b78f4b21bd16e46dda3fa0876096d)

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135585
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 45613274794636ba98d0e978fe872511297d275d)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135549
Reviewed-by: Xisco Fauli 
(cherry picked from commit 7df50ecd9dea623058dc7bf9095fd13d9bb49860)

Change-Id: I1facf1584a349d1d087438f4e6fd3a63a80c6f7e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136219
Tested-by: Thorsten Behrens 
Reviewed-by: Thorsten Behrens 

diff --git a/sw/qa/extras/uiwriter/uiwriter2.cxx 
b/sw/qa/extras/uiwriter/uiwriter2.cxx
index bdea12f0370a..81c4c3e31c33 100644
--- a/sw/qa/extras/uiwriter/uiwriter2.cxx
+++ b/sw/qa/extras/uiwriter/uiwriter2.cxx
@@ -820,6 +820,94 @@ CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf131912)
 CPPUNIT_ASSERT_EQUAL(OUString("foo"), pWrtShell->GetCursor()->GetText());
 }
 
+CPPUNIT_TEST_FIXTURE(SwUiWriterTest2, testTdf140007)
+{
+typedef sal_uLong SwNodeOffset;
+SwDoc* const pDoc = createDoc();
+SwWrtShell* const pWrtShell = pDoc->GetDocShell()->GetWrtShell();
+
+pWrtShell->Insert("foo");
+pWrtShell->SplitNode();
+pWrtShell->Insert("bar");
+pWrtShell->SplitNode();
+pWrtShell->Insert("baz");
+CPPUNIT_ASSERT_EQUAL(sal_uLong(13), pDoc->GetNodes().Count());
+CPPUNIT_ASSERT_EQUAL(OUString("foo"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("bar"),
+ 
pDoc->GetNodes()[SwNodeOffset(10)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(11)]->GetTextNode()->GetText());
+
+pWrtShell->SttEndDoc(true);
+pWrtShell->EndPara(false);
+pWrtShell->Right(CRSR_SKIP_CHARS, /*bSelect=*/true, 1, 
/*bBasicCall=*/false);
+pWrtShell->Replace(" ", true);
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(12), pDoc->GetNodes().Count());
+CPPUNIT_ASSERT_EQUAL(OUString("foo bar"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(10)]->GetTextNode()->GetText());
+
+pWrtShell->SttEndDoc(true);
+pWrtShell->EndPara(false);
+pWrtShell->Right(CRSR_SKIP_CHARS, /*bSelect=*/true, 1, 
/*bBasicCall=*/false);
+pWrtShell->Replace(" ", true);
+CPPUNIT_ASSERT_EQUAL(OUString("foo bar baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(11), pDoc->GetNodes().Count());
+
+pWrtShell->Undo();
+
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(12), pDoc->GetNodes().Count());
+CPPUNIT_ASSERT_EQUAL(OUString("foo bar"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(10)]->GetTextNode()->GetText());
+
+pWrtShell->Undo();
+
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(13), pDoc->GetNodes().Count());
+CPPUNIT_ASSERT_EQUAL(OUString("foo"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("bar"),
+ 
pDoc->GetNodes()[SwNodeOffset(10)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(11)]->GetTextNode()->GetText());
+
+pWrtShell->Redo();
+
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(12), pDoc->GetNodes().Count());
+CPPUNIT_ASSERT_EQUAL(OUString("foo bar"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(10)]->GetTextNode()->GetText());
+
+pWrtShell->Redo();
+
+CPPUNIT_ASSERT_EQUAL(OUString("foo bar baz"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(11), pDoc->GetNodes().Count());
+
+pWrtShell->Undo();
+
+CPPUNIT_ASSERT_EQUAL(SwNodeOffset(12), pDoc->GetNodes().Count());
+CPPUNIT_ASSERT_EQUAL(OUString("foo bar"),
+ 
pDoc->GetNodes()[SwNodeOffset(9)]->GetTextNode()->GetText());
+CPPUNIT_ASSERT_EQUAL(OUString("baz"),
+ 

[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/source/core/undo/undobj.cxx |   12 
 1 file changed, 8 insertions(+), 4 deletions(-)

New commits:
commit c6a2033221c86166a2226023f34314eb59380fa2
Author: Michael Stahl 
AuthorDate: Wed Jun 8 17:34:32 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:40:25 2022 +0200

(related: tdf#139514) sw: fix Undo of delete with at-para fly

Nonobviously, there are situations where the anchor node must be
preserved and restored when it's not on the node that is being deleted.

(probably regression from commit 91b2325808a75174f284c48c8b8afc118fad74e4)

Change-Id: I39f09ddb631204c8ad522f9ec7068d235ca94ad2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135509
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 12acdce71dd6b6af2c52ba8fa3248d3166418543)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135518
Reviewed-by: Miklos Vajna 
(cherry picked from commit 054c248127c78521b4a4e7aacd8936bd54259996)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136218
Tested-by: Michael Stahl 

diff --git a/sw/source/core/undo/undobj.cxx b/sw/source/core/undo/undobj.cxx
index 3fe7f107f052..d611cb4a496b 100644
--- a/sw/source/core/undo/undobj.cxx
+++ b/sw/source/core/undo/undobj.cxx
@@ -981,10 +981,14 @@ void SwUndoSaveContent::DelContentIndex( const 
SwPosition& rMark,
 // Moving the anchor?
 else if 
(!((DelContentType::CheckNoCntnt|DelContentType::ExcludeFlyAtStartEnd)
 & nDelContentType) &&
-// at least for calls from SwUndoDelete,
-// this should work - other Undos don't
-// remember the order of the cursor
-(rPoint.nNode.GetIndex() == 
pAPos->nNode.GetIndex())
+// for SwUndoDelete: rPoint is the node that
+// will be Joined - so anchor should be moved
+// off it - but UndoImpl() split will insert
+// new node *before* existing one so a no-op
+// may need to be done here to add it to
+// history for Undo.
+(rPoint.nNode.GetIndex() == 
pAPos->nNode.GetIndex()
+ || pStt->nNode.GetIndex() == 
pAPos->nNode.GetIndex())
 // Do not try to move the anchor to a table!
 && rMark.nNode.GetNode().IsTextNode())
 {


[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/inc sw/qa sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/inc/IDocumentContentOperations.hxx   |   11 +++
 sw/inc/editsh.hxx   |4 -
 sw/qa/core/uwriter.cxx  |2 
 sw/source/core/doc/DocumentContentOperationsManager.cxx |   46 
 sw/source/core/docnode/ndsect.cxx   |2 
 sw/source/core/docnode/ndtbl.cxx|2 
 sw/source/core/edit/autofmt.cxx |4 -
 sw/source/core/edit/eddel.cxx   |9 +--
 sw/source/core/edit/edglbldc.cxx|2 
 sw/source/core/edit/editsh.cxx  |4 -
 sw/source/core/frmedt/fecopy.cxx|2 
 sw/source/core/inc/DocumentContentOperationsManager.hxx |9 +--
 sw/source/core/inc/UndoDelete.hxx   |3 +
 sw/source/core/layout/atrfrm.cxx|2 
 sw/source/core/undo/undel.cxx   |   21 +--
 sw/source/core/undo/unins.cxx   |2 
 sw/source/core/undo/unredln.cxx |4 -
 sw/source/core/undo/untbl.cxx   |   14 ++--
 sw/source/uibase/dochdl/swdtflvr.cxx|2 
 sw/source/uibase/lingu/hhcwrp.cxx   |4 -
 sw/source/uibase/ribbar/inputwin.cxx|4 -
 sw/source/uibase/wrtsh/delete.cxx   |   26 -
 sw/source/uibase/wrtsh/select.cxx   |4 -
 23 files changed, 109 insertions(+), 74 deletions(-)

New commits:
commit 6bd76edc7ffd82690068618e5b322787228bb5d2
Author: Michael Stahl 
AuthorDate: Tue Jun 7 19:01:24 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:40:02 2022 +0200

tdf#133957 sw: don't delete flys on Backspace/Delete keys

Also fixes: tdf#134007 tdf#138835 tdf#139514

When a character is deleted via the keyboard by Backspace or Delete key,
an artificial selection is created in SwWrtShell::DelLeft()/DelRight().

Ideally this should not delete flys that may be anchored to the
paragraphs, but unfortunately this may happen if there are only 2 empty
paragraphs in the section, because then the artificial selection cannot
be distinguished by the SwDoc implementation from a selection from
Ctrl+A (Select All), which *should* delete the flys.

So introduce a new flag that needs to be passed down multiple layers so
that SwUndoDelete can use it to determine if flys should be deleted, and
translating it to a flag that had been introduced to preserve flys in
ReplaceRange() previously.

There are a couple more callers that look like they want to "replace"
some text, so guess a bit at where to set this new flag.

(note: of course fly anchored *as char* must be deleted via keys.)

(regression from commit e75dd1fc992f168f24d66595265a978071cdd277)

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135476
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit 85376a02348810812d515ee72140dbf56f2b6040)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135517
Reviewed-by: Miklos Vajna 
(cherry picked from commit 5192cd430e8cab0ed04f8c70c5194397455ac705)

Change-Id: Ib4467476b12a12aefbbcb74ab9802f9318cf9aa0
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136217
Tested-by: Michael Stahl 
Reviewed-by: Michael Stahl 

diff --git a/sw/inc/IDocumentContentOperations.hxx 
b/sw/inc/IDocumentContentOperations.hxx
index b6857c346a33..6e6b3c1bad3e 100644
--- a/sw/inc/IDocumentContentOperations.hxx
+++ b/sw/inc/IDocumentContentOperations.hxx
@@ -69,6 +69,16 @@ namespace o3tl
 template<> struct typed_flags : 
is_typed_flags {};
 }
 
+enum class SwDeleteFlags
+{
+Default = 0,
+ArtificialSelection = (1<<0), ///< keyboard delete, artificial selection, 
avoid deleting flys
+};
+namespace o3tl
+{
+template<> struct typed_flags : 
is_typed_flags {};
+}
+
 /** Text operation/manipulation interface
 */
 class IDocumentContentOperations
@@ -130,6 +140,7 @@ public:
 Needed for hiding of deletion redlines
 */
 virtual bool DeleteAndJoin( SwPaM&,
+SwDeleteFlags flags = SwDeleteFlags::Default,
 const bool bForceJoinNext = false ) = 0;
 
 virtual bool MoveRange(SwPaM&, SwPosition&, SwMoveFlags) = 0;
diff --git a/sw/inc/editsh.hxx b/sw/inc/editsh.hxx
index c2224d0575bc..71bf75a3a8b6 100644
--- a/sw/inc/editsh.hxx
+++ b/sw/inc/editsh.hxx
@@ -152,7 +152,7 @@ class SW_DLLPUBLIC SwEditShell : public SwCursorShell
  that will be used by GetGraphic() and GetGraphicSize(). */
 SAL_DLLPRIVATE SwGrfNode *GetGrfNode_() const ;
 
-SAL_DLLPRIVATE void DeleteSel( SwPaM& rPam, bool* pUndo = nullptr );
+SAL_DLLPRIVATE void DeleteSel(SwPaM& rPam, bool isArtificialSelection, 
bool* pUndo = nullptr);
 
 

[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-6-4+backports' - sw/source

2022-06-21 Thread Michael Stahl (via logerrit)
 sw/source/core/doc/docnew.cxx |   27 ++-
 1 file changed, 14 insertions(+), 13 deletions(-)

New commits:
commit 97a8db45e6ad8fb397edf5e239384637353f0b2f
Author: Michael Stahl 
AuthorDate: Wed Jun 1 20:30:50 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:39:46 2022 +0200

tdf#148309 sw_redlinehide: fix mail merge performance regression

The problem is that in the call to pTargetShell->EndAllAction(), the
cursor is on the node before the newly appended one that has the page
break, and only this node is formatted (via GetFormatted()), so no new
page frames are inserted in the layout, which then creates massive
performance problems later, particularly since the bugdoc contains
multiple at-page anchored flys.

(regression from commit 42448f48bb48a13d6618a181b12840db6d85c574)

Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135356
Tested-by: Michael Stahl 
Reviewed-by: Michael Stahl 
(cherry picked from commit ff525d0d70ea9d189a430bde944b56d048b03e55)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135273
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 
(cherry picked from commit 57cd4735a7312174e63d2a1a3dd3831443169530)

Revert "trac#34262 Revert "tdf#144565 sw_redlinehide: fix mailmerge when 
flys anchored at last node""

This reverts commit 228537cebb6ffa75e7e2270fb7bfd6e91fa6ba2c.
The performance regression should be fixed with above commit.

Change-Id: I05cd2a515a7f67132ab1c8c6fa0d675252ea3a15
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136181
Reviewed-by: Thorsten Behrens 
Tested-by: Thorsten Behrens 

diff --git a/sw/source/core/doc/docnew.cxx b/sw/source/core/doc/docnew.cxx
index 984f2335bc45..6b041fa96c51 100644
--- a/sw/source/core/doc/docnew.cxx
+++ b/sw/source/core/doc/docnew.cxx
@@ -1054,19 +1054,19 @@ SwNodeIndex SwDoc::AppendDoc(const SwDoc& rSource, 
sal_uInt16 const nStartPageNu
 {
 SwNodeIndex aBreakIdx( GetNodes().GetEndOfContent(), -1 );
 SwPosition aBreakPos( aBreakIdx );
-// InsertPageBreak just works on SwTextNode nodes, so make
-// sure the last node is one!
-bool bIsTextNode = aBreakIdx.GetNode().IsTextNode();
-if ( !bIsTextNode )
-getIDocumentContentOperations().AppendTextNode( aBreakPos );
-const OUString name = pTargetPageDesc->GetName();
-pTargetShell->InsertPageBreak( , nStartPageNumber );
-if ( !bIsTextNode )
-{
-pTargetShell->SttEndDoc( false );
---aBreakIdx;
-GetNodes().Delete( aBreakIdx );
-}
+// insert new node - will be removed at the end...
+// (don't SplitNode() as it may move flys to the wrong node)
+getIDocumentContentOperations().AppendTextNode(aBreakPos);
+SwFormatPageDesc pageDesc(pTargetPageDesc);
+pageDesc.SetNumOffset(nStartPageNumber);
+// set break on the last paragraph
+getIDocumentContentOperations().InsertPoolItem(SwPaM(aBreakPos),
+pageDesc, SetAttrMode::DEFAULT, pTargetShell->GetLayout());
+// tdf#148309 move to the last node - so that the "flush page 
break"
+// code below will format the frame of the node with the page 
break,
+// which is required for new page frames to be created!  Else 
layout
+// performance will be terrible.
+pTargetShell->SttEndDoc(false);
 
 // There is now a new empty text node on the new page. If it has
 // any marks, those are from the previous page: move them back
@@ -1097,6 +1097,7 @@ SwNodeIndex SwDoc::AppendDoc(const SwDoc& rSource, 
sal_uInt16 const nStartPageNu
 if ( !bDeletePrevious )
 {
 SAL_INFO( "sw.pageframe", "(Flush pagebreak AKA EndAllAction" 
);
+
assert(pTargetShell->GetCursor()->GetPoint()->nNode.GetNode().GetTextNode()->GetSwAttrSet().HasItem(RES_PAGEDESC));
 pTargetShell->EndAllAction();
 SAL_INFO( "sw.pageframe",  "Flush changes AKA EndAllAction)" );
 pTargetShell->StartAllAction();


[Libreoffice-commits] core.git: vcl/source

2022-06-21 Thread Andrea Gelmini (via logerrit)
 vcl/source/app/salvtables.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit cbe00c2eea06bc59c4bf3b36fbb5f2d1d2d87ef4
Author: Andrea Gelmini 
AuthorDate: Tue Jun 21 21:02:34 2022 +0200
Commit: Julien Nabet 
CommitDate: Tue Jun 21 23:36:48 2022 +0200

Fix typo

Change-Id: I116e7fe8ecc5d9440e537613a55b880512a1706d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136260
Tested-by: Julien Nabet 
Reviewed-by: Julien Nabet 

diff --git a/vcl/source/app/salvtables.cxx b/vcl/source/app/salvtables.cxx
index 0cca56bcf5b9..6cd499a1dcc7 100644
--- a/vcl/source/app/salvtables.cxx
+++ b/vcl/source/app/salvtables.cxx
@@ -114,7 +114,7 @@ void SalFrame::SetRepresentedURL(const OUString&)
 OUString SalFrame::DumpSetPosSize(tools::Long nX, tools::Long nY, tools::Long 
nWidth,
   tools::Long nHeight, sal_uInt16 nFlags)
 {
-// assuming the 4 integers normally don't have more then 4 digits, but 
might be negative
+// assuming the 4 integers normally don't have more than 4 digits, but 
might be negative
 OUStringBuffer aBuffer(4 * 5 + 5);
 if (nFlags & SAL_FRAME_POSSIZE_WIDTH)
 aBuffer << nWidth << "x";


[Libreoffice-commits] core.git: Branch 'libreoffice-7-3' - basctl/source vcl/source

2022-06-21 Thread Samuel Mehrbrodt (via logerrit)
 basctl/source/basicide/iderdll.cxx  |1 +
 basctl/source/basicide/macrodlg.cxx |7 +++
 vcl/source/app/salvtables.cxx   |2 ++
 vcl/source/treelist/treelist.cxx|6 ++
 4 files changed, 16 insertions(+)

New commits:
commit f894fa135bea1b33c51fd94e2138a7c9b16b2776
Author: Samuel Mehrbrodt 
AuthorDate: Mon Jun 20 10:27:26 2022 +0200
Commit: Thorsten Behrens 
CommitDate: Tue Jun 21 23:32:20 2022 +0200

Fix crash when no valid EntryDescriptor found

When opening macro run dlg, the last selected entry is displayed again.
When no entry was found, a crash occured in some situations
(GetLastEntryDescriptor() returned garbage).

Initialize m_aLastEntryDesc properly, and make sure the method returns
when no last selected macro was found.

Also fix some nullptr crashes which occurred during UITests

Change-Id: I7bd1a0b8824725f9935876ae26d8222410a3bc25
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136140
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt 
(cherry picked from commit 499ecbf3a36990c29dc7e1fb9b0ecb1d297c2848)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136170
Reviewed-by: Xisco Fauli 
(cherry picked from commit 2545a7700f8a4872fd18cb8b1fffeaa4599136d9)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136236
Reviewed-by: Thorsten Behrens 

diff --git a/basctl/source/basicide/iderdll.cxx 
b/basctl/source/basicide/iderdll.cxx
index 022045050e9a..8e8a3dc3e114 100644
--- a/basctl/source/basicide/iderdll.cxx
+++ b/basctl/source/basicide/iderdll.cxx
@@ -145,6 +145,7 @@ ExtraData* Dll::GetExtraData ()
 
 
 ExtraData::ExtraData () :
+m_aLastEntryDesc(EntryDescriptor()),
 bChoosingMacro(false),
 bShellInCriticalSection(false)
 {
diff --git a/basctl/source/basicide/macrodlg.cxx 
b/basctl/source/basicide/macrodlg.cxx
index 511abf83591e..b0abae8998c0 100644
--- a/basctl/source/basicide/macrodlg.cxx
+++ b/basctl/source/basicide/macrodlg.cxx
@@ -153,6 +153,13 @@ void MacroChooser::RestoreMacroDescription()
 aDesc = pData->GetLastEntryDescriptor();
 }
 
+// No valid EntryDescriptor found
+if (aDesc.GetMethodName().isEmpty())
+{
+m_xMacroNameEdit->select_region(0, 0);
+return;
+}
+
 m_xBasicBox->SetCurrentEntry(aDesc);
 BasicSelectHdl(m_xBasicBox->get_widget());
 
diff --git a/vcl/source/app/salvtables.cxx b/vcl/source/app/salvtables.cxx
index 20e8db2e7726..6875b9fb9b45 100644
--- a/vcl/source/app/salvtables.cxx
+++ b/vcl/source/app/salvtables.cxx
@@ -4638,6 +4638,8 @@ void SalInstanceTreeView::set_text(const weld::TreeIter& 
rIter, const OUString&
 OUString SalInstanceTreeView::get_id(const weld::TreeIter& rIter) const
 {
 const SalInstanceTreeIter& rVclIter = static_cast(rIter);
+if (!rVclIter.iter)
+return OUString();
 const OUString* pStr = static_cast(rVclIter.iter->GetUserData());
 if (pStr)
 return *pStr;
diff --git a/vcl/source/treelist/treelist.cxx b/vcl/source/treelist/treelist.cxx
index 54ebcb075e92..a876b0675980 100644
--- a/vcl/source/treelist/treelist.cxx
+++ b/vcl/source/treelist/treelist.cxx
@@ -101,6 +101,8 @@ bool SvTreeList::IsEntryVisible( const SvListView* pView, 
SvTreeListEntry* pEntr
 
 sal_uInt16 SvTreeList::GetDepth( const SvTreeListEntry* pEntry ) const
 {
+if (!pEntry)
+return 0;
 DBG_ASSERT(pEntry && pEntry!=pRootItem.get(),"GetDepth:Bad Entry");
 sal_uInt16 nDepth = 0;
 while( pEntry && pEntry->pParent != pRootItem.get() )
@@ -1505,6 +1507,8 @@ SvTreeListEntries& SvTreeList::GetChildList( 
SvTreeListEntry* pParent )
 
 const SvTreeListEntry* SvTreeList::GetParent( const SvTreeListEntry* pEntry ) 
const
 {
+if (!pEntry)
+return nullptr;
 const SvTreeListEntry* pParent = pEntry->pParent;
 if (pParent == pRootItem.get())
 pParent = nullptr;
@@ -1513,6 +1517,8 @@ const SvTreeListEntry* SvTreeList::GetParent( const 
SvTreeListEntry* pEntry ) co
 
 SvTreeListEntry* SvTreeList::GetParent( SvTreeListEntry* pEntry )
 {
+if (!pEntry)
+return nullptr;
 SvTreeListEntry* pParent = pEntry->pParent;
 if (pParent == pRootItem.get())
 pParent = nullptr;


[Libreoffice-bugs] [Bug 149665] Cannot enter a single quotation mark followed by on or more digits into a cell via keyboard

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149665

OfficeUser  changed:

   What|Removed |Added

Summary|Cannot enter a single   |Cannot enter a single
   |quotation mark followed by  |quotation mark followed by
   |on or more digits into a|on or more digits into a
   |cell|cell via keyboard

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: sw/source

2022-06-21 Thread Stephan Bergmann (via logerrit)
 sw/source/core/unocore/unosett.cxx |   16 
 1 file changed, 8 insertions(+), 8 deletions(-)

New commits:
commit 62e2288a9b0b7cef37af432f2f6671c0fded51f5
Author: Stephan Bergmann 
AuthorDate: Tue Jun 21 21:58:07 2022 +0200
Commit: Stephan Bergmann 
CommitDate: Tue Jun 21 23:07:17 2022 +0200

Use proper type for aSvxToUnoAdjust

...following up on a comment in the commit message of
37ec4442d70339dc8ec5fb8e4ec8984420b6e14d "o3tl: ensure that the initializer 
of
enumarray contains enough elements":  "In sw/source/core/unocore/unosett.cxx
with its odd mix of saL_Int16 and USHRT_MAX, lets keep things that way for 
now
(probably awaiting later clean up) and use casts to avoid the implicit
narrowing."  The USHRT_MAX values always mapped to -1 when the elements of
aSvxToUnoAdjust are used to initialize sal_Int16 nINT16 later on, so make 
that
explicit here (but an explicit cast from int to sal_Int16 is still needed 
for
them, to avoid narrowing failures from within the enumarray ctor).

Change-Id: Ib401ff94c429f8415199a5d35997d17266e04279
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136262
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 

diff --git a/sw/source/core/unocore/unosett.cxx 
b/sw/source/core/unocore/unosett.cxx
index e06e5fe42e50..a871d4f1c844 100644
--- a/sw/source/core/unocore/unosett.cxx
+++ b/sw/source/core/unocore/unosett.cxx
@@ -240,14 +240,14 @@ static SwPageDesc* lcl_GetPageDesc(SwDoc* pDoc, const 
uno::Any& aValue)
 }
 
 // Numbering
-const o3tl::enumarray aSvxToUnoAdjust
-{
-static_cast(text::HoriOrientation::LEFT),   //3
-static_cast(text::HoriOrientation::RIGHT),  //1
-static_cast(USHRT_MAX),
-static_cast(text::HoriOrientation::CENTER), //2
-static_cast(USHRT_MAX),
-static_cast(USHRT_MAX)
+const o3tl::enumarray aSvxToUnoAdjust
+{
+text::HoriOrientation::LEFT,   //3
+text::HoriOrientation::RIGHT,  //1
+sal_Int16(-1),
+text::HoriOrientation::CENTER, //2
+sal_Int16(-1),
+sal_Int16(-1)
 };
 
 const unsigned short aUnoToSvxAdjust[] =


[Libreoffice-bugs] [Bug 149665] New: Cannot enter a single quotation mark followed by on or more digits into a cell

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149665

Bug ID: 149665
   Summary: Cannot enter a single quotation mark followed by on or
more digits into a cell
   Product: LibreOffice
   Version: 7.3.4.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: norbert.n...@gmx.de

It is impossible to enter for example "'1" as cell content via keyboard.

Steps to reproduce:

- Entering "''1" leads to "''1" (Note: Do not enter the double quotation
marks!"

Expected result:
Entering "''1" should lead to a cell content of "'1". If a digit as one or more
leading single quotation marks, the first should single quotation mark should
always be truncated. This is how Excel and Only Office behave.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149664] New: Cannot enter a single quotation mark followed by on or more digits into a cell

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149664

Bug ID: 149664
   Summary: Cannot enter a single quotation mark followed by on or
more digits into a cell
   Product: LibreOffice
   Version: 7.3.4.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: norbert.n...@gmx.de

It is impossible to enter for example "'1" as cell content via keyboard.

Steps to reproduce:

- Entering "''1" leads to "''1" (Note: Do not enter the double quotation
marks!"

Expected result:
Entering "''1" should lead to a cell content of "'1". If a digit as one or more
leading single quotation marks, the first should single quotation mark should
always be truncated. This is how Excel and Only Office behave.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149647] LibreOffice Calc cursor not positioned correctly

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149647

Mike Kaganski  changed:

   What|Removed |Added

 OS|Windows (All)   |All
 CC||noelgran...@gmail.com
   Keywords||bibisected, bisected

--- Comment #9 from Mike Kaganski  ---
Regression after commit feec8e3c34e08b621098a17f1011dccd0b4f7f4c
  Author Noel Grandin 
  Date   Tue Nov 12 16:00:55 2019 +0200
reduce iteration in ScViewData::GetScrPos

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149432] system font in Hebrew is not a standard font

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149432

--- Comment #14 from Eyal Rozenberg  ---
Also note that the font used in the user interface is not the font you choose
in the "Basic Fonts" dialog. That dialog is for the fonts used in the document,
not in the UI.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149432] system font in Hebrew is not a standard font

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149432

--- Comment #13 from Eyal Rozenberg  ---
(In reply to Dieter from comment #12)
> Eyal, could you help with this report? Can you confirm the problem?

No, I cannot confirm this. And - many people would have complained about this
had it been a widespread occurrence. However, to be fair, I've only tried this
in Windows 10. It is theoretically possible, though unlikely, that the bug
manifests with Windows 7 ("6.1"). I'll ask someone with a Win 7 machine on the
Hebrew channel to try it.

Shlomo, have you seen this only on a single computer, or on my than one system?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149647] LibreOffice Calc cursor not positioned correctly

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149647

Mike Kaganski  changed:

   What|Removed |Added

   Keywords|possibleRegression  |regression
Version|7.3.3.2 release |7.0.0.3 release

--- Comment #8 from Mike Kaganski  ---
No repro with Version: Version: 6.4.0.3 (x64)
Build ID: b0a288ab3d2d4774cb44b62f04d5d28733ac6df8
CPU threads: 12; OS: Windows 10.0 Build 19044; UI render: GL; VCL: win; 
Locale: ru-RU (ru_RU); UI-Language: en-US
Calc: threaded

Repro using Version: 7.0.0.3 (x64)
Build ID: 8061b3e9204bef6b321a21033174034a5e2ea88e
CPU threads: 12; OS: Windows 10.0 Build 19044; UI render: default; VCL: win
Locale: en-US (ru_RU); UI: en-US
Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 133557] Rejecting track & changes in Calc slow

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=133557

Julien Nabet  changed:

   What|Removed |Added

 CC||nem...@numbertext.org

--- Comment #5 from Julien Nabet  ---
László: I noticed patches from you but on sw part. Do you also know sc part or
do you know whom to ping?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 112812] [META] Hebrew language-specific RTL issues

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112812

Eyal Rozenberg  changed:

   What|Removed |Added

 Depends on||149432


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=149432
[Bug 149432] system font in Hebrew is not a standard font
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 129661] [META] Right-To-Left (RTL) user interface issues

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=129661

Eyal Rozenberg  changed:

   What|Removed |Added

 Depends on||149432


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=149432
[Bug 149432] system font in Hebrew is not a standard font
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149432] system font in Hebrew is not a standard font

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149432

Eyal Rozenberg  changed:

   What|Removed |Added

 Blocks||129661, 112812


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=112812
[Bug 112812] [META] Hebrew language-specific RTL issues
https://bugs.documentfoundation.org/show_bug.cgi?id=129661
[Bug 129661] [META] Right-To-Left (RTL) user interface issues
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 133557] Rejecting track & changes in Calc slow

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=133557

--- Comment #4 from Julien Nabet  ---
Created attachment 180883
  --> https://bugs.documentfoundation.org/attachment.cgi?id=180883=edit
Flamegraph

Here's a Flamegraph retrieved on pc Debian x86-64 with master sources updated
today.
I started the trace when choosing "Reject all".

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149648] Background of cells are shown in black when there is no value

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149648

raal  changed:

   What|Removed |Added

   Keywords|possibleRegression  |bibisected, bisected,
   ||regression
 OS|Windows (All)   |All
 CC||r...@post.cz
  Regression By||Vasily Melenchuk

--- Comment #2 from raal  ---
This seems to have begun at the below commit.
Adding Cc: to Vasily Melenchuk; Could you possibly take a look at this one?
Thanks

3323df1e6e342d468d9c45c3afe8f01b81dc93ff is the first bad commit
commit 3323df1e6e342d468d9c45c3afe8f01b81dc93ff
Author: Jenkins Build User 
Date:   Mon Oct 4 10:03:15 2021 +0200

source sha:eab60b2663596220cc4d33676524adebc8349a89

https://git.libreoffice.org/core/+/eab60b2663596220cc4d33676524adebc8349a89
  tdf#143104 Fix xlsx import/export of color filter colors

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-ux-advise] [Bug 144991] EDITING: The "Increase Indent" and "Decrease Indent" increment step should be the width of 3 space chars (using the default style font) when working in an xlsx doc

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144991

--- Comment #7 from Mike Kaganski  ---
(In reply to Eyal Rozenberg from comment #6)
> Why the magic number 3

Because that's what ECMA-376 Part 1 "Office Open XML File Formats —
Fundamentals and Markup Language Reference" sect. 18.8.1 "Alignment" tells:

> indent (Indent) An integer value, where an increment of 1 represents 3 spaces.
> Indicates the number of spaces (of the normal style font) of indentation for
> text in a cell. The number of spaces to indent is calculated as following:
> Number of spaces to indent = indent value * 3

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-bugs] [Bug 144991] EDITING: The "Increase Indent" and "Decrease Indent" increment step should be the width of 3 space chars (using the default style font) when working in an xlsx document

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144991

--- Comment #7 from Mike Kaganski  ---
(In reply to Eyal Rozenberg from comment #6)
> Why the magic number 3

Because that's what ECMA-376 Part 1 "Office Open XML File Formats —
Fundamentals and Markup Language Reference" sect. 18.8.1 "Alignment" tells:

> indent (Indent) An integer value, where an increment of 1 represents 3 spaces.
> Indicates the number of spaces (of the normal style font) of indentation for
> text in a cell. The number of spaces to indent is calculated as following:
> Number of spaces to indent = indent value * 3

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149642] Want to be able to set area, line & text transparencies altogether

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149642

--- Comment #2 from Eyal Rozenberg  ---
(In reply to Rafael Lima from comment #1)
> I'm not sure the global transparency you proposed would be ODF compliant.

It may not be ODF-compliant - but I believe it's quite natural for a user to
want to make an entire object partially-transparent, not just one aspect of it.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-ux-advise] [Bug 144991] EDITING: The "Increase Indent" and "Decrease Indent" increment step should be the width of 3 space chars (using the default style font) when working in an xlsx doc

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144991

--- Comment #6 from Eyal Rozenberg  ---
A lay-person's question: Why the magic number 3 ? Is 2 or 4 spaces inherently
bad, or is this actually configured somehow?

-- 
You are receiving this mail because:
You are on the CC list for the bug.

[Libreoffice-bugs] [Bug 144991] EDITING: The "Increase Indent" and "Decrease Indent" increment step should be the width of 3 space chars (using the default style font) when working in an xlsx document

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144991

--- Comment #6 from Eyal Rozenberg  ---
A lay-person's question: Why the magic number 3 ? Is 2 or 4 spaces inherently
bad, or is this actually configured somehow?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149621] PDF documentation files downloaded from motherboard manufacturers require password that doesn't exist.

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149621

--- Comment #7 from Julien Nabet  ---
Sorry, it wasn't for you specifically but for LO devs. But of course, if you
know coding and would like to involve, don't hesitate to contribute! :-)
If interested this link may help:
https://wiki.documentfoundation.org/Development/GetInvolved
LO is mainly (at least 95%) coded in C++.

About PDF specs, you can find it here:
https://opensource.adobe.com/dc-acrobat-sdk-docs/standards/pdfstandards/pdf/PDF32000_2008.pdf
This one is Adobe version, there's also iso version but you must pay to have it
if I read well the forums.
It seems there are some slight differences but not sure it's relevant here.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146940] Libreoffice became increadibly slow on using emoticons

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146940

Rafael Lima  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||0265

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 140265] Emoji selector takes too long to load and respond

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=140265

Rafael Lima  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||6940

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 136922] Emoji selector is slow

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=136922

Rafael Lima  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=14
   ||0265

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 140265] Emoji selector takes too long to load and respond

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=140265

Rafael Lima  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=13
   ||6922

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 140265] Emoji selector takes too long to load and respond

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=140265

--- Comment #8 from Rafael Lima  ---
Still slow in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: cb83063cc0eb4e93bd44bc0cb9b7c4841230cdef
CPU threads: 16; OS: Linux 5.13; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149663] 1377x

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149663

Rafael Lima  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |INVALID

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: io/source

2022-06-21 Thread Chris Sherlock (via logerrit)
 io/source/stm/odata.cxx |3 +--
 1 file changed, 1 insertion(+), 2 deletions(-)

New commits:
commit 2bf296ddd10f0a56bdf799712bac1ccb10ed8b72
Author: Chris Sherlock 
AuthorDate: Mon Jun 13 17:09:08 2022 +1000
Commit: Stephan Bergmann 
CommitDate: Tue Jun 21 21:16:26 2022 +0200

io: drop unnecessary cast

Change-Id: Iaceab1b85e41dbb2102fe829f7abc2f8e3898a52
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/135693
Tested-by: Jenkins
Reviewed-by: Stephan Bergmann 

diff --git a/io/source/stm/odata.cxx b/io/source/stm/odata.cxx
index 754cda450a94..7d39a5835852 100644
--- a/io/source/stm/odata.cxx
+++ b/io/source/stm/odata.cxx
@@ -25,7 +25,6 @@
 #include 
 #include 
 #include 
-#include 
 
 #include 
 #include 
@@ -1094,7 +1093,7 @@ Reference< XPersistObject >  
OObjectInputStream::readObject()
 {
 // grow to the right size
 Reference< XPersistObject > xEmpty;
-m_aPersistVector.insert( m_aPersistVector.end(), 
static_cast(nId - nSize + 1), xEmpty );
+m_aPersistVector.insert( m_aPersistVector.end(), nId - 
nSize + 1, xEmpty );
 }
 
 m_aPersistVector[nId] = xLoadedObj;


[Libreoffice-bugs] [Bug 149660] FILEOPEN DOCX IF field conditions not read

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149660

Cor Nouws  changed:

   What|Removed |Added

   Keywords||filter:docx
 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEW
 CC||c...@nouenoff.nl
Version|7.5.0.0 alpha0+ Master  |Inherited From OOo

--- Comment #2 from Cor Nouws  ---
confirmed in Version: 7.4.0.0.alpha1+ / LibreOffice Community
Build ID: fc2716740bd513154ae4ab5404d9c575678172e0
CPU threads: 8; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: nl-NL (en_US.UTF-8); UI: nl-NL
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 131149] Super/Subscript and Line Spacing options in menu can't be unchecked (with text box objects)

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=131149

--- Comment #5 from Brian  ---
Created attachment 180882
  --> https://bugs.documentfoundation.org/attachment.cgi?id=180882=edit
Example .odf file showing inconsistent behavior of text in drawing objects

Bug is still present in 7.3.4.2

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 147873] LO 7.2.5.2 Calc Eventual Freezing Crash in Mac silicon M1 Mac Book Pro

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=147873

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEEDINFO
 Ever confirmed|0   |1
 CC||79045_79...@mail.ru

--- Comment #1 from Roman Kuznetsov <79045_79...@mail.ru> ---
Your summary and the description say us about different things. Please clarify
what problem do you have in fact

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 148427] EDITING: Calc freezes when attempting to "Paste Special"

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148427

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 CC||79045_79...@mail.ru
 Status|UNCONFIRMED |NEEDINFO
 Ever confirmed|0   |1

--- Comment #4 from Roman Kuznetsov <79045_79...@mail.ru> ---
(In reply to nlo366 from comment #2)
> Second, I do not
> get the same issue with the blank Calc spreadsheet. I get it in my work, and
> admittedly rather large (5.7Mb), file.

So please attach your file here

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 148487] when you try to insert a special character, Libre Office closes

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148487

--- Comment #7 from Roman Kuznetsov <79045_79...@mail.ru> ---
No repro anyway in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ab12274ffda3a1a5a1ff33203acf9de665d4a636
CPU threads: 4; OS: Mac OS X 12.4; UI render: default; VCL: osx
Locale: ru-RU (ru_RU.UTF-8); UI: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149662] Standard filter treating text badly if being composed of digits only

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149662

m.a.riosv  changed:

   What|Removed |Added

 CC||miguelangelrv@libreoffice.o
   ||rg
 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #2 from m.a.riosv  ---
Reproducible
Version: 7.3.4.2 (x64) / LibreOffice Community
Build ID: 728fec16bd5f605073805c3c9e7c4212a0120dc5
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: es-ES (es_ES); UI: en-US Calc: CL
Version: 7.5.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: e4d23c27288b99c3ed3cfa332ff308b31c01f97d
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Vulkan; VCL: win
Locale: es-ES (es_ES); UI: en-US Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149663] 1377x

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149663

josiahluca...@gmail.com changed:

   What|Removed |Added

URL||https://www.devopsdiggers.c
   ||om/1377x-proxy/

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: registry/source

2022-06-21 Thread Chris Sherlock (via logerrit)
 registry/source/keyimpl.cxx  |   57 
 registry/source/reflread.cxx |   53 +
 registry/source/registry.cxx |   42 ---
 registry/source/regkey.cxx   |   76 ---
 4 files changed, 3 insertions(+), 225 deletions(-)

New commits:
commit e02a7a55c23d23dd7afa2627a7e10c70e1cba001
Author: Chris Sherlock 
AuthorDate: Sun Jan 9 06:51:17 2022 +1100
Commit: Stephan Bergmann 
CommitDate: Tue Jun 21 20:22:34 2022 +0200

registry: remove unnecessary comments

Change-Id: I03b49dcf4bb668b0e4967eb4e88639a5a7e36c8e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/128174
Reviewed-by: Stephan Bergmann 
Tested-by: Jenkins

diff --git a/registry/source/keyimpl.cxx b/registry/source/keyimpl.cxx
index abf8e0d1ef12..37efeda82ff1 100644
--- a/registry/source/keyimpl.cxx
+++ b/registry/source/keyimpl.cxx
@@ -34,9 +34,6 @@ using namespace store;
 
 namespace { char const VALUE_PREFIX[] = "$VL_"; }
 
-
-//  ORegKey()
-
 ORegKey::ORegKey(const OUString& keyName, ORegistry* pReg)
 : m_refCount(1)
 , m_name(keyName)
@@ -46,41 +43,26 @@ ORegKey::ORegKey(const OUString& keyName, ORegistry* pReg)
 {
 }
 
-
-//  ~ORegKey()
-
 ORegKey::~ORegKey()
 {
 SAL_WARN_IF(m_refCount != 0, "registry", "registry::ORegKey::dtor(): 
refcount not zero.");
 }
 
-
-//  releaseKey
-
 RegError ORegKey::releaseKey(RegKeyHandle hKey)
 {
 return m_pRegistry->releaseKey(hKey);
 }
 
-
-//  createKey
-
 RegError ORegKey::createKey(std::u16string_view keyName, RegKeyHandle* 
phNewKey)
 {
 return m_pRegistry->createKey(this, keyName, phNewKey);
 }
 
-
-//  openKey
-
 RegError ORegKey::openKey(std::u16string_view keyName, RegKeyHandle* phOpenKey)
 {
 return m_pRegistry->openKey(this, keyName, phOpenKey);
 }
 
-
-//  openSubKeys
-
 RegError ORegKey::openSubKeys(std::u16string_view keyName, RegKeyHandle** 
phOpenSubKeys, sal_uInt32* pnSubKeys)
 {
 RegError _ret = RegError::NO_ERROR;
@@ -139,9 +121,6 @@ RegError ORegKey::openSubKeys(std::u16string_view keyName, 
RegKeyHandle** phOpen
 return RegError::NO_ERROR;
 }
 
-
-//  getKeyNames
-
 RegError ORegKey::getKeyNames(std::u16string_view keyName,
   rtl_uString*** pSubKeyNames,
   sal_uInt32* pnSubKeys)
@@ -196,25 +175,16 @@ RegError ORegKey::getKeyNames(std::u16string_view keyName,
 return RegError::NO_ERROR;
 }
 
-
-//  closeKey
-
 RegError ORegKey::closeKey(RegKeyHandle hKey)
 {
 return m_pRegistry->closeKey(hKey);
 }
 
-
-//  deleteKey
-
 RegError ORegKey::deleteKey(std::u16string_view keyName)
 {
 return m_pRegistry->deleteKey(this, keyName);
 }
 
-
-//  getValueType
-
 RegError ORegKey::getValueInfo(std::u16string_view valueName, RegValueType* 
pValueType, sal_uInt32* pValueSize) const
 {
 OStoreStreamrValue;
@@ -267,9 +237,6 @@ RegError ORegKey::getValueInfo(std::u16string_view 
valueName, RegValueType* pVal
 return RegError::NO_ERROR;
 }
 
-
-//  setValue
-
 RegError ORegKey::setValue(std::u16string_view valueName, RegValueType vType, 
RegValue value, sal_uInt32 vSize)
 {
 OStoreStreamrValue;
@@ -338,9 +305,6 @@ RegError ORegKey::setValue(std::u16string_view valueName, 
RegValueType vType, Re
 return RegError::NO_ERROR;
 }
 
-
-//  setLongListValue
-
 RegError ORegKey::setLongListValue(std::u16string_view valueName, sal_Int32 
const * pValueList, sal_uInt32 len)
 {
 OStoreStreamrValue;
@@ -393,9 +357,6 @@ RegError ORegKey::setLongListValue(std::u16string_view 
valueName, sal_Int32 cons
 return RegError::NO_ERROR;
 }
 
-
-//  setStringListValue
-
 RegError ORegKey::setStringListValue(
 std::u16string_view valueName, char** pValueList, sal_uInt32 len)
 {
@@ -458,9 +419,6 @@ RegError ORegKey::setStringListValue(
 return RegError::NO_ERROR;
 }
 
-
-//  setUnicodeListValue
-
 RegError ORegKey::setUnicodeListValue(std::u16string_view valueName, 
sal_Unicode** pValueList, sal_uInt32 len)
 {
 OStoreStreamrValue;
@@ -522,9 +480,6 @@ RegError ORegKey::setUnicodeListValue(std::u16string_view 
valueName, sal_Unicode
 return RegError::NO_ERROR;
 }
 
-
-//  getValue
-
 RegError ORegKey::getValue(std::u16string_view valueName, RegValue value) const
 {
 OStoreStreamrValue;
@@ -601,9 +556,6 @@ RegError ORegKey::getValue(std::u16string_view valueName, 
RegValue value) const
 return RegError::NO_ERROR;
 }
 
-
-//  getLongListValue
-
 RegError ORegKey::getLongListValue(std::u16string_view valueName, sal_Int32** 
pValueList, sal_uInt32* pLen) const
 {
 OStoreStreamrValue;
@@ -704,9 +656,6 @@ RegError ORegKey::getLongListValue(std::u16string_view 
valueName, sal_Int32** pV
 return RegError::NO_ERROR;
 }
 
-
-//  getStringListValue
-
 RegError ORegKey::getStringListValue(std::u16string_view valueName, char*** 
pValueList, sal_uInt32* pLen) const
 {
 OStoreStreamrValue;
@@ -801,9 +750,6 @@ RegError 

[Libreoffice-bugs] [Bug 149663] New: 1377x

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149663

Bug ID: 149663
   Summary: 1377x
   Product: Document Liberation Project
   Version: unspecified
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: General
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: josiahluca...@gmail.com

1377x Proxy is one of the best entertainment sites that helps users download
all kinds of files like TV shows, software, web series, movies, music, TV
series, etc.
https://www.devopsdiggers.com/1377x-proxy/

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 148600] Wrong encoding in the GUI

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148600

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Whiteboard| QA:needsComment|

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 148600] Wrong encoding in the GUI

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148600

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 CC||79045_79...@mail.ru
 Status|UNCONFIRMED |NEEDINFO
 Ever confirmed|0   |1

--- Comment #1 from Roman Kuznetsov <79045_79...@mail.ru> ---
Please attach some screenshots/video with your problem

(interesting how you differ Cyrillic A and Latin A)

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 131899] incorrect background colour in thumbnails of recent documents in start centre

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=131899

--- Comment #5 from Rafael Lima  ---
Still repro

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ca47989ad60b1414f92be22a1fbf4c1d1a92dd97
CPU threads: 12; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149648] Background of cells are shown in black when there is no value

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149648

Rafael Lima  changed:

   What|Removed |Added

Summary|content changed by calc |Background of cells are
   ||shown in black when there
   ||is no value
 Ever confirmed|0   |1
   Keywords||possibleRegression
 Status|UNCONFIRMED |NEW

--- Comment #1 from Rafael Lima  ---
Confirmed with

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ca47989ad60b1414f92be22a1fbf4c1d1a92dd97
CPU threads: 12; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: CL

In LO the background of these cells is black until a value is entered.

In MSO the background is yellow.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149472] Freezes on selection Writer

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149472

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 CC||79045_79...@mail.ru

--- Comment #1 from Roman Kuznetsov <79045_79...@mail.ru> ---
No repro in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ab12274ffda3a1a5a1ff33203acf9de665d4a636
CPU threads: 4; OS: Mac OS X 12.3.1; UI render: default; VCL: osx
Locale: ru-RU (ru_RU.UTF-8); UI: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149645] insert photo album does not rotate images

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149645

Rafael Lima  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEEDINFO

--- Comment #2 from Rafael Lima  ---
Simon, can you please try again on a clean profile? Or maybe on a different
computer?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149642] Want to be able to set area, line & text transparencies altogether

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149642

--- Comment #1 from Rafael Lima  ---
This is a complicated one because each is a different property.

I'm not sure the global transparency you proposed would be ODF compliant.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149594] Writer crashes on mailmerge when adding salutation to email message

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149594

Roman Kuznetsov <79045_79...@mail.ru> changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
 Ever confirmed|0   |1

--- Comment #3 from Roman Kuznetsov <79045_79...@mail.ru> ---
Confirm the crash in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ab12274ffda3a1a5a1ff33203acf9de665d4a636
CPU threads: 4; OS: Mac OS X 12.3.1; UI render: default; VCL: osx
Locale: ru-RU (ru_RU.UTF-8); UI: en-US
Calc: threaded

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149647] LibreOffice Calc cursor not positioned correctly

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149647

Rafael Lima  changed:

   What|Removed |Added

   Keywords||possibleRegression
Summary|LibreOffice Calc shift of   |LibreOffice Calc cursor not
   |current row |positioned correctly

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149647] LibreOffice Calc shift of current row

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149647

--- Comment #7 from Rafael Lima  ---
Also repro in "gen" and "gtk3".

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149647] LibreOffice Calc shift of current row

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149647

Rafael Lima  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 CC||rafael.palma.l...@gmail.com
 Status|UNCONFIRMED |NEW

--- Comment #6 from Rafael Lima  ---
Confirmed with

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ca47989ad60b1414f92be22a1fbf4c1d1a92dd97
CPU threads: 12; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: CL

This is a mess... Go around row 178 in the sample ODS file and click anywhere
in the sheet and you'll notice the bug.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-commits] core.git: helpcontent2

2022-06-21 Thread Olivier Hallot (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 1dbfc4fdc9710e670e25bff5636e0336efee8a18
Author: Olivier Hallot 
AuthorDate: Tue Jun 21 14:42:27 2022 -0300
Commit: Gerrit Code Review 
CommitDate: Tue Jun 21 19:42:27 2022 +0200

Update git submodules

* Update helpcontent2 from branch 'master'
  to a5a05714cad12c2205389b98557400c4069304ea
  - Fix bad link

Change-Id: I6470492a640355d98a652183d688684874e843b7
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/136257
Tested-by: Jenkins
Reviewed-by: Olivier Hallot 

diff --git a/helpcontent2 b/helpcontent2
index 4ab7e2cf0ecd..a5a05714cad1 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 4ab7e2cf0ecd45c000eb7605c1918265569e52f7
+Subproject commit a5a05714cad12c2205389b98557400c4069304ea


[Libreoffice-commits] help.git: source/text

2022-06-21 Thread Olivier Hallot (via logerrit)
 source/text/shared/01/0601.xhp |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit a5a05714cad12c2205389b98557400c4069304ea
Author: Olivier Hallot 
AuthorDate: Tue Jun 21 13:27:46 2022 -0300
Commit: Olivier Hallot 
CommitDate: Tue Jun 21 19:42:25 2022 +0200

Fix bad link

Change-Id: I6470492a640355d98a652183d688684874e843b7
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/136257
Tested-by: Jenkins
Reviewed-by: Olivier Hallot 

diff --git a/source/text/shared/01/0601.xhp 
b/source/text/shared/01/0601.xhp
index b3656b618..62d5e2b84 100644
--- a/source/text/shared/01/0601.xhp
+++ b/source/text/shared/01/0601.xhp
@@ -94,7 +94,7 @@
   Click to undo the last change in the current sentence. Click again to 
undo the previous change in the same sentence.
   
  
- Spelling 
Options
+ Spelling 
Options
  AutoCorrect 
Commands
  Thesaurus
   


[Libreoffice-commits] core.git: Branch 'libreoffice-7-4' - wizards/source

2022-06-21 Thread Jean-Pierre Ledure (via logerrit)
 wizards/source/sfdocuments/SF_Calc.xba |8 +---
 1 file changed, 5 insertions(+), 3 deletions(-)

New commits:
commit cb62be830fae6e42e94c42813923e635e895eae7
Author: Jean-Pierre Ledure 
AuthorDate: Tue Jun 21 18:29:03 2022 +0200
Commit: Jean-Pierre Ledure 
CommitDate: Tue Jun 21 19:34:04 2022 +0200

ScriptForge - (SF_Calc)FIX wrong sheet name validation

When the name of a sheet contains a "." or a space, when passed
as an argument, the name must be surrounded with sinle quotes
(like in usual Calc formulas).

The validation of the sheet name goes thru the comparison with
the list of existing sheet names in the document.

The comparison compared erroneously the name with quotes and
sheet name without quotes.

Gave an unjustified user error.

Example:
calc.Sheet("'Commits 7.4'")

Cfr. commit on master: https://gerrit.libreoffice.org/c/core/+/136233

Change-Id: I7a47c9dfe1c7b507e99a37d5714046dd6d8e7567
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/136258
Tested-by: Jean-Pierre Ledure 
Reviewed-by: Jean-Pierre Ledure 
Tested-by: Jenkins

diff --git a/wizards/source/sfdocuments/SF_Calc.xba 
b/wizards/source/sfdocuments/SF_Calc.xba
index 4b42163753bb..0b7b88ae8f76 100644
--- a/wizards/source/sfdocuments/SF_Calc.xba
+++ b/wizards/source/sfdocuments/SF_Calc.xba
@@ -,6 +,7 @@ Private Function _ValidateSheet(Optional ByRef 
pvSheetName As Variant _
  DUPLICATESHEETERROR A sheet with 
the given name exists already
 
 Dim vSheets As Variant   List of sheets
+Dim sSheet As String Sheet name without 
single quotes
 Dim lSheet As Long   Index in list 
of sheets
 Dim vTypes As VariantArray of accepted 
variable types
 Dim bValid As BooleanReturn value
@@ -4474,12 +4475,13 @@ Try:
pvSheetName = 
_Component.CurrentController.ActiveSheet.Name
Else
vSheets = _Component.getSheets.getElementNames()
+   sSheet = Replace(pvSheetName, , 
)
If pvNew Then
-   If ScriptForge.SF_Array.Contains(vSheets, 
pvSheetName) Then GoTo CatchDuplicate
+   If ScriptForge.SF_Array.Contains(vSheets, 
sSheet) Then GoTo CatchDuplicate
Else
-   If Not 
ScriptForge.SF_Utils._Validate(pvSheetName, psArgName, V_STRING, vSheets) Then 
GoTo Finally
+   If Not ScriptForge.SF_Utils._Validate(sSheet, 
psArgName, V_STRING, vSheets) Then GoTo Finally
If pvResetSheet Then
-   lSheet = 
ScriptForge.SF_Array.IndexOf(vSheets, pvSheetName, CaseSensitive := False)
+   lSheet = 
ScriptForge.SF_Array.IndexOf(vSheets, sSheet, CaseSensitive := False)
pvSheetName = vSheets(lSheet)
End If
End If


[Libreoffice-bugs] [Bug 99953] Misaligned cell background color when inserting Calc table in Writer document, depending on zoom level

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=99953

--- Comment #19 from Rafael Lima  ---
Still repro in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: ca47989ad60b1414f92be22a1fbf4c1d1a92dd97
CPU threads: 12; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Calc: CL

-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 97423] Top and left borders get thinner when copy/paste table from Calc to Writer

2022-06-21 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=97423

Rafael Lima  changed:

   What|Removed |Added

   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=99
   ||953

-- 
You are receiving this mail because:
You are the assignee for the bug.

  1   2   3   >