Re: Windows and 256 color displays

2023-09-08 Thread Ilmari Lauhakangas

On 9.9.2023 3.35, Chris Sherlock wrote:

Do we still support Windows on a 256 color display?

If not, we could remove a huge chunk of dithering code in ImplInitSalGDI() and 
vastly simplify multiplatform support.


We advertise 256 colors for all operating systems: 
https://www.libreoffice.org/get-help/system-requirements


Whether that is true for non-Windows anymore, I don't know.

Doing some searches, Windows 7 can be fully changed to 256 colors via 
Screen Resolution - Advanced Settings - Adapter - List All Modes, but 
Win 10/11 only offer compatibility mode for applications.


Win 7 popularity relative to other Win versions is crashing finally, 
being 3.5% now, which *is* roughly the same that Win XP was in early 
2018 when LibreOffice removed support for it and Vista.


Of course dropping 256 color support doesn't have to be tied to a 
platform bump.


Ilmari


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

2023-09-08 Thread Jim Raykowski (via logerrit)
 sw/source/uibase/wrtsh/wrtsh1.cxx |7 +++
 1 file changed, 7 insertions(+)

New commits:
commit 10689e0f24e96781664e734fe23d109af6df77f1
Author: Jim Raykowski 
AuthorDate: Wed Sep 6 21:02:48 2023 -0800
Commit: Jim Raykowski 
CommitDate: Sat Sep 9 02:41:12 2023 +0200

Improve the HasFoldedOutlineContentSelected function

Return false immediatly if there are no outline nodes in the document
since then there can be no folded outline content selected.

Adjust selection that includes the end of content node so that the
end of content node is not included in the selection. This fixes a
crash that occurs when outline folding is enabled and Master
document 'Text' is deleted by using the Navigator popup menu item
'Delete'.

Change-Id: I28be127566c3820b3e78834495bbb354cf2d01aa
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156631
Tested-by: Jenkins
Reviewed-by: Jim Raykowski 

diff --git a/sw/source/uibase/wrtsh/wrtsh1.cxx 
b/sw/source/uibase/wrtsh/wrtsh1.cxx
index af96916885d5..923fabb06167 100644
--- a/sw/source/uibase/wrtsh/wrtsh1.cxx
+++ b/sw/source/uibase/wrtsh/wrtsh1.cxx
@@ -2643,12 +2643,19 @@ bool SwWrtShell::GetAttrOutlineContentVisible(const 
size_t nPos) const
 
 bool SwWrtShell::HasFoldedOutlineContentSelected() const
 {
+// No need to check for selection over folded outline content when there 
are no outline nodes.
+if (GetDoc()->GetNodes().GetOutLineNds().empty())
+return false;
 for(const SwPaM& rPaM : GetCursor()->GetRingContainer())
 {
 SwPaM aPaM(*rPaM.GetMark(), *rPaM.GetPoint());
 aPaM.Normalize();
 SwNodeIndex aPointIdx(aPaM.GetPoint()->GetNode());
 SwNodeIndex aMarkIdx(aPaM.GetMark()->GetNode());
+// Prevent crash in the for loop below by adjusting the mark if it is 
set to the end of
+// content node.
+if (aMarkIdx.GetNode() == GetDoc()->GetNodes().GetEndOfContent())
+--aMarkIdx;
 if (aPointIdx == aMarkIdx)
 continue;
 // Return true if any nodes in PaM are folded outline content nodes.


Windows and 256 color displays

2023-09-08 Thread Chris Sherlock
Do we still support Windows on a 256 color display?

If not, we could remove a huge chunk of dithering code in ImplInitSalGDI() and 
vastly simplify multiplatform support.

Chris

Sent from my iPhone

[Libreoffice-commits] core.git: odk/examples odk/Package_examples.mk solenv/clang-format

2023-09-08 Thread Hossein (via logerrit)
 odk/Package_examples.mk |8 
 odk/examples/OLE/activex/Makefile   |   12 ++--
 odk/examples/OLE/activex/SOActiveX.cxx  |2 +-
 odk/examples/OLE/activex/StdAfx2.cxx|6 +++---
 odk/examples/OLE/activex/so_activex.cxx |2 +-
 solenv/clang-format/excludelist |2 +-
 6 files changed, 16 insertions(+), 16 deletions(-)

New commits:
commit 65325f9c2f9aff6782fa5df910e8f2f5e63dfd93
Author: Hossein 
AuthorDate: Thu Sep 7 17:16:24 2023 +0200
Commit: Hossein 
CommitDate: Sat Sep 9 01:20:46 2023 +0200

Rename .cpp examples to .cxx

The files are renamed, and the associated lines in Makefiles and
elsewhere are also changed.

Change-Id: Ieec38055690e03afd8982320ee0836904a3f41eb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156670
Tested-by: Jenkins
Reviewed-by: Hossein 

diff --git a/odk/Package_examples.mk b/odk/Package_examples.mk
index 457aea37e166..f533a91a346c 100644
--- a/odk/Package_examples.mk
+++ b/odk/Package_examples.mk
@@ -366,17 +366,17 @@ $(eval $(call 
gb_Package_add_files_with_dir,odk_examples,$(SDKDIRNAME)/examples,
 DevelopersGuide/UCB/data/data.txt \
 OLE/activex/Makefile \
 OLE/activex/README.txt \
-OLE/activex/SOActiveX.cpp \
+OLE/activex/SOActiveX.cxx \
 OLE/activex/SOActiveX.h \
 OLE/activex/SOActiveX.rgs \
-OLE/activex/SOComWindowPeer.cpp \
+OLE/activex/SOComWindowPeer.cxx \
 OLE/activex/SOComWindowPeer.h \
 OLE/activex/SOComWindowPeer.rgs \
-OLE/activex/StdAfx2.cpp \
+OLE/activex/StdAfx2.cxx \
 OLE/activex/StdAfx2.h \
 OLE/activex/example.html \
 OLE/activex/resource.h \
-OLE/activex/so_activex.cpp \
+OLE/activex/so_activex.cxx \
 OLE/activex/so_activex.def \
 OLE/activex/so_activex.idl \
 OLE/activex/so_activex.rc \
diff --git a/odk/examples/OLE/activex/Makefile 
b/odk/examples/OLE/activex/Makefile
index 15167bc93ba4..ca893a44362a 100644
--- a/odk/examples/OLE/activex/Makefile
+++ b/odk/examples/OLE/activex/Makefile
@@ -50,12 +50,12 @@ OUT_COMP_SLO=$(OUT_SLO)/$(COMPONENT_NAME)
 
 MIDLFILES = so_activex.idl
 
-CXXFILES = SOActiveX.cpp \
-SOComWindowPeer.cpp \
-so_activex.cpp \
-StdAfx2.cpp
+CXXFILES = SOActiveX.cxx \
+SOComWindowPeer.cxx \
+so_activex.cxx \
+StdAfx2.cxx
 
-SLOFILES = $(patsubst %.cpp,$(OUT_COMP_SLO)/%.$(OBJ_EXT),$(CXXFILES))
+SLOFILES = $(patsubst %.cxx,$(OUT_COMP_SLO)/%.$(OBJ_EXT),$(CXXFILES))
 
 # Targets
 .PHONY: ALL
@@ -76,7 +76,7 @@ $(OUT_COMP_GEN)/so_activex.res : so_activex.rc 
$(OUT_COMP_INC)/so_activex.h
-$(MKDIR) $(subst /,$(PS),$(@D))
rc $(CC_INCLUDES) /R /FO$@ so_activex.rc   
 
-$(OUT_COMP_SLO)/%.$(OBJ_EXT) : %.cpp
+$(OUT_COMP_SLO)/%.$(OBJ_EXT) : %.cxx
-$(MKDIR) $(subst /,$(PS),$(@D))
$(CC) $(CC_FLAGS) $(CC_INCLUDES) -I$(OUT_COMP_INC) $(CC_DEFINES) 
$(CC_OUTPUT_SWITCH)$(subst /,$(PS),$@) $<
 
diff --git a/odk/examples/OLE/activex/SOActiveX.cpp 
b/odk/examples/OLE/activex/SOActiveX.cxx
similarity index 99%
rename from odk/examples/OLE/activex/SOActiveX.cpp
rename to odk/examples/OLE/activex/SOActiveX.cxx
index 81159e489f2e..e29a439f3552 100644
--- a/odk/examples/OLE/activex/SOActiveX.cpp
+++ b/odk/examples/OLE/activex/SOActiveX.cxx
@@ -33,7 +33,7 @@
  *
  */
 
-// SOActiveX.cpp : Implementation of CSOActiveX
+// SOActiveX.cxx : Implementation of CSOActiveX
 
 #include "stdafx2.h"
 #include "so_activex.h"
diff --git a/odk/examples/OLE/activex/SOComWindowPeer.cpp 
b/odk/examples/OLE/activex/SOComWindowPeer.cxx
similarity index 100%
rename from odk/examples/OLE/activex/SOComWindowPeer.cpp
rename to odk/examples/OLE/activex/SOComWindowPeer.cxx
diff --git a/odk/examples/OLE/activex/StdAfx2.cpp 
b/odk/examples/OLE/activex/StdAfx2.cxx
similarity index 95%
rename from odk/examples/OLE/activex/StdAfx2.cpp
rename to odk/examples/OLE/activex/StdAfx2.cxx
index 3911bcf24003..46743f22c2a0 100644
--- a/odk/examples/OLE/activex/StdAfx2.cpp
+++ b/odk/examples/OLE/activex/StdAfx2.cxx
@@ -33,7 +33,7 @@
  *
  */
 
-// stdafx1.cpp : source file that includes just the standard includes
+// stdafx1.cxx : source file that includes just the standard includes
 //  stdafx1.pch will be the pre-compiled header
 //  stdafx1.obj will contain the pre-compiled type information
 
@@ -41,9 +41,9 @@
 
 #ifdef _ATL_STATIC_REGISTRY
 #include 
-#include 
+#include 
 #endif
 
-#include 
+#include 
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */
diff --git a/odk/examples/OLE/activex/so_activex.cpp 
b/odk/examples/OLE/activex/so_activex.cxx
similarity index 98%
rename from odk/examples/OLE/activex/so_activex.cpp
rename to odk/examples/OLE/activex/so_activex.cxx
index b2cf16a80349..614d9866a7ec 100644
--- a/odk/examples/OLE/activex/so_activex.cpp
+++ b/odk/exam

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

2023-09-08 Thread Noel Grandin (via logerrit)
 dbaccess/source/core/dataaccess/databasecontext.cxx   |9 +
 dbaccess/source/core/dataaccess/databaseregistrations.cxx |8 
 dbaccess/source/core/dataaccess/databaseregistrations.hxx |4 ++--
 dbaccess/source/core/inc/databasecontext.hxx  |2 --
 4 files changed, 7 insertions(+), 16 deletions(-)

New commits:
commit debd9a749cbaaa6ec97992b6908293ec5d41bffc
Author: Noel Grandin 
AuthorDate: Fri Sep 8 11:25:50 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 22:15:45 2023 +0200

no need to use XAggregation for DatabaseRegistrations

the delegating class already implements the interface and forwards the
method calls

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

diff --git a/dbaccess/source/core/dataaccess/databasecontext.cxx 
b/dbaccess/source/core/dataaccess/databasecontext.cxx
index dce39f3badd4..5677b5fe316f 100644
--- a/dbaccess/source/core/dataaccess/databasecontext.cxx
+++ b/dbaccess/source/core/dataaccess/databasecontext.cxx
@@ -177,12 +177,7 @@ ODatabaseContext::ODatabaseContext( const Reference< 
XComponentContext >& _rxCon
 #endif
 
 osl_atomic_increment( &m_refCount );
-{
-m_xDBRegistrationAggregate.set( createDataSourceRegistrations( 
m_aContext ), UNO_SET_THROW );
-m_xDatabaseRegistrations.set( m_xDBRegistrationAggregate, 
UNO_QUERY_THROW );
-
-m_xDBRegistrationAggregate->setDelegator( *this );
-}
+m_xDatabaseRegistrations =  createDataSourceRegistrations( m_aContext );
 osl_atomic_decrement( &m_refCount );
 }
 
@@ -193,8 +188,6 @@ ODatabaseContext::~ODatabaseContext()
 #endif
 
 m_xDatabaseDocumentLoader.clear();
-m_xDBRegistrationAggregate->setDelegator( nullptr );
-m_xDBRegistrationAggregate.clear();
 m_xDatabaseRegistrations.clear();
 }
 
diff --git a/dbaccess/source/core/dataaccess/databaseregistrations.cxx 
b/dbaccess/source/core/dataaccess/databaseregistrations.cxx
index 40fa526d1ba7..9e53a8429ef3 100644
--- a/dbaccess/source/core/dataaccess/databaseregistrations.cxx
+++ b/dbaccess/source/core/dataaccess/databaseregistrations.cxx
@@ -27,7 +27,7 @@
 
 #include 
 #include 
-#include 
+#include 
 #include 
 #include 
 #include 
@@ -67,8 +67,8 @@ namespace dbaccess
 }
 
 // DatabaseRegistrations - declaration
-typedef ::cppu::WeakAggImplHelper1  <   XDatabaseRegistrations
->   DatabaseRegistrations_Base;
+typedef ::cppu::WeakImplHelper<   XDatabaseRegistrations
+   >   DatabaseRegistrations_Base;
 
 namespace {
 
@@ -359,7 +359,7 @@ namespace dbaccess
 }
 
 // DatabaseRegistrations - factory
-Reference< XAggregation > createDataSourceRegistrations( const 
Reference & _rxContext )
+Reference< XDatabaseRegistrations > createDataSourceRegistrations( const 
Reference & _rxContext )
 {
 return new DatabaseRegistrations( _rxContext );
 }
diff --git a/dbaccess/source/core/dataaccess/databaseregistrations.hxx 
b/dbaccess/source/core/dataaccess/databaseregistrations.hxx
index 709bc9f98490..52def1de5aa2 100644
--- a/dbaccess/source/core/dataaccess/databaseregistrations.hxx
+++ b/dbaccess/source/core/dataaccess/databaseregistrations.hxx
@@ -20,11 +20,11 @@
 #pragma once
 
 #include 
-#include 
+#include 
 
 namespace dbaccess
 {
-css::uno::Reference
+css::uno::Reference
 createDataSourceRegistrations(const 
css::uno::Reference& _rxContext);
 
 } // namespace dbaccess
diff --git a/dbaccess/source/core/inc/databasecontext.hxx 
b/dbaccess/source/core/inc/databasecontext.hxx
index 2f9bf326c9e0..d94df018d1c0 100644
--- a/dbaccess/source/core/inc/databasecontext.hxx
+++ b/dbaccess/source/core/inc/databasecontext.hxx
@@ -86,8 +86,6 @@ private:
 css::uno::Reference< css::uno::XComponentContext >
 m_aContext;
 
-css::uno::Reference< css::uno::XAggregation >
-m_xDBRegistrationAggregate;
 css::uno::Reference< css::sdb::XDatabaseRegistrations >
 m_xDatabaseRegistrations;
 


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

2023-09-08 Thread Noel Grandin (via logerrit)
 filter/source/pdf/pdfdecomposer.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit d3e0130b1e5a0beb7995565b6e365aec248eece0
Author: Noel Grandin 
AuthorDate: Fri Sep 8 14:02:13 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 22:15:27 2023 +0200

XPdfDecomposer does not need to implement XAggreggation

Checked on jenkins using 'make check' and

+void SAL_CALL setDelegator(css::uno::Reference 
const &) final { assert(false); }
+

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

diff --git a/filter/source/pdf/pdfdecomposer.cxx 
b/filter/source/pdf/pdfdecomposer.cxx
index c1a02c776262..e13ffbb8e162 100644
--- a/filter/source/pdf/pdfdecomposer.cxx
+++ b/filter/source/pdf/pdfdecomposer.cxx
@@ -40,6 +40,11 @@ public:
 XPdfDecomposer(const XPdfDecomposer&) = delete;
 XPdfDecomposer& operator=(const XPdfDecomposer&) = delete;
 
+void SAL_CALL setDelegator(css::uno::Reference 
const&) final
+{
+assert(false);
+}
+
 // XPdfDecomposer
 uno::Sequence> SAL_CALL
 getDecomposition(const uno::Reference& 
xDataContainer,


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

2023-09-08 Thread Darshan-upadhyay1110 (via logerrit)
 cui/source/dialogs/QrCodeGenDialog.cxx |   27 +++
 cui/source/factory/dlgfact.cxx |4 ++--
 cui/source/factory/dlgfact.hxx |2 +-
 cui/source/inc/QrCodeGenDialog.hxx |5 -
 sw/source/uibase/uiview/viewdlg2.cxx   |6 --
 vcl/jsdialog/enabled.cxx   |1 +
 6 files changed, 39 insertions(+), 6 deletions(-)

New commits:
commit 6ed38adb5578d0b52d11d8f2077e345f9a8c7ade
Author: Darshan-upadhyay1110 
AuthorDate: Thu Sep 7 13:00:50 2023 +0530
Commit: Szymon Kłos 
CommitDate: Fri Sep 8 20:38:53 2023 +0200

Enable QR and barcode dialog for online

 - enable QR and barcode genration dialog for online
 - Change Qr code genration dialog to async

Change-Id: Ia46b8e27a3002adcc893e5ef4c2545d7edcc3e41
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156642
Reviewed-by: Szymon Kłos 
Tested-by: Jenkins CollaboraOffice 

diff --git a/cui/source/dialogs/QrCodeGenDialog.cxx 
b/cui/source/dialogs/QrCodeGenDialog.cxx
index 817be7f21ede..43f6f26279ae 100644
--- a/cui/source/dialogs/QrCodeGenDialog.cxx
+++ b/cui/source/dialogs/QrCodeGenDialog.cxx
@@ -231,6 +231,33 @@ short QrCodeGenDialog::run()
 #endif
 }
 
+bool QrCodeGenDialog::runAsync(const std::shared_ptr& 
rController,
+   const std::function& rFunc)
+{
+#if ENABLE_ZXING
+
+weld::GenericDialogController::runAsync(rController, [rController, 
rFunc](sal_Int32 nResult) {
+if (nResult == RET_OK)
+{
+try
+{
+rController->Apply();
+}
+catch (const std::exception&)
+{
+std::unique_ptr 
xBox(Application::CreateMessageDialog(
+rController->GetParent(), VclMessageType::Warning, 
VclButtonsType::Ok,
+CuiResId(RID_CUISTR_QRCODEDATALONG)));
+xBox->run();
+}
+}
+
+rFunc(nResult);
+});
+#endif
+return true;
+}
+
 void QrCodeGenDialog::Apply()
 {
 #if ENABLE_ZXING
diff --git a/cui/source/factory/dlgfact.cxx b/cui/source/factory/dlgfact.cxx
index 091caf4d2bb7..a571b99887ae 100644
--- a/cui/source/factory/dlgfact.cxx
+++ b/cui/source/factory/dlgfact.cxx
@@ -113,7 +113,6 @@ IMPL_ABSTDLG_CLASS(AbstractGraphicFilterDialog)
 IMPL_ABSTDLG_CLASS(AbstractHangulHanjaConversionDialog)
 IMPL_ABSTDLG_CLASS(AbstractInsertObjectDialog)
 IMPL_ABSTDLG_CLASS(AbstractLinksDialog)
-IMPL_ABSTDLG_CLASS(AbstractQrCodeGenDialog)
 IMPL_ABSTDLG_CLASS(AbstractScreenshotAnnotationDlg)
 IMPL_ABSTDLG_CLASS(AbstractSignatureLineDialog)
 IMPL_ABSTDLG_CLASS(AbstractSignSignatureLineDialog)
@@ -132,6 +131,7 @@ IMPL_ABSTDLG_CLASS(AbstractSvxZoomDialog)
 IMPL_ABSTDLG_CLASS(AbstractTitleDialog)
 IMPL_ABSTDLG_CLASS(AbstractURLDlg)
 
IMPL_ABSTDLG_CLASS_ASYNC(AbstractPasswordToOpenModifyDialog,weld::DialogController)
+IMPL_ABSTDLG_CLASS_ASYNC(AbstractQrCodeGenDialog,QrCodeGenDialog)
 IMPL_ABSTDLG_CLASS_ASYNC(AbstractPasteDialog,SfxDialogController)
 IMPL_ABSTDLG_CLASS_ASYNC(AbstractScriptSelectorDialog,SfxDialogController)
 IMPL_ABSTDLG_CLASS_ASYNC(AbstractSpellDialog,SfxDialogController)
@@ -1490,7 +1490,7 @@ VclPtr 
AbstractDialogFactory_Impl::CreateQrCodeGenDialo
 weld::Window* pParent, const Reference xModel, bool bEditExisting)
 {
 return VclPtr::Create(
-std::make_unique(pParent, xModel, bEditExisting));
+std::make_shared(pParent, xModel, bEditExisting));
 }
 
 VclPtr 
AbstractDialogFactory_Impl::CreateAdditionsDialog(
diff --git a/cui/source/factory/dlgfact.hxx b/cui/source/factory/dlgfact.hxx
index 991ec082ad48..0d60011a922b 100644
--- a/cui/source/factory/dlgfact.hxx
+++ b/cui/source/factory/dlgfact.hxx
@@ -406,7 +406,7 @@ 
DECL_ABSTDLG_CLASS(AbstractSignatureLineDialog,SignatureLineDialog)
 };
 
 // AbstractQrCodeGenDialog_Impl
-DECL_ABSTDLG_CLASS(AbstractQrCodeGenDialog,QrCodeGenDialog)
+DECL_ABSTDLG_CLASS_ASYNC(AbstractQrCodeGenDialog,QrCodeGenDialog)
 };
 
 // AbstractSignSignatureLineDialog_Impl
