[Libreoffice-commits] core.git: chart2/Library_chartcontroller.mk chart2/source chart2/uiconfig chart2/UIConfig_chart2.mk

2022-08-14 Thread Tomaž Vajngerl (via logerrit)
 chart2/Library_chartcontroller.mk|2 
 chart2/UIConfig_chart2.mk|3 
 chart2/source/controller/dialogs/dlg_InsertDataTable.cxx |   61 +++
 chart2/source/controller/dialogs/res_DataTableProperties.cxx |  111 +
 chart2/source/controller/dialogs/tp_DataTable.cxx|   78 ---
 chart2/source/controller/dialogs/tp_DataTable.hxx|   11 
 chart2/source/controller/inc/dlg_InsertDataTable.hxx |   47 ++
 chart2/source/controller/inc/res_DataTableProperties.hxx |   47 ++
 chart2/source/controller/main/ChartController_Insert.cxx |   64 ++-
 chart2/uiconfig/ui/dlg_InsertDataTable.ui|  215 +++
 10 files changed, 548 insertions(+), 91 deletions(-)

New commits:
commit 6c00d75738bb182b5c02848b1720e6623baed4b9
Author: Tomaž Vajngerl 
AuthorDate: Fri Jul 1 22:00:38 2022 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 15 07:42:32 2022 +0200

chart2: InsertDataTableDialog for inserting/removing the data table

Change-Id: Ie3c033c587b150723e7aa39cd5ddf5774104db9a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138257
Tested-by: Jenkins
Reviewed-by: Tomaž Vajngerl 

diff --git a/chart2/Library_chartcontroller.mk 
b/chart2/Library_chartcontroller.mk
index 4bc8c7497915..cc25cdd907ae 100644
--- a/chart2/Library_chartcontroller.mk
+++ b/chart2/Library_chartcontroller.mk
@@ -109,6 +109,7 @@ $(eval $(call 
gb_Library_add_exception_objects,chartcontroller,\
 chart2/source/controller/dialogs/dlg_DataSource \
 chart2/source/controller/dialogs/dlg_InsertAxis_Grid \
 chart2/source/controller/dialogs/dlg_InsertDataLabel \
+chart2/source/controller/dialogs/dlg_InsertDataTable \
 chart2/source/controller/dialogs/dlg_InsertErrorBars \
 chart2/source/controller/dialogs/dlg_InsertLegend \
 chart2/source/controller/dialogs/dlg_InsertTitle \
@@ -122,6 +123,7 @@ $(eval $(call 
gb_Library_add_exception_objects,chartcontroller,\
 chart2/source/controller/dialogs/RangeSelectionListener \
 chart2/source/controller/dialogs/res_BarGeometry \
 chart2/source/controller/dialogs/res_DataLabel \
+chart2/source/controller/dialogs/res_DataTableProperties \
 chart2/source/controller/dialogs/res_ErrorBar \
 chart2/source/controller/dialogs/res_LegendPosition \
 chart2/source/controller/dialogs/res_Titles \
diff --git a/chart2/UIConfig_chart2.mk b/chart2/UIConfig_chart2.mk
index 59af510c5845..274efd02638f 100644
--- a/chart2/UIConfig_chart2.mk
+++ b/chart2/UIConfig_chart2.mk
@@ -44,7 +44,9 @@ $(eval $(call gb_UIConfig_add_uifiles,modules/schart,\
chart2/uiconfig/ui/combobox \
chart2/uiconfig/ui/datarangedialog \
chart2/uiconfig/ui/dlg_DataLabel \
+   chart2/uiconfig/ui/dlg_InsertDataTable \
chart2/uiconfig/ui/dlg_InsertErrorBars \
+   chart2/uiconfig/ui/dlg_InsertLegend \
chart2/uiconfig/ui/imagefragment \
chart2/uiconfig/ui/insertaxisdlg \
chart2/uiconfig/ui/insertgriddlg \
@@ -58,7 +60,6 @@ $(eval $(call gb_UIConfig_add_uifiles,modules/schart,\
chart2/uiconfig/ui/smoothlinesdlg \
chart2/uiconfig/ui/steppedlinesdlg \
chart2/uiconfig/ui/titlerotationtabpage \
-   chart2/uiconfig/ui/dlg_InsertLegend \
chart2/uiconfig/ui/tp_3D_SceneAppearance \
chart2/uiconfig/ui/tp_3D_SceneGeometry \
chart2/uiconfig/ui/tp_3D_SceneIllumination \
diff --git a/chart2/source/controller/dialogs/dlg_InsertDataTable.cxx 
b/chart2/source/controller/dialogs/dlg_InsertDataTable.cxx
new file mode 100644
index ..4b5e928db057
--- /dev/null
+++ b/chart2/source/controller/dialogs/dlg_InsertDataTable.cxx
@@ -0,0 +1,61 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include 
+
+namespace chart
+{
+InsertDataTableDialog::InsertDataTableDialog(weld::Window* pWindow)
+: GenericDialogController(pWindow, 
"modules/schart/ui/dlg_InsertDataTable.ui",
+  "InsertDataTableDialog")
+, m_aDataTablePropertiesResources(*m_xBuilder)
+, m_xCbShowDataTable(m_xBuilder->weld_check_button("showDataTable"))
+{
+m_xCbShowDataTable->connect_toggled(LINK(this, InsertDataTableDialog, 
ShowDataTableToggle));
+init(m_aData);
+}
+
+IMPL_LINK_NOARG(InsertDataTableDialog, ShowDataTableToggle, weld::Toggleable&, 
void)
+{
+changeEnabled();
+}
+
+void InsertDataTableDialog::changeEnabled()
+{
+bool bEnable = m_xCbShowDataTable->get_active();
+m_aDataTablePropertiesResources.setChecksSensitive(bEnable);
+m_aData.mbShow = bEnable;
+}
+
+void InsertDataTableDialog::init(DataTableDialogData const& rData)
+{
+ 

[Libreoffice-bugs] [Bug 82321] row add/lock bug when deleting field from row to be added

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=82321

Robert Großkopf  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |WORKSFORME

--- Comment #11 from Robert Großkopf  ---
Tried it with LO 7.4.0.2, LO 7.3.5.2 and LO 7.1.5.2. Gives an error when trying
to switch to previous row:

Error inserting the new record at
/home/buildslave/source/libo-core/connectivity/source/commontools/dbtools.cxx:746
SQL Status: HY010

Function sequence error. at
/home/buildslave/source/libo-core/connectivity/source/commontools/dbexception.cxx:358

Bug couldn't be reproduced any more.

I will set this one as WORKSFORME.

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

[Libreoffice-bugs] [Bug 108441] [META] Database table bugs and enhancements

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=108441
Bug 108441 depends on bug 82321, which changed state.

Bug 82321 Summary: row add/lock bug when deleting field from row to be added
https://bugs.documentfoundation.org/show_bug.cgi?id=82321

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |WORKSFORME

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

[Libreoffice-bugs] [Bug 86066] [META] Bugs and improvements to the status bar

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=86066
Bug 86066 depends on bug 127856, which changed state.

Bug 127856 Summary: Pushing More... in language selection in Calc Status bar 
leads to Language settings instead of cell properties
https://bugs.documentfoundation.org/show_bug.cgi?id=127856

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |WORKSFORME

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

[Libreoffice-bugs] [Bug 127856] Pushing More... in language selection in Calc Status bar leads to Language settings instead of cell properties

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=127856

andis.lazd...@gmail.com changed:

   What|Removed |Added

 Resolution|--- |WORKSFORME
 Status|NEW |RESOLVED

--- Comment #12 from andis.lazd...@gmail.com ---
Works as supposed, in the same way in Calc and Writer.

Version: 7.4.0.3 / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 8; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: lv-LV (en_US.UTF-8); UI: en-US
Calc: threaded

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

[Libreoffice-bugs] [Bug 135459] There is no a Build ID in About dialog

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=135459

Dieter  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 135460] BORDERS TAB IN PARAGRAPH DIALOG: Make selection of borders in User-defined preview more visible (see comment 7)

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=135460

Dieter  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 135359] BORDERS "Line arrangement" display shows "Set no borders" when dialog is reopened, irrespective of actual border setting

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=135359

Dieter  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 103729] [META] HarfBuzz-based common text layout regressions

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103729

Khaled Hosny  changed:

   What|Removed |Added

   Assignee|kha...@aliftype.com |libreoffice-b...@lists.free
   ||desktop.org

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

[Libreoffice-bugs] [Bug 150381] LibreOffice 7.3.5.2 started giving a warning when clicking link to HTML file

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150381

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 150381] LibreOffice 7.3.5.2 started giving a warning when clicking link to HTML file

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150381

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

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

[Libreoffice-bugs] [Bug 149055] Embedded chart no longer works properly

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149055

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 149055] Embedded chart no longer works properly

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149055

--- 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 148311] calc . REQUEST for Copy & Past configurability

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148311

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

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

[Libreoffice-bugs] [Bug 148311] calc . REQUEST for Copy & Past configurability

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148311

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 146770] tarda en abrir archivos y rutina de párrafo tarda 8 segundos en abrir

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146770

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 138391] duplex printing does not work

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=138391

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 146770] tarda en abrir archivos y rutina de párrafo tarda 8 segundos en abrir

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146770

--- Comment #4 from QA Administrators  ---
Dear elvialh55,

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 138391] duplex printing does not work

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=138391

--- Comment #4 from QA Administrators  ---
Dear billfrew90,

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 147120] Repairs document to old version and no clue about new document.

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=147120

--- Comment #2 from QA Administrators  ---
Dear rackaitis,

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 91111] Importing Docbook ignores lists

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=9

