Re: Machine learning is used in gerrit build

2023-08-18 Thread Baole Fang
Hi Noel,

The reason to choose it is that it is most likely to fail among all the
builds. If it fails, then there is no need to run others. If we choose a
job that is stable and likely to pass every patch, then all other builds
also need to be run, saving no computation.

I'm open to all advice on the build design.

Best,
Baole Fang

On Fri, Aug 18, 2023, 4:26 AM Noel Grandin  wrote:

> Hi
>
> Why are we fronting the ml jenkins job with the least reliable subjob?
>
> Surely we should be using the gcc job - which is faster and more reliable.
>
> Regards, Noel Grandin
>
>


Re: Machine learning is used in gerrit build

2023-08-18 Thread Noel Grandin
HI

My problem with this design assumes that the only thing we are optimising
for is reducing the load on the build servers.

But we are not a build-service organisation. Jenkins is a service that is
subordinate to the needs of developers.

And what developers need is feedback about which platforms their changes
are working or not working on.

Just recently I have to resume my builds 5 or 6 times to get past the ML
stage, only then to discover that I made a mistake that affected some other
platform, and then having to change things again, and restart the process.

I am really sorry you went to all this effort, but I do not think that this
ML plugin for jenkins is a good idea.

If I had known that this was in progress, I would have spoken up sooner.

If we need better jenkins throughput, TDF should just buy some more/better
hardware.

Regards, Noel Grandin


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

2023-08-18 Thread Caolán McNamara (via logerrit)
 officecfg/registry/data/org/openoffice/Office/Accelerators.xcu |   26 
+-
 1 file changed, 13 insertions(+), 13 deletions(-)

New commits:
commit 3614953bf1f4ada22247d04a772d1e3e1d361cef
Author: Caolán McNamara 
AuthorDate: Fri Aug 18 13:27:46 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 18:17:36 2023 +0200

tdf#151059 don't use Ctrl+Alt+3 etc by default for gallery

because on some keyboard layouts ctrl+alt is basically
equivalent to altgr and removes the possibility to enter
specific letters.

use Alt+3 etc because that 1-9 sequence seems free and
unproblematic vs other combinations.

If that doesn't work out then maybe we don't need one per sidebar entry
and just have a single entry to jump to the sidebar toolbar and a follow
up of press of 1-9 to activate a specific deck.

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

diff --git a/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu 
b/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
index a26b90b298d6..647b46927e5f 100644
--- a/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
@@ -344,25 +344,25 @@ Ctrl+Shift+e aka E_SHIFT_MOD1 under GTK/IBUS is for some 
emoji thing
   .uno:CommandPopup
 
   
-  
+  
 
   I10N SHORTCUTS - NO 
TRANSLATE
   .uno:SidebarDeck.PropertyDeck
 
   
-  
+  
 
   I10N SHORTCUTS - NO 
TRANSLATE
   .uno:SidebarDeck.StyleListDeck
 
   
-  
+  
 
   I10N SHORTCUTS - NO 
TRANSLATE
   .uno:SidebarDeck.GalleryDeck
 
   
-  
+  
 
   I10N SHORTCUTS - NO 
TRANSLATE
   .uno:SidebarDeck.NavigatorDeck
@@ -605,7 +605,7 @@ Ctrl+Shift+e aka E_SHIFT_MOD1 under GTK/IBUS is for some 
emoji thing
 
   
   
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.ScFunctionsDeck
@@ -1629,7 +1629,7 @@ Ctrl+Shift+e aka E_SHIFT_MOD1 under GTK/IBUS is for some 
emoji thing
 
   
   
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.ShapesDeck
@@ -2908,25 +2908,25 @@ Ctrl+Shift+e aka E_SHIFT_MOD1 under GTK/IBUS is for 
some emoji thing
 
   
   
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.ShapesDeck
   
 
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.SdSlideTransitionDeck
   
 
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.SdCustomAnimationDeck
   
 
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.SdMasterPagesDeck
@@ -6010,19 +6010,19 @@ Ctrl+Shift+e aka E_SHIFT_MOD1 under GTK/IBUS is for 
some emoji thing
 .uno:Grow
   
 
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.WriterPageDeck
   
 
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.InspectorDeck
   
 
-
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.A11yCheckDeck


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

2023-08-18 Thread Caolán McNamara (via logerrit)
 include/sfx2/lokhelper.hxx |3 +++
 sfx2/source/view/lokhelper.cxx |9 -
 2 files changed, 11 insertions(+), 1 deletion(-)

New commits:
commit 560b5112f6738f02a12adec7d5be04e2411c93b9
Author: Caolán McNamara 
AuthorDate: Thu Aug 17 20:22:02 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 18:32:18 2023 +0200

add a SfxLokHelper::notifyInvalidation which can take a 'part'

no change in behaviour intended

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

diff --git a/include/sfx2/lokhelper.hxx b/include/sfx2/lokhelper.hxx
index b46097938695..26b222bc8aa9 100644
--- a/include/sfx2/lokhelper.hxx
+++ b/include/sfx2/lokhelper.hxx
@@ -138,6 +138,9 @@ public:
 /// Emits a LOK_CALLBACK_DOCUMENT_SIZE_CHANGED for all views of the same 
document with the same part
 static void notifyPartSizeChangedAllViews(vcl::ITiledRenderable* pDoc, int 
nPart);
 /// Emits a LOK_CALLBACK_INVALIDATE_TILES, but tweaks it according to 
setOptionalFeatures() if needed.
+static void notifyInvalidation(SfxViewShell const* pThisView, int nPart, 
tools::Rectangle const *);
+/// Emits a LOK_CALLBACK_INVALIDATE_TILES, but tweaks it according to 
setOptionalFeatures() if needed
+/// uses the Part reported by pThisView
 static void notifyInvalidation(SfxViewShell const* pThisView, 
tools::Rectangle const *);
 /// Notifies all views with the given type and payload.
 static void notifyAllViews(int nType, const OString& rPayload);
diff --git a/sfx2/source/view/lokhelper.cxx b/sfx2/source/view/lokhelper.cxx
index 5bf6e8a1a2c4..cc3920fd0f35 100644
--- a/sfx2/source/view/lokhelper.cxx
+++ b/sfx2/source/view/lokhelper.cxx
@@ -610,12 +610,19 @@ void SfxLokHelper::notifyWindow(const SfxViewShell* 
pThisView,
 }
 
 void SfxLokHelper::notifyInvalidation(SfxViewShell const* pThisView, 
tools::Rectangle const* pRect)
+{
+// -1 means all parts
+const int nPart = comphelper::LibreOfficeKit::isPartInInvalidation() ? 
pThisView->getPart() : INT_MIN;
+SfxLokHelper::notifyInvalidation(pThisView, nPart, pRect);
+}
+
+void SfxLokHelper::notifyInvalidation(SfxViewShell const* pThisView, const int 
nInPart, tools::Rectangle const* pRect)
 {
 if (DisableCallbacks::disabled())
 return;
 
 // -1 means all parts
-const int nPart = comphelper::LibreOfficeKit::isPartInInvalidation() ? 
pThisView->getPart() : INT_MIN;
+const int nPart = comphelper::LibreOfficeKit::isPartInInvalidation() ? 
nInPart : INT_MIN;
 const int nMode = pThisView->getEditMode();
 pThisView->libreOfficeKitViewInvalidateTilesCallback(pRect, nPart, nMode);
 }


Re: Machine learning is used in gerrit build

2023-08-18 Thread Kohei Yoshida

Hello,

On 09.08.2023 10:57, Baole Fang wrote:


Feel free to contact me if you have any questions!


Just out of curiosity, what is the overall accuracy of your model?  Do 
you have a confusion matrix or something similar that shows the 
performance of your model?


Thanks,
Kohei


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

2023-08-18 Thread Michael Stahl (via logerrit)
 sw/qa/extras/layout/data/linked_frames_section_bug.odt |binary
 sw/qa/extras/layout/layout3.cxx|   18 +
 sw/source/core/layout/sectfrm.cxx  |1 
 3 files changed, 19 insertions(+)

New commits:
commit 987fe1175de2db53235cc6f2441335fcc3548d64
Author: Michael Stahl 
AuthorDate: Fri Aug 18 16:44:16 2023 +0200
Commit: Michael Stahl 
CommitDate: Fri Aug 18 18:55:57 2023 +0200

tdf#156419 sw: layout: don't prevent moving between linked flys

... when there is a page break on the next page.

Of course you can put a section into a fly, and then link multiple flys.

(regression from commit 325fe7ab507fd8f2ca17a3db32181edf30169525)

Change-Id: I5cb854fc7ee5caa2af4e04f4da2d8a8083c0d007
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155842
Tested-by: Jenkins
Reviewed-by: Michael Stahl 

diff --git a/sw/qa/extras/layout/data/linked_frames_section_bug.odt 
b/sw/qa/extras/layout/data/linked_frames_section_bug.odt
new file mode 100644
index ..639332ad5516
Binary files /dev/null and 
b/sw/qa/extras/layout/data/linked_frames_section_bug.odt differ
diff --git a/sw/qa/extras/layout/layout3.cxx b/sw/qa/extras/layout/layout3.cxx
index 9c75ff31a322..4ddb46b025b9 100644
--- a/sw/qa/extras/layout/layout3.cxx
+++ b/sw/qa/extras/layout/layout3.cxx
@@ -1228,6 +1228,24 @@ CPPUNIT_TEST_FIXTURE(SwLayoutWriter3, testTdf156725)
 
"/root/page[2]/body/txt/anchored/fly/column[2]/body/section/column[2]/body/txt",
 1);
 }
 