diff --git a/cui/source/inc/QrCodeGenDialog.hxx 
b/cui/source/inc/QrCodeGenDialog.hxx
index 001df88c5581..e477c0b44879 100644
--- a/cui/source/inc/QrCodeGenDialog.hxx
+++ b/cui/source/inc/QrCodeGenDialog.hxx
@@ -24,9 +24,11 @@ public:
 bool bEditExisting);
 
 virtual short run() override;
+static bool runAsync(const std::shared_ptr& rController,
+ const std::function& rFunc);
 
-protected:
 void Apply();
+weld::Widget* GetParent() { return mpParent; }
 
 private:
 css::uno::Reference m_xModel;
@@ -34,6 +36,7 @@ private:
 std::unique_ptr m_xECC[4];
 std::unique_ptr m_xSpinBorder;
 std::unique_ptr m_xComboType;
+
 #if ENABLE_ZXING
 weld::Widget* mpParent;
 #endif
diff --git a/sw/source/uibase/uiview/viewdlg2.cxx 
b/sw/source/uibase/uiview/viewdlg2.cxx
index 19678d38fdcb..597731645ebe 100644
--- a/sw/source/uibase/uiview/viewdlg2.cxx
+++ b/sw/source/uibase/uiview/viewdlg2.

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

2023-09-08 Thread Noel Grandin (via logerrit)
 include/editeng/unotext.hxx |3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

New commits:
commit 59bf2fbb5f6ebe62b4176c040054ff1c80e2d29a
Author: Noel Grandin 
AuthorDate: Fri Sep 8 13:52:00 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 20:20:56 2023 +0200

SvUnoTextRangeEnumeration does not need to implement XAggreggation

Checked on jenkins using 'make check' and

+void SAL_CALL setDelegator(css::uno::Reference 
const &) final { assert(false); }
+

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

diff --git a/include/editeng/unotext.hxx b/include/editeng/unotext.hxx
index 213c5efa6b4f..59e670a0f06d 100644
--- a/include/editeng/unotext.hxx
+++ b/include/editeng/unotext.hxx
@@ -43,6 +43,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -611,7 +612,7 @@ public:
 };
 
 
-class SvxUnoTextRangeEnumeration final : public ::cppu::WeakAggImplHelper1< 
css::container::XEnumeration >
+class SvxUnoTextRangeEnumeration final : public ::cppu::WeakImplHelper< 
css::container::XEnumeration >
 {
 private:
 std::unique_ptr  mpEditSource;


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

2023-09-08 Thread Noel Grandin (via logerrit)
 emfio/source/emfuno/xemfparser.cxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit b438f10910234241f7367d2598c81e0bf1e44cd4
Author: Noel Grandin 
AuthorDate: Fri Sep 8 11:27:38 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 20:20:20 2023 +0200

XEmfParser does not need to implement XAggreggation

Checked on jenkins using 'make check' and

+void SAL_CALL setDelegator(css::uno::Reference 
const &) final { assert(false); }
+

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

diff --git a/emfio/source/emfuno/xemfparser.cxx 
b/emfio/source/emfuno/xemfparser.cxx
index c8641d3f2acc..077446078936 100644
--- a/emfio/source/emfuno/xemfparser.cxx
+++ b/emfio/source/emfuno/xemfparser.cxx
@@ -21,7 +21,7 @@
 
 #include 
 #include 
-#include 
+#include 
 #include 
 
 #include 
@@ -43,7 +43,7 @@ namespace emfio::emfreader
 {
 namespace {
 
-class XEmfParser : public ::cppu::WeakAggImplHelper2< 
graphic::XEmfParser, lang::XServiceInfo >
+class XEmfParser : public ::cppu::WeakImplHelper< graphic::XEmfParser, 
lang::XServiceInfo >
 {
 private:
 uno::Reference< uno::XComponentContext > context_;


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

2023-09-08 Thread Noel Grandin (via logerrit)
 sc/qa/uitest/calc_tests8/navigator.py |3 ++-
 sw/qa/uitest/navigator/tdf144672.py   |   30 +++---
 sw/qa/uitest/navigator/tdf154521.py   |7 ---
 3 files changed, 21 insertions(+), 19 deletions(-)

New commits:
commit f2ba17fecb60d936599efb975d96a3ea06c2a43f
Author: Noel Grandin 
AuthorDate: Fri Sep 8 14:41:58 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 19:27:44 2023 +0200

disable more flakey navigator UITest

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

diff --git a/sc/qa/uitest/calc_tests8/navigator.py 
b/sc/qa/uitest/calc_tests8/navigator.py
index 313b0d572e31..d0a3280619a5 100644
--- a/sc/qa/uitest/calc_tests8/navigator.py
+++ b/sc/qa/uitest/calc_tests8/navigator.py
@@ -142,7 +142,8 @@ class navigator(UITestCase):
 xColumn.executeAction("TYPE", mkPropertyValues({"TEXT":"B"}))
 xColumn.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"RETURN"}))
 xToolkit.processEventsToIdle()
-self.assertEqual(get_state_as_dict(xColumn)['Value'], '2')
+# disable flakey UITest
+#self.assertEqual(get_state_as_dict(xColumn)['Value'], '2')
 
 xRow.executeAction("TYPE", mkPropertyValues({"KEYCODE":"CTRL+A"}))
 xRow.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"BACKSPACE"}))
diff --git a/sw/qa/uitest/navigator/tdf144672.py 
b/sw/qa/uitest/navigator/tdf144672.py
index 4bded66dcb08..790840ad3329 100644
--- a/sw/qa/uitest/navigator/tdf144672.py
+++ b/sw/qa/uitest/navigator/tdf144672.py
@@ -40,21 +40,21 @@ class tdf144672(UITestCase):
 # tdf#129625: Without the fix in place, this test would have 
failed with
 # AssertionError: 'true' != 'false'
 self.assertEqual('true', 
get_state_as_dict(xReferences)['IsSemiTransparent'])
-
-xIndexes = self.get_item(xContentTree, 'Indexes')
-self.assertEqual('Indexes', get_state_as_dict(xIndexes)['Text'])
-self.assertEqual('false', 
get_state_as_dict(xIndexes)['IsSemiTransparent'])
-
-xIndexes.executeAction("EXPAND", tuple())
-
-self.assertEqual('3', get_state_as_dict(xIndexes)['Children'])
-
-for i in range(3):
-# Without the fix in place, this test would have failed here 
with
-# AssertionError: 'false' != 'true'
-self.assertEqual('false', 
get_state_as_dict(xIndexes.getChild(i))['IsSemiTransparent'])
-
-xIndexes.executeAction("COLLAPSE", tuple())
+# disable flakey UITest
+#xIndexes = self.get_item(xContentTree, 'Indexes')
+#self.assertEqual('Indexes', get_state_as_dict(xIndexes)['Text'])
+#self.assertEqual('false', 
get_state_as_dict(xIndexes)['IsSemiTransparent'])
+#
+#xIndexes.executeAction("EXPAND", tuple())
+#
+#self.assertEqual('3', get_state_as_dict(xIndexes)['Children'])
+#
+#for i in range(3):
+## Without the fix in place, this test would have failed here 
with
+## AssertionError: 'false' != 'true'
+#self.assertEqual('false', 
get_state_as_dict(xIndexes.getChild(i))['IsSemiTransparent'])
+#
+#xIndexes.executeAction("COLLAPSE", tuple())
 
 self.xUITest.executeCommand(".uno:Sidebar")
 
diff --git a/sw/qa/uitest/navigator/tdf154521.py 
b/sw/qa/uitest/navigator/tdf154521.py
index 7b2125646ec6..c2140f2ce889 100644
--- a/sw/qa/uitest/navigator/tdf154521.py
+++ b/sw/qa/uitest/navigator/tdf154521.py
@@ -155,9 +155,10 @@ class tdf154521(UITestCase):
 
 # Try the same selection with Bookmark 2
 xContentTree.executeAction("TYPE", mkPropertyValues({"KEYCODE": 
"UP"}))
-self.ui_test.wait_until_property_is_updated(xContentTree, 
"SelectEntryText", "Bookmark 2")
-
self.assertEqual(get_state_as_dict(xContentTree)["SelectEntryText"], "Bookmark 
2")
-
self.assertEqual(get_state_as_dict(xContentTree)["SelectionCount"], "1")
+# disable flakey UITest
+#self.ui_test.wait_until_property_is_updated(xContentTree, 
"SelectEntryText", "Bookmark 2")
+#
self.assertEqual(get_state_as_dict(xContentTree)["SelectEntryText"], "Bookmark 
2")
+#
self.assertEqual(get_state_as_dict(xContentTree)["SelectionCount"], "1")
 xContentTree.executeAction("TYPE", mkPropertyValues({"KEYCODE": 
"RETURN"}))
 self.ui_test.wait_until_property_is_updated(xContentTree, 
"SelectEntryText", "Bookmark 2")
 
self.assertEqual(get_state_as_dict(xContentTree)["SelectEntryText"], "Bookmark 
2")


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

2023-09-08 Thread Noel Grandin (via logerrit)
 chart2/source/view/main/PropertyMapper.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 9e24e0160b3cb0d2bbd5ffe584d98a126783d279
Author: Noel Grandin 
AuthorDate: Fri Sep 8 13:47:38 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 18:39:37 2023 +0200

improve log message

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

diff --git a/chart2/source/view/main/PropertyMapper.cxx 
b/chart2/source/view/main/PropertyMapper.cxx
index 2700769f595c..c3241049057a 100644
--- a/chart2/source/view/main/PropertyMapper.cxx
+++ b/chart2/source/view/main/PropertyMapper.cxx
@@ -119,7 +119,7 @@ void PropertyMapper::setMappedProperties(
 }
 catch( const uno::Exception& )
 {
-TOOLS_WARN_EXCEPTION("chart2", "" );
+TOOLS_WARN_EXCEPTION("chart2", "exception mapping property from " 
<< rSource << " to " << rTarget);
 }
 }
 if (nN == 0)


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

2023-09-08 Thread Armin Le Grand (allotropia) (via logerrit)
 include/svl/itemset.hxx  |   44 +
 svl/source/items/itemset.cxx |  972 +++
 svl/source/items/poolcach.cxx|2 
 sw/inc/swatrset.hxx  |2 
 sw/source/core/attr/swatrset.cxx |   12 
 5 files changed, 530 insertions(+), 502 deletions(-)

New commits:
commit c1f3b34f871d2a6bb9ee7b912492be1164eec96f
Author: Armin Le Grand (allotropia) 
AuthorDate: Tue Aug 29 11:17:59 2023 +0200
Commit: Armin Le Grand 
CommitDate: Fri Sep 8 17:38:24 2023 +0200

ITEM: preparations for more/easier changes II

Again this is a change to improve understandabilty/changeability
of SfxItemSet code plus some cleanups. Still did a callgrind round
to check - it slightly got faster. In a start/load(complex SW doc)/
show/shutdown cycle compared with master I get 96.722 mio cycles
compared with 99.851 mio in master.

Main focus was to isolate two aspects: preparation and cleanup of
an Item for usage in an SfxItemSet. For that we now have
  implCreateItemEntry: to do all needed actions to create/prepare
an Item for membership, including evtl. AddRef/Cloning/using
ItemPool stuff.
  implCleanupItemEntry: to do all needed actios to correctly
clean that Item up again.
All formally accesses distributed over SfxItemSet (and other places)
are cleaned-up to use these. The Item-counter in DBG code that I
already added helped a lot to do this.

Also cleaned up PutImpl to 1st check if action is necessary (Item is
already in place) or not, reducing spot to cleanup an Item that
was handed over as bPassingOwnership to one place.

I also added a 2nd flag, bItemIsSetMember, that tells if the Item
given as input is already member of an SfxItemSet, in that case a
shortcut can be used (increase AddRef, use).

Adapted all places AFAP to use the new container interface
(begin(), end(), ...) where useful.
Made GetItemState inline and directly use the tooling method. Same
for InvalidateItem.
Added much more comments to describe what's going on or to hint at
problems (check for CAUTION).
Removed PutDirect - not needed anymore, probably was there to not get
recursive death loop with callbacks in SW.
More smaller changes.

Checked with all apps, played around. Still, stuff may come up, so
I put on gerrit  the tests will show and give further hints. At last
SfxItemSet is a minefield :-)