--- Comment #14 from QA Administrators  ---
Dear Stéphane Bortzmeyer,

To make sure we're focusing on the bugs that affect our users today,
LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed
bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this
bug report. During that time, it's possible that the bug has been fixed, or the
details of the problem have changed. We'd really appreciate your help in
getting confirmation that the bug is still present.

If you have time, please do the following:

Test to see if the bug is still present with the latest version of LibreOffice
from https://www.libreoffice.org/download/

If the bug is present, please leave a comment that includes the information
from Help - About LibreOffice.

If the bug is NOT present, please set the bug's Status field to
RESOLVED-WORKSFORME and leave a comment that includes the information from Help
- About LibreOffice.

Please DO NOT

Update the version field
Reply via email (please reply directly on the bug tracker)
Set the bug's Status field to RESOLVED - FIXED (this status has a particular
meaning that is not 
appropriate in this case)


If you want to do more to help you can test to see if your issue is a
REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your
bug pertains to a feature added after 3.3) from
https://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to 'inherited from OOo';
4b. If the bug was not present in 3.3 - add 'regression' to keyword


Feel free to come ask questions or to say hello in our QA chat:
https://web.libera.chat/?settings=#libreoffice-qa

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

Warm Regards,
QA Team

MassPing-UntouchedBug

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

[Libreoffice-bugs] [Bug 135932] Open File option in front screen does nothing

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=135932

--- Comment #5 from QA Administrators  ---
Dear Dan Essin,

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 80931] VIEWING: Size of caption depending on amount of description text & above/below object

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=80931

--- Comment #8 from QA Administrators  ---
Dear Dominik,

To make sure we're focusing on the bugs that affect our users today,
LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed
bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this
bug report. During that time, it's possible that the bug has been fixed, or the
details of the problem have changed. We'd really appreciate your help in
getting confirmation that the bug is still present.

If you have time, please do the following:

Test to see if the bug is still present with the latest version of LibreOffice
from https://www.libreoffice.org/download/

If the bug is present, please leave a comment that includes the information
from Help - About LibreOffice.

If the bug is NOT present, please set the bug's Status field to
RESOLVED-WORKSFORME and leave a comment that includes the information from Help
- About LibreOffice.

Please DO NOT

Update the version field
Reply via email (please reply directly on the bug tracker)
Set the bug's Status field to RESOLVED - FIXED (this status has a particular
meaning that is not 
appropriate in this case)


If you want to do more to help you can test to see if your issue is a
REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your
bug pertains to a feature added after 3.3) from
https://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to 'inherited from OOo';
4b. If the bug was not present in 3.3 - add 'regression' to keyword


Feel free to come ask questions or to say hello in our QA chat:
https://web.libera.chat/?settings=#libreoffice-qa

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

Warm Regards,
QA Team

MassPing-UntouchedBug

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

[Libreoffice-bugs] [Bug 106038] Selected range of text disappears when going to Print Preview and back

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=106038

--- Comment #7 from QA Administrators  ---
Dear Thomas Lendo,

To make sure we're focusing on the bugs that affect our users today,
LibreOffice QA is asking bug reporters and confirmers to retest open, confirmed
bugs which have not been touched for over a year.

There have been thousands of bug fixes and commits since anyone checked on this
bug report. During that time, it's possible that the bug has been fixed, or the
details of the problem have changed. We'd really appreciate your help in
getting confirmation that the bug is still present.

If you have time, please do the following:

Test to see if the bug is still present with the latest version of LibreOffice
from https://www.libreoffice.org/download/

If the bug is present, please leave a comment that includes the information
from Help - About LibreOffice.

If the bug is NOT present, please set the bug's Status field to
RESOLVED-WORKSFORME and leave a comment that includes the information from Help
- About LibreOffice.

Please DO NOT

Update the version field
Reply via email (please reply directly on the bug tracker)
Set the bug's Status field to RESOLVED - FIXED (this status has a particular
meaning that is not 
appropriate in this case)


If you want to do more to help you can test to see if your issue is a
REGRESSION. To do so:
1. Download and install oldest version of LibreOffice (usually 3.3 unless your
bug pertains to a feature added after 3.3) from
https://downloadarchive.documentfoundation.org/libreoffice/old/

2. Test your bug
3. Leave a comment with your results.
4a. If the bug was present with 3.3 - set version to 'inherited from OOo';
4b. If the bug was not present in 3.3 - add 'regression' to keyword


Feel free to come ask questions or to say hello in our QA chat:
https://web.libera.chat/?settings=#libreoffice-qa

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

Warm Regards,
QA Team

MassPing-UntouchedBug

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

[Libreoffice-bugs] [Bug 150102] LibreOffice Impress has white line on bottom and right edge of projection monitor

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150102

NoWorries  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #6 from NoWorries  ---
I now have LO Impress Version 7.4.0.3 and on Wayland Kubuntu 22.10 it is
working correctly with no white lines on the screen edges showing.  I have the
Format set to On-screen Show (16:9) and I make sure that no packages such as
libreofice-kf5, libreofice-kde5, libreoffice-kde4, and libreoffice-kde are NOT
installed.

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

[Libreoffice-bugs] [Bug 150408] "Legal numbering style" is not applied to list levels with tag

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150408

Tex2002ans  changed:

   What|Removed |Added

   Hardware|x86-64 (AMD64)  |All
 Blocks||108770
 OS|Linux (All) |All
Version|unspecified |6.0.7.3 release


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=108770
[Bug 108770] [META] DOCX (OOXML) bullet and numbering list-related issues
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 108770] [META] DOCX (OOXML) bullet and numbering list-related issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=108770

Tex2002ans  changed:

   What|Removed |Added

 Depends on||150408


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=150408
[Bug 150408] "Legal numbering style" is not applied to list levels with
 tag
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 150408] "Legal numbering style" is not applied to list levels with tag

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150408

--- Comment #2 from Tex2002ans  ---
Created attachment 181777
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181777=edit
LO 7.3.5 vs. Word 2016 Comparison

I confirm the bug:

Version: 7.3.5.2 (x64) / LibreOffice Community
Build ID: 184fe81b8c8c30d8b5082578aee2fed2ea847c01
CPU threads: 8; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: threaded

- - -

Attached is an image showing the difference in list numbering between LO +
Microsoft Word.

Word 2016:

- 1.
-- 1.1.
--- 1.1.1
- 2.
-- 2.1.
--- 2.1.1.

LO 7.3.5:

- 1.
-- 1.a.<--- Here
--- 1.a.i. <--- Here
- 2.
-- 2.a.<--- Here
--- 2.a.i. <--- Here

- - -

Other discussion originally happened in this Reddit post:

https://www.reddit.com/r/libreoffice/comments/wmwlf1/does_libreoffices_cli_support_islgl_legal/

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

[Libreoffice-bugs] [Bug 149055] Embedded chart no longer works properly

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149055

--- Comment #3 from John  ---
Created attachment 181776
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181776=edit
Requested sample file

I've enclosed a copy of the file. If you open it in the current version of
LibreOffice the vertical lines in the chart will be elongated. The original the
lines were within the colored part of the chart. On opening you will see they
are elongated past that area. Make a minor change to  a cell and save, close
and reopen the spreadsheet. You will observe the lines have lengthened again.
Each iteration will cause the lines to grow longer.

The lines originally marked the boundary between months. As they lengthen they
become skewed.

There are no instructions about where to upload the file so I'm enclosing it
with this email. I'll also try to upload it to the bug report.

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

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

2022-08-14 Thread Caolán McNamara (via logerrit)
 sc/qa/unit/data/xls/pass/ofz49713-1.xls |binary
 sc/source/core/data/table2.cxx  |   20 
 2 files changed, 16 insertions(+), 4 deletions(-)

New commits:
commit 6c81a09e3ef239a2d7a991d00fe3620a67298b99
Author: Caolán McNamara 
AuthorDate: Sun Aug 14 20:46:49 2022 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 15 00:50:46 2022 +0200

ofz#49713 Heap-use-after-free

the dtor of ScAttrArray where the std::vector ends up
will call ScDocumentPool::Remove on each entries pPattern, assuming
that a matching ScDocumentPool::Put was called on each, something
that is elided if we just do a simply copy here.

probably a problem since:

commit dddee125cc32f1ad5228e598a7de04e9654e65c1
Date:   Thu Mar 10 15:03:25 2022 +0100

load ods/xlsx with full row attributes without allocating all columns

Change-Id: I3a5e2e3fa4d40343f30f9eefbabd1579d8a97e02
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138262
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/sc/qa/unit/data/xls/pass/ofz49713-1.xls 
b/sc/qa/unit/data/xls/pass/ofz49713-1.xls
new file mode 100644
index ..b32574013e95
Binary files /dev/null and b/sc/qa/unit/data/xls/pass/ofz49713-1.xls differ
diff --git a/sc/source/core/data/table2.cxx b/sc/source/core/data/table2.cxx
index c7c8a92b03dc..4eb5671f0dc7 100644
--- a/sc/source/core/data/table2.cxx
+++ b/sc/source/core/data/table2.cxx
@@ -2897,6 +2897,20 @@ void ScTable::ApplyPatternArea( SCCOL nStartCol, SCROW 
nStartRow, SCCOL nEndCol,
 CreateColumnIfNotExists(i).ApplyPatternArea(nStartRow, nEndRow, rAttr, 
pDataArray, pIsChanged);
 }
 