+CPPUNIT_TEST_FIXTURE(SwLayoutWriter3, testTdf156419)
+{
+createSwDoc("linked_frames_section_bug.odt");
+
+xmlDocUniquePtr pXmlDoc = parseLayoutDump();
+assertXPath(pXmlDoc, "/root/page", 2);
+// there are 2 flys on page 1, and 1 on page 2, all linked
+assertXPath(pXmlDoc, 
"/root/page[1]/body/txt/anchored/fly[1]/section/column", 2);
+assertXPath(pXmlDoc, 
"/root/page[1]/body/txt/anchored/fly[1]/section/column[1]/body/txt", 11);
+assertXPath(pXmlDoc, 
"/root/page[1]/body/txt/anchored/fly[1]/section/column[2]/body/txt", 11);
+assertXPath(pXmlDoc, 
"/root/page[1]/body/txt/anchored/fly[2]/section/column", 2);
+assertXPath(pXmlDoc, 
"/root/page[1]/body/txt/anchored/fly[2]/section/column[1]/body/txt", 12);
+assertXPath(pXmlDoc, 
"/root/page[1]/body/txt/anchored/fly[2]/section/column[2]/body/txt", 12);
+assertXPath(pXmlDoc, 
"/root/page[2]/body/txt/anchored/fly[1]/section/column", 2);
+assertXPath(pXmlDoc, 
"/root/page[2]/body/txt/anchored/fly[1]/section/column[1]/body/txt", 2);
+assertXPath(pXmlDoc, 
"/root/page[2]/body/txt/anchored/fly[1]/section/column[2]/body/txt", 1);
+}
+
 CPPUNIT_TEST_FIXTURE(SwLayoutWriter3, testTdf145826)
 {
 createSwDoc("tdf145826.odt");
diff --git a/sw/source/core/layout/sectfrm.cxx 
b/sw/source/core/layout/sectfrm.cxx
index 9023550cacd1..7afcb6b89800 100644
--- a/sw/source/core/layout/sectfrm.cxx
+++ b/sw/source/core/layout/sectfrm.cxx
@@ -1758,6 +1758,7 @@ SwLayoutFrame *SwFrame::GetNextSctLeaf( MakePageType 
eMakePage )
 // creating / moving the cell frame.
 // It doesn't make sense to move to a page that starts with break?
 if (pNxtPg != FindPageFrame() // tdf#156725 not between columns!
+&& !FindFlyFrame() // tdf#156419 linked fly frames don't care!
 && (WrongPageDesc(pNxtPg) || HasPageBreakBefore(*pNxtPg))
 && !bLayLeafTableAllowed)
 {


[Libreoffice-commits] core.git: Branch 'distro/collabora/co-23.05' - sw/CppunitTest_sw_core_text.mk sw/qa sw/source

2023-08-18 Thread Miklos Vajna (via logerrit)
 sw/CppunitTest_sw_core_text.mk |1 
 sw/qa/core/text/data/floattable-leftover-para-portion.docx |binary
 sw/qa/core/text/porrst.cxx |   57 +
 sw/source/core/text/porrst.cxx |2 
 4 files changed, 59 insertions(+), 1 deletion(-)

New commits:
commit 352bcab5bc59ef3e0042328503c6535e28dce931
Author: Miklos Vajna 
AuthorDate: Wed Aug 16 09:34:49 2023 +0200
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 15:40:52 2023 +0200

tdf#156682 sw floattable: fix missing del of master anchor para por on split

Pressing "enter" at the end of B1 in the bugdoc resulted in a split
floating table, which had the anchor text duplicated (both on page 1 and
page2).

What happened is that the master anchor frame didn't have its paragraph
portion cleared, the follow anchor frame had a correct frame offset (of
0).

Fix the problem in SwTextFrame::FormatEmpty(), similar to what commit
00b9b4791079c2dc26b1ed4c123450cabf7d (sw: call FormatEmpty() in
SwTextFrame::Format() for split fly masters, 2023-02-09), did; here what
we want to avoid is setting some properties on the paragraph preventing
the deletion of non-last anchor paragraph portions.

As a side effect, this also fixes the hang from the bugreport, after
pressing "enter" 5 times at the end of B1.

(cherry picked from commit 1cf29168840f84c2e946e2678b99988e83503c96)

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

diff --git a/sw/CppunitTest_sw_core_text.mk b/sw/CppunitTest_sw_core_text.mk
index 477e25b91612..37b2fa566094 100644
--- a/sw/CppunitTest_sw_core_text.mk
+++ b/sw/CppunitTest_sw_core_text.mk
@@ -17,6 +17,7 @@ $(eval $(call 
gb_CppunitTest_add_exception_objects,sw_core_text, \
 sw/qa/core/text/frmform \
 sw/qa/core/text/itratr \
 sw/qa/core/text/itrform2 \
+sw/qa/core/text/porrst \
 sw/qa/core/text/text \
 ))
 
diff --git a/sw/qa/core/text/data/floattable-leftover-para-portion.docx 
b/sw/qa/core/text/data/floattable-leftover-para-portion.docx
new file mode 100644
index ..51eee56cf003
Binary files /dev/null and 
b/sw/qa/core/text/data/floattable-leftover-para-portion.docx differ
diff --git a/sw/qa/core/text/porrst.cxx b/sw/qa/core/text/porrst.cxx
new file mode 100644
index ..1c343dc0d9f2
--- /dev/null
+++ b/sw/qa/core/text/porrst.cxx
@@ -0,0 +1,57 @@
+/* -*- 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 
+
+#include 
+
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+#include 
+
+namespace
+{
+/// Covers sw/source/core/text/porrst.cxx fixes.
+class Test : public SwModelTestBase
+{
+public:
+Test()
+: SwModelTestBase("/sw/qa/core/text/data/")
+{
+}
+};
+
+CPPUNIT_TEST_FIXTURE(Test, testFloattableLeftoverParaPortion)
+{
+// Given a document with a multi-page floating table, the anchor of the 
table has some text:
+createSwDoc("floattable-leftover-para-portion.docx");
+
+// When laying out that document:
+calcLayout();
+
+// Then make sure all anchor text goes to the second page:
+SwDoc* pDoc = getSwDoc();
+SwRootFrame* pLayout = pDoc->getIDocumentLayoutAccess().GetCurrentLayout();
+auto pPage = dynamic_cast(pLayout->Lower());
+CPPUNIT_ASSERT(pPage);
+SwFrame* pBody = pPage->FindBodyCont();
+SwFrame* pPara1 = pBody->GetLower();
+auto pPara2 = pPara1->GetNext()->DynCastTextFrame();
+CPPUNIT_ASSERT(pPara2);
+// Without the accompanying fix in place, this test would have failed, the 
first page's anchor
+// also had some (duplicated) anchor text.
+CPPUNIT_ASSERT(!pPara2->GetPara());
+}
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/core/text/porrst.cxx b/sw/source/core/text/porrst.cxx
index 4bdc0fa40f77..9c04fc0458c6 100644
--- a/sw/source/core/text/porrst.cxx
+++ b/sw/source/core/text/porrst.cxx
@@ -413,7 +413,7 @@ bool SwTextFrame::FormatEmpty()
 // Split fly frames: non-last parts of the anchor want this optimization 
to clear the old
 // content.
 bool bHasNonLastSplitFlyDrawObj = HasNonLastSplitFlyDrawObj();
-if ((HasFollow() && !bHasNonLastSplitFlyDrawObj) || GetMergedPara() || 
GetTextNodeFirst()->GetpSwpHints() ||
+if ((HasFollow() && !bHasNonLastSplitFlyDrawObj) || GetMergedPara() || 
(GetTextNodeFirst()->GetpSwpHints() && !bHasNonLastSplitFlyDrawObj) ||
 nullptr != GetTextNodeForParaProps()->GetNumRule() ||
 

Cool! LibreOffice is free, open source and very good office suite!

2023-08-18 Thread Turritopsis Dohrnii Teo En Ming
Subject: Cool! LibreOffice is free, open source and very good office suite!

Good day from Singapore,

I have problems with both of my Microsoft Office Home and Business
2019 licenses. So I had installed a trial version of Microsoft Office
on my refurbished Aftershock gaming laptop but it expired on 11 Aug
2023. I couldn't use it any more.

I needed an office suite but I do not want to spend SGD$300++
purchasing Microsoft Office licenses. Then I thought of LibreOffice. I
quickly downloaded and installed LibreOffice 7.5.5.2.

With LibreOffice installed, I used LibreOffice Calc to edit and save
my Excel worksheet. When I opened the edited Excel worksheet using
Microsoft Excel on my Samsung Galaxy A32 5G Android phone, the Excel
worksheet didn't look significantly different. In fact, the Excel
worksheet edited by LibreOffice Calc looks the same as if it had been
edited by Microsoft Excel.

I have decided to uninstall the trial version of Microsoft Office Home
and Business 2019 (as it doesn't work anymore) and use LibreOffice
permanently!

By the way, does LibreOffice have the equivalent of Microsoft Outlook?

Thank you LibreOffice development team!

Regards,

Mr. Turritopsis Dohrnii Teo En Ming
Targeted Individual in Singapore
Blogs:
https://tdtemcerts.blogspot.com
https://tdtemcerts.wordpress.com
GIMP also stands for Government-Induced Medical Problems.


[Libreoffice-commits] core.git: Branch 'feature/cib_contract57d' - sw/qa sw/source

2023-08-18 Thread Michael Stahl (via logerrit)
 sw/qa/extras/layout/data/tdf156725.fodt |  163 
 sw/qa/extras/layout/layout.cxx  |   21 
 sw/source/core/layout/sectfrm.cxx   |4 
 3 files changed, 187 insertions(+), 1 deletion(-)

New commits:
commit 807c2254ccd1a02e7af5a944f52948ef343ad384
Author: Michael Stahl 
AuthorDate: Thu Aug 17 18:32:35 2023 +0200
Commit: Michael Stahl 
CommitDate: Fri Aug 18 15:47:59 2023 +0200

tdf#156725 sw: layout: don't prevent moving between columns ...

... on the same page when there is a page break on the next page.

The existing, presumably pointless/always-false in case of same page,
check of WrongPageDesc() let me assume that this would only be reached
if the current page and the target page are different, but that was a
mistake.

(regression from commit 325fe7ab507fd8f2ca17a3db32181edf30169525)

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

Change-Id: I4df53b77bed006095c34976011aa0cd5e12879e7

diff --git a/sw/qa/extras/layout/data/tdf156725.fodt 
b/sw/qa/extras/layout/data/tdf156725.fodt
new file mode 100644
index ..9f60e7011954
--- /dev/null
+++ b/sw/qa/extras/layout/data/tdf156725.fodt
@@ -0,0 +1,163 @@
+
+http://www.w3.org/TR/css3-text/; 
xmlns:grddl="http://www.w3.org/2003/g/data-view#; 
xmlns:xhtml="http://www.w3.org/1999/xhtml; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; 
xmlns:xsd="http://www.w3.org/2001/XMLSchema; 
xmlns:xforms="http://www.w3.org/2002/xforms; 
xmlns:dom="http://www.w3.org/2001/xml-events; 
xmlns:script="urn:oasis:names:tc:opendocument:xmlns:script:1.0" 
xmlns:form="urn:oasis:names:tc:opendocument:xmlns:form:1.0" 
xmlns:math="http://www.w3.org/1998/Math/MathML; 
xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0" 
xmlns:ooo="http://openoffice.org/2004/office; 
xmlns:fo="urn:oasis:names:tc:opendocument:xmlns:xsl-fo-compatible:1.0" 
xmlns:config="urn:oasis:names:tc:opendocument:xmlns:config:1.0" 
xmlns:ooow="http://openoffice.org/2004/writer; 
xmlns:xlink="http://www.w3.org/1999/xlink; 
xmlns:drawooo="http://openoffice.org/2010/draw; 
xmlns:oooc="http://openoffice.org/2004/calc; 
xmlns:dc="http://purl.org/dc/elements/1.1/; xmlns:c
 alcext="urn:org:documentfoundation:names:experimental:calc:xmlns:calcext:1.0" 
xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" 
xmlns:text="urn:oasis:names:tc:opendocument:xmlns:text:1.0" 
xmlns:of="urn:oasis:names:tc:opendocument:xmlns:of:1.2" 
xmlns:tableooo="http://openoffice.org/2009/table; 
xmlns:draw="urn:oasis:names:tc:opendocument:xmlns:drawing:1.0" 
xmlns:dr3d="urn:oasis:names:tc:opendocument:xmlns:dr3d:1.0" 
xmlns:rpt="http://openoffice.org/2005/report; 
xmlns:formx="urn:openoffice:names:experimental:ooxml-odf-interop:xmlns:form:1.0"
 xmlns:svg="urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0" 
xmlns:chart="urn:oasis:names:tc:opendocument:xmlns:chart:1.0" 
xmlns:officeooo="http://openoffice.org/2009/office; 
xmlns:table="urn:oasis:names:tc:opendocument:xmlns:table:1.0" 
xmlns:field="urn:openoffice:names:experimental:ooo-ms-interop:xmlns:field:1.0" 
xmlns:number="urn:oasis:names:tc:opendocument:xmlns:datastyle:1.0" 
xmlns:meta="urn:oasis:names:tc:opendocument:xmlns:
 meta:1.0" 
xmlns:loext="urn:org:documentfoundation:names:experimental:office:xmlns:loext:1.0"
 office:version="1.3" office:mimetype="application/vnd.oasis.opendocument.text">
+ 
2023-08-17T18:14:28.1153022102023-08-17T18:20:06.109127302PT4M30S2LibreOfficeDev/24.2.0.0.alpha0$Linux_X86_64
 
LibreOffice_project/79452241ad33f9eaace2ba8bd1336be69c99ed4d
+ 
+  
+  
+  
+ 
+ 
+  
+   
+   
+
+   
+   
+  
+  
+   
+   
+  
+  
+   
+  
+  
+   
+  
+  
+  
+  
+   
+  
+  
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+  
+  
+  
+  
+  
+   
+
+
+
+
+
+
+
+
+
+
+
+
+   
+  
+ 
+ 
+  
+   
+  
+  
+   
+  
+  
+   
+
+ 
+ 
+
+   
+  
+  
+   
+
+ 
+ 
+
+   
+  
+  
+   
+
+   
+   
+   
+  
+  
+   
+  
+ 
+ 
+  
+ 
+ 
+  
+   
+
+
+
+
+
+   
+   
+   
+ 
+  
+   a
+   b
+   c
+   d
+  
+ 
+
+  
+ 
+
\ No newline at end of file
diff --git a/sw/qa/extras/layout/layout.cxx b/sw/qa/extras/layout/layout.cxx
index f72715fcba43..def657eec623 100644
--- a/sw/qa/extras/layout/layout.cxx
+++ b/sw/qa/extras/layout/layout.cxx
@@ -3672,6 +3672,27 @@ CPPUNIT_TEST_FIXTURE(SwLayoutWriter, testTdf156724)
 assertXPath(pXmlDoc, "/root/page", 2);
 }
 
+CPPUNIT_TEST_FIXTURE(SwLayoutWriter, testTdf156725)
+{
+

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

2023-08-18 Thread Caolán McNamara (via logerrit)
 filter/source/svg/svgwriter.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 30febe5fc4ac3117ff5dabcd1fec4acfc7b759d8
Author: Caolán McNamara 
AuthorDate: Wed Aug 16 15:04:58 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 18:24:04 2023 +0200

cool#6616 compare graphics for equality with operator==...

instead of checking equal CheckSums.

For better or worse while Bitmap::operator== also considers equal
CheckSums as equal Bitmaps operator== can quickly return true if
both Bitmaps share the same underlying SalBitmap which is nearly
always true in the example here

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

diff --git a/filter/source/svg/svgwriter.cxx b/filter/source/svg/svgwriter.cxx
index 4f21f0493b06..62aa05115fbb 100644
--- a/filter/source/svg/svgwriter.cxx
+++ b/filter/source/svg/svgwriter.cxx
@@ -2916,7 +2916,7 @@ void SVGActionWriter::ImplWriteBmp( const BitmapEx& 
rBmpEx,
 if (aGraphic.GetType() == GraphicType::Bitmap)
 {
 const BitmapEx& rGraphicBitmap = aGraphic.GetBitmapExRef();
-if (rGraphicBitmap.GetChecksum() == rBmpEx.GetChecksum())
+if (rGraphicBitmap == rBmpEx)
 {
 bool bPNG = false;
 GfxLink aGfxLink = aGraphic.GetGfxLink();


[Libreoffice-commits] core.git: 2 commits - desktop/source include/ucbhelper ucbhelper/source ucb/source

2023-08-18 Thread Michael Stahl (via logerrit)
 desktop/source/app/crashreport.cxx |6 ++---
 include/ucbhelper/proxydecider.hxx |6 ++---
 ucb/source/ucp/cmis/cmis_content.cxx   |5 
 ucb/source/ucp/cmis/cmis_repo_content.cxx  |5 
 ucb/source/ucp/webdav-curl/CurlSession.cxx |   34 +++--
 ucb/source/ucp/webdav-curl/CurlSession.hxx |4 +--
 ucbhelper/source/client/proxydecider.cxx   |   17 ++
 7 files changed, 40 insertions(+), 37 deletions(-)

New commits:
commit f88af95552f9b46e1714964d84c447327b50ed40
Author: Michael Stahl 
AuthorDate: Fri Aug 18 15:09:43 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Aug 18 18:33:27 2023 +0200

ucbhelper,ucb,desktop: InternetProxyServer is problematic

It turns out that every single client of InternetProxyDecider simply
concatenates the 2 members of InternetProxyServer into a single string
and passes it on to curl_easy_setopt(CURLOPT_PROXY), which will happily
take a URL including scheme and everything.

It turns out that the awful GetUnixSystemProxy() tries to cut off the
scheme in a terrible way, but GetPACProxy() does no such thing and
WINHTTP_PROXY_INFO::lpszProxy may or may not contain scheme in its
entries; fix this to only separate the port and leave the rest alone.

So why do we need a InternetProxyServer struct?  Because officecfg has
separate entries that correspond to its members, and so
InternetProxyDecider gets separate events on its listener interface when
any of them changes, which is easiest to handle if it stores these
separately.

So just return a concatenated URL with or without scheme in getProxy().

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

diff --git a/desktop/source/app/crashreport.cxx 
b/desktop/source/app/crashreport.cxx
index 59a2aa771b32..680492b3b80a 100644
--- a/desktop/source/app/crashreport.cxx
+++ b/desktop/source/app/crashreport.cxx
@@ -139,7 +139,7 @@ void CrashReporter::writeCommonInfo()
 static constexpr OUStringLiteral url = u"crashreport.libreoffice.org";
 const sal_Int32 port = 443;
 
-const ucbhelper::InternetProxyServer proxy_server = 
proxy_decider.getProxy(protocol, url, port);
+const OUString proxy_server = proxy_decider.getProxy(protocol, url, port);
 
 // save the new Keys
 vmaKeyValues atlast = maKeyValues;
@@ -152,9 +152,9 @@ void CrashReporter::writeCommonInfo()
 addKeyValue("BuildID", utl::Bootstrap::getBuildIdData(""), AddItem);
 addKeyValue("URL", protocol + "://" + url + "/submit/", AddItem);
 
-if (!proxy_server.aName.isEmpty())
+if (!proxy_server.isEmpty())
 {
-addKeyValue("Proxy", proxy_server.aName + ":" + 
OUString::number(proxy_server.nPort), AddItem);
+addKeyValue("Proxy", proxy_server, AddItem);
 }
 
 // write the new keys at the end
diff --git a/include/ucbhelper/proxydecider.hxx 
b/include/ucbhelper/proxydecider.hxx
index 9a1abad3264a..3fe3025833ce 100644
--- a/include/ucbhelper/proxydecider.hxx
+++ b/include/ucbhelper/proxydecider.hxx
@@ -116,10 +116,10 @@ public:
   * If host is not empty this parameter must always contain a valid
   * port number, for instance the default port for the requested
   * protocol(i.e. 80 or http).
-  * @return a InternetProxyServer struct. If member aName of the
-  * InternetProxyServer is empty no proxy server is to be used.
+  * @return an URL, with or without scheme.
+  * If empty no proxy server is to be used.
   */
-InternetProxyServer
+OUString
 getProxy( const OUString & rProtocol,
   const OUString & rHost,
   sal_Int32 nPort ) const;
diff --git a/ucb/source/ucp/cmis/cmis_content.cxx 
b/ucb/source/ucp/cmis/cmis_content.cxx
index 815d89502f2c..b5ad2cb738ce 100644
--- a/ucb/source/ucp/cmis/cmis_content.cxx
+++ b/ucb/source/ucp/cmis/cmis_content.cxx
@@ -304,11 +304,8 @@ namespace cmis
 // Set the proxy if needed. We are doing that all times as the proxy 
data shouldn't be cached.
 ucbhelper::InternetProxyDecider aProxyDecider( m_xContext );
 INetURLObject aBindingUrl( m_aURL.getBindingUrl( ) );
-const ucbhelper::InternetProxyServer& rProxy = aProxyDecider.getProxy(
+const OUString sProxy = aProxyDecider.getProxy(
 INetURLObject::GetScheme( aBindingUrl.GetProtocol( ) ), 
aBindingUrl.GetHost(), aBindingUrl.GetPort() );
-OUString sProxy = rProxy.aName;
-if ( rProxy.nPort > 0 )
-sProxy += ":" + OUString::number( rProxy.nPort );
 libcmis::SessionFactory::setProxySettings( OUSTR_TO_STDSTR( sProxy ), 
std::string(), std::string(), std::string() );
 
 // Look for a cached session, key is binding url + repo id
diff --git 

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

2023-08-18 Thread Caolán McNamara (via logerrit)
 sc/source/ui/inc/gridwin.hxx   |2 ++
 sc/source/ui/view/gridwin4.cxx |   10 --
 2 files changed, 10 insertions(+), 2 deletions(-)

New commits:
commit 28d41e60416ef10ae2a59d1c4ad34a2eb75dc408
Author: Caolán McNamara 
AuthorDate: Thu Aug 17 20:31:55 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 18:32:56 2023 +0200

add a LogicInvalidatePart which can take a 'part'

no change in behaviour intended

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

diff --git a/sc/source/ui/inc/gridwin.hxx b/sc/source/ui/inc/gridwin.hxx
index ddc9815c2052..979767c6801b 100644
--- a/sc/source/ui/inc/gridwin.hxx
+++ b/sc/source/ui/inc/gridwin.hxx
@@ -320,6 +320,8 @@ class SAL_DLLPUBLIC_RTTI ScGridWindow : public 
vcl::DocWindow, public DropTarget
 voidSetupInitialPageBreaks(const ScDocument& rDoc, SCTAB nTab);
 DECL_DLLPRIVATE_LINK(InitiatePageBreaksTimer, Timer*, void);
 
+voidLogicInvalidatePart(const tools::Rectangle* pRectangle, 
int nPart);
+
 protected:
 virtual voidPrePaint(vcl::RenderContext& rRenderContext) override;
 virtual voidPaint(vcl::RenderContext& rRenderContext, const 
tools::Rectangle& rRect) override;
diff --git a/sc/source/ui/view/gridwin4.cxx b/sc/source/ui/view/gridwin4.cxx
index bc2521531a37..a86902d57944 100644
--- a/sc/source/ui/view/gridwin4.cxx
+++ b/sc/source/ui/view/gridwin4.cxx
@@ -1727,7 +1727,7 @@ void ScGridWindow::PaintTile( VirtualDevice& rDevice,
 }
 }
 
-void ScGridWindow::LogicInvalidate(const tools::Rectangle* pRectangle)
+void ScGridWindow::LogicInvalidatePart(const tools::Rectangle* pRectangle, int 
nPart)
 {
 tools::Rectangle aRectangle;
 tools::Rectangle* pResultRectangle;
@@ -1759,7 +1759,13 @@ void ScGridWindow::LogicInvalidate(const 
tools::Rectangle* pRectangle)
 }
 
 ScTabViewShell* pViewShell = mrViewData.GetViewShell();
-SfxLokHelper::notifyInvalidation(pViewShell, pResultRectangle);
+SfxLokHelper::notifyInvalidation(pViewShell, nPart, pResultRectangle);
+}
+
+void ScGridWindow::LogicInvalidate(const tools::Rectangle* pRectangle)
+{
+ScTabViewShell* pViewShell = mrViewData.GetViewShell();
+LogicInvalidatePart(pRectangle, pViewShell->getPart());
 }
 
 void ScGridWindow::SetCellSelectionPixel(int nType, int nPixelX, int nPixelY)


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

2023-08-18 Thread Noel Grandin (via logerrit)
 uui/source/iahndl.cxx |   13 ++---
 1 file changed, 10 insertions(+), 3 deletions(-)

New commits:
commit 40b2464bf346f1aeda44f719a301e2e60f4d0fd0
Author: Noel Grandin 
AuthorDate: Thu Aug 17 14:06:23 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Aug 18 18:34:13 2023 +0200

ignore exception in UUIInteractionHelper::handlerequest

otherwise I sometimes get a crash during
make UITest_sw_options
when an exception in configmgr goes up the stack and hits
the noexcept lamba in the vcl/
stuff that executes Application::PostUserEvent stuff

#1  in configmgr::RootAccess::getNode()
#2  in configmgr::Access::isValue()
#3  in configmgr::configuration_provider::(anonymous 
namespace)::Service::createInstanceWithArguments
#4  in non-virtual thunk to configmgr::configuration_provider::(anonymous 
namespace)::Service::createInstanceWithArguments
#5  in UUIInteractionHelper::getInteractionHandlerList
#6  in UUIInteractionHelper::tryOtherInteractionHandler
#6  in UUIInteractionHelper::handleRequest_impl
#7  in UUIInteractionHelper::handlerequest

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

diff --git a/uui/source/iahndl.cxx b/uui/source/iahndl.cxx
index dfa3b96742d6..b6e88bcd3e40 100644
--- a/uui/source/iahndl.cxx
+++ b/uui/source/iahndl.cxx
@@ -141,9 +141,16 @@ void UUIInteractionHelper::handlerequest(
 = static_cast< UUIInteractionHelper * >(pInteractionHelper);
 bool bDummy = false;
 OUString aDummy;
-pHND->bHandled
-= pUUI->handleRequest_impl(pHND->m_rRequest, false, bDummy, aDummy);
-pHND->set();
+try
+{
+pHND->bHandled
+= pUUI->handleRequest_impl(pHND->m_rRequest, false, bDummy, 
aDummy);
+pHND->set();
+}
+catch (css::uno::Exception&)
+{
+TOOLS_WARN_EXCEPTION("uui", "");
+}
 }
 
 bool


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

2023-08-18 Thread jucasaca (via logerrit)
 connectivity/source/parse/sqlbison.y |2 ++
 connectivity/source/parse/sqlflex.l  |1 +
 2 files changed, 3 insertions(+)

New commits:
commit ccf30787a701860d19941a1d36ad3d1db2fb2e06
Author: jucasaca 
AuthorDate: Sun Jul 30 19:20:29 2023 +0200
Commit: Julien Nabet 
CommitDate: Fri Aug 18 08:23:18 2023 +0200

tdf#156534 Add Firebird's DATEADD function to SQL parser

Add DATEADD function to SQL parser, so the function can be used in
Query's Design and can produce  editable queries

Change-Id: I14f9d88f6a96021b0f9199fd79df18c8f4bb5ea3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155071
Tested-by: Jenkins
Reviewed-by: Julien Nabet 

diff --git a/connectivity/source/parse/sqlbison.y 
b/connectivity/source/parse/sqlbison.y
index fe3b32b79ef6..00666ed87a09 100644
--- a/connectivity/source/parse/sqlbison.y
+++ b/connectivity/source/parse/sqlbison.y
@@ -156,6 +156,7 @@ using namespace connectivity;
 %token  SQL_TOKEN_DAYNAME  SQL_TOKEN_DAYOFMONTH  
SQL_TOKEN_DAYOFWEEK  SQL_TOKEN_DAYOFYEAR SQL_TOKEN_EXTRACT
 %token  SQL_TOKEN_HOUR SQL_TOKEN_MILLISECOND SQL_TOKEN_MINUTE  
SQL_TOKEN_MONTH  SQL_TOKEN_MONTHNAME SQL_TOKEN_NOW SQL_TOKEN_QUARTER 
SQL_TOKEN_DATEDIFF
 %token  SQL_TOKEN_SECOND SQL_TOKEN_TIMESTAMPADD 
SQL_TOKEN_TIMESTAMPDIFF SQL_TOKEN_TIMEVALUE SQL_TOKEN_WEEK SQL_TOKEN_WEEKDAY 
SQL_TOKEN_YEAR SQL_TOKEN_YEARDAY
+%token  SQL_TOKEN_DATEADD
 
 /* numeric functions */
 %token  SQL_TOKEN_ABS SQL_TOKEN_ACOS SQL_TOKEN_ASIN SQL_TOKEN_ATAN 
SQL_TOKEN_ATAN2 SQL_TOKEN_CEILING
@@ -1978,6 +1979,7 @@ date_function_1Argument:
 date_function:
SQL_TOKEN_TIMESTAMPADD
|   SQL_TOKEN_TIMESTAMPDIFF
+   |   SQL_TOKEN_DATEADD
;
 numeric_function_0Argument:
SQL_TOKEN_PI
diff --git a/connectivity/source/parse/sqlflex.l 
b/connectivity/source/parse/sqlflex.l
index 34a4067ea21c..44daeb236bf7 100644
--- a/connectivity/source/parse/sqlflex.l
+++ b/connectivity/source/parse/sqlflex.l
@@ -186,6 +186,7 @@ CURSOR  {SQL_NEW_KEYWORD(SQL_TOKEN_CURSOR);  }
 
 D   {SQL_NEW_KEYWORD(SQL_TOKEN_D);  }
 DATE{SQL_NEW_KEYWORD(SQL_TOKEN_DATE);  }
+DATEADD {SQL_NEW_KEYWORD(SQL_TOKEN_DATEADD);  }
 DATEDIFF{SQL_NEW_KEYWORD(SQL_TOKEN_DATEDIFF);  }
 DATEVALUE   {SQL_NEW_KEYWORD(SQL_TOKEN_DATEVALUE);  }
 DAY {SQL_NEW_KEYWORD(SQL_TOKEN_DAY);  }


[Libreoffice-commits] core.git: basic/source comphelper/source embedserv/source oox/source scripting/source sc/source sdext/source sd/source sfx2/source sot/source svtools/source svx/source sw/source

