[Libreoffice-commits] core.git: sw/UITest_writer_tests2.mk sw/UITest_writer_tests3.mk sw/UITest_writer_tests4.mk sw/UITest_writer_tests6.mk

2023-03-21 Thread Miklos Vajna (via logerrit)
 sw/UITest_writer_tests2.mk |2 ++
 sw/UITest_writer_tests3.mk |2 ++
 sw/UITest_writer_tests4.mk |2 ++
 sw/UITest_writer_tests6.mk |2 ++
 4 files changed, 8 insertions(+)

New commits:
commit f7ec6a19dca9619bce038dd0aee88765a93255f6
Author: Miklos Vajna 
AuthorDate: Mon Mar 20 20:05:27 2023 +0100
Commit: Miklos Vajna 
CommitDate: Tue Mar 21 07:16:08 2023 +

UITest_writer_tests{2,3,4,6}: enable oneprocess mode

Execution time changes:

- from 4m8,209s to 2m23,235s
- from 4m0,718s to 2m35,536s
- from 2m52,488s to 1m58,201s
- from 2m40,288s to 1m48,718s

for me.

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

diff --git a/sw/UITest_writer_tests2.mk b/sw/UITest_writer_tests2.mk
index 771508014d23..641d394578aa 100644
--- a/sw/UITest_writer_tests2.mk
+++ b/sw/UITest_writer_tests2.mk
@@ -14,3 +14,5 @@ $(eval $(call 
gb_UITest_add_modules,writer_tests2,$(SRCDIR)/sw/qa/uitest,\
 $(eval $(call gb_UITest_set_defs,writer_tests2, \
 TDOC="$(SRCDIR)/sw/qa/uitest/data" \
 ))
+
+$(eval $(call gb_UITest_use_oneprocess,writer_tests2))
diff --git a/sw/UITest_writer_tests3.mk b/sw/UITest_writer_tests3.mk
index b68e7379c3a6..58126977f7f9 100644
--- a/sw/UITest_writer_tests3.mk
+++ b/sw/UITest_writer_tests3.mk
@@ -14,3 +14,5 @@ $(eval $(call 
gb_UITest_add_modules,writer_tests3,$(SRCDIR)/sw/qa/uitest,\
 $(eval $(call gb_UITest_set_defs,writer_tests3, \
 TDOC="$(SRCDIR)/sw/qa/uitest/data" \
 ))
+
+$(eval $(call gb_UITest_use_oneprocess,writer_tests3))
diff --git a/sw/UITest_writer_tests4.mk b/sw/UITest_writer_tests4.mk
index e08e955fd1cd..4bc9e89ccf65 100644
--- a/sw/UITest_writer_tests4.mk
+++ b/sw/UITest_writer_tests4.mk
@@ -14,3 +14,5 @@ $(eval $(call 
gb_UITest_add_modules,writer_tests4,$(SRCDIR)/sw/qa/uitest,\
 $(eval $(call gb_UITest_set_defs,writer_tests4, \
 TDOC="$(SRCDIR)/sw/qa/uitest/data" \
 ))
+
+$(eval $(call gb_UITest_use_oneprocess,writer_tests4))
diff --git a/sw/UITest_writer_tests6.mk b/sw/UITest_writer_tests6.mk
index 1b17cd262333..10e7112a1ce3 100644
--- a/sw/UITest_writer_tests6.mk
+++ b/sw/UITest_writer_tests6.mk
@@ -14,3 +14,5 @@ $(eval $(call 
gb_UITest_add_modules,writer_tests6,$(SRCDIR)/sw/qa/uitest,\
 $(eval $(call gb_UITest_set_defs,writer_tests6, \
 TDOC="$(SRCDIR)/sw/qa/uitest/data" \
 ))
+
+$(eval $(call gb_UITest_use_oneprocess,writer_tests6))


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

2023-03-21 Thread Szymon Kłos (via logerrit)
 vcl/inc/jsdialog/jsdialogbuilder.hxx |1 
 vcl/inc/salvtables.hxx   |   54 
 vcl/jsdialog/executor.cxx|   22 +
 vcl/jsdialog/jsdialogbuilder.cxx |   12 
 vcl/source/app/salvtables.cxx|  438 +--
 5 files changed, 304 insertions(+), 223 deletions(-)

New commits:
commit b9484dfdc2545a1c8eb954b260b5e122d1ede56c
Author: Szymon Kłos 
AuthorDate: Thu Mar 9 14:17:23 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 07:18:06 2023 +

Move SalInstanceScrolledWindow declaration to header file

Change-Id: I8848358cf04ad0298867903350476f1f3b38b7da
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148608
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149160
Tested-by: Jenkins

diff --git a/vcl/inc/salvtables.hxx b/vcl/inc/salvtables.hxx
index 1bbf37b4f7fb..3c421fc89c90 100644
--- a/vcl/inc/salvtables.hxx
+++ b/vcl/inc/salvtables.hxx
@@ -2108,4 +2108,58 @@ public:
 virtual void set_image(const css::uno::Reference& 
rImage) override;
 };
 
+class SalInstanceScrolledWindow : public SalInstanceContainer, public virtual 
weld::ScrolledWindow
+{
+private:
+VclPtr m_xScrolledWindow;
+Link m_aOrigVScrollHdl;
+Link m_aOrigHScrollHdl;
+bool m_bUserManagedScrolling;
+
+DECL_LINK(VscrollHdl, ScrollBar*, void);
+DECL_LINK(HscrollHdl, ScrollBar*, void);
+
+static void customize_scrollbars(ScrollBar& rScrollBar, const Color& 
rButtonTextColor,
+ const Color& rBackgroundColor, const 
Color& rShadowColor,
+ const Color& rFaceColor);
+
+public:
+SalInstanceScrolledWindow(VclScrolledWindow* pScrolledWindow, 
SalInstanceBuilder* pBuilder,
+  bool bTakeOwnership, bool bUserManagedScrolling);
+
+virtual void hadjustment_configure(int value, int lower, int upper, int 
step_increment,
+   int page_increment, int page_size) 
override;
+virtual int hadjustment_get_value() const override;
+virtual void hadjustment_set_value(int value) override;
+virtual int hadjustment_get_upper() const override;
+virtual void hadjustment_set_upper(int upper) override;
+virtual int hadjustment_get_page_size() const override;
+virtual void hadjustment_set_page_size(int size) override;
+virtual void hadjustment_set_page_increment(int size) override;
+virtual void hadjustment_set_step_increment(int size) override;
+virtual void set_hpolicy(VclPolicyType eHPolicy) override;
+virtual VclPolicyType get_hpolicy() const override;
+
+virtual void vadjustment_configure(int value, int lower, int upper, int 
step_increment,
+   int page_increment, int page_size) 
override;
+virtual int vadjustment_get_value() const override;
+virtual void vadjustment_set_value(int value) override;
+virtual int vadjustment_get_upper() const override;
+virtual void vadjustment_set_upper(int upper) override;
+virtual int vadjustment_get_lower() const override;
+virtual void vadjustment_set_lower(int lower) override;
+virtual int vadjustment_get_page_size() const override;
+virtual void vadjustment_set_page_size(int size) override;
+virtual void vadjustment_set_page_increment(int size) override;
+virtual void vadjustment_set_step_increment(int size) override;
+
+virtual void set_vpolicy(VclPolicyType eVPolicy) override;
+virtual VclPolicyType get_vpolicy() const override;
+virtual int get_scroll_thickness() const override;
+virtual void set_scroll_thickness(int nThickness) override;
+virtual void customize_scrollbars(const Color& rBackgroundColor, const 
Color& rShadowColor,
+  const Color& rFaceColor) override;
+virtual ~SalInstanceScrolledWindow() override;
+};
+
 /* vim:set shiftwidth=4 softtabstop=4 expandtab cinoptions=b1,g0,N-s 
cinkeys+=0=break: */
diff --git a/vcl/source/app/salvtables.cxx b/vcl/source/app/salvtables.cxx
index 61fa56dd3223..4a31b3d2cd3d 100644
--- a/vcl/source/app/salvtables.cxx
+++ b/vcl/source/app/salvtables.cxx
@@ -2247,257 +2247,251 @@ public:
 
 virtual int get_position() const override { return 
m_xPaned->get_position(); }
 };
+}
 
-class SalInstanceScrolledWindow : public SalInstanceContainer, public virtual 
weld::ScrolledWindow
-{
-private:
-VclPtr m_xScrolledWindow;
-Link m_aOrigVScrollHdl;
-Link m_aOrigHScrollHdl;
-bool m_bUserManagedScrolling;
-
-DECL_LINK(VscrollHdl, ScrollBar*, void);
-DECL_LINK(HscrollHdl, ScrollBar*, void);
-
-static void customize_scrollbars(ScrollBar& rScrollBar, const Color& 
rButtonTextColor,
- const Color& rBackgroundColor, const 
Color& rShadowColor,
- const Color& rFaceColor)
-{
-rScrollB

Re: FYI: Compatibility of Gradients

2023-03-21 Thread Joao S. O. Bueno
Thanks for that  -

It is really insightful and rather complete.

On Mon, Mar 20, 2023 at 4:44 PM Regina Henschel 
wrote:

> Hi all,
>
> I've been working on importing and exporting gradients for Fontwork
> shapes lately. In the process, I realized how incompatible gradients are
> in Microsoft Office and LibreOffice.
>
> To get an overview, I wrote down a comparison. Since this might be of
> interest to you as well, I put it in the wiki.
> https://wiki.documentfoundation.org/User:Regina/Compatibility_of_Gradients
>
> Kind regards,
> Regina
>


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

2023-03-21 Thread Miklos Vajna (via logerrit)
 sw/qa/extras/ooxmlexport/ooxmlexport9.cxx |   16 
 1 file changed, 12 insertions(+), 4 deletions(-)

New commits:
commit 8bc607225e6bc9ba343e2292f9185b6491108e44
Author: Miklos Vajna 
AuthorDate: Tue Mar 21 08:17:46 2023 +0100
Commit: Miklos Vajna 
CommitDate: Tue Mar 21 08:17:28 2023 +

sw floattable, CppunitTest_sw_ooxmlexport9: assert the layout in 
testTdf107889

The point is that the table is multi-page, which is preserved. The
detail that it's not in a fly is no longer true.

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

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
index 9013dc83c126..da430214d75f 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
@@ -1156,10 +1156,18 @@ DECLARE_OOXMLEXPORT_TEST(testTdf107033, 
"tdf107033.docx")
 }
 
 #if HAVE_MORE_FONTS
-DECLARE_OOXMLEXPORT_TEST(testTdf107889, "tdf107889.docx")
-{
-// This was 1, multi-page table was imported as a floating one.
-CPPUNIT_ASSERT_EQUAL(0, getShapes());
+CPPUNIT_TEST_FIXTURE(Test, testTdf107889)
+{
+SwModelTestBase::FlySplitGuard aGuard;
+auto verify = [this]() {
+// This was 1, multi-page table was imported as a non-split fly.
+xmlDocUniquePtr pXmlDoc = parseLayoutDump();
+assertXPath(pXmlDoc, "//tab", 2);
+};
+createSwDoc("tdf107889.docx");
+verify();
+reload(mpFilter, "tdf107889.docx");
+verify();
 }
 #endif
 


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

2023-03-21 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtkinst.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit fd32093df9fdf5d46ed4def9fd8dada7d0d5e361
Author: Caolán McNamara 
AuthorDate: Mon Mar 20 11:31:25 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 08:36:05 2023 +

tdf#154232 signal_drag_begin callback can delete current GtkTargetList

so fetch the up to date one on calling gtk_drag_begin_with_coordinates

==3221972== Invalid read of size 8
==3221972==at 0x23F281DA: gtk_drag_begin_internal (gtkdnd.c:1801)
==3221972==by 0x23F2886F: gtk_drag_begin_with_coordinates 
(gtkdnd.c:1995)
==3221972==by 0x23845075: (anonymous 
namespace)::GtkInstanceWidget::signal_motion(_GdkEventMotion const*) 
(gtkinst.cxx:3139)
==3221972==by 0x23844F09: (anonymous 
namespace)::GtkInstanceWidget::signalMotion(_GtkWidget*, _GdkEventMotion*, 
void*) (gtkinst.cxx:3130)
==3221972==by 0x23C0FC56: _gtk_marshal_BOOLEAN__BOXED 
(gtkmarshalers.c:84)
==3221972==by 0x1378C05F: g_closure_invoke (gclosure.c:832)
==3221972==by 0x137B8F65: signal_emit_unlocked_R.isra.0 (gsignal.c:3796)
==3221972==by 0x137A8ED5: g_signal_emit_valist (gsignal.c:3559)
==3221972==by 0x137A96F2: g_signal_emit (gsignal.c:3606)
==3221972==by 0x23EF1073: gtk_widget_event_internal.part.0.lto_priv.0 
(gtkwidget.c:7812)
==3221972==by 0x23D825AD: UnknownInlinedFun (gtkmain.c:2588)
==3221972==by 0x23D825AD: propagate_event.lto_priv.0 (gtkmain.c:2691)
==3221972==by 0x23D83320: UnknownInlinedFun (gtkmain.c:1921)
==3221972==by 0x23D83320: gtk_main_do_event (gtkmain.c:1691)
==3221972==  Address 0x2c7d8070 is 0 bytes inside a block of size 16 free'd
==3221972==at 0x48460E4: free (vg_replace_malloc.c:884)
==3221972==by 0x1383388C: g_free (gmem.c:229)
==3221972==by 0x1384E093: g_slice_free1 (gslice.c:1185)
==3221972==by 0x23CD7A08: gtk_drag_source_set (gtkdragsource.c:162)
==3221972==by 0x23ED1EAB: gtk_tree_view_enable_model_drag_source 
(gtktreeview.c:14303)
==3221972==by 0x23877F41: (anonymous 
namespace)::GtkInstanceTreeView::drag_source_set(std::__debug::vector<_GtkTargetEntry,
 std::allocator<_GtkTargetEntry> > const&, GdkDragAction) (gtkinst.cxx:16279)
==3221972==by 0x238440B3: (anonymous 
namespace)::GtkInstanceWidget::do_enable_drag_source(rtl::Reference
 const&, unsigned char) (gtkinst.cxx:2707)
==3221972==by 0x23877E99: (anonymous 
namespace)::GtkInstanceTreeView::enable_drag_source(rtl::Reference&,
 unsigned char) (gtkinst.cxx:16270)
==3221972==by 0x36AAE6D2: SwContentTree::DragBeginHdl(bool&) 
(content.cxx:1227)
==3221972==by 0x36AAE2CE: SwContentTree::LinkStubDragBeginHdl(void*, 
bool&) (content.cxx:1180)
==3221972==by 0x238D38DA: Link::Call(bool&) const 
(link.hxx:111)
==3221972==by 0x23878D2D: (anonymous 
namespace)::GtkInstanceTreeView::do_signal_drag_begin(bool&) (gtkinst.cxx:16520)

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

diff --git a/vcl/unx/gtk3/gtkinst.cxx b/vcl/unx/gtk3/gtkinst.cxx
index 3177ec5272d2..5fe9ff9779fe 100644
--- a/vcl/unx/gtk3/gtkinst.cxx
+++ b/vcl/unx/gtk3/gtkinst.cxx
@@ -3132,12 +3132,12 @@ private:
 
 bool signal_motion(const GdkEventMotion* pEvent)
 {
-GtkTargetList* pDragData = (m_eDragAction != 0 && m_nPressedButton != 
-1 && m_xDragSource.is()) ? gtk_drag_source_get_target_list(m_pWidget) : 
nullptr;
+const bool bDragData = m_eDragAction != 0 && m_nPressedButton != -1 && 
m_xDragSource.is() && gtk_drag_source_get_target_list(m_pWidget);
 bool bUnsetDragIcon(false);
-if (pDragData && gtk_drag_check_threshold(m_pWidget, m_nPressStartX, 
m_nPressStartY, pEvent->x, pEvent->y) && !do_signal_drag_begin(bUnsetDragIcon))
+if (bDragData && gtk_drag_check_threshold(m_pWidget, m_nPressStartX, 
m_nPressStartY, pEvent->x, pEvent->y) && !do_signal_drag_begin(bUnsetDragIcon))
 {
 GdkDragContext* pContext = 
gtk_drag_begin_with_coordinates(m_pWidget,
-   
pDragData,
+   
gtk_drag_source_get_target_list(m_pWidget),

m_eDragAction,

m_nPressedButton,

const_cast(reinterpret_cast(pEvent)),


[Libreoffice-commits] core.git: Branch 'libreoffice-7-4' - vcl/unx

2023-03-21 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtkinst.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit 9cb9bda78a7e47c4948e6ef2702f07460b22050a
Author: Caolán McNamara 
AuthorDate: Mon Mar 20 11:31:25 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 08:36:25 2023 +

tdf#154232 signal_drag_begin callback can delete current GtkTargetList

so fetch the up to date one on calling gtk_drag_begin_with_coordinates

==3221972== Invalid read of size 8
==3221972==at 0x23F281DA: gtk_drag_begin_internal (gtkdnd.c:1801)
==3221972==by 0x23F2886F: gtk_drag_begin_with_coordinates 
(gtkdnd.c:1995)
==3221972==by 0x23845075: (anonymous 
namespace)::GtkInstanceWidget::signal_motion(_GdkEventMotion const*) 
(gtkinst.cxx:3139)
==3221972==by 0x23844F09: (anonymous 
namespace)::GtkInstanceWidget::signalMotion(_GtkWidget*, _GdkEventMotion*, 
void*) (gtkinst.cxx:3130)
==3221972==by 0x23C0FC56: _gtk_marshal_BOOLEAN__BOXED 
(gtkmarshalers.c:84)
==3221972==by 0x1378C05F: g_closure_invoke (gclosure.c:832)
==3221972==by 0x137B8F65: signal_emit_unlocked_R.isra.0 (gsignal.c:3796)
==3221972==by 0x137A8ED5: g_signal_emit_valist (gsignal.c:3559)
==3221972==by 0x137A96F2: g_signal_emit (gsignal.c:3606)
==3221972==by 0x23EF1073: gtk_widget_event_internal.part.0.lto_priv.0 
(gtkwidget.c:7812)
==3221972==by 0x23D825AD: UnknownInlinedFun (gtkmain.c:2588)
==3221972==by 0x23D825AD: propagate_event.lto_priv.0 (gtkmain.c:2691)
==3221972==by 0x23D83320: UnknownInlinedFun (gtkmain.c:1921)
==3221972==by 0x23D83320: gtk_main_do_event (gtkmain.c:1691)
==3221972==  Address 0x2c7d8070 is 0 bytes inside a block of size 16 free'd
==3221972==at 0x48460E4: free (vg_replace_malloc.c:884)
==3221972==by 0x1383388C: g_free (gmem.c:229)
==3221972==by 0x1384E093: g_slice_free1 (gslice.c:1185)
==3221972==by 0x23CD7A08: gtk_drag_source_set (gtkdragsource.c:162)
==3221972==by 0x23ED1EAB: gtk_tree_view_enable_model_drag_source 
(gtktreeview.c:14303)
==3221972==by 0x23877F41: (anonymous 
namespace)::GtkInstanceTreeView::drag_source_set(std::__debug::vector<_GtkTargetEntry,
 std::allocator<_GtkTargetEntry> > const&, GdkDragAction) (gtkinst.cxx:16279)
==3221972==by 0x238440B3: (anonymous 
namespace)::GtkInstanceWidget::do_enable_drag_source(rtl::Reference
 const&, unsigned char) (gtkinst.cxx:2707)
==3221972==by 0x23877E99: (anonymous 
namespace)::GtkInstanceTreeView::enable_drag_source(rtl::Reference&,
 unsigned char) (gtkinst.cxx:16270)
==3221972==by 0x36AAE6D2: SwContentTree::DragBeginHdl(bool&) 
(content.cxx:1227)
==3221972==by 0x36AAE2CE: SwContentTree::LinkStubDragBeginHdl(void*, 
bool&) (content.cxx:1180)
==3221972==by 0x238D38DA: Link::Call(bool&) const 
(link.hxx:111)
==3221972==by 0x23878D2D: (anonymous 
namespace)::GtkInstanceTreeView::do_signal_drag_begin(bool&) (gtkinst.cxx:16520)

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

diff --git a/vcl/unx/gtk3/gtkinst.cxx b/vcl/unx/gtk3/gtkinst.cxx
index 8697f7f7eb47..1841ebbaf7f4 100644
--- a/vcl/unx/gtk3/gtkinst.cxx
+++ b/vcl/unx/gtk3/gtkinst.cxx
@@ -3120,12 +3120,12 @@ private:
 
 bool signal_motion(const GdkEventMotion* pEvent)
 {
-GtkTargetList* pDragData = (m_eDragAction != 0 && m_nPressedButton != 
-1 && m_xDragSource.is()) ? gtk_drag_source_get_target_list(m_pWidget) : 
nullptr;
+const bool bDragData = m_eDragAction != 0 && m_nPressedButton != -1 && 
m_xDragSource.is() && gtk_drag_source_get_target_list(m_pWidget);
 bool bUnsetDragIcon(false);
-if (pDragData && gtk_drag_check_threshold(m_pWidget, m_nPressStartX, 
m_nPressStartY, pEvent->x, pEvent->y) && !do_signal_drag_begin(bUnsetDragIcon))
+if (bDragData && gtk_drag_check_threshold(m_pWidget, m_nPressStartX, 
m_nPressStartY, pEvent->x, pEvent->y) && !do_signal_drag_begin(bUnsetDragIcon))
 {
 GdkDragContext* pContext = 
gtk_drag_begin_with_coordinates(m_pWidget,
-   
pDragData,
+   
gtk_drag_source_get_target_list(m_pWidget),

m_eDragAction,

m_nPressedButton,

const_cast(reinterpret_cast(pEvent)),


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 vcl/jsdialog/enabled.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 38eaa71245954d0dd8805b2140039f126a8ea0e3
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 20:32:32 2023 +0530
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 08:41:03 2023 +

jsdialog: enabled spelling dialog

Change-Id: I36c35e82aa17d8e6cb76dbb24f33344870c49629
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149172
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 116cc6a5b768..4e5c79c1bf9b 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -55,6 +55,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"cui/ui/queryduplicatedialog.ui"
 || rUIFile == u"cui/ui/similaritysearchdialog.ui"
 || rUIFile == u"cui/ui/specialcharacters.ui"
+|| rUIFile == u"cui/ui/spellingdialog.ui"
 || rUIFile == u"cui/ui/splitcellsdialog.ui"
 || rUIFile == u"cui/ui/textflowpage.ui"
 || rUIFile == u"cui/ui/transparencytabpage.ui"


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

2023-03-21 Thread Noel Grandin (via logerrit)
 chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx|6 
 chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx |2 
 chart2/source/controller/itemsetwrapper/TextLabelItemConverter.cxx |2 
 chart2/source/inc/Diagram.hxx  |   15 +
 chart2/source/inc/DiagramHelper.hxx|   16 -
 chart2/source/model/main/Diagram.cxx   |   85 
+
 chart2/source/model/template/BarChartTypeTemplate.cxx  |6 
 chart2/source/model/template/StockChartTypeTemplate.cxx|2 
 chart2/source/tools/DiagramHelper.cxx  |   94 
--
 chart2/source/tools/TitleHelper.cxx|4 
 chart2/source/view/main/ChartView.cxx  |4 
 11 files changed, 113 insertions(+), 123 deletions(-)

New commits:
commit df12a23157f4515ba5e78c0bd293dfd1609de668
Author: Noel Grandin 
AuthorDate: Thu Mar 16 09:32:07 2023 +0200
Commit: Noel Grandin 
CommitDate: Tue Mar 21 08:44:53 2023 +

move getVertical/setVertical inside chart2::Diagram

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

diff --git a/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx 
b/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx
index 657751ed3c23..7cc7c57e2862 100644
--- a/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx
+++ b/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx
@@ -1385,9 +1385,9 @@ void WrappedVerticalProperty::setPropertyValue( const 
Any& rOuterValue, const Re
 
 bool bFound = false;
 bool bAmbiguous = false;
-bool bOldVertical = DiagramHelper::getVertical( xDiagram, bFound, 
bAmbiguous );
+bool bOldVertical = xDiagram->getVertical( bFound, bAmbiguous );
 if( bFound && ( bOldVertical != bNewVertical || bAmbiguous ) )
-DiagramHelper::setVertical( xDiagram, bNewVertical );
+xDiagram->setVertical( bNewVertical );
 }
 
 Any WrappedVerticalProperty::getPropertyValue( const Reference< 
beans::XPropertySet >& /*xInnerPropertySet*/ ) const
@@ -1397,7 +1397,7 @@ Any WrappedVerticalProperty::getPropertyValue( const 
Reference< beans::XProperty
 {
 bool bFound = false;
 bool bAmbiguous = false;
-bool bVertical = DiagramHelper::getVertical( xDiagram, bFound, 
bAmbiguous );
+bool bVertical = xDiagram->getVertical( bFound, bAmbiguous );
 if( bFound )
 m_aOuterValue <<= bVertical;
 }
diff --git a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
index b0155174cd94..f415cd194386 100644
--- a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
@@ -242,7 +242,7 @@ DataPointItemConverter::DataPointItemConverter(
 rtl::Reference< ChartType > xChartType( xDiagram->getChartTypeOfSeries( 
xSeries ) );
 bool bFound = false;
 bool bAmbiguous = false;
-bool bSwapXAndY = DiagramHelper::getVertical( xDiagram, bFound, bAmbiguous 
);
+bool bSwapXAndY = xDiagram->getVertical( bFound, bAmbiguous );
 m_aAvailableLabelPlacements = 
ChartTypeHelper::getSupportedLabelPlacements( xChartType, bSwapXAndY, xSeries );
 
 m_bForbidPercentValue = ChartTypeHelper::getAxisType( xChartType, 0 ) != 
AxisType::CATEGORY;
diff --git a/chart2/source/controller/itemsetwrapper/TextLabelItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/TextLabelItemConverter.cxx
index 929f1c7c48b9..b4097d21da41 100644
--- a/chart2/source/controller/itemsetwrapper/TextLabelItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/TextLabelItemConverter.cxx
@@ -215,7 +215,7 @@ TextLabelItemConverter::TextLabelItemConverter(
 rtl::Reference< ChartType > 
xChartType(xDiagram->getChartTypeOfSeries(xSeries));
 bool bFound = false;
 bool bAmbiguous = false;
-bool bSwapXAndY = DiagramHelper::getVertical(xDiagram, bFound, bAmbiguous);
+bool bSwapXAndY = xDiagram->getVertical(bFound, bAmbiguous);
 maAvailableLabelPlacements = 
ChartTypeHelper::getSupportedLabelPlacements(xChartType, bSwapXAndY, xSeries);
 
 mbForbidPercentValue = ChartTypeHelper::getAxisType(xChartType, 0) != 
AxisType::CATEGORY;
diff --git a/chart2/source/inc/Diagram.hxx b/chart2/source/inc/Diagram.hxx
index 9ff0f94bb020..c24d10af9d83 100644
--- a/chart2/source/inc/Diagram.hxx
+++ b/chart2/source/inc/Diagram.hxx
@@ -300,6 +300,21 @@ public:
  */
 void setStackMode(StackMode eStackMode);
 
+
+/** Sets the "SwapXAndYAxis" property at all coordinate systems found in 
the
+given diagram.
+
+"vertical==true" for bar charts, "vertical==false" for column charts
+ */
+void setVer

[Libreoffice-commits] core.git: Branch 'libreoffice-7-5' - download.lst

2023-03-21 Thread Michael Stahl (via logerrit)
 download.lst |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit f2736b09046af9e9461fb13faae37b09ac941163
Author: Michael Stahl 
AuthorDate: Mon Mar 20 11:52:22 2023 +0100
Commit: Xisco Fauli 
CommitDate: Tue Mar 21 08:53:45 2023 +

curl: upgrade to release 8.0.0

Fixes CVE-2023-27535.

Also hopefully fixes excessive storage consumption during build:
o build: drop the use of XC_AMEND_DISTCLEAN [62]

Change-Id: I8792e95bc7634ee496488e80fec5a1310b24a31c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149153
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
(cherry picked from commit bbe0d0534abe6480c4502ce8fb543a736d3399d8)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149105
Reviewed-by: Xisco Fauli 

diff --git a/download.lst b/download.lst
index 5719657456f3..71eec4128749 100644
--- a/download.lst
+++ b/download.lst
@@ -75,8 +75,8 @@ CPPUNIT_TARBALL := cppunit-1.15.1.tar.gz
 # three static lines
 # so that git cherry-pick
 # will not run into conflicts
-CURL_SHA256SUM := 
1dae31b2a7c1fe269de99c0c31bb488346aab3459b5ffca909d6938249ae415f
-CURL_TARBALL := curl-7.88.1.tar.xz
+CURL_SHA256SUM := 
759690f9a375a720f8bcce9f953897b0d93f31eed9649b74f846d54bbf63bbcc
+CURL_TARBALL := curl-8.0.0.tar.xz
 # three static lines
 # so that git cherry-pick
 # will not run into conflicts


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

2023-03-21 Thread Tor Lillqvist (via logerrit)
 offapi/com/sun/star/accessibility/XAccessibleContext.idl |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 660f73385562314e01abb4cef9d443b0c0f42b06
Author: Tor Lillqvist 
AuthorDate: Tue Mar 21 09:48:04 2023 +0200
Commit: Tor Lillqvist 
CommitDate: Tue Mar 21 09:10:53 2023 +

There is no XAccessibleStateType

What is meant is surely AccessibleStateType.

Change-Id: I57903dd7e1eb9b0a3cac42a96633093569f770eb
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149194
Tested-by: Jenkins
Reviewed-by: Tor Lillqvist 

diff --git a/offapi/com/sun/star/accessibility/XAccessibleContext.idl 
b/offapi/com/sun/star/accessibility/XAccessibleContext.idl
index 84c8a21cfdea..6b1b16701717 100644
--- a/offapi/com/sun/star/accessibility/XAccessibleContext.idl
+++ b/offapi/com/sun/star/accessibility/XAccessibleContext.idl
@@ -152,10 +152,10 @@ interface XAccessibleContext : 
::com::sun::star::uno::XInterface
 /** Returns the set of states that are currently active for this
 object.
 
-See the documentation of XAccessibleStateType for a
+See the documentation of AccessibleStateType for a
 description of the individual states.
 
-@see XAccessibleStateType
+@see AccessibleStateType
 */
 hyper getAccessibleStateSet ();
 


[Libreoffice-commits] core.git: Branch 'distro/cib/libreoffice-6-4' - sc/qa sc/source

2023-03-21 Thread Regina Henschel (via logerrit)
 sc/qa/unit/data/ods/tdf117948_CollapseBeforeShape.ods |binary
 sc/qa/unit/scshapetest.cxx|  112 ++
 sc/source/filter/xml/xmlexprt.cxx |   47 ++-
 3 files changed, 150 insertions(+), 9 deletions(-)

New commits:
commit 393d061a0e34917b1029f399b86058bd94a16d51
Author: Regina Henschel 
AuthorDate: Sat Nov 7 01:08:53 2020 +0100
Commit: Thorsten Behrens 
CommitDate: Tue Mar 21 09:25:26 2023 +

tdf#117948 Do not treat hidden rows as zero in ODF export

The object geometry in ODF file format has values so as if no hidden
columns or rows exists. But for rendering the object geometry has to
treat hidden rows and columns as zero. This patch changes the object
geometry temporarily to the 'no hidden columns and rows' mode for
export and restores the original geometry afterwards.
The patch considers hidden columns left from the shape and hidden rows
above the shape. So the object is shifted. Considering hidden columns
or rows in the area, which is covered by the shape, is still missing.
That would possibly require scaling.

cherry-pick from commit: 1cb6bb9576871ff5d56c9810ff1791abc3dd64fd
(tdf#117948 Do not treat hidden rows as zero in ODF export)

Change-Id: Icdb3f08404ca4d212d25a1967bfdc0bfc7186007
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/105427
Tested-by: Jenkins
Reviewed-by: Regina Henschel 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149163
Tested-by: Gabor Kelemen 
Reviewed-by: Thorsten Behrens 

diff --git a/sc/qa/unit/data/ods/tdf117948_CollapseBeforeShape.ods 
b/sc/qa/unit/data/ods/tdf117948_CollapseBeforeShape.ods
new file mode 100644
index ..bbc1af87cf07
Binary files /dev/null and 
b/sc/qa/unit/data/ods/tdf117948_CollapseBeforeShape.ods differ
diff --git a/sc/qa/unit/scshapetest.cxx b/sc/qa/unit/scshapetest.cxx
index 55a2b0a8537c..f348854bb9e3 100644
--- a/sc/qa/unit/scshapetest.cxx
+++ b/sc/qa/unit/scshapetest.cxx
@@ -13,6 +13,7 @@
 #include 
 #include 
 #include 
+#include 
 
 #include 
 #include 
@@ -29,14 +30,18 @@ class ScShapeTest : public CalcUnoApiTest
 {
 public:
 ScShapeTest();
+void saveAndReload(css::uno::Reference& xComponent,
+   const OUString& rFilter);
 
 virtual void tearDown() override;
 
+void testTdf117948_CollapseBeforeShape();
 void testFitToCellSize();
 void testCustomShapeCellAnchoredRotatedShape();
 void testFormSizeWithHiddenCol();
 
 CPPUNIT_TEST_SUITE(ScShapeTest);
+CPPUNIT_TEST(testTdf117948_CollapseBeforeShape);
 CPPUNIT_TEST(testFitToCellSize);
 CPPUNIT_TEST(testCustomShapeCellAnchoredRotatedShape);
 CPPUNIT_TEST(testFormSizeWithHiddenCol);
@@ -51,6 +56,21 @@ ScShapeTest::ScShapeTest()
 {
 }
 
+void ScShapeTest::saveAndReload(css::uno::Reference& 
xComponent,
+const OUString& rFilter)
+{
+utl::TempFile aTempFile;
+aTempFile.EnableKillingFile();
+css::uno::Sequence aArgs(1);
+aArgs[0].Name = "FilterName";
+aArgs[0].Value <<= rFilter; // e.g. "calc8"
+css::uno::Reference xStorable(xComponent, 
css::uno::UNO_QUERY_THROW);
+xStorable->storeAsURL(aTempFile.GetURL(), aArgs);
+css::uno::Reference xCloseable(xComponent, 
css::uno::UNO_QUERY_THROW);
+xCloseable->close(true);
+xComponent = loadFromDesktop(aTempFile.GetURL(), 
"com.sun.star.sheet.SpreadsheetDocument");
+}
+
 static OUString lcl_compareRectWithTolerance(const tools::Rectangle& rExpected,
  const tools::Rectangle& rActual,
  const sal_Int32 nTolerance)
@@ -74,6 +94,98 @@ static OUString lcl_compareRectWithTolerance(const 
tools::Rectangle& rExpected,
 return sErrors;
 }
 
+static void lcl_AssertRectEqualWithTolerance(const OString& sInfo,
+ const tools::Rectangle& rExpected,
+ const tools::Rectangle& rActual,
+ const sal_Int32 nTolerance)
+{
+// Left
+OString sMsg = sInfo + " Left expected " + 
OString::number(rExpected.Left()) + " actual "
+   + OString::number(rActual.Left()) + " Tolerance " + 
OString::number(nTolerance);
+CPPUNIT_ASSERT_MESSAGE(sMsg.getStr(), labs(rExpected.Left() - 
rActual.Left()) <= nTolerance);
+// Top
+sMsg = sInfo + " Top expected " + OString::number(rExpected.Top()) + " 
actual "
+   + OString::number(rActual.Top()) + " Tolerance " + 
OString::number(nTolerance);
+CPPUNIT_ASSERT_MESSAGE(sMsg.getStr(), labs(rExpected.Top() - 
rActual.Top()) <= nTolerance);
+// Width
+sMsg = sInfo + " Width expected " + OString::number(rExpected.GetWidth()) 
+ " actual "
+   + OString::number(rActual.GetWidth()) + " Tolerance " + 
OString::number(nTolerance);
+CPPUNIT_ASSERT_MESSAGE(sMsg.getS

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

2023-03-21 Thread Szymon Kłos (via logerrit)
 include/vcl/layout.hxx   |1 
 vcl/inc/jsdialog/jsdialogbuilder.hxx |   22 
 vcl/jsdialog/jsdialogbuilder.cxx |   40 -
 vcl/source/window/layout.cxx |   47 +++
 4 files changed, 48 insertions(+), 62 deletions(-)

New commits:
commit 8f0a337afd8cf40a74e6668b4441903a1b49f4ae
Author: Szymon Kłos 
AuthorDate: Thu Mar 9 15:43:15 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 09:35:41 2023 +

jsdialog: dump properties of scrolled window

Change-Id: Id6d072e951efec4c1de7641c94017c82840fa50b
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148609
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149190
Tested-by: Jenkins

diff --git a/include/vcl/layout.hxx b/include/vcl/layout.hxx
index 2296f8097a4a..f5090b24eac2 100644
--- a/include/vcl/layout.hxx
+++ b/include/vcl/layout.hxx
@@ -498,6 +498,7 @@ public:
 //to override it
 void setUserManagedScrolling(bool bUserManagedScrolling) { 
m_bUserManagedScrolling = bUserManagedScrolling;}
 void doSetAllocation(const Size &rAllocation, bool bRetryOnFailure);
+virtual void DumpAsPropertyTree(::tools::JsonWriter& rJsonWriter) override;
 private:
 virtual Size calculateRequisition() const override;
 virtual void setAllocation(const Size &rAllocation) override;
diff --git a/vcl/source/window/layout.cxx b/vcl/source/window/layout.cxx
index 039cb2690a9d..c77a4a0d30e1 100644
--- a/vcl/source/window/layout.cxx
+++ b/vcl/source/window/layout.cxx
@@ -2167,6 +2167,53 @@ void VclScrolledWindow::Paint(vcl::RenderContext& 
rRenderContext, const tools::R
 nBorderWidth << " is larger than expected " << m_nBorderWidth);
 }
 
+namespace {
+void lcl_dumpScrollbar(::tools::JsonWriter& rJsonWriter, ScrollBar& rScrollBar)
+{
+rJsonWriter.put("lower", rScrollBar.GetRangeMin());
+rJsonWriter.put("upper", rScrollBar.GetRangeMax());
+rJsonWriter.put("step_increment", rScrollBar.GetLineSize());
+rJsonWriter.put("page_increment", rScrollBar.GetPageSize());
+rJsonWriter.put("value", rScrollBar.GetThumbPos());
+rJsonWriter.put("page_size", rScrollBar.GetVisibleSize());
+}
+};
+
+void VclScrolledWindow::DumpAsPropertyTree(::tools::JsonWriter& rJsonWriter)
+{
+VclBin::DumpAsPropertyTree(rJsonWriter);
+
+{
+auto aVertical = rJsonWriter.startNode("vertical");
+
+ScrollBar& rScrollBar = getVertScrollBar();
+lcl_dumpScrollbar(rJsonWriter, rScrollBar);
+
+WinBits nWinBits = GetStyle();
+if (nWinBits & WB_VSCROLL)
+rJsonWriter.put("policy", "always");
+else if (nWinBits & WB_AUTOVSCROLL)
+rJsonWriter.put("policy", "auto");
+else
+rJsonWriter.put("policy", "never");
+}
+
+{
+auto aHorizontal = rJsonWriter.startNode("horizontal");
+
+ScrollBar& rScrollBar = getHorzScrollBar();
+lcl_dumpScrollbar(rJsonWriter, rScrollBar);
+
+WinBits nWinBits = GetStyle();
+if (nWinBits & WB_HSCROLL)
+rJsonWriter.put("policy", "always");
+else if (nWinBits & WB_AUTOHSCROLL)
+rJsonWriter.put("policy", "auto");
+else
+rJsonWriter.put("policy", "never");
+}
+}
+
 void VclViewport::setAllocation(const Size &rAllocation)
 {
 vcl::Window *pChild = get_child();
commit 103e656e4f9e073354dc6a683889fd3fb922ecf2
Author: Szymon Kłos 
AuthorDate: Tue Mar 7 13:09:20 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 09:35:30 2023 +

jsdialog: don't send full updates on tab change

All tabs are now sent with the first full message.
So client can switch between tabs by showing another
tabpage content. We don't need to send structure again.

Change-Id: I0f3dfc8be4816fba5813a83b41722407631edb73
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148421
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149189
Tested-by: Jenkins

diff --git a/vcl/inc/jsdialog/jsdialogbuilder.hxx 
b/vcl/inc/jsdialog/jsdialogbuilder.hxx
index 0864c3d7ab8a..19458a938c02 100644
--- a/vcl/inc/jsdialog/jsdialogbuilder.hxx
+++ b/vcl/inc/jsdialog/jsdialogbuilder.hxx
@@ -575,34 +575,12 @@ public:
 
 class JSNotebook final : public JSWidget
 {
-Link m_aLeavePageOverridenHdl;
-Link m_aEnterPageOverridenHdl;
-
-DECL_LINK(LeaveHdl, const OString&, bool);
-DECL_LINK(EnterHdl, const OString&, bool);
-
 public:
 JSNotebook(JSDialogSender* pSender, ::TabControl* pControl, 
SalInstanceBuilder* pBuilder,
bool bTakeOwnership);
 
-virtual void set_current_page(int nPage) override;
-
-virtual void set_current_page(const OString& rIdent) override;
-
 virtual void remove_page(const OString& rIdent) override;
-
 virtual void insert_page(const OStri

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

2023-03-21 Thread Caolán McNamara (via logerrit)
 cui/source/options/optgdlg.cxx |   20 ++--
 1 file changed, 2 insertions(+), 18 deletions(-)

New commits:
commit 4fd7d1a9e64ed256d442ef997c80bce92afe839b
Author: Caolán McNamara 
AuthorDate: Tue Mar 21 08:42:14 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 09:53:47 2023 +

cid#1524485 'Constant' variable guards dead code

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

diff --git a/cui/source/options/optgdlg.cxx b/cui/source/options/optgdlg.cxx
index b0bca101bdaa..04de401c4121 100644
--- a/cui/source/options/optgdlg.cxx
+++ b/cui/source/options/optgdlg.cxx
@@ -665,7 +665,6 @@ std::unique_ptr OfaViewTabPage::Create( 
weld::Container* pPage, weld
 bool OfaViewTabPage::FillItemSet( SfxItemSet* )
 {
 bool bModified = false;
-bool bMenuOptModified = false;
 bool bDarkModeOptModified = false;
 bool bRepaintWindows(false);
 std::shared_ptr 
xChanges(comphelper::ConfigurationChanges::create());
@@ -799,23 +798,8 @@ bool OfaViewTabPage::FillItemSet( SfxItemSet* )
 
 xChanges->commit();
 
-if (bMenuOptModified || bDarkModeOptModified)
-{
-// Set changed settings to the application instance
-AllSettings aAllSettings = Application::GetSettings();
-
-if (bMenuOptModified)
-{
-StyleSettings aStyleSettings = aAllSettings.GetStyleSettings();
-aAllSettings.SetStyleSettings(aStyleSettings);
-}
-
-if (bDarkModeOptModified)
-MiscSettings::SetDarkMode(m_xAppearanceStyleLB->get_active());
-
-Application::MergeSystemSettings( aAllSettings );
-Application::SetSettings(aAllSettings);
-}
+if (bDarkModeOptModified)
+MiscSettings::SetDarkMode(m_xAppearanceStyleLB->get_active());
 
 if ( bAppearanceChanged )
 {


[Libreoffice-commits] core.git: Branch 'libreoffice-7-5' - vcl/unx

2023-03-21 Thread Caolán McNamara (via logerrit)
 vcl/unx/gtk3/gtkinst.cxx |6 +++---
 1 file changed, 3 insertions(+), 3 deletions(-)

New commits:
commit c4f29ffc62af42365c983f4dc3514b2f8633c095
Author: Caolán McNamara 
AuthorDate: Mon Mar 20 11:31:25 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 09:53:58 2023 +

tdf#154232 signal_drag_begin callback can delete current GtkTargetList

so fetch the up to date one on calling gtk_drag_begin_with_coordinates

==3221972== Invalid read of size 8
==3221972==at 0x23F281DA: gtk_drag_begin_internal (gtkdnd.c:1801)
==3221972==by 0x23F2886F: gtk_drag_begin_with_coordinates 
(gtkdnd.c:1995)
==3221972==by 0x23845075: (anonymous 
namespace)::GtkInstanceWidget::signal_motion(_GdkEventMotion const*) 
(gtkinst.cxx:3139)
==3221972==by 0x23844F09: (anonymous 
namespace)::GtkInstanceWidget::signalMotion(_GtkWidget*, _GdkEventMotion*, 
void*) (gtkinst.cxx:3130)
==3221972==by 0x23C0FC56: _gtk_marshal_BOOLEAN__BOXED 
(gtkmarshalers.c:84)
==3221972==by 0x1378C05F: g_closure_invoke (gclosure.c:832)
==3221972==by 0x137B8F65: signal_emit_unlocked_R.isra.0 (gsignal.c:3796)
==3221972==by 0x137A8ED5: g_signal_emit_valist (gsignal.c:3559)
==3221972==by 0x137A96F2: g_signal_emit (gsignal.c:3606)
==3221972==by 0x23EF1073: gtk_widget_event_internal.part.0.lto_priv.0 
(gtkwidget.c:7812)
==3221972==by 0x23D825AD: UnknownInlinedFun (gtkmain.c:2588)
==3221972==by 0x23D825AD: propagate_event.lto_priv.0 (gtkmain.c:2691)
==3221972==by 0x23D83320: UnknownInlinedFun (gtkmain.c:1921)
==3221972==by 0x23D83320: gtk_main_do_event (gtkmain.c:1691)
==3221972==  Address 0x2c7d8070 is 0 bytes inside a block of size 16 free'd
==3221972==at 0x48460E4: free (vg_replace_malloc.c:884)
==3221972==by 0x1383388C: g_free (gmem.c:229)
==3221972==by 0x1384E093: g_slice_free1 (gslice.c:1185)
==3221972==by 0x23CD7A08: gtk_drag_source_set (gtkdragsource.c:162)
==3221972==by 0x23ED1EAB: gtk_tree_view_enable_model_drag_source 
(gtktreeview.c:14303)
==3221972==by 0x23877F41: (anonymous 
namespace)::GtkInstanceTreeView::drag_source_set(std::__debug::vector<_GtkTargetEntry,
 std::allocator<_GtkTargetEntry> > const&, GdkDragAction) (gtkinst.cxx:16279)
==3221972==by 0x238440B3: (anonymous 
namespace)::GtkInstanceWidget::do_enable_drag_source(rtl::Reference
 const&, unsigned char) (gtkinst.cxx:2707)
==3221972==by 0x23877E99: (anonymous 
namespace)::GtkInstanceTreeView::enable_drag_source(rtl::Reference&,
 unsigned char) (gtkinst.cxx:16270)
==3221972==by 0x36AAE6D2: SwContentTree::DragBeginHdl(bool&) 
(content.cxx:1227)
==3221972==by 0x36AAE2CE: SwContentTree::LinkStubDragBeginHdl(void*, 
bool&) (content.cxx:1180)
==3221972==by 0x238D38DA: Link::Call(bool&) const 
(link.hxx:111)
==3221972==by 0x23878D2D: (anonymous 
namespace)::GtkInstanceTreeView::do_signal_drag_begin(bool&) (gtkinst.cxx:16520)

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

diff --git a/vcl/unx/gtk3/gtkinst.cxx b/vcl/unx/gtk3/gtkinst.cxx
index a55c01cf8934..08ed12f21065 100644
--- a/vcl/unx/gtk3/gtkinst.cxx
+++ b/vcl/unx/gtk3/gtkinst.cxx
@@ -3122,12 +3122,12 @@ private:
 
 bool signal_motion(const GdkEventMotion* pEvent)
 {
-GtkTargetList* pDragData = (m_eDragAction != 0 && m_nPressedButton != 
-1 && m_xDragSource.is()) ? gtk_drag_source_get_target_list(m_pWidget) : 
nullptr;
+const bool bDragData = m_eDragAction != 0 && m_nPressedButton != -1 && 
m_xDragSource.is() && gtk_drag_source_get_target_list(m_pWidget);
 bool bUnsetDragIcon(false);
-if (pDragData && gtk_drag_check_threshold(m_pWidget, m_nPressStartX, 
m_nPressStartY, pEvent->x, pEvent->y) && !do_signal_drag_begin(bUnsetDragIcon))
+if (bDragData && gtk_drag_check_threshold(m_pWidget, m_nPressStartX, 
m_nPressStartY, pEvent->x, pEvent->y) && !do_signal_drag_begin(bUnsetDragIcon))
 {
 GdkDragContext* pContext = 
gtk_drag_begin_with_coordinates(m_pWidget,
-   
pDragData,
+   
gtk_drag_source_get_target_list(m_pWidget),

m_eDragAction,

m_nPressedButton,

const_cast(reinterpret_cast(pEvent)),


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

2023-03-21 Thread Miklos Vajna (via logerrit)
 sw/qa/extras/ooxmlexport/ooxmlexport10.cxx   |   53 ++---
 sw/qa/extras/ooxmlexport/ooxmlexport8.cxx|   61 ---
 sw/source/filter/ww8/docxtableexport.cxx |2 
 writerfilter/source/dmapper/DomainMapperTableHandler.cxx |4 
 4 files changed, 83 insertions(+), 37 deletions(-)

New commits:
commit 3777923b34eb72ee2034ea0dbf13d1d6ab122025
Author: Miklos Vajna 
AuthorDate: Mon Mar 20 12:14:53 2023 +0100
Commit: Miklos Vajna 
CommitDate: Tue Mar 21 10:06:59 2023 +

sw floattable: fix handling of nested non-floating tables at cell start

CppunitTest_sw_ooxmlexport8's testN779642 failed in the
SW_FORCE_FLY_SPLIT=1 case.

The bugdoc has a floating table, then an inner, non-floating table. The
conversion to floating tables was disabled by the
c1eebcdac9f2b289fd363399130c485ca5ff444c (tdf#79329 DOCX import: fix
missing outer table with floattable at cell start, 2016-11-08).

Given that SwFormatFlySplit doesn't need the delayed text frame
conversion, we can get rid of this workaround.

Fix a similar problem on the export side as well, where the positioning
properties of the outer table leaked into the inner table. Once both the
import and export are fixed, the test passes.

(cherry picked from commit 48818dd359fbf0f37e1b318de89ab2ea7d735f58)

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

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx
index 8cd6d4961f38..dbcd48aca0b6 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport8.cxx
@@ -720,34 +720,41 @@ DECLARE_OOXMLEXPORT_TEST(testN793998, "n793998.docx")
 CPPUNIT_ASSERT(nTextPortion + nTabPortion > nParagraph - nRightMargin);
 }
 
-DECLARE_OOXMLEXPORT_TEST(testN779642, "n779642.docx")
+CPPUNIT_TEST_FIXTURE(Test, testN779642)
 {
-uno::Reference xTextTablesSupplier(mxComponent, 
uno::UNO_QUERY);
-
-// First problem: check that we have 2 tables, nesting caused the
-// creation of outer one to fail
-uno::Reference 
xTables(xTextTablesSupplier->getTextTables(), uno::UNO_QUERY);
-CPPUNIT_ASSERT_EQUAL_MESSAGE("Wrong number of imported tables", 
sal_Int32(2), xTables->getCount());
-
-// Second problem: check that the outer table is in a frame, at the bottom 
of the page
-uno::Reference 
xTextTable(xTextTablesSupplier->getTextTables()->getByName("Table2"), 
uno::UNO_QUERY);
-uno::Reference xAnchor(xTextTable->getAnchor(), 
uno::UNO_QUERY);
-uno::Any aFrame = xAnchor->getPropertyValue("TextFrame");
-uno::Reference xFrame;
-aFrame >>= xFrame;
-sal_Int16 nValue;
-xFrame->getPropertyValue("VertOrient") >>= nValue;
-CPPUNIT_ASSERT_EQUAL_MESSAGE("Wrong vertical orientation", 
text::VertOrientation::BOTTOM, nValue);
-xFrame->getPropertyValue("VertOrientRelation") >>= nValue;
-CPPUNIT_ASSERT_EQUAL_MESSAGE("Wrong vertical orientation relation", 
text::RelOrientation::PAGE_PRINT_AREA, nValue);
-
-// tdf#106572 - perhaps not the best test to hijack since this file
-// produces an error in Word, but it nicely matches danger points,
-// and has a different first footer, so nice visual confirmation.
-discardDumpedLayout();
-xmlDocUniquePtr pXmlDoc = parseLayoutDump();
-// There is no footer text on the first page.
-assertXPath(pXmlDoc, "/root/page[1]/footer/txt", 0);
+SwModelTestBase::FlySplitGuard aGuard;
+auto verify = [this]() {
+uno::Reference 
xTextTablesSupplier(mxComponent, uno::UNO_QUERY);
+
+// First problem: check that we have 2 tables, nesting caused the
+// creation of outer one to fail
+uno::Reference 
xTables(xTextTablesSupplier->getTextTables(), uno::UNO_QUERY);
+CPPUNIT_ASSERT_EQUAL_MESSAGE("Wrong number of imported tables", 
sal_Int32(2), xTables->getCount());
+
+// Second problem: check that the outer table is in a frame, at the 
bottom of the page
+uno::Reference 
xTextTable(xTextTablesSupplier->getTextTables()->getByName("Table2"), 
uno::UNO_QUERY);
+uno::Reference xAnchor(xTextTable->getAnchor(), 
uno::UNO_QUERY);
+uno::Any aFrame = xAnchor->getPropertyValue("TextFrame");
+uno::Reference xFrame;
+aFrame >>= xFrame;
+sal_Int16 nValue;
+xFrame->getPropertyValue("VertOrient") >>= nValue;
+CPPUNIT_ASSERT_EQUAL_MESSAGE("Wrong vertical orientation", 
text::VertOrientation::BOTTOM, nValue);
+xFrame->getPropertyValue("VertOrientRelation") >>= nValue;
+CPPUNIT_ASSERT_EQUAL_MESSAGE("Wrong vertical orientation relation", 
text::RelOrientation::PAGE_PRINT_AREA, nValue);
+
+// tdf#106572 - perhaps not the best test to hijack since this file
+// produce

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

2023-03-21 Thread Xisco Fauli (via logerrit)
 sc/qa/unit/subsequent_filters_test4.cxx |   15 ---
 1 file changed, 4 insertions(+), 11 deletions(-)

New commits:
commit 53946b0698d63849266f465c16386dd8e9319f19
Author: Xisco Fauli 
AuthorDate: Tue Mar 21 09:49:31 2023 +0100
Commit: Xisco Fauli 
CommitDate: Tue Mar 21 10:57:02 2023 +

CppunitTest_sc_subsequent_filters_test4: simplify code a bit

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

diff --git a/sc/qa/unit/subsequent_filters_test4.cxx 
b/sc/qa/unit/subsequent_filters_test4.cxx
index 21a918086304..98f66c600f47 100644
--- a/sc/qa/unit/subsequent_filters_test4.cxx
+++ b/sc/qa/unit/subsequent_filters_test4.cxx
@@ -984,15 +984,12 @@ CPPUNIT_TEST_FIXTURE(ScFiltersTest4, 
testColorScaleNumWithRefXLSX)
 CPPUNIT_TEST_FIXTURE(ScFiltersTest4, testTdf153514)
 {
 ScDocument aDoc;
-OUString aFullUrl = 
m_directories.getURLFromSrc(u"sc/qa/unit/data/xml/tdf153514.xml");
-
-OUString aValidPath;
-osl::FileBase::getSystemPathFromFileURL(aFullUrl, aValidPath);
+OUString aPath = createFilePath(u"xml/tdf153514.xml");
 
 ScOrcusFilters* pOrcus = ScFormatFilter::Get().GetOrcusFilters();
 CPPUNIT_ASSERT(pOrcus);
 
-pOrcus->importODS_Styles(aDoc, aValidPath);
+pOrcus->importODS_Styles(aDoc, aPath);
 ScStyleSheetPool* pStyleSheetPool = aDoc.GetStyleSheetPool();
 
 ScStyleSheet* pStyleSheet;
@@ -1022,16 +1019,12 @@ CPPUNIT_TEST_FIXTURE(ScFiltersTest4, testTdf153514)
 CPPUNIT_TEST_FIXTURE(ScFiltersTest4, testOrcusODSStyleInterface)
 {
 ScDocument aDoc;
-OUString aFullUrl = 
m_directories.getURLFromSrc(u"sc/qa/unit/data/xml/styles.xml");
-
-/* This loop below trims file:// from the start because orcus doesn't 
accept such a URL */
-OUString aValidPath;
-osl::FileBase::getSystemPathFromFileURL(aFullUrl, aValidPath);
+OUString aPath = createFilePath(u"xml/styles.xml");
 
 ScOrcusFilters* pOrcus = ScFormatFilter::Get().GetOrcusFilters();
 CPPUNIT_ASSERT(pOrcus);
 
-pOrcus->importODS_Styles(aDoc, aValidPath);
+pOrcus->importODS_Styles(aDoc, aPath);
 ScStyleSheetPool* pStyleSheetPool = aDoc.GetStyleSheetPool();
 
 /* Test cases for Style "Name1"


[Libreoffice-commits] core.git: Branch 'distro/lhm/libreoffice-7-5+backports' - 41 commits - cui/source dbaccess/source download.lst extras/source filter/source framework/source helpcontent2 i18npool/

2023-03-21 Thread Caolán McNamara (via logerrit)
 cui/source/customize/acccfg.cxx |8 
 dbaccess/source/ui/querydesign/JoinTableView.cxx|4 
 download.lst|4 
 extras/source/tipoftheday/expand_formula_bar.svg|1 
 extras/source/tipoftheday/toolbarmode.png   |binary
 extras/source/tipoftheday/toolbarmode.svg   |1 
 extras/source/toolbarmode/default.png   |binary
 extras/source/toolbarmode/default.svg   |1 
 extras/source/toolbarmode/notebookbar.png   |binary
 extras/source/toolbarmode/notebookbar.svg   |1 
 extras/source/toolbarmode/notebookbar_compact.png   |binary
 extras/source/toolbarmode/notebookbar_compact.svg   |1 
 extras/source/toolbarmode/notebookbar_groupedbar_compact.png|binary
 extras/source/toolbarmode/notebookbar_groupedbar_compact.svg|1 
 extras/source/toolbarmode/notebookbar_groupedbar_full.png   |binary
 extras/source/toolbarmode/notebookbar_groupedbar_full.svg   |1 
 extras/source/toolbarmode/notebookbar_groups.png|binary
 extras/source/toolbarmode/notebookbar_groups.svg|1 
 extras/source/toolbarmode/notebookbar_single.png|binary
 extras/source/toolbarmode/notebookbar_single.svg|1 
 extras/source/toolbarmode/sidebar.png   |binary
 extras/source/toolbarmode/sidebar.svg   |  270 
 extras/source/toolbarmode/single.png|binary
 extras/source/toolbarmode/single.svg|1 
 filter/source/pdf/impdialog.cxx |  104 
 filter/source/pdf/impdialog.hxx |8 
 filter/source/pdf/pdfexport.cxx |3 
 filter/source/xslt/odf2xhtml/export/xhtml/body.xsl  |   11 
 filter/source/xslt/odf2xhtml/export/xhtml/header.xsl|   31 
 filter/source/xslt/odf2xhtml/export/xhtml/opendoc2xhtml.xsl |6 
 framework/source/accelerators/keymapping.cxx|1 
 helpcontent2|2 
 i18npool/source/localedata/data/hsb_DE.xml  |   22 
 icon-themes/breeze/cmd/32/ar/jumpdownthislevel.png  |binary
 icon-themes/breeze/cmd/32/ar/jumpupthislevel.png|binary
 icon-themes/breeze/cmd/32/jumpdownthislevel.png |binary
 icon-themes/breeze/cmd/32/jumpupthislevel.png   |binary
 icon-themes/breeze/cmd/ar/lc_jumpdownthislevel.png  |binary
 icon-themes/breeze/cmd/ar/lc_jumpupthislevel.png|binary
 icon-themes/breeze/cmd/ar/sc_jumpdownthislevel.png  |binary
 icon-themes/breeze/cmd/ar/sc_jumpupthislevel.png|binary
 icon-themes/breeze/cmd/lc_jumpdownthislevel.png |binary
 icon-themes/breeze/cmd/lc_jumpupthislevel.png   |binary
 icon-themes/breeze/cmd/sc_jumpdownthislevel.png |binary
 icon-themes/breeze/cmd/sc_jumpupthislevel.png   |binary
 icon-themes/breeze/links.txt|   18 
 icon-themes/breeze_dark/cmd/32/ar/jumpdownthislevel.png |binary
 icon-themes/breeze_dark/cmd/32/ar/jumpupthislevel.png   |binary
 icon-themes/breeze_dark/cmd/32/jumpdownthislevel.png|binary
 icon-themes/breeze_dark/cmd/32/jumpupthislevel.png  |binary
 icon-themes/breeze_dark/cmd/ar/lc_jumpdownthislevel.png |binary
 icon-themes/breeze_dark/cmd/ar/lc_jumpupthislevel.png   |binary
 icon-themes/breeze_dark/cmd/ar/sc_jumpdownthislevel.png |binary
 icon-themes/breeze_dark/cmd/ar/sc_jumpupthislevel.png   |binary
 icon-themes/breeze_dark/cmd/lc_jumpdownthislevel.png|binary
 icon-themes/breeze_dark/cmd/lc_jumpupthislevel.png  |binary
 icon-themes/breeze_dark/cmd/sc_jumpdownthislevel.png|binary
 icon-themes/breeze_dark/cmd/sc_jumpupthislevel.png  |binary
 icon-themes/breeze_dark/links.txt   |   18 
 icon-themes/breeze_dark_svg/cmd/32/ar/jumpdownthislevel.svg |1 
 icon-themes/breeze_dark_svg/cmd/32/ar/jumpupthislevel.svg   |1 
 icon-themes/breeze_dark_svg/cmd/32/jumpdownthislevel.svg|1 
 icon-themes/breeze_dark_svg/cmd/32/jumpupthislevel.svg  |1 
 icon-themes/breeze_dark_svg/cmd/ar/lc_jumpdownthislevel.svg |1 
 icon-themes/breeze_dark_svg/cmd/ar/lc_jumpupthislevel.svg   | 

[Libreoffice-commits] core.git: Branch 'distro/cib/libreoffice-6-4' - configure.ac

2023-03-21 Thread Thorsten Behrens (via logerrit)
 configure.ac |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit d64d7fe3819ca2a771a0c839122374fd9972f649
Author: Thorsten Behrens 
AuthorDate: Tue Mar 21 12:24:05 2023 +0100
Commit: Thorsten Behrens 
CommitDate: Tue Mar 21 12:25:09 2023 +0100

Bump version to 6.4.0.16

Change-Id: Ib19bb43fcae58bf20df1bf7a1f5863f9cd6215aa

diff --git a/configure.ac b/configure.ac
index d2938c0fe5bd..42df61c1050d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -9,7 +9,7 @@ dnl in order to create a configure script.
 # several non-alphanumeric characters, those are split off and used only for 
the
 # ABOUTBOXPRODUCTVERSIONSUFFIX in openoffice.lst. Why that is necessary, no 
idea.
 
-AC_INIT([CIB Office],[6.4.0.15],[],[],[https://www.cib.de/office/])
+AC_INIT([CIB Office],[6.4.0.16],[],[],[https://www.cib.de/office/])
 
 dnl libnumbertext needs autoconf 2.68, but that can pick up autoconf268 just 
fine if it is installed
 dnl whereas aclocal (as run by autogen.sh) insists on using autoconf and fails 
hard


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

2023-03-21 Thread Miklos Vajna (via logerrit)
 sw/qa/extras/ooxmlexport/ooxmlexport9.cxx |   18 +++---
 1 file changed, 15 insertions(+), 3 deletions(-)

New commits:
commit 3911b23c89da02eca92a0069bf1035155b7fddae
Author: Miklos Vajna 
AuthorDate: Tue Mar 21 09:20:14 2023 +0100
Commit: Miklos Vajna 
CommitDate: Tue Mar 21 12:29:15 2023 +

sw floattable, CppunitTest_sw_ooxmlexport9: assert can-split in 
testTdf109063

It should not be a problem if the table is in a frame, what matters is
that the frame is marked to split.

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

diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
index da430214d75f..c821953f1160 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport9.cxx
@@ -123,10 +123,22 @@ DECLARE_SW_ROUNDTRIP_TEST(testBadDocm, "bad.docm", 
nullptr, DocmTest)
 CPPUNIT_ASSERT_EQUAL(OUString("MS Word 2007 XML VBA"), 
pTextDoc->GetDocShell()->GetMedium()->GetFilter()->GetName());
 }
 
-DECLARE_OOXMLEXPORT_TEST(testTdf109063, "tdf109063.docx")
+CPPUNIT_TEST_FIXTURE(Test, testTdf109063)
 {
-// This was 1, near-page-width table was imported as a TextFrame.
-CPPUNIT_ASSERT_EQUAL(0, getShapes());
+SwModelTestBase::FlySplitGuard aGuard;
+auto verify = [this]() {
+// A near-page-width table should be allowed to split:
+uno::Reference xDocument(mxComponent, 
uno::UNO_QUERY);
+uno::Reference 
xFrame(xDocument->getTextFrames()->getByName("Frame1"),
+   uno::UNO_QUERY);
+bool bIsSplitAllowed{};
+xFrame->getPropertyValue("IsSplitAllowed") >>= bIsSplitAllowed;
+CPPUNIT_ASSERT(bIsSplitAllowed);
+};
+createSwDoc("tdf109063.docx");
+verify();
+reload(mpFilter, "tdf109063.docx");
+verify();
 }
 
 CPPUNIT_TEST_FIXTURE(DocmTest, testTdf108269)


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

2023-03-21 Thread Szymon Kłos (via logerrit)
 vcl/inc/jsdialog/jsdialogbuilder.hxx |1 +
 vcl/jsdialog/executor.cxx|7 +--
 vcl/jsdialog/jsdialogbuilder.cxx |5 +
 3 files changed, 11 insertions(+), 2 deletions(-)

New commits:
commit 33f97ddd549b4ebd72b5fea327378667d55796ff
Author: Szymon Kłos 
AuthorDate: Thu Mar 9 12:04:36 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 12:36:31 2023 +

jsdialog: set text for textview on every key press

for that we need to set text without sending back notification
to not disturb user in typing

we need to keep the selection at the same place

Signed-off-by: Szymon Kłos 
Change-Id: Ib5c4a53e61dbc2cf28d450fb44fc5a2957ff57b6
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148533
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149191
Tested-by: Jenkins

diff --git a/vcl/inc/jsdialog/jsdialogbuilder.hxx 
b/vcl/inc/jsdialog/jsdialogbuilder.hxx
index 19458a938c02..0c6cbaa48080 100644
--- a/vcl/inc/jsdialog/jsdialogbuilder.hxx
+++ b/vcl/inc/jsdialog/jsdialogbuilder.hxx
@@ -666,6 +666,7 @@ public:
 JSTextView(JSDialogSender* pSender, ::VclMultiLineEdit* pTextView, 
SalInstanceBuilder* pBuilder,
bool bTakeOwnership);
 virtual void set_text(const OUString& rText) override;
+void set_text_without_notify(const OUString& rText);
 virtual void replace_selection(const OUString& rText) override;
 };
 
diff --git a/vcl/jsdialog/executor.cxx b/vcl/jsdialog/executor.cxx
index 5598afa8ba19..e0d0cbb323f3 100644
--- a/vcl/jsdialog/executor.cxx
+++ b/vcl/jsdialog/executor.cxx
@@ -361,12 +361,15 @@ bool ExecuteAction(const std::string& nWindowId, const 
OString& rWidget, StringM
 }
 }
 
-auto pTextView = dynamic_cast(pWidget);
+auto pTextView = dynamic_cast(pWidget);
 if (pTextView)
 {
 if (sAction == "change")
 {
-pTextView->set_text(rData["data"]);
+int rStartPos, rEndPos;
+pTextView->get_selection_bounds(rStartPos, rEndPos);
+pTextView->set_text_without_notify(rData["data"]);
+pTextView->select_region(rStartPos, rEndPos);
 LOKTrigger::trigger_changed(*pTextView);
 return true;
 }
diff --git a/vcl/jsdialog/jsdialogbuilder.cxx b/vcl/jsdialog/jsdialogbuilder.cxx
index 2d73c26613da..3cfe67c3b56c 100644
--- a/vcl/jsdialog/jsdialogbuilder.cxx
+++ b/vcl/jsdialog/jsdialogbuilder.cxx
@@ -1752,6 +1752,11 @@ void JSTextView::set_text(const OUString& rText)
 sendUpdate();
 }
 
+void JSTextView::set_text_without_notify(const OUString& rText)
+{
+SalInstanceTextView::set_text(rText);
+}
+
 void JSTextView::replace_selection(const OUString& rText)
 {
 SalInstanceTextView::replace_selection(rText);


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

2023-03-21 Thread Szymon Kłos (via logerrit)
 cui/source/dialogs/SpellDialog.cxx |   30 +++---
 cui/source/inc/SpellDialog.hxx |1 +
 vcl/jsdialog/enabled.cxx   |2 ++
 3 files changed, 22 insertions(+), 11 deletions(-)

New commits:
commit ac46f272a4946a8f18a89ba08fa376a3e879b276
Author: Szymon Kłos 
AuthorDate: Tue Mar 21 09:51:31 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 12:37:17 2023 +

jsdialog: enable spelling options dialog

and make it async

Change-Id: Idbf8661aa106d69e60ab6037052fd3d6dec28c06
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149205
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 

diff --git a/cui/source/dialogs/SpellDialog.cxx 
b/cui/source/dialogs/SpellDialog.cxx
index f443aa4f..34564c712ee7 100644
--- a/cui/source/dialogs/SpellDialog.cxx
+++ b/cui/source/dialogs/SpellDialog.cxx
@@ -225,6 +225,12 @@ SpellDialog::SpellDialog(SpellDialogChildWindow* 
pChildWindow,
 
 SpellDialog::~SpellDialog()
 {
+if (m_xOptionsDlg)
+{
+m_xOptionsDlg->response(RET_CANCEL);
+m_xOptionsDlg.reset();
+}
+
 if (m_pInitHdlEvent)
 Application::RemoveUserEvent(m_pInitHdlEvent);
 if (pImpl)
@@ -462,19 +468,21 @@ IMPL_LINK_NOARG(SpellDialog, CheckGrammarHdl, 
weld::Toggleable&, void)
 
 void SpellDialog::StartSpellOptDlg_Impl()
 {
-SfxItemSetFixed aSet( 
SfxGetpApp()->GetPool() );
-SfxSingleTabDialogController aDlg(m_xDialog.get(), &aSet, 
"cui/ui/spelloptionsdialog.ui", "SpellOptionsDialog");
+auto xSet = 
std::make_shared>( 
SfxGetpApp()->GetPool() );
+m_xOptionsDlg = 
std::make_shared(m_xDialog.get(), xSet.get(), 
"cui/ui/spelloptionsdialog.ui", "SpellOptionsDialog");
 
-std::unique_ptr xPage = 
SvxLinguTabPage::Create(aDlg.get_content_area(), &aDlg, &aSet);
+std::unique_ptr xPage = 
SvxLinguTabPage::Create(m_xOptionsDlg->get_content_area(), m_xOptionsDlg.get(), 
xSet.get());
 static_cast(xPage.get())->HideGroups( GROUP_MODULES );
-aDlg.SetTabPage(std::move(xPage));
-if (RET_OK == aDlg.run())
-{
-InitUserDicts();
-const SfxItemSet* pOutSet = aDlg.GetOutputItemSet();
-if(pOutSet)
-OfaTreeOptionsDialog::ApplyLanguageOptions(*pOutSet);
-}
+m_xOptionsDlg->SetTabPage(std::move(xPage));
+weld::GenericDialogController::runAsync(m_xOptionsDlg, [this, xSet] 
(sal_uInt32 nResult) {
+if (RET_OK == nResult)
+{
+InitUserDicts();
+const SfxItemSet* pOutSet = m_xOptionsDlg->GetOutputItemSet();
+if(pOutSet)
+OfaTreeOptionsDialog::ApplyLanguageOptions(*pOutSet);
+}
+});
 }
 
 namespace
diff --git a/cui/source/inc/SpellDialog.hxx b/cui/source/inc/SpellDialog.hxx
index 3c6a7b229ebb..a7e6cd398e2f 100644
--- a/cui/source/inc/SpellDialog.hxx
+++ b/cui/source/inc/SpellDialog.hxx
@@ -165,6 +165,7 @@ private:
 std::unique_ptr m_xClosePB;
 std::unique_ptr m_xToolbar;
 std::unique_ptr m_xSentenceEDWeld;
+std::shared_ptr m_xOptionsDlg;
 
 DECL_LINK(ChangeHdl, weld::Button&, void);
 DECL_LINK(DoubleClickChangeHdl, weld::TreeView&, bool);
diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 4e5c79c1bf9b..935a81ba6e6d 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -44,6 +44,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"cui/ui/imagetabpage.ui"
 || rUIFile == u"cui/ui/macroselectordialog.ui"
 || rUIFile == u"cui/ui/numberingformatpage.ui"
+|| rUIFile == u"cui/ui/optlingupage.ui"
 || rUIFile == u"cui/ui/pageformatpage.ui"
 || rUIFile == u"cui/ui/paragalignpage.ui"
 || rUIFile == u"cui/ui/paraindentspacing.ui"
@@ -56,6 +57,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"cui/ui/similaritysearchdialog.ui"
 || rUIFile == u"cui/ui/specialcharacters.ui"
 || rUIFile == u"cui/ui/spellingdialog.ui"
+|| rUIFile == u"cui/ui/spelloptionsdialog.ui"
 || rUIFile == u"cui/ui/splitcellsdialog.ui"
 || rUIFile == u"cui/ui/textflowpage.ui"
 || rUIFile == u"cui/ui/transparencytabpage.ui"


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 vcl/jsdialog/enabled.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 8c358c1616a16beb10752f0a60bbacccba15d6ea
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 18:50:12 2023 +0530
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 12:39:25 2023 +

jsdialog: enable watermark dialog (writer)

Change-Id: Ie1f5a299fd68bdfd21c04d1ef79514953a2ac5c3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149168
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 935a81ba6e6d..6595cf7971c0 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -148,6 +148,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"modules/swriter/ui/templatedialog8.ui"
 || rUIFile == u"modules/swriter/ui/textgridpage.ui"
 || rUIFile == u"modules/swriter/ui/translationdialog.ui"
+|| rUIFile == u"modules/swriter/ui/watermarkdialog.ui"
 // sfx
 || rUIFile == u"sfx/ui/cmisinfopage.ui"
 || rUIFile == u"sfx/ui/custominfopage.ui"


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 sw/source/ui/frmdlg/column.cxx |2 +-
 sw/uiconfig/swriter/ui/columndialog.ui |   20 +---
 vcl/jsdialog/enabled.cxx   |1 +
 3 files changed, 19 insertions(+), 4 deletions(-)

New commits:
commit f50d009885c499e55db35571ace83f2ed7effbfe
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 18:35:31 2023 +0530
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 12:59:32 2023 +

jsdialog: enable column dialog (writer)

Change-Id: I4b91d271d766d8efeaddc3c3775513184036c5f7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149167
Reviewed-by: Szymon Kłos 
Tested-by: Szymon Kłos 

diff --git a/sw/source/ui/frmdlg/column.cxx b/sw/source/ui/frmdlg/column.cxx
index 5692a906e279..c2a24ad18ef6 100644
--- a/sw/source/ui/frmdlg/column.cxx
+++ b/sw/source/ui/frmdlg/column.cxx
@@ -89,7 +89,7 @@ SwColumnDlg::SwColumnDlg(weld::Window* pParent, SwWrtShell& 
rSh)
 , m_bSectionChanged(false)
 , m_bSelSectionChanged(false)
 , m_bFrameChanged(false)
-, m_xContentArea(m_xDialog->weld_content_area())
+, m_xContentArea(m_xBuilder->weld_container("content"))
 , m_xOkButton(m_xBuilder->weld_button("ok"))
 {
 SwRect aRect;
diff --git a/sw/uiconfig/swriter/ui/columndialog.ui 
b/sw/uiconfig/swriter/ui/columndialog.ui
index 5bf4d5d9d999..2162f1e5b2fd 100644
--- a/sw/uiconfig/swriter/ui/columndialog.ui
+++ b/sw/uiconfig/swriter/ui/columndialog.ui
@@ -17,6 +17,23 @@
 False
 vertical
 12
+
+  
+True
+False
+vertical
+12
+
+  
+
+  
+  
+False
+True
+begin
+0
+  
+
 
   
 False
@@ -74,9 +91,6 @@
 0
   
 
-
-  
-
   
 
 
diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 6595cf7971c0..7efb9de2bb84 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -128,6 +128,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"modules/swriter/ui/captionoptions.ui"
 || rUIFile == u"modules/swriter/ui/characterproperties.ui"
 || rUIFile == u"modules/swriter/ui/charurlpage.ui"
+|| rUIFile == u"modules/swriter/ui/columndialog.ui"
 || rUIFile == u"modules/swriter/ui/columnpage.ui"
 || rUIFile == u"modules/swriter/ui/contentcontroldlg.ui"
 || rUIFile == u"modules/swriter/ui/contentcontrollistitemdlg.ui"


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 vcl/jsdialog/enabled.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 41b7f70981cffb9a04b6b132317915ca51519eab
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 17:47:17 2023 +0530
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 13:02:59 2023 +

jsdialog: enabled insert break(manual break) dialog(writer)

Signed-off-by: Pranam Lashkari 
Change-Id: I65213cd8822e574eff333b96a8718ebff2946f5c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149166
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 7efb9de2bb84..c93e80eb17fd 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -137,6 +137,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"modules/swriter/ui/footnoteareapage.ui"
 || rUIFile == u"modules/swriter/ui/footnotesendnotestabpage.ui"
 || rUIFile == u"modules/swriter/ui/indentpage.ui"
+|| rUIFile == u"modules/swriter/ui/insertbreak.ui"
 || rUIFile == u"modules/swriter/ui/insertcaption.ui"
 || rUIFile == u"modules/swriter/ui/insertsectiondialog.ui"
 || rUIFile == u"modules/swriter/ui/linenumbering.ui"


[Libreoffice-commits] core.git: download.lst

2023-03-21 Thread Michael Stahl (via logerrit)
 download.lst |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit e5005c76bd60a004f6025728e794ba3e4d0dfff1
Author: Michael Stahl 
AuthorDate: Tue Mar 21 10:46:46 2023 +0100
Commit: Michael Stahl 
CommitDate: Tue Mar 21 13:19:29 2023 +

curl: upgrade to release 8.0.1

Apparently 8.0.0 had a serious regression.

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

diff --git a/download.lst b/download.lst
index 08dfd4c0078c..4f2a9169d40f 100644
--- a/download.lst
+++ b/download.lst
@@ -75,8 +75,8 @@ CPPUNIT_TARBALL := cppunit-1.15.1.tar.gz
 # three static lines
 # so that git cherry-pick
 # will not run into conflicts
-CURL_SHA256SUM := 
759690f9a375a720f8bcce9f953897b0d93f31eed9649b74f846d54bbf63bbcc
-CURL_TARBALL := curl-8.0.0.tar.xz
+CURL_SHA256SUM := 
0a381cd82f4d00a9a334438b8ca239afea5bfefcfa9a1025f2bf118e79e0b5f0
+CURL_TARBALL := curl-8.0.1.tar.xz
 # three static lines
 # so that git cherry-pick
 # will not run into conflicts


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 vcl/jsdialog/enabled.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 2e75e849e85da86ea33b447350b0e2f7eaede69e
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 20:36:45 2023 +0530
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 13:22:52 2023 +

jsdialog: enabled thesaurus dialog

Change-Id: I01059e7a4aaffd5c4a2f5ccff55f83ed3db9bf38
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149173
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index c93e80eb17fd..529cffe1dfb7 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -60,6 +60,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"cui/ui/spelloptionsdialog.ui"
 || rUIFile == u"cui/ui/splitcellsdialog.ui"
 || rUIFile == u"cui/ui/textflowpage.ui"
+|| rUIFile == u"cui/ui/thesaurus.ui"
 || rUIFile == u"cui/ui/transparencytabpage.ui"
 || rUIFile == u"cui/ui/twolinespage.ui"
 || rUIFile == u"cui/ui/widgettestdialog.ui"


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

2023-03-21 Thread Miklos Vajna (via logerrit)
 sw/qa/extras/ooxmlimport/ooxmlimport2.cxx |   19 +--
 1 file changed, 17 insertions(+), 2 deletions(-)

New commits:
commit 28b16870553f436b8dd0f74894896136057402a3
Author: Miklos Vajna 
AuthorDate: Tue Mar 21 13:30:49 2023 +0100
Commit: Miklos Vajna 
CommitDate: Tue Mar 21 13:50:20 2023 +

sw floattable, CppunitTest_sw_ooxmlimport2: assert the layout in 
testTdf114217

The point is that the table is multi-page, which is preserved. The
detail that it's not in a fly is no longer true.

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

diff --git a/sw/qa/extras/ooxmlimport/ooxmlimport2.cxx 
b/sw/qa/extras/ooxmlimport/ooxmlimport2.cxx
index f723fe320492..09de11d4c0bb 100644
--- a/sw/qa/extras/ooxmlimport/ooxmlimport2.cxx
+++ b/sw/qa/extras/ooxmlimport/ooxmlimport2.cxx
@@ -37,6 +37,10 @@
 #include 
 #include 
 #include 
+#include 
+#include 
+#include 
+#include 
 
 class Test : public SwModelTestBase
 {
@@ -667,9 +671,20 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf121804)
 
 CPPUNIT_TEST_FIXTURE(Test, testTdf114217)
 {
+SwModelTestBase::FlySplitGuard aGuard;
+// The floating table was not split between page 1 and page 2.
 createSwDoc("tdf114217.docx");
-// This was 1, multi-page table was imported as a floating one.
-CPPUNIT_ASSERT_EQUAL(0, getShapes());
+SwDoc* pDoc = getSwDoc();
+SwRootFrame* pLayout = pDoc->getIDocumentLayoutAccess().GetCurrentLayout();
+auto pPage1 = dynamic_cast(pLayout->Lower());
+CPPUNIT_ASSERT(pPage1);
+const SwSortedObjs& rPage1Objs = *pPage1->GetSortedObjs();
+CPPUNIT_ASSERT_EQUAL(static_cast(1), rPage1Objs.size());
+auto pPage1Fly = dynamic_cast(rPage1Objs[0]);
+CPPUNIT_ASSERT(pPage1Fly);
+auto pTab1 = dynamic_cast(pPage1Fly->GetLower());
+CPPUNIT_ASSERT(pTab1);
+CPPUNIT_ASSERT(pTab1->HasFollow());
 }
 
 CPPUNIT_TEST_FIXTURE(Test, testTdf119200)


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

2023-03-21 Thread Patrick Luby (via logerrit)
 sc/source/ui/view/tabview.cxx |   28 
 1 file changed, 20 insertions(+), 8 deletions(-)

New commits:
commit d82a734c07b85cbd7861699b7fa6d3ebbb3122f2
Author: Patrick Luby 
AuthorDate: Mon Mar 20 16:48:30 2023 -0400
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 14:23:51 2023 +

tdf#152406 Disable anti-jitter code for scroll wheel events

After moving thousands of columns to the right via horizontal
scroll wheel or trackpad swipe events, most vertical scroll
wheel or trackpad swipe events will trigger the anti-jitter code
because nScrollPos and nPrevDragPos will be equal and nDelta
will be overriden and set to zero. So, only use the anti-jitter
code for mouse drag events.

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

diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index 66ddcda2d223..9142393a9152 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -1182,16 +1182,28 @@ void ScTabView::ScrollHdl(ScrollAdaptor* pScroll)
 
 tools::Long nScrollPos = GetScrollBarPos( *pScroll ) + 
nScrollMin;
 nDelta = nScrollPos - nViewPos;
-if ( nScrollPos > nPrevDragPos )
-{
-if (nDelta<0) nDelta=0;
-}
-else if ( nScrollPos < nPrevDragPos )
+
+// tdf#152406 Disable anti-jitter code for scroll wheel events
+// After moving thousands of columns to the right via
+// horizontal scroll wheel or trackpad swipe events, most
+// vertical scroll wheel or trackpad swipe events will trigger
+// the anti-jitter code because nScrollPos and nPrevDragPos
+// will be equal and nDelta will be overriden and set to zero.
+// So, only use the anti-jitter code for mouse drag events.
+if ( eType == ScrollType::Drag )
 {
-if (nDelta>0) nDelta=0;
+if ( nScrollPos > nPrevDragPos )
+{
+if (nDelta<0) nDelta=0;
+}
+else if ( nScrollPos < nPrevDragPos )
+{
+if (nDelta>0) nDelta=0;
+}
+else
+nDelta = 0;
 }
-else
-nDelta = 0;
+
 nPrevDragPos = nScrollPos;
 }
 break;


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

2023-03-21 Thread Noel Grandin (via logerrit)
 chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx   |4 -
 chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx |   12 +--
 chart2/source/controller/chartapiwrapper/WrappedStockProperties.cxx |   12 +--
 chart2/source/controller/dialogs/ChartTypeDialogController.cxx  |4 -
 chart2/source/controller/dialogs/DataBrowserModel.cxx   |4 -
 chart2/source/controller/dialogs/dlg_DataSource.cxx |6 -
 chart2/source/controller/dialogs/tp_ChartType.cxx   |4 -
 chart2/source/controller/sidebar/ChartTypePanel.cxx |   10 +-
 chart2/source/inc/Diagram.hxx   |   18 
 chart2/source/inc/DiagramHelper.hxx |   17 
 chart2/source/model/main/ChartModel_Persistence.cxx |2 
 chart2/source/model/main/Diagram.cxx|   39 
+-
 chart2/source/tools/DiagramHelper.cxx   |   38 
-
 13 files changed, 85 insertions(+), 85 deletions(-)

New commits:
commit cde0f27c84599d0d9894148e510254749f7e2784
Author: Noel Grandin 
AuthorDate: Thu Mar 16 09:38:50 2023 +0200
Commit: Noel Grandin 
CommitDate: Tue Mar 21 14:24:53 2023 +

move getTemplate inside chart2::Diagram

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

diff --git a/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx 
b/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx
index 5ce060e9891f..fbb12b5d4a51 100644
--- a/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx
+++ b/chart2/source/controller/chartapiwrapper/ChartDocumentWrapper.cxx
@@ -1192,8 +1192,8 @@ uno::Reference< uno::XInterface > SAL_CALL 
ChartDocumentWrapper::createInstance(
 rtl::Reference< Diagram > xDiagram = 
ChartModelHelper::findDiagram( xChartDoc );
 ThreeDLookScheme e3DScheme = ThreeDHelper::detectScheme( 
xDiagram );
 rtl::Reference< ::chart::ChartTypeManager > 
xTemplateManager = xChartDoc->getTypeManager();
-DiagramHelper::tTemplateWithServiceName 
aTemplateWithService(
-DiagramHelper::getTemplateForDiagram( xDiagram, 
xTemplateManager ));
+Diagram::tTemplateWithServiceName aTemplateWithService(
+xDiagram->getTemplate( xTemplateManager ));
 if( aTemplateWithService.xChartTypeTemplate.is())
 aTemplateWithService.xChartTypeTemplate->resetStyles2( 
xDiagram );//#i109371#
 xTemplate->changeDiagram( xDiagram );
diff --git a/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx 
b/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx
index 7cc7c57e2862..8ef1429d7565 100644
--- a/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx
+++ b/chart2/source/controller/chartapiwrapper/DiagramWrapper.cxx
@@ -588,8 +588,8 @@ OUString SAL_CALL DiagramWrapper::getDiagramType()
 }
 
 rtl::Reference< ::chart::ChartTypeManager > xChartTypeManager = 
xChartDoc->getTypeManager();
-DiagramHelper::tTemplateWithServiceName aTemplateAndService =
-DiagramHelper::getTemplateForDiagram( xDiagram, xChartTypeManager 
);
+Diagram::tTemplateWithServiceName aTemplateAndService =
+xDiagram->getTemplate( xChartTypeManager );
 
 aRet = lcl_getDiagramType( aTemplateAndService.sServiceName );
 }
@@ -1455,8 +1455,8 @@ bool WrappedNumberOfLinesProperty::detectInnerValue( 
uno::Any& rInnerValue ) con
 if( !aSeriesVector.empty() )
 {
 rtl::Reference< ::chart::ChartTypeManager > xChartTypeManager = 
xChartDoc->getTypeManager();
-DiagramHelper::tTemplateWithServiceName aTemplateAndService =
-DiagramHelper::getTemplateForDiagram( xDiagram, 
xChartTypeManager );
+Diagram::tTemplateWithServiceName aTemplateAndService =
+xDiagram->getTemplate( xChartTypeManager );
 if( aTemplateAndService.sServiceName == 
"com.sun.star.chart2.template.ColumnWithLine" )
 {
 try
@@ -1494,8 +1494,8 @@ void WrappedNumberOfLinesProperty::setPropertyValue( 
const Any& rOuterValue, con
 return;
 
 rtl::Reference< ::chart::ChartTypeManager > xChartTypeManager = 
xChartDoc->getTypeManager();
-DiagramHelper::tTemplateWithServiceName aTemplateAndService =
-DiagramHelper::getTemplateForDiagram( xDiagram, xChartTypeManager 
);
+Diagram::tTemplateWithServiceName aTemplateAndService =
+xDiagram->getTemplate( xChartTypeManager );
 
 rtl::Reference< ChartTypeTemplate > xTemplate;
 if( aTemplateAndService.sServiceName == 
"com.sun.star.chart2.template.Colu

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

2023-03-21 Thread Caolán McNamara (via logerrit)
 vcl/source/window/layout.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f7e5c9bee73c9834312a78adb8d040ed78c6c0ef
Author: Caolán McNamara 
AuthorDate: Tue Mar 21 12:58:54 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 14:32:57 2023 +

typo: then->them

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

diff --git a/vcl/source/window/layout.cxx b/vcl/source/window/layout.cxx
index c77a4a0d30e1..8c102bd6004c 100644
--- a/vcl/source/window/layout.cxx
+++ b/vcl/source/window/layout.cxx
@@ -402,7 +402,7 @@ void VclBox::DumpAsPropertyTree(tools::JsonWriter& 
rJsonWriter)
 sal_uInt16 VclBox::getDefaultAccessibleRole() const
 {
 #if defined(_WIN32)
-//fdo#74284 call Boxes Panels, keep then as "Filler" under
+//fdo#74284 call Boxes Panels, keep them as "Filler" under
 //at least Linux seeing as that's what Gtk does for GtkBoxes
 return css::accessibility::AccessibleRole::PANEL;
 #else


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

2023-03-21 Thread Caolán McNamara (via logerrit)
 vcl/source/window/layout.cxx |   11 ---
 1 file changed, 8 insertions(+), 3 deletions(-)

New commits:
commit 5f5e1a3204b98b9e74bded50ba0eddd4fb371301
Author: Caolán McNamara 
AuthorDate: Tue Mar 21 13:08:17 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 14:57:46 2023 +

gtk4: use PANEL a11y role for VclBox under gtk4

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

diff --git a/vcl/source/window/layout.cxx b/vcl/source/window/layout.cxx
index 8c102bd6004c..6befd3399520 100644
--- a/vcl/source/window/layout.cxx
+++ b/vcl/source/window/layout.cxx
@@ -401,12 +401,17 @@ void VclBox::DumpAsPropertyTree(tools::JsonWriter& 
rJsonWriter)
 
 sal_uInt16 VclBox::getDefaultAccessibleRole() const
 {
+// fdo#74284 call Boxes Panels, keep them as "Filler" under
+// at least Linux seeing as that's what Gtk3 did for GtkBoxes.
+// Though now with Gtk4 that uses GTK_ACCESSIBLE_ROLE_GROUP
+// which maps to ATSPI_ROLE_PANEL
 #if defined(_WIN32)
-//fdo#74284 call Boxes Panels, keep them as "Filler" under
-//at least Linux seeing as that's what Gtk does for GtkBoxes
 return css::accessibility::AccessibleRole::PANEL;
 #else
-return css::accessibility::AccessibleRole::FILLER;
+static sal_uInt16 eRole = Application::GetToolkitName() == "gtk4" ?
+  css::accessibility::AccessibleRole::PANEL :
+  css::accessibility::AccessibleRole::FILLER;
+return eRole;
 #endif
 }
 


[Libreoffice-commits] core.git: Branch 'distro/cib/libreoffice-6-4' - sd/qa xmloff/source

2023-03-21 Thread Mike Kaganski (via logerrit)
 sd/qa/uitest/impress_tests/data/tdf153161_FlushToSave.odp |binary
 sd/qa/uitest/impress_tests/tdf153161.py   |   41 ++
 xmloff/source/draw/shapeexport.cxx|   12 
 3 files changed, 53 insertions(+)

New commits:
commit cd69836ee9e98a4c6ceaa716da8838ddac842054
Author: Mike Kaganski 
AuthorDate: Wed Jan 25 01:37:00 2023 +0300
Commit: Thorsten Behrens 
CommitDate: Tue Mar 21 14:57:49 2023 +

tdf#153161: (Ab)use a call to XTextRange::getString to flush edits

Restore the call (without checking its returned value), removed in commit
d194474aabd699806cb3631bc8641dd0548b8026 ("tdf#151100: xText->getString()
may be empty for content needing export", 2022-09-22), side effects of
which obviously allow some object's changes to get flushed and saved.

Change-Id: I62f27cd056c32ad76f79a4862e2f4a0964eaadef
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/146106
Tested-by: Mike Kaganski 
Reviewed-by: Mike Kaganski 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149142
Tested-by: Thorsten Behrens 
Reviewed-by: Thorsten Behrens 

diff --git a/sd/qa/uitest/impress_tests/data/tdf153161_FlushToSave.odp 
b/sd/qa/uitest/impress_tests/data/tdf153161_FlushToSave.odp
new file mode 100644
index ..1fd5c20c2a52
Binary files /dev/null and 
b/sd/qa/uitest/impress_tests/data/tdf153161_FlushToSave.odp differ
diff --git a/sd/qa/uitest/impress_tests/tdf153161.py 
b/sd/qa/uitest/impress_tests/tdf153161.py
new file mode 100644
index ..23ffdef879d1
--- /dev/null
+++ b/sd/qa/uitest/impress_tests/tdf153161.py
@@ -0,0 +1,41 @@
+# -*- tab-width: 4; indent-tabs-mode: nil; py-indent-offset: 4 -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+from libreoffice.uno.propertyvalue import mkPropertyValues
+from uitest.framework import UITestCase
+import org.libreoffice.unotest
+import pathlib
+
+def get_url_for_data_file(file_name):
+return 
pathlib.Path(org.libreoffice.unotest.makeCopyFromTDOC(file_name)).as_uri()
+
+class TestTdf153161(UITestCase):
+
+def testTdf153161(self):
+
+document = 
self.ui_test.load_file(get_url_for_data_file('tdf153161_FlushToSave.odp'))
+oldText = document.DrawPages[0].getByIndex(1).String
+self.assertTrue(oldText.startswith('在没有版本控制系统的时期'))
+
+xImpressDoc = self.xUITest.getTopFocusWindow()
+xEditWin = xImpressDoc.getChild('impress_win')
+xEditWin.executeAction('SELECT', mkPropertyValues({'OBJECT':'Unnamed 
Drawinglayer object 1'}))
+
+# Type something, getting into text editing mode (appending) 
automatically
+xEditWin.executeAction('TYPE', mkPropertyValues({'TEXT': 'Foo Bar'}))
+xToolkit = 
self.xContext.ServiceManager.createInstance('com.sun.star.awt.Toolkit')
+xToolkit.processEventsToIdle()
+self.xUITest.executeCommand('.uno:Save')
+self.xUITest.executeCommand('.uno:Reload')
+
+# Reload and check that the edit was saved
+document = self.ui_test.get_component()
+self.assertEqual(oldText + 'Foo Bar', 
document.DrawPages[0].getByIndex(1).String)
+
+# vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/xmloff/source/draw/shapeexport.cxx 
b/xmloff/source/draw/shapeexport.cxx
index 61193072f84e..887c49e61a49 100644
--- a/xmloff/source/draw/shapeexport.cxx
+++ b/xmloff/source/draw/shapeexport.cxx
@@ -304,6 +304,18 @@ void XMLShapeExport::collectShapeAutoStyles(const 
uno::Reference< drawing::XShap
 uno::Reference< text::XText > xText(xShape, uno::UNO_QUERY);
 if (xText.is())
 {
+try
+{
+// tdf#153161: it seems that the call to XTextRange::getString 
flushes the changes
+// for some objects, that otherwise fail to get exported 
correctly. Maybe at some
+// point it would make sense to find a better place for more 
targeted flush.
+xText->getString();
+}
+catch (uno::RuntimeException const&)
+{
+// E.g., SwXTextFrame that contains only a table will throw; 
this is not an error
+}
+
 uno::Reference< beans::XPropertySetInfo > xPropSetInfo( 
xPropSet->getPropertySetInfo() );
 
 if( xPropSetInfo.is() && 
xPropSetInfo->hasPropertyByName("IsEmptyPresentationObject") )


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

2023-03-21 Thread Szymon Kłos (via logerrit)
 include/vcl/jsdialog/executor.hxx|   10 ++
 include/vcl/weld.hxx |2 +
 vcl/inc/jsdialog/jsdialogbuilder.hxx |   27 
 vcl/jsdialog/executor.cxx|   21 
 vcl/jsdialog/jsdialogbuilder.cxx |   57 +++
 5 files changed, 117 insertions(+)

New commits:
commit b46defbc331c3c91553bc8967cdf48eacea87afd
Author: Szymon Kłos 
AuthorDate: Fri Mar 10 13:53:04 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 15:00:32 2023 +

jsdialog: update Entry on replace_selection

Signed-off-by: Szymon Kłos 
Change-Id: Ie0c3586fbf4bda1a9d68dceec61e08ff9f75bb97
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148634
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149201
Tested-by: Jenkins

diff --git a/vcl/inc/jsdialog/jsdialogbuilder.hxx 
b/vcl/inc/jsdialog/jsdialogbuilder.hxx
index 783aeaacfff7..6f59b7d5ca64 100644
--- a/vcl/inc/jsdialog/jsdialogbuilder.hxx
+++ b/vcl/inc/jsdialog/jsdialogbuilder.hxx
@@ -572,6 +572,7 @@ public:
 bool bTakeOwnership);
 virtual void set_text(const OUString& rText) override;
 void set_text_without_notify(const OUString& rText);
+virtual void replace_selection(const OUString& rText) override;
 };
 
 class JSListBox final : public JSWidget
diff --git a/vcl/jsdialog/jsdialogbuilder.cxx b/vcl/jsdialog/jsdialogbuilder.cxx
index d623e9723a7e..9a0f61d90caa 100644
--- a/vcl/jsdialog/jsdialogbuilder.cxx
+++ b/vcl/jsdialog/jsdialogbuilder.cxx
@@ -1488,6 +1488,12 @@ void JSEntry::set_text(const OUString& rText)
 
 void JSEntry::set_text_without_notify(const OUString& rText) { 
SalInstanceEntry::set_text(rText); }
 
+void JSEntry::replace_selection(const OUString& rText)
+{
+SalInstanceEntry::replace_selection(rText);
+sendUpdate();
+}
+
 JSListBox::JSListBox(JSDialogSender* pSender, ::ListBox* pListBox, 
SalInstanceBuilder* pBuilder,
  bool bTakeOwnership)
 : JSWidget(pSender, pListBox, 
pBuilder,
commit 74e8476d23b72f06328dcbe183597847471771e1
Author: Szymon Kłos 
AuthorDate: Fri Mar 10 10:03:49 2023 +0100
Commit: Szymon Kłos 
CommitDate: Tue Mar 21 15:00:25 2023 +

jsdialog: add wrapper for ScrolledWindow

Change-Id: I59489e4cf1e5fb492272108e3a19afb4212753f5
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/148610
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149200

diff --git a/include/vcl/jsdialog/executor.hxx 
b/include/vcl/jsdialog/executor.hxx
index 9bccad3f1865..50f71272bb61 100644
--- a/include/vcl/jsdialog/executor.hxx
+++ b/include/vcl/jsdialog/executor.hxx
@@ -27,6 +27,16 @@ public:
 
 static void trigger_changed(weld::IconView& rIconView) { 
rIconView.signal_selection_changed(); }
 
+static void trigger_scrollv(weld::ScrolledWindow& rScrolledWindow)
+{
+rScrolledWindow.signal_vadjustment_changed();
+}
+
+static void trigger_scrollh(weld::ScrolledWindow& rScrolledWindow)
+{
+rScrolledWindow.signal_hadjustment_changed();
+}
+
 static void trigger_toggled(weld::Toggleable& rButton) { 
rButton.signal_toggled(); }
 
 static void trigger_row_activated(weld::TreeView& rTreeView)
diff --git a/include/vcl/weld.hxx b/include/vcl/weld.hxx
index 4acece25f052..d22fa5eb616e 100644
--- a/include/vcl/weld.hxx
+++ b/include/vcl/weld.hxx
@@ -404,6 +404,8 @@ public:
 
 class VCL_DLLPUBLIC ScrolledWindow : virtual public Container
 {
+friend class ::LOKTrigger;
+
 Link m_aVChangeHdl;
 Link m_aHChangeHdl;
 
diff --git a/vcl/inc/jsdialog/jsdialogbuilder.hxx 
b/vcl/inc/jsdialog/jsdialogbuilder.hxx
index 0c6cbaa48080..783aeaacfff7 100644
--- a/vcl/inc/jsdialog/jsdialogbuilder.hxx
+++ b/vcl/inc/jsdialog/jsdialogbuilder.hxx
@@ -40,6 +40,7 @@ class ComboBox;
 class VclMultiLineEdit;
 class SvTabListBox;
 class IconView;
+class VclScrolledWindow;
 
 namespace vcl
 {
@@ -304,6 +305,8 @@ public:
 virtual std::unique_ptr weld_tree_view(const OString& id) 
override;
 virtual std::unique_ptr weld_expander(const OString& id) 
override;
 virtual std::unique_ptr weld_icon_view(const OString& id) 
override;
+virtual std::unique_ptr
+weld_scrolled_window(const OString& id, bool bUserManagedScrolling = 
false) override;
 virtual std::unique_ptr weld_radio_button(const 
OString& id) override;
 virtual std::unique_ptr weld_frame(const OString& id) 
override;
 virtual std::unique_ptr weld_menu_button(const OString& 
id) override;
@@ -368,6 +371,14 @@ public:
 {
 }
 
+JSWidget(JSDialogSender* pSender, VclClass* pObject, SalInstanceBuilder* 
pBuilder,
+ bool bTakeOwnership, bool bUserManagedScrolling)
+: BaseInstanceClass(pObject, pBuilder, bTakeOwnership, 
bUserManagedScrolling)
+, m_bIsFreezed(false)
+, m_pSender(pSender)
+{
+}
+
 JSWidget(JSDialogSender* pSender, VclClass* pObje

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

2023-03-21 Thread Caolán McNamara (via logerrit)
 sc/source/filter/excel/xiescher.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit 5ccbdcf182851375486386c0df8bb3f04cf2f9cd
Author: Caolán McNamara 
AuthorDate: Tue Mar 21 13:29:23 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 15:42:28 2023 +

crashreporting: CreateEmbeddedObject might fail

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

diff --git a/sc/source/filter/excel/xiescher.cxx 
b/sc/source/filter/excel/xiescher.cxx
index 9647ca5565ba..718f464e0c9c 100644
--- a/sc/source/filter/excel/xiescher.cxx
+++ b/sc/source/filter/excel/xiescher.cxx
@@ -1763,6 +1763,9 @@ rtl::Reference XclImpChartObj::DoCreateSdrObj( 
XclImpDffConverter& rD
 Reference< XEmbeddedObject > xEmbObj = 
pDocShell->GetEmbeddedObjectContainer().
 CreateEmbeddedObject( SvGlobalName( SO3_SCH_CLASSID 
).GetByteSequence(), aEmbObjName, &sBaseURL );
 
+if (!xEmbObj)
+return xSdrObj;
+
 /*  Set the size to the embedded object, this prevents that font sizes
 of text objects are changed in the chart when the object is
 inserted into the draw page. */


[Libreoffice-commits] core.git: basegfx/source drawinglayer/inc drawinglayer/source include/basegfx svx/source

2023-03-21 Thread Armin Le Grand (allotropia) (via logerrit)
 basegfx/source/tools/gradienttools.cxx  |   76 +++
 drawinglayer/inc/texture/texture.hxx|3 
 drawinglayer/source/processor2d/vclpixelprocessor2d.cxx |   14 +
 drawinglayer/source/texture/texture.cxx |  173 +---
 include/basegfx/utils/gradienttools.hxx |   12 -
 svx/source/sdr/primitive2d/sdrattributecreator.cxx  |   75 ++
 6 files changed, 269 insertions(+), 84 deletions(-)

New commits:
commit 808c5bf66370f78f36e98887db0848ee7e55bb3a
Author: Armin Le Grand (allotropia) 
AuthorDate: Tue Mar 21 13:13:15 2023 +0100
Commit: Armin Le Grand 
CommitDate: Tue Mar 21 15:53:06 2023 +

MCGR: Model data changes for ColorSteps (II)

The biggest change here is to allow multiple ColorStops
with the same Offset. That allows to define gradients
with 'hard' color changes at any place using two
ColorStops with the same Offset and different Colors.
This required quite some adaptions, but works well.

Also removed in this context checking for all Colors
being the same to not mix up things. Also works well.

Also changed the need for having Start/EndColors AKA
ColorStops for 0.0 and 1.0 in place, instead 'imply'
the 1st ColorStop to also define the StartColor and
the last one the EndColor. This allows e.g. Gradient
definitions with two GradientStops at the same
Offset e.g. 0.5 with different colors to already
define a full Gradient.

Also added a tooling method to reverse ColorSteps,
which changes the order and mirrors the Offsets
(what even keeps an existing sort valid).
This is useful e.g. for GradientAxial which is the
only one where for decomposition the Gradient had
to be interpreted 'reverse' since it's defined
from center to edge, but for creating correct filled
polygons to represent this the inverse order had to
be used, creating polygons from edge to center.
This led to 'wild' code for this one of six cases
and prevented unifications with the other cases
(also made your brain flip).
Thus I adapted this now to use the reversed
ColorSteps consequently, and the same principle
loops than the other implementations to make things
easier for the future and to use common tooling.

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

diff --git a/basegfx/source/tools/gradienttools.cxx 
b/basegfx/source/tools/gradienttools.cxx
index d43f7899121a..6055143d5abd 100644
--- a/basegfx/source/tools/gradienttools.cxx
+++ b/basegfx/source/tools/gradienttools.cxx
@@ -264,6 +264,17 @@ namespace basegfx
 
 namespace utils
 {
+/* Tooling method to reverse ColorStops, including offsets.
+   When also mirroring offsets a valid sort keeps valid.
+*/
+void reverseColorStops(ColorStops& rColorStops)
+{
+// can use std::reverse, but also need to adapt offset(s)
+std::reverse(rColorStops.begin(), rColorStops.end());
+for (auto& candidate : rColorStops)
+candidate = ColorStop(1.0 - candidate.getStopOffset(), 
candidate.getStopColor());
+}
+
 /* Tooling method to convert UNO API data to ColorStops
This will try to extract ColorStop data from the given
Any, so if it's of type awt::Gradient2 that data will be
@@ -374,13 +385,8 @@ namespace basegfx
  be removed)
- contains no ColorStops with offset > 1.0 (will
  be removed)
-   - contains no two ColorStops with identical offsets
- (will be removed, 1st one/smallest offset wins
- which is also given by sort tooling)
+   - ColorStops with identical offsets are now allowed
- will be sorted from lowest offset to highest
-   - if all colors are the same, the content will
- be reduced to a single entry with offset 0.0
- (force to StartColor)
 
Some more notes:
- It can happen that the result is empty
@@ -404,8 +410,15 @@ namespace basegfx
 if (1 == rColorStops.size())
 {
 // no gradient at all, but preserve given color
-// and force it to be the StartColor
-rColorStops[0] = ColorStop(0.0, rColorStops[0].getStopColor());
+// evtl. correct offset to be in valid range [0.0 .. 1.0]
+// NOTE: This does not move it to 0.0 or 1.0, it *can* still
+//   be somewhere in-between what is allowed
+rColorStops[0] = ColorStop(
+std::max(0.0, std::min(1.0, 
rColorStops[0].getStopOffset())),
+rColorStops[0].getStopColor());
+
+// done
+return;
 }
 
  

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

2023-03-21 Thread Caolán McNamara (via logerrit)
 sc/source/filter/excel/xiescher.cxx |3 +++
 1 file changed, 3 insertions(+)

New commits:
commit fa1760d0c7ac8b9fb2e88f7d6d7f9de14479cbf6
Author: Caolán McNamara 
AuthorDate: Tue Mar 21 13:29:23 2023 +
Commit: Xisco Fauli 
CommitDate: Tue Mar 21 15:55:23 2023 +

crashreporting: CreateEmbeddedObject might fail

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

diff --git a/sc/source/filter/excel/xiescher.cxx 
b/sc/source/filter/excel/xiescher.cxx
index 7940758bffc5..2838347b3f14 100644
--- a/sc/source/filter/excel/xiescher.cxx
+++ b/sc/source/filter/excel/xiescher.cxx
@@ -1763,6 +1763,9 @@ rtl::Reference XclImpChartObj::DoCreateSdrObj( 
XclImpDffConverter& rD
 Reference< XEmbeddedObject > xEmbObj = 
pDocShell->GetEmbeddedObjectContainer().
 CreateEmbeddedObject( SvGlobalName( SO3_SCH_CLASSID 
).GetByteSequence(), aEmbObjName, &sBaseURL );
 
+if (!xEmbObj)
+return xSdrObj;
+
 /*  Set the size to the embedded object, this prevents that font sizes
 of text objects are changed in the chart when the object is
 inserted into the draw page. */


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

2023-03-21 Thread László Németh (via logerrit)
 sw/qa/extras/ooxmlexport/data/tdf153255.docx  |binary
 sw/qa/extras/ooxmlexport/ooxmlexport14.cxx|   35 ++
 writerfilter/source/dmapper/DomainMapper_Impl.cxx |   14 ++--
 3 files changed, 46 insertions(+), 3 deletions(-)

New commits:
commit ed0372bac123b402fe3cd694a455e8328117752d
Author: László Németh 
AuthorDate: Mon Mar 20 17:27:32 2023 +0100
Commit: László Németh 
CommitDate: Tue Mar 21 15:57:27 2023 +

tdf#153255 DOCX import: fix footnote order

Simple unit test for tdf#152203 allowed to create
bad ordering algorithm by accident without noticing
the problem, which caused problem for most of the DOCX
files, where serialization in footnotes.xml doesn't follow
the order of the footnotes.

Regression from commit 9b39ce0e66acfe812e1d50e530dc2ccdef3e1357
"tdf#76260 DOCX import: fix slow footnote import".

Follow-up to commit 09ae3c01940bbc25ffde51963683b04e3cb4bb6a
"tdf#152203 DOCX import: fix mixed footnotes/endnotes".

Change-Id: I0628d96e3b8ad4df38d26e9288b47d39b958c441
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149176
Tested-by: Jenkins
Reviewed-by: László Németh 

diff --git a/sw/qa/extras/ooxmlexport/data/tdf153255.docx 
b/sw/qa/extras/ooxmlexport/data/tdf153255.docx
new file mode 100644
index ..e9f2892d5670
Binary files /dev/null and b/sw/qa/extras/ooxmlexport/data/tdf153255.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlexport14.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlexport14.cxx
index ffb48ecad109..0fd98a48f759 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlexport14.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlexport14.cxx
@@ -1317,6 +1317,41 @@ CPPUNIT_TEST_FIXTURE(Test, testTdf152506)
 CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg5"), 
xLastButOne->getString().trim() );
 }
 
+CPPUNIT_TEST_FIXTURE(Test, testTdf153255)
+{
+loadAndSave("tdf153255.docx");
+xmlDocUniquePtr pXml = parseExport("word/footnotes.xml");
+CPPUNIT_ASSERT(pXml);
+
+uno::Reference xFootnotesSupplier(mxComponent, 
uno::UNO_QUERY);
+uno::Reference xFootnotes = 
xFootnotesSupplier->getFootnotes();
+uno::Reference xLastFootnote(xFootnotes->getByIndex(5), 
uno::UNO_QUERY);
+// This was "Footnote for pg2" (replaced footnotes)
+CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg 6"), 
xLastFootnote->getString().trim() );
+
+uno::Reference xLastButOne(xFootnotes->getByIndex(4), 
uno::UNO_QUERY);
+// This was "Footnote for pg 6" (replaced footnotes)
+CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg5"), 
xLastButOne->getString().trim() );
+
+// check all the remaining footnotes
+
+uno::Reference xFootnote1(xFootnotes->getByIndex(0), 
uno::UNO_QUERY);
+// This was "Footnote for pg3" (replaced footnotes)
+CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg1"), 
xFootnote1->getString().trim() );
+
+uno::Reference xFootnote2(xFootnotes->getByIndex(1), 
uno::UNO_QUERY);
+// This was "Footnote for pg5" (replaced footnotes)
+CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg2"), 
xFootnote2->getString().trim() );
+
+uno::Reference xFootnote3(xFootnotes->getByIndex(2), 
uno::UNO_QUERY);
+// This was "Footnote for pg4." (replaced footnotes)
+CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg3"), 
xFootnote3->getString().trim() );
+
+uno::Reference xFootnote4(xFootnotes->getByIndex(3), 
uno::UNO_QUERY);
+// This was "Footnote for pg1" (replaced footnotes)
+CPPUNIT_ASSERT_EQUAL( OUString("Footnote for pg4."), 
xFootnote4->getString().trim() );
+}
+
 // skip test for macOS (missing fonts?)
 #if !defined(MACOSX)
 DECLARE_OOXMLEXPORT_TEST(testTdf146346, "tdf146346.docx")
diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx 
b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index 6f0c1dd0952f..ffcd883289bd 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -3755,15 +3755,23 @@ void DomainMapper_Impl::RemoveTemporaryFootOrEndnotes()
 
 static void lcl_convertToNoteIndices(std::deque& rNoteIds, 
sal_Int32& rFirstNoteIndex)
 {
-// convert arbitrary footnote identifiers to 0, 1, 2...
-// indices, keeping their possible random order
+// rNoteIds contains XML footnote identifiers in the loaded order of the 
footnotes
+// (the same order as in footnotes.xml), i.e. it maps temporary footnote 
positions to the
+// identifiers. For example: Ids[0] = 100; Ids[1] = -1, Ids[2] = 5.
+// To copy the footnotes in their final place, create an array, which map 
the (normalized)
+// footnote identifiers to the temporary footnote positions. Using the 
previous example,
+// Pos[0] = 1; Pos[1] = 2; Pos[2] = 0 (where [0], [1], [2] are the 
normalized
+// -1, 5 and 100 identifiers).
 std::deque aSortedIds = rNoteIds;
 std::sort(aSortedIds.begin(), aSortedIds.end());
 std::map aMapIds;
+// normalize footnote identifiers 

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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 sw/source/ui/frmdlg/column.cxx |2 +-
 sw/uiconfig/swriter/ui/columndialog.ui |   20 +---
 vcl/jsdialog/enabled.cxx   |1 +
 3 files changed, 19 insertions(+), 4 deletions(-)

New commits:
commit 6c4464a14ce324100df4cf3934c3ddeeac82ab08
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 18:35:31 2023 +0530
Commit: Pranam Lashkari 
CommitDate: Tue Mar 21 17:03:01 2023 +

jsdialog: enable column dialog (writer)

Change-Id: I4b91d271d766d8efeaddc3c3775513184036c5f7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149167
Reviewed-by: Szymon Kłos 
Tested-by: Szymon Kłos 
(cherry picked from commit f50d009885c499e55db35571ace83f2ed7effbfe)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149235
Tested-by: Jenkins
Reviewed-by: Pranam Lashkari 

diff --git a/sw/source/ui/frmdlg/column.cxx b/sw/source/ui/frmdlg/column.cxx
index 8a82dd04d95c..338e8011ad20 100644
--- a/sw/source/ui/frmdlg/column.cxx
+++ b/sw/source/ui/frmdlg/column.cxx
@@ -89,7 +89,7 @@ SwColumnDlg::SwColumnDlg(weld::Window* pParent, SwWrtShell& 
rSh)
 , m_bSectionChanged(false)
 , m_bSelSectionChanged(false)
 , m_bFrameChanged(false)
-, m_xContentArea(m_xDialog->weld_content_area())
+, m_xContentArea(m_xBuilder->weld_container("content"))
 , m_xOkButton(m_xBuilder->weld_button("ok"))
 {
 SwRect aRect;
diff --git a/sw/uiconfig/swriter/ui/columndialog.ui 
b/sw/uiconfig/swriter/ui/columndialog.ui
index 5bf4d5d9d999..2162f1e5b2fd 100644
--- a/sw/uiconfig/swriter/ui/columndialog.ui
+++ b/sw/uiconfig/swriter/ui/columndialog.ui
@@ -17,6 +17,23 @@
 False
 vertical
 12
+
+  
+True
+False
+vertical
+12
+
+  
+
+  
+  
+False
+True
+begin
+0
+  
+
 
   
 False
@@ -74,9 +91,6 @@
 0
   
 
-
-  
-
   
 
 
diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 9a3699301de1..fdb587c8b1e1 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -102,6 +102,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"modules/swriter/ui/captionoptions.ui"
 || rUIFile == u"modules/swriter/ui/characterproperties.ui"
 || rUIFile == u"modules/swriter/ui/charurlpage.ui"
+|| rUIFile == u"modules/swriter/ui/columndialog.ui"
 || rUIFile == u"modules/swriter/ui/columnpage.ui"
 || rUIFile == u"modules/swriter/ui/contentcontroldlg.ui"
 || rUIFile == u"modules/swriter/ui/contentcontrollistitemdlg.ui"


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 vcl/jsdialog/enabled.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit 11fc8254548d241a972a5f1483609497d8d93ea2
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 20:36:45 2023 +0530
Commit: Pranam Lashkari 
CommitDate: Tue Mar 21 17:03:16 2023 +

jsdialog: enabled thesaurus dialog

Change-Id: I01059e7a4aaffd5c4a2f5ccff55f83ed3db9bf38
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149173
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
(cherry picked from commit 2e75e849e85da86ea33b447350b0e2f7eaede69e)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149237
Tested-by: Jenkins
Reviewed-by: Pranam Lashkari 

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index fdb587c8b1e1..a99417b50a27 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -59,6 +59,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"cui/ui/similaritysearchdialog.ui"
 || rUIFile == u"cui/ui/splitcellsdialog.ui"
 || rUIFile == u"cui/ui/textflowpage.ui"
+|| rUIFile == u"cui/ui/thesaurus.ui"
 || rUIFile == u"cui/ui/transparencytabpage.ui"
 || rUIFile == u"cui/ui/twolinespage.ui"
 || rUIFile == u"cui/ui/widgettestdialog.ui"


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

2023-03-21 Thread Noel Grandin (via logerrit)
 chart2/source/controller/dialogs/DataBrowser.cxx|   14 -
 chart2/source/controller/dialogs/DialogModel.cxx|2 
 chart2/source/controller/dialogs/ObjectNameProvider.cxx |4 
 chart2/source/controller/dialogs/tp_DataSource.cxx  |3 
 chart2/source/controller/main/ChartController_Tools.cxx |2 
 chart2/source/controller/sidebar/ChartSeriesPanel.cxx   |2 
 chart2/source/inc/DataSeries.hxx|8 
 chart2/source/inc/DataSeriesHelper.hxx  |   10 -
 chart2/source/inc/Diagram.hxx   |4 
 chart2/source/inc/RegressionCurveHelper.hxx |5 
 chart2/source/model/main/DataSeries.cxx |  132 +++
 chart2/source/model/main/Diagram.cxx|   19 ++
 chart2/source/tools/DataSeriesHelper.cxx|  140 
 chart2/source/tools/RegressionCurveHelper.cxx   |   18 --
 chart2/source/view/charttypes/VSeriesPlotter.cxx|   10 -
 15 files changed, 180 insertions(+), 193 deletions(-)

New commits:
commit 0f691fe670526c1c0ba5c9bab8dfe8209bfe698e
Author: Noel Grandin 
AuthorDate: Thu Mar 16 11:04:21 2023 +0200
Commit: Noel Grandin 
CommitDate: Tue Mar 21 17:25:46 2023 +

move getLabel inside chart2::DataSeries

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

diff --git a/chart2/source/controller/dialogs/DataBrowser.cxx 
b/chart2/source/controller/dialogs/DataBrowser.cxx
index ce7687162eb1..ab82a772ad02 100644
--- a/chart2/source/controller/dialogs/DataBrowser.cxx
+++ b/chart2/source/controller/dialogs/DataBrowser.cxx
@@ -651,11 +651,10 @@ void DataBrowser::RenewTable()
 spHeader->SetColor( nColor );
 spHeader->SetChartType( elemHeader.m_xChartType, 
elemHeader.m_bSwapXAndYAxis );
 spHeader->SetSeriesName(
-DataSeriesHelper::getDataSeriesLabel(
-elemHeader.m_xDataSeries,
-(elemHeader.m_xChartType.is() ?
+elemHeader.m_xDataSeries->getLabelForRole(
+elemHeader.m_xChartType.is() ?
  
elemHeader.m_xChartType->getRoleOfSequenceForSeriesLabel() :
- OUString("values-y";
+ OUString("values-y")));
 // index is 1-based, as 0 is for the column that contains the 
row-numbers
 spHeader->SetRange( elemHeader.m_nStartColumn + 1, 
elemHeader.m_nEndColumn + 1 );
 spHeader->SetGetFocusHdl( aFocusLink );
@@ -1276,11 +1275,10 @@ void DataBrowser::RenewSeriesHeaders()
 spHeader->SetColor( nColor );
 spHeader->SetChartType( elemHeader.m_xChartType, 
elemHeader.m_bSwapXAndYAxis );
 spHeader->SetSeriesName(
-DataSeriesHelper::getDataSeriesLabel(
-elemHeader.m_xDataSeries,
-(elemHeader.m_xChartType.is() ?
+elemHeader.m_xDataSeries->getLabelForRole(
+elemHeader.m_xChartType.is() ?
  
elemHeader.m_xChartType->getRoleOfSequenceForSeriesLabel() :
- OUString( "values-y";
+ OUString( "values-y")));
 spHeader->SetRange( elemHeader.m_nStartColumn + 1, 
elemHeader.m_nEndColumn + 1 );
 spHeader->SetGetFocusHdl( aFocusLink );
 spHeader->SetEditChangedHdl( aSeriesHeaderChangedLink );
diff --git a/chart2/source/controller/dialogs/DialogModel.cxx 
b/chart2/source/controller/dialogs/DialogModel.cxx
index f64e328f1d28..334fc0b1b0e2 100644
--- a/chart2/source/controller/dialogs/DialogModel.cxx
+++ b/chart2/source/controller/dialogs/DialogModel.cxx
@@ -434,7 +434,7 @@ std::vector< DialogModel::tSeriesWithChartTypeByName >
 {
 aResult.push_back(
 ::chart::DialogModel::tSeriesWithChartTypeByName(
-::chart::DataSeriesHelper::getDataSeriesLabel( 
dataSeries, aRole ),
+dataSeries->getLabelForRole( aRole ),
 std::make_pair( dataSeries, rxChartType )));
 }
 }
diff --git a/chart2/source/controller/dialogs/ObjectNameProvider.cxx 
b/chart2/source/controller/dialogs/ObjectNameProvider.cxx
index b1a023136706..2a2b2dcce89a 100644
--- a/chart2/source/controller/dialogs/ObjectNameProvider.cxx
+++ b/chart2/source/controller/dialogs/ObjectNameProvider.cxx
@@ -73,8 +73,8 @@ OUString lcl_getDataSeriesName( std::u16string_view 
rObjectCID, const rtl::Refer
 rtl::Reference< ChartType > xChartType( 
xDiagram->getChartTypeOfSeries( xSeries ) );
 if( xChartType.is() )
 {
-aRet = ::chart::DataSeriesHelper::getDataSeriesLabel(
-xSeries, xChartType->getRoleOfSequenceForSeriesLabel() ) ;
+aRet = xSeries->getLabelForRole(
+

[Libreoffice-commits] core.git: Branch 'distro/collabora/co-22.05' - bin/lo-all-static-libs solenv/bin

2023-03-21 Thread Patrick Luby (via logerrit)
 bin/lo-all-static-libs|1 +
 solenv/bin/native-code.py |1 +
 2 files changed, 2 insertions(+)

New commits:
commit 260be930139e2760882d20c9ea3c2b2db796cdf6
Author: Patrick Luby 
AuthorDate: Tue Mar 21 08:38:21 2023 -0400
Commit: Andras Timar 
CommitDate: Tue Mar 21 19:54:34 2023 +

Partial fix: iOS missing text direction listbox in Format > Page Style > 
Page

Add the static libraries that are required by the LibreOffice's
CTL and CJK text layout features to the iOS app add link list.

Reference-to: https://github.com/CollaboraOnline/online/issues/6050
Change-Id: I21fddbfd29a1d326b509840127bd136c327cd3d2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149210
Reviewed-by: Patrick Luby 
Reviewed-by: Andras Timar 
Tested-by: Andras Timar 

diff --git a/bin/lo-all-static-libs b/bin/lo-all-static-libs
index af4ff25f348b..839e21169e61 100755
--- a/bin/lo-all-static-libs
+++ b/bin/lo-all-static-libs
@@ -119,6 +119,7 @@ echo $INSTDIR/$LIBO_LIB_FOLDER/lib*.a \
  $WORKDIR/UnpackedTarball/libexttextcat/src/.libs/*.a \
  $WORKDIR/UnpackedTarball/libmspub/src/lib/.libs/*.a \
  $WORKDIR/UnpackedTarball/libmwaw/src/lib/.libs/*.a \
+ $WORKDIR/UnpackedTarball/libnumbertext/src/.libs/*.a \
  $WORKDIR/UnpackedTarball/libodfgen/src/.libs/*.a \
  $WORKDIR/UnpackedTarball/liborcus/src/*/.libs/*.a \
  $WORKDIR/UnpackedTarball/librevenge/src/*/.libs/*.a \
diff --git a/solenv/bin/native-code.py b/solenv/bin/native-code.py
index 669bddc7b00d..3c569be67a15 100755
--- a/solenv/bin/native-code.py
+++ b/solenv/bin/native-code.py
@@ -311,6 +311,7 @@ core_constructor_list = [
 "lingucomponent_LangGuess_get_implementation",
 "lingucomponent_Hyphenator_get_implementation",
 "lingucomponent_LanguageToolGrammarChecker_get_implementation",
+"lingucomponent_NumberText_get_implementation",
 # package/source/xstor/xstor.component
 "package_OStorageFactory_get_implementation",
 # package/util/package2.component


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

2023-03-21 Thread Mike Kaganski (via logerrit)
 writerfilter/source/dmapper/DomainMapper_Impl.cxx |   17 ++---
 1 file changed, 10 insertions(+), 7 deletions(-)

New commits:
commit 24d2e8ea7a17ded531d19cba3f1cb30b17833eea
Author: Mike Kaganski 
AuthorDate: Tue Mar 21 19:18:14 2023 +0300
Commit: Mike Kaganski 
CommitDate: Tue Mar 21 20:13:36 2023 +

Simplify a bit

Also takes care of one place where dereferencing wasn't safe

Change-Id: I4ed67ad709610bea9cbed8184710f53d18d76d11
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149225
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 

diff --git a/writerfilter/source/dmapper/DomainMapper_Impl.cxx 
b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
index ffcd883289bd..37f7059118c8 100644
--- a/writerfilter/source/dmapper/DomainMapper_Impl.cxx
+++ b/writerfilter/source/dmapper/DomainMapper_Impl.cxx
@@ -6218,7 +6218,6 @@ void DomainMapper_Impl::handleToc
 OUString sTemplate;
 OUString sChapterNoSeparator;
 OUString sFigureSequence;
-uno::Reference< beans::XPropertySet > xTOC;
 OUString aBookmarkName;
 
 //  \a Builds a table of figures but does not include the 
captions's label and number
@@ -6317,6 +6316,8 @@ void DomainMapper_Impl::handleToc
 
 const OUString aTocTitle = extractTocTitle();
 
+uno::Reference xTOC;
+
 if (m_xTextFactory.is() && ! m_aTextAppendStack.empty())
 {
 const auto& xTextAppend = GetTopTextAppend();
@@ -6352,13 +6353,17 @@ void DomainMapper_Impl::handleToc
 }
 
 m_bStartTOC = true;
+pContext->SetTOC(xTOC);
+m_bParaHadField = false;
 
-if (xTOC.is())
-xTOC->setPropertyValue(getPropertyName( PROP_TITLE ), 
uno::Any(aTocTitle));
+if (!xTOC)
+return;
+
+xTOC->setPropertyValue(getPropertyName( PROP_TITLE ), uno::Any(aTocTitle));
 
 if (!aBookmarkName.isEmpty())
 xTOC->setPropertyValue(getPropertyName(PROP_TOC_BOOKMARK), 
uno::Any(aBookmarkName));
-if( !bTableOfFigures && xTOC.is() )
+if (!bTableOfFigures)
 {
 xTOC->setPropertyValue( getPropertyName( PROP_LEVEL ), uno::Any( 
nMaxLevel ) );
 xTOC->setPropertyValue( getPropertyName( PROP_CREATE_FROM_OUTLINE ), 
uno::Any( bFromOutline ));
@@ -6427,7 +6432,7 @@ void DomainMapper_Impl::handleToc
 }
 }
 }
-else if (bTableOfFigures && xTOC.is())
+else // if (bTableOfFigures)
 {
 if (!sFigureSequence.isEmpty())
 xTOC->setPropertyValue(getPropertyName(PROP_LABEL_CATEGORY),
@@ -6452,8 +6457,6 @@ void DomainMapper_Impl::handleToc
 xLevelFormats->replaceByIndex( 1, uno::Any( aNewLevel ) );
 }
 }
-pContext->SetTOC( xTOC );
-m_bParaHadField = false;
 }
 
 uno::Reference DomainMapper_Impl::createSectionForRange(


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

2023-03-21 Thread Caolán McNamara (via logerrit)
 sc/source/ui/view/tabview.cxx |   40 +++-
 1 file changed, 19 insertions(+), 21 deletions(-)

New commits:
commit f2b97ecf340e1fd714c636060ae09382518a2b46
Author: Caolán McNamara 
AuthorDate: Tue Mar 21 14:30:38 2023 +
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 20:21:36 2023 +

unmerge default and ScrollType::Drag cases for calc scrollbars

before:

commit 8c4e8818fe9f5ac1f6cdf908299cc109d67f1e50
Author: Caolán McNamara 
Date:   Thu Aug 4 16:38:47 2022 +0100

nDelta overwritten on all branches except default

the default case was different to the Drag case, and tdf#152406
uncovers a problem using the Drag case for scroll wheel, so undo
the merge of default of Drag.

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

diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index 9142393a9152..dea6dbb7810d 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -1170,7 +1170,6 @@ void ScTabView::ScrollHdl(ScrollAdaptor* pScroll)
 if ( pScroll == aVScrollBottom.get() ) nDelta = 
aViewData.VisibleCellsY( SC_SPLIT_BOTTOM );
 if (nDelta==0) nDelta=1;
 break;
-default: // added to avoid warnings
 case ScrollType::Drag:
 {
 // only scroll in the correct direction, do not jitter around 
hidden ranges
@@ -1182,31 +1181,30 @@ void ScTabView::ScrollHdl(ScrollAdaptor* pScroll)
 
 tools::Long nScrollPos = GetScrollBarPos( *pScroll ) + 
nScrollMin;
 nDelta = nScrollPos - nViewPos;
-
-// tdf#152406 Disable anti-jitter code for scroll wheel events
-// After moving thousands of columns to the right via
-// horizontal scroll wheel or trackpad swipe events, most
-// vertical scroll wheel or trackpad swipe events will trigger
-// the anti-jitter code because nScrollPos and nPrevDragPos
-// will be equal and nDelta will be overriden and set to zero.
-// So, only use the anti-jitter code for mouse drag events.
-if ( eType == ScrollType::Drag )
+if ( nScrollPos > nPrevDragPos )
 {
-if ( nScrollPos > nPrevDragPos )
-{
-if (nDelta<0) nDelta=0;
-}
-else if ( nScrollPos < nPrevDragPos )
-{
-if (nDelta>0) nDelta=0;
-}
-else
-nDelta = 0;
+if (nDelta<0) nDelta=0;
 }
-
+else if ( nScrollPos < nPrevDragPos )
+{
+if (nDelta>0) nDelta=0;
+}
+else
+nDelta = 0;
 nPrevDragPos = nScrollPos;
 }
 break;
+default:
+// Note tdf#152406 no anti-jitter code, unlike ScrollType::Drag,
+// for scroll wheel events.
+// After moving thousands of columns to the right via horizontal
+// scroll wheel or trackpad swipe events, most vertical scroll
+// wheel or trackpad swipe events would trigger the anti-jitter
+// code because nScrollPos and nPrevDragPos would be equal and
+// nDelta will be overriden and set to zero. So, only use the
+// anti-jitter code for mouse drag events.
+nDelta = GetScrollBarPos(*pScroll) - nViewPos;
+break;
 }
 
 if (nDelta)


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

2023-03-21 Thread Patrick Luby (via logerrit)
 sc/source/ui/view/tabview.cxx |   28 
 1 file changed, 20 insertions(+), 8 deletions(-)

New commits:
commit 12fdd2792c392a5b355110836faa40db5866e57e
Author: Patrick Luby 
AuthorDate: Mon Mar 20 16:48:30 2023 -0400
Commit: Caolán McNamara 
CommitDate: Tue Mar 21 20:21:47 2023 +

tdf#152406 Disable anti-jitter code for scroll wheel events

After moving thousands of columns to the right via horizontal
scroll wheel or trackpad swipe events, most vertical scroll
wheel or trackpad swipe events will trigger the anti-jitter code
because nScrollPos and nPrevDragPos will be equal and nDelta
will be overriden and set to zero. So, only use the anti-jitter
code for mouse drag events.

Change-Id: I9a22b31e1e012a97a058ab36e040629a71f5d24f
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149183
Tested-by: Jenkins
Tested-by: Caolán McNamara 
Reviewed-by: Caolán McNamara 
(cherry picked from commit d82a734c07b85cbd7861699b7fa6d3ebbb3122f2)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149238

diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index 17a414a5f5e4..aad41a9c1236 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -1182,16 +1182,28 @@ void ScTabView::ScrollHdl(ScrollAdaptor* pScroll)
 
 tools::Long nScrollPos = GetScrollBarPos( *pScroll ) + 
nScrollMin;
 nDelta = nScrollPos - nViewPos;
-if ( nScrollPos > nPrevDragPos )
-{
-if (nDelta<0) nDelta=0;
-}
-else if ( nScrollPos < nPrevDragPos )
+
+// tdf#152406 Disable anti-jitter code for scroll wheel events
+// After moving thousands of columns to the right via
+// horizontal scroll wheel or trackpad swipe events, most
+// vertical scroll wheel or trackpad swipe events will trigger
+// the anti-jitter code because nScrollPos and nPrevDragPos
+// will be equal and nDelta will be overriden and set to zero.
+// So, only use the anti-jitter code for mouse drag events.
+if ( eType == ScrollType::Drag )
 {
-if (nDelta>0) nDelta=0;
+if ( nScrollPos > nPrevDragPos )
+{
+if (nDelta<0) nDelta=0;
+}
+else if ( nScrollPos < nPrevDragPos )
+{
+if (nDelta>0) nDelta=0;
+}
+else
+nDelta = 0;
 }
-else
-nDelta = 0;
+
 nPrevDragPos = nScrollPos;
 }
 break;


[Libreoffice-commits] core.git: Branch 'distro/collabora/co-22.05' - download.lst

2023-03-21 Thread Michael Stahl (via logerrit)
 download.lst |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 5e8c5742bd87a71ffcb69a34ee8baf69ff295149
Author: Michael Stahl 
AuthorDate: Mon Mar 20 11:52:22 2023 +0100
Commit: Andras Timar 
CommitDate: Tue Mar 21 20:54:23 2023 +

curl: upgrade to release 8.0.0

Fixes CVE-2023-27535.

Also hopefully fixes excessive storage consumption during build:
o build: drop the use of XC_AMEND_DISTCLEAN [62]

Change-Id: I8792e95bc7634ee496488e80fec5a1310b24a31c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149153
Tested-by: Jenkins
Reviewed-by: Michael Stahl 
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149211
Tested-by: Andras Timar 
Reviewed-by: Andras Timar 

diff --git a/download.lst b/download.lst
index 87e91ede9118..e1ad51789623 100644
--- a/download.lst
+++ b/download.lst
@@ -37,8 +37,8 @@ export CPPUNIT_SHA256SUM := 
89c5c6665337f56fd2db36bc3805a5619709d51fb136e5193707
 export CPPUNIT_TARBALL := cppunit-1.15.1.tar.gz
 export CT2N_SHA256SUM := 
71b238efd2734be9800af07566daea8d6685aeed28db5eb5fa0e6453f4d85de3
 export CT2N_TARBALL := 
1f467e5bb703f12cbbb09d5cf67ecf4a-converttexttonumber-1-5-0.oxt
-export CURL_SHA256SUM := 
1dae31b2a7c1fe269de99c0c31bb488346aab3459b5ffca909d6938249ae415f
-export CURL_TARBALL := curl-7.88.1.tar.xz
+export CURL_SHA256SUM := 
759690f9a375a720f8bcce9f953897b0d93f31eed9649b74f846d54bbf63bbcc
+export CURL_TARBALL := curl-8.0.0.tar.xz
 export EBOOK_SHA256SUM := 
7e8d8ff34f27831aca3bc6f9cc532c2f90d2057c778963b884ff3d1e34dfe1f9
 export EBOOK_TARBALL := libe-book-0.1.3.tar.xz
 export EPOXY_SHA256SUM := 
a7ced37f4102b745ac86d6a70a9da399cc139ff168ba6b8002b4d8d43c900c15


[Libreoffice-commits] core.git: Branch 'distro/collabora/co-22.05' - download.lst

2023-03-21 Thread Michael Stahl (via logerrit)
 download.lst |4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

New commits:
commit 13658021cf7028369ee85a5478f5c0c9ed96c11a
Author: Michael Stahl 
AuthorDate: Tue Mar 21 10:46:46 2023 +0100
Commit: Andras Timar 
CommitDate: Tue Mar 21 20:54:42 2023 +

curl: upgrade to release 8.0.1

Apparently 8.0.0 had a serious regression.

Change-Id: Icc761f5e5e01b5d9bebecc13f7cba608f5834f54
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149212
Tested-by: Andras Timar 
Reviewed-by: Andras Timar 

diff --git a/download.lst b/download.lst
index e1ad51789623..4fc4aad6587a 100644
--- a/download.lst
+++ b/download.lst
@@ -37,8 +37,8 @@ export CPPUNIT_SHA256SUM := 
89c5c6665337f56fd2db36bc3805a5619709d51fb136e5193707
 export CPPUNIT_TARBALL := cppunit-1.15.1.tar.gz
 export CT2N_SHA256SUM := 
71b238efd2734be9800af07566daea8d6685aeed28db5eb5fa0e6453f4d85de3
 export CT2N_TARBALL := 
1f467e5bb703f12cbbb09d5cf67ecf4a-converttexttonumber-1-5-0.oxt
-export CURL_SHA256SUM := 
759690f9a375a720f8bcce9f953897b0d93f31eed9649b74f846d54bbf63bbcc
-export CURL_TARBALL := curl-8.0.0.tar.xz
+export CURL_SHA256SUM := 
0a381cd82f4d00a9a334438b8ca239afea5bfefcfa9a1025f2bf118e79e0b5f0
+export CURL_TARBALL := curl-8.0.1.tar.xz
 export EBOOK_SHA256SUM := 
7e8d8ff34f27831aca3bc6f9cc532c2f90d2057c778963b884ff3d1e34dfe1f9
 export EBOOK_TARBALL := libe-book-0.1.3.tar.xz
 export EPOXY_SHA256SUM := 
a7ced37f4102b745ac86d6a70a9da399cc139ff168ba6b8002b4d8d43c900c15


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

2023-03-21 Thread Pranam Lashkari (via logerrit)
 vcl/jsdialog/enabled.cxx |2 ++
 1 file changed, 2 insertions(+)

New commits:
commit 65758e25b16db552af7699facbe068e3d4aebc3a
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 17:47:17 2023 +0530
Commit: Pranam Lashkari 
CommitDate: Tue Mar 21 22:11:43 2023 +

jsdialog: enabled insert break(manual break) dialog(writer)

Signed-off-by: Pranam Lashkari 
Change-Id: I65213cd8822e574eff333b96a8718ebff2946f5c
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149166
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
(cherry picked from commit 41b7f70981cffb9a04b6b132317915ca51519eab)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149236
Tested-by: Jenkins

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index 1dad92a7347c..54ffe823aecf 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -112,6 +112,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"modules/swriter/ui/footnoteareapage.ui"
 || rUIFile == u"modules/swriter/ui/footnotesendnotestabpage.ui"
 || rUIFile == u"modules/swriter/ui/indentpage.ui"
+|| rUIFile == u"modules/swriter/ui/insertbreak.ui"
 || rUIFile == u"modules/swriter/ui/insertcaption.ui"
 || rUIFile == u"modules/swriter/ui/insertsectiondialog.ui"
 || rUIFile == u"modules/swriter/ui/linenumbering.ui"
commit e8d56d600f16e137acf765842273feeca5d98b37
Author: Pranam Lashkari 
AuthorDate: Mon Mar 20 18:50:12 2023 +0530
Commit: Pranam Lashkari 
CommitDate: Tue Mar 21 22:11:32 2023 +

jsdialog: enable watermark dialog (writer)

Change-Id: Ie1f5a299fd68bdfd21c04d1ef79514953a2ac5c3
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149168
Tested-by: Szymon Kłos 
Reviewed-by: Szymon Kłos 
(cherry picked from commit 8c358c1616a16beb10752f0a60bbacccba15d6ea)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149234
Tested-by: Jenkins
Reviewed-by: Pranam Lashkari 

diff --git a/vcl/jsdialog/enabled.cxx b/vcl/jsdialog/enabled.cxx
index a99417b50a27..1dad92a7347c 100644
--- a/vcl/jsdialog/enabled.cxx
+++ b/vcl/jsdialog/enabled.cxx
@@ -124,6 +124,7 @@ bool isBuilderEnabled(std::u16string_view rUIFile, bool 
bMobile)
 || rUIFile == u"modules/swriter/ui/templatedialog8.ui"
 || rUIFile == u"modules/swriter/ui/textgridpage.ui"
 || rUIFile == u"modules/swriter/ui/translationdialog.ui"
+|| rUIFile == u"modules/swriter/ui/watermarkdialog.ui"
 // sfx
 || rUIFile == u"sfx/ui/cmisinfopage.ui"
 || rUIFile == u"sfx/ui/custominfopage.ui"


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

2023-03-21 Thread Maxim Monastirsky (via logerrit)
 include/svx/svdpool.hxx |1 
 include/svx/xpool.hxx   |3 
 sc/inc/globstr.hrc  |1 
 sc/inc/servuno.hxx  |2 
 sc/qa/extras/scstylefamiliesobj.cxx |2 
 sc/source/core/data/stlpool.cxx |2 
 sc/source/core/data/stlsheet.cxx|   42 
 sc/source/ui/unoobj/servuno.cxx |4 
 sc/source/ui/unoobj/styleuno.cxx|  183 +++-
 9 files changed, 212 insertions(+), 28 deletions(-)

New commits:
commit 6db5e15e5b91f34232e90225a723e0673ecfc2f0
Author: Maxim Monastirsky 
AuthorDate: Fri Mar 17 14:50:25 2023 +0200
Commit: Maxim Monastirsky 
CommitDate: Tue Mar 21 23:10:01 2023 +

sc drawstyles: Implement the styles family

Change-Id: I2af6072685e4c79cfe2f177c0aae27044f86d840
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149184
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky 

diff --git a/include/svx/svdpool.hxx b/include/svx/svdpool.hxx
index e61b6fe97005..3d2201edef5b 100644
--- a/include/svx/svdpool.hxx
+++ b/include/svx/svdpool.hxx
@@ -20,7 +20,6 @@
 #pragma once
 
 #include 
-#include 
 
 class XLineAttrSetItem;
 class XFillAttrSetItem;
diff --git a/include/svx/xpool.hxx b/include/svx/xpool.hxx
index e8fc6f5e7225..13f5fa3d7a7d 100644
--- a/include/svx/xpool.hxx
+++ b/include/svx/xpool.hxx
@@ -20,6 +20,7 @@
 #ifndef INCLUDED_SVX_XPOOL_HXX
 #define INCLUDED_SVX_XPOOL_HXX
 
+#include 
 #include 
 #include 
 
@@ -29,7 +30,7 @@
 |*
 \/
 
-class XOutdevItemPool : public SfxItemPool
+class SVXCORE_DLLPUBLIC XOutdevItemPool : public SfxItemPool
 {
 protected:
 std::vector* mpLocalPoolDefaults;
diff --git a/sc/inc/globstr.hrc b/sc/inc/globstr.hrc
index 95f7138c81c9..88eca11e5a89 100644
--- a/sc/inc/globstr.hrc
+++ b/sc/inc/globstr.hrc
@@ -443,6 +443,7 @@
 #define STR_FORM_SCROLLBAR  NC_("STR_FORM_SCROLLBAR", 
"Scroll Bar")
 #define STR_STYLE_FAMILY_CELL   NC_("STR_STYLE_FAMILY_CELL", 
"Cell Styles")
 #define STR_STYLE_FAMILY_PAGE   NC_("STR_STYLE_FAMILY_PAGE", 
"Page Styles")
+#define STR_STYLE_FAMILY_GRAPHICS   
NC_("STR_STYLE_FAMILY_GRAPHICS", "Drawing Styles")
 #define STR_ERR_DATAPILOTSOURCE NC_("STR_ERR_DATAPILOTSOURCE", 
"Pivot table source data is invalid.")
 #define STR_OPTIONS_WARN_SEPARATORS 
NC_("STR_OPTIONS_WARN_SEPARATORS", "Because the current formula separator 
settings conflict with the locale, the formula separators have been reset to 
their default values.")
 #define STR_UNDO_INSERT_CURRENT_DATE
NC_("STR_UNDO_INSERT_CURRENT_DATE", "Insert Current Date")
diff --git a/sc/inc/servuno.hxx b/sc/inc/servuno.hxx
index b8a6d49fd172..b705f3ef01b4 100644
--- a/sc/inc/servuno.hxx
+++ b/sc/inc/servuno.hxx
@@ -32,7 +32,7 @@ public:
 enum class Type
 {
 SHEET , URLFIELD , PAGEFIELD , PAGESFIELD , DATEFIELD , TIMEFIELD , 
TITLEFIELD , FILEFIELD ,
-SHEETFIELD , CELLSTYLE , PAGESTYLE ,
+SHEETFIELD , CELLSTYLE , PAGESTYLE , GRAPHICSTYLE ,
 // sheet
 AUTOFORMAT , AUTOFORMATS, CELLRANGES , FUNCTIONDESCRIPTIONS , 
GLOBALSHEETSETTINGS ,
 RECENTFUNCTIONS ,
diff --git a/sc/qa/extras/scstylefamiliesobj.cxx 
b/sc/qa/extras/scstylefamiliesobj.cxx
index 89ec9a2b46f7..72fb135bd56d 100644
--- a/sc/qa/extras/scstylefamiliesobj.cxx
+++ b/sc/qa/extras/scstylefamiliesobj.cxx
@@ -82,7 +82,7 @@ private:
 ScStyleFamiliesObj::ScStyleFamiliesObj()
 : UnoApiTest("/sc/qa/extras/testdocuments")
 , XElementAccess(cppu::UnoType::get())
-, XIndexAccess(2)
+, XIndexAccess(3)
 , XNameAccess("CellStyles")
 , XServiceInfo("ScStyleFamiliesObj", "com.sun.star.style.StyleFamilies")
 {
diff --git a/sc/source/core/data/stlpool.cxx b/sc/source/core/data/stlpool.cxx
index 8f554896ff21..e7f6d26aa0c7 100644
--- a/sc/source/core/data/stlpool.cxx
+++ b/sc/source/core/data/stlpool.cxx
@@ -101,7 +101,7 @@ rtl::Reference ScStyleSheetPool::Create( 
const OUString&   rN
  SfxStyleSearchBits nMaskP )
 {
 rtl::Reference pSheet = new ScStyleSheet( rName, *this, 
eFamily, nMaskP );
-if ( eFamily == SfxStyleFamily::Para && ScResId(STR_STYLENAME_STANDARD) != 
rName )
+if ( eFamily != SfxStyleFamily::Page && ScResId(STR_STYLENAME_STANDARD) != 
rName )
 pSheet->SetParent( ScResId(STR_STYLENAME_STANDARD) );
 
 return pSheet;
diff --git a/sc/source/core/data/stlsheet.cxx b/sc/source/core/data/stlsheet.cxx
index 4ee18ef0f0fb..665b806ba540 100644
--- a/sc/source/core/data/stlsheet.cxx
+++ b/sc/source/core/data/stlsheet.cxx
@@ -26,6 +26,10 @@
 #include 
 #include 
 #include 
+#include 
+#include 
+#include 
+#include 
 #include 
 #include 
 #include 
@@ -77,6 +81,7 @@ bool ScStyleSheet::HasParentSupport () const
 switch ( GetFamily() )
 {
 case SfxStyleFamily::Para: bHasParentSup

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

2023-03-21 Thread Maxim Monastirsky (via logerrit)
 sc/source/core/data/drwlayer.cxx |5 +
 1 file changed, 5 insertions(+)

New commits:
commit f460bd163afb06dc2dccc877269106af38767484
Author: Maxim Monastirsky 
AuthorDate: Sun Mar 19 19:52:09 2023 +0200
Commit: Maxim Monastirsky 
CommitDate: Tue Mar 21 23:10:20 2023 +

sc drawstyles: Connect the styles pool to the draw layer

Change-Id: Ie87bb028264b74fd3e15e23a3ad60ab3c48bb14d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149202
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky 

diff --git a/sc/source/core/data/drwlayer.cxx b/sc/source/core/data/drwlayer.cxx
index e6203697a4e7..653571cabd38 100644
--- a/sc/source/core/data/drwlayer.cxx
+++ b/sc/source/core/data/drwlayer.cxx
@@ -72,6 +72,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 
 #include 
@@ -280,6 +281,8 @@ ScDrawLayer::ScDrawLayer( ScDocument* pDocument, OUString 
_aName ) :
 
 rPool.FreezeIdRanges(); // the pool is also used 
directly
 
+SetStyleSheetPool(pDocument ? pDocument->GetStyleSheetPool() : new 
ScStyleSheetPool(rPool, pDocument));
+
 SdrLayerAdmin& rAdmin = GetLayerAdmin();
 rAdmin.NewLayer("vorne",SC_LAYER_FRONT.get());
 rAdmin.NewLayer("hinten",   SC_LAYER_BACK.get());
@@ -292,9 +295,11 @@ ScDrawLayer::ScDrawLayer( ScDocument* pDocument, OUString 
_aName ) :
 ScModule* pScMod = SC_MOD();
 Outliner& rOutliner = GetDrawOutliner();
 rOutliner.SetCalcFieldValueHdl( LINK( pScMod, ScModule, CalcFieldValueHdl 
) );
+
rOutliner.SetStyleSheetPool(static_cast(GetStyleSheetPool()));
 
 Outliner& rHitOutliner = GetHitTestOutliner();
 rHitOutliner.SetCalcFieldValueHdl( LINK( pScMod, ScModule, 
CalcFieldValueHdl ) );
+
rHitOutliner.SetStyleSheetPool(static_cast(GetStyleSheetPool()));
 
 // set FontHeight pool defaults without changing static SdrEngineDefaults
 SfxItemPool* pOutlinerPool = rOutliner.GetEditTextObjectPool();


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

2023-03-21 Thread Maxim Monastirsky (via logerrit)
 framework/source/uielement/styletoolbarcontroller.cxx |3 
 sc/inc/globstr.hrc|1 
 sc/sdi/drawsh.sdi |   12 -
 sc/sdi/drtxtob.sdi|   12 -
 sc/sdi/tabvwsh.sdi|1 
 sc/source/ui/drawfunc/drawsh2.cxx |2 
 sc/source/ui/drawfunc/fusel.cxx   |8 +
 sc/source/ui/undo/undostyl.cxx|   16 +-
 sc/source/ui/view/tabview3.cxx|2 
 sc/source/ui/view/tabvwsha.cxx|  143 +-
 10 files changed, 164 insertions(+), 36 deletions(-)

New commits:
commit 1cfeb4bd8ce7f7727a81136bd3e2d6ebea976895
Author: Maxim Monastirsky 
AuthorDate: Fri Mar 17 14:28:36 2023 +0200
Commit: Maxim Monastirsky 
CommitDate: Tue Mar 21 23:10:39 2023 +

sc drawstyles: Handle style commands and undo

Change-Id: Ibd81f7aba9ef2efca89556c31cd9b09867419d0e
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149203
Tested-by: Jenkins
Reviewed-by: Maxim Monastirsky 

diff --git a/framework/source/uielement/styletoolbarcontroller.cxx 
b/framework/source/uielement/styletoolbarcontroller.cxx
index 31aed5336f10..3ff1e777eb6d 100644
--- a/framework/source/uielement/styletoolbarcontroller.cxx
+++ b/framework/source/uielement/styletoolbarcontroller.cxx
@@ -34,7 +34,8 @@ OUString MapFamilyToCommand( std::u16string_view rFamily )
 return ".uno:CharStyle";
 else if ( rFamily == u"PageStyles" )
 return ".uno:PageStyle";
-else if ( rFamily == u"FrameStyles" )
+else if ( rFamily == u"FrameStyles" ||
+  rFamily == u"GraphicStyles" )  // In sc
 return ".uno:FrameStyle";
 else if ( rFamily == u"NumberingStyles" )
 return ".uno:ListStyle";
diff --git a/sc/inc/globstr.hrc b/sc/inc/globstr.hrc
index 88eca11e5a89..16fce3f32974 100644
--- a/sc/inc/globstr.hrc
+++ b/sc/inc/globstr.hrc
@@ -84,6 +84,7 @@
 #define STR_UNDO_EDITCELLSTYLE  NC_("STR_UNDO_EDITCELLSTYLE", 
"Edit Cell Style")
 #define STR_UNDO_APPLYPAGESTYLE NC_("STR_UNDO_APPLYPAGESTYLE", 
"Apply Page Style")
 #define STR_UNDO_EDITPAGESTYLE  NC_("STR_UNDO_EDITPAGESTYLE", 
"Edit Page Style")
+#define STR_UNDO_EDITGRAPHICSTYLE   
NC_("STR_UNDO_EDITGRAPHICSTYLE", "Edit Drawing Style")
 #define STR_UNDO_DETADDPRED NC_("STR_UNDO_DETADDPRED", 
"Trace Precedents")
 #define STR_UNDO_DETDELPRED NC_("STR_UNDO_DETDELPRED", 
"Remove Precedent")
 #define STR_UNDO_DETADDSUCC NC_("STR_UNDO_DETADDSUCC", 
"Trace Dependents")
diff --git a/sc/sdi/drawsh.sdi b/sc/sdi/drawsh.sdi
index ccd5d69605d4..13f1d6b150df 100644
--- a/sc/sdi/drawsh.sdi
+++ b/sc/sdi/drawsh.sdi
@@ -35,18 +35,6 @@ interface TableDraw
 SID_OPENDLG_EDIT_PRINTAREA  [ StateMethod = StateDisableItems; ]
 // others:
 SID_DRAW_CHART  [ StateMethod = StateDisableItems; ]
-SID_STYLE_FAMILY2   [ StateMethod = StateDisableItems; ]
-SID_STYLE_FAMILY4   [ StateMethod = StateDisableItems; ]
-SID_STYLE_APPLY [ StateMethod = StateDisableItems; ]
-SID_STYLE_WATERCAN  [ StateMethod = StateDisableItems; ]
-SID_STYLE_NEW_BY_EXAMPLE[ StateMethod = StateDisableItems; ]
-SID_STYLE_UPDATE_BY_EXAMPLE [ StateMethod = StateDisableItems; ]
-SID_STYLE_NEW   [ StateMethod = StateDisableItems; ]
-SID_STYLE_EDIT  [ StateMethod = StateDisableItems; ]
-SID_STYLE_DELETE[ StateMethod = StateDisableItems; ]
-SID_STYLE_HIDE  [ StateMethod = StateDisableItems; ]
-SID_STYLE_SHOW  [ StateMethod = StateDisableItems; ]
-
 
 SID_TEXT_STANDARD   [ ExecMethod = ExecDrawAttr; StateMethod = 
NoState; ]
 SID_DRAWTEXT_ATTR_DLG   [ ExecMethod = ExecDrawAttr; StateMethod = 
NoState; ]
diff --git a/sc/sdi/drtxtob.sdi b/sc/sdi/drtxtob.sdi
index 5280430413bc..4670d216fba2 100644
--- a/sc/sdi/drtxtob.sdi
+++ b/sc/sdi/drtxtob.sdi
@@ -33,18 +33,6 @@ interface TableDrawText
 // others:
 SID_DRAW_CHART  [ StateMethod = StateDisableItems; ]
 SID_OPENDLG_FUNCTION[ StateMethod = StateDisableItems; ]
-SID_STYLE_FAMILY2   [ StateMethod = StateDisableItems; ]
-SID_STYLE_FAMILY4   [ StateMethod = StateDisableItems; ]
-SID_STYLE_APPLY [ StateMethod = StateDisableItems; ]
-SID_STYLE_WATERCAN  [ StateMethod = StateDisableItems; ]
-SID_STYLE_NEW_BY_EXAMPLE[ StateMethod = StateDisableItems; ]
-SID_STYLE_UPDATE_BY_EXAMPLE [ StateMethod = StateDisableItems; ]
-SID_STYLE_NEW   [ StateMethod = StateDisableItems; ]
-SID_STYLE_EDIT  [ StateMethod = StateDisableItems; ]
-SID_STYLE_DELETE[ StateMethod = StateDisableItems; ]
-SID_STY

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

2023-03-21 Thread Mike Kaganski (via logerrit)
 basic/qa/vba_tests/strconv.vb|   21 ++-
 basic/source/runtime/methods.cxx |  210 ---
 2 files changed, 124 insertions(+), 107 deletions(-)

New commits:
commit 458f2b8f69fbf0a220f18fbf250e4369fd0491cd
Author: Mike Kaganski 
AuthorDate: Wed Mar 15 16:30:26 2023 +0300
Commit: Mike Kaganski 
CommitDate: Wed Mar 22 04:31:07 2023 +

Refactor and fix VBA StrConv

This properly handles null bytes that are expected
when converting between byte strings and Unicode.

It properly handles TransliterationFlags, which are
not a bitset.

In vbProperCase, it uses the correct method to
lowercase the string, working not only with ASCII.

Change-Id: I04e8cdca66ef9863a6516b15205a2a543ed97680
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/149224
Tested-by: Jenkins
Reviewed-by: Mike Kaganski 

diff --git a/basic/qa/vba_tests/strconv.vb b/basic/qa/vba_tests/strconv.vb
index b0295df428b2..9b7dfaf21866 100644
--- a/basic/qa/vba_tests/strconv.vb
+++ b/basic/qa/vba_tests/strconv.vb
@@ -18,25 +18,30 @@ End Function
 Sub verify_testStrConv()
 On Error GoTo errorHandler
 
-TestUtil.AssertEqual(StrConv("abc EFG hij", vbUpperCase),  "ABC EFG HIJ", 
"StrConv(""abc EFG hij"", vbUpperCase)")
-TestUtil.AssertEqual(StrConv("abc EFG hij", vbLowerCase),  "abc efg hij", 
"StrConv(""abc EFG hij"", vbLowerCase)")
-TestUtil.AssertEqual(StrConv("abc EFG hij", vbProperCase), "Abc Efg Hij", 
"StrConv(""abc EFG hij"", vbProperCase)")
+TestUtil.AssertEqual(StrConv("abc EFG hij αβγ ΔΕΖ ηθι", vbUpperCase),  
"ABC EFG HIJ ΑΒΓ ΔΕΖ ΗΘΙ", "StrConv(""abc EFG hij αβγ ΔΕΖ ηθι"", vbUpperCase)")
+TestUtil.AssertEqual(StrConv("abc EFG hij αβγ ΔΕΖ ηθι", vbLowerCase),  
"abc efg hij αβγ δεζ ηθι", "StrConv(""abc EFG hij αβγ ΔΕΖ ηθι"", vbLowerCase)")
+TestUtil.AssertEqual(StrConv("abc EFG hij αβγ ΔΕΖ ηθι", vbProperCase), 
"Abc Efg Hij Αβγ Δεζ Ηθι", "StrConv(""abc EFG hij αβγ ΔΕΖ ηθι"", vbProperCase)")
 
 ' Converts narrow (single-byte) characters in string to wide
 TestUtil.AssertEqual(StrConv("ABCDEVB¥ì¥¹¥­¥å©", vbWide), 
"ABCDEVB¥ì¥¹¥­¥å©", "StrConv(""ABCDEVB¥ì¥¹¥­¥å©"", vbWide)")
+TestUtil.AssertEqual(StrConv("ABCDEVB¥ì¥¹¥­¥å©", vbWide + vbLowerCase), 
"abcdevb¥ì¥¹¥­¥å©", "StrConv(""ABCDEVB¥ì¥¹¥­¥å©"", vbWide + vbLowerCase)")
 
 ' Converts wide (double-byte) characters in string to narrow (single-byte) 
characters
 TestUtil.AssertEqual(StrConv("ABCD@$%23'?EG", vbNarrow), "ABCD@$%23'?EG", 
"StrConv(""ABCD@$%23'?EG"", vbNarrow)")
+TestUtil.AssertEqual(StrConv("ABCD@$%23'?EG", vbNarrow + vbLowerCase), 
"abcd@$%23'?eg", "StrConv(""ABCD@$%23'?EG"", vbNarrow + vbLowerCase)")
 
 ' Converts Hiragana characters in string to Katakana characters
 TestUtil.AssertEqual(StrConv("かたかな", vbKatakana), "カタカナ", 
"StrConv(""かたかな"", vbKatakana)")
+TestUtil.AssertEqual(StrConv("かたかな abc", vbKatakana + vbUpperCase + 
vbWide), "カタカナ ABC", "StrConv(""かたかな abc"", vbKatakana + vbUpperCase + vbWide)")
 
 ' Converts Katakana characters in string to Hiragana characters
 TestUtil.AssertEqual(StrConv("カタカナ", vbHiragana), "かたかな", 
"StrConv(""カタカナ"", vbHiragana)")
+TestUtil.AssertEqual(StrConv("カタカナ ABC", vbLowerCase + vbNarrow), "カタカナ 
abc", "StrConv(""カタカナ ABC"", vbLowerCase + vbNarrow)")
 
-' Assumes CP-1252 encoding associated with en-US locale used in unit tests.
 Dim x() As Byte
-x = StrConv("ÉϺ£ÊÐABC", vbFromUnicode)
+Const Cp1252TestString = "ÉϺ£ÊÐABC"
+
+x = StrConv(Cp1252TestString, vbFromUnicode, &h0409)' CP-1252 encoding 
associated with en-US locale
 TestUtil.AssertEqual(UBound(x), 8, "UBound(x)")
 TestUtil.AssertEqual(x(0), 201, "x(0)")
 TestUtil.AssertEqual(x(1), 207, "x(1)")
@@ -47,7 +52,11 @@ Sub verify_testStrConv()
 TestUtil.AssertEqual(x(6), 65, "x(6)")
 TestUtil.AssertEqual(x(7), 66, "x(7)")
 TestUtil.AssertEqual(x(8), 67, "x(8)")
-TestUtil.AssertEqual(StrConv(x, vbUnicode), "ÉϺ£ÊÐABC", "StrConv(x, 
vbUnicode)")
+TestUtil.AssertEqual(StrConv(x, vbUnicode, &h0409), Cp1252TestString, 
"StrConv(x, vbUnicode, &h0409)")
+
+x = StrConv(Cp1252TestString, vbUnicode, &h0409)
+TestUtil.AssertEqual(UBound(x), 35, "UBound(x)")
+TestUtil.AssertEqual(StrConv(x, vbFromUnicode, &h0409), Cp1252TestString, 
"StrConv(x, vbFromUnicode, &h0409)")
 
 Exit Sub
 errorHandler:
diff --git a/basic/source/runtime/methods.cxx b/basic/source/runtime/methods.cxx
index b6d9383d5b37..0b4f89d40115 100644
--- a/basic/source/runtime/methods.cxx
+++ b/basic/source/runtime/methods.cxx
@@ -59,6 +59,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 #include 
 #include 
@@ -4084,6 +4085,40 @@ void SbRtl_QBColor(StarBASIC *, SbxArray & rPar, bool)
 rPar.Get(0)->PutLong(nRGB);
 }
 
+static std::vector byteArray2Vec(SbxArray* pArr)
+{
+std::vector result;
+if (pArr)
+{
+const sal_uInt32 nCount = pArr->Count();
+result.

Origin/master build fail

2023-03-21 Thread Franklin Weng

Hi,

Since yesterday when I built from origin/master it always halted with 
the following messages:


[BIN] sc
[MOD] sc
[BIN] postprocess
[GAL] backgrounds
[GAL] sounds
[MOD] postprocess
Aborted (core dumped)
terminate called after throwing an instance of 
'com::sun::star::uno::RuntimeException'
make[1]: *** [/opt/git/libreoffice.core/solenv/gbuild/Gallery.mk:57: 
/opt/git/libreoffice.core/workdir/Gallery/sounds.done] Error 1
make[1]: *** Waiting for unfinished jobs
Aborted (core dumped)
terminate called after throwing an instance of 
'com::sun::star::uno::RuntimeException'
make[1]: *** [/opt/git/libreoffice.core/solenv/gbuild/Gallery.mk:57: 
/opt/git/libreoffice.core/workdir/Gallery/backgrounds.done] Error 1
make: *** [Makefile:289: build] Error 2


If I checked out tag libreoffice-7.5.1.2 it could build successfully, so 
I think it should not be caused by my building environment.


Does anyone experience the same problem?


Thanks, Franklin

--
Franklin Weng
Member, Certification Committee
The Document Foundation, Kurfürstendamm 188, 10707 Berlin, DE
Gemeinnützige rechtsfähige Stiftung des bürgerlichen Rechts
Legal details:https://www.documentfoundation.org/imprint



OpenPGP_signature
Description: OpenPGP digital signature


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

2023-03-21 Thread Andrea Gelmini (via logerrit)
 0 files changed

New commits:
commit dd490103342900a1bab2138555803e53a49894dd
Author: Andrea Gelmini 
AuthorDate: Wed Mar 15 18:58:07 2023 +0100
Commit: Julien Nabet 
CommitDate: Wed Mar 22 05:35:19 2023 +

Remove executable bits from odt

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

diff --git a/sw/qa/extras/uiwriter/data/tdf40142.odt 
b/sw/qa/extras/uiwriter/data/tdf40142.odt
old mode 100755
new mode 100644


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

2023-03-21 Thread Andrea Gelmini (via logerrit)
 sc/source/ui/view/tabview.cxx |2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 30ab88edbbd60e62302a8d08a3cca39bfb0d45a5
Author: Andrea Gelmini 
AuthorDate: Tue Mar 21 23:08:25 2023 +0100
Commit: Julien Nabet 
CommitDate: Wed Mar 22 05:38:22 2023 +

Fix typo

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

diff --git a/sc/source/ui/view/tabview.cxx b/sc/source/ui/view/tabview.cxx
index dea6dbb7810d..69aef742b345 100644
--- a/sc/source/ui/view/tabview.cxx
+++ b/sc/source/ui/view/tabview.cxx
@@ -1201,7 +1201,7 @@ void ScTabView::ScrollHdl(ScrollAdaptor* pScroll)
 // scroll wheel or trackpad swipe events, most vertical scroll
 // wheel or trackpad swipe events would trigger the anti-jitter
 // code because nScrollPos and nPrevDragPos would be equal and
-// nDelta will be overriden and set to zero. So, only use the
+// nDelta will be overridden and set to zero. So, only use the
 // anti-jitter code for mouse drag events.
 nDelta = GetScrollBarPos(*pScroll) - nViewPos;
 break;


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

2023-03-21 Thread Noel Grandin (via logerrit)
 chart2/source/controller/chartapiwrapper/DataSeriesPointWrapper.cxx|2 
 chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx   |2 
 chart2/source/controller/inc/SeriesOptionsItemConverter.hxx|3 -
 chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx |4 +
 chart2/source/controller/itemsetwrapper/SeriesOptionsItemConverter.cxx |   13 
++
 chart2/source/inc/ChartModelHelper.hxx |5 
--
 chart2/source/inc/Diagram.hxx  |7 
---
 chart2/source/inc/RegressionCurveHelper.hxx|2 
 chart2/source/model/main/Diagram.cxx   |   21 
--
 chart2/source/tools/ChartModelHelper.cxx   |8 
---
 chart2/source/tools/RegressionCurveHelper.cxx  |4 -
 11 files changed, 19 insertions(+), 52 deletions(-)

New commits:
commit 949c73f9c35d640c841f03d505e905aa2e1b7a3f
Author: Noel Grandin 
AuthorDate: Thu Mar 16 11:37:03 2023 +0200
Commit: Noel Grandin 
CommitDate: Wed Mar 22 05:57:16 2023 +

use more concrete types in chart2

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

diff --git 
a/chart2/source/controller/chartapiwrapper/DataSeriesPointWrapper.cxx 
b/chart2/source/controller/chartapiwrapper/DataSeriesPointWrapper.cxx
index 8961c2731f71..6c0dbe7bce25 100644
--- a/chart2/source/controller/chartapiwrapper/DataSeriesPointWrapper.cxx
+++ b/chart2/source/controller/chartapiwrapper/DataSeriesPointWrapper.cxx
@@ -318,7 +318,7 @@ Any WrappedAttachedAxisProperty::getPropertyValue( const 
Reference< beans::XProp
 
 void WrappedAttachedAxisProperty::setPropertyValue( const Any& rOuterValue, 
const Reference< beans::XPropertySet >& xInnerPropertySet ) const
 {
-uno::Reference< chart2::XDataSeries > xDataSeries( xInnerPropertySet, 
uno::UNO_QUERY );
+rtl::Reference< ::chart::DataSeries > xDataSeries( 
dynamic_cast<::chart::DataSeries*>(xInnerPropertySet.get()) );
 
 sal_Int32 nChartAxisAssign = css::chart::ChartAxisAssign::PRIMARY_Y;
 if( ! (rOuterValue >>= nChartAxisAssign) )
diff --git 
a/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx 
b/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx
index 50d4b2c0edd2..79c45ea1d888 100644
--- a/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx
+++ b/chart2/source/controller/chartapiwrapper/WrappedSymbolProperties.cxx
@@ -286,7 +286,7 @@ beans::PropertyState 
WrappedSymbolTypeProperty::getPropertyState( const Referenc
 m_spChart2ModelContact)
 {
 rtl::Reference< ::chart::Diagram > xDiagram( 
m_spChart2ModelContact->getDiagram() );
-Reference< chart2::XDataSeries > xSeries( xInnerPropertyState, 
uno::UNO_QUERY );
+rtl::Reference< ::chart::DataSeries > xSeries( 
dynamic_cast(xInnerPropertyState.get()) );
 rtl::Reference< ChartType > xChartType( 
xDiagram->getChartTypeOfSeries( xSeries ) );
 if( ChartTypeHelper::isSupportingSymbolProperties( xChartType, 2 ) )
 return beans::PropertyState_DIRECT_VALUE;
diff --git a/chart2/source/controller/inc/SeriesOptionsItemConverter.hxx 
b/chart2/source/controller/inc/SeriesOptionsItemConverter.hxx
index 886e6ec91b83..b55457ac5b30 100644
--- a/chart2/source/controller/inc/SeriesOptionsItemConverter.hxx
+++ b/chart2/source/controller/inc/SeriesOptionsItemConverter.hxx
@@ -27,6 +27,7 @@ namespace com::sun::star::frame { class XModel; }
 namespace com::sun::star::uno { class XComponentContext; }
 namespace chart { class ChartModel; }
 namespace chart { class BaseCoordinateSystem; }
+namespace chart { class DataSeries; }
 
 namespace chart::wrapper
 {
@@ -37,7 +38,7 @@ public:
 SeriesOptionsItemConverter(
 const rtl::Reference<::chart::ChartModel> & xChartModel,
 css::uno::Reference< css::uno::XComponentContext > xContext,
-const css::uno::Reference< css::beans::XPropertySet > & rPropertySet,
+const rtl::Reference<::chart::DataSeries> & rPropertySet,
 SfxItemPool& rItemPool );
 virtual ~SeriesOptionsItemConverter() override;
 
diff --git a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx 
b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
index f415cd194386..e71646cf7a8f 100644
--- a/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
+++ b/chart2/source/controller/itemsetwrapper/DataPointItemConverter.cxx
@@ -234,8 +234,10 @@ DataPointItemConverter::DataPointItemConverter(
 m_aConverters.emplace_back( new 
CharacterPropertyItemConverter(rPropertySet, rItemPool, pRefSize, 
"ReferencePageSize"));
 if( bDataSeries )
 {
+assert(dynamic_cast(rPropertySet.get()));
 m_aConverters.emplace_back( new Sta

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

2023-03-21 Thread Michael Weghorn (via logerrit)
 winaccessibility/inc/AccObjectWinManager.hxx|4 ++--
 winaccessibility/source/service/AccObjectWinManager.cxx |8 
 2 files changed, 6 insertions(+), 6 deletions(-)

New commits:
commit 02edbf7dc7cf1a3bcc1b943e79c027e3c144d544
Author: Michael Weghorn 
AuthorDate: Tue Mar 21 19:32:11 2023 +0100
Commit: Michael Weghorn 
CommitDate: Wed Mar 22 06:36:58 2023 +

tdf#154303 wina11y: Don't truncate 64-bit states

commit 8d8e6c84e512c1a8b33aac75965b84481d1a1d13
Date:   Sun Jul 3 20:29:28 2022 +0200

[API CHANGE] Drop css::accessibility::XAccessibleStateSet

had introduced the use of a 64-bit integer with
bit flags for the accessible states.

However, `AccObjectWinManager::DecreaseState` and
`AccObjectWinManager::IncreaseState` were still using
unsigned short, resulting in truncation.

As one result, the `AccessibleEventId::STATE_CHANGED`
event with state `AccessibleStateType::VISIBLE`
from `OAccessibleMenuBaseComponent::SetVisible` would no
longer result in the `STATE_SYSTEM_INVISIBLE` being unset
and thus the the JAWS screen reader would no longer announce
the focused (but considered invisible) menu item.

Fix this by switching the param for those two methods to
sal_Int64 as well.

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

diff --git a/winaccessibility/inc/AccObjectWinManager.hxx 
b/winaccessibility/inc/AccObjectWinManager.hxx
index 76e3e548fcdf..5c162dfed1ea 100644
--- a/winaccessibility/inc/AccObjectWinManager.hxx
+++ b/winaccessibility/inc/AccObjectWinManager.hxx
@@ -102,8 +102,8 @@ public:
 
 LRESULT Get_ToATInterface(HWND hWnd, long lParam, WPARAM wParam);
 
-void  DecreaseState( css::accessibility::XAccessible* pXAcc,unsigned short 
pState );
-void  IncreaseState( css::accessibility::XAccessible* pXAcc,unsigned short 
pState );
+void  DecreaseState(css::accessibility::XAccessible* pXAcc, sal_Int64 
nState);
+void  IncreaseState(css::accessibility::XAccessible* pXAcc, sal_Int64 
nState);
 void  UpdateState( css::accessibility::XAccessible* pXAcc );
 
 void  SetValue( css::accessibility::XAccessible* pXAcc, css::uno::Any pAny 
);
diff --git a/winaccessibility/source/service/AccObjectWinManager.cxx 
b/winaccessibility/source/service/AccObjectWinManager.cxx
index de527270a8e0..b0ee01525de9 100644
--- a/winaccessibility/source/service/AccObjectWinManager.cxx
+++ b/winaccessibility/source/service/AccObjectWinManager.cxx
@@ -796,11 +796,11 @@ AccObjectWinManager::CreateAccEventListener(XAccessible* 
pXAcc)
* @param pState Changed state.
* @return
*/
-void AccObjectWinManager::DecreaseState( XAccessible* pXAcc,unsigned short 
pState )
+void AccObjectWinManager::DecreaseState(XAccessible* pXAcc, sal_Int64 nState)
 {
 AccObject* pAccObj = GetAccObjByXAcc( pXAcc );
 if( pAccObj )
-pAccObj->DecreaseState( pState );
+pAccObj->DecreaseState(nState);
 }
 
 /**
@@ -811,11 +811,11 @@ void AccObjectWinManager::DecreaseState( XAccessible* 
pXAcc,unsigned short pStat
* @param pState Changed state.
* @return
*/
-void AccObjectWinManager::IncreaseState( XAccessible* pXAcc,unsigned short 
pState )
+void AccObjectWinManager::IncreaseState(XAccessible* pXAcc, sal_Int64 nState)
 {
 AccObject* pAccObj = GetAccObjByXAcc( pXAcc );
 if( pAccObj )
-pAccObj->IncreaseState( pState );
+pAccObj->IncreaseState(nState);
 }
 
 void  AccObjectWinManager::UpdateState( css::accessibility::XAccessible* pXAcc 
)