+namespace
+{
+std::vector duplicateScAttrEntries(ScDocument& rDocument, 
const std::vector& rOrigData)
+{
+std::vector aData(rOrigData);
+for (size_t nIdx = 0; nIdx < aData.size(); ++nIdx)
+{
+ScPatternAttr aNewPattern(*aData[nIdx].pPattern);
+aData[nIdx].pPattern = ()->Put(aNewPattern);
+}
+return aData;
+}
+}
+
 void ScTable::SetAttrEntries( SCCOL nStartCol, SCCOL nEndCol, 
std::vector && vNewData)
 {
 if (!ValidCol(nStartCol) || !ValidCol(nEndCol))
@@ -2908,8 +2922,7 @@ void ScTable::SetAttrEntries( SCCOL nStartCol, SCCOL 
nEndCol, std::vector(vNewData));
+aCol[i].SetAttrEntries(duplicateScAttrEntries(rDocument, 
vNewData));
 aDefaultColData.SetAttrEntries(std::move(vNewData));
 }
 else
@@ -2922,8 +2935,7 @@ void ScTable::SetAttrEntries( SCCOL nStartCol, SCCOL 
nEndCol, std::vector(vNewData));
+aCol[i].SetAttrEntries(duplicateScAttrEntries(rDocument, 
vNewData));
 aCol[nEndCol].SetAttrEntries( std::move(vNewData));
 }
 }


[Libreoffice-bugs] [Bug 113298] RTL: Automatic language detection based on keyboard layout

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=113298

--- Comment #7 from Eyal Rozenberg  ---
Still the same behavior with:

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 5c68399e6bea3aa18477487400f8bb143d6ed84e
CPU threads: 4; OS: Linux 5.18; UI render: default; VCL: gtk3
Locale: en-IL (en_IL); UI: en-US

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

Infra call on Tue, Aug 16 at 16:30 UTC

2022-08-14 Thread Guilhem Moulin
Hi there,

The next infra call will take place at `date -d "Tue, 16 Aug 2022 16:30 UTC"`
(18:30 Berlin time).

We'll meet at https://jitsi.documentfoundation.org/infra and write the minutes
to https://pad.documentfoundation.org/p/infra .  Agenda TBA.

See you there!
Cheers,
-- 
Guilhem.


signature.asc
Description: PGP signature


[Libreoffice-bugs] [Bug 133184] Arabic Tatweel disconnected from following letter when surrounded with Japanese quotation marks

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=133184

--- Comment #14 from Khaled Hosny  ---
Created attachment 181775
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181775=edit
Screenshot showing rendering as expected

Looks fine here, can’t reproduce with various fonts. If this is still an issue,
please attach LibreOffice document to reproduce it.

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

segfault in unopkg.bin

2022-08-14 Thread Terrence Enger
Hi,

I managed to provoke unopkg.bin to segfault while I was doing `make`
in instdir/sdk/examples/cpp/complextoolbarcontrols/.  Is this a
candidate for a bug report?  Or, is it a case of: I deserve whatever I
do to myself?

FWIW, my mistake was to define envvar URE_BOOTSTRAP with an embedded
equal sign where it should have had a colon.

Thank you for your attention and guidance.
Terry.



Re: IMPORTANT UPDATE LibreOffice Conference

2022-08-14 Thread Italo Vignoli
I have added the hotel list to the conference website. You can also use 
Booking.Com as an alternative.


On 8/5/22 09:06, Stephan Bergmann wrote:

On 7/1/22 18:00, Italo Vignoli wrote:
[...]
We will share all logistic details before July 15. Together with the 
logistic details, we will share infos about places to see, and food & 
drinks to enjoy.
[...]> As soon as we share logistic details, we also suggest people to 
look for
hotel rooms based on the list of affordable hotels close to the metro 
we are putting together. Most of the hotels will be rather small, as 
large hotels are usually quite expensive.

[...]

Hi Italo,

Is there an update on such a hotel list?  Might be helpful when one is 
looking for an appropriate hotel to book.



--
Italo Vignoli - LibreOffice Marketing & PR
mobile/signal +39.348.5653829 - emailit...@libreoffice.org
GPG Key ID - 0xAAB8D5C0
DB75 1534 3FD0 EA5F 56B5 FDA6 DE82 934C AAB8 D5C0


[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 133184, which changed state.

Bug 133184 Summary: Arabic Tatweel disconnected from following letter when 
surrounded with Japanese quotation marks
https://bugs.documentfoundation.org/show_bug.cgi?id=133184

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |WORKSFORME

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

[Libreoffice-bugs] [Bug 71732] [META] Bugs related to text rendering, typography and font features in LO

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=71732
Bug 71732 depends on bug 133184, which changed state.

Bug 133184 Summary: Arabic Tatweel disconnected from following letter when 
surrounded with Japanese quotation marks
https://bugs.documentfoundation.org/show_bug.cgi?id=133184

   What|Removed |Added

 Status|NEEDINFO|RESOLVED
 Resolution|--- |WORKSFORME

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

[Libreoffice-bugs] [Bug 133184] Arabic Tatweel disconnected from following letter when surrounded with Japanese quotation marks

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=133184

Khaled Hosny  changed:

   What|Removed |Added

 Resolution|--- |WORKSFORME
 Status|NEEDINFO|RESOLVED
 CC||kha...@aliftype.com

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

[Libreoffice-bugs] [Bug 105948] [META] Undo/Redo bugs and enhancements

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=105948

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 Depends on||135765


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=135765
[Bug 135765] bullets in table not restored after undo cell merging
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 103100] [META] Writer table bugs

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103100

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 Depends on||135765


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=135765
[Bug 135765] bullets in table not restored after undo cell merging
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 135765] bullets in table not restored after undo cell merging

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=135765

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 CC||kelem...@ubuntu.com
 Blocks||105948, 103100

--- Comment #5 from Gabor Kelemen (allotropia)  ---
Still in

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: cdf5b6a2a6f0ba2838323ce4398948c3d3a56f8e
CPU threads: 8; OS: Linux 5.4; UI render: default; VCL: gtk3
Locale: hu-HU (hu_HU.UTF-8); UI: en-US
Calc: threaded


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=103100
[Bug 103100] [META] Writer table bugs
https://bugs.documentfoundation.org/show_bug.cgi?id=105948
[Bug 105948] [META] Undo/Redo bugs and enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 150389] loose centering with new entry in a table in writer

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150389

--- Comment #1 from seth_...@yahoo.com ---
New detail: The problem only occurs when new rows are added to an existing
table.Only then will the previously centered text suddenly become left
justified.

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810

Eyal Rozenberg  changed:

   What|Removed |Added

 Depends on||87731


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=87731
[Bug 87731] Tag appearing in Arabic characters when change size of fonts
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 43808] [META] Right-To-Left and Complex Text Layout language issues (RTL/CTL)

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=43808

Eyal Rozenberg  changed:

   What|Removed |Added

 Depends on|87731   |


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=87731
[Bug 87731] Tag appearing in Arabic characters when change size of fonts
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 144998] Impress Ubuntu at save us the last modifications disappear from both last versions

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144998

BogdanB  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |WORKSFORME

--- Comment #8 from BogdanB  ---
Based on your last comment and the fact that noone will install an unsupported
version I will close the bug as Worksforme. Please be free to reopen as
unconfirmed if you want.

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

[Libreoffice-bugs] [Bug 150411] Table scrambled at undo. Track Changes involved

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150411

Telesto  changed:

   What|Removed |Added

   Keywords||bibisectRequest
Version|7.5.0.0 alpha0+ Master  |7.4.0.0 alpha1+

--- Comment #3 from Telesto  ---
Also in
Version: 7.4.0.0.alpha1+ (x64) / LibreOffice Community
Build ID: 0c17712465b07e8a7ebe078b00123965b112f0ac
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

not in
Version: 7.1.8.0.0+ (x64) / LibreOffice Community
Build ID: a94b58277c7aeaa83ce14347cd0b8f7137969d03
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

Note: in 7.1.8 track Changes Record didn't register table deletion

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

[Libreoffice-bugs] [Bug 150411] Table scrambled at undo. Track Changes involved

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150411

Telesto  changed:

   What|Removed |Added

Summary|Table scrambled at undo |Table scrambled at undo.
   ||Track Changes involved

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

[Libreoffice-bugs] [Bug 150411] Table scrambled at undo

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150411

--- Comment #1 from Telesto  ---
Created attachment 181773
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181773=edit
Example file

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

[Libreoffice-bugs] [Bug 150411] New: Table scrambled at undo

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150411

Bug ID: 150411
   Summary: Table scrambled at undo
   Product: LibreOffice
   Version: 7.5.0.0 alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: tele...@surfxs.nl

Description:
Table scrambled at undo

Steps to Reproduce:
1. Open the attached file
2. Place the cursor in the table on page 2
3. Menu Table -> Delete -> Table
4. CTRL+Z
5. Edit -> Track Changes -> Record -> Toggle record on & show ON
6. Delete the table again
7. Press undo

Actual Results:
See screenshot

Expected Results:
Same table as before on file open


Reproducible: Always


User Profile Reset: No



Additional Info:
Version: 7.5.0.0.alpha0+ (x64) / LibreOffice Community
Build ID: 86b2bfd34a4f07c54f03c8c8dfe48e0810834628
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Raster; VCL: win
Locale: nl-NL (nl_NL); UI: en-US
Calc: CL

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

[Libreoffice-bugs] [Bug 103538] Use OpenType Sanitiser to sanitize embedded fonts

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103538

Khaled Hosny  changed:

   What|Removed |Added

 CC||kha...@aliftype.com
 Resolution|--- |WONTFIX
 Status|NEW |RESOLVED