2023-08-18 Thread Noel Grandin (via logerrit)
 basic/source/classes/sbxmod.cxx |4 
 comphelper/source/container/embeddedobjectcontainer.cxx |6 -
 comphelper/source/misc/instancelocker.cxx   |7 -
 comphelper/source/misc/storagehelper.cxx|4 
 embedserv/source/embed/docholder.cxx|   31 ---
 oox/source/drawingml/chart/converterbase.cxx|8 -
 oox/source/ole/oleobjecthelper.cxx  |8 -
 oox/source/ole/vbamodule.cxx|8 -
 sc/source/filter/excel/xichart.cxx  |   13 ---
 sc/source/filter/ftools/fapihelper.cxx  |4 
 sc/source/filter/oox/numberformatsbuffer.cxx|8 -
 sc/source/filter/oox/pivotcachebuffer.cxx   |8 -
 sc/source/filter/oox/pivottablebuffer.cxx   |8 -
 sc/source/filter/oox/worksheethelper.cxx|9 --
 sc/source/filter/xml/xmlwrap.cxx|   11 +-
 sc/source/ui/docshell/docsh.cxx |8 -
 sc/source/ui/docshell/docsh4.cxx|8 -
 sc/source/ui/vba/vbaeventshelper.cxx|   16 ---
 scripting/source/vbaevents/eventhelper.cxx  |9 --
 sd/source/console/PresenterHelper.cxx   |   15 ---
 sd/source/console/PresenterScreen.cxx   |4 
 sd/source/console/PresenterSlideShowView.cxx|4 
 sd/source/core/stlsheet.cxx |6 -
 sd/source/filter/eppt/epptso.cxx|8 -
 sd/source/ui/annotations/annotationmanager.cxx  |   22 -
 sd/source/ui/slideshow/slideshowviewimpl.cxx|7 -
 sdext/source/minimizer/impoptimizer.cxx |   18 ++--
 sdext/source/minimizer/optimizationstats.cxx|6 -
 sfx2/source/dialog/filedlghelper.cxx|7 -
 sfx2/source/doc/objmisc.cxx |4 
 sfx2/source/doc/objstor.cxx |   66 +---
 sot/source/unoolestorage/xolesimplestorage.cxx  |7 -
 svtools/source/misc/bindablecontrolhelper.cxx   |6 -
 svx/source/smarttags/SmartTagMgr.cxx|   26 +-
 svx/source/tbxctrls/tbcontrl.cxx|8 -
 svx/source/xml/xmlxtimp.cxx |8 -
 sw/source/core/ole/ndole.cxx|   10 +-
 sw/source/core/swg/SwXMLTextBlocks.cxx  |8 +
 sw/source/core/unocore/unochart.cxx |9 --
 sw/source/filter/xml/wrtxml.cxx |5 -
 toolkit/source/controls/grid/gridcontrol.cxx|9 --
 toolkit/source/hatchwindow/documentcloser.cxx   |9 --
 unotools/source/config/confignode.cxx   |9 --
 unotools/source/config/lingucfg.cxx |8 +
 unotools/source/config/useroptions.cxx  |8 -
 vbahelper/source/msforms/vbauserform.cxx|6 -
 vbahelper/source/vbahelper/vbacolorformat.cxx   |8 -
 vbahelper/source/vbahelper/vbadocumentbase.cxx  |   13 ---
 vbahelper/source/vbahelper/vbadocumentsbase.cxx |   31 +++
 writerfilter/source/dmapper/DomainMapper_Impl.cxx   |6 -
 writerfilter/source/dmapper/ModelEventListener.cxx  |   10 --
 writerfilter/source/dmapper/OLEHandler.cxx  |4 
 xmloff/source/chart/SchXMLTableContext.cxx  |8 +
 xmloff/source/core/XMLEmbeddedObjectImportContext.cxx   |   14 ---
 54 files changed, 167 insertions(+), 408 deletions(-)

New commits:
commit 7fc6063914432d58d86cfcbd728d967e7c86ebfd
Author: Noel Grandin 
AuthorDate: Thu Aug 17 15:42:01 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Aug 18 11:03:02 2023 +0200

use more Reference::query instead of UNO_QUERY_THROW

since querying with exceptions is consideably more expensive

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

diff --git a/basic/source/classes/sbxmod.cxx b/basic/source/classes/sbxmod.cxx
index 15e4719569ba..020f3814ec8c 100644
--- a/basic/source/classes/sbxmod.cxx
+++ b/basic/source/classes/sbxmod.cxx
@@ -395,8 +395,8 @@ static uno::Reference< vba::XVBACompatibility > 
getVBACompatibility( const uno::
 uno::Reference< vba::XVBACompatibility > xVBACompat;
 try
 {
-uno::Reference< beans::XPropertySet > xModelProps( rxModel, 
uno::UNO_QUERY_THROW );
-xVBACompat.set( xModelProps->getPropertyValue( "BasicLibraries" ), 
uno::UNO_QUERY );
+if (auto xModelProps = rxModel.query() )
+xVBACompat.set( xModelProps->getPropertyValue( "BasicLibraries" ), 
uno::UNO_QUERY );
 }
 catch(const uno::Exception& )
 {
diff --git 

Re: Building Doxygen documentation

2023-08-18 Thread Chris Tapp
On 17 Aug 2023, at 17:42, Michael Stahl  wrote:
> 
> On 17/08/2023 18:12, Chris Tapp wrote:
>> I am trying to build the doxygen documentation using:
>> make odk
>> as instructed on https://wiki.documentfoundation.org/Development/Doxygen
> 
> this wiki page is very confused.
> 
> almost everything of it is about documenting *internal* LO code with doxygen, 
> except for the "Building" section, which is about building the *public* API 
> documentation with doxygen - 2 different things!
> 
> the internal LO code documentation can be built with "make docs".

Thanks, that gets me a bit further - I am now told that "You need the graphviz 
tools to create the nice inheritance graphs”. How do I install it into the LODE 
build environment?

> 
>> However, the following is reported:
>> *** Executable unoapploader must be registered in Repository.mk or 
>> RepositoryExternal.mk.  Stop.
>> What do I need to add to these files to fix this?
> 
> the problem is, you can't just build a single module (odk is a top-level 
> directory aka module) without having a complete build of everything first 
> (just "make").
> 
> note that "docs" is not a module, but just a target in the top-level 
> makefile, so you should be able run that one without building LO (just from 
> memory, i haven't done this in ages).


I should have mentioned that I have already run a complete build.

Chris

[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - sw/qa sw/source

2023-08-18 Thread László Németh (via logerrit)
 sw/qa/uitest/writer_tests7/tdf156783.py |   63 
 sw/source/core/crsr/crsrsh.cxx  |2 -
 2 files changed, 64 insertions(+), 1 deletion(-)

New commits:
commit 2748138cb5c1ad03ef794f8411e28e02de608f39
Author: László Németh 
AuthorDate: Wed Aug 16 21:41:26 2023 +0200
Commit: László Németh 
CommitDate: Fri Aug 18 11:27:39 2023 +0200

tdf#156783 sw tracked table column: fix crash setting border

or border padding without correct table cursor.

Table formatting is allowed on text selections starting
with tables, but adding border or border padding
resulted crashing. This regression became more visible
with hidden deleted table columns, where SelectTable
(e.g. Ctrl-A) doesn't select the whole table, but only
the text content of the visible cells, and in the case
of hidden right table columns, without table cursor.

Manual test: insert two tables in the document, and in the
first cell of the first table, select the whole document
by  pressing Ctrl-A 2 times or more. In table settings,
enable table border or modify padding to crash LibreOffice.

Regression since version 7.5.

Change-Id: If8ebb52c7662ff37ac42348c8f412549b793b6d9
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155749
Tested-by: Jenkins
Reviewed-by: László Németh 
(cherry picked from commit 3a2d3926bbaf6907a50b595d867d7e36cc4ffeaa)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155773

diff --git a/sw/qa/uitest/writer_tests7/tdf156783.py 
b/sw/qa/uitest/writer_tests7/tdf156783.py
new file mode 100644
index ..2ccdb7dcf6e1
--- /dev/null
+++ b/sw/qa/uitest/writer_tests7/tdf156783.py
@@ -0,0 +1,63 @@
+# -*- tab-width: 4; indent-tabs-mode: nil; py-indent-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/.
+#
+from uitest.framework import UITestCase
+from uitest.uihelper.common import get_state_as_dict
+from libreoffice.uno.propertyvalue import mkPropertyValues
+from uitest.uihelper.common import select_pos
+from uitest.uihelper.common import get_url_for_data_file
+
+# Bug 156783 - crash fix: setting table border on a table without correct 
table cursor
+
+class tdf156783(UITestCase):
+def test_tdf156783(self):
+with 
self.ui_test.load_file(get_url_for_data_file("TC-table-del-add.docx")) as 
self.document:
+
+xToolkit = 
self.xContext.ServiceManager.createInstance('com.sun.star.awt.Toolkit')
+
+# accept all tracked changes
+self.xUITest.executeCommand(".uno:AcceptAllTrackedChanges")
+
+# select all tables
+self.xUITest.executeCommand(".uno:SelectAll")
+self.xUITest.executeCommand(".uno:SelectAll")
+self.xUITest.executeCommand(".uno:SelectAll")
+
+# dialog Table Properties - Borders
+with 
self.ui_test.execute_dialog_through_command(".uno:TableDialog", 
close_button="ok") as xDialog:
+tabcontrol = xDialog.getChild("tabcontrol")
+select_pos(tabcontrol, "3")
+
+sync = xDialog.getChild("sync")
+rightmf = xDialog.getChild("rightmf")
+leftmf = xDialog.getChild("leftmf")
+topmf = xDialog.getChild("topmf")
+bottommf = xDialog.getChild("bottommf")
+sync.executeAction("CLICK", tuple())
+
+rightmf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"CTRL+A"}))
+rightmf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"BACKSPACE"}))
+rightmf.executeAction("TYPE", 
mkPropertyValues({"TEXT":"72pt"}))
+leftmf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"CTRL+A"}))
+leftmf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"BACKSPACE"}))
+leftmf.executeAction("TYPE", mkPropertyValues({"TEXT":"72pt"}))
+topmf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"CTRL+A"}))
+topmf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"BACKSPACE"}))
+topmf.executeAction("TYPE", mkPropertyValues({"TEXT":"72pt"}))
+bottommf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"CTRL+A"}))
+bottommf.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"BACKSPACE"}))
+bottommf.executeAction("TYPE", 
mkPropertyValues({"TEXT":"72pt"}))
+
+self.assertEqual(get_state_as_dict(rightmf)["Text"], u"72pt")
+self.assertEqual(get_state_as_dict(leftmf)["Text"], u"72pt")
+self.assertEqual(get_state_as_dict(topmf)["Text"], u"72pt")
+self.assertEqual(get_state_as_dict(bottommf)["Text"], u"72pt")
+
+# 

[Libreoffice-commits] core.git: Branch 'distro/collabora/co-23.05' - vcl/source

2023-08-18 Thread Caolán McNamara (via logerrit)
 vcl/source/graphic/BinaryDataContainer.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 934d43c0834da5ee04cc7362941b8e63e44e635c
Author: Caolán McNamara 
AuthorDate: Wed Aug 16 16:31:54 2023 +0100
Commit: Miklos Vajna 
CommitDate: Fri Aug 18 08:22:47 2023 +0200

SAL_DEBUG "is never meant to remain in the code"

Change-Id: I8c0e513d36c087e9ea8019325cdb3122bec744be
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155772
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Miklos Vajna 

diff --git a/vcl/source/graphic/BinaryDataContainer.cxx 
b/vcl/source/graphic/BinaryDataContainer.cxx
index 43538f9b12fd..7af8b9149bfe 100644
--- a/vcl/source/graphic/BinaryDataContainer.cxx
+++ b/vcl/source/graphic/BinaryDataContainer.cxx
@@ -45,7 +45,6 @@ struct BinaryDataContainer::Impl
 // Horrifying data loss ...
 SAL_WARN_IF(pStream->GetError(), "vcl",
 "Inconsistent system - failed to swap image back in");
-SAL_DEBUG("Swap in: " << pStream->GetError());
 }
 
 void swapOut()


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

2023-08-18 Thread Caolán McNamara (via logerrit)
 desktop/source/lib/init.cxx|   27 ++-
 vcl/source/graphic/BinaryDataContainer.cxx |1 -
 2 files changed, 26 insertions(+), 2 deletions(-)

New commits:
commit a43a9dd8205f089c92737cb5519ed2a96a8d8ff2
Author: Caolán McNamara 
AuthorDate: Thu Aug 17 15:57:28 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 09:58:08 2023 +0200

logs full of "OConfigurationNode::hasByName: object is invalid"

warn:legacy.osl:3509:3314:unotools/source/config/confignode.cxx:408: 
OConfigurationNode::hasByName: object is invalid!
warn:legacy.osl:3509:3314:unotools/source/config/confignode.cxx:409: 
OConfigurationNode::hasByName: object is invalid!
warn:legacy.osl:3509:3314:comphelper/source/misc/types.cxx:90: 
comphelper::getBOOL : invalid argument !

in cool, open a document, switch from light to dark with sidebar open,
click on a paragraph and then on another with different properties, e.g.
normal vs heading and each change of a sidebar context triggers the
warning over and over again.

The config has no mention of notebookbar_online.ui but we set
notebookbar_online.ui as the default to use for a notebookbar
so replace mentions in the config of notebookbar.ui with
notebookbar_online.ui for consistency when we do that and we
can drop the constant log spam.

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

diff --git a/desktop/source/lib/init.cxx b/desktop/source/lib/init.cxx
index 6d5295222f74..86a32708e59b 100644
--- a/desktop/source/lib/init.cxx
+++ b/desktop/source/lib/init.cxx
@@ -82,6 +82,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 
@@ -202,6 +203,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -7427,7 +7429,23 @@ static void activateNotebookbar(std::u16string_view rApp)
 
 if (aAppNode.isValid())
 {
-aAppNode.setNodeValue("Active", 
Any(OUString("notebookbar_online.ui")));
+OUString sNoteBookbarName("notebookbar_online.ui");
+aAppNode.setNodeValue("Active", Any(sNoteBookbarName));
+
+const utl::OConfigurationNode aImplsNode = aAppNode.openNode("Modes");
+const Sequence aModeNodeNames( aImplsNode.getNodeNames() );
+
+for (const auto& rModeNodeName : aModeNodeNames)
+{
+const utl::OConfigurationNode 
aImplNode(aImplsNode.openNode(rModeNodeName));
+if (!aImplNode.isValid())
+continue;
+
+OUString aCommandArg = 
comphelper::getString(aImplNode.getNodeValue("CommandArg"));
+if (aCommandArg == "notebookbar.ui")
+aImplNode.setNodeValue("CommandArg", Any(sNoteBookbarName));
+}
+
 aAppNode.commit();
 }
 }