Had to adapt SfxItemSet::implCreateItemEntry when input Item is
a StaticDefaultItem. SfxItemPool::PutImpl needs to be called in
that case.

Had to correct bItemIsSetMember in all cases if the transfer
of SfxPoolItems is between SfxItemSets with different
SfxItemPools. This is and will be necessary as long as the
Items are stored at the pool.

After a hard deep-dive I found the error: m_nCount was not in
all cases correct, invalid items get counted.

 Win build *insists* for initialzation of local var aEntry in
 SfxItemSet::PutImpl, triggers warning C4701:
 "potentially uninitialized local variable 'aEntry' used". This
 is not the case here, but I know of no way to silence the
 warning in another way, so added an extra-call to begin().

Re-added to use static pool defaults directly, possible After
the fix 6d8c6e8d60956fd36094035a526c1a29a902204b, thanks for
that. This avoids some cloning of Items.

CAUTION: static default items are not *that* static as it seems
(or: should be). If they are freed with the Pool (see
::ReleaseDefaults) they will be deleted. If the target pool is
different from the source pool static default items from the
source pool can be used that then might be deleted (sigh).

A solution would be to change all pools to really work on
static instances of default items.

Another one would be to know here that the
targetPool != sourcePool, so maybe extend
bItemIsSetMember -> bSamePool.

A good test for this is CppunitTest_chart2_uichart/testTdf98690.
Until solving/cleaning up we unfortunately *have* to continue
to clone static default items...

Change-Id: Ibd8dc6d612f594a5ad88c75fcee8726d89a6090c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156306
Tested-by: Jenkins
Reviewed-by: Armin Le Grand 

diff --git a/include/svl/itemset.hxx b/include/svl/itemset.hxx
index 37141a0a79b3..ba14c21b8e6c 100644
--- a/include/svl/itemset.hxx
+++ b/include/svl/itemset.hxx
@@ -85,9 +85,7 @@ private:
 const SfxItemSet&   operator=(const SfxItemSet &) = delete;
 
 protected:
-voidPutDirect(const SfxPoolItem &rItem);
-
-virtual const SfxPoolItem*  PutImpl( const SfxPoolItem&, sal_uInt16 
nWhich, bool bPassingOwnership );
+virtual const SfxPoolItem*  PutImpl( const SfxPoolItem&, sal_uInt16 
nWhich, bool bItemIsSetMember, 

[Libreoffice-commits] core.git: Branch 'distro/collabora/co-23.05' - sc/CppunitTest_sc_theme_import_export_test.mk

2023-09-08 Thread Mike Kaganski (via logerrit)
 sc/CppunitTest_sc_theme_import_export_test.mk |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 4c884a7924e4f10217e7785ffa81f137db6c32eb
Author: Mike Kaganski 
AuthorDate: Thu Jun 29 12:04:02 2023 +0300
Commit: Mike Kaganski 
CommitDate: Fri Sep 8 16:35:05 2023 +0200

Fix build

Failing like

  ThemeImportExportTest.o : error LNK2019: unresolved external symbol 
"__declspec(dllimport) public: class Color const & __cdecl 
SvxBrushItem::GetColor(void)const " 
(__imp_?GetColor@SvxBrushItem@@QEBAAEBVColor@@XZ) referenced in function 
"public: void __cdecl testCellBackgroundThemeColor::TestBody(void)" 
(?TestBody@testCellBackgroundThemeColor@@QEAAXXZ)
  ThemeImportExportTest.o : error LNK2019: unresolved external symbol 
"__declspec(dllimport) public: class model::ComplexColor const & __cdecl 
SvxBrushItem::getComplexColor(void)const " 
(__imp_?getComplexColor@SvxBrushItem@@QEBAAEBVComplexColor@model@@XZ) 
referenced in function "public: void __cdecl 
testCellBackgroundThemeColor::TestBody(void)" 
(?TestBody@testCellBackgroundThemeColor@@QEAAXXZ)
  
C:\lo\src\build\workdir\LinkTarget\CppunitTest\test_sc_theme_import_export_test.dll
 : fatal error LNK1120: 2 unresolved externals

Change-Id: I7a64405e3e8bf774163d8f101681c2c0de892767
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/153736
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156704
Tested-by: Jenkins CollaboraOffice 

diff --git a/sc/CppunitTest_sc_theme_import_export_test.mk 
b/sc/CppunitTest_sc_theme_import_export_test.mk
index 22b0a2eea6a2..9b2ce60f595f 100644
--- a/sc/CppunitTest_sc_theme_import_export_test.mk
+++ b/sc/CppunitTest_sc_theme_import_export_test.mk
@@ -29,6 +29,7 @@ $(eval $(call 
gb_CppunitTest_use_libraries,sc_theme_import_export_test, \
 comphelper \
 cppu \
 cppuhelper \
+editeng \
 sal \
 salhelper \
 sax \


Re: CppunitTest_dbaccess_hsqldb_test failure

2023-09-08 Thread Dan Horák
On Fri, 8 Sep 2023 14:02:54 +0200
Dan Horák  wrote:

> Hello,
> 
> I am seeing CppunitTest_dbaccess_hsqldb_test failing on some systems and
> on some occasions.
> 
> ...
> Running scope as unit: 
> -home-jenkins-workspace-libreoffice\x2dupstream\x2dbundled-label-LO\x2dppc64le-workdir-CppunitTest-dbaccess_hsqldb_test.test:20230907213220:2142227.scope
> [_RUN_] HSQLDBTest::testEmptyDBConnection
> HSQLDBTest::testEmptyDBConnection finished in: 1353ms
> OK (1)
>PID: 2142247 (cppunittester)
>UID: 1003 (jenkins)
>GID: 1003 (jenkins)
> Signal: 5 (TRAP)
>  Timestamp: Thu 2023-09-07 21:32:22 UTC (15s ago)
>   Command Line: 
> /home/jenkins/workspace/libreoffice-upstream-bundled/label/LO-ppc64le/workdir/LinkTarget/Executable/cppunittester
>  
> /home/jenkins/workspace/libreoffice-upstream-bundled/label/LO-ppc64le/workdir/LinkTarget/CppunitTest/libtest_dbaccess_hsqldb_test.so
>  --headless 
> -env:BRAND_BASE_DIR=file:///home/jenkins/workspace/libreoffice-upstream-bundled/label/LO-ppc64le/instdir
>  -env:BRAND_SHARE_SUBDIR=share 
> -env:BRAND_SHARE_RESOURCE_SUBDIR=program/resource 
> ...
> 
> 
> I suspect it's because the time to complete the check is over 1000ms.
> When the run time captured in
> workdir/CppunitTest/dbaccess_hsqldb_test.test.log
> is bellow 1000 (usually 600-800ms), the test passes. The obvious
> question is where I can extend the timeout?

hmm, or it is not a timeout, got a passed test with added 5 sec sleep
in HSQLDBTest::testEmptyDBConnection() ...


Dan


[Libreoffice-commits] core.git: solenv/flatpak-manifest.in

2023-09-08 Thread Stephan Bergmann (via logerrit)
 solenv/flatpak-manifest.in |7 +++
 1 file changed, 7 insertions(+)

New commits:
commit 93aa47535267435fd327368875e46a33f86b39d6
Author: Stephan Bergmann 
AuthorDate: Fri Sep 8 12:27:26 2023 +0200
Commit: Stephan Bergmann 
CommitDate: Fri Sep 8 15:56:19 2023 +0200

Flatpak build needs external/java_websocket

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

diff --git a/solenv/flatpak-manifest.in b/solenv/flatpak-manifest.in
index 84c35e99a7e6..bb5a174cba3f 100644
--- a/solenv/flatpak-manifest.in
+++ b/solenv/flatpak-manifest.in
@@ -681,6 +681,13 @@
 "dest": "external/tarballs",
 "dest-filename": "@FROZEN_TARBALL@"
 },
+{
+"url": 
"https://dev-www.libreoffice.org/src/@JAVA_WEBSOCKET_TARBALL@";,
+"sha256": "@JAVA_WEBSOCKET_SHA256SUM@",
+"type": "file",
+"dest": "external/tarballs",
+"dest-filename": "@JAVA_WEBSOCKET_TARBALL@"
+},
 {
 "url": 
"https://dev-www.libreoffice.org/extern/@OPENSYMBOL_TTF@";,
 "sha256": "@OPENSYMBOL_SHA256SUM@",


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

2023-09-08 Thread Noel Grandin (via logerrit)
 xmloff/source/text/XMLIndexSimpleEntryContext.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 425ca75d71497aebae5f5fdfdb20f65a6422a9d3
Author: Noel Grandin 
AuthorDate: Fri Sep 8 13:29:49 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 15:36:48 2023 +0200

WARN->INFO in XMLIndexSimpleEntryContext

This is not generally a problem

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

diff --git a/xmloff/source/text/XMLIndexSimpleEntryContext.cxx 
b/xmloff/source/text/XMLIndexSimpleEntryContext.cxx
index d8f2e79bc259..6c59f13380f3 100644
--- a/xmloff/source/text/XMLIndexSimpleEntryContext.cxx
+++ b/xmloff/source/text/XMLIndexSimpleEntryContext.cxx
@@ -75,7 +75,7 @@ void XMLIndexSimpleEntryContext::startFastElement(
 m_bCharStyleNameOK = false;
 }
 else
-XMLOFF_WARN_UNKNOWN("xmloff", aIter);
+XMLOFF_INFO_UNKNOWN("xmloff", aIter);
 }
 
 // if we have a style name, set it!


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

2023-09-08 Thread Mike Kaganski (via logerrit)
 sc/qa/unit/ThemeImportExportTest.cxx |   26 +-
 1 file changed, 13 insertions(+), 13 deletions(-)

New commits:
commit 48720d869df42bdcf2d9444e508068e91a3bd9c3
Author: Mike Kaganski 
AuthorDate: Tue Jul 4 11:50:01 2023 +0200
Commit: Mike Kaganski 
CommitDate: Fri Sep 8 15:19:04 2023 +0200

Fix build on Win64, where int != sal_Int32

Change-Id: Id835da668042a656ace0f09a7fa9bc852b968b3e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/153926
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 
(cherry picked from commit b68e4a2d0179558180085bfe28a32a1510aeb3e1)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/154251
Reviewed-by: Michael Stahl 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156703
Tested-by: Jenkins CollaboraOffice 

diff --git a/sc/qa/unit/ThemeImportExportTest.cxx 
b/sc/qa/unit/ThemeImportExportTest.cxx
index 7d0eb47ee2c9..e8e1a31dbbff 100644
--- a/sc/qa/unit/ThemeImportExportTest.cxx
+++ b/sc/qa/unit/ThemeImportExportTest.cxx
@@ -194,9 +194,9 @@ void checkCellBackgroundThemeColor(ScDocument* pDoc)
 auto& rTransformations = aComplexColor.getTransformations();
 CPPUNIT_ASSERT_EQUAL(size_t(2), rTransformations.size());
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumMod, 
rTransformations[0].meType);
-CPPUNIT_ASSERT_EQUAL(20, roundToPercent(rTransformations[0].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(20), 
roundToPercent(rTransformations[0].mnValue));
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumOff, 
rTransformations[1].meType);
-CPPUNIT_ASSERT_EQUAL(80, roundToPercent(rTransformations[1].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(80), 
roundToPercent(rTransformations[1].mnValue));
 }
 
 // A3
@@ -213,7 +213,7 @@ void checkCellBackgroundThemeColor(ScDocument* pDoc)
 auto& rTransformations = aComplexColor.getTransformations();
 CPPUNIT_ASSERT_EQUAL(size_t(1), rTransformations.size());
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumMod, 
rTransformations[0].meType);
-CPPUNIT_ASSERT_EQUAL(50, roundToPercent(rTransformations[0].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(50), 
roundToPercent(rTransformations[0].mnValue));
 }
 }
 
@@ -265,9 +265,9 @@ void checkCellTextThemeColor(ScDocument* pDoc)
 auto& rTransformations = aComplexColor.getTransformations();
 CPPUNIT_ASSERT_EQUAL(size_t(2), rTransformations.size());
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumMod, 
rTransformations[0].meType);
-CPPUNIT_ASSERT_EQUAL(60, roundToPercent(rTransformations[0].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(60), 
roundToPercent(rTransformations[0].mnValue));
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumOff, 
rTransformations[1].meType);
-CPPUNIT_ASSERT_EQUAL(40, roundToPercent(rTransformations[1].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(40), 
roundToPercent(rTransformations[1].mnValue));
 }
 
 // B3
@@ -284,7 +284,7 @@ void checkCellTextThemeColor(ScDocument* pDoc)
 auto& rTransformations = aComplexColor.getTransformations();
 CPPUNIT_ASSERT_EQUAL(size_t(1), rTransformations.size());
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumMod, 
rTransformations[0].meType);
-CPPUNIT_ASSERT_EQUAL(50, roundToPercent(rTransformations[0].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(50), 
roundToPercent(rTransformations[0].mnValue));
 }
 }
 
@@ -350,9 +350,9 @@ void checkCellBorderThemeColor(ScDocument* pDoc)
 auto& rTransformations = aComplexColor.getTransformations();
 CPPUNIT_ASSERT_EQUAL(size_t(2), rTransformations.size());
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumMod, 
rTransformations[0].meType);
-CPPUNIT_ASSERT_EQUAL(20, 
roundToPercent(rTransformations[0].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(20), 
roundToPercent(rTransformations[0].mnValue));
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumOff, 
rTransformations[1].meType);
-CPPUNIT_ASSERT_EQUAL(80, 
roundToPercent(rTransformations[1].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(80), 
roundToPercent(rTransformations[1].mnValue));
 }
 {
 auto* pTop = pBoxItem->GetRight();
@@ -363,9 +363,9 @@ void checkCellBorderThemeColor(ScDocument* pDoc)
 auto& rTransformations = aComplexColor.getTransformations();
 CPPUNIT_ASSERT_EQUAL(size_t(2), rTransformations.size());
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumMod, 
rTransformations[0].meType);
-CPPUNIT_ASSERT_EQUAL(20, 
roundToPercent(rTransformations[0].mnValue));
+CPPUNIT_ASSERT_EQUAL(sal_Int32(20), 
roundToPercent(rTransformations[0].mnValue));
 CPPUNIT_ASSERT_EQUAL(model::TransformationType::LumOff, 
rTransformations[1].meType);
-CPPUNIT_ASSER

[Libreoffice-commits] core.git: Branch 'distro/collabora/co-23.05' - filter/source filter/uiconfig offapi/com offapi/UnoApi_offapi.mk sc/inc sc/qa sc/source sd/qa sw/qa

2023-09-08 Thread NickWingate (via logerrit)
 filter/source/pdf/impdialog.cxx   |   62 +++---
 filter/source/pdf/impdialog.hxx   |   12 -
 filter/source/pdf/pdfexport.cxx   |9 
 filter/uiconfig/ui/pdfgeneralpage.ui  |   62 ++
 offapi/UnoApi_offapi.mk   |1 
 offapi/com/sun/star/sheet/XSheetRange.idl |   42 
 sc/inc/viewuno.hxx|5 ++
 sc/qa/uitest/calc_tests4/exportToPDF.py   |2 
 sc/source/ui/unoobj/docuno.cxx|   13 +-
 sc/source/ui/unoobj/viewuno.cxx   |   32 +++
 sc/source/ui/view/printfun.cxx|2 
 sd/qa/uitest/impress_tests/exportToPDF.py |2 
 sw/qa/uitest/writer_tests4/exportToPDF.py |2 
 13 files changed, 217 insertions(+), 29 deletions(-)

New commits:
commit dd9f787875a9a953d1fb84de44e04fd2a0c2f8c1
Author: NickWingate 
AuthorDate: Tue Aug 29 15:31:08 2023 +0100
Commit: Szymon Kłos 
CommitDate: Fri Sep 8 15:04:41 2023 +0200