--- Comment #3 from Khaled Hosny  ---
No security issue about embedded fonts in 5 years, probably not worth it.

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

[Libreoffice-bugs] [Bug 144998] Impress Ubuntu at save us the last modifications disappear from both last versions

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=144998

szelev71  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED

--- Comment #7 from szelev71  ---
I mark it as unconfirmed, but nobody else tried it on the same version of
Ubuntu with the same version of LO.
By the way, on the newer Ubuntu and LO it works without this bug.

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 137530, which changed state.

Bug 137530 Summary: FORMATTING: Arabic Letters are sometimes disjoined in 
justified alignment in LibreOffice Impress
https://bugs.documentfoundation.org/show_bug.cgi?id=137530

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 137530, which changed state.

Bug 137530 Summary: FORMATTING: Arabic Letters are sometimes disjoined in 
justified alignment in LibreOffice Impress
https://bugs.documentfoundation.org/show_bug.cgi?id=137530

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 103492, which changed state.

Bug 103492 Summary: Arabic words not formed well in textboxes when sentence 
begins with English word
https://bugs.documentfoundation.org/show_bug.cgi?id=103492

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 113064] [META] RTL/CTL textbox related issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=113064
Bug 113064 depends on bug 103492, which changed state.

Bug 103492 Summary: Arabic words not formed well in textboxes when sentence 
begins with English word
https://bugs.documentfoundation.org/show_bug.cgi?id=103492

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

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

2022-08-14 Thread Khaled Hosny (via logerrit)
 vcl/source/filter/svm/SvmReader.cxx |   21 +
 vcl/source/filter/svm/SvmWriter.cxx |8 +++-
 2 files changed, 28 insertions(+), 1 deletion(-)

New commits:
commit 62ff1053eeba26cecd7264db08e73da154835816
Author: Khaled Hosny 
AuthorDate: Sat Aug 13 23:06:27 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 22:01:19 2022 +0200

svm: serialize MetaTextArrayAction’s Kashida array

Otherwise reading SVM files would lose the Kashida but not the DX
adjustments and would leave blanks in their place.

Change-Id: Id37e09fed12da5e256d65e2c45c91869364c8d06
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138242
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/vcl/source/filter/svm/SvmReader.cxx 
b/vcl/source/filter/svm/SvmReader.cxx
index f02451ea36ff..a7b990c3ddba 100644
--- a/vcl/source/filter/svm/SvmReader.cxx
+++ b/vcl/source/filter/svm/SvmReader.cxx
@@ -707,6 +707,27 @@ rtl::Reference 
SvmReader::TextArrayHandler(const ImplMetaReadData* p
 if (!aArray.empty())
 pAction->SetDXArray(std::move(aArray));
 
+if (aCompat.GetVersion() >= 3) // Version 3
+{
+sal_Int32 nKashidaAryLen(0);
+mrStream.ReadInt32(nKashidaAryLen);
+nTmpLen = std::min(nKashidaAryLen, sal_Int32(aArray.size()));
+if (nTmpLen)
+{
+// aKashidaArray, if not empty, must be the same size as aArray
+std::vector aKashidaArray(aArray.size(), 0);
+
+// [-loplugin:fakebool] false positive:
+sal_Bool val(sal_False);
+for (sal_Int32 i = 0; i < nTmpLen; i++)
+{
+mrStream.ReadUChar(val);
+aKashidaArray[i] = val;
+}
+pAction->SetKashidaArray(std::move(aKashidaArray));
+}
+}
+
 return pAction;
 }
 
diff --git a/vcl/source/filter/svm/SvmWriter.cxx 
b/vcl/source/filter/svm/SvmWriter.cxx
index 118db1325d71..6dec8279c151 100644
--- a/vcl/source/filter/svm/SvmWriter.cxx
+++ b/vcl/source/filter/svm/SvmWriter.cxx
@@ -988,7 +988,7 @@ void SvmWriter::TextArrayHandler(const MetaTextArrayAction* 
pAction, const ImplM
 
 const sal_Int32 nAryLen = !rDXArray.empty() ? pAction->GetLen() : 0;
 
-VersionCompatWrite aCompat(mrStream, 2);
+VersionCompatWrite aCompat(mrStream, 3);
 TypeSerializer aSerializer(mrStream);
 aSerializer.writePoint(pAction->GetPoint());
 mrStream.WriteUniOrByteString(pAction->GetText(), pData->meActualCharSet);
@@ -1000,6 +1000,12 @@ void SvmWriter::TextArrayHandler(const 
MetaTextArrayAction* pAction, const ImplM
 mrStream.WriteInt32(rDXArray[i]);
 
 write_uInt16_lenPrefixed_uInt16s_FromOUString(mrStream, 
pAction->GetText()); // version 2
+
+// Version 3
+const auto& rKashidaArray = pAction->GetKashidaArray();
+mrStream.WriteInt32(sal_Int32(rKashidaArray.size()));
+for (const auto& val : rKashidaArray)
+mrStream.WriteUChar(val);
 }
 
 void SvmWriter::StretchTextHandler(const MetaStretchTextAction* pAction,


[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 106653, which changed state.

Bug 106653 Summary: RTL Arabic text has some gaps between characters
https://bugs.documentfoundation.org/show_bug.cgi?id=106653

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 71732] [META] Bugs related to text rendering, typography and font features in LO

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=71732
Bug 71732 depends on bug 106653, which changed state.

Bug 106653 Summary: RTL Arabic text has some gaps between characters
https://bugs.documentfoundation.org/show_bug.cgi?id=106653

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 139627, which changed state.

Bug 139627 Summary: Justified paragraph rendering defect when inserting Arabic 
diacritics
https://bugs.documentfoundation.org/show_bug.cgi?id=139627

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 43808] [META] Right-To-Left and Complex Text Layout language issues (RTL/CTL)

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=43808
Bug 43808 depends on bug 139627, which changed state.

Bug 139627 Summary: Justified paragraph rendering defect when inserting Arabic 
diacritics
https://bugs.documentfoundation.org/show_bug.cgi?id=139627

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150381] LibreOffice 7.3.5.2 started giving a warning when clicking link to HTML file

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150381

Timur  changed:

   What|Removed |Added

Summary|LibreOffice 7.3.5.2 dropped |LibreOffice 7.3.5.2 started
   |a feature I use a lot when  |giving a warning when
   |editing.|clicking link to HTML file

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 137528, which changed state.

Bug 137528 Summary: Arabic Letters are sometimes disjoined and the vowel marks 
are misplaced in justified paragraph in Text Box
https://bugs.documentfoundation.org/show_bug.cgi?id=137528

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 113046] renaming a style

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=113046
Bug 113046 depends on bug 137528, which changed state.

Bug 137528 Summary: Arabic Letters are sometimes disjoined and the vowel marks 
are misplaced in justified paragraph in Text Box
https://bugs.documentfoundation.org/show_bug.cgi?id=137528

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 145962] [META] Bugs related to kerning/font spacing (cramped/to width/wiggling)

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145962
Bug 145962 depends on bug 137528, which changed state.

Bug 137528 Summary: Arabic Letters are sometimes disjoined and the vowel marks 
are misplaced in justified paragraph in Text Box
https://bugs.documentfoundation.org/show_bug.cgi?id=137528

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 146199, which changed state.

Bug 146199 Summary: Farsi character overlapped with underscore/dash in an 
Impress presentation
https://bugs.documentfoundation.org/show_bug.cgi?id=146199

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 146199, which changed state.

Bug 146199 Summary: Farsi character overlapped with underscore/dash in an 
Impress presentation
https://bugs.documentfoundation.org/show_bug.cgi?id=146199

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 145647, which changed state.

Bug 145647 Summary: Arabic tatweel rendered as whitespace with preceding 
English text
https://bugs.documentfoundation.org/show_bug.cgi?id=145647

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 145647, which changed state.

Bug 145647 Summary: Arabic tatweel rendered as whitespace with preceding 
English text
https://bugs.documentfoundation.org/show_bug.cgi?id=145647

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 103378] [META] PDF export bugs and enhancements

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=103378

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 Depends on||146575


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=146575
[Bug 146575] Writer won't export hyperlinks to pdf
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 146575] Writer won't export hyperlinks to pdf

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=146575

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 Blocks||103378


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=103378
[Bug 103378] [META] PDF export bugs and enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 127176, which changed state.

Bug 127176 Summary: 'Noto Nastaliq Urdu' font does not render correctly when 
justified.
https://bugs.documentfoundation.org/show_bug.cgi?id=127176

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 71732] [META] Bugs related to text rendering, typography and font features in LO

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=71732
Bug 71732 depends on bug 127176, which changed state.

Bug 127176 Summary: 'Noto Nastaliq Urdu' font does not render correctly when 
justified.
https://bugs.documentfoundation.org/show_bug.cgi?id=127176

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 112849, which changed state.

Bug 112849 Summary: Kashida characters appears at particular zoom levels (310%)
https://bugs.documentfoundation.org/show_bug.cgi?id=112849

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 112849, which changed state.

Bug 112849 Summary: Kashida characters appears at particular zoom levels (310%)
https://bugs.documentfoundation.org/show_bug.cgi?id=112849

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 71732] [META] Bugs related to text rendering, typography and font features in LO

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=71732
Bug 71732 depends on bug 112849, which changed state.

Bug 112849 Summary: Kashida characters appears at particular zoom levels (310%)
https://bugs.documentfoundation.org/show_bug.cgi?id=112849

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 108604, which changed state.

Bug 108604 Summary: wrong kashida placement when exporting to PDF
https://bugs.documentfoundation.org/show_bug.cgi?id=108604

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 108604, which changed state.