@@ -7866,6 +7884,13 @@ static int lo_initialize(LibreOfficeKit* pThis, const 
char* pAppPath, const char
 
 if (bNotebookbar)
 {
+std::shared_ptr 
batch(comphelper::ConfigurationChanges::create());
+
officecfg::Office::UI::ToolbarMode::ActiveWriter::set("notebookbar_online.ui", 
batch);
+
officecfg::Office::UI::ToolbarMode::ActiveCalc::set("notebookbar_online.ui", 
batch);
+
officecfg::Office::UI::ToolbarMode::ActiveImpress::set("notebookbar_online.ui", 
batch);
+
officecfg::Office::UI::ToolbarMode::ActiveDraw::set("notebookbar_online.ui", 
batch);
+batch->commit();
+
 activateNotebookbar(u"Writer");
 activateNotebookbar(u"Calc");
 activateNotebookbar(u"Impress");
commit 60d91bed3fdfb0a3d89c51831b27d2187bd24c40
Author: Caolán McNamara 
AuthorDate: Wed Aug 16 16:31:54 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 09:58:00 2023 +0200

SAL_DEBUG "is never meant to remain in the code"

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

diff --git a/vcl/source/graphic/BinaryDataContainer.cxx 
b/vcl/source/graphic/BinaryDataContainer.cxx
index 83cacb491ac2..c9e75bc4ef77 100644
--- a/vcl/source/graphic/BinaryDataContainer.cxx
+++ b/vcl/source/graphic/BinaryDataContainer.cxx
@@ -45,7 +45,6 @@ struct BinaryDataContainer::Impl
 // Horrifying data loss ...
 SAL_WARN_IF(pStream->GetError(), "vcl",
 "Inconsistent system - failed to swap image back in");
-SAL_DEBUG("Swap in: " << pStream->GetError());
 }
 
 void swapOut()


[Libreoffice-commits] core.git: cui/uiconfig

2023-08-18 Thread Samuel Mehrbrodt (via logerrit)
 cui/uiconfig/ui/textflowpage.ui |  416 
 1 file changed, 208 insertions(+), 208 deletions(-)

New commits:
commit 5444b73219fcd50ddbb1fe5e685a401cfd8c9cf3
Author: Samuel Mehrbrodt 
AuthorDate: Thu Aug 17 14:28:36 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Aug 18 08:28:31 2023 +0200

Resave with latest glade

Change-Id: I466d873791351f1eee02cc6bff2adc43e539eece
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155768
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt 

diff --git a/cui/uiconfig/ui/textflowpage.ui b/cui/uiconfig/ui/textflowpage.ui
index 1be6c04228b5..0115952cb0c2 100644
--- a/cui/uiconfig/ui/textflowpage.ui
+++ b/cui/uiconfig/ui/textflowpage.ui
@@ -1,90 +1,90 @@
 
-
+
 
   
   
 1
 9
 2
-1
-10
+1
+10
   
   
 99
-1
-10
+1
+10
   
   
 55535
 1
-1
-10
+1
+10
   
   
 1
 9
 2
-1
-10
+1
+10
   
   
 2
 9
 2
-1
-10
+1
+10
   
   
 2
 9
 2
-1
-10
+1
+10
   
   
 4
 99
 4
-1
-10
+1
+10
   
   
 55.88
-1
-10
+1
+10
   
-  
+  
   
 True
-False
-6
-12
-12
+False
+6
+12
+12
 
   
 True
-False
+False
 GDK_EXPOSURE_MASK
-0
-none
+0
+none
 
-  
+  
   
 True
-False
-6
-12
+False
 12
 6
+6
+12
 
   
 A_utomatically
 True
-True
-False
-True
+True
+False
+True
 True
-True
+True
 
   
 Automatically 
inserts hyphens where they are needed in a paragraph.
@@ -92,20 +92,20 @@
 
   
   
-0
-0
+0
+0
 2
   
 
 
   
 True
-True
+True
 start
 25
-True
-adjustment7
+True
 True
+adjustment7
 
   
 Enter the 
minimum word length in characters that can be hyphenated.
@@ -113,19 +113,19 @@
 
   
   
-0
-6
+0
+6
   
 
 
   
 True
-True
+True
 start
 25
-True
-adjustment2
+True
 True
+adjustment2
 
   
 Enter the 
maximum number of consecutive lines that can be hyphenated.
@@ -133,19 +133,19 @@
 
   
   
-0
-5
+0
+5
   
 
 
   
 True
-True
+True
 start
 25
-True
-adjustment4
+True
 True
+adjustment4
 
   
 Enter the 
minimum number of characters that must appear at the beginning of the line 
after the hyphen.
@@ -153,19 +153,19 @@
 
   
   
-0
-4
+0
+4
   
 
 
   
 True
-True
+True
 start
 25
-True
-adjustment1
+True
 True
+adjustment1
 
   
 Enter the 
minimum number of characters to leave at the end of the line before a hyphen is 
inserted.
@@ -173,78 +173,78 @@
 
   
   
-0
-3
+0
+3
   
 
 
   
 True
-False
+False
 start
 C_haracters at line end
-True
-spinLineEnd
+True
+spinLineEnd
   
   
-1
-3
+1
+3
   

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

2023-08-18 Thread Heiko Tietze (via logerrit)
 sc/source/ui/view/tabvwsh3.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 27fa859bb85ba7ac5b8cb22db5ae85750c7effba
Author: Heiko Tietze 
AuthorDate: Thu Jul 27 11:08:35 2023 +0200
Commit: Heiko Tietze 
CommitDate: Fri Aug 18 09:21:23 2023 +0200

Resolves tdf#156357 - Submit Show Formula to document

Change-Id: Ic893e573b2409342d61877313a3b60e83bb13690
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154984
Tested-by: Jenkins
Reviewed-by: Heiko Tietze 

diff --git a/sc/source/ui/view/tabvwsh3.cxx b/sc/source/ui/view/tabvwsh3.cxx
index 7395325f775c..255dbed88508 100644
--- a/sc/source/ui/view/tabvwsh3.cxx
+++ b/sc/source/ui/view/tabvwsh3.cxx
@@ -721,6 +721,8 @@ void ScTabViewShell::Execute( SfxRequest& rReq )
 ScViewOptions aSetOpts = rOpts;
 aSetOpts.SetOption( VOPT_FORMULAS, bFormulaMode );
 rViewData.SetOptions( aSetOpts );
+ScDocument& rDoc = rViewData.GetDocument();
+rDoc.SetViewOptions(aSetOpts);
 
 rViewData.GetDocShell()->PostPaintGridAll();
 


[Libreoffice-commits] core.git: cui/source editeng/qa editeng/source include/svl include/svx sfx2/source starmath/source svl/Library_svl.mk svl/qa svl/source svx/source sw/qa sw/source vcl/source

2023-08-18 Thread Armin Le Grand (allotropia) (via logerrit)
 cui/source/dialogs/srchxtra.cxx |   20 +-
 cui/source/options/optgdlg.cxx  |1 
 editeng/qa/unit/core-test.cxx   |1 
 editeng/source/editeng/editattr.cxx |1 
 editeng/source/editeng/editdoc.cxx  |1 
 editeng/source/editeng/eerdll.cxx   |1 
 editeng/source/editeng/impedit2.cxx |1 
 include/svl/itemset.hxx |   18 ++
 include/svl/poolitem.hxx|   37 
 include/svl/voiditem.hxx|   53 ++
 include/svx/srchdlg.hxx |   22 +-
 sfx2/source/appl/appbas.cxx |1 
 sfx2/source/control/bindings.cxx|1 
 sfx2/source/control/sfxstatuslistener.cxx   |1 
 sfx2/source/control/shell.cxx   |1 
 sfx2/source/control/statcach.cxx|1 
 sfx2/source/statbar/stbitem.cxx |1 
 sfx2/source/toolbox/tbxitem.cxx |1 
 sfx2/source/view/lokhelper.cxx  |2 
 starmath/source/view.cxx|1 
 svl/Library_svl.mk  |1 
 svl/qa/unit/items/test_itempool.cxx |1 
 svl/source/items/itemset.cxx|  124 ++--
 svl/source/items/poolitem.cxx   |   60 +++
 svl/source/items/voiditem.cxx   |   59 +++
 svx/source/dialog/srchdlg.cxx   |   52 +++---
 svx/source/mnuctrls/clipboardctl.cxx|1 
 svx/source/stbctrls/zoomctrl.cxx|1 
 svx/source/svdraw/svdattr.cxx   |1 
 sw/qa/extras/tiledrendering/tiledrendering.cxx  |1 
 sw/source/core/bastyp/init.cxx  |1 
 sw/source/core/doc/DocumentContentOperationsManager.cxx |7 
 sw/source/uibase/app/apphdl.cxx |1 
 sw/source/uibase/app/applab.cxx |1 
 sw/source/uibase/lingu/olmenu.cxx   |1 
 sw/source/uibase/ribbar/workctrl.cxx|1 
 vcl/source/app/svapp.cxx|   19 ++
 37 files changed, 341 insertions(+), 157 deletions(-)

New commits:
commit c351f920c426542f0d3685bb9df1363d3a6393f8
Author: Armin Le Grand (allotropia) 
AuthorDate: Mon Aug 14 18:21:13 2023 +0200
Commit: Armin Le Grand 
CommitDate: Fri Aug 18 10:37:44 2023 +0200

ITEM: preparations for more/easier changes

This change is not about speed improvements but diverse
preparations to make changes/reading/understanding easier.
It does not change speed AFAIK.

Added a global static debug-only counter to allow getting
an overview over number of all allocated SfxPoolItem's
and the still alloated ones at office shutdown. The values
are used in Application::~Application to make a short info
statement. It allows to be able to quickly detect if an
error in future changes may lead to memory losses - these
would show in dramaitically higher numbers then (hopefully)
immediately.

Moved SfxVoidItem to own source/header.

Added container library interface support to SfxItemSet,
adapted already some methods to use it - not all possible,
I will commit & get status from gerrit 1st if all still works
and then continue.

Changed INVALID_POOL_ITEM from -1 to use a global unique
incarnation of an isolated derivation from SfxPoolItem. It
allows to avoid the (-1) pointer hack. Since still just
pointers are compared it's not worse. NOTE: That way, more
'special' SfxPoolItem's may be used for more States - a
candidate is e.g. SfxVoidItem(0) which represents ::DISABLED
state -- unfortunately not only, it is also used (mainly for
UI stuff) with 'real' WhichIDs - hard to sort out, will have
to stay that way for now AFAIK.

Changed INVALID_POOL_ITEM stuff to use a static extern
incarnated item in combination with a inline method
to return it, called GetGlobalStaticInvalidItemInstance().

Isolated create/cleanup of a SfxPoolItem entry in
SfxItemSet to further modularize/simplify that. It is
currently from constructor & destructor but already shows
that PoolDefaults are handled differently - probably an
error. Still, for now, do no change in behaviour (yet).

Got regular 'killed by the Kill-Wrapper' messages from
gerrit, seems to have to do with UITest_sw_findReplace.
That python/c++ scripting stuff is hard to debug, but
finally I identified the problem has to do with
the INVALID_POOL_ITEM change. It was in

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

2023-08-18 Thread Michael Stahl (via logerrit)
 sw/qa/extras/layout/data/tdf156725.fodt |  163 
 sw/qa/extras/layout/layout3.cxx |   21 
 sw/source/core/layout/sectfrm.cxx   |4 
 3 files changed, 187 insertions(+), 1 deletion(-)

New commits:
commit db83c41d460103df5d80f5bd99816575c4ead5cd
Author: Michael Stahl 
AuthorDate: Thu Aug 17 18:32:35 2023 +0200
Commit: Michael Stahl 
CommitDate: Fri Aug 18 10:58:12 2023 +0200

tdf#156725 sw: layout: don't prevent moving between columns ...

... on the same page when there is a page break on the next page.

The existing, presumably pointless/always-false in case of same page,
check of WrongPageDesc() let me assume that this would only be reached
if the current page and the target page are different, but that was a
mistake.

(regression from commit 325fe7ab507fd8f2ca17a3db32181edf30169525)

Change-Id: I4df53b77bed006095c34976011aa0cd5e12879e7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155809
Tested-by: Jenkins
Reviewed-by: Michael Stahl 

diff --git a/sw/qa/extras/layout/data/tdf156725.fodt 
b/sw/qa/extras/layout/data/tdf156725.fodt
new file mode 100644
index ..9f60e7011954
--- /dev/null
+++ b/sw/qa/extras/layout/data/tdf156725.fodt
@@ -0,0 +1,163 @@
+
+http://www.w3.org/TR/css3-text/; 
xmlns:grddl="http://www.w3.org/2003/g/data-view#; 
xmlns:xhtml="http://www.w3.org/1999/xhtml; 
xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance; 
xmlns:xsd="http://www.w3.org/2001/XMLSchema; 
xmlns:xforms="http://www.w3.org/2002/xforms; 
xmlns:dom="http://www.w3.org/2001/xml-events; 
xmlns:script="urn:oasis:names:tc:opendocument:xmlns:script:1.0" 
xmlns:form="urn:oasis:names:tc:opendocument:xmlns:form:1.0" 
xmlns:math="http://www.w3.org/1998/Math/MathML; 
xmlns:office="urn:oasis:names:tc:opendocument:xmlns:office:1.0" 
xmlns:ooo="http://openoffice.org/2004/office; 
xmlns:fo="urn:oasis:names:tc:opendocument:xmlns:xsl-fo-compatible:1.0" 
xmlns:config="urn:oasis:names:tc:opendocument:xmlns:config:1.0" 
xmlns:ooow="http://openoffice.org/2004/writer; 
xmlns:xlink="http://www.w3.org/1999/xlink; 
xmlns:drawooo="http://openoffice.org/2010/draw; 
xmlns:oooc="http://openoffice.org/2004/calc; 
xmlns:dc="http://purl.org/dc/elements/1.1/; xmlns:c
 alcext="urn:org:documentfoundation:names:experimental:calc:xmlns:calcext:1.0" 
xmlns:style="urn:oasis:names:tc:opendocument:xmlns:style:1.0" 
xmlns:text="urn:oasis:names:tc:opendocument:xmlns:text:1.0" 
xmlns:of="urn:oasis:names:tc:opendocument:xmlns:of:1.2" 
xmlns:tableooo="http://openoffice.org/2009/table; 
xmlns:draw="urn:oasis:names:tc:opendocument:xmlns:drawing:1.0" 
xmlns:dr3d="urn:oasis:names:tc:opendocument:xmlns:dr3d:1.0" 
xmlns:rpt="http://openoffice.org/2005/report; 
xmlns:formx="urn:openoffice:names:experimental:ooxml-odf-interop:xmlns:form:1.0"
 xmlns:svg="urn:oasis:names:tc:opendocument:xmlns:svg-compatible:1.0" 
xmlns:chart="urn:oasis:names:tc:opendocument:xmlns:chart:1.0" 
xmlns:officeooo="http://openoffice.org/2009/office; 
xmlns:table="urn:oasis:names:tc:opendocument:xmlns:table:1.0" 
xmlns:field="urn:openoffice:names:experimental:ooo-ms-interop:xmlns:field:1.0" 
xmlns:number="urn:oasis:names:tc:opendocument:xmlns:datastyle:1.0" 
xmlns:meta="urn:oasis:names:tc:opendocument:xmlns:
 meta:1.0" 
xmlns:loext="urn:org:documentfoundation:names:experimental:office:xmlns:loext:1.0"
 office:version="1.3" office:mimetype="application/vnd.oasis.opendocument.text">
+ 
2023-08-17T18:14:28.1153022102023-08-17T18:20:06.109127302PT4M30S2LibreOfficeDev/24.2.0.0.alpha0$Linux_X86_64
 
LibreOffice_project/79452241ad33f9eaace2ba8bd1336be69c99ed4d
+ 
+  
+  
+  
+ 
+ 
+  
+   
+   
+
+   
+   
+  
+  
+   
+   
+  
+  
+   
+  
+  
+   
+  
+  
+  
+  
+   
+  
+  
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+   
+
+ 
+
+   
+  
+  
+  
+  
+  
+   
+
+
+
+
+
+
+
+
+
+
+
+
+   
+  
+ 
+ 
+  
+   
+  
+  
+   
+  
+  
+   
+
+ 
+ 
+
+   
+  
+  
+   
+
+ 
+ 
+
+   
+  
+  
+   
+
+   
+   
+   
+  
+  
+   
+  
+ 
+ 
+  
+ 
+ 
+  
+   
+
+
+
+
+
+   
+   
+   
+ 
+  
+   a
+   b
+   c
+   d
+  
+ 
+
+  
+ 
+
\ No newline at end of file
diff --git a/sw/qa/extras/layout/layout3.cxx b/sw/qa/extras/layout/layout3.cxx
index 9e47f435df94..9c75ff31a322 100644
--- a/sw/qa/extras/layout/layout3.cxx
+++ b/sw/qa/extras/layout/layout3.cxx
@@ -1207,6 +1207,27 @@ CPPUNIT_TEST_FIXTURE(SwLayoutWriter3, testTdf156724)
 assertXPath(pXmlDoc, "/root/page", 2);
 }
 