Add export sheet range option to calc

User can specify which sheets to export e.g. '2-5,7'
exports sheets 2,3,4,5,7.
Note: this is different from exporting pages as one
sheet may contain several pages worth of content.

Also fix a bug where exporting only a selected sheet
causes the next sheet to be exported. e.g.:
Sheet 1 is empty, Sheet 2 has content. Exporting Sheet 1
results in Sheet 2's content being exported

Signed-off-by: NickWingate 
Change-Id: Iecd42188ddbbbcd70eb37bec80783e29e3cb5b19
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156255
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Szymon Kłos 

diff --git a/filter/source/pdf/impdialog.cxx b/filter/source/pdf/impdialog.cxx
index 2723afbf0ab8..d9f3f3576a4e 100644
--- a/filter/source/pdf/impdialog.cxx
+++ b/filter/source/pdf/impdialog.cxx
@@ -106,8 +106,10 @@ ImpPDFTabDialog::ImpPDFTabDialog(weld::Window* pParent, 
const Sequence< Property
 mbCanCopyOrExtract( false ),
 mbCanExtractForAccessibility( true ),
 
-mbIsRangeChecked( false ),
+mbIsPageRangeChecked( false ),
 msPageRange( ' ' ),
+mbIsSheetRangeChecked( false ),
+msSheetRange( ' ' ),
 
 mbSelectionIsChecked( false ),
 mbExportRelativeFsysLinks( false ),
@@ -492,8 +494,10 @@ Sequence< PropertyValue > ImpPDFTabDialog::GetFilterData()
 comphelper::makePropertyValue("RestrictPermissions", 
mbRestrictPermissions),
 comphelper::makePropertyValue("PreparedPermissionPassword", 
maPreparedOwnerPassword)
 };
-if( mbIsRangeChecked )
+if( mbIsPageRangeChecked )
 aRet.push_back(comphelper::makePropertyValue("PageRange", 
msPageRange));
+if( mbIsSheetRangeChecked )
+aRet.push_back(comphelper::makePropertyValue("SheetRange", 
msSheetRange));
 else if( mbSelectionIsChecked )
 aRet.push_back(comphelper::makePropertyValue("Selection", 
maSelection));
 