Bug 108604 Summary: wrong kashida placement when exporting to PDF
https://bugs.documentfoundation.org/show_bug.cgi?id=108604

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 106309, which changed state.

Bug 106309 Summary: Wrong placement of kashida causes defect in Arabic/Persian 
text rendering
https://bugs.documentfoundation.org/show_bug.cgi?id=106309

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 112810] [META] Arabic language-specific RTL issues

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112810
Bug 112810 depends on bug 106309, which changed state.

Bug 106309 Summary: Wrong placement of kashida causes defect in Arabic/Persian 
text rendering
https://bugs.documentfoundation.org/show_bug.cgi?id=106309

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 71732] [META] Bugs related to text rendering, typography and font features in LO

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=71732
Bug 71732 depends on bug 106309, which changed state.

Bug 106309 Summary: Wrong placement of kashida causes defect in Arabic/Persian 
text rendering
https://bugs.documentfoundation.org/show_bug.cgi?id=106309

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150406] Style Inspector shows with Default Paragraph Style with Times of New Roman, but actual font is David (without DF accordantly to Style Inspector)

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150406

--- Comment #8 from Telesto  ---
Created attachment 181772
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181772=edit
Example file B

Before I start spamming the bug tracker.. another attempt to illustrate the
issue

1. Open Writer
2. Tools -> Options -> Language settings -> Languages -> Make sure complex text
layout being enabled set for Hebrew [not sure if it matters, but well)
3. Open the attached file
4. Open Sidebar -> Style Inspector
5. Walk through the file with arrows.. Nothing about Liberation Serif (OK)
6. Save the file to DOC
7. File -> Reload
8. Walk through the file with arrows.. Notice Character Direct Formatting
appearing regularly in the area between words

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 87731, which changed state.

Bug 87731 Summary: Tag appearing in Arabic characters when change size of fonts
https://bugs.documentfoundation.org/show_bug.cgi?id=87731

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150285] [META] Problems with Justified Arabic/Persian text

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150285
Bug 150285 depends on bug 104921, which changed state.

Bug 104921 Summary: Make Arabic text justification more robust
https://bugs.documentfoundation.org/show_bug.cgi?id=104921

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 116969] Justification by alternative methods

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=116969
Bug 116969 depends on bug 104921, which changed state.

Bug 104921 Summary: Make Arabic text justification more robust
https://bugs.documentfoundation.org/show_bug.cgi?id=104921

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 145962] [META] Bugs related to kerning/font spacing (cramped/to width/wiggling)

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=145962
Bug 145962 depends on bug 104921, which changed state.

Bug 104921 Summary: Make Arabic text justification more robust
https://bugs.documentfoundation.org/show_bug.cgi?id=104921

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 150400] File menu is missing some items

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150400

Timur  changed:

   What|Removed |Added

Summary|Embeded Fonts   |File menu is missing some
   ||items

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

[Libreoffice-bugs] [Bug 150400] Embeded Fonts

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150400

Timur  changed:

   What|Removed |Added

 Status|RESOLVED|UNCONFIRMED
 Resolution|NOTABUG |---

--- Comment #6 from Timur  ---
This is not reproducible by someone  else hence cannot be confirmed. 
Did you really reset, delete, rename LO user profile?

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

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

2022-08-14 Thread Noel Grandin (via logerrit)
 sw/inc/unocrsrhelper.hxx|4 +--
 sw/source/core/crsr/crsrsh.cxx  |4 +--
 sw/source/core/crsr/swcrsr.cxx  |2 -
 sw/source/core/doc/DocumentContentOperationsManager.cxx |2 -
 sw/source/core/doc/DocumentRedlineManager.cxx   |   10 
 sw/source/core/doc/doccomp.cxx  |8 +++---
 sw/source/core/doc/docredln.cxx |2 -
 sw/source/core/frmedt/fetab.cxx |2 -
 sw/source/core/layout/frmtool.cxx   |2 -
 sw/source/core/layout/trvlfrm.cxx   |2 -
 sw/source/core/text/itratr.cxx  |2 -
 sw/source/core/text/redlnitr.cxx|2 -
 sw/source/core/undo/undel.cxx   |4 +--
 sw/source/core/undo/undobj.cxx  |   20 
 sw/source/core/undo/unins.cxx   |8 +++---
 sw/source/core/unocore/unoobj.cxx   |   10 
 sw/source/core/unocore/unoparagraph.cxx |6 ++--
 sw/source/uibase/docvw/edtwin.cxx   |7 ++---
 sw/source/uibase/uiview/formatclipboard.cxx |2 -
 sw/source/uibase/uiview/viewstat.cxx|2 -
 20 files changed, 50 insertions(+), 51 deletions(-)

New commits:
commit d0c07ec1cad386a0ec3bf34c6aa6e7b50859ef09
Author: Noel Grandin 
AuthorDate: Sun Aug 14 18:02:27 2022 +0200
Commit: Noel Grandin 
CommitDate: Sun Aug 14 21:28:37 2022 +0200

use more GetContentIndex

Change-Id: I48e36a95872b5c2bdb6b92c66403d4c08eedd38c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138255
Tested-by: Jenkins
Reviewed-by: Noel Grandin 

diff --git a/sw/inc/unocrsrhelper.hxx b/sw/inc/unocrsrhelper.hxx
index 014421371ba0..af3c779f2288 100644
--- a/sw/inc/unocrsrhelper.hxx
+++ b/sw/inc/unocrsrhelper.hxx
@@ -86,10 +86,10 @@ namespace SwUnoCursorHelper
 voidGetCurPageStyle(SwPaM const & rPaM, OUString 
);
 
 inline bool IsStartOfPara(SwPaM& rUnoCursor)
-{ return 
rUnoCursor.GetPoint()->nContent == 0;}
+{ return 
rUnoCursor.GetPoint()->GetContentIndex() == 0;}
 inline bool IsEndOfPara(SwPaM& rUnoCursor)
 { return 
rUnoCursor.GetPointContentNode() &&
-rUnoCursor.GetPoint()->nContent == 
rUnoCursor.GetPointContentNode()->Len();}
+
rUnoCursor.GetPoint()->GetContentIndex() == 
rUnoCursor.GetPointContentNode()->Len();}
 
 voidresetCursorPropertyValue(const 
SfxItemPropertyMapEntry& rEntry, SwPaM& rPam);
 /// @throws css::lang::IllegalArgumentException
diff --git a/sw/source/core/crsr/crsrsh.cxx b/sw/source/core/crsr/crsrsh.cxx
index 013f65bf6e5b..d9a040adfad4 100644
--- a/sw/source/core/crsr/crsrsh.cxx
+++ b/sw/source/core/crsr/crsrsh.cxx
@@ -1107,7 +1107,7 @@ bool SwCursorShell::IsSttPara() const
 }
 }
 }
-return m_pCurrentCursor->GetPoint()->nContent == 0;
+return m_pCurrentCursor->GetPoint()->GetContentIndex() == 0;
 }
 
 bool SwCursorShell::IsEndPara() const
@@ -1126,7 +1126,7 @@ bool SwCursorShell::IsEndPara() const
 }
 }
 }
-return m_pCurrentCursor->GetPoint()->nContent == 
m_pCurrentCursor->GetPointContentNode()->Len();
+return m_pCurrentCursor->GetPoint()->GetContentIndex() == 
m_pCurrentCursor->GetPointContentNode()->Len();
 }
 
 bool SwCursorShell::IsEndOfTable() const
diff --git a/sw/source/core/crsr/swcrsr.cxx b/sw/source/core/crsr/swcrsr.cxx
index 9cd4a326a533..5a47e9bdb965 100644
--- a/sw/source/core/crsr/swcrsr.cxx
+++ b/sw/source/core/crsr/swcrsr.cxx
@@ -1280,7 +1280,7 @@ bool SwCursor::IsInWordWT(sal_Int16 nWordType, 
SwRootFrame const*const pLayout)
 bool SwCursor::IsStartEndSentence(bool bEnd, SwRootFrame const*const pLayout) 
const
 {
 bool bRet = bEnd ?
-GetPointContentNode() && GetPoint()->nContent == 
GetPointContentNode()->Len() :
+GetPointContentNode() && GetPoint()->GetContentIndex() == 
GetPointContentNode()->Len() :
 GetPoint()->GetContentIndex() == 0;
 
 if ((pLayout != nullptr && pLayout->HasMergedParas()) || !bRet)
diff --git a/sw/source/core/doc/DocumentContentOperationsManager.cxx 
b/sw/source/core/doc/DocumentContentOperationsManager.cxx
index bd8b154b1323..1972be388df2 100644
--- a/sw/source/core/doc/DocumentContentOperationsManager.cxx
+++ b/sw/source/core/doc/DocumentContentOperationsManager.cxx
@@ -2452,7 +2452,7 @@ bool DocumentContentOperationsManager::MoveRange( SwPaM& 
rPaM, SwPosition& rPos,
 // tdf#99692 don't Move() back if that would end up in another node
 // because moving 

[Libreoffice-bugs] [Bug 149957] Screen Size Does Not Carry to Next Session with LO 7.4

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149957

Timur  changed:

   What|Removed |Added

   Keywords||possibleRegression

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

[Libreoffice-bugs] [Bug 149424] Layout loop in doc files

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149424

--- Comment #16 from Gabor Kelemen (allotropia)  ---
A fresh debug build spews this on the console every second or two:

warn:legacy.osl:23929:23929:sw/source/core/layout/layact.cxx:755: LoopControl_3
in Interrupt formatting in SwLayAction::InternalAction

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

[Libreoffice-bugs] [Bug 132261] [META] Table layouting: PrepareMake, MakeAll loops and crashes

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=132261

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 Depends on||149424


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=149424
[Bug 149424] Layout loop in doc files
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 149424] Layout loop in doc files

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=149424

Gabor Kelemen (allotropia)  changed:

   What|Removed |Added

 Blocks||132261
 CC||kelem...@ubuntu.com


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=132261
[Bug 132261] [META] Table layouting: PrepareMake, MakeAll loops and crashes
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 150381] LibreOffice 7.3.5.2 dropped a feature I use a lot when editing.