+CPPUNIT_TEST_FIXTURE(SwLayoutWriter3, testTdf156725)
+{
+createSwDoc("tdf156725.fodt");
+
+xmlDocUniquePtr pXmlDoc = parseLayoutDump();
+  

Re: Building Doxygen documentation

2023-08-18 Thread Ilmari Lauhakangas

On 18.8.2023 12.02, Chris Tapp wrote:

On 17 Aug 2023, at 17:42, Michael Stahl  wrote:


On 17/08/2023 18:12, Chris Tapp wrote:

I am trying to build the doxygen documentation using:
make odk
as instructed on https://wiki.documentfoundation.org/Development/Doxygen


this wiki page is very confused.

almost everything of it is about documenting *internal* LO code with doxygen, except for 
the "Building" section, which is about building the *public* API documentation 
with doxygen - 2 different things!

the internal LO code documentation can be built with "make docs".


Thanks, that gets me a bit further - I am now told that "You need the graphviz 
tools to create the nice inheritance graphs”. How do I install it into the LODE 
build environment?


On Windows, run the Cygwin setup and search for the graphviz package.

You can also view the docs online: https://api.libreoffice.org/

Ilmari


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

2023-08-18 Thread Michael Stahl (via logerrit)
 sw/source/core/layout/fly.cxx |   13 +
 1 file changed, 13 insertions(+)

New commits:
commit ee82accbd0d67eaf0f5a164a82b86abd00c3b0c9
Author: Michael Stahl 
AuthorDate: Thu Aug 17 13:30:38 2023 +0200
Commit: Michael Stahl 
CommitDate: Fri Aug 18 10:57:21 2023 +0200

tdf#156728 sw: layout: invalidate fly on header/footer size

On page 30 there are at-page anchored shapes Shape22, Shape23 and
Shape25.

Shape22 and Shape23 are positioned relative to the bottom of the page
text area, and that is working; Shape25 is positioned with an offset to
the top of the page text area, and that doesn't work.

The problem is that the shapes are positioned when the header on the
page hasn't been formatted yet, and once the header is formatted, the
position of the Shape25 is not invalidated, so it's a header-height too
far up.

(regression from commit 425a252c3cc5e5a79a533965026dd4af6b8df739)

Change-Id: Ic907d98b1bcd4bb544fd1af9bf93ebc7b75bf362
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155762
Tested-by: Jenkins
Reviewed-by: Michael Stahl 

diff --git a/sw/source/core/layout/fly.cxx b/sw/source/core/layout/fly.cxx
index c2d27eaf2f0e..a06c2a9e7f70 100644
--- a/sw/source/core/layout/fly.cxx
+++ b/sw/source/core/layout/fly.cxx
@@ -2718,10 +2718,23 @@ void SwLayoutFrame::NotifyLowerObjs( const bool 
_bUnlockPosOfObjs )
 {
 assert( dynamic_cast( pObj) &&
 "GetFrameFormat().GetVertOrient().GetRelationOrient());
+if (nO == text::RelOrientation::PAGE_PRINT_AREA
+|| nO == text::RelOrientation::PAGE_PRINT_AREA_BOTTOM
+|| nO == text::RelOrientation::PAGE_PRINT_AREA_TOP)
+{
+isPositionedByHF = true;
+}
+}
 // #i26945# - use  to check, if
 // fly frame is lower of layout frame resp. if fly frame is
 // at a different page registered as its anchor frame is on.
 if ( IsAnLower( pAnchorFrame ) ||
+ isPositionedByHF ||
  pAnchorFrame->FindPageFrame() != pPageFrame )
 {
 // #i44016#


Re: Machine learning is used in gerrit build

2023-08-18 Thread Khaled Hosny



> On 18 Aug 2023, at 6:21 PM, Baole Fang  wrote:
> 
> Hi Noel,
> 
> The reason to choose it is that it is most likely to fail among all the 
> builds. If it fails, then there is no need to run others.

This assumes it is failing because something in the change being built, but it 
often fails for random reasons, so it is not doing the developer any service 
not running the other job. Choosing a more reliable job would at least increase 
the chance of catching actual issues in the change quicker.

Regards,
Khaled

[Libreoffice-commits] core.git: 2 commits - config_host.mk.in configure.ac solenv/gbuild

2023-08-18 Thread Michael Weghorn (via logerrit)
 config_host.mk.in |1 -
 configure.ac  |9 ++---
 solenv/gbuild/JavaClassSet.mk |2 +-
 3 files changed, 3 insertions(+), 9 deletions(-)

New commits:
commit f09e2e1d92a7d7f99b05c739952eed876f741f75
Author: Michael Weghorn 
AuthorDate: Fri Aug 18 09:51:43 2023 +0200
Commit: Michael Weghorn 
CommitDate: Fri Aug 18 20:14:38 2023 +0200

configure.ac: Update Java check for JDK >= 9 version string

Since Java 9, the major version number is the actual one,
while up to including Java 8, that wasn't the case
(Java 8 would use a version string of 1.8.x).

Update the version check accordingly, which was still
assuming the old versioning scheme, i.e. would pass
for any Java version >= 1.9.0.
(This shouldn't make any difference in practice yet,
but will once the minimium version is increased.)

Change-Id: I79eeb247315499caddd63d6abbb1e14ea6a72a4a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155826
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 

diff --git a/configure.ac b/configure.ac
index 673a16b925e3..77d916ac8b5a 100644
--- a/configure.ac
+++ b/configure.ac
@@ -8647,8 +8647,8 @@ if test "$ENABLE_JAVA" != "" -a "$cross_compiling" != 
"yes"; then
 _jdk=`$JAVAINTERPRETER -version 2>&1 | $AWK -F'"' '{ print \$2 }' 
| $SED '/^$/d' | $SED s/[[-A-Za-z]]*//`
 _jdk_ver=`echo "$_jdk" | $AWK -F. '{ print (($1 * 100) + $2) * 100 
+ $3;}'`
 
-if test "$_jdk_ver" -lt 10900; then
-AC_MSG_ERROR([JDK is too old, you need at least 9 ($_jdk_ver < 
10900)])
+if test "$_jdk_ver" -lt 9; then
+AC_MSG_ERROR([JDK is too old, you need at least 9 ($_jdk_ver < 
9)])
 fi
 dnl TODO: Presumably, the Security Manager will not merely be 
disallowed, but be
 dnl completely removed in some Java version > 18 (see
commit 94c94633a4e6f1622c7cc31ce6efd41862691d59
Author: Michael Weghorn 
AuthorDate: Fri Aug 18 09:30:17 2023 +0200
Commit: Michael Weghorn 
CommitDate: Fri Aug 18 20:14:30 2023 +0200

Drop JAVA_CLASSPATH_NOT_SET which is always TRUE

The variable was introduced in

commit b97cade9183d14ad21ae71892348a9a81689488b
Date:   Sun Apr 7 12:14:36 2013 -0400

Supress warnings about unset classpath

I get the below warning for every single jar file without this 
patch.
warning: [options] bootstrap class path not set in conjunction with
-source 1.5
1 warning

Change-Id: I71c01aeea993640f1ec86fe1d8a977656861358d

, at a time when Java 5 was the minimum version,
likely to suppress warnings with JDK versions >= 6
(greater than 1.6.0 to be exact...).
The version check was then just bumped along with the Java
baseline bumps in

commit aafc10c9edb61e13ac557c7e43c8d4a31dce4f37
Date:   Fri Jul 26 10:19:39 2019 +0200

Bump Java baseline to Java 8

and

commit 941b567a41569260fef14a7337511e8f42337323
Date:   Thu May 7 09:42:22 2020 +0200

tdf#131572 Add java 9 module info for libreoffice.jar

for no apparent reason.

In any case, the variable will always be set to TRUE,
since the Java version string is using the actual
major version number as the first digit since
JDK 9 (while JDK 8 would still use 1.8.x as version
string), s.a. upcoming commit
Change-Id I79eeb247315499caddd63d6abbb1e14ea6a72a4a
("configure.ac: Update Java check for JDK >= 9 version string").

Drop the variable and set the Java params
in `solenv/gbuild/JavaClassSet.mk` unconditionally.

Change-Id: Ib4aad07da3937289fc6ff29dd80abdd4c35f1773
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155825
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 

diff --git a/config_host.mk.in b/config_host.mk.in
index 8b345a690eff..f4e6841b22d6 100644
--- a/config_host.mk.in
+++ b/config_host.mk.in
@@ -353,7 +353,6 @@ export JAVADOCISGJDOC=@JAVADOCISGJDOC@
 export JAVACFLAGS=@JAVACFLAGS@
 export JAVAIFLAGS=@JAVAIFLAGS@
 export JAVAIFLAGS_FOR_BUILD=@JAVAIFLAGS_FOR_BUILD@
-export JAVA_CLASSPATH_NOT_SET=@JAVA_CLASSPATH_NOT_SET@
 export JAVAINTERPRETER=@JAVAINTERPRETER@
 export JAVA_HOME=@JAVA_HOME@
 export JAVA_HOME_FOR_BUILD=@JAVA_HOME_FOR_BUILD@
diff --git a/configure.ac b/configure.ac
index 7f5c0ff66430..673a16b925e3 100644
--- a/configure.ac
+++ b/configure.ac
@@ -5850,7 +5850,6 @@ if test "$cross_compiling" = "yes"; then
 ANT
 ANT_HOME
 ANT_LIB
-JAVA_CLASSPATH_NOT_SET
 JAVA_SOURCE_VER
 JAVA_TARGET_VER
 JAVACFLAGS
@@ -8651,9 +8650,6 @@ if test "$ENABLE_JAVA" != "" -a "$cross_compiling" != 
"yes"; then
 if test "$_jdk_ver" -lt 10900; then
 AC_MSG_ERROR([JDK is too old, you need at least 9 

[Libreoffice-commits] core.git: Branch 'libreoffice-7-6' - svgio/qa svgio/source

2023-08-18 Thread Xisco Fauli (via logerrit)
 svgio/qa/cppunit/SvgImportTest.cxx|   21 +
 svgio/qa/cppunit/data/tdf117920.svg   |7 +++
 svgio/source/svgreader/svgdocumenthandler.cxx |7 +--
 3 files changed, 29 insertions(+), 6 deletions(-)

New commits:
commit 06075cd2ffc490feb7560f92ffc894b1e6751f90
Author: Xisco Fauli 
AuthorDate: Thu Aug 17 09:24:18 2023 +0200
Commit: Xisco Fauli 
CommitDate: Fri Aug 18 21:38:46 2023 +0200

tdf#117920: Revert "related tdf#156579: just break for switch element"

This reverts commit e222f60ac2a71032dbdf62d7c7e28978b5a0b19c.

To make the switch element behave like a
group as least, so the transform attribute is parsed
Change-Id: Id8c21ac6da217d8f7bc350ff41022335bf1e0f1f

Change-Id: If1e53f9baad8995f26f9b885cfef6265040736bc
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155770
Tested-by: Jenkins
Reviewed-by: Xisco Fauli 

diff --git a/svgio/qa/cppunit/SvgImportTest.cxx 
b/svgio/qa/cppunit/SvgImportTest.cxx
index bbaf2ec5d96d..73b3150b3a8d 100644
--- a/svgio/qa/cppunit/SvgImportTest.cxx
+++ b/svgio/qa/cppunit/SvgImportTest.cxx
@@ -962,6 +962,27 @@ CPPUNIT_TEST_FIXTURE(Test, testNoneColor)
 assertXPath(pDocument, 
"/primitive2D/transform/mask/polypolygonstroke/line", "width", "3");
 }
 
+CPPUNIT_TEST_FIXTURE(Test, testTdf117920)
+{
+Primitive2DSequence aSequence = 
parseSvg(u"/svgio/qa/cppunit/data/tdf117920.svg");
+CPPUNIT_ASSERT_EQUAL(1, static_cast(aSequence.getLength()));
+
+drawinglayer::Primitive2dXmlDump dumper;
+xmlDocUniquePtr pDocument = 
dumper.dumpAndParse(Primitive2DContainer(aSequence));
+
+CPPUNIT_ASSERT (pDocument);
+
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy11", "1");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy12", "0");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy13", "-18");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy21", "0");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy22", "1");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy23", "-6");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy31", "0");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy32", "0");
+assertXPath(pDocument, "/primitive2D/transform/transform", "xy33", "1");
+}
+
 CPPUNIT_TEST_FIXTURE(Test, testTdf97936)
 {
 // check that both rectangles are rendered in the viewBox
diff --git a/svgio/qa/cppunit/data/tdf117920.svg 
b/svgio/qa/cppunit/data/tdf117920.svg
new file mode 100644
index ..487e0f6cb925
--- /dev/null
+++ b/svgio/qa/cppunit/data/tdf117920.svg
@@ -0,0 +1,7 @@
+http://www.w3.org/2000/svg;
+ xmlns:xlink="http://www.w3.org/1999/xlink;>
+   
+   
+   
+
diff --git a/svgio/source/svgreader/svgdocumenthandler.cxx 
b/svgio/source/svgreader/svgdocumenthandler.cxx
index f86d2603de2a..bd7a9c5791c9 100644
--- a/svgio/source/svgreader/svgdocumenthandler.cxx
+++ b/svgio/source/svgreader/svgdocumenthandler.cxx
@@ -199,6 +199,7 @@ namespace
 mpTarget->parseAttributes(xAttribs);
 break;
 }
+case SVGToken::Switch: //TODO: Support switch element
 case SVGToken::Defs:
 case SVGToken::G:
 {
@@ -442,12 +443,6 @@ namespace
 break;
 }
 
-case SVGToken::Switch:
-{
-//TODO: Support switch element
-break;
-}
-
 default:
 {
 mpTarget = new SvgNode(SVGToken::Unknown, maDocument, 
mpTarget);


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

2023-08-18 Thread Caolán McNamara (via logerrit)
 officecfg/registry/data/org/openoffice/Office/Accelerators.xcu  |6 
++
 officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu |5 
+
 2 files changed, 11 insertions(+)

New commits:
commit 1d361ad42a687bd6730d251d060ce867cc0b882b
Author: Caolán McNamara 
AuthorDate: Fri Aug 18 09:35:28 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 22:57:55 2023 +0200

Related: tdf#151059 assign Alt+7 to new manage changes deck

and move a11y one to 8 to match visual order

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

diff --git a/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu 
b/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
index 647b46927e5f..ed3f30f8f08f 100644
--- a/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/Accelerators.xcu
@@ -6023,6 +6023,12 @@ Ctrl+Shift+e aka E_SHIFT_MOD1 under GTK/IBUS is for some 
emoji thing
   
 
 
+  
+I10N SHORTCUTS - NO 
TRANSLATE
+.uno:SidebarDeck.SwManageChangesDeck
+  
+
+
   
 I10N SHORTCUTS - NO 
TRANSLATE
 .uno:SidebarDeck.A11yCheckDeck
diff --git 
a/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu 
b/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu
index b3e8ce0e2e94..c0d820c0e43b 100644
--- a/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu
+++ b/officecfg/registry/data/org/openoffice/Office/UI/WriterCommands.xcu
@@ -4204,6 +4204,11 @@
   Open the Style Inspector Deck
 
   
+  
+
+  Open the Manage Changes Deck
+
+  
   
 
   Open the Accessibility Check Deck


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

2023-08-18 Thread Caolán McNamara (via logerrit)
 cui/source/dialogs/AdditionsDialog.cxx |   10 --
 cui/source/inc/AdditionsDialog.hxx |1 +
 2 files changed, 9 insertions(+), 2 deletions(-)

New commits:
commit 5296b0ffe3ce031cab29d64d62e1bdc3fb2595fe
Author: Caolán McNamara 
AuthorDate: Fri Aug 18 11:07:07 2023 +0100
Commit: Noel Grandin 
CommitDate: Fri Aug 18 19:22:53 2023 +0200

don't access network during a UITest

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

diff --git a/cui/source/dialogs/AdditionsDialog.cxx 
b/cui/source/dialogs/AdditionsDialog.cxx
index 73b16ed06e90..f0dedf626acf 100644
--- a/cui/source/dialogs/AdditionsDialog.cxx
+++ b/cui/source/dialogs/AdditionsDialog.cxx
@@ -275,6 +275,11 @@ 
SearchAndParseThread::SearchAndParseThread(AdditionsDialog* pDialog, const bool
 , m_bExecute(true)
 , m_bIsFirstLoading(isFirstLoading)
 {
+// if we are running a UITest, e.g. UITest_sw_options then
+// don't attempt to downloading anything
+static const bool bUITest = getenv("LIBO_TEST_UNIT");
+
+m_bUITest = bUITest;
 }
 
 SearchAndParseThread::~SearchAndParseThread() {}
@@ -284,7 +289,8 @@ void SearchAndParseThread::Append(AdditionInfo& 
additionInfo)
 if (!m_bExecute)
 return;
 OUString aPreviewFile;
-bool bResult = getPreviewFile(additionInfo, aPreviewFile); // info vector 
json data
+bool bResult
+= !m_bUITest && getPreviewFile(additionInfo, aPreviewFile); // info 
vector json data
 
 if (!bResult)
 {
@@ -398,7 +404,7 @@ void SearchAndParseThread::execute()
 
 if (m_bIsFirstLoading)
 {
-std::string sResponse = ucbGet(m_pAdditionsDialog->m_sURL);
+std::string sResponse = !m_bUITest ? 
ucbGet(m_pAdditionsDialog->m_sURL) : "";
 parseResponse(sResponse, m_pAdditionsDialog->m_aAllExtensionsVector);
 std::sort(m_pAdditionsDialog->m_aAllExtensionsVector.begin(),
   m_pAdditionsDialog->m_aAllExtensionsVector.end(),
diff --git a/cui/source/inc/AdditionsDialog.hxx 
b/cui/source/inc/AdditionsDialog.hxx
index 302d11bbc6ea..559a4ca911ff 100644
--- a/cui/source/inc/AdditionsDialog.hxx
+++ b/cui/source/inc/AdditionsDialog.hxx
@@ -138,6 +138,7 @@ private:
 AdditionsDialog* m_pAdditionsDialog;
 std::atomic m_bExecute;
 bool m_bIsFirstLoading;
+bool m_bUITest;
 
 void Search();
 void Append(AdditionInfo& additionInfo);


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

2023-08-18 Thread Samuel Mehrbrodt (via logerrit)
 oox/source/export/drawingml.cxx|   15 +++
 sd/qa/unit/data/pptx/tdf151134.odp |binary
 sd/qa/unit/export-tests-ooxml1.cxx |9 +
 3 files changed, 20 insertions(+), 4 deletions(-)

New commits:
commit 0367a168cd01f4a2c16101726714f696ba023a6f
Author: Samuel Mehrbrodt 
AuthorDate: Fri Aug 18 13:42:23 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Aug 18 22:15:59 2023 +0200

tdf#151134 Always write inset properties when exporting placeholder shapes

The default value works for text boxes, but not for these special
presentation placeholder objects - here the values need to be written
explicitly.

Change-Id: Id606bd5e6b63dee2fd3b385435ce13ecec63b5f1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155838
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt 

diff --git a/oox/source/export/drawingml.cxx b/oox/source/export/drawingml.cxx
index 351c1b7cf265..6b464cf774ba 100644
--- a/oox/source/export/drawingml.cxx
+++ b/oox/source/export/drawingml.cxx
@@ -4010,6 +4010,9 @@ void DrawingML::WriteText(const Reference& 
rXIface, bool bBodyPr, bo
 sVertOverflow = "clip";
 }
 
+// tdf#151134 When writing placeholder shapes, inset must be 
explicitly specified
+bool bRequireInset = GetProperty(rXPropSet, "IsPresentationObject") && 
rXPropSet->getPropertyValue("IsPresentationObject").get();
+
 mpFS->startElementNS( (nXmlNamespace ? nXmlNamespace : XML_a), 
XML_bodyPr,
XML_numCol, 
sax_fastparser::UseIf(OString::number(nCols), nCols > 0),
XML_spcCol, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nColSpacing)),
 nCols > 0 && nColSpacing >= 0),
@@ -4017,10 +4020,14 @@ void DrawingML::WriteText(const Reference& 
rXIface, bool bBodyPr, bo
XML_horzOverflow, sHorzOverflow,
XML_vertOverflow, sVertOverflow,
XML_fromWordArt, sax_fastparser::UseIf("1", 
bFromWordArt),
-   XML_lIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nLeft)), 
nLeft != constDefaultLeftRightInset),
-   XML_rIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nRight)), 
nRight != constDefaultLeftRightInset),
-   XML_tIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nTop)), 
nTop != constDefaultTopBottomInset),
-   XML_bIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nBottom)),
 nBottom != constDefaultTopBottomInset),
+   XML_lIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nLeft)),
+   bRequireInset 
|| nLeft != constDefaultLeftRightInset),
+   XML_rIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nRight)),
+   bRequireInset 
|| nRight != constDefaultLeftRightInset),
+   XML_tIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nTop)),
+   bRequireInset 
|| nTop != constDefaultTopBottomInset),
+   XML_bIns, 
sax_fastparser::UseIf(OString::number(oox::drawingml::convertHmmToEmu(nBottom)),
+   bRequireInset 
|| nBottom != constDefaultTopBottomInset),
XML_anchor, sAnchor,
XML_anchorCtr, sax_fastparser::UseIf("1", 
bAnchorCtr),
XML_vert, sWritingMode,
diff --git a/sd/qa/unit/data/pptx/tdf151134.odp 
b/sd/qa/unit/data/pptx/tdf151134.odp
new file mode 100644
index ..56ea86a3144c
Binary files /dev/null and b/sd/qa/unit/data/pptx/tdf151134.odp differ
diff --git a/sd/qa/unit/export-tests-ooxml1.cxx 
b/sd/qa/unit/export-tests-ooxml1.cxx
index ffc3530f40b6..1e7c05390d39 100644
--- a/sd/qa/unit/export-tests-ooxml1.cxx
+++ b/sd/qa/unit/export-tests-ooxml1.cxx
@@ -1666,6 +1666,15 @@ CPPUNIT_TEST_FIXTURE(SdOOXMLExportTest1, testTdf137675)
 "none");
 }
 
+CPPUNIT_TEST_FIXTURE(SdOOXMLExportTest1, testTdf151134)
+{
+createSdImpressDoc("pptx/tdf151134.odp");
+save("Impress Office Open XML");
+
+xmlDocUniquePtr pXmlDoc = parseExport("ppt/slides/slide1.xml");
+assertXPath(pXmlDoc, "/p:sld/p:cSld/p:spTree/p:sp[1]/p:txBody/a:bodyPr", 
"lIns", "91440");
+}
+
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */


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

2023-08-18 Thread Michael Weghorn (via logerrit)
 vcl/source/window/dockmgr.cxx |8 
 1 file changed, 8 insertions(+)

New commits:
commit 0008ff3597e9d89481d5fccb5cefa7f058400a43
Author: Michael Weghorn 
AuthorDate: Fri Aug 18 15:32:12 2023 +0100
Commit: Michael Weghorn 
CommitDate: Fri Aug 18 20:13:55 2023 +0200

tdf#141101 a11y: Ensure focused event after showing popup content

If the `FloatWinPopupFlags::GrabFocus` flag is set,
don't only pass it to `FloatingWindow::StartPopupMode`,
but explicitly grab focus again after showing the
docking window (client window of the floating window).

At least proper handling for accessible focus/focused
events depends on the window being visible at the point
it receives focus.

For winaccessibility, the a11y event listener only
gets registered once the window gets shown
(`VCLXAccessibleComponent::ProcessWindowChildEvent`
forwards a `VclEventId::WindowShow` as
`accessibility::AccessibleEventId::CHILD` event, which
winaccessibility then processes and registers an
event listener for the new child).

This makes NVDA on Windows announce focus for the
"Automatic" button in the color popup in Writer's
character style dialog also when the popup shows
for the first time.

Change-Id: I2e4028eb4ec68a8b586cbd1a0dc5351d1453937e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155843
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 

diff --git a/vcl/source/window/dockmgr.cxx b/vcl/source/window/dockmgr.cxx
index ba9aa2aa9450..7cde1910fed3 100644
--- a/vcl/source/window/dockmgr.cxx
+++ b/vcl/source/window/dockmgr.cxx
@@ -823,6 +823,10 @@ void ImplDockingWindowWrapper::StartPopupMode( ToolBox 
*pParentToolBox, FloatWin
 
 mpFloatWin->StartPopupMode( pParentToolBox, nFlags );
 GetWindow()->Show();
+// grab focus (again) after showing docking window, as e.g. a11y focus
+// events require window to be visible first
+if (nFlags & FloatWinPopupFlags::GrabFocus)
+mpFloatWin->GrabFocus();
 
 if( pParentToolBox->IsKeyEvent() )
 {
@@ -841,6 +845,10 @@ void ImplDockingWindowWrapper::StartPopupMode( const 
tools::Rectangle& rRect, Fl
 ImplPreparePopupMode();
 mpFloatWin->StartPopupMode( rRect, nFlags );
 GetWindow()->Show();
+// grab focus (again) after showing docking window, as e.g. a11y focus
+// events require window to be visible first
+if (nFlags & FloatWinPopupFlags::GrabFocus)
+mpFloatWin->GrabFocus();
 }
 
 IMPL_LINK_NOARG(ImplDockingWindowWrapper, PopupModeEnd, FloatingWindow*, void)


Re: Machine learning is used in gerrit build

2023-08-18 Thread Xisco Fauli

Hello,

On 18/8/23 18:48, Noel Grandin wrote:


Just recently I have to resume my builds 5 or 6 times to get past the 
ML stage, only then to discover that I made a mistake that affected 
some other platform, and then having to change things again, and 
restart the process.


Maybe having a way to force to build on all platforms as we do with 
android could help.


See 
https://lists.freedesktop.org/archives/libreoffice/2023-August/090731.html


--
Xisco Faulí
LibreOffice QA Team
IRC: x1sc0


[Libreoffice-commits] core.git: 2 commits - svx/source

2023-08-18 Thread Michael Weghorn (via logerrit)
 svx/source/dialog/frmsel.cxx |5 -
 1 file changed, 4 insertions(+), 1 deletion(-)

New commits:
commit 912374164f16a4157f04c74994e15f8658b37f03
Author: Michael Weghorn 
AuthorDate: Fri Aug 18 20:02:17 2023 +0200
Commit: Michael Weghorn 
CommitDate: Fri Aug 18 22:20:45 2023 +0200

svx: Use FRAMEBORDERTYPE_COUNT instead of magic number

... just as is used for `maAllBorders` below.

Change-Id: I0741a039cc0ee9ad01055414f471dab45f420ed6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155849
Tested-by: Jenkins
Reviewed-by: Michael Weghorn 

diff --git a/svx/source/dialog/frmsel.cxx b/svx/source/dialog/frmsel.cxx
index 55e73043044b..1fde68ed7c0d 100644
--- a/svx/source/dialog/frmsel.cxx
+++ b/svx/source/dialog/frmsel.cxx
@@ -238,7 +238,7 @@ FrameSelectorImpl::FrameSelectorImpl( FrameSelector& 
rFrameSel ) :
 mbAutoSelect( true ),
 mbHCMode( false )
 #if !ENABLE_WASM_STRIP_ACCESSIBILITY
-,maChildVec( 8 )
+,maChildVec(FRAMEBORDERTYPE_COUNT)
 #endif
 {
 maAllBorders.resize( FRAMEBORDERTYPE_COUNT, nullptr );
commit 79cc574270582d03408286544d99227881f13bb8
Author: Michael Weghorn 
AuthorDate: Fri Aug 18 18:00:17 2023 +0200
Commit: Michael Weghorn 
CommitDate: Fri Aug 18 22:20:37 2023 +0200

related tdf#156683 a11y: Dispose FrameSelectorImpl a11y children

Don't just invalidate, but also dispose the a11y children
in `FrameSelectorImpl::~FrameSelectorImpl`.

This fixes a crash on exit that's reproducible with
the gtk3 VCL plugin e.g. like this (maybe some
steps aren't even needed):

1) start LO Writer with the gtk3 VCL plugin
2) right-click, select "Character" > "Character"
   in context menu
3) switch to "Font Effects" tab
4) press space key with focus in the font color combobox
   to open the popup