@@ -517,9 +521,11 @@ 
ImpPDFTabGeneralPage::ImpPDFTabGeneralPage(weld::Container* pPage, weld::DialogC
 , mbIsWriter(false)
 , mpParent(nullptr)
 , mxRbAll(m_xBuilder->weld_radio_button("all"))
-, mxRbRange(m_xBuilder->weld_radio_button("range"))
+, mxRbPageRange(m_xBuilder->weld_radio_button("pagerange"))
+, mxRbSheetRange(m_xBuilder->weld_radio_button("sheetrange"))
 , mxRbSelection(m_xBuilder->weld_radio_button("selection"))
 , mxEdPages(m_xBuilder->weld_entry("pages"))
+, mxEdSheets(m_xBuilder->weld_entry("sheets"))
 , 
mxRbLosslessCompression(m_xBuilder->weld_radio_button("losslesscompress"))
 , mxRbJPEGCompression(m_xBuilder->weld_radio_button("jpegcompress"))
 , mxQualityFrame(m_xBuilder->weld_widget("qualityframe"))
@@ -550,7 +556,7 @@ ImpPDFTabGeneralPage::ImpPDFTabGeneralPage(weld::Container* 
pPage, weld::DialogC
 , mxFtWatermark(m_xBuilder->weld_label("watermarklabel"))
 , mxEdWatermark(m_xBuilder->weld_entry("watermarkentry"))
 , mxSlidesFt(m_xBuilder->weld_label("slides"))
-, mxSheetsFt(m_xBuilder->weld_label("selectedsheets"))
+, mxSheetsSelectionFt(m_xBuilder->weld_label("selectedsheets"))
 {
 }
 
@@ -565,11 +571,13 @@ void 
ImpPDFTabGeneralPage::SetFilterConfigItem(ImpPDFTabDialog* pParent)
 mpParent = pParent;
 
 // init this class data
-mxRbRange->connect_toggled( LINK( this, ImpPDFTabGeneralPage, 
TogglePagesHdl ) );
+mxRbPageRange->connect_toggled( LINK( this, ImpPDFTabGeneralPage, 
TogglePagesHdl ) );
+mxRbSheetRange->connect_toggled( LINK( this, ImpPDFTabGeneralPage, 
ToggleSheetsHdl ) );
 
 mxRbAll->set_active(true);
 mxRbAll->connect_toggled( LINK( this, ImpPDFTabGeneralPage, ToggleAllHdl ) 
);
 TogglePagesHdl();
+ToggleSheetsHdl();
 
 mxRbSelection->set_sensitive( pParent->mbSelectionPresent );
 if ( pParent->mbSelectionPresent )
@@ -654,7 +662,7 @@ void 
ImpPDFTabGeneralPage::SetFilterConfigItem(ImpPDFTabDialog* pParent)
 
 if ( mbIsPresent

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

2023-09-08 Thread Noel Grandin (via logerrit)
 sw/source/uibase/uno/unomod.cxx |   38 +++---
 1 file changed, 19 insertions(+), 19 deletions(-)

New commits:
commit edffc763fb56453dc2eba15bb8c9450639c69db1
Author: Noel Grandin 
AuthorDate: Fri Sep 8 13:28:05 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 14:54:41 2023 +0200

improve exception message

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

diff --git a/sw/source/uibase/uno/unomod.cxx b/sw/source/uibase/uno/unomod.cxx
index 499f4ebea0e7..63250ede8526 100644
--- a/sw/source/uibase/uno/unomod.cxx
+++ b/sw/source/uibase/uno/unomod.cxx
@@ -284,11 +284,11 @@ void SwXPrintSettings::_preSetValues ()
 
 namespace
 {
-bool tryBoolAccess(const uno::Any &rValue)
+bool tryBoolAccess(std::u16string_view rName, const uno::Any &rValue)
 {
 const std::optional xPrSet = o3tl::tryAccess(rValue);
 if (!xPrSet.has_value())
-throw lang::IllegalArgumentException();
+throw 
lang::IllegalArgumentException(OUString(OUString::Concat(rName) + " no value"), 
nullptr, 0);
 return *xPrSet;
 }
 }
@@ -299,62 +299,62 @@ void SwXPrintSettings::_setSingleValue( const 
comphelper::PropertyInfo & rInfo,
 {
 case HANDLE_PRINTSET_LEFT_PAGES:
 {
-mpPrtOpt->SetPrintLeftPage(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintLeftPage(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_RIGHT_PAGES:
 {
-mpPrtOpt->SetPrintRightPage(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintRightPage(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_REVERSED:
 {
-mpPrtOpt->SetPrintReverse(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintReverse(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_PROSPECT:
 {
-mpPrtOpt->SetPrintProspect(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintProspect(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_GRAPHICS:
 {
-mpPrtOpt->SetPrintGraphic(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintGraphic(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_TABLES:
 {
-mpPrtOpt->SetPrintTable(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintTable(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_DRAWINGS:
 {
-mpPrtOpt->SetPrintDraw(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintDraw(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_CONTROLS:
 {
-mpPrtOpt->SetPrintControl(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintControl(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_PAGE_BACKGROUND:
 {
-mpPrtOpt->SetPrintPageBackground(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintPageBackground(tryBoolAccess(rInfo.maName, 
rValue));
 }
 break;
 case HANDLE_PRINTSET_BLACK_FONTS:
 {
-mpPrtOpt->SetPrintBlackFont(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintBlackFont(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_SINGLE_JOBS:
 {
-mpPrtOpt->SetPrintSingleJobs(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintSingleJobs(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_PAPER_FROM_SETUP:
 {
-mpPrtOpt->SetPaperFromSetup(tryBoolAccess(rValue));
+mpPrtOpt->SetPaperFromSetup(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_ANNOTATION_MODE:
@@ -363,14 +363,14 @@ void SwXPrintSettings::_setSingleValue( const 
comphelper::PropertyInfo & rInfo,
 rValue >>= nTmp;
 SwPostItMode nVal = static_cast(nTmp);
 if(nVal > SwPostItMode::EndPage)
-throw lang::IllegalArgumentException();
+throw lang::IllegalArgumentException(OUString::number(nTmp) + 
" > SwPostItMode::EndPage", nullptr, 0);
 
 mpPrtOpt->SetPrintPostIts(nVal);
 }
 break;
 case HANDLE_PRINTSET_EMPTY_PAGES:
 {
-mpPrtOpt->SetPrintEmptyPages(tryBoolAccess(rValue));
+mpPrtOpt->SetPrintEmptyPages(tryBoolAccess(rInfo.maName, rValue));
 }
 break;
 case HANDLE_PRINTSET_FAX_NAME:
@@ -384,17 +384,17 @@ void SwXPrintSettings::_setSingleValue( const 
comphelper::PropertyInfo & rInfo,
 break;
 case HANDLE_PRINTSET_PROSPECT_RTL:
 {
-   

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

2023-09-08 Thread Samuel Mehrbrodt (via logerrit)
 sw/source/core/access/AccessibilityIssue.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 2768ada1943a4dfb7ac2fc080656e151dd55efb8
Author: Samuel Mehrbrodt 
AuthorDate: Thu Sep 7 13:56:30 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Sep 8 14:07:11 2023 +0200

tdf#157134 Fix selecting shape when image was selected before

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

diff --git a/sw/source/core/access/AccessibilityIssue.cxx 
b/sw/source/core/access/AccessibilityIssue.cxx
index 731930a9500e..6ec39238cda9 100644
--- a/sw/source/core/access/AccessibilityIssue.cxx
+++ b/sw/source/core/access/AccessibilityIssue.cxx
@@ -91,6 +91,8 @@ void AccessibilityIssue::gotoIssue() const
 case IssueObject::SHAPE:
 {
 SwWrtShell* pWrtShell = 
TempIssueObject.m_pDoc->GetDocShell()->GetWrtShell();
+if (pWrtShell->IsFrameSelected())
+pWrtShell->LeaveSelFrameMode();
 pWrtShell->GotoDrawingObject(TempIssueObject.m_sObjectID);
 if (comphelper::LibreOfficeKit::isActive())
 pWrtShell->ShowCursor();


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

2023-09-08 Thread Noel Grandin (via logerrit)
 include/cppuhelper/queryinterface.hxx |  248 ++
 sc/inc/miscuno.hxx|   10 -
 sc/source/ui/unoobj/cellsuno.cxx  |  133 ++
 sc/source/ui/unoobj/cursuno.cxx   |   11 -
 sc/source/ui/unoobj/dapiuno.cxx   |9 -
 sc/source/ui/unoobj/docuno.cxx|   37 ++---
 sc/source/ui/unoobj/viewuno.cxx   |   50 --
 7 files changed, 385 insertions(+), 113 deletions(-)

New commits:
commit fbe8cc7feb9ead5918b0c52d4211caf4a10c8212
Author: Noel Grandin 
AuthorDate: Fri Sep 8 11:28:30 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 14:03:14 2023 +0200

inline SC_QUERYINTERFACE macros

which means we need a few more cppu::queryInterface variants

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

diff --git a/include/cppuhelper/queryinterface.hxx 
b/include/cppuhelper/queryinterface.hxx
index df9c9916cf8e..25961154b4d1 100644
--- a/include/cppuhelper/queryinterface.hxx
+++ b/include/cppuhelper/queryinterface.hxx
@@ -519,6 +519,254 @@ inline css::uno::Any SAL_CALL queryInterface(
 return css::uno::Any();
 }
 
+/** Compares demanded type to given template argument types.
+@return acquired interface of demanded type or empty Any
+*/
+template< class Interface1, class Interface2, class Interface3, class 
Interface4, class Interface5,
+  class Interface6, class Interface7, class Interface8, class 
Interface9, class Interface10,
+  class Interface11, class Interface12, class Interface13 >
+inline css::uno::Any SAL_CALL queryInterface(
+const css::uno::Type & rType,
+Interface1 * p1, Interface2 * p2, Interface3 * p3, Interface4 * p4, 
Interface5 * p5,
+Interface6 * p6, Interface7 * p7, Interface8 * p8, Interface9 * p9, 
Interface10 * p10,
+Interface11 * p11, Interface12 * p12, Interface13 * p13 )
+{
+if (rType == Interface1::static_type())
+return css::uno::Any( &p1, rType );
+else if (rType == Interface2::static_type())
+return css::uno::Any( &p2, rType );
+else if (rType == Interface3::static_type())
+return css::uno::Any( &p3, rType );
+else if (rType == Interface4::static_type())
+return css::uno::Any( &p4, rType );
+else if (rType == Interface5::static_type())
+return css::uno::Any( &p5, rType );
+else if (rType == Interface6::static_type())
+return css::uno::Any( &p6, rType );
+else if (rType == Interface7::static_type())
+return css::uno::Any( &p7, rType );
+else if (rType == Interface8::static_type())
+return css::uno::Any( &p8, rType );
+else if (rType == Interface9::static_type())
+return css::uno::Any( &p9, rType );
+else if (rType == Interface10::static_type())
+return css::uno::Any( &p10, rType );
+else if (rType == Interface11::static_type())
+return css::uno::Any( &p11, rType );
+else if (rType == Interface12::static_type())
+return css::uno::Any( &p12, rType );
+else if (rType == Interface13::static_type())
+return css::uno::Any( &p13, rType );
+else
+return css::uno::Any();
+}
+
+/** Compares demanded type to given template argument types.
+@return acquired interface of demanded type or empty Any
+*/
+template< class Interface1, class Interface2, class Interface3, class 
Interface4, class Interface5,
+  class Interface6, class Interface7, class Interface8, class 
Interface9, class Interface10,
+  class Interface11, class Interface12, class Interface13, class 
Interface14 >
+inline css::uno::Any SAL_CALL queryInterface(
+const css::uno::Type & rType,
+Interface1 * p1, Interface2 * p2, Interface3 * p3, Interface4 * p4, 
Interface5 * p5,
+Interface6 * p6, Interface7 * p7, Interface8 * p8, Interface9 * p9, 
Interface10 * p10,
+Interface11 * p11, Interface12 * p12, Interface13 * p13, Interface14 * p14 
)
+{
+if (rType == Interface1::static_type())
+return css::uno::Any( &p1, rType );
+else if (rType == Interface2::static_type())
+return css::uno::Any( &p2, rType );
+else if (rType == Interface3::static_type())
+return css::uno::Any( &p3, rType );
+else if (rType == Interface4::static_type())
+return css::uno::Any( &p4, rType );
+else if (rType == Interface5::static_type())
+return css::uno::Any( &p5, rType );
+else if (rType == Interface6::static_type())
+return css::uno::Any( &p6, rType );
+else if (rType == Interface7::static_type())
+return css::uno::Any( &p7, rType );
+else if (rType == Interface8::static_type())
+return css::uno::Any( &p8, rType );
+else if (rType == Interface9::static_type())
+return css::uno::Any( &p9, rType );
+else if (rType == Interface10::static_type())
+return css::uno::Any( &p10, rType 

CppunitTest_dbaccess_hsqldb_test failure

2023-09-08 Thread Dan Horák
Hello,

I am seeing CppunitTest_dbaccess_hsqldb_test failing on some systems and
on some occasions.

...
Running scope as unit: 
-home-jenkins-workspace-libreoffice\x2dupstream\x2dbundled-label-LO\x2dppc64le-workdir-CppunitTest-dbaccess_hsqldb_test.test:20230907213220:2142227.scope
[_RUN_] HSQLDBTest::testEmptyDBConnection
HSQLDBTest::testEmptyDBConnection finished in: 1353ms
OK (1)
   PID: 2142247 (cppunittester)
   UID: 1003 (jenkins)
   GID: 1003 (jenkins)
Signal: 5 (TRAP)
 Timestamp: Thu 2023-09-07 21:32:22 UTC (15s ago)
  Command Line: 
/home/jenkins/workspace/libreoffice-upstream-bundled/label/LO-ppc64le/workdir/LinkTarget/Executable/cppunittester
 
/home/jenkins/workspace/libreoffice-upstream-bundled/label/LO-ppc64le/workdir/LinkTarget/CppunitTest/libtest_dbaccess_hsqldb_test.so
 --headless 
-env:BRAND_BASE_DIR=file:///home/jenkins/workspace/libreoffice-upstream-bundled/label/LO-ppc64le/instdir
 -env:BRAND_SHARE_SUBDIR=share 
-env:BRAND_SHARE_RESOURCE_SUBDIR=program/resource 
...


I suspect it's because the time to complete the check is over 1000ms.
When the run time captured in
workdir/CppunitTest/dbaccess_hsqldb_test.test.log
is bellow 1000 (usually 600-800ms), the test passes. The obvious
question is where I can extend the timeout?


Thanks,

Dan


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

2023-09-08 Thread Julien Nabet (via logerrit)
 cui/source/tabpages/tparea.cxx |7 ++-
 1 file changed, 2 insertions(+), 5 deletions(-)

New commits:
commit b69e14038288387b5f288a06821fb5df66dcf94e
Author: Julien Nabet 
AuthorDate: Thu Sep 7 16:34:53 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Sep 8 13:26:15 2023 +0200

tdf#157138: Can't switch from Use Background to None area fill

Revert partly tdf#151260 fix

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

diff --git a/cui/source/tabpages/tparea.cxx b/cui/source/tabpages/tparea.cxx
index 23946aa7c4b7..9107bace195c 100644
--- a/cui/source/tabpages/tparea.cxx
+++ b/cui/source/tabpages/tparea.cxx
@@ -240,11 +240,8 @@ DeactivateRC SvxAreaTabPage::DeactivatePage( SfxItemSet* 
_pSet )
 {
 XFillStyleItem aStyleItem( drawing::FillStyle_NONE );
 _pSet->Put( aStyleItem );
-if (_pSet->HasItem(XATTR_FILLUSESLIDEBACKGROUND))
-{
-XFillUseSlideBackgroundItem aFillBgItem( false );
-_pSet->Put( aFillBgItem );
-}
+XFillUseSlideBackgroundItem aFillBgItem( false );
+_pSet->Put( aFillBgItem );
 }
 break;
 }


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

2023-09-08 Thread Samuel Mehrbrodt (via logerrit)
 cui/source/dialogs/dlgname.cxx   |5 -
 cui/source/factory/dlgfact.cxx   |4 ++--
 cui/source/factory/dlgfact.hxx   |2 +-
 include/cui/dlgname.hxx  |3 ++-
 include/svx/svxdlg.hxx   |2 +-
 sw/inc/AccessibilityCheckStrings.hrc |3 ++-
 sw/source/core/access/AccessibilityIssue.cxx |6 +++---
 7 files changed, 15 insertions(+), 10 deletions(-)

New commits:
commit c7a608a6691c790783c63f504010bc796c36af25
Author: Samuel Mehrbrodt 
AuthorDate: Thu Sep 7 12:49:46 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Sep 8 13:24:21 2023 +0200

tdf#155503 Add title to document title dialog

Otherwise the title was just "Name"

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

diff --git a/cui/source/dialogs/dlgname.cxx b/cui/source/dialogs/dlgname.cxx
index fa12a158445f..4e6847e0f5cc 100644
--- a/cui/source/dialogs/dlgname.cxx
+++ b/cui/source/dialogs/dlgname.cxx
@@ -25,7 +25,8 @@
 |*
 \/
 
-SvxNameDialog::SvxNameDialog(weld::Window* pParent, const OUString& rName, 
const OUString& rDesc)
+SvxNameDialog::SvxNameDialog(weld::Window* pParent, const OUString& rName, 
const OUString& rDesc,
+ const OUString& rTitle)
 : GenericDialogController(pParent, "cui/ui/namedialog.ui", "NameDialog")
 , m_xEdtName(m_xBuilder->weld_entry("name_entry"))
 , m_xFtDescription(m_xBuilder->weld_label("description_label"))
@@ -36,6 +37,8 @@ SvxNameDialog::SvxNameDialog(weld::Window* pParent, const 
OUString& rName, const
 m_xEdtName->select_region(0, -1);
 ModifyHdl(*m_xEdtName);
 m_xEdtName->connect_changed(LINK(this, SvxNameDialog, ModifyHdl));
+if (!rTitle.isEmpty())
+set_title(rTitle);
 }
 
 IMPL_LINK_NOARG(SvxNameDialog, ModifyHdl, weld::Entry&, void)
diff --git a/cui/source/factory/dlgfact.cxx b/cui/source/factory/dlgfact.cxx
index d0b0fceed8a3..f558f9e4a37d 100644
--- a/cui/source/factory/dlgfact.cxx
+++ b/cui/source/factory/dlgfact.cxx
@@ -1092,9 +1092,9 @@ VclPtr 
AbstractDialogFactory_Impl::CreateSvxEditDictionaryDia
 }
 
 VclPtr 
AbstractDialogFactory_Impl::CreateSvxNameDialog(weld::Window* pParent,
-const OUString& rName, const OUString& 
rDesc)
+const OUString& rName, const OUString& 
rDesc, const OUString& rTitle)
 {
-return 
VclPtr::Create(std::make_unique(pParent,
 rName, rDesc));
+return 
VclPtr::Create(std::make_unique(pParent,
 rName, rDesc, rTitle));
 }
 
 VclPtr 
AbstractDialogFactory_Impl::CreateSvxObjectNameDialog(weld::Window* pParent, 
const OUString& rName)
diff --git a/cui/source/factory/dlgfact.hxx b/cui/source/factory/dlgfact.hxx
index cd3f638aced5..4adc0d82ee47 100644
--- a/cui/source/factory/dlgfact.hxx
+++ b/cui/source/factory/dlgfact.hxx
@@ -521,7 +521,7 @@ public:
 virtual VclPtr 
CreateSvxNewDictionaryDialog(weld::Window* pParent) override;
 virtual VclPtr 
CreateSvxEditDictionaryDialog(weld::Window* pParent, const OUString& rName) 
override;
 virtual VclPtr CreateSvxNameDialog(weld::Window* 
pParent,
-const OUString& rName, const 
OUString& rDesc) override;
+const OUString& rName, const 
OUString& rDesc, const OUString& rTitle = "") override;
 // #i68101#
 virtual VclPtr 
CreateSvxObjectNameDialog(weld::Window* pParent, const OUString& rName) 
override;
 virtual VclPtr 
CreateSvxObjectTitleDescDialog(weld::Window* pParent, const OUString& rTitle, 
const OUString& rDescription, bool isDecorative) override;
diff --git a/include/cui/dlgname.hxx b/include/cui/dlgname.hxx
index 9083177ee980..16938734ff97 100644
--- a/include/cui/dlgname.hxx
+++ b/include/cui/dlgname.hxx
@@ -36,7 +36,8 @@ private:
 DECL_LINK(ModifyHdl, weld::Entry&, void);
 
 public:
-SvxNameDialog(weld::Window* pWindow, const OUString& rName, const 
OUString& rDesc);
+SvxNameDialog(weld::Window* pWindow, const OUString& rName, const 
OUString& rDesc,
+  const OUString& rTitle = "");
 
 OUString GetName() const { return m_xEdtName->get_text(); }
 
diff --git a/include/svx/svxdlg.hxx b/include/svx/svxdlg.hxx
index d5a0b5fec7fc..436bfd430368 100644
--- a/include/svx/svxdlg.hxx
+++ b/include/svx/svxdlg.hxx
@@ -377,7 +377,7 @@ public:
 virtual VclPtr 
CreateSvxNewDictionaryDialog(weld::Window* pParent) = 0;
 virtual VclPtr 
CreateSvxEditDictionaryDialog(weld::Window* pParent, const OUString& rName) = 0;
 virtual VclPtr CreateSvxNameDialog(weld::Window* 
pParent,
-const OUString& rName, const 
OUString& rDesc ) = 0;
+const OUSt

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

2023-09-08 Thread Samuel Mehrbrodt (via logerrit)
 sw/source/core/access/AccessibilityIssue.cxx |   11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

New commits:
commit 48129cabb9a5636207d3b8f4f9fa4e4abac61b8a
Author: Samuel Mehrbrodt 
AuthorDate: Thu Sep 7 11:35:52 2023 +0200
Commit: Samuel Mehrbrodt 
CommitDate: Fri Sep 8 13:23:18 2023 +0200

tdf#157085 Make doc title dialog modal

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

diff --git a/sw/source/core/access/AccessibilityIssue.cxx 
b/sw/source/core/access/AccessibilityIssue.cxx
index 080ce2ec7a9e..a915289f26ea 100644
--- a/sw/source/core/access/AccessibilityIssue.cxx
+++ b/sw/source/core/access/AccessibilityIssue.cxx
@@ -236,8 +236,11 @@ void AccessibilityIssue::quickFixIssue() const
 case IssueObject::DOCUMENT_TITLE:
 {
 OUString aDesc = SwResId(STR_ENTER_DOCUMENT_TITLE);
-SvxNameDialog aNameDialog(m_pParent, "", aDesc);
-if (aNameDialog.run() == RET_OK)
+SvxAbstractDialogFactory* pFact = 
SvxAbstractDialogFactory::Create();
+SwWrtShell* pWrtShell = m_pDoc->GetDocShell()->GetWrtShell();
+ScopedVclPtr aNameDialog(
+
pFact->CreateSvxNameDialog(pWrtShell->GetView().GetFrameWeld(), "", aDesc));
+if (aNameDialog->Execute() == RET_OK)
 {
 SwDocShell* pShell = m_pDoc->GetDocShell();
 if (!pShell)
@@ -247,7 +250,9 @@ void AccessibilityIssue::quickFixIssue() const
 pShell->GetModel(), uno::UNO_QUERY_THROW);
 const uno::Reference 
xDocumentProperties(
 xDPS->getDocumentProperties());
-xDocumentProperties->setTitle(aNameDialog.GetName());
+OUString sName;
+aNameDialog->GetName(sName);
+xDocumentProperties->setTitle(sName);
 
 
m_pDoc->getOnlineAccessibilityCheck()->resetAndQueueDocumentLevel();
 }


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

2023-09-08 Thread Noel Grandin (via logerrit)
 sc/inc/miscuno.hxx   |8 
 sc/source/core/data/dpoutput.cxx |2 +-
 2 files changed, 1 insertion(+), 9 deletions(-)

New commits:
commit 7585667c83578e7f073409b2b94a4d34e7124115
Author: Noel Grandin 
AuthorDate: Fri Sep 8 11:30:25 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 13:12:05 2023 +0200

replace ScUnoHelpFunctions::VectorToSequence..

with the more standard comphelper::containerToSequence

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

diff --git a/sc/inc/miscuno.hxx b/sc/inc/miscuno.hxx
index 1a17e7afe7ea..362212c1564b 100644
--- a/sc/inc/miscuno.hxx
+++ b/sc/inc/miscuno.hxx
@@ -195,14 +195,6 @@ public:
 SetOptionalPropertyValue(rPropSet, rPropName, any);
 }
 
-template
-static css::uno::Sequence VectorToSequence( const 
std::vector& rVector )
-{
-if (rVector.empty())
-return css::uno::Sequence();
-
-return css::uno::Sequence(&rVector[0], 
static_cast(rVector.size()));
-}
 private:
 static sal_Int32GetEnumPropertyImpl( const css::uno::Reference< 
css::beans::XPropertySet>& xProp,
 const OUString& rName, sal_Int32 
nDefault );
diff --git a/sc/source/core/data/dpoutput.cxx b/sc/source/core/data/dpoutput.cxx
index c964dda14c7a..c74f43d10bd7 100644
--- a/sc/source/core/data/dpoutput.cxx
+++ b/sc/source/core/data/dpoutput.cxx
@@ -501,7 +501,7 @@ uno::Sequence 
getVisiblePageMembersAsResults( const uno::Re
 // All members are visible.  Return empty result.
 return uno::Sequence();
 
-return ScUnoHelpFunctions::VectorToSequence(aRes);
+return comphelper::containerToSequence(aRes);
 }
 
 }


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

2023-09-08 Thread Miklos Vajna (via logerrit)
 oox/source/drawingml/fillproperties.cxx |4 +++-
 sd/qa/unit/data/pptx/tdf156649.pptx |binary
 sd/qa/unit/export-tests.cxx |   14 ++
 sw/qa/core/layout/flycnt.cxx|8 
 sw/source/core/layout/tabfrm.cxx|4 +++-
 sw/source/core/text/txtfly.cxx  |   17 +
 6 files changed, 45 insertions(+), 2 deletions(-)

New commits:
commit 8ac547a463c4061961f924e858a377525ea821b1
Author: Miklos Vajna 
AuthorDate: Thu Sep 7 08:25:18 2023 +0200
Commit: Xisco Fauli 
CommitDate: Fri Sep 8 12:54:20 2023 +0200

sw floattable, nesting: fix position of the inner follow table

The bugdoc was no longer crashing, but the inner table's follow part on
page 2 was not visible.

The problem was that lcl_ArrangeLowers() didn't try to update the fly's
position when the cell's position changed, now we do this for split
flys.

The other problem was that as SwTextFly::GetTop() is called by
SwTextFly::InitAnchoredObjList(), the inner flys were ignored while
collecting the intersecting fly frames for a paragraph, leading to an
overlap between the inner follow fly and the inner anchor text. This is
now fixed by explicitly checking for the splitfly-in-splitfly case.

With this, the ODT bugdoc now renders correctly when opened. (This is
related to tdf#55160.)

Change-Id: I60997e7913984872319250b0fb1cb91e02512800
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156632
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins
(cherry picked from commit 3aa3f0a1638a8d8006955b62bb647526768be3d8)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156607
Reviewed-by: Xisco Fauli 

diff --git a/sw/qa/core/layout/flycnt.cxx b/sw/qa/core/layout/flycnt.cxx
index 0839e6a34b36..797d5d8c5691 100644
--- a/sw/qa/core/layout/flycnt.cxx
+++ b/sw/qa/core/layout/flycnt.cxx
@@ -1060,6 +1060,14 @@ CPPUNIT_TEST_FIXTURE(Test, testSplitFlyNested)
 CPPUNIT_ASSERT(pPage2Fly1);
 CPPUNIT_ASSERT(pPage2Fly1->GetAnchorFrameContainingAnchPos()->IsInFly());
 CPPUNIT_ASSERT(pPage2Fly1->GetPrecede());
+
+// Without the accompanying fix in place, this test would have failed with:
+// - Expected greater than: 6204
+// - Actual  : 1725
+// i.e. the inner follow fly had a bad position, it was outside the page 
rectangle, it was not
+// rendered and this way the inner anchor had no fly portion, either.
+CPPUNIT_ASSERT_GREATER(pPage2->getFrameArea().Top(), 
pPage2Fly1->getFrameArea().Top());
+
 auto pPage2Fly2 = 
rPage2Objs[1]->DynCastFlyFrame()->DynCastFlyAtContentFrame();
 CPPUNIT_ASSERT(pPage2Fly2);
 CPPUNIT_ASSERT(!pPage2Fly2->GetAnchorFrameContainingAnchPos()->IsInFly());
diff --git a/sw/source/core/layout/tabfrm.cxx b/sw/source/core/layout/tabfrm.cxx
index e592467751b7..f7e889094adc 100644
--- a/sw/source/core/layout/tabfrm.cxx
+++ b/sw/source/core/layout/tabfrm.cxx
@@ -5303,10 +5303,12 @@ static bool lcl_ArrangeLowers( SwLayoutFrame *pLay, 
tools::Long lYStart, bool bI
 // on the object positioning.
 // #i52904# - no direct move of objects,
 // whose vertical position doesn't depend on anchor 
frame.
+// Also move split flys directly, otherwise the 
follows would not be moved
+// at all.
 const bool bDirectMove =
 FAR_AWAY != pFly->getFrameArea().Top() &&
 bVertPosDepOnAnchor &&
-!pFly->ConsiderObjWrapInfluenceOnObjPos();
+(!pFly->ConsiderObjWrapInfluenceOnObjPos() || 
pFly->IsFlySplitAllowed());
 if ( bDirectMove )
 {
 {
diff --git a/sw/source/core/text/txtfly.cxx b/sw/source/core/text/txtfly.cxx
index e5bf03162db4..a290ac4d2010 100644
--- a/sw/source/core/text/txtfly.cxx
+++ b/sw/source/core/text/txtfly.cxx
@@ -679,6 +679,23 @@ bool SwTextFly::GetTop( const SwAnchoredObject* 
_pAnchoredObj,
 bool bEvade = !mpCurrAnchoredObj ||
   Is_Lower_Of( mpCurrAnchoredObj->DynCastFlyFrame(), 
pNew);
 
+auto pFly = _pAnchoredObj->DynCastFlyFrame();
+if (pFly && pFly->IsFlySplitAllowed())
+{
+// Check if _pAnchoredObj is a split fly inside an other split 
fly. Always collect such
+// flys, otherwise the inner anchor text will overlap with the 
inner fly.
+SwFrame* pFlyAnchor = const_cast(_pAnchoredObj)
+->GetAnchorFrameContainingAnchPos();
+if (pFlyAnchor && pFlyAnchor->IsInFly())
+{
+auto pOuterFly = pFlyAnchor->FindFlyFrame();
+if (pOuterFly && pOuterFly->IsFlySplitAllowed())
+{
+return true;
+}
+

[Libreoffice-commits] core.git: Branch 'distro/allotropia/zeta-7-4' - sfx2/source

2023-09-08 Thread Balazs Varga (via logerrit)
 sfx2/source/dialog/backingwindow.cxx |   18 +-
 1 file changed, 1 insertion(+), 17 deletions(-)

New commits:
commit 096c70435c7f1e0cd1f91933dd9f1aff4eaf1ceb
Author: Balazs Varga 
AuthorDate: Thu Sep 7 23:17:52 2023 +0200
Commit: Balazs Varga 
CommitDate: Fri Sep 8 12:53:22 2023 +0200

Remove external link from start center brand image

Change-Id: I601a65c526ba1c3b6e49f02a0401f04379c3e82b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156683
Reviewed-by: Gabor Kelemen 
Tested-by: Balazs Varga 

diff --git a/sfx2/source/dialog/backingwindow.cxx 
b/sfx2/source/dialog/backingwindow.cxx
index a69fd7eac7f7..32a5e28236f6 100644
--- a/sfx2/source/dialog/backingwindow.cxx
+++ b/sfx2/source/dialog/backingwindow.cxx
@@ -78,7 +78,7 @@ public:
 OutputDevice& rDevice = pDrawingArea->get_ref_device();
 rDevice.SetBackground(Wallpaper(rStyleSettings.GetWindowColor()));
 
-SetPointer(PointerStyle::RefHand);
+// SetPointer(PointerStyle::RefHand);
 }
 
 virtual void Resize() override
@@ -119,22 +119,6 @@ public:
 weld::CustomWidgetController::StyleUpdated();
 }
 
-virtual bool MouseButtonUp(const MouseEvent& rMEvt) override
-{
-if (rMEvt.IsLeft())
-{
-OUString sURL = 
officecfg::Office::Common::Menus::ReleaseNotesURL::get();
-// localizeWebserviceURI(sURL);
-
-Reference const 
xSystemShellExecute(
-css::system::SystemShellExecute::create(
-::comphelper::getProcessComponentContext()));
-xSystemShellExecute->execute(sURL, OUString(),
- 
css::system::SystemShellExecuteFlags::URIS_ONLY);
-}
-return true;
-}
-
 virtual void Paint(vcl::RenderContext& rRenderContext, const 
tools::Rectangle&) override
 {
 rRenderContext.DrawBitmapEx(Point(0, 0), maBrandImage);


[Libreoffice-commits] core.git: include/xmloff

2023-09-08 Thread Noel Grandin (via logerrit)
 include/xmloff/unoatrcn.hxx |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 033959aae1744a7bc7315e9abd051805805b33a3
Author: Noel Grandin 
AuthorDate: Fri Sep 8 09:32:50 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 12:40:33 2023 +0200

SvUnoAttributeContainer does not need to implement XAggreggation

Checked on jenkins using 'make check' and

+void SAL_CALL setDelegator(css::uno::Reference 
const &) final { assert(false); }
+

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

diff --git a/include/xmloff/unoatrcn.hxx b/include/xmloff/unoatrcn.hxx
index b09a06250215..454885c26641 100644
--- a/include/xmloff/unoatrcn.hxx
+++ b/include/xmloff/unoatrcn.hxx
@@ -31,12 +31,12 @@
 
 #include 
 
-#include 
+#include 
 
 extern css::uno::Reference< css::uno::XInterface >  
SvUnoAttributeContainer_CreateInstance();
 
 class XMLOFF_DLLPUBLIC SvUnoAttributeContainer final :
-public ::cppu::WeakAggImplHelper2<
+public ::cppu::WeakImplHelper<
 css::lang::XServiceInfo,
 css::container::XNameContainer >
 {


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

2023-09-08 Thread Noel Grandin (via logerrit)
 vcl/source/uitest/uno/uiobject_uno.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 74895033b99ba8b1496dc71699f52402b6d3db11
Author: Noel Grandin 
AuthorDate: Fri Sep 8 11:14:56 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 12:28:44 2023 +0200

add SAL_WARN to UIObjectUnoObj::getChild

to help track down an intermittent UITest failure on jenkins, which
would otherwise assert inside the UIObjectUnoObj constructor

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

diff --git a/vcl/source/uitest/uno/uiobject_uno.cxx 
b/vcl/source/uitest/uno/uiobject_uno.cxx
index 0d23fe9272a7..1a59520f1943 100644
--- a/vcl/source/uitest/uno/uiobject_uno.cxx
+++ b/vcl/source/uitest/uno/uiobject_uno.cxx
@@ -56,6 +56,7 @@ css::uno::Reference SAL_CALL 
UIObjectUnoObj::getChild(
 {
 SolarMutexGuard aGuard;
 std::unique_ptr pObj = mpObj->get_child(rID);
+SAL_WARN_IF(!pObj, "vcl", "child " << rID << " of parent " << 
mpObj->dumpState() << " does not exist");
 return new UIObjectUnoObj(std::move(pObj));
 }
 


[Libreoffice-commits] core.git: helpcontent2

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

New commits:
commit 777252f9f091ddaa51423e1978e233fa9510cabe
Author: Olivier Hallot 
AuthorDate: Fri Sep 8 06:58:50 2023 -0300
Commit: Gerrit Code Review 
CommitDate: Fri Sep 8 11:58:50 2023 +0200

Update git submodules

* Update helpcontent2 from branch 'master'
  to ba616b2299e419762f2eca9ba5b55ed5a4f31c9f
  - tdf#155875 UI cmds Writer/Table in Help(35)

+ refactoring
+ Table - AutoFormat Styles (part 35)
  + uno:AutoFormat refers to similar dialogs in Calc and Writer
  + but each on its module
  + merged in shared/01/AutoFormat.xhp with app switch
  + removed obsolete files (merged)

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

diff --git a/helpcontent2 b/helpcontent2
index 8daa94aa1ff1..ba616b2299e4 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 8daa94aa1ff1135d3d2552182c7f2d23cc21d6f2
+Subproject commit ba616b2299e419762f2eca9ba5b55ed5a4f31c9f


[Libreoffice-commits] help.git: AllLangHelp_scalc.mk AllLangHelp_shared.mk AllLangHelp_swriter.mk source/text

2023-09-08 Thread Olivier Hallot (via logerrit)
 AllLangHelp_scalc.mk |1 
 AllLangHelp_shared.mk|1 
 AllLangHelp_swriter.mk   |1 
 source/text/scalc/00/0405.xhp|   82 +++--
 source/text/scalc/01/0511.xhp|   78 -
 source/text/shared/00/00040503.xhp   |   31 ++
 source/text/shared/01/AutoFormat.xhp |  160 +++
 source/text/shared/01/Protect.xhp|7 +
 source/text/swriter/01/05150101.xhp  |  103 --
 source/text/swriter/main0110.xhp |3 
 10 files changed, 232 insertions(+), 235 deletions(-)

New commits:
commit ba616b2299e419762f2eca9ba5b55ed5a4f31c9f
Author: Olivier Hallot 
AuthorDate: Thu Sep 7 19:45:50 2023 -0300
Commit: Olivier Hallot 
CommitDate: Fri Sep 8 11:58:50 2023 +0200

tdf#155875 UI cmds Writer/Table in Help(35)

+ refactoring
+ Table - AutoFormat Styles (part 35)
  + uno:AutoFormat refers to similar dialogs in Calc and Writer
  + but each on its module
  + merged in shared/01/AutoFormat.xhp with app switch
  + removed obsolete files (merged)

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

diff --git a/AllLangHelp_scalc.mk b/AllLangHelp_scalc.mk
index f69425a60f..04a1ba353b 100644
--- a/AllLangHelp_scalc.mk
+++ b/AllLangHelp_scalc.mk
@@ -111,7 +111,6 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,scalc,\
 helpcontent2/source/text/scalc/01/05080300 \
 helpcontent2/source/text/scalc/01/05080400 \
 helpcontent2/source/text/scalc/01/0510 \
-helpcontent2/source/text/scalc/01/0511 \
 helpcontent2/source/text/scalc/01/0512 \
 helpcontent2/source/text/scalc/01/05120100 \
 helpcontent2/source/text/scalc/01/05120200 \
diff --git a/AllLangHelp_shared.mk b/AllLangHelp_shared.mk
index d19ddefb7c..f6779849d3 100644
--- a/AllLangHelp_shared.mk
+++ b/AllLangHelp_shared.mk
@@ -336,6 +336,7 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,shared,\
 helpcontent2/source/text/shared/01/0708 \
 helpcontent2/source/text/shared/01/about_meta_tags \
 helpcontent2/source/text/shared/01/addsignatureline \
+helpcontent2/source/text/shared/01/AutoFormat \
 helpcontent2/source/text/shared/01/certificatepath \
 helpcontent2/source/text/shared/01/classificationbar \
 helpcontent2/source/text/shared/01/classificationdialog \
diff --git a/AllLangHelp_swriter.mk b/AllLangHelp_swriter.mk
index 0fcd7e2805..c93efbc5f7 100644
--- a/AllLangHelp_swriter.mk
+++ b/AllLangHelp_swriter.mk
@@ -144,7 +144,6 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,swriter,\
 helpcontent2/source/text/swriter/01/0514 \
 helpcontent2/source/text/swriter/01/0515 \
 helpcontent2/source/text/swriter/01/05150100 \
-helpcontent2/source/text/swriter/01/05150101 \
 helpcontent2/source/text/swriter/01/05150104 \
 helpcontent2/source/text/swriter/01/05150200 \
 helpcontent2/source/text/swriter/01/05150300 \
diff --git a/source/text/scalc/00/0405.xhp 
b/source/text/scalc/00/0405.xhp
index cf3956717f..fb6506f971 100644
--- a/source/text/scalc/00/0405.xhp
+++ b/source/text/scalc/00/0405.xhp
@@ -20,7 +20,7 @@
 
 
   
-Format Menu
+Format Menu
 /text/scalc/00/0405.xhp
   
   
@@ -30,50 +30,50 @@
 
 Format Menu
 
-Choose 
Format - Cells.
+Choose Format - 
Cells.
 Press CommandCtrl+1.
 
-Choose Format - Cells - Cell Protection 
tab.
-Choose Format - Rows.
-Choose Format - Rows - Optimal 
Height.
+Choose 
Format - Cells - Cell Protection tab.
+Choose 
Format - Rows.
+Choose Format - Rows - Optimal 
Height.
 
-Choose 
Format - Rows - Hide.
-Choose 
Format - Columns - Hide.
-Choose 
Sheet - Hide Sheet.
+Choose Format - Rows - 
Hide.
+Choose Format - Columns - 
Hide.
+Choose Sheet - Hide 
Sheet.
 
 
-Choose 
Format - Rows - Show.
-Choose 
Format - Columns - Show.
+Choose Format - Rows - 
Show.
+Choose Format - Columns - 
Show.
 
-Choose Format - Columns.
+Choose 
Format - Columns.
 
-Choose 
Format - Columns - Optimal Width.
-Double-click 
right column separator in column headers.
+Choose Format - Columns - 
Optimal Width.
+Double-click right column 
separator in column headers.
 
-Choose Sheet.
+Choose 
Sheet.
 
 
-Choose Format - Page Style.
-Choose Format - Page Style - Sheet 
tab.
-Choose Format - Print Ranges.
-Choose Format - Print Ranges - 
Define.
-Choose Format - Print Ranges - 
Add.
-Choose Format - Print Ranges - 
Clear.
-Choose Format - Print Ranges - 
Edit.
+Choose 
Format - Page Style.
+Choose 
Format - Page Style - Sheet tab.
+Choose 
Format - Print Ranges.
+Choose 
Format - Print Ranges - Define.
+Choose 
Format - Print Ranges - Add.
+Choose 
Format - Print Ranges - Clear.
+Choose 
Format - Print Ranges - Edit.
 
 
 
-  Choose Format - Conditional - 
Condition.
+  Choose Format - Conditional - 
Condition.
   On the 
Formatting

[Libreoffice-commits] core.git: helpcontent2

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

New commits:
commit 19e32f6a3661a5f161db0d5a722f25fb315bd45a
Author: Olivier Hallot 
AuthorDate: Fri Sep 8 06:58:21 2023 -0300
Commit: Gerrit Code Review 
CommitDate: Fri Sep 8 11:58:21 2023 +0200

Update git submodules

* Update helpcontent2 from branch 'master'
  to 8daa94aa1ff1135d3d2552182c7f2d23cc21d6f2
  - tdf#155875 UI cmds Writer/Table in Help(34)

+ refactoring
+ Table - Protection (part 34)
  + protection is called by "uno.Protect" in menus
  + in Calc: protect sheet dialog
  + in Writer: protect cell directly
  + merged in shared/01/Protect.xhp with app switch
  + removed obsolete files

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

diff --git a/helpcontent2 b/helpcontent2
index 5f31e41620bb..8daa94aa1ff1 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit 5f31e41620bb668c611701c87a9f2b1cbf3c93c9
+Subproject commit 8daa94aa1ff1135d3d2552182c7f2d23cc21d6f2


[Libreoffice-commits] help.git: AllLangHelp_scalc.mk AllLangHelp_shared.mk AllLangHelp_swriter.mk source/text

2023-09-08 Thread Olivier Hallot (via logerrit)
 AllLangHelp_scalc.mk  |1 
 AllLangHelp_shared.mk |1 
 AllLangHelp_swriter.mk|1 
 source/text/scalc/01/0606.xhp |6 -
 source/text/scalc/01/06060100.xhp |  111 
 source/text/scalc/guide/cell_protect.xhp  |4 
 source/text/scalc/menu/sheet_tab_menu.xhp |2 
 source/text/shared/01/Protect.xhp |  137 ++
 source/text/swriter/00/0405.xhp   |   37 +++-
 source/text/swriter/01/0510.xhp   |2 
 source/text/swriter/01/05100300.xhp   |   41 
 source/text/swriter/01/05100400.xhp   |   31 +++---
 source/text/swriter/main0110.xhp  |7 -
 13 files changed, 196 insertions(+), 185 deletions(-)

New commits:
commit 8daa94aa1ff1135d3d2552182c7f2d23cc21d6f2
Author: Olivier Hallot 
AuthorDate: Thu Sep 7 14:16:13 2023 -0300
Commit: Olivier Hallot 
CommitDate: Fri Sep 8 11:58:21 2023 +0200

tdf#155875 UI cmds Writer/Table in Help(34)

+ refactoring
+ Table - Protection (part 34)
  + protection is called by "uno.Protect" in menus
  + in Calc: protect sheet dialog
  + in Writer: protect cell directly
  + merged in shared/01/Protect.xhp with app switch
  + removed obsolete files

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

diff --git a/AllLangHelp_scalc.mk b/AllLangHelp_scalc.mk
index 6192819d40..f69425a60f 100644
--- a/AllLangHelp_scalc.mk
+++ b/AllLangHelp_scalc.mk
@@ -133,7 +133,6 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,scalc,\
 helpcontent2/source/text/scalc/01/0604 \
 helpcontent2/source/text/scalc/01/0605 \
 helpcontent2/source/text/scalc/01/0606 \
-helpcontent2/source/text/scalc/01/06060100 \
 helpcontent2/source/text/scalc/01/06060200 \
 helpcontent2/source/text/scalc/01/0607 \
 helpcontent2/source/text/scalc/01/0608 \
diff --git a/AllLangHelp_shared.mk b/AllLangHelp_shared.mk
index c40c7c4209..d19ddefb7c 100644
--- a/AllLangHelp_shared.mk
+++ b/AllLangHelp_shared.mk
@@ -383,6 +383,7 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,shared,\
 helpcontent2/source/text/shared/01/pasterowabove \
 helpcontent2/source/text/shared/01/pastecolumnleft \
 helpcontent2/source/text/shared/01/profile_safe_mode \
+helpcontent2/source/text/shared/01/Protect \
 helpcontent2/source/text/shared/01/qrcode \
 helpcontent2/source/text/shared/01/quickcolorselector \
 helpcontent2/source/text/shared/01/ref_epub_export \
diff --git a/AllLangHelp_swriter.mk b/AllLangHelp_swriter.mk
index 26ffea7b41..0fcd7e2805 100644
--- a/AllLangHelp_swriter.mk
+++ b/AllLangHelp_swriter.mk
@@ -129,7 +129,6 @@ $(eval $(call gb_AllLangHelp_add_helpfiles,swriter,\
 helpcontent2/source/text/swriter/01/05090201 \
 helpcontent2/source/text/swriter/01/05090300 \
 helpcontent2/source/text/swriter/01/0510 \
-helpcontent2/source/text/swriter/01/05100300 \
 helpcontent2/source/text/swriter/01/05100400 \
 helpcontent2/source/text/swriter/01/05110100 \
 helpcontent2/source/text/swriter/01/05110300 \
diff --git a/source/text/scalc/01/0606.xhp 
b/source/text/scalc/01/0606.xhp
index b57bd65a69..fc4972f06a 100644
--- a/source/text/scalc/01/0606.xhp
+++ b/source/text/scalc/01/0606.xhp
@@ -18,7 +18,7 @@
  *   except in compliance with the License. You may obtain a copy of
  *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
  -->
-   
+
 
 
 
@@ -38,8 +38,8 @@
   
   
 
-Sheets
-
+Sheets
+
 Documents
 
 
diff --git a/source/text/scalc/01/06060100.xhp 
b/source/text/scalc/01/06060100.xhp
deleted file mode 100644
index 0e39d0d27e..00
--- a/source/text/scalc/01/06060100.xhp
+++ /dev/null
@@ -1,111 +0,0 @@
-
-
-
-
-
-Protecting Sheet
-/text/scalc/01/06060100.xhp
-
-
-
-
-
-
-
-insert rows;protected sheet
-insert columns;protected sheet
-delete columns;protected sheet
-delete rows;protected sheet
-protected sheet;insert columns
-protected sheet;insert rows
-protected sheet;delete rows
-protected sheet;delete columns
-protect;sheet
-selection;in protected cells
-
-
-Protect Sheet
-Protects the cells in the current 
sheet from being modified.
-
-Choose Tools - 
Protect Sheet to open the Protect Sheet dialog in which you 
then specify sheet protection with or without a password, and select the 
elements of the sheet to protect.
-
-  
-  
-
-To protect cells from further editing, the 
Protected check box must be checked on the Format - Cells - Cell 
Protection tab page or on the Format Cells context 
menu.
-
-Protect this sheet and contents of the protected 
cells
-Mark this checkbox to 
activate protection of sheet and cell contents.
-Password / Confirm
-Allows you to enter a 
password to protect the sheet from unauthorized changes. Confirm the passwo

[Libreoffice-commits] core.git: helpcontent2

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

New commits:
commit 87f9e703bfa13652e2bca59e89f45b196f9d9580
Author: Olivier Hallot 
AuthorDate: Fri Sep 8 06:57:46 2023 -0300
Commit: Gerrit Code Review 
CommitDate: Fri Sep 8 11:57:46 2023 +0200

Update git submodules

* Update helpcontent2 from branch 'master'
  to 5f31e41620bb668c611701c87a9f2b1cbf3c93c9
  - tdf#155875 UI cmds Writer/Table in Help(33)

+ refactoring
+ Table - split/merge table (part 33)

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

diff --git a/helpcontent2 b/helpcontent2
index f9f74aac7105..5f31e41620bb 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit f9f74aac7105208ded70127e5acd6cd2bae8b093
+Subproject commit 5f31e41620bb668c611701c87a9f2b1cbf3c93c9


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

2023-09-08 Thread Olivier Hallot (via logerrit)
 source/text/shared/01/05100200.xhp  |6 +++--
 source/text/swriter/00/0405.xhp |   26 ++--
 source/text/swriter/01/0519.xhp |   38 ++--
 source/text/swriter/01/0520.xhp |   22 ++--
 source/text/swriter/main0110.xhp|6 +
 5 files changed, 60 insertions(+), 38 deletions(-)

New commits:
commit 5f31e41620bb668c611701c87a9f2b1cbf3c93c9
Author: Olivier Hallot 
AuthorDate: Thu Sep 7 10:00:58 2023 -0300
Commit: Olivier Hallot 
CommitDate: Fri Sep 8 11:57:46 2023 +0200

tdf#155875 UI cmds Writer/Table in Help(33)

+ refactoring
+ Table - split/merge table (part 33)

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

diff --git a/source/text/shared/01/05100200.xhp 
b/source/text/shared/01/05100200.xhp
index 482a0b3d7f..662c62bcd0 100644
--- a/source/text/shared/01/05100200.xhp
+++ b/source/text/shared/01/05100200.xhp
@@ -44,8 +44,10 @@
 
 
 
-Split Cells
-Displays the Split Cells Dialog where the split 
can be defined as either horizontally or vertically and the number the each 
cell will be split into.
+
+Split Cells
+Displays the Split Cells Dialog where the split 
can be defined as either horizontally or vertically and the number the each 
cell will be split into.
+
 
 
 
diff --git a/source/text/swriter/00/0405.xhp 
b/source/text/swriter/00/0405.xhp
index c947f2d49b..167397b411 100644
--- a/source/text/swriter/00/0405.xhp
+++ b/source/text/swriter/00/0405.xhp
@@ -260,8 +260,30 @@
 
 
 
-  Choose Table - Split Table.
-  Choose Table - Merge Table.
+
+
+   
+   Choose 
Table - Split Table.
+   
+   Choose 
Table - Split Table.
+   
+   
+  
+ 
+Icon Split 
Table
+ 
+ 
+Split 
Table
+ 
+  
+   
+   
+   On the 
Table deck of the Properties panel, 
click on Split Table.
+
+
+   
+   Choose Table - Merge Table.
+
   Choose Table - Properties - Table 
tab.
   Choose Table - Properties - Columns 
tab.
   Choose Table - Properties - Text Flow 
tab.
diff --git a/source/text/swriter/01/0519.xhp 
b/source/text/swriter/01/0519.xhp
index 38851d7ed9..cad232864c 100644
--- a/source/text/swriter/01/0519.xhp
+++ b/source/text/swriter/01/0519.xhp
@@ -1,6 +1,4 @@
 
-
-
 
-
 
 
 
-Split Table
+Split Table
 /text/swriter/01/0519.xhp
 
 
@@ -31,32 +28,35 @@
 
 
 
-tables; splitting
+tables; 
splitting
 splitting tables; at cursor position
 dividing tables
 
-
-
-
+
+
+
 Split 
Table
-Splits the current table into two separate tables at the 
cursor position. You can also access this command by right-clicking in 
a table cell.
+Splits the current table into two separate tables at the 
cursor position. You can also access this command by right-clicking in 
a table cell.
 
 
-  
+  
 
 Mode
-
+
 Copy heading
-Includes the first row of the 
original table as the first row of the second table.
-
+Includes the first row of the 
original table as the first row of the second table.
+
 Custom heading (apply style)
-Inserts a blank 
header row in the second table that is formatted with the style of the first 
row in the original table.
-
+Inserts a blank 
header row in the second table that is formatted with the style of the first 
row in the original table.
+
 Custom heading
-Inserts an additional blank 
row in the second table.
-
+Inserts an additional blank 
row in the second table.
+
 No heading
-Splits the table without copying 
the header row.
-When you split a 
table that contains formulas, the formulas may be affected.
+Splits the table without copying 
the header row.
+When you split a table that contains formulas, the 
formulas may be affected.
+
+
+
 
 
diff --git a/source/text/swriter/01/0520.xhp 
b/source/text/swriter/01/0520.xhp
index 4f6620d93a..53f9304606 100644
--- a/source/text/swriter/01/0520.xhp
+++ b/source/text/swriter/01/0520.xhp
@@ -1,6 +1,4 @@
 
-
-
 
-
 
 
 
-Merge Table 
+Merge Table 
 /text/swriter/01/0520.xhp
 
 
@@ -31,18 +28,21 @@
 
 
 
-tables; merging
+tables; 
merging
 merging; tables
 
-
-
-
+
+
+
 Merge 
Table
-Combines two consecutive tables into a single table. The tables must be 
directly next to each other and not separated by an empty 
paragraph.
+Combines two 
consecutive tables into a single table. The tables must be directly next to 
each other and not separated by an empty paragraph.
 
 
-  
+  
 
-If you choose 
this command when the cursor is in the middle of three consecutive tables, you 
are prompted to select the table that you want to merge with.
+If you choose this command when 
the cursor is in the middle of three consecutive tables, you are prompted to 
select the table that

[Libreoffice-commits] core.git: Branch 'feature/cib_contract891c' - 3 commits - configure.ac solenv/bin sysui/CustomTarget_deb.mk sysui/CustomTarget_rpm.mk sysui/CustomTarget_solaris.mk

2023-09-08 Thread Michael Weghorn (via logerrit)
 configure.ac  |9 ++
 solenv/bin/modules/installer/download.pm  |   37 +-
 solenv/bin/modules/installer/epmfile.pm   |   19 ++---
 solenv/bin/modules/installer/simplepackage.pm |   10 ++-
 sysui/CustomTarget_deb.mk |7 ++--
 sysui/CustomTarget_rpm.mk |2 -
 sysui/CustomTarget_solaris.mk |2 -
 7 files changed, 35 insertions(+), 51 deletions(-)

New commits:
commit 55200cee2ff3229bc32323eb74501b62b370b419
Author: Michael Weghorn 
AuthorDate: Thu Feb 15 16:37:23 2018 +0100
Commit: Vasily Melenchuk 
CommitDate: Fri Sep 8 12:06:04 2023 +0300

tdf#115554: Fix ownership in desktop-integration tars

Use fakeroot to call tar when creating the tar.gz archives
holding the desktop-integration parts to have root:root
as owner of the files inside those.

Change-Id: Idcdcb4dd398f330773671266dafb354711b0f093
Reviewed-on: https://gerrit.libreoffice.org/49827
Reviewed-by: Michael Stahl 
Tested-by: Michael Stahl 
Reviewed-by: Thorsten Behrens 

diff --git a/sysui/CustomTarget_deb.mk b/sysui/CustomTarget_deb.mk
index dd75ee913c62..92cda34a3bfe 100644
--- a/sysui/CustomTarget_deb.mk
+++ b/sysui/CustomTarget_deb.mk
@@ -25,7 +25,7 @@ $(foreach product,$(PRODUCTLIST),\
 ))
 
 $(deb_WORKDIR)/%-desktop-integration.tar.gz: 
$(deb_WORKDIR)/%$(PKGVERSIONSHORT)-debian-menus_$(PKGVERSION)-$(LIBO_VERSION_PATCH)_all.deb
-   $(GNUTAR) -C $(deb_WORKDIR) -cf - $(notdir $<) | gzip > $@
+   fakeroot $(GNUTAR) -C $(deb_WORKDIR) -cf - $(notdir $<) | gzip > $@
 
 $(deb_WORKDIR)/%/DEBIAN/postrm: $(deb_SRCDIR)/postrm
cat $< | tr -d "\015" | \
diff --git a/sysui/CustomTarget_rpm.mk b/sysui/CustomTarget_rpm.mk
index 8cc56ed80e34..a76b4161e2af 100644
--- a/sysui/CustomTarget_rpm.mk
+++ b/sysui/CustomTarget_rpm.mk
@@ -54,6 +54,6 @@ $(foreach product,$(PRODUCTLIST),\
 ))
 
 $(rpm_WORKDIR)/%-desktop-integration.tar.gz:
-   $(GNUTAR) -C $(rpm_WORKDIR)/$* -cf - $(foreach 
distro,$(RPMDISTROS),$*$(PKGVERSIONSHORT)-$(distro)-menus-$(PKGVERSION)-$(LIBO_VERSION_PATCH).noarch.rpm)
 | gzip > $@
+   fakeroot $(GNUTAR) -C $(rpm_WORKDIR)/$* -cf - $(foreach 
distro,$(RPMDISTROS),$*$(PKGVERSIONSHORT)-$(distro)-menus-$(PKGVERSION)-$(LIBO_VERSION_PATCH).noarch.rpm)
 | gzip > $@
 
 # vim: set noet sw=4 ts=4:
diff --git a/sysui/CustomTarget_solaris.mk b/sysui/CustomTarget_solaris.mk
index ed264fe95960..306a30429174 100644
--- a/sysui/CustomTarget_solaris.mk
+++ b/sysui/CustomTarget_solaris.mk
@@ -50,6 +50,6 @@ $(solaris_WORKDIR)/%/prototype: $(solaris_SRCDIR)/prototype
 
 $(solaris_WORKDIR)/%-desktop-integration.tar.gz: 
$(solaris_WORKDIR)/%/copyright $(solaris_WORKDIR)/%/pkginfo 
$(solaris_WORKDIR)/%/depend $(solaris_WORKDIR)/%/mailcap 
$(solaris_WORKDIR)/%/postinstall $(solaris_WORKDIR)/%/postremove 
$(solaris_WORKDIR)/%/prototype $(call 
gb_CustomTarget_get_workdir,sysui/share)/%/openoffice.org.xml
 #  pkgmk -l 1073741824 -r $(solaris_WORKDIR) -f 
$(solaris_WORKDIR)/$*/prototype -o -d $(solaris_WORKDIR) ARCH=all 
VERSION=$(PKGVERSION.$*)
-   $(GNUTAR) -cf - -C $(solaris_WORKDIR) $*$(LIBO_MAJOR) -desktop-int | 
gzip > $@
+   fakeroot $(GNUTAR) -cf - -C $(solaris_WORKDIR) $*$(LIBO_MAJOR) 
-desktop-int | gzip > $@
 
 # vim: set noet sw=4 ts=4:
commit 4debe1fe558ae23e13a65dc79ef78ae600a231a5
Author: Michael Weghorn 
AuthorDate: Tue Feb 13 20:29:10 2018 +0100
Commit: Vasily Melenchuk 
CommitDate: Fri Sep 8 12:04:05 2023 +0300

tdf#115554:  Create tar archives using 'fakeroot'

'tar' must be run as root or faked to be run as root,
to make sure that file ownerships and permissions inside
the created tar archives are as desired (root:root).

Have fakeroot take care about creating an appropriate
environment rather than using the custom "libgetuid",
to no longer have to care about tar internals by ourselves.

This fixes the problem that file ownerships are incorrect
when tar version >= 1.24 is used for

* tar archives holding all the generated deb/rpm packages
  (created in 'download.pm')
* tar archives created by using the
  '--with-pacakage-format=archive' autogen option
  (created in 'simplepackage.pm')

Change-Id: Id20ccce4d002ff95c75292eda8080ca299eee3a5
Reviewed-on: https://gerrit.libreoffice.org/49682
Reviewed-by: Michael Stahl 
Tested-by: Jenkins 
Reviewed-by: Thorsten Behrens 

diff --git a/configure.ac b/configure.ac
index 94a65c0c0d9a..d81ed648bae7 100644
--- a/configure.ac
+++ b/configure.ac
@@ -4457,6 +4457,15 @@ msi - Windows .msi
 ;;
 esac
 done
+# fakeroot is needed to ensure correct file ownerships/permissions
+# inside deb packages and tar archives created on Linux and Solaris.
+if test "$OS" = "LINUX" || test "$OS" = "SOLARIS"; then
+AC_PATH_PROG(FAKEROOT, fakeroot, no)
+i

[Libreoffice-commits] core.git: offapi/com

2023-09-08 Thread Stephan Bergmann (via logerrit)
 offapi/com/sun/star/datatransfer/XTransferableTextSupplier.idl |2 -
 offapi/com/sun/star/document/XDocumentProperties2.idl  |2 -
 offapi/com/sun/star/frame/status/Template.idl  |2 -
 offapi/com/sun/star/sdbc/DataType.idl  |   20 
+-
 4 files changed, 13 insertions(+), 13 deletions(-)

New commits:
commit 2bd5b489db722905a81e40f9d6dd90c5a73a2d89
Author: Stephan Bergmann 
AuthorDate: Fri Sep 8 09:03:55 2023 +0200
Commit: Stephan Bergmann 
CommitDate: Fri Sep 8 11:16:50 2023 +0200

Fix some misspelled LibreOffice @since tags

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

diff --git a/offapi/com/sun/star/datatransfer/XTransferableTextSupplier.idl 
b/offapi/com/sun/star/datatransfer/XTransferableTextSupplier.idl
index 18255d9074bc..7b4c84ecb23f 100644
--- a/offapi/com/sun/star/datatransfer/XTransferableTextSupplier.idl
+++ b/offapi/com/sun/star/datatransfer/XTransferableTextSupplier.idl
@@ -11,7 +11,7 @@
 
 module com { module sun { module star { module datatransfer {
 
-/** @since LO 7.2
+/** @since LibreOffice 7.2
  */
 interface XTransferableTextSupplier
 {
diff --git a/offapi/com/sun/star/document/XDocumentProperties2.idl 
b/offapi/com/sun/star/document/XDocumentProperties2.idl
index c0d2a18b4af2..5b4acbfe2e47 100644
--- a/offapi/com/sun/star/document/XDocumentProperties2.idl
+++ b/offapi/com/sun/star/document/XDocumentProperties2.idl
@@ -11,7 +11,7 @@ module com {   module sun {   module star {   module document 
{
 
 /** Extends XDocumentProperties interface to provide additional attributes
 
-@since LibreOffice 24.02
+@since LibreOffice 24.2
 */
 interface XDocumentProperties2: com::sun::star::document::XDocumentProperties
 {
diff --git a/offapi/com/sun/star/frame/status/Template.idl 
b/offapi/com/sun/star/frame/status/Template.idl
index ca8939d6bf9d..dfee1152adf4 100644
--- a/offapi/com/sun/star/frame/status/Template.idl
+++ b/offapi/com/sun/star/frame/status/Template.idl
@@ -41,7 +41,7 @@ struct Template
 
 /** specifies an identifier name in English (only for standard style).
 
-@since LO 7.2
+@since LibreOffice 7.2
  */
 string StyleNameIdentifier;
 };
diff --git a/offapi/com/sun/star/sdbc/DataType.idl 
b/offapi/com/sun/star/sdbc/DataType.idl
index 375a76f30a86..794b824a6b71 100644
--- a/offapi/com/sun/star/sdbc/DataType.idl
+++ b/offapi/com/sun/star/sdbc/DataType.idl
@@ -129,70 +129,70 @@ published constants DataType
 
 /** indicates a type representing an SQL DATALINK.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long DATALINK   =   70;
 
 
 /** indicates a type representing an SQL ROWID.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long ROWID  =   -8;
 
 
 /** indicates a type representing an SQL NCHAR.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long NCHAR  =  -15;
 
 
 /** indicates a type representing an SQL NVARCHAR.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long NVARCHAR   =   -9;
 
 
 /** indicates a type representing an SQL LONGNVARCHAR.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long LONGNVARCHAR   =  -16;
 
 
 /** indicates a type representing an SQL NCLOB.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long NCLOB  = 2011;
 
 
 /** indicates a type representing an SQL XML.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long SQLXML = 2009;
 
 
 /** indicates a type representing an SQL REF CURSOR.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long REF_CURSOR = 2012;
 
 
 /** indicates a type representing an SQL TIME WITH TIMEZONE.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long TIME_WITH_TIMEZONE = 2013;
 
 
 /** indicates a type representing an SQL TIMESTAMP WITH TIMEZONE.
  *
- * @since LO 24.02
+ * @since LibreOffice 24.2
  */
 const long TIMESTAMP_WITH_TIMEZONE= 2014;
 


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

2023-09-08 Thread Noel Grandin (via logerrit)
 drawinglayer/source/drawinglayeruno/xprimitive2drenderer.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 380b586d6e69412aa5f4b9558ebb192ed8bcfc38
Author: Noel Grandin 
AuthorDate: Thu Sep 7 13:02:41 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 10:53:15 2023 +0200

XPrimitive2DRenderer does not need to implement XAggreggation

Checked on jenkins using 'make check' and

+void SAL_CALL setDelegator(css::uno::Reference 
const &) final { assert(false); }
+

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

diff --git a/drawinglayer/source/drawinglayeruno/xprimitive2drenderer.cxx 
b/drawinglayer/source/drawinglayeruno/xprimitive2drenderer.cxx
index 2eb1373df0ae..58df9f4a7b28 100644
--- a/drawinglayer/source/drawinglayeruno/xprimitive2drenderer.cxx
+++ b/drawinglayer/source/drawinglayeruno/xprimitive2drenderer.cxx
@@ -43,7 +43,7 @@ namespace drawinglayer::unorenderer
 namespace {
 
 class XPrimitive2DRenderer:
-public cppu::WeakAggImplHelper2<
+public cppu::WeakImplHelper<
 css::graphic::XPrimitive2DRenderer, css::lang::XServiceInfo>
 {
 public:


[Libreoffice-commits] core.git: include/svl

2023-09-08 Thread Noel Grandin (via logerrit)
 include/svl/numuno.hxx |7 ---
 1 file changed, 4 insertions(+), 3 deletions(-)

New commits:
commit 3f0d41c14d2198fcf86104050dd42cdb82b3e9a4
Author: Noel Grandin 
AuthorDate: Thu Sep 7 16:08:25 2023 +0200
Commit: Noel Grandin 
CommitDate: Fri Sep 8 10:52:53 2023 +0200

document SvNumberFormatsSupplierObj

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

diff --git a/include/svl/numuno.hxx b/include/svl/numuno.hxx
index a439b95599ae..c7c04194187b 100644
--- a/include/svl/numuno.hxx
+++ b/include/svl/numuno.hxx
@@ -35,9 +35,10 @@ namespace comphelper
 }
 
 
-//  SvNumberFormatsSupplierObj: aggregate to document,
-//  construct with SvNumberFormatter
-
+/**
+ * Factory for XNumberFormats objects.
+ * Implements XAggregation because it is aggregated to ScModelObj
+ */
 class SVL_DLLPUBLIC SvNumberFormatsSupplierObj : public 
cppu::WeakAggImplHelper2<
 css::util::XNumberFormatsSupplier,
 css::lang::XUnoTunnel>


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

2023-09-08 Thread Gökay Şatır (via logerrit)
 sw/source/uibase/docvw/AnnotationMenuButton.cxx |   28 ++--
 1 file changed, 3 insertions(+), 25 deletions(-)

New commits:
commit e653ddd24ab4b75db212431461035ecbd9d625fa
Author: Gökay Şatır 
AuthorDate: Thu Sep 7 11:22:25 2023 +0300
Commit: Miklos Vajna 
CommitDate: Fri Sep 8 10:41:50 2023 +0200

We can allow one to answer their own comment.

Other office implementations allow this.

I think we can enable reply button for the owner of the comment, unless 
there is a good reason to not to.

Signed-off-by: Gökay Şatır 
Change-Id: I6969122efe24d2804b4092e0afe0709a2fd1ef98
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156688
Tested-by: Jenkins CollaboraOffice 
Reviewed-by: Miklos Vajna 

diff --git a/sw/source/uibase/docvw/AnnotationMenuButton.cxx 
b/sw/source/uibase/docvw/AnnotationMenuButton.cxx
index 1696f7e6d338..11d755cbd7e2 100644
--- a/sw/source/uibase/docvw/AnnotationMenuButton.cxx
+++ b/sw/source/uibase/docvw/AnnotationMenuButton.cxx
@@ -66,13 +66,11 @@ IMPL_LINK_NOARG(SwAnnotationWin, ToggleHdl, 
weld::Toggleable&, void)
 if (!mxMenuButton->get_active())
 return;
 
-bool bReplyVis = true;
-
 bool bReadOnly = IsReadOnly();
 if (bReadOnly)
 {
 mxMenuButton->set_item_visible("reply", false);
-bReplyVis = false;
+mxMenuButton->set_item_visible("sep1", false); // Separator after 
reply button.
 mxMenuButton->set_item_visible("resolve", false);
 mxMenuButton->set_item_visible("unresolve", false);
 mxMenuButton->set_item_visible("resolvethread", false);
@@ -81,6 +79,8 @@ IMPL_LINK_NOARG(SwAnnotationWin, ToggleHdl, 
weld::Toggleable&, void)
 }
 else
 {
+mxMenuButton->set_item_visible("reply", !IsProtected());
+mxMenuButton->set_item_visible("sep1", !IsProtected());
 mxMenuButton->set_item_visible("resolve", !IsResolved());
 mxMenuButton->set_item_visible("unresolve", IsResolved());
 mxMenuButton->set_item_visible("resolvethread", !IsThreadResolved());
@@ -92,28 +92,6 @@ IMPL_LINK_NOARG(SwAnnotationWin, ToggleHdl, 
weld::Toggleable&, void)
 mxMenuButton->set_item_visible("deleteby", !bReadOnly);
 mxMenuButton->set_item_visible("deleteall", !bReadOnly);
 mxMenuButton->set_item_visible("formatall", !bReadOnly);
-
-if (IsProtected())
-{
-mxMenuButton->set_item_visible("reply", false);
-bReplyVis = false;
-}
-else
-{
-SvtUserOptions aUserOpt;
-OUString sAuthor;
-if ((sAuthor = aUserOpt.GetFullName()).isEmpty())
-{
-if ((sAuthor = aUserOpt.GetID()).isEmpty())
-{
-sAuthor = SwResId(STR_REDLINE_UNKNOWN_AUTHOR);
-}
-}
-// do not allow to reply to ourself and no answer possible if this 
note is in a protected section
-bReplyVis = sAuthor != GetAuthor();
-mxMenuButton->set_item_visible("reply", bReplyVis);
-}
-mxMenuButton->set_item_visible("sep1", bReplyVis);
 }
 
 IMPL_LINK(SwAnnotationWin, KeyInputHdl, const KeyEvent&, rKeyEvt, bool)


[Libreoffice-commits] core.git: odk/examples

2023-09-08 Thread Hossein (via logerrit)
 odk/examples/DevelopersGuide/FirstSteps/FirstLoadComponent.py |   62 ++
 1 file changed, 62 insertions(+)

New commits:
commit 7dc3be651b050c7b4d383d8ee25b69728418acc7
Author: Hossein 
AuthorDate: Fri Sep 8 01:16:29 2023 +0200
Commit: Hossein 
CommitDate: Fri Sep 8 10:16:06 2023 +0200

tdf#143123 Port DevelopersGuide/FirstSteps/FirstLoadComponent to Python

Porting FirstSteps/FirstLoadComponent.java to Python is done. More
information on this example can be found on LibreOffice DevGuide:

LibreOffice Developer's Guide: Chapter 1 - First Steps - First Contact
https://wiki.documentfoundation.org/Documentation/DevGuide/First_Steps

This is the last of the 3 examples in the first chapter which are ported
to Python.

Change-Id: Iac0682943d4286168bbdadf792423670f41bdee6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/156686
Tested-by: Hossein 
Reviewed-by: Hossein 

diff --git a/odk/examples/DevelopersGuide/FirstSteps/FirstLoadComponent.py 
b/odk/examples/DevelopersGuide/FirstSteps/FirstLoadComponent.py
new file mode 100755
index ..a5d93d799f8c
--- /dev/null
+++ b/odk/examples/DevelopersGuide/FirstSteps/FirstLoadComponent.py
@@ -0,0 +1,62 @@
+# -*- 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/.
+#
+
+import uno
+import officehelper
+import sys
+import traceback
+from com.sun.star.sheet.CellFlags import FORMULA
+
+
+def main():
+try:
+remote_context = officehelper.bootstrap()
+if remote_context is None:
+print("ERROR: Could not bootstrap default Office.")
+sys.exit(1)
+srv_mgr = remote_context.getServiceManager()
+desktop = 
srv_mgr.createInstanceWithContext("com.sun.star.frame.Desktop", remote_context)
+spreadsheet_component = 
desktop.loadComponentFromURL("private:factory/scalc", "_blank", 0, tuple())
+spreadsheets = spreadsheet_component.getSheets()
+spreadsheets.insertNewByName("MySheet", 0)
+elem_type = spreadsheets.getElementType()
+print(elem_type)
+sheet = spreadsheets.getByName("MySheet")
+cell = sheet.getCellByPosition(0, 0)
+cell.setValue(21)
+cell = sheet.getCellByPosition(0, 1)
+cell.setValue(21)
+cell = sheet.getCellByPosition(0, 2)
+cell.setFormula("=sum(A1:A2)")
+
+cell.setPropertyValue("CellStyle", "Result")
+
+spreadsheet_controller = spreadsheet_component.getCurrentController()
+spreadsheet_controller.setActiveSheet(sheet)
+cell.setPropertyValue("VertJustify", 
"com.sun.star.table.CellVertJustify.TOP")
+formula_cells = sheet.queryContentCells(FORMULA)
+formulas = formula_cells.getCells()
+formula_enum = formulas.createEnumeration()
+
+while formula_enum.hasMoreElements():
+formula_cell = formula_enum.nextElement()
+print("Formula cell in column " + 
str(formula_cell.getCellAddress().Column)
+  + ", row " + str(formula_cell.getCellAddress().Row)
+  + " contains " + cell.getFormula())
+
+except Exception as e:
+print(e)
+traceback.print_exc()
+sys.exit(1)
+
+
+if __name__ == "__main__":
+main()
+
+# vim: set shiftwidth=4 softtabstop=4 expandtab:


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

2023-09-08 Thread Miklos Vajna (via logerrit)
 sw/qa/core/layout/flycnt.cxx  |7 +++
 sw/source/core/layout/objectformatter.cxx |7 ++-
 2 files changed, 13 insertions(+), 1 deletion(-)

New commits:
commit facdac2443d50339f81415d09c1869d19dded7bf
Author: Miklos Vajna 
AuthorDate: Fri Sep 8 08:26:40 2023 +0200
Commit: Miklos Vajna 
CommitDate: Fri Sep 8 09:25:19 2023 +0200

sw floattable, nesting: fix PDF export

The problem was that the bugdoc rendered fine when opened interactively,
but not when converted to PDF via 'soffice --convert-to pdf'.

The direct cause was that the interactive case also enabled the
navigator that did one more calculation for each fly, and that fixes the
layout. The headless export to PDF has no navigator, so that didn't
happen.

The root of the problem seems to be that:

1) There is a bad left position for the inner follow fly (it's outside
   the page rectangle).

2) SwTabFrame::Split() for the outer tab normally calls
   lcl_RecalcSplitLine() which moves flys away + back.

3) Moving away happens in lcl_InvalidateLowerObjs() in the
   _bMoveObjsOutOfRange case, here we move away both fly 10 and 20 (both
  inner flys).

4) Moving back happens in
   SwToContentAnchoredObjectPosition::CalcPosition(), but that's not
   invoked for the inner follow fly.

Fix the problem by extending SwObjectFormatter::FormatObjsAtFrame_() to
be more consistent here and move back both flys.

This is related to tdf#55160.

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

diff --git a/sw/qa/core/layout/flycnt.cxx b/sw/qa/core/layout/flycnt.cxx
index 797d5d8c5691..1c69606c8a24 100644
--- a/sw/qa/core/layout/flycnt.cxx
+++ b/sw/qa/core/layout/flycnt.cxx
@@ -1068,6 +1068,13 @@ CPPUNIT_TEST_FIXTURE(Test, testSplitFlyNested)
 // rendered and this way the inner anchor had no fly portion, either.
 CPPUNIT_ASSERT_GREATER(pPage2->getFrameArea().Top(), 
pPage2Fly1->getFrameArea().Top());
 
+// Without the accompanying fix in place, this test would have failed with:
+// - Expected less than: 12523
+// - Actual  : 15312
+// i.e. the inner follow fly was not "moved back" to its place to have the 
wanted 4400 position,
+// which makes the "Inner A2" text visible.
+CPPUNIT_ASSERT_LESS(pPage2->getFrameArea().Right(), 
pPage2Fly1->getFrameArea().Right());
+
 auto pPage2Fly2 = 
rPage2Objs[1]->DynCastFlyFrame()->DynCastFlyAtContentFrame();
 CPPUNIT_ASSERT(pPage2Fly2);
 CPPUNIT_ASSERT(!pPage2Fly2->GetAnchorFrameContainingAnchPos()->IsInFly());
diff --git a/sw/source/core/layout/objectformatter.cxx 
b/sw/source/core/layout/objectformatter.cxx
index 15ca544a2d9c..b99da6a0fe4b 100644
--- a/sw/source/core/layout/objectformatter.cxx
+++ b/sw/source/core/layout/objectformatter.cxx
@@ -390,7 +390,12 @@ bool SwObjectFormatter::FormatObjsAtFrame_( SwTextFrame* 
_pMasterTextFrame )
 pAnchorCharFrame != pAnchoredObj->GetAnchorFrame() &&
 pAnchorCharFrame->FindBodyFrame() ==
 
static_cast(pAnchoredObj->AnchorFrame())->FindBodyFrame();
-if ( bAnchoredAtFollowInSameBodyAsMaster )
+// Make sure that in case nested split flys are moved "out of range" in
+// lcl_InvalidateLowerObjs(), then we moved them back here.
+SwFlyFrame* pFly = pAnchoredObj->DynCastFlyFrame();
+bool bSplitFly = pFly && pFly->IsFlySplitAllowed();
+bool bNestedSplitFly = bSplitFly && pAnchorCharFrame && 
pAnchorCharFrame->IsInFly();
+if (bAnchoredAtFollowInSameBodyAsMaster && !bNestedSplitFly)
 {
 continue;
 }