2022-08-14 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=150381

--- Comment #6 from JIM7  ---
Created attachment 181771
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181771=edit
Edited sample

Help/About is at the bottom of the Sample.
Hyperlink is not present in .odt.
This happens when I am directly editing html files.
I tested this file.
Hyperlink no longer works.
I'm also attaching the file that the hyperlink is supposed to access.

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

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

2022-08-14 Thread Khaled Hosny (via logerrit)
 editeng/source/editeng/impedit3.cxx |2 --
 1 file changed, 2 deletions(-)

New commits:
commit 895843ad466954b63f6f99a1b6319b7813d0dbe1
Author: Khaled Hosny 
AuthorDate: Sat Aug 13 00:00:27 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 21:13:04 2022 +0200

tdf#103492: Messed Arabic letter spacing in text starting with LTR 
character(s)

This partially reverts the still relevant parts of commit

commit 41007842ed9bb5d6165792a197769f72dae55a2c
Author: Martin Hosken 
Date:   Thu Sep 10 10:14:18 2015 +0700

Refactor graphite integration and update graphite

Change-Id: I09db7262c2a6180a078620ba086dd7a4c132bd82
Reviewed-on: https://gerrit.libreoffice.org/18459
Tested-by: Jenkins 
Reviewed-by: László Németh 
Reviewed-by: Martin Hosken 

No mention of the commit about this change or its relevance to Graphite
integration. But the separate Graphite layout engine is long gone and
Graphite handling goes through HarfBuzz now, so whatever the change was
for is probably no longer relevant.

I reverted only one the two changes in impedit3.cxx, as the other one
does not seem to make any difference for this issue.

Change-Id: Id70a711d6231cce52c9a32dbc3604860b1da9f4a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138217
Reviewed-by: Caolán McNamara 
Tested-by: Caolán McNamara 

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 6195b0e8e606..5ff196f02040 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -838,8 +838,6 @@ bool ImpEditEngine::CreateLines( sal_Int32 nPara, 
sal_uInt32 nStartPosY )
 EditLine aSaveLine( *pLine );
 SvxFont aTmpFont( pNode->GetCharAttribs().GetDefFont() );
 
-ImplInitLayoutMode(*GetRefDevice(), nPara, nIndex);
-
 std::vector aBuf( pNode->Len() );
 
 bool bSameLineAgain = false;// For TextRanger, if the height changes.


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

2022-08-14 Thread Khaled Hosny (via logerrit)
 vcl/source/gdi/CommonSalLayout.cxx |   10 +-
 1 file changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 3836b99f335ac9e6890545607d07124e455eb531
Author: Khaled Hosny 
AuthorDate: Fri Aug 12 14:00:25 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 21:12:37 2022 +0200

Use doubles when calculating the number of Kashidas to insert

Makes it a bit more stable across zoom levels and avoids excessive
overlaps due to calculating one copy too many because of rounding
errors.