5) use Tab key to get to the view of colors
6) move around with arrow keys a bit
7) close the dialog and Writer

Reverse-continuing to `atk_object_wrapper_dispose`
(frame #10 in the below backtrace) in rr
showed the involved a11y object:

uno::Reference to (svx::a11y::AccFrameSelectorChild *) 0x5557832b9150

Backtrace (somewhat similar to the one in tdf#156683):

#0  0x76b15cd6 in 
comphelper::OInterfaceContainerHelper4::disposeAndClear(std::unique_lock&,
 com::sun::star::lang::EventObject const&)
(this=0x7fffca88, rGuard=..., rEvt=...) at 
.../include/comphelper/interfacecontainer4.hxx:397
#1  0x76b13c6f in 
comphelper::AccessibleEventNotifier::revokeClientNotifyDisposing(unsigned int, 
com::sun::star::uno::Reference const&)
(_nClient=1058, _rxEventSource=uno::Reference to 
(svx::a11y::AccFrameSelectorChild *) 0x5cecfdf0) at 
.../comphelper/source/misc/accessibleeventnotifier.cxx:204
#2  0x76b1028a in 
comphelper::OCommonAccessibleComponent::disposing() (this=0x5cecfdf0) at 
.../comphelper/source/misc/accessiblecomponenthelper.cxx:61
#3  0x76674aeb in cppu::WeakComponentImplHelperBase::dispose() 
(this=0x5cecfdf0) at .../cppuhelper/source/implbase.cxx:104
#4  0x72ddf0e0 in 
cppu::PartialWeakComponentImplHelper::dispose() 
(this=0x5cecfdf0)
at .../include/cppuhelper/compbase.hxx:90
#5  0x76674834 in cppu::WeakComponentImplHelperBase::release() 
(this=0x5cecfdf0) at .../cppuhelper/source/implbase.cxx:79
#6  0x72ddf642 in 
cppu::PartialWeakComponentImplHelper::release() 
(this=0x5cecfdf0)
at .../include/cppuhelper/compbase.hxx:86
#7  0x72ddf378 in 
cppu::ImplInheritanceHelper::release() 
(this=0x5cecfdf0)
at .../include/cppuhelper/implbase.hxx:171
#8  0x72ddf0a0 in 
cppu::ImplInheritanceHelper::release() (this=0x5cecfdf0)
at .../include/cppuhelper/implbase.hxx:171
#9  0x7fffe3f29535 in 
com::sun::star::uno::Reference::clear()
 (this=0x5d40a010) at .../include/com/sun/star/uno/Reference.hxx:231
#10 0x7fffe3f288de in atk_object_wrapper_dispose(AtkObjectWrapper*) 
(wrapper=0x5d409fa0) at .../vcl/unx/gtk3/a11y/atkwrapper.cxx:1078
#11 0x7fffe3f27527 in atk_object_wrapper_finalize(GObject*) 
(obj=0x5d409fa0) at .../vcl/unx/gtk3/a11y/atkwrapper.cxx:674
#12 0x7138c48c in g_object_unref () at 
/lib/x86_64-linux-gnu/libgobject-2.0.so.0
#13 0x7fffe3f003f7 in AtkListener::~AtkListener() 
(this=0x5ce6ecb0, __in_chrg=) at 
.../vcl/unx/gtk3/a11y/atklistener.cxx:57
#14 0x7fffe3f0042e in AtkListener::~AtkListener() 
(this=0x5ce6ecb0, __in_chrg=) at 
.../vcl/unx/gtk3/a11y/atklistener.cxx:58
#15 0x76756838 in cppu::OWeakObject::release() 
(this=0x5ce6ecb0) at .../cppuhelper/source/weak.cxx:230
#16 0x7fffe3f0a7de in 
cppu::WeakImplHelper::release()
 (this=0x5ce6ecb0) at 

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

2023-08-18 Thread Caolán McNamara (via logerrit)
 sc/source/ui/inc/gridwin.hxx   |3 +--
 sc/source/ui/view/tabview3.cxx |7 ---
 2 files changed, 5 insertions(+), 5 deletions(-)

New commits:
commit 34d32740d89876c3d4fd2743a07d6e2578601683
Author: Caolán McNamara 
AuthorDate: Thu Aug 17 20:43:22 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 23:00:57 2023 +0200

on invalidating other views, specific what Tab is invalidated

the other views might be viewing other tabs and it is only
the tab ('part') this view is on which is invalidated

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

diff --git a/sc/source/ui/inc/gridwin.hxx b/sc/source/ui/inc/gridwin.hxx
index 979767c6801b..deb3264c7759 100644
--- a/sc/source/ui/inc/gridwin.hxx
+++ b/sc/source/ui/inc/gridwin.hxx
@@ -320,8 +320,6 @@ class SAL_DLLPUBLIC_RTTI ScGridWindow : public 
vcl::DocWindow, public DropTarget
 voidSetupInitialPageBreaks(const ScDocument& rDoc, SCTAB nTab);
 DECL_DLLPRIVATE_LINK(InitiatePageBreaksTimer, Timer*, void);
 
-voidLogicInvalidatePart(const tools::Rectangle* pRectangle, 
int nPart);
-
 protected:
 virtual voidPrePaint(vcl::RenderContext& rRenderContext) override;
 virtual voidPaint(vcl::RenderContext& rRenderContext, const 
tools::Rectangle& rRect) override;
@@ -380,6 +378,7 @@ public:
 
 /// @see Window::LogicInvalidate().
 void LogicInvalidate(const tools::Rectangle* pRectangle) override;
+void LogicInvalidatePart(const tools::Rectangle* pRectangle, int nPart);
 
 /// Update the cell selection according to what handles have been dragged.
 /// @see vcl::ITiledRenderable::setTextSelection() for the values of nType.
diff --git a/sc/source/ui/view/tabview3.cxx b/sc/source/ui/view/tabview3.cxx
index 38c8fcca449e..9a321f73eefe 100644
--- a/sc/source/ui/view/tabview3.cxx
+++ b/sc/source/ui/view/tabview3.cxx
@@ -2200,6 +2200,7 @@ void ScTabView::KillEditView( bool bNoPaint )
 SCROW nRow1 = aViewData.GetEditStartRow();
 SCCOL nCol2 = aViewData.GetEditEndCol();
 SCROW nRow2 = aViewData.GetEditEndRow();
+SCTAB nTab = aViewData.GetTabNo();
 bool bPaint[4];
 bool bNotifyAcc = false;
 tools::Rectangle aRectangle[4];
@@ -2237,16 +2238,16 @@ void ScTabView::KillEditView( bool bNoPaint )
 if (comphelper::LibreOfficeKit::isActive())
 {
 const tools::Rectangle& rInvRect = aRectangle[i];
-pGridWin[i]->LogicInvalidate();
+pGridWin[i]->LogicInvalidatePart(, nTab);
 
 // invalidate other views
 auto lInvalidateWindows =
-[] (ScTabView* pTabView)
+[nTab, ] (ScTabView* pTabView)
 {
 for (VclPtr const & pWin: 
pTabView->pGridWin)
 {
 if (pWin)
-pWin->LogicInvalidate();
+pWin->LogicInvalidatePart(, nTab);
 }
 };
 
commit a35311beef378c15ecbb3bb0560035c17f4657ca
Author: Caolán McNamara 
AuthorDate: Thu Aug 17 20:33:14 2023 +0100
Commit: Caolán McNamara 
CommitDate: Fri Aug 18 23:00:48 2023 +0200

for LibreOfficeKit::isActive we can skip to LogicInvalidate

it should be the same effect

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

diff --git a/sc/source/ui/view/tabview3.cxx b/sc/source/ui/view/tabview3.cxx
index 03a15b4437d9..38c8fcca449e 100644
--- a/sc/source/ui/view/tabview3.cxx
+++ b/sc/source/ui/view/tabview3.cxx
@@ -2237,7 +2237,7 @@ void ScTabView::KillEditView( bool bNoPaint )
 if (comphelper::LibreOfficeKit::isActive())
 {
 const tools::Rectangle& rInvRect = aRectangle[i];
-pGridWin[i]->Invalidate(rInvRect);
+pGridWin[i]->LogicInvalidate();
 
 // invalidate other views
 auto lInvalidateWindows =
@@ -2246,7 +2246,7 @@ void ScTabView::KillEditView( bool bNoPaint )
 for (VclPtr const & pWin: 
pTabView->pGridWin)
 {
 if (pWin)
-pWin->Invalidate(rInvRect);
+pWin->LogicInvalidate();
 }
 };
 


[Libreoffice-commits] core.git: svx/uiconfig

2023-08-18 Thread Samuel Mehrbrodt (via logerrit)
 svx/uiconfig/ui/accessibilitycheckentry.ui |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 56d5b243688e6904620920fbbe9dc3f97f18e932
Author: Samuel Mehrbrodt 
AuthorDate: Tue Aug 15 10:40:01 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Aug 18 12:40:41 2023 +0200

Add dots to "Fix" button to indicate it opens a dialog

Change-Id: Ibafcd3998396f4d763faad4296c3f7355cd90cf1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155718
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt 

diff --git a/svx/uiconfig/ui/accessibilitycheckentry.ui 
b/svx/uiconfig/ui/accessibilitycheckentry.ui
index 1eb5511cfd1d..f5b79bd9a203 100644
--- a/svx/uiconfig/ui/accessibilitycheckentry.ui
+++ b/svx/uiconfig/ui/accessibilitycheckentry.ui
@@ -46,7 +46,7 @@
 
 
   
-Fix
+Fix…
 True
 True
 True


Re: Building Doxygen documentation

2023-08-18 Thread Pierre-Yves Foucou


hello,
was there a  breaking reason to explicitely mention 7.*4* on 
https://api.libreoffice.org/ ?
and/or does it just need a refresh with 7.*5* ?  (then, how much would 
that cost ? ;)

thx


Le 18. 08. 23 à 11:44, Ilmari Lauhakangas a écrit :

On 18.8.2023 12.02, Chris Tapp wrote:

On 17 Aug 2023, at 17:42, Michael Stahl  wrote:


On 17/08/2023 18:12, Chris Tapp wrote:

I am trying to build the doxygen documentation using:
make odk
as instructed on 
https://wiki.documentfoundation.org/Development/Doxygen


this wiki page is very confused.

almost everything of it is about documenting *internal* LO code with 
doxygen, except for the "Building" section, which is about building 
the *public* API documentation with doxygen - 2 different things!


the internal LO code documentation can be built with "make docs".


Thanks, that gets me a bit further - I am now told that "You need the 
graphviz tools to create the nice inheritance graphs”. How do I 
install it into the LODE build environment?


On Windows, run the Cygwin setup and search for the graphviz package.

You can also view the docs online: https://api.libreoffice.org/

Ilmari


--
Cordialement,

__
Schweizerisches Bundesgericht
Service Informatique - AB
Pierre-Yves Foucou
f...@bger.ch+41 (21) 318 95 44



Re: Machine learning is used in gerrit build

2023-08-18 Thread Noel Grandin
Hi

Why are we fronting the ml jenkins job with the least reliable subjob?

Surely we should be using the gcc job - which is faster and more reliable.

Regards, Noel Grandin


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

2023-08-18 Thread Andrea Gelmini (via logerrit)
 vcl/source/app/svapp.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit c8bbabd64ca642b8e9dec5cc6479177315be1210
Author: Andrea Gelmini 
AuthorDate: Fri Aug 18 11:29:43 2023 +0200
Commit: Julien Nabet 
CommitDate: Fri Aug 18 13:53:57 2023 +0200

Fix typos

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

diff --git a/vcl/source/app/svapp.cxx b/vcl/source/app/svapp.cxx
index 927cc4a32b60..541256d8c26a 100644
--- a/vcl/source/app/svapp.cxx
+++ b/vcl/source/app/svapp.cxx
@@ -189,7 +189,7 @@ Application::~Application()
 // is no memory loss. These counters are more to be able to have an eye
 // on amounts of SfxPoolItems used during office usage and to be able to
 // detect if an error in future changes may lead to memory losses - these
-// would show in dramaitically higher numbers then imediately
+// would show in dramatically higher numbers then immediately
 SAL_WARN("vcl", "ITEM: " << getAllocatedSfxPoolItemCount() << " 
SfxPoolItems still allocated at shutdown");
 SAL_WARN("vcl", "ITEM: " << getUsedSfxPoolItemCount() << " SfxPoolItems 
were incarnated during office usage");
 #endif


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

2023-08-18 Thread Andrea Gelmini (via logerrit)
 sc/source/filter/oox/pivotcachebuffer.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit b0cfba866a664b6d70a7b10a20da1971c891ab5e
Author: Andrea Gelmini 
AuthorDate: Fri Aug 18 11:31:10 2023 +0200
Commit: Julien Nabet 
CommitDate: Fri Aug 18 13:53:26 2023 +0200

Fix typo

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

diff --git a/sc/source/filter/oox/pivotcachebuffer.cxx 
b/sc/source/filter/oox/pivotcachebuffer.cxx
index e796cfe53da0..3655ffc40391 100644
--- a/sc/source/filter/oox/pivotcachebuffer.cxx
+++ b/sc/source/filter/oox/pivotcachebuffer.cxx
@@ -720,7 +720,7 @@ OUString PivotCacheField::createParentGroupField( const 
Reference< XDataPilotFie
 previous field if that is grouped too. To find the correct
 group, the first item used to create the group is searched.
 Calc provides the original item names of the base field
-when the group is querried for its members. Its does not
+when the group is queried for its members. Its does not
 provide the names of members that are already groups in the
 field used to create the new groups. (Is this a bug?)
 Therefore, a name from the passed list of original item


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

2023-08-18 Thread Miklos Vajna (via logerrit)
 sfx2/source/view/lokhelper.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 20f6e960cda073012a56597bf6544524240ce246
Author: Miklos Vajna 
AuthorDate: Thu Aug 17 08:24:05 2023 +0200
Commit: Miklos Vajna 
CommitDate: Fri Aug 18 13:25:13 2023 +0200

sfx2: fix crash in SfxLokHelper::notifyOtherView()

Crashreport signature:

Fatal signal received: SIGSEGV code: 1 for address: 0x0
program/libmergedlo.so
SfxLokHelper::notifyOtherView(SfxViewShell const*, SfxViewShell 
const*, int, boost::property_tree::basic_ptree > const&)
sfx2/source/view/lokhelper.cxx:473
program/libmergedlo.so
ImpEditView::ShowCursor(bool, bool)
editeng/source/editeng/impedit.cxx:1423
program/../program/libsclo.so
ScViewData::SetEditEngine(ScSplitPos, ScEditEngineDefaulter*, 
vcl::Window*, short, int)
sc/source/ui/view/viewdata.cxx:1635
program/libsclo.so
ScTabView::UpdateEditView()
sc/source/ui/view/tabview3.cxx:2190

Change-Id: I219a1e9685db9bc3ea03d6ab67ff5db74aac9343
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155823
Tested-by: Jenkins
Reviewed-by: Miklos Vajna 

diff --git a/sfx2/source/view/lokhelper.cxx b/sfx2/source/view/lokhelper.cxx
index a6d9ef825e35..5bf6e8a1a2c4 100644
--- a/sfx2/source/view/lokhelper.cxx
+++ b/sfx2/source/view/lokhelper.cxx
@@ -463,7 +463,7 @@ void SfxLokHelper::notifyOtherView(const SfxViewShell* 
pThisView, SfxViewShell c
int nType, const 
boost::property_tree::ptree& rTree)
 {
 assert(pThisView != nullptr && "pThisView must be valid");
-if (DisableCallbacks::disabled())
+if (DisableCallbacks::disabled() || !pOtherView)
 return;
 
 const int viewId = SfxLokHelper::getView(pThisView);


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

2023-08-18 Thread Andrea Gelmini (via logerrit)
 vcl/source/app/svapp.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit bfa592ab0ce94d1adb905cbd7c1c4a2fd7f16192
Author: Andrea Gelmini 
AuthorDate: Fri Aug 18 11:30:18 2023 +0200
Commit: Julien Nabet 
CommitDate: Fri Aug 18 13:52:47 2023 +0200

Fix typo

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

diff --git a/vcl/source/app/svapp.cxx b/vcl/source/app/svapp.cxx
index 4b7fc313bbb1..927cc4a32b60 100644
--- a/vcl/source/app/svapp.cxx
+++ b/vcl/source/app/svapp.cxx
@@ -185,7 +185,7 @@ Application::~Application()
 //   const Cell OBJ_CELL_NONE;
 // being a static held SfxPoolItem which is not yet de-initialized here
 // number often is (1), even higher with other modules loaded (like 5).
-// These get de-allocated reliaby in module-deinitializations, so this
+// These get de-allocated reliably in module-deinitializations, so this
 // is no memory loss. These counters are more to be able to have an eye
 // on amounts of SfxPoolItems used during office usage and to be able to
 // detect if an error in future changes may lead to memory losses - these


Re: Building Doxygen documentation

2023-08-18 Thread Michael Stahl

On 18/08/2023 11:02, Chris Tapp wrote:

On 17 Aug 2023, at 17:42, Michael Stahl  wrote:


On 17/08/2023 18:12, Chris Tapp wrote:



However, the following is reported:
*** Executable unoapploader must be registered in Repository.mk or 
RepositoryExternal.mk.  Stop.
What do I need to add to these files to fix this?


the problem is, you can't just build a single module (odk is a top-level directory aka 
module) without having a complete build of everything first (just "make").

note that "docs" is not a module, but just a target in the top-level makefile, 
so you should be able run that one without building LO (just from memory, i haven't done 
this in ages).



I should have mentioned that I have already run a complete build.


okay i guess that means you have --disable-odk then, or i think that's 
the default nowadays, you'd need --enable-odk in autogen.input to be 
able to build that.




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

2023-08-18 Thread Olivier Hallot (via logerrit)
 source/text/scalc/01/04060112.xhp |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 01eaa54adbab405db6aad6c2d476bd4ee775e61f
Author: Olivier Hallot 
AuthorDate: Fri Aug 18 15:09:52 2023 +0200
Commit: Olivier Hallot 
CommitDate: Fri Aug 18 15:22:57 2023 +0200

Mute l10n on string

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

diff --git a/source/text/scalc/01/04060112.xhp 
b/source/text/scalc/01/04060112.xhp
index 42cfa5ec52..20c7f3f71e 100644
--- a/source/text/scalc/01/04060112.xhp
+++ b/source/text/scalc/01/04060112.xhp
@@ -116,7 +116,7 @@
 void CALLTYPE fn(out, in1, in2, 
...)
 Output: Resulting value
 Input: Any number of types (double, char*, 
double*, char**, Cell area), where the Cell area is an array of types double 
array, string array, or cell array.
-GetFunctionCount()
+GetFunctionCount()
 Returns the 
number of functions without the management functions of the reference 
parameter. Each function has a unique number between 0 and nCount-1. This 
number will be needed for the GetFunctionData and GetParameterDescription functions 
later.
 
 Syntax
@@ -127,7 +127,7 @@
 
 USHORT nCount:
 Output: Reference to a variable, which is 
supposed to contain the number of Add-In functions. For example: If the Add-In 
provides 5 functions for $[officename] Calc, then nCount=5.
-GetFunctionData()
+GetFunctionData()
 Determines all 
the important information about an Add-In function.
 
 Syntax
@@ -148,7 +148,7 @@
 char* pInternalName:
 Output: Function name as seen by the user, as it 
appears in the Function Wizard. May contain umlauts.
 The pFuncName 
and pInternalName parameters are char arrays, which are implemented with size 
256 in $[officename] Calc.
-GetParameterDescription()
+GetParameterDescription()
 Provides a 
brief description of the Add-In function and its parameters. As an option, this 
function can be used to show a function and parameter description in the 
Function Wizard.
 
 Syntax


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

2023-08-18 Thread Miklos Vajna (via logerrit)
 sw/CppunitTest_sw_core_layout.mk  |1 
 sw/qa/core/layout/data/split-table-border.odt |binary
 sw/qa/core/layout/paintfrm.cxx|   71 +++
 sw/source/core/layout/paintfrm.cxx|   80 ++
 4 files changed, 152 insertions(+)

New commits:
commit 53798fef2cc0b5b0b9706081a4af5ceca964a41b
Author: Miklos Vajna 
AuthorDate: Fri Aug 18 11:21:25 2023 +0200
Commit: Miklos Vajna 
CommitDate: Fri Aug 18 14:52:56 2023 +0200

Related: tdf#156351 sw floattable: fix missing top border in follow table

The problem was that in case a table has a table border but no cell
border, then the follow table was missing a top border in case the
follow table had multiple rows.

This was already working for single-row follow tables since commit
a4da71fb824f2d4ecc7c01f4deb2865ba52f5f4c (INTEGRATION: CWS fmebugs04
(1.115.46); FILE MERGED 2008/05/13 13:56:19 fme 1.115.46.2: #i9860# Top
border for tables - correction 2008/05/13 13:49:23 fme 1.115.46.1:
i#9860 Top border for tables, 2008-06-06).

Fix the problem by checking for the case when the first row in a follow
table has no own bottom border and then take that bottom border from the
last row instead, if that has a bottom border.

The reported case around the missing bottom border in the master table
still needs work.

Change-Id: Ib418da99067a42efcf3bd0e2d6a9467797de619a
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155824
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins

diff --git a/sw/CppunitTest_sw_core_layout.mk b/sw/CppunitTest_sw_core_layout.mk
index be208df04165..9431b6b83526 100644
--- a/sw/CppunitTest_sw_core_layout.mk
+++ b/sw/CppunitTest_sw_core_layout.mk
@@ -16,6 +16,7 @@ $(eval $(call 
gb_CppunitTest_use_common_precompiled_header,sw_core_layout))
 $(eval $(call gb_CppunitTest_add_exception_objects,sw_core_layout, \
 sw/qa/core/layout/flycnt \
 sw/qa/core/layout/layout \
+sw/qa/core/layout/paintfrm \
 sw/qa/core/layout/tabfrm \
 ))
 
diff --git a/sw/qa/core/layout/data/split-table-border.odt 
b/sw/qa/core/layout/data/split-table-border.odt
new file mode 100644
index ..743d54bd38c5
Binary files /dev/null and b/sw/qa/core/layout/data/split-table-border.odt 
differ
diff --git a/sw/qa/core/layout/paintfrm.cxx b/sw/qa/core/layout/paintfrm.cxx
new file mode 100644
index ..15b9df6a4fb4
--- /dev/null
+++ b/sw/qa/core/layout/paintfrm.cxx
@@ -0,0 +1,71 @@
+/* -*- 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 
+
+#include 
+
+#include 
+#include 
+
+namespace
+{
+/// Covers sw/source/core/layout/paintfrm.cxx fixes.
+class Test : public SwModelTestBase
+{
+public:
+Test()
+: SwModelTestBase("/sw/qa/core/layout/data/")
+{
+}
+};
+
+CPPUNIT_TEST_FIXTURE(Test, testSplitTableBorder)
+{
+// Given a document with a split table, table borders are defined, but 
cell borders are not:
+createSwDoc("split-table-border.odt");
+SwXTextDocument* pTextDoc = 
dynamic_cast(mxComponent.get());
+SwDocShell* pShell = pTextDoc->GetDocShell();
+
+// When rendering that document:
+std::shared_ptr xMetaFile = pShell->GetPreviewMetaFile();
+
+// Then make sure that the follow table has a top border:
+MetafileXmlDump aDumper;
+xmlDocUniquePtr pXmlDoc = dumpAndParse(aDumper, *xMetaFile);
+xmlXPathObjectPtr pXmlObj = getXPathNode(pXmlDoc, 
"//polyline[@style='solid']/point");
+xmlNodeSetPtr pXmlNodes = pXmlObj->nodesetval;
+int nHorizontalBorders = 0;
+// Count the horizontal borders:
+for (int i = 0; i < xmlXPathNodeSetGetLength(pXmlNodes); i += 2)
+{
+xmlNodePtr pStart = pXmlNodes->nodeTab[i];
+xmlNodePtr pEnd = pXmlNodes->nodeTab[i + 1];
+xmlChar* pStartY = xmlGetProp(pStart, BAD_CAST("y"));
+xmlChar* pEndY = xmlGetProp(pEnd, BAD_CAST("y"));
+sal_Int32 nStartY = o3tl::toInt32(reinterpret_cast(pStartY));
+sal_Int32 nEndY = o3tl::toInt32(reinterpret_cast(pEndY));
+if (nStartY != nEndY)
+{
+// Vertical border.
+continue;
+}
+
+++nHorizontalBorders;
+}
+xmlXPathFreeObject(pXmlObj);
+// Without the accompanying fix in place, this test would have failed with:
+// - Expected: 3
+// - Actual  : 2
+// i.e. the top border in the follow table was missing.
+CPPUNIT_ASSERT_EQUAL(3, nHorizontalBorders);
+}
+}
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/sw/source/core/layout/paintfrm.cxx 
b/sw/source/core/layout/paintfrm.cxx
index e1f6e12827fe..51dabd080b01 100644
--- 

[Libreoffice-commits] core.git: cui/source cui/uiconfig

2023-08-18 Thread Samuel Mehrbrodt (via logerrit)
 cui/source/inc/paragrph.hxx  |8 +++
 cui/source/tabpages/paragrph.cxx |   40 +++
 cui/uiconfig/ui/textflowpage.ui  |   16 +--
 3 files changed, 34 insertions(+), 30 deletions(-)

New commits:
commit 47edf86a62784aa275de7cc89df01a4fcd4e90c8
Author: Samuel Mehrbrodt 
AuthorDate: Thu Aug 17 14:56:11 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Aug 18 15:23:41 2023 +0200

tdf#156795, tdf#156109 Avoid typesetter language: orphan, widow control

Instead use descriptive language to make it obvious what these options are 
for.
Also invert the logic of the "Do not split paragraph" option as suggested 
in tdf#156109.

Change-Id: I619c0558597aff26b8b7fafea3548d8bf1f24a36
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/155769
Tested-by: Jenkins
Reviewed-by: Samuel Mehrbrodt 

diff --git a/cui/source/inc/paragrph.hxx b/cui/source/inc/paragrph.hxx
index 2b6f26fd25d8..9e78c8311015 100644
--- a/cui/source/inc/paragrph.hxx
+++ b/cui/source/inc/paragrph.hxx
@@ -219,7 +219,7 @@ private:
 weld::TriStateEnabled aPageBreakState;
 weld::TriStateEnabled aApplyCollState;
 weld::TriStateEnabled aPageNumState;
-weld::TriStateEnabled aKeepTogetherState;
+weld::TriStateEnabled aAllowSplitState;
 weld::TriStateEnabled aKeepParaState;
 weld::TriStateEnabled aOrphanState;
 weld::TriStateEnabled aWidowState;
@@ -255,7 +255,7 @@ private:
 std::unique_ptr m_xPagenumEdit;
 
 // paragraph division
-std::unique_ptr m_xKeepTogetherBox;
+std::unique_ptr m_xAllowSplitBox;
 std::unique_ptr m_xKeepParaBox;
 
 // orphan/widow
@@ -271,12 +271,12 @@ private:
 void PageNumBoxClickHdl();
 void ApplyCollClickHdl();
 void PageBreakHdl();
-void KeepTogetherHdl();
+void AllowSplitHdl();
 void OrphanHdl();
 void WidowHdl();
 
 DECL_LINK(PageBreakHdl_Impl, weld::Toggleable&, void);
-DECL_LINK(KeepTogetherHdl_Impl, weld::Toggleable&, void);
+DECL_LINK(AllowSplitHdl_Impl, weld::Toggleable&, void);
 DECL_LINK(WidowHdl_Impl, weld::Toggleable&, void);
 DECL_LINK(OrphanHdl_Impl, weld::Toggleable&, void);
 DECL_LINK(HyphenClickHdl_Impl, weld::Toggleable&, void);
diff --git a/cui/source/tabpages/paragrph.cxx b/cui/source/tabpages/paragrph.cxx
index ff06ee623c8e..957431e565a9 100644
--- a/cui/source/tabpages/paragrph.cxx
+++ b/cui/source/tabpages/paragrph.cxx
@@ -1753,9 +1753,9 @@ bool SvxExtParagraphTabPage::FillItemSet( SfxItemSet* 
rOutSet )
 
 // paragraph split
 _nWhich = GetWhich( SID_ATTR_PARA_SPLIT );
-eState = m_xKeepTogetherBox->get_state();
+eState = m_xAllowSplitBox->get_state();
 
-if (m_xKeepTogetherBox->get_state_changed_from_saved())
+if (m_xAllowSplitBox->get_state_changed_from_saved())
 {
 pOld = GetOldItem( *rOutSet, SID_ATTR_PARA_SPLIT );
 
@@ -2056,13 +2056,13 @@ void SvxExtParagraphTabPage::Reset( const SfxItemSet* 
rSet )
 {
 const SvxFormatSplitItem& rSplit =
 static_cast(rSet->Get( _nWhich ));
-aKeepTogetherState.bTriStateEnabled = false;
+aAllowSplitState.bTriStateEnabled = false;
 
 if ( !rSplit.GetValue() )
-m_xKeepTogetherBox->set_state(TRISTATE_TRUE);
+m_xAllowSplitBox->set_state(TRISTATE_FALSE);
 else
 {
-m_xKeepTogetherBox->set_state(TRISTATE_FALSE);
+m_xAllowSplitBox->set_state(TRISTATE_TRUE);
 // default widows and orphans to enabled
 m_xWidowBox->set_sensitive(true);
 m_xOrphanBox->set_sensitive(true);
@@ -2113,12 +2113,12 @@ void SvxExtParagraphTabPage::Reset( const SfxItemSet* 
rSet )
 aOrphanState.eState = m_xOrphanBox->get_state();
 }
 else if ( SfxItemState::DONTCARE == eItemState )
-m_xKeepTogetherBox->set_state(TRISTATE_INDET);
+m_xAllowSplitBox->set_state(TRISTATE_INDET);
 else
-m_xKeepTogetherBox->set_sensitive(false);
+m_xAllowSplitBox->set_sensitive(false);
 
 // so that everything is enabled correctly
-KeepTogetherHdl();
+AllowSplitHdl();
 WidowHdl();
 OrphanHdl();
 ChangesApplied();
@@ -2144,7 +2144,7 @@ void SvxExtParagraphTabPage::ChangesApplied()
 m_xApplyCollBox->save_value();
 m_xPageNumBox->save_state();
 m_xPagenumEdit->save_value();
-m_xKeepTogetherBox->save_state();
+m_xAllowSplitBox->save_state();
 m_xKeepParaBox->save_state();
 m_xWidowBox->save_state();
 m_xOrphanBox->save_state();
@@ -2202,7 +2202,7 @@ 
SvxExtParagraphTabPage::SvxExtParagraphTabPage(weld::Container* pPage, weld::Dia
 , m_xPageNumBox(m_xBuilder->weld_check_button("labelPageNum"))
 , m_xPagenumEdit(m_xBuilder->weld_spin_button("spinPageNumber"))
 // Options
-, m_xKeepTogetherBox(m_xBuilder->weld_check_button("checkSplitPara"))
+, m_xAllowSplitBox(m_xBuilder->weld_check_button("checkSplitPara"))
 , 

[Libreoffice-commits] core.git: helpcontent2

2023-08-18 Thread Olivier Hallot (via logerrit)
 helpcontent2 |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit abff4a7423a31fc88da63ab92028c22c575131ee
Author: Olivier Hallot 
AuthorDate: Fri Aug 18 15:22:57 2023 +0200
Commit: Gerrit Code Review 
CommitDate: Fri Aug 18 15:22:57 2023 +0200

Update git submodules

* Update helpcontent2 from branch 'master'
  to 01eaa54adbab405db6aad6c2d476bd4ee775e61f
  - Mute l10n on string

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

diff --git a/helpcontent2 b/helpcontent2
index ebdeebb9b004..01eaa54adbab 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit ebdeebb9b004df574ec90ead903ffa9c9b1f3235
+Subproject commit 01eaa54adbab405db6aad6c2d476bd4ee775e61f


Re: Building Doxygen documentation

2023-08-18 Thread Chris Tapp
> On 18 Aug 2023, at 11:57, Michael Stahl  wrote:
> 
> On 18/08/2023 11:02, Chris Tapp wrote:
>> On 17 Aug 2023, at 17:42, Michael Stahl  wrote:
>>> 
>>> On 17/08/2023 18:12, Chris Tapp wrote:
> 
 However, the following is reported:
 *** Executable unoapploader must be registered in Repository.mk or 
 RepositoryExternal.mk.  Stop.
 What do I need to add to these files to fix this?
>>> 
>>> the problem is, you can't just build a single module (odk is a top-level 
>>> directory aka module) without having a complete build of everything first 
>>> (just "make").
>>> 
>>> note that "docs" is not a module, but just a target in the top-level 
>>> makefile, so you should be able run that one without building LO (just from 
>>> memory, i haven't done this in ages).
>> I should have mentioned that I have already run a complete build.
> 
> okay i guess that means you have --disable-odk then, or i think that's the 
> default nowadays, you'd need --enable-odk in autogen.input to be able to 
> build that.

I’ve added —enable-odk to autogen.input, run “make clean” and then “make docs”, 
but I’m still being told “You need the graphviz tools to create the nice 
inheritance graphs”.

Chris



Re: Building Doxygen documentation

2023-08-18 Thread Chris Tapp
> On 18 Aug 2023, at 10:44, Ilmari Lauhakangas 
>  wrote:
> 
> On 18.8.2023 12.02, Chris Tapp wrote:
>> On 17 Aug 2023, at 17:42, Michael Stahl  wrote:
>>> 
>>> On 17/08/2023 18:12, Chris Tapp wrote:
 I am trying to build the doxygen documentation using:
 make odk
 as instructed on https://wiki.documentfoundation.org/Development/Doxygen
>>> 
>>> this wiki page is very confused.
>>> 
>>> almost everything of it is about documenting *internal* LO code with 
>>> doxygen, except for the "Building" section, which is about building the 
>>> *public* API documentation with doxygen - 2 different things!
>>> 
>>> the internal LO code documentation can be built with "make docs".
>> Thanks, that gets me a bit further - I am now told that "You need the 
>> graphviz tools to create the nice inheritance graphs”. How do I install it 
>> into the LODE build environment?
> 
> On Windows, run the Cygwin setup and search for the graphviz package.

My bad - I should have said I’m building under MacOS.

> You can also view the docs online: https://api.libreoffice.org/
> 
> Ilmari



Re: Building Doxygen documentation

2023-08-18 Thread Patrick Luby




On 8/18/23 7:15 PM, Chris Tapp wrote:

I’ve added —enable-odk to autogen.input, run “make clean” and then “make docs”, 
but I’m still being told “You need the graphviz tools to create the nice 
inheritance graphs”.


So I tried "make docs" and I get the same error. I looked at the script 
that make executes and it is looking for the "dot" command.


Maybe the missing piece is that a "graphviz" package needs to be 
installed via Homebrew or MacPorts?


Patrick


[Libreoffice-bugs] [Bug 139532] [META] DOCX Floating table related issues

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=139532
Bug 139532 depends on bug 80717, which changed state.

Bug 80717 Summary: FILEOPEN: DOC DOCX file has incorrect vertical position 
relative to 'page' of floating table (comment 28)
https://bugs.documentfoundation.org/show_bug.cgi?id=80717

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 80717] FILEOPEN: DOC DOCX file has incorrect vertical position relative to 'page' of floating table (comment 28)

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=80717

Justin L  changed:

   What|Removed |Added

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

--- Comment #30 from Justin L  ---
Interesting fact: in MS Word 2010 these two .doc files take up two pages, while
comment 12 shows it on one page. Hmmm - even more interesting is that the PDF
that MSO2010 generates all fits on one page.

(In reply to Justin L from comment #28)
> Probably each of the duplicates here will need to be re-assessed one by one.
Done

> comment 11's GE Bilokin Nadia-fixed.doc had the first table hidden
Fixed

The DOCX version PageBug-new.docx (comment 14) looks fine in LO - one table
follows the other (and spills over onto page 2). [In MSO2010, the second table
splits in two, with only the first row showing up on page one as mentioned
earlier.]

The DOC version PageBug.doc (comment 13) looks more or less like MSO2010 -
except that the second table doesn't split into two.

However, all of this multi-page stuff was split off into bug 121318 which was
marked as NOTOURBUG, and indeed this sounds like a Microsoft Bug since DOC
files are opened differently by the latest versions of MS Word.

I'm going to mark this as FIXED, since the subject and comment 0 complaint is
fixed.

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

[Libreoffice-bugs] [Bug 112700] [META] DOC (binary) table-related issues

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=112700
Bug 112700 depends on bug 80717, which changed state.

Bug 80717 Summary: FILEOPEN: DOC DOCX file has incorrect vertical position 
relative to 'page' of floating table (comment 28)
https://bugs.documentfoundation.org/show_bug.cgi?id=80717

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 77760] FILEOPEN: DOCX Text and Tables switch position - table with unchecked "Move with text"

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=77760

Justin L  changed:

   What|Removed |Added

 CC||jl...@mail.com,
   ||vmik...@collabora.com

--- Comment #15 from Justin L  ---
repro 24.2+ with comment 14's 39006 anformulairechairessud 4-6.docx

Amazing - somehow this table has failed to become floating, even after all of
the changes made to float them. I don't see any relevant exceptions...

Block table

  



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

[Libreoffice-bugs] [Bug 137523] Part of the footer of the first page is not shown at doc opening even is visible at save

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=137523

Michael Stahl (allotropia)  changed:

   What|Removed |Added

 Attachment #166408|application/octet-stream|application/vnd.oasis.opend
  mime type||ocument.text

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

[Libreoffice-bugs] [Bug 156812] New: Installation couldn't competed because of corrupted "libreofficedev1.cab" file

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156812

Bug ID: 156812
   Summary: Installation couldn't competed because of corrupted
"libreofficedev1.cab" file
   Product: LibreOffice
   Version: 24.2.0.0 alpha0+ Master
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Installation
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: rajasekara...@gmail.com

Created attachment 189028
  --> https://bugs.documentfoundation.org/attachment.cgi?id=189028=edit
installation error

Tried to install LO 24.2.0.0.alpha0 on my Windows 10(x64) system.This LO
24.2.0.0.alpha0 was released on 18.08.2023 but couldn't complete because the
installation wizard showed like "libreofficedev1.cab is required for this
installation is corrupted and can't be used etc".I attached the screenshots for
this problem.After I clicked Abort,Retry or Ignore,it showed "Internal Error
2350".

I did the typical installation but it existed again.

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

[Libreoffice-bugs] [Bug 137859] Imported .docx doesn't look the same

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=137859

--- Comment #10 from Justin L  ---
Created attachment 189030
  --> https://bugs.documentfoundation.org/attachment.cgi?id=189030=edit
Bug_DOCX.mso2010.pdf: LO looks like this too, after a layout refresh

This looks nearly perfect now (at least compared to MSO 2010) (and DOES look
perfect if you switch to View - web and then back to View - Normal).

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

[Libreoffice-bugs] [Bug 102847] [META] Quick Find, Search and Replace

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=102847

BogdanB  changed:

   What|Removed |Added

 Depends on||156780


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=156780
[Bug 156780] Libre Office never ends searching during Find and Replace
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 107656] [META] Frame bugs and enhancements

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=107656
Bug 107656 depends on bug 156419, which changed state.

Bug 156419 Summary: Section within Linked Frames Will Not Continue Beyond Page
https://bugs.documentfoundation.org/show_bug.cgi?id=156419

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 107738] [META] Section bugs and enhancements

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=107738
Bug 107738 depends on bug 156419, which changed state.

Bug 156419 Summary: Section within Linked Frames Will Not Continue Beyond Page
https://bugs.documentfoundation.org/show_bug.cgi?id=156419

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 156419] Section within Linked Frames Will Not Continue Beyond Page

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156419

Michael Stahl (allotropia)  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
   Assignee|libreoffice-b...@lists.free |michael.st...@allotropia.de
   |desktop.org |
 Resolution|--- |FIXED

--- Comment #5 from Michael Stahl (allotropia)  ---
fixed on master

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

[Libreoffice-bugs] [Bug 156806] in German translation of UI dialogue for deleting whole colums lacks a word

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156806

Julien Nabet  changed:

   What|Removed |Added

 CC||serval2...@yahoo.fr

--- Comment #1 from Julien Nabet  ---
On pc Debian x86-64 with master sources updated today, I don't reproduce this.

It seems already fixed according to
https://translations.documentfoundation.org/translate/libo_ui-master/officecfgregistrydataorgopenofficeofficeui/de/?checksum=8dbd2d936db28535=note%3AAG4Qy_by=-priority%2Cposition

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

[Libreoffice-ux-advise] Cool! LibreOffice is free, open source and very good office suite!

2023-08-18 Thread Turritopsis Dohrnii Teo En Ming
Subject: Cool! LibreOffice is free, open source and very good office suite!

Good day from Singapore,

I have problems with both of my Microsoft Office Home and Business
2019 licenses. So I had installed a trial version of Microsoft Office
on my refurbished Aftershock gaming laptop but it expired on 11 Aug
2023. I couldn't use it any more.

I needed an office suite but I do not want to spend SGD$300++
purchasing Microsoft Office licenses. Then I thought of LibreOffice. I
quickly downloaded and installed LibreOffice 7.5.5.2.

With LibreOffice installed, I used LibreOffice Calc to edit and save
my Excel worksheet. When I opened the edited Excel worksheet using
Microsoft Excel on my Samsung Galaxy A32 5G Android phone, the Excel
worksheet didn't look significantly different. In fact, the Excel
worksheet edited by LibreOffice Calc looks the same as if it had been
edited by Microsoft Excel.

I have decided to uninstall the trial version of Microsoft Office Home
and Business 2019 (as it doesn't work anymore) and use LibreOffice
permanently!

By the way, does LibreOffice have the equivalent of Microsoft Outlook?

Thank you LibreOffice development team!

Regards,

Mr. Turritopsis Dohrnii Teo En Ming
Targeted Individual in Singapore
Blogs:
https://tdtemcerts.blogspot.com
https://tdtemcerts.wordpress.com
GIMP also stands for Government-Induced Medical Problems.


[Libreoffice-bugs] [Bug 156810] FILESAVE

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156810

m.a.riosv  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 CC||miguelangelrv@libreoffice.o
   ||rg
 Resolution|--- |DUPLICATE

--- Comment #1 from m.a.riosv  ---


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

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

[Libreoffice-bugs] [Bug 148170] Calc: setting print range with the whole column not remember in XLS/X

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=148170

m.a.riosv  changed:

   What|Removed |Added

 CC||cbuat...@free.fr

--- Comment #7 from m.a.riosv  ---
*** Bug 156810 has been marked as a duplicate of this bug. ***

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

[Libreoffice-qa] Cool! LibreOffice is free, open source and very good office suite!

2023-08-18 Thread Turritopsis Dohrnii Teo En Ming
Subject: Cool! LibreOffice is free, open source and very good office suite!

Good day from Singapore,

I have problems with both of my Microsoft Office Home and Business
2019 licenses. So I had installed a trial version of Microsoft Office
on my refurbished Aftershock gaming laptop but it expired on 11 Aug
2023. I couldn't use it any more.

I needed an office suite but I do not want to spend SGD$300++
purchasing Microsoft Office licenses. Then I thought of LibreOffice. I
quickly downloaded and installed LibreOffice 7.5.5.2.

With LibreOffice installed, I used LibreOffice Calc to edit and save
my Excel worksheet. When I opened the edited Excel worksheet using
Microsoft Excel on my Samsung Galaxy A32 5G Android phone, the Excel
worksheet didn't look significantly different. In fact, the Excel
worksheet edited by LibreOffice Calc looks the same as if it had been
edited by Microsoft Excel.

I have decided to uninstall the trial version of Microsoft Office Home
and Business 2019 (as it doesn't work anymore) and use LibreOffice
permanently!

By the way, does LibreOffice have the equivalent of Microsoft Outlook?

Thank you LibreOffice development team!

Regards,

Mr. Turritopsis Dohrnii Teo En Ming
Targeted Individual in Singapore
Blogs:
https://tdtemcerts.blogspot.com
https://tdtemcerts.wordpress.com
GIMP also stands for Government-Induced Medical Problems.


[Libreoffice-bugs] [Bug 75331] FILEOPEN: DOCX - floating table with text wrapping 'around' and vertical position relative to 'page' position is wrong (see Comment 15)

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=75331

Justin L  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|NEW |RESOLVED

--- Comment #31 from Justin L  ---


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

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

[Libreoffice-bugs] [Bug 55160] Fileopen: docx layout not imported correctly - support for nested floating tables

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=55160

Justin L  changed:

   What|Removed |Added

 CC||mikhail.v.gavri...@gmail.co
   ||m

--- Comment #16 from Justin L  ---
*** Bug 75331 has been marked as a duplicate of this bug. ***

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

[Libreoffice-bugs] [Bug 120466] [META] DOCX (OOXML) anchor and text wrapping bugs and enhancements

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=120466
Bug 120466 depends on bug 75331, which changed state.

Bug 75331 Summary: FILEOPEN: DOCX - floating table with text wrapping 'around' 
and vertical position relative to 'page' position is wrong (see Comment 15)
https://bugs.documentfoundation.org/show_bug.cgi?id=75331

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |DUPLICATE

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

[Libreoffice-bugs] [Bug 139532] [META] DOCX Floating table related issues

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=139532
Bug 139532 depends on bug 75331, which changed state.

Bug 75331 Summary: FILEOPEN: DOCX - floating table with text wrapping 'around' 
and vertical position relative to 'page' position is wrong (see Comment 15)
https://bugs.documentfoundation.org/show_bug.cgi?id=75331

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |DUPLICATE

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

[Libreoffice-bugs] [Bug 151059] Can't type "^" character with AltGr+3 using Slovenian keyboard, opening Gallery instead

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=151059

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:24.2.0

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

[Libreoffice-bugs] [Bug 151059] Can't type "^" character with AltGr+3 using Slovenian keyboard, opening Gallery instead

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=151059

--- Comment #7 from Commit Notification 
 ---
Caolán McNamara committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/3614953bf1f4ada22247d04a772d1e3e1d361cef

tdf#151059 don't use Ctrl+Alt+3 etc by default for gallery

It will be available in 24.2.0.

The patch should be included in the daily builds available at
https://dev-builds.libreoffice.org/daily/ in the next 24-48 hours. More
information about daily builds can be found at:
https://wiki.documentfoundation.org/Testing_Daily_Builds

Affected users are encouraged to test the fix and report feedback.

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

[Libreoffice-bugs] [Bug 156812] Installation couldn't competed because of corrupted "libreofficedev1.cab" file

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156812

--- Comment #1 from Rajasekaran Karunanithi  ---
Created attachment 189029
  --> https://bugs.documentfoundation.org/attachment.cgi?id=189029=edit
installation error2

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

[Libreoffice-bugs] [Bug 108357] [META] Writer frame/image/OLE style bugs and enhancements

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=108357

BogdanB  changed:

   What|Removed |Added

 Depends on||137981


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=137981
[Bug 137981] Image inside frame still present after cutting frame (but
invisible)
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 137981] Image inside frame still present after cutting frame (but invisible)

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=137981

BogdanB  changed:

   What|Removed |Added

 Blocks||108357
 CC||buzea.bog...@libreoffice.or
   ||g

--- Comment #7 from BogdanB  ---
Still in
Version: 7.5.5.2 (X86_64) / LibreOffice Community
Build ID: ca8fe7424262805f223b9a2334bc7181abbcbf5e
CPU threads: 4; OS: Linux 6.2; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=108357
[Bug 108357] [META] Writer frame/image/OLE style bugs and enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 156419] Section within Linked Frames Will Not Continue Beyond Page

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156419

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:24.2.0

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

[Libreoffice-bugs] [Bug 139532] [META] DOCX Floating table related issues

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=139532
Bug 139532 depends on bug 141348, which changed state.

Bug 141348 Summary: Fileopen DOCX: Floating tables overlap since 4.3
https://bugs.documentfoundation.org/show_bug.cgi?id=141348

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 141348] Fileopen DOCX: Floating tables overlap since 4.3

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=141348

Justin L  changed:

   What|Removed |Added

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

--- Comment #12 from Justin L  ---
Fixed via backports to LO 7.6

I focused on the overlapping tables on page 3. That was fixed by 24.2
commit a4af5432753408c4eea8a8d56c2f48202160c5fe
Author: Miklos Vajna on Mon Jul 17 09:00:33 2023 +0200
 tdf#120262 sw floattable, legacy: fix text wrap around fly when no content
fits

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

[Libreoffice-bugs] [Bug 116292] FILEOPEN DOCX: floating table near bottom of page doesn't split across pages

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=116292

Justin L  changed:

   What|Removed |Added

 CC||jl...@mail.com,
   ||vmik...@collabora.com

--- Comment #12 from Justin L  ---
still repro 24.2+

@Miklos: I think you want to look at this bug report, as it shows situations
where a floating table should flow to the next page when reaching bottom
margin.

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

[Libreoffice-bugs] [Bug 156810] New: FILESAVE

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156810

Bug ID: 156810
   Summary: FILESAVE
   Product: LibreOffice
   Version: 7.5.5.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: cbuat...@free.fr

Print Ranges are correctly memorised in format .xls, but not in .xlsx after a
file save. The Print ranges "user defined" is reset to "entire sheet" clearing
all the ranges defined.

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

[Libreoffice-bugs] [Bug 156809] CALC VLOOKUP() & HLOOKUP() column or row insertion corrupts references - they do not expand

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156809

m.a.riosv  changed:

   What|Removed |Added

 Resolution|--- |NOTABUG
 Status|UNCONFIRMED |RESOLVED
 CC||miguelangelrv@libreoffice.o
   ||rg

--- Comment #2 from m.a.riosv  ---
You have defined the third parameter with a value, so it doesn't change, no
matter what you do inserting/deleting rows/columns.
In S4 use a formula like =VLOOKUP(A1;E2:H13;COLUMNS(E1:H1))


In the future, please go to ask.libreoffice.org and ask your question before
come here. So you can confirm it is really a bug.

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

[Libreoffice-bugs] [Bug 156811] New: Document wrongly marked as modified, only because template has changed.

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156811

Bug ID: 156811
   Summary: Document wrongly marked as modified, only because
template has changed.
   Product: LibreOffice
   Version: 7.3.7.2 release
  Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: UNCONFIRMED
  Severity: minor
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: 2pet...@gmx.co.uk

Description:
I often need to consult old reference Writer documents. On opening, I get a
dialogue, "Template has been modified". I click Keep Old Styles, but the
document is marked as modified (orange dot on the Save icon) immediately it
appears. When I exit, I annoyingly get the "Save document?" dialogue.

I DO NOT want to change the document because its Modify date may be important.

Is it possible to not modify the document if that is what I intend?

Steps to Reproduce:
1.Find an old document
2.Change the template
3.Open the document, selecting Keep Old Styles from the Template Changed
dialogue
4.Notice the document is marked as modified.

Actual Results:
The "Save document?" dialogue appears on exit, even though the document has not
been changed.

Expected Results:
You should be able to exit with being prompted to save.


Reproducible: Always


User Profile Reset: Yes

Additional Info:
Exit immediately when the document has not been changed.

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

[Libreoffice-bugs] [Bug 156310] Cross reference does not jump to number range in a frame

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156310

--- Comment #5 from Miklos Vajna  ---
Hm, I don't recall an intentional change from me here. But we could bibisect
it. :-)

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

[Libreoffice-ux-advise] [Bug 156310] Cross reference does not jump to number range in a frame

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156310

--- Comment #5 from Miklos Vajna  ---
Hm, I don't recall an intentional change from me here. But we could bibisect
it. :-)

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

[Libreoffice-bugs] [Bug 88173] [META] DOCX (OOXML) format limitations

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=88173

Justin L  changed:

   What|Removed |Added

 Depends on||91178


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=91178
[Bug 91178] FILEOPEN: docx table export loses inline table spacing (b/c MS
formats can't define top/bottom margin for inline tables)
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 139532] [META] DOCX Floating table related issues

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=139532

Justin L  changed:

   What|Removed |Added

 Depends on|91178   |


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=91178
[Bug 91178] FILEOPEN: docx table export loses inline table spacing (b/c MS
formats can't define top/bottom margin for inline tables)
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 91178] FILEOPEN: docx table export loses inline table spacing (b/c MS formats can't define top/bottom margin for inline tables)

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=91178

Justin L  changed:

   What|Removed |Added

   Priority|medium  |lowest
 Status|NEW |RESOLVED
 Resolution|--- |NOTABUG
Summary|FILEOPEN: docx table export |FILEOPEN: docx table export
   |loses table spacing |loses inline table spacing
   ||(b/c MS formats can't
   ||define top/bottom margin
   ||for inline tables)
   Hardware|Other   |All
 Blocks|139532  |88173

--- Comment #8 from Justin L  ---
While Gabor's comment 7 may be correct, I'm quite sure this report should just
be closed as NOTABUG.

Isn't this a limitation of the MS formats? I mean, what is the DOCX setting
that would do such a thing? In the UI I don't notice anything that allows a top
and bottom margin for an inline table.

Floating a table simply to preserve top and bottom margins is a very heavy
price to pay - it will cause many more problems than what it is worth.


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=88173
[Bug 88173] [META] DOCX (OOXML) format limitations
https://bugs.documentfoundation.org/show_bug.cgi?id=139532
[Bug 139532] [META] DOCX Floating table related issues
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 88173] [META] DOCX (OOXML) format limitations

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=88173
Bug 88173 depends on bug 91178, which changed state.

Bug 91178 Summary: FILEOPEN: docx table export loses inline table spacing (b/c 
MS formats can't define top/bottom margin for inline tables)
https://bugs.documentfoundation.org/show_bug.cgi?id=91178

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |NOTABUG

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

[Libreoffice-bugs] [Bug 116583] When using special condensed spacing, underlined text leads to a gap toward the end of the word

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=116583

BogdanB  changed:

   What|Removed |Added

 Blocks||71732
Summary|When utilizing special  |When using special
   |condensed spacing,  |condensed spacing,
   |underlined text results in  |underlined text leads to a
   |having a gap toward the end |gap toward the end of the
   |of the word.|word
 CC||buzea.bog...@libreoffice.or
   ||g

--- Comment #8 from BogdanB  ---
Aslo in
Version: 7.5.5.2 (X86_64) / LibreOffice Community
Build ID: ca8fe7424262805f223b9a2334bc7181abbcbf5e
CPU threads: 4; OS: Linux 6.2; UI render: default; VCL: gtk3
Locale: ro-RO (ro_RO.UTF-8); UI: en-US
Calc: threaded


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=71732
[Bug 71732] [META] Bugs related to text rendering, typography and font features
in LO
-- 
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

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=71732

BogdanB  changed:

   What|Removed |Added

 Depends on||116583


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=116583
[Bug 116583] When using special condensed spacing, underlined text leads to a
gap toward the end of the word
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 131470] Swapping columns across multiple sheets works correctly only on the active sheet

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=131470

BogdanB  changed:

   What|Removed |Added

 CC||buzea.bog...@libreoffice.or
   ||g
Summary|Swapping columns across |Swapping columns across
   |multiple sheets works   |multiple sheets works
   |correctly only on the   |correctly only on the
   |active sheet.   |active sheet
 Blocks||108364


Referenced Bugs:

https://bugs.documentfoundation.org/show_bug.cgi?id=108364
[Bug 108364] [META] Table/Row/Column/Cell management function bugs and
enhancements
-- 
You are receiving this mail because:
You are the assignee for the bug.

[Libreoffice-bugs] [Bug 60251] [META] ACCESSIBILITY: Tracking Windows OS accessibility and AT issues

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=60251
Bug 60251 depends on bug 156561, which changed state.

Bug 156561 Summary: NVDA doesn't announce popup content when it's made visible 
after the parent widget
https://bugs.documentfoundation.org/show_bug.cgi?id=156561

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 156770] Sidebar doesn't maintain its user fixed size for example after reload the document or the next time it opens.

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156770

--- Comment #2 from Davide  ---
This issue appears from release 7.5.4.2. I just checked 7.6.0.3 and it remains.
The same is true for the horizontal space in the gallery side bar of Draw, but
I'll post a different bug report for this.

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

[Libreoffice-bugs] [Bug 108519] [META] Page break bugs and enhancements

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=108519
Bug 108519 depends on bug 156728, which changed state.

Bug 156728 Summary: Document layout changed: shape is vertically centered 
instead of aligned to bottom
https://bugs.documentfoundation.org/show_bug.cgi?id=156728

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 156728] Document layout changed: shape is vertically centered instead of aligned to bottom

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156728

Michael Stahl (allotropia)  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Assignee|libreoffice-b...@lists.free |michael.st...@allotropia.de
   |desktop.org |

--- Comment #2 from Michael Stahl (allotropia)  ---
fixed on master

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

[Libreoffice-bugs] [Bug 156783] LO crash selecting table with the whole document and changing table border

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156783

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:24.2.0   |target:24.2.0 target:7.6.1

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

[Libreoffice-bugs] [Bug 153718] EDITING In a Calc table, the default document language setting is not honored throughout the table. Instead, some cells are set to English(USA)

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=153718

--- Comment #8 from Buovjaga  ---
(In reply to Ben Opp from comment #4)
> Created attachment 186296 [details]
> This sample document exhibited the described behavior, but does not when I
> open it now.

So you don't see the problem anymore? I see German (Austria) as the language in
the cells.

(In reply to Ben Opp from comment #7)
> Created attachment 186298 [details]
> clicking on document language in bottom UI panel, these are the choices
> available
> 
> 4 more German dictionaries and English(USA) dictionary are installed, but
> not available for selecting.
> There is a mismatch between the installed dictionaries and the ones I can
> set as document language

What if you click Mehr... and select from there?

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

[Libreoffice-bugs] [Bug 108253] [META] Calc cell formula bugs and enhancements

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=108253
Bug 108253 depends on bug 156357, which changed state.

Bug 156357 Summary: Cannot copy formula by copying cell when displaying with 
Show Formula
https://bugs.documentfoundation.org/show_bug.cgi?id=156357

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

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

[Libreoffice-bugs] [Bug 156803] New: Table Cell Background Color select does not remember selected color

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156803

Bug ID: 156803
   Summary: Table Cell Background Color select does not remember
selected color
   Product: LibreOffice
   Version: 7.5.5.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: vkera...@gmail.com

I am creating lots of Writer tables and I use different colors for various
cells/rows/columns.
If I start working with one color, I think LO should remember my last choice.
But when I click somewhere out of the table the selector is going back to
yellow.
Even when I click on a cell which already has some background color the
selector stays on yellow.
I think it should stay on the selected color or if it has to change I think it
should only change when the user clicks on a cell with background and to choose
this particular color.

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

[Libreoffice-bugs] [Bug 156789] CALC FORMATTING: Cell background color not rendered when OS high contrast display mode is on

2023-08-18 Thread bugzilla-daemon
https://bugs.documentfoundation.org/show_bug.cgi?id=156789

--- Comment #17 from Heiko Tietze  ---
Agree with Ady that this is implemented in a weird way. Plus, switching on via
"[x] Automatically detect high contrast mode of OS" under tools > options >
a11y is pretty awkward. 

Besides the question if we need to modify colors at all I believe the
configuration should not be affected, in other words the dialog should still
show the actual color.

Michael, what is your recommendation? Keep colors depending on HC and fix
issues (see c14) or drop it and limit HC to the UI only (my preference).

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

  1   2   3   4   >