Change-Id: I6a21a89c1dfda085b1f7fa9570c9875a0d39f671
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138192
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/vcl/source/gdi/CommonSalLayout.cxx 
b/vcl/source/gdi/CommonSalLayout.cxx
index 0bea121896b4..eff9dab9acbb 100644
--- a/vcl/source/gdi/CommonSalLayout.cxx
+++ b/vcl/source/gdi/CommonSalLayout.cxx
@@ -754,7 +754,7 @@ void GenericSalLayout::ApplyDXArray(const DC* pDXArray, 
const sal_Bool* pKashida
 return;
 
 // Find Kashida glyph width and index.
-DeviceCoordinate nKashidaWidth = 0;
+double nKashidaWidth = 0;
 hb_codepoint_t nKashidaIndex = 0;
 if (hb_font_get_glyph(GetFont().GetHbFont(), 0x0640, 0, ))
 nKashidaWidth = GetFont().GetKashidaWidth();
@@ -771,7 +771,7 @@ void GenericSalLayout::ApplyDXArray(const DC* pDXArray, 
const sal_Bool* pKashida
 auto pGlyphIter = m_GlyphItems.begin() + nInserted + pKashida.first;
 
 // The total Kashida width.
-DeviceCoordinate nTotalWidth = pKashida.second;
+double nTotalWidth = pKashida.second;
 
 // Number of times to repeat each Kashida.
 int nCopies = 1;
@@ -780,12 +780,12 @@ void GenericSalLayout::ApplyDXArray(const DC* pDXArray, 
const sal_Bool* pKashida
 
 // See if we can improve the fit by adding an extra Kashidas and
 // squeezing them together a bit.
-DeviceCoordinate nOverlap = 0;
-DeviceCoordinate nShortfall = nTotalWidth - nKashidaWidth * nCopies;
+double nOverlap = 0;
+double nShortfall = nTotalWidth - nKashidaWidth * nCopies;
 if (nShortfall > 0)
 {
 ++nCopies;
-DeviceCoordinate nExcess = nCopies * nKashidaWidth - nTotalWidth;
+double nExcess = nCopies * nKashidaWidth - nTotalWidth;
 if (nExcess > 0)
 nOverlap = nExcess / (nCopies - 1);
 }


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

2022-08-14 Thread Khaled Hosny (via logerrit)
 vcl/source/font/fontinstance.cxx   |2 +-
 vcl/source/gdi/CommonSalLayout.cxx |2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 3d0e482df81a28e6d0eb86cda71ac49d3363ae50
Author: Khaled Hosny 
AuthorDate: Fri Aug 12 02:51:50 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 21:12:21 2022 +0200

tdf#106653: Try harder not to return 0 for Kashida width

The font in the bug document has a very small Kashida advance width and
this gets rounded to zero at smaller zoom level. Try to round up the
value to avoid returning zero too eagerly.

Change-Id: I78c309d66a9e94812827c7c6724c67c4d09bcff9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138169
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/vcl/source/font/fontinstance.cxx b/vcl/source/font/fontinstance.cxx
index 0907d657ad02..44c2c8cc757d 100644
--- a/vcl/source/font/fontinstance.cxx
+++ b/vcl/source/font/fontinstance.cxx
@@ -73,7 +73,7 @@ int LogicalFontInstance::GetKashidaWidth() const
 {
 double nXScale = 0;
 GetScale(, nullptr);
-nWidth = hb_font_get_glyph_h_advance(pHbFont, nIndex) * nXScale;
+nWidth = std::ceil(hb_font_get_glyph_h_advance(pHbFont, nIndex) * 
nXScale);
 }
 
 return nWidth;
diff --git a/vcl/source/gdi/CommonSalLayout.cxx 
b/vcl/source/gdi/CommonSalLayout.cxx
index b3d63dac30a0..0bea121896b4 100644
--- a/vcl/source/gdi/CommonSalLayout.cxx
+++ b/vcl/source/gdi/CommonSalLayout.cxx
@@ -761,7 +761,7 @@ void GenericSalLayout::ApplyDXArray(const DC* pDXArray, 
const sal_Bool* pKashida
 
 if (nKashidaWidth <= 0)
 {
-SAL_WARN("vcl.gdi", "Asked to insert Kashidas in a font with 
zero-width Kashida");
+SAL_WARN("vcl.gdi", "Asked to insert Kashidas in a font with bogus 
Kashida width");
 return;
 }
 


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

2022-08-14 Thread Khaled Hosny (via logerrit)
 include/vcl/vcllayout.hxx  |2 
 vcl/inc/sallayout.hxx  |4 -
 vcl/source/gdi/CommonSalLayout.cxx |   82 +++--
 vcl/source/gdi/sallayout.cxx   |9 ++--
 vcl/source/outdev/font.cxx |   26 +--
 5 files changed, 54 insertions(+), 69 deletions(-)

New commits:
commit 5e2b7a656024b621bdeeb6efd977331191b66d9d
Author: Khaled Hosny 
AuthorDate: Fri Aug 12 02:12:04 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 21:11:52 2022 +0200

Streamline Kashida validation logic

We are asked to validate the position *after* which Kashida will be
inserted, but HarfBuzz will tell us which glyph we can insert Kashida
*before*. So align both by passing down the position before and after
and make the loop iterating over glyph items a lot simpler.

As a bonus, the new code allow Kashida insertion across layout change in
both sides, old code allowed it only at the start of the layout.

Change-Id: I9f632610b92c0f4c512e50456bf7d207175f17ac
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138168
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/include/vcl/vcllayout.hxx b/include/vcl/vcllayout.hxx
index 87d9345f4b9b..cbadb47ee0ba 100644
--- a/include/vcl/vcllayout.hxx
+++ b/include/vcl/vcllayout.hxx
@@ -92,7 +92,7 @@ public:
 virtual DeviceCoordinate FillDXArray( std::vector* 
pDXArray ) const = 0;
 virtual DeviceCoordinate GetTextWidth() const { return FillDXArray( 
nullptr ); }
 virtual voidGetCaretPositions( int nArraySize, sal_Int32* pCaretXArray 
) const = 0;
-virtual boolIsKashidaPosValid ( int /*nCharPos*/ ) const { return 
true; } // i60594
+virtual boolIsKashidaPosValid ( int /*nCharPos*/, int /*nNextCharPos*/ 
) const = 0; // i60594
 
 // methods using glyph indexing
 virtual boolGetNextGlyph(const GlyphItem** pGlyph, DevicePoint& rPos, 
int& nStart,
diff --git a/vcl/inc/sallayout.hxx b/vcl/inc/sallayout.hxx
index 885de2446db7..b49c1c663ee4 100644
--- a/vcl/inc/sallayout.hxx
+++ b/vcl/inc/sallayout.hxx
@@ -68,7 +68,7 @@ public:
  const LogicalFontInstance** ppGlyphFont = 
nullptr,
  const vcl::font::PhysicalFontFace** 
pFallbackFont = nullptr) const override;
 boolGetOutline(basegfx::B2DPolyPolygonVector&) const override;
-boolIsKashidaPosValid(int nCharPos) const override;
+boolIsKashidaPosValid(int nCharPos, int nNextCharPos) const 
override;
 SalLayoutGlyphs GetGlyphs() const final override;
 
 // used only by OutputDevice::ImplLayout, TODO: make friend
@@ -115,7 +115,7 @@ public:
 voidDrawText(SalGraphics&) const final override;
 SalLayoutGlyphs GetGlyphs() const final override;
 
-boolIsKashidaPosValid(int nCharPos) const final override;
+boolIsKashidaPosValid(int nCharPos, int nNextCharPos) const 
final override;
 
 // used by upper layers
 DeviceCoordinate GetTextWidth() const final override;
diff --git a/vcl/source/gdi/CommonSalLayout.cxx 
b/vcl/source/gdi/CommonSalLayout.cxx
index da7334352858..b3d63dac30a0 100644
--- a/vcl/source/gdi/CommonSalLayout.cxx
+++ b/vcl/source/gdi/CommonSalLayout.cxx
@@ -719,22 +719,19 @@ void GenericSalLayout::ApplyDXArray(const DC* pDXArray, 
const sal_Bool* pKashida
 m_GlyphItems[i].addNewWidth(nDiff);
 m_GlyphItems[i].adjustLinearPosX(nDelta + nDiff);
 
-// Adjust the X position of the rest of the glyphs in the cluster.
 size_t j = i;
 while (j > 0)
 {
 --j;
-if (!m_GlyphItems[j].IsInCluster())
+if (!(m_GlyphItems[j].IsDiacritic() || 
m_GlyphItems[j].IsInCluster()))
 break;
-m_GlyphItems[j].adjustLinearPosX(nDelta + nDiff);
-}
 
-// Move any non-spacing marks to keep attached to this cluster.
-while (j > 0)
-{
-if (!m_GlyphItems[j].IsDiacritic())
-break;
-m_GlyphItems[j--].adjustLinearPosX(nDiff);
+if (m_GlyphItems[j].IsInCluster())
+// Adjust X position of the remainder of the cluster.
+m_GlyphItems[j].adjustLinearPosX(nDelta + nDiff);
+else
+// Move non-spacing marks to keep attached to this cluster.
+m_GlyphItems[j].adjustLinearPosX(nDiff);
 }
 
 // This is a Kashida insertion position, mark it. Kashida glyphs
@@ -807,56 +804,27 @@ void GenericSalLayout::ApplyDXArray(const DC* pDXArray, 
const sal_Bool* pKashida
 }
 }
 
-bool GenericSalLayout::IsKashidaPosValid(int nCharPos) const
+// Kashida will be inserted between nCharPos and nNextCharPos.
+bool GenericSalLayout::IsKashidaPosValid(int nCharPos, int 

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

2022-08-14 Thread Khaled Hosny (via logerrit)
 editeng/source/editeng/impedit3.cxx |   18 ++
 sw/source/core/text/porlay.cxx  |   18 ++
 2 files changed, 20 insertions(+), 16 deletions(-)

New commits:
commit 72e56537d4bb9411229346da977d1d669ccfca9a
Author: Khaled Hosny 
AuthorDate: Thu Aug 11 18:52:17 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 21:11:15 2022 +0200

tdf#139627: Don’t set Kashida insertion position after combining marks

We were skipping the marks when checking for previous char, but not when
setting the index to previous char.

Fixes also the second part of tdf#106653.

Change-Id: I73436ba9df6ccf104ea6d4dfbd76a59c86040a44
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138154
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index d712dfa5..6195b0e8e606 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -2339,10 +2339,9 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // restore selection for proper iteration at the end of the function
 aWordSel.Max().SetIndex( nSavPos );
 
-sal_Int32 nIdx = 0;
+sal_Int32 nIdx = 0, nPrevIdx = 0;
 sal_Int32 nKashidaPos = -1;
-sal_Unicode cCh;
-sal_Unicode cPrevCh = 0;
+sal_Unicode cCh, cPrevCh = 0;
 
 int nPriorityLevel = 7;// 0..6 = level found
// 7 not found
@@ -2390,7 +2389,7 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // check if character is connectable to previous character,
 if ( lcl_ConnectToPrev( cCh, cPrevCh ) )
 {
-nKashidaPos = aWordSel.Min().GetIndex() + nIdx - 1;
+nKashidaPos = aWordSel.Min().GetIndex() + nPrevIdx;
 nPriorityLevel = 2;
 }
 }
@@ -2411,7 +2410,7 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // check if character is connectable to previous character,
 if ( lcl_ConnectToPrev( cCh, cPrevCh ) )
 {
-nKashidaPos = aWordSel.Min().GetIndex() + nIdx - 1;
+nKashidaPos = aWordSel.Min().GetIndex() + nPrevIdx;
 nPriorityLevel = 3;
 }
 }
@@ -2431,7 +2430,7 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // check if character is connectable to previous 
character,
 if ( lcl_ConnectToPrev( cCh, cPrevCh ) )
 {
-nKashidaPos = aWordSel.Min().GetIndex() + nIdx - 1;
+nKashidaPos = aWordSel.Min().GetIndex() + nPrevIdx;
 nPriorityLevel = 4;
 }
 }
@@ -2453,7 +2452,7 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // check if character is connectable to previous character,
 if ( lcl_ConnectToPrev( cCh, cPrevCh ) )
 {
-nKashidaPos = aWordSel.Min().GetIndex() + nIdx - 1;
+nKashidaPos = aWordSel.Min().GetIndex() + nPrevIdx;
 nPriorityLevel = 5;
 }
 }
@@ -2469,7 +2468,7 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // check if character is connectable to previous character,
 if ( lcl_ConnectToPrev( cCh, cPrevCh ) )
 {
-nKashidaPos = aWordSel.Min().GetIndex() + nIdx - 1;
+nKashidaPos = aWordSel.Min().GetIndex() + nPrevIdx;
 nPriorityLevel = 6;
 }
 }
@@ -2478,7 +2477,10 @@ void ImpEditEngine::ImpFindKashidas( ContentNode* pNode, 
sal_Int32 nStart, sal_I
 // Do not consider vowel marks when checking if a character
 // can be connected to previous character.
 if ( !isTransparentChar ( cCh) )
+{
 cPrevCh = cCh;
+nPrevIdx = nIdx;
+}
 
 ++nIdx;
 } // end of current word
diff --git a/sw/source/core/text/porlay.cxx b/sw/source/core/text/porlay.cxx
index 3db6a07600e9..d1e4b967f235 100644
--- a/sw/source/core/text/porlay.cxx
+++ b/sw/source/core/text/porlay.cxx
@@ -1509,10 +1509,9 @@ void SwScriptInfo::InitScriptInfo(const SwTextNode& 
rNode,
 {
 const OUString& rWord = aScanner.GetWord();
 
-sal_Int32 nIdx = 0;
+ 

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

2022-08-14 Thread Khaled Hosny (via logerrit)
 editeng/source/editeng/impedit3.cxx |  248 
 1 file changed, 199 insertions(+), 49 deletions(-)

New commits:
commit 640a2bc1cd093875b701bb958b6156d5752c0fac
Author: Khaled Hosny 
AuthorDate: Thu Aug 11 14:53:19 2022 +0200
Commit: Caolán McNamara 
CommitDate: Sun Aug 14 21:10:43 2022 +0200

tdf#137528: Sync Kashida algorithm in editeng with sw

It seems that editeng is using an old copy of the code in sw and it was
never updated.

With this change, the text in the test document from tdf#137528 looks
essentially identical in both cases.

Fixes also tdf#146199.

Change-Id: I38541bfb35eae36681af84e73a7965d36152448f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138145
Tested-by: Jenkins
Reviewed-by: Caolán McNamara 

diff --git a/editeng/source/editeng/impedit3.cxx 
b/editeng/source/editeng/impedit3.cxx
index 9d71560c108b..d712dfa5 100644
--- a/editeng/source/editeng/impedit3.cxx
+++ b/editeng/source/editeng/impedit3.cxx
@@ -75,6 +75,8 @@
 
 #include 
 
+#include 
+
 using namespace ::com::sun::star;
 using namespace ::com::sun::star::uno;
 using namespace ::com::sun::star::beans;
@@ -254,19 +256,93 @@ static Point lcl_ImplCalcRotatedPos( Point rPos, Point 
rOrigin, double nSin, dou
 return aTranslatedPos;
 }
 
-static bool lcl_IsLigature( sal_Unicode cCh, sal_Unicode cNextCh ) // For 
Kashidas from sw/source/core/text/porlay.txt
+// For Kashidas from sw/source/core/text/porlay.cxx
+
+#define IS_JOINING_GROUP(c, g) ( u_getIntPropertyValue( (c), 
UCHAR_JOINING_GROUP ) == U_JG_##g )
+#define isAinChar(c)IS_JOINING_GROUP((c), AIN)
+#define isAlefChar(c)   IS_JOINING_GROUP((c), ALEF)
+#define isDalChar(c)IS_JOINING_GROUP((c), DAL)
+#if U_ICU_VERSION_MAJOR_NUM >= 58
+#define isFehChar(c)   (IS_JOINING_GROUP((c), FEH) || 
IS_JOINING_GROUP((c), AFRICAN_FEH))
+#else
+#define isFehChar(c)IS_JOINING_GROUP((c), FEH)
+#endif
+#define isGafChar(c)IS_JOINING_GROUP((c), GAF)
+#define isHehChar(c)IS_JOINING_GROUP((c), HEH)
+#define isKafChar(c)IS_JOINING_GROUP((c), KAF)
+#define isLamChar(c)IS_JOINING_GROUP((c), LAM)
+#if U_ICU_VERSION_MAJOR_NUM >= 58
+#define isQafChar(c)   (IS_JOINING_GROUP((c), QAF) || 
IS_JOINING_GROUP((c), AFRICAN_QAF))
+#else
+#define isQafChar(c)IS_JOINING_GROUP((c), QAF)
+#endif
+#define isRehChar(c)IS_JOINING_GROUP((c), REH)
+#define isTahChar(c)IS_JOINING_GROUP((c), TAH)
+#define isTehMarbutaChar(c) IS_JOINING_GROUP((c), TEH_MARBUTA)
+#define isWawChar(c)IS_JOINING_GROUP((c), WAW)
+#define isSeenOrSadChar(c)  (IS_JOINING_GROUP((c), SAD) || 
IS_JOINING_GROUP((c), SEEN))
+
+// Beh and characters that behave like Beh in medial form.
+static bool isBehChar(sal_Unicode cCh)
+{
+bool bRet = false;
+switch (u_getIntPropertyValue(cCh, UCHAR_JOINING_GROUP))
+{
+case U_JG_BEH:
+case U_JG_NOON:
+#if U_ICU_VERSION_MAJOR_NUM >= 58
+case U_JG_AFRICAN_NOON:
+#endif
+case U_JG_NYA:
+case U_JG_YEH:
+case U_JG_FARSI_YEH:
+case U_JG_BURUSHASKI_YEH_BARREE:
+bRet = true;
+break;
+default:
+bRet = false;
+break;
+}
+
+return bRet;
+}
+
+// Yeh and characters that behave like Yeh in final form.
+static bool isYehChar(sal_Unicode cCh)
+{
+bool bRet = false;
+switch (u_getIntPropertyValue(cCh, UCHAR_JOINING_GROUP))
+{
+case U_JG_YEH:
+case U_JG_FARSI_YEH:
+case U_JG_YEH_BARREE:
+case U_JG_BURUSHASKI_YEH_BARREE:
+case U_JG_YEH_WITH_TAIL:
+bRet = true;
+break;
+default:
+bRet = false;
+break;
+}
+
+return bRet;
+}
+
+static bool isTransparentChar ( sal_Unicode cCh )
+{
+return u_getIntPropertyValue( cCh, UCHAR_JOINING_TYPE ) == 
U_JT_TRANSPARENT;
+}
+
+static bool lcl_IsLigature( sal_Unicode cCh, sal_Unicode cNextCh )
 {
 // Lam + Alef
-return ( 0x644 == cCh && 0x627 == cNextCh ) ||
-// Beh + Reh
-   ( 0x628 == cCh && 0x631 == cNextCh );
+return ( isLamChar ( cCh ) && isAlefChar ( cNextCh ));
 }
 
-static bool lcl_ConnectToPrev( sal_Unicode cCh, sal_Unicode cPrevCh )  // For 
Kashidas from sw/source/core/text/porlay.txt
+static bool lcl_ConnectToPrev( sal_Unicode cCh, sal_Unicode cPrevCh )
 {
-// Alef, Dal, Thal, Reh, Zain, and Waw do not connect to the left
-bool bRet = 0x627 != cPrevCh && 0x62F != cPrevCh && 0x630 != cPrevCh &&
-0x631 != cPrevCh && 0x632 != cPrevCh && 0x648 != cPrevCh;
+const int32_t nJoiningType = u_getIntPropertyValue( cPrevCh, 
UCHAR_JOINING_TYPE );
+bool bRet = nJoiningType != U_JT_RIGHT_JOINING && nJoiningType != 
U_JT_NON_JOINING;
 
 // check for ligatures cPrevChar + cChar
 if ( bRet )
@@ -2236,8 +2312,15 @@ void ImpEditEngine::ImpAdjustBlocks( ParaPortion* 
pParaPortion, EditLine* pLine,
 pLine->SetTextWidth( pLine->GetTextWidth() + 

[Libreoffice-commits] core.git: canvas/source drawinglayer/source editeng/inc editeng/source emfio/source include/drawinglayer include/editeng include/vcl sc/source sd/source sfx2/source svgio/source

2022-08-14 Thread Khaled Hosny (via logerrit)
 canvas/source/cairo/cairo_textlayout.cxx  |2 
 canvas/source/directx/dx_textlayout_drawhelper.cxx|1 
 canvas/source/vcl/textlayout.cxx  |1 
 drawinglayer/source/primitive2d/textbreakuphelper.cxx |   10 +
 drawinglayer/source/primitive2d/textdecoratedprimitive2d.cxx  |7 
 drawinglayer/source/primitive2d/textlayoutdevice.cxx  |6 
 drawinglayer/source/primitive2d/textprimitive2d.cxx   |   13 -
 drawinglayer/source/primitive2d/textstrikeoutprimitive2d.cxx  |1 
 drawinglayer/source/processor2d/vclprocessor2d.cxx|3 
 drawinglayer/source/tools/emfphelperdata.cxx  |4 
 drawinglayer/source/tools/wmfemfhelper.cxx|7 
 editeng/inc/editdoc.hxx   |4 
 editeng/source/editeng/editeng.cxx|3 
 editeng/source/editeng/impedit3.cxx   |   33 ++-
 editeng/source/items/svxfont.cxx  |   13 -
 editeng/source/outliner/outleeng.cxx  |5 
 editeng/source/outliner/outleeng.hxx  |3 
 editeng/source/outliner/outliner.cxx  |9 
 emfio/source/reader/mtftools.cxx  |4 
 include/drawinglayer/primitive2d/textdecoratedprimitive2d.hxx |2 
 include/drawinglayer/primitive2d/textlayoutdevice.hxx |3 
 include/drawinglayer/primitive2d/textprimitive2d.hxx  |8 
 include/editeng/editeng.hxx   |4 
 include/editeng/outliner.hxx  |7 
 include/editeng/svxfont.hxx   |3 
 include/vcl/metaact.hxx   |   11 -
 include/vcl/outdev.hxx|   12 -
 include/vcl/pdfwriter.hxx |1 
 include/vcl/rendercontext/SalLayoutFlags.hxx  |3 
 sc/source/ui/view/hintwin.cxx |4 
 sd/source/ui/view/sdview.cxx  |1 
 sfx2/source/control/thumbnailviewitem.cxx |1 
 svgio/source/svgreader/svgcharacternode.cxx   |2 
 svtools/source/control/ruler.cxx  |4 
 svx/source/diagram/IDiagramHelper.cxx |3 
 svx/source/svdraw/svdotextdecomposition.cxx   |   14 +
 svx/source/svdraw/svdotextpathdecomposition.cxx   |5 
 svx/source/tbxctrls/StylesPreviewWindow.cxx   |3 
 sw/source/core/inc/scriptinfo.hxx |2 
 sw/source/core/layout/paintfrm.cxx|1 
 sw/source/core/text/itradj.cxx|6 
 sw/source/core/text/porlay.cxx|6 
 sw/source/core/text/portxt.cxx|2 
 sw/source/core/txtnode/fntcache.cxx   |   32 ++-
 sw/source/uibase/docvw/HeaderFooterWin.cxx|2 
 sw/source/uibase/docvw/UnfloatTableButton.cxx |2 
 vcl/inc/ImplLayoutArgs.hxx|2 
 vcl/inc/impglyphitem.hxx  |4 
 vcl/inc/pdf/pdfwriter_impl.hxx|2 
 vcl/inc/sallayout.hxx |2 
 vcl/qa/cppunit/complextext.cxx|   37 ---
 vcl/qa/cppunit/svm/svmtest.cxx|2 
 vcl/qa/cppunit/text.cxx   |2 
 vcl/source/control/imp_listbox.cxx|2 
 vcl/source/filter/eps/eps.cxx |   12 -
 vcl/source/filter/svm/SvmConverter.cxx|2 
 vcl/source/gdi/CommonSalLayout.cxx|   98 +++---
 vcl/source/gdi/gdimtf.cxx |4 
 vcl/source/gdi/impglyphitem.cxx   |   25 --
 vcl/source/gdi/metaact.cxx|   12 +
 vcl/source/gdi/pdfwriter.cxx  |3 
 vcl/source/gdi/pdfwriter_impl.cxx |8 
 vcl/source/gdi/pdfwriter_impl2.cxx|2 
 vcl/source/gdi/textlayout.cxx |2 
 vcl/source/outdev/text.cxx|   42 ++--
 vcl/source/outdev/transparent.cxx |2 
 vcl/source/text/ImplLayoutArgs.cxx|   32 +++
 vcl/source/window/menuitemlist.cxx|2 
 vcl/source/window/status.cxx  |2 
 69 files changed, 346 insertions(+), 228 deletions(-)

New 

  1   2   >