[Libreoffice-bugs] [Bug 115481] Paper size output does not match the paper size selected

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

--- Comment #24 from João Paulo  ---
Created attachment 181970
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181970=edit
Print dialog shows LibreOffice will try to print as an A4 document in landscape
instead of Envelope C6/5

Sorry, I sent the same screenshot three times.  Those new screenshots are
correct.

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

[Libreoffice-bugs] [Bug 115481] Paper size output does not match the paper size selected

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

João Paulo  changed:

   What|Removed |Added

 Attachment #181842|0   |1
is obsolete||

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

[Libreoffice-bugs] [Bug 148626] Printing Envelopes

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

--- Comment #5 from João Paulo  ---
Created attachment 181969
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181969=edit
Print dialog shows incorrect paper size after creating a custom paper size on
the printer driver settings

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

[Libreoffice-bugs] [Bug 148626] Printing Envelopes

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

--- Comment #4 from João Paulo  ---
Created attachment 181968
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181968=edit
Print dialog shows LibreOffice will try to print as an A4 document in landscape
instead of Envelope C6/5

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

[Libreoffice-commits] core.git: Branch 'libreoffice-7-4' - helpcontent2

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

New commits:
commit 8f2c68e5d85602b2472db5e5836c934cae5805a2
Author: Olivier Hallot 
AuthorDate: Tue Aug 23 01:20:02 2022 -0300
Commit: Gerrit Code Review 
CommitDate: Tue Aug 23 06:20:02 2022 +0200

Update git submodules

* Update helpcontent2 from branch 'libreoffice-7-4'
  to 278086ec01e796c7668de4760a1f2cde09bbf370
  - Update LO7.4 new features video

(no translation needed)

Change-Id: Ic1290ef2d90aca04f1bf6df1aae5d19f64f5919f
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/138486
Tested-by: Jenkins
Reviewed-by: Olivier Hallot 
(cherry picked from commit a37e043d12c969f3c12686ff09bc9087130b91aa)
Reviewed-on: https://gerrit.libreoffice.org/c/help/+/138425
Reviewed-by: Adolfo Jayme Barrientos 

diff --git a/helpcontent2 b/helpcontent2
index a4989eb67de8..278086ec01e7 16
--- a/helpcontent2
+++ b/helpcontent2
@@ -1 +1 @@
-Subproject commit a4989eb67de865d030f3a5012734da7b72d7583b
+Subproject commit 278086ec01e796c7668de4760a1f2cde09bbf370


[Libreoffice-bugs] [Bug 107733] [META] Hyperlink bugs and enhancements

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

Bug 43431 Summary: EDITING: Hyperlinks do not work if DRAW picture in 
background (wrongly select object)
https://bugs.documentfoundation.org/show_bug.cgi?id=43431

   What|Removed |Added

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

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

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

2022-08-22 Thread Caolán McNamara (via logerrit)
 vcl/unx/generic/gdi/cairotextrender.cxx |   19 ++-
 1 file changed, 10 insertions(+), 9 deletions(-)

New commits:
commit 4ec4fee0cf48142e2f4c594fe401f72872dc1bd9
Author: Caolán McNamara 
AuthorDate: Mon Aug 22 13:22:04 2022 +0100
Commit: Adolfo Jayme Barrientos 
CommitDate: Tue Aug 23 06:18:52 2022 +0200

tdf#150507 set hint style even if the platform returns no default font 
options

if the platform (e.g. gen/kf5) doesn't return anything for
GetCairoFontOptions() but we have need for specific font options then we
also need to set them

Change-Id: Ia805341d2ec75bdaf3d9de62955a247c33b106d1
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138543
Tested-by: Jenkins
Reviewed-by: Adolfo Jayme Barrientos 

diff --git a/vcl/unx/generic/gdi/cairotextrender.cxx 
b/vcl/unx/generic/gdi/cairotextrender.cxx
index 7300562d5d68..1e2e75c01d79 100644
--- a/vcl/unx/generic/gdi/cairotextrender.cxx
+++ b/vcl/unx/generic/gdi/cairotextrender.cxx
@@ -191,14 +191,15 @@ void CairoTextRender::DrawTextLayout(const 
GenericSalLayout& rLayout, const SalG
 __lsan_disable();
 #endif
 
-if (const cairo_font_options_t* pFontOptions = 
GetSalInstance()->GetCairoFontOptions())
-{
-const StyleSettings& rStyleSettings = 
Application::GetSettings().GetStyleSettings();
-bool bDisableAA = !rStyleSettings.GetUseFontAAFromSystem() && 
!rGraphics.getAntiAlias();
+const StyleSettings& rStyleSettings = 
Application::GetSettings().GetStyleSettings();
+const bool bDisableAA = !rStyleSettings.GetUseFontAAFromSystem() && 
!rGraphics.getAntiAlias();
+const bool bResolutionIndependentLayoutEnabled = 
rGraphics.getTextRenderModeForResolutionIndependentLayoutEnabled();
 
-const bool bResolutionIndependentLayoutEnabled = 
rGraphics.getTextRenderModeForResolutionIndependentLayoutEnabled();
-cairo_hint_style_t eHintStyle = 
cairo_font_options_get_hint_style(pFontOptions);
-cairo_hint_metrics_t eHintMetricsStyle = 
cairo_font_options_get_hint_metrics(pFontOptions);
+const cairo_font_options_t* pFontOptions = 
GetSalInstance()->GetCairoFontOptions();
+if (pFontOptions || bDisableAA || bResolutionIndependentLayoutEnabled)
+{
+cairo_hint_style_t eHintStyle = pFontOptions ? 
cairo_font_options_get_hint_style(pFontOptions) : CAIRO_HINT_STYLE_DEFAULT;
+cairo_hint_metrics_t eHintMetricsStyle = pFontOptions ? 
cairo_font_options_get_hint_metrics(pFontOptions) : CAIRO_HINT_METRICS_DEFAULT;
 bool bAllowedHintStyle = !bResolutionIndependentLayoutEnabled || 
(eHintStyle == CAIRO_HINT_STYLE_NONE);
 bool bAllowedHintMetricStyle = !bResolutionIndependentLayoutEnabled || 
(eHintMetricsStyle == CAIRO_HINT_METRICS_OFF);
 
@@ -206,7 +207,7 @@ void CairoTextRender::DrawTextLayout(const 
GenericSalLayout& rLayout, const SalG
 {
 // Disable font AA in case global AA setting is supposed to affect
 // font rendering (not the default) and AA is disabled.
-cairo_font_options_t* pOptions = 
cairo_font_options_copy(pFontOptions);
+cairo_font_options_t* pOptions = pFontOptions ? 
cairo_font_options_copy(pFontOptions) : cairo_font_options_create();
 if (bDisableAA)
 cairo_font_options_set_antialias(pOptions, 
CAIRO_ANTIALIAS_NONE);
 if (!bAllowedHintMetricStyle)
@@ -216,7 +217,7 @@ void CairoTextRender::DrawTextLayout(const 
GenericSalLayout& rLayout, const SalG
 cairo_set_font_options(cr, pOptions);
 cairo_font_options_destroy(pOptions);
 }
-else
+else if (pFontOptions)
 cairo_set_font_options(cr, pFontOptions);
 }
 


[Libreoffice-bugs] [Bug 43431] EDITING: Hyperlinks do not work if DRAW picture in background (wrongly select object)

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

Rainer Bielefeld Retired  changed:

   What|Removed |Added

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

--- Comment #11 from Rainer Bielefeld Retired  
---
No longer reproducible at least since LibO Version 4.0.0.3 (Build ID:
7545bee9c2a0782548772a21bc84a9dcc583b89 (WIN 10)

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

[Libreoffice-bugs] [Bug 150295] JDBC exception stack trace lost

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

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

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

[Libreoffice-bugs] [Bug 150305] A marked Fontwork shape is not in 'Current Selection' in Development tools

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

QA Administrators  changed:

   What|Removed |Added

 Whiteboard|| QA:needsComment

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

[Libreoffice-bugs] [Bug 147613] Table properties dialog: Tab columns does not show more than 4 columns anymore

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

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

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

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

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

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

Warm Regards,
QA Team

MassPing-NeedInfo-Ping

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

[Libreoffice-bugs] [Bug 150531] Incorrectly rendered SVG images with Skia enabled

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

QA Administrators  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED

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

[Libreoffice-bugs] [Bug 150531] Incorrectly rendered SVG images with Skia enabled

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

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

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

[Libreoffice-bugs] [Bug 125431] An unwanted image appears in exported PDF

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

--- Comment #18 from QA Administrators  ---
Dear 510jrb2301,

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

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

If you have time, please do the following:

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

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

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

Please DO NOT

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


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

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


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

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

Warm Regards,
QA Team

MassPing-UntouchedBug

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

[Libreoffice-bugs] [Bug 143144] Input list field: Add confirmation dialog in order to prevent loss of data

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

Dieter  changed:

   What|Removed |Added

   Priority|medium  |low

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

[Libreoffice-bugs] [Bug 143144] Input list field: Add confirmation dialog in order to prevent loss of data

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

Dieter  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |WONTFIX
 CC||dgp-m...@gmx.de

--- Comment #7 from Dieter  ---
No further input since more than a year, so nobody suppoted that idea. I agree
with Heiko and Buovjaga, that it's not useful to implement every wish. So let's
close it as WF.

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

[Libreoffice-ux-advise] [Bug 142446] In folding outlines, enable specifying, document-wide, the level to which text should be collapsed.

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

Dieter  changed:

   What|Removed |Added

   Keywords||needsUXEval
 CC||dgp-m...@gmx.de,
   ||libreoffice-ux-advise@lists
   ||.freedesktop.org,
   ||rayk...@gmail.com

--- Comment #2 from Dieter  ---
Let's ask design-team and Jim Raykowski.

cc: Design-Team, Jim Raykowski

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

[Libreoffice-bugs] [Bug 142446] In folding outlines, enable specifying, document-wide, the level to which text should be collapsed.

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

Dieter  changed:

   What|Removed |Added

   Keywords||needsUXEval
 CC||dgp-m...@gmx.de,
   ||libreoffice-ux-advise@lists
   ||.freedesktop.org,
   ||rayk...@gmail.com

--- Comment #2 from Dieter  ---
Let's ask design-team and Jim Raykowski.

cc: Design-Team, Jim Raykowski

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

[Libreoffice-bugs] [Bug 150322] EDITING: An ordered list will reset the footnote sequence

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

Dieter  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEEDINFO

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

[Libreoffice-bugs] [Bug 150322] EDITING: An ordered list will reset the footnote sequence

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

Dieter  changed:

   What|Removed |Added

 CC||dgp-m...@gmx.de

--- Comment #1 from Dieter  ---
Created attachment 181967
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181967=edit
Sample document

I can't confirm it with

Version: 7.4.0.3 (x64) / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 4; OS: Windows 10.0 Build 19044; UI render: Skia/Raster; VCL: win
Locale: de-DE (de_DE); UI: en-GB
Calc: CL

Steps
1. Open attached document
2. Mark fourth item -> context menu -> list -> Restart Numbering

Actual and expected result
No change in footnotes

KK, could you please retest with document?
=> NEEDINFO

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

[Libreoffice-bugs] [Bug 103369] [META] Bullet, numbered, and outline list bugs and enhancements

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

Bug 57317 Summary: Writer FILESAVE: bullets list saved as numbered list (in 
specific ODT)
https://bugs.documentfoundation.org/show_bug.cgi?id=57317

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 57317] Writer FILESAVE: bullets list saved as numbered list (in specific ODT)

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

Justin L  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED
   Assignee|libreoffice-b...@lists.free |jl...@mail.com
   |desktop.org |

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

[Libreoffice-bugs] [Bug 57317] Writer FILESAVE: bullets list saved as numbered list (in specific ODT)

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

--- Comment #10 from Commit Notification 
 ---
Justin Luth committed a patch related to this issue.
It has been pushed to "master":

https://git.libreoffice.org/core/commit/46dc9f3bbac67e9240adc44ab017f905482ef786

tdf#57317 odf export: avoid autolist conflict with NumberingStyles

It will be available in 7.5.0.

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

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

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

[Libreoffice-bugs] [Bug 57317] Writer FILESAVE: bullets list saved as numbered list (in specific ODT)

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

Commit Notification  changed:

   What|Removed |Added

 Whiteboard||target:7.5.0

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

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

2022-08-22 Thread Justin Luth (via logerrit)
 include/xmloff/xmlnume.hxx|4 --
 sw/qa/extras/odfexport/data/tdf57317_autoListName.odt |binary
 sw/qa/extras/odfexport/odfexport2.cxx |   31 ++
 xmloff/source/style/xmlnume.cxx   |9 +
 xmloff/source/text/XMLTextListAutoStylePool.cxx   |   20 +++
 xmloff/source/text/txtstyle.cxx   |2 -
 6 files changed, 55 insertions(+), 11 deletions(-)

New commits:
commit 46dc9f3bbac67e9240adc44ab017f905482ef786
Author: Justin Luth 
AuthorDate: Mon Aug 22 12:52:22 2022 -0400
Commit: Justin Luth 
CommitDate: Tue Aug 23 04:09:04 2022 +0200

tdf#57317 odf export: avoid autolist conflict with NumberingStyles

Somehow you can have autoStyles writing into the styles.xml.
Well, the code to prevent duplicates for that needs to exist
for autoStyles written into document.xml as well.

Change-Id: I9602b7023a1ef009ad644374dc5081afb8a76916
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138703
Tested-by: Jenkins
Reviewed-by: Justin Luth 

diff --git a/include/xmloff/xmlnume.hxx b/include/xmloff/xmlnume.hxx
index 68818f1076ff..85f78047ee80 100644
--- a/include/xmloff/xmlnume.hxx
+++ b/include/xmloff/xmlnume.hxx
@@ -63,9 +63,7 @@ public:
 const css::uno::Reference< css::container::XIndexReplace > & 
xNumRule,
 bool bOutline=false );
 
-void exportStyles( bool bUsed,
-   XMLTextListAutoStylePool *pPool,
-   bool bExportChapterNumbering );
+void exportStyles(bool bUsed, bool bExportChapterNumbering);
 void exportNumberingRule(
 const OUString& rName, bool bIsHidden,
 const css::uno::Reference< css::container::XIndexReplace > & 
xNumRule );
diff --git a/sw/qa/extras/odfexport/data/tdf57317_autoListName.odt 
b/sw/qa/extras/odfexport/data/tdf57317_autoListName.odt
new file mode 100644
index ..3aafbc519387
Binary files /dev/null and 
b/sw/qa/extras/odfexport/data/tdf57317_autoListName.odt differ
diff --git a/sw/qa/extras/odfexport/odfexport2.cxx 
b/sw/qa/extras/odfexport/odfexport2.cxx
index 2a2f015f6b84..5d2f91b92d5d 100644
--- a/sw/qa/extras/odfexport/odfexport2.cxx
+++ b/sw/qa/extras/odfexport/odfexport2.cxx
@@ -94,6 +94,37 @@ DECLARE_ODFEXPORT_TEST(testTdf143605, "tdf143605.odt")
 CPPUNIT_ASSERT_EQUAL(OUString("."), getProperty(getParagraph(1), 
"ListLabelString"));
 }
 
+CPPUNIT_TEST_FIXTURE(Test, testTdf57317_autoListName)
+{
+load(mpTestDocumentPath, "tdf57317_autoListName.odt");
+// The list style (from styles.xml) overrides a duplicate named auto-style
+//uno::Any aNumStyle = getStyles("NumberingStyles")->getByName("L1");
+//CPPUNIT_ASSERT(aNumStyle.hasValue());
+uno::Reference xPara(getParagraph(1), uno::UNO_QUERY);
+CPPUNIT_ASSERT_EQUAL(OUString(">1<"), getProperty(xPara, 
"ListLabelString"));
+CPPUNIT_ASSERT_EQUAL(OUString("L1"), getProperty(xPara, 
"NumberingStyleName"));
+
+dispatchCommand(mxComponent, ".uno:SelectAll", {});
+dispatchCommand(mxComponent, ".uno:DefaultBullet", {});
+
+// This was failing with a duplicate auto numbering style name of L1 
instead of a unique name,
+// thus it was showing the same info as before the bullet modification.
+reload(mpFilter, "");
+xPara.set(getParagraph(1), uno::UNO_QUERY);
+CPPUNIT_ASSERT_EQUAL(OUString(""), getProperty(xPara, 
"ListLabelString"));
+
+uno::Reference 
xLevels(xPara->getPropertyValue("NumberingRules"),
+uno::UNO_QUERY);
+uno::Sequence aProps;
+xLevels->getByIndex(0) >>= aProps;
+for (beans::PropertyValue const& rProp : std::as_const(aProps))
+{
+if (rProp.Name == "BulletChar")
+return;
+}
+CPPUNIT_FAIL("no BulletChar property");
+}
+
 CPPUNIT_TEST_FIXTURE(Test, testListFormatDocx)
 {
 loadAndReload("listformat.docx");
diff --git a/xmloff/source/style/xmlnume.cxx b/xmloff/source/style/xmlnume.cxx
index 682fc648af41..9ec9350374db 100644
--- a/xmloff/source/style/xmlnume.cxx
+++ b/xmloff/source/style/xmlnume.cxx
@@ -782,9 +782,7 @@ void SvxXMLNumRuleExport::exportOutline()
 }
 }
 
-void SvxXMLNumRuleExport::exportStyles( bool bUsed,
-XMLTextListAutoStylePool *pPool,
-bool bExportChapterNumbering )
+void SvxXMLNumRuleExport::exportStyles( bool bUsed, bool 
bExportChapterNumbering )
 {
 if( bExportChapterNumbering )
 exportOutline();
@@ -821,11 +819,8 @@ void SvxXMLNumRuleExport::exportStyles( bool bUsed,
 xStyles->getByIndex( i ) >>= xStyle;
 
 if( !bUsed || xStyle->isInUse() )
-{
 exportStyle( xStyle );
-if( pPool )
-pPool->RegisterName( xStyle->getName() );
-}
+
 }
 }
 
diff --git a/xmloff/source/text/XMLTextListAutoStylePool.cxx 

[Libreoffice-bugs] [Bug 150556] New: Libreoffice 7.3.5.2 x64 can't open or save ODS files

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

Bug ID: 150556
   Summary: Libreoffice 7.3.5.2 x64 can't open or save ODS files
   Product: LibreOffice
   Version: 7.4.0.3 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: jer...@creative-afternoon.com

Description:
Libreoffice 7.3.5.2 x64 problem
LO will not start as normal - blank screen with rotating whirligig
LO will start in safe mode
Writer opens new doc and can save it - also recovers doc at next
startup(?)
Presentation ditto
Drawing ditto
Formula does not open and hangs LO
Database opens new doc and can save it
Spreadsheet opens new doc and can save it but NOT reopen it
Tried repair in control panel programs and features - trundled but did not work
After repair LO does load and will create ODS file but cannot open one (app
hangs)
Note the problem with ODS files was happening before I updated to 7.3.5.2

Steps to Reproduce:
1.Start W10
2.Open LibreOffice
3.Open new ODS
4: App hangs and has to be shut down

Actual Results:
App hangs

Expected Results:
Calc would fileopen


Reproducible: Always


User Profile Reset: Yes



Additional Info:
gerrit.libreoffice.org / core / 184fe81b8c8c30d8b5082578aee2fed2ea847c01

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

[Libreoffice-bugs] [Bug 150555] Crash after resizing Writer's main window

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

Przemas  changed:

   What|Removed |Added

   Assignee|libreoffice-b...@lists.free |przemaspierw...@gmail.com
   |desktop.org |

--- Comment #1 from Przemas  ---
Created attachment 181966
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181966=edit
LO logo after bug

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

[Libreoffice-bugs] [Bug 106306] RTL: Wrong text language detection for punctuation at the beginning of sentence

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

--- Comment #15 from Hossein  ---
Still reproducible with the latest LO 7.5 master:

Version: 7.5.0.0.alpha0+ / LibreOffice Community
Build ID: 947a6455d23bff290319313734c8c30e8f495773
CPU threads: 8; OS: Linux 5.15; UI render: default; VCL: gtk3
Locale: fa-IR (en_US.UTF-8); UI: en-US
Calc: threaded

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

[Libreoffice-bugs] [Bug 150555] New: Crash after resizing Writer's main window

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

Bug ID: 150555
   Summary: Crash after resizing Writer's main window
   Product: LibreOffice
   Version: 7.4.0.3 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Writer
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: przemaspierw...@gmail.com

Description:
After multiple resizing of the main window painting is distorted. No letters no
icons only random points. After that "show" Writer is sometimes crashed.

Steps to Reproduce:
1.Run Writer
2.Resize window horizontally and vertically several times
3.

Actual Results:
no letters no icons even in menu only non-client (term from Win API) elements
exist

Expected Results:
normal rendering client of the window


Reproducible: Always


User Profile Reset: Yes



Additional Info:
Version: 7.4.0.3 (x64) / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 4; OS: Windows 6.3 Build 9600; UI render: Skia/Vulkan; VCL: win
Locale: pl-PL (pl_PL); UI: pl-PL
Calc: CL

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

[Libreoffice-bugs] [Bug 91400] LibreOffice doesn’t use system font since OS X 10.10+

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

خالد حسني  changed:

   What|Removed |Added

 OS|All |macOS (All)

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

[Libreoffice-bugs] [Bug 150554] Specific image anchoring properties (alignment & keep text) change when saving file to .docx

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

Dave Seitman  changed:

   What|Removed |Added

   Keywords||filter:docx

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

[Libreoffice-bugs] [Bug 52582] Problem with combining unencoded characters in Brahmi Graphite font

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

--- Comment #11 from خالد حسني  ---
It is been 10 years and these combining marks are included in Unicode 14, so
the original issue is no longer reproducible. I think I know what was breaking
the encoded combining marks, though, so if you can make a different test with
characters that are still unencoded, I might be able to finally fix it.

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

[Libreoffice-bugs] [Bug 150554] Specific image anchoring properties (alignment & keep text) change when saving file to .docx

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

--- Comment #5 from Dave Seitman  ---
Created attachment 181965
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181965=edit
screenshot of document page showing image locations changed

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

[Libreoffice-bugs] [Bug 150554] Specific image anchoring properties (alignment & keep text) change when saving file to .docx

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

--- Comment #4 from Dave Seitman  ---
Created attachment 181964
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181964=edit
screenshot to show correct placement of images

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

[Libreoffice-bugs] [Bug 150554] Specific image anchoring properties (alignment & keep text) change when saving file to .docx

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

--- Comment #3 from Dave Seitman  ---
Created attachment 181963
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181963=edit
File I was working with - properties changed when previously saved

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

[Libreoffice-bugs] [Bug 150554] Specific image anchoring properties (alignment & keep text) change when saving file to .docx

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

--- Comment #2 from Dave Seitman  ---
Created attachment 181962
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181962=edit
File with properties altered after conversion from .odt to .docx

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

[Libreoffice-bugs] [Bug 150554] Specific image anchoring properties (alignment & keep text) change when saving file to .docx

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

--- Comment #1 from Dave Seitman  ---
Created attachment 181961
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181961=edit
File with "correct" image placement properties

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

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

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

Bug 119342 Summary: EDITING: Pasted HTML table into Calc can't optimize width
https://bugs.documentfoundation.org/show_bug.cgi?id=119342

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |INSUFFICIENTDATA

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

[Libreoffice-bugs] [Bug 119342] EDITING: Pasted HTML table into Calc can't optimize width

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

خالد حسني  changed:

   What|Removed |Added

 Resolution|--- |INSUFFICIENTDATA
 Status|NEW |RESOLVED

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

[Libreoffice-bugs] [Bug 150167] createUnoService returns wrong type for CharacterProperties and ParagraphProperties

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

Maxim Monastirsky  changed:

   What|Removed |Added

 CC||momonas...@gmail.com

--- Comment #8 from Maxim Monastirsky  ---
The term "service" used here is a bit misleading. There is no such thing as a
"ParagraphProperties service" that you can instantiate. ParagraphProperties is
just a *specification* that can be implemented by different components (like
paragraphs, cells etc.). That means that if such "service" references some
interfaces or properties, you can expect them to be present in components that
claim to implement this service (among others). That's it.

Regarding createUnoService, I believe it just picks a random component whose
entry in a .component file has that service listed.

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

[Libreoffice-bugs] [Bug 83066] [META] CJK (Chinese, Japanese, Korean, and Vietnamese) language issues

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

Bug 114272 Summary: [macOS] Some Asian characters are garbled with western 
fonts.
https://bugs.documentfoundation.org/show_bug.cgi?id=114272

   What|Removed |Added

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

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

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

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

Bug 114272 Summary: [macOS] Some Asian characters are garbled with western 
fonts.
https://bugs.documentfoundation.org/show_bug.cgi?id=114272

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 114272] [macOS] Some Asian characters are garbled with western fonts.

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

خالد حسني  changed:

   What|Removed |Added

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

--- Comment #22 from خالد حسني  ---
Can’t reproduce this with 7.3.5 on macOS. Please reopen if still reproducible
on recent LibreOffice.

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

[Libreoffice-bugs] [Bug 142020] Cannot combine ï with a dot below in most of the fonts

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

خالد حسني  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |NOTOURBUG

--- Comment #13 from خالد حسني  ---
It is up to the fonts to handle such combining marks, we have no say on this.

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

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

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

Bug 142020 Summary: Cannot combine ï with a dot below in most of the fonts
https://bugs.documentfoundation.org/show_bug.cgi?id=142020

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |NOTOURBUG

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

[Libreoffice-bugs] [Bug 150280] After pasting image 3x, save, close and reopen, middle Image not appearing

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

Scott Clewell  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED
Summary|Image not appearing |After pasting image 3x,
   ||save, close and reopen,
   ||middle Image not appearing

--- Comment #8 from Scott Clewell  ---
I understand. I've added more of your details in the bug summary and placed
this back to "UNCONFIRMED" so others may help out.
Thank you

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

[Libreoffice-bugs] [Bug 150167] createUnoService returns wrong type for CharacterProperties and ParagraphProperties

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

--- Comment #7 from eseb63  ---
maybe, but "why the createUnoService does not return the good service" is a
question for the support team i think ?

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

[Libreoffice-ux-advise] [Bug 127066] UI: Font size in input line (formular bar) larger than in other UI elements

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

V Stuart Foote  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 101646] UI option "Scaling" was removed

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

V Stuart Foote  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 150552] UI fonts rendered too small on high resolution retina displays using macOS

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

V Stuart Foote  changed:

   What|Removed |Added

 CC||vstuart.fo...@utsa.edu
   See Also||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=10
   ||1646,
   ||https://bugs.documentfounda
   ||tion.org/show_bug.cgi?id=12
   ||7066

--- Comment #3 from V Stuart Foote  ---
Reasonable, but hope we don't have a repeat of bug 127066, but probably not
since in the interim the inputlince has been welded.

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

[Libreoffice-bugs] [Bug 150553] EDITING: Text corruption when editing text field containing specific characters

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

--- Comment #2 from Eric Lasota  ---
Forgot to mention: This does not happen if I press the left or right arrow keys
to move the editing carat before step 7.

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

[Libreoffice-bugs] [Bug 150553] EDITING: Text corruption when editing text field containing specific characters

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

--- Comment #1 from Eric Lasota  ---
Created attachment 181960
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181960=edit
Test document

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

[Libreoffice-bugs] [Bug 150553] New: EDITING: Text corruption when editing text field containing specific characters

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

Bug ID: 150553
   Summary: EDITING: Text corruption when editing text field
containing specific characters
   Product: LibreOffice
   Version: 7.4.0.3 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: ejlas...@gmail.com

Description:
Double-clicking a text cell to edit it may put the character in some kind of
invalid "mid-character" position that corrupts the text.  See reproduction
steps.

Steps to Reproduce:
1. Install LibreOffice 7.4.0.3 on Windows.
2. Open the attached CSV.  It contains 1 cel with 6 consecutive musical note
characters.
3. In the import dialog, ensure the character set is set to UTF-8
4. Notice that the cell preview displays the content incorrectly at this step.
5. Click OK
6. Double-click in cell A1 anywhere between two of the note characters.
7. Press the "a" key

Actual Results:
The character preceding the editing carat is replaced with 2 damaged characters
and an "a" character between them.  Additionally, the cell edit box above the
spreadsheet will no longer match the cell contents.

Expected Results:
The "a" character should be inserted between the note characters where the
editing carat indicates.


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
Version: 7.4.0.3 (x64) / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 8; OS: Windows 10.0 Build 19044; UI render: Skia/Vulkan; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: threaded

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

[Libreoffice-bugs] [Bug 150550] After drawing a shape, the shape tool should be unselected from the sidebar

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

--- Comment #2 from Maxim Monastirsky  ---
(In reply to Rafael Lima from comment #1)
> If you click the tool twice, it will remain enabled and you can
> draw as many instances of the same shape as you like.
> 
> Is this indeed a feature?
Yes.

> I could not find documentation for it.
Seems to be documented at
https://help.libreoffice.org/latest/en-US/text/simpress/main0210.html

> [1] https://help.libreoffice.org/latest/en-US/text/shared/02/0114.html
That page seems to be for Writer and Calc only.

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

[Libreoffice-bugs] [Bug 150552] UI fonts rendered too small on high resolution retina displays using macOS

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

--- Comment #2 from Thorsten Wagner  ---
Created attachment 181959
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181959=edit
Screenshot 2

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

[Libreoffice-bugs] [Bug 150552] UI fonts rendered too small on high resolution retina displays using macOS

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

--- Comment #1 from Thorsten Wagner  ---
Created attachment 181958
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181958=edit
Screenshot 1

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

[Libreoffice-bugs] [Bug 150552] New: UI fonts rendered too small on high resolution retina displays using macOS

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

Bug ID: 150552
   Summary: UI fonts rendered too small on high resolution retina
displays using macOS
   Product: LibreOffice
   Version: 7.5.0.0 alpha0+ Master
  Hardware: All
OS: macOS (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: UI
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: thorsten.wagne...@gmail.com

Description:
UI fonts are rendered too small on laptop retina displays with native
resolution greater than 220 ppi (see screenshot 1, red boxes). Font rendering
within document area is ok, because these fonts have to be rendered at their
physical extent and independent from display resolution. For better readability
UI font rendering should be like in screenshot 2, red boxes with larger fonts
similar to font rendering on external displays.

Steps to Reproduce:
1. Open any LO application like Writer, Calc, Impress, Draw
2. Look at statusbar or sidebar

Actual Results:
UI fonts are rendered too small on displays with native resolution greater than
220 ppi (see screenshot 1, red boxes).

Expected Results:
UI fonts should be rendered larger and as large as on external displays or on
displays with native resolution less or equal than 220 ppi (see screenshot 2,
red boxese).


Reproducible: Always


User Profile Reset: No



Additional Info:
I will provide a patch resulting in screenshot 2 in short.

Reason is VCL font handling for UI fonts using integer values in points (pt).
Due to a rounding issue, fonts are rendered using a smaller height after
recalculation of font height for display resolution.

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

[Libreoffice-bugs] [Bug 150547] Unusable documentation for scripting

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

--- Comment #1 from Rafael Lima  ---
Documentation about scripting in Basic/Python has improved significantly in
recent years, but still documenting the entire API is a daunting task.

TBH it's unrealistic that we'll have help pages and examples for every method
available in the API.

If you want to write scripts using Basic, I recommend you check out Pitonyak's
book, which is a wonderful learning resource.

https://www.pitonyak.org/oo.php

If you want to write Calc scripts, I also recommend checking out the Calc
service of the ScriptForge library (which is thoroughly documented):

https://help.libreoffice.org/latest/en-US/text/sbasic/shared/03/sf_calc.html

Finally, if you run into issues to which you cannot find an answer, post a
question at:

https://ask.libreoffice.org/

There are a lot of knowledgeable people willing to help with scripting
questions.

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

[Libreoffice-bugs] [Bug 150551] Exporting shape with gradient fill as PDF results in wrong output

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

Rafael Lima  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 150545] Incorrect handling of "Automatic" increment in gradient fill

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

Rafael Lima  changed:

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 150551] New: Exporting shape with gradient fill as PDF results in wrong output

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

Bug ID: 150551
   Summary: Exporting shape with gradient fill as PDF results in
wrong output
   Product: LibreOffice
   Version: 7.3.5.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Impress
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: rafael.palma.l...@gmail.com

Created attachment 181956
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181956=edit
Screenshot showing the problem

Export of shapes with gradient fill to PDF does not seem to work.

Steps to reproduce:
1) Create a blank presentation; insert a single shape to the slide
2) Right-click the shape - Area
3) Select "Gradient"; Type "Radial"; Increment "Automatic"; Center X/Y: 50% for
both; From color "Dark Blue 1"; To color "Dark Blue 3" (other colors show the
same behavior)
4) Click OK; notice that the gradient will look fine in Impress
5) Export to PDF; open the PDF file and notice that the gradient was not
exported correctly

See attached image comparing what I see in Impress and what I get as the PDF
output.

This bug is probably related to bug 150545.

System info

Version: 7.3.5.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 12; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.5-0ubuntu0.22.04.1
Calc: threaded

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

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

2022-08-22 Thread Caolán McNamara (via logerrit)
 sc/qa/uitest/inputLine/tdf131326.py |4 +++-
 sc/qa/uitest/inputLine/tdf54197.py  |2 ++
 sc/qa/uitest/inputLine/tdf67346.py  |2 ++
 3 files changed, 7 insertions(+), 1 deletion(-)

New commits:
commit 3cc6b870c48b23418f65d2c3cd6eab72ef0680de
Author: Caolán McNamara 
AuthorDate: Mon Aug 22 20:21:39 2022 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 22 22:30:01 2022 +0200

make an effort to fix the UITest_inputLine intermittent failure

debugging what normally happens I see that the EditEngine belonging to
this inputbar widget is thrown away by calc when the content of the cell
changes and is only recreated on the next "Paint" of the inputbar.

It's plausible that throwing processEventsToIdle at the problem solves
this.

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

diff --git a/sc/qa/uitest/inputLine/tdf131326.py 
b/sc/qa/uitest/inputLine/tdf131326.py
index 6fbfe25f02bd..90809ab42c79 100755
--- a/sc/qa/uitest/inputLine/tdf131326.py
+++ b/sc/qa/uitest/inputLine/tdf131326.py
@@ -17,10 +17,12 @@ class tdf131326(UITestCase):
 with self.ui_test.create_doc_in_start_center("calc"):
 xCalcDoc = self.xUITest.getTopFocusWindow()
 gridwin = xCalcDoc.getChild("grid_window")
-xInputWin = xCalcDoc.getChild("sc_input_window")
 
 enter_text_to_cell(gridwin, "A1", "=1")
 self.xUITest.executeCommand(".uno:ConvertFormulaToValue")
+xToolkit = 
self.xContext.ServiceManager.createInstance('com.sun.star.awt.Toolkit')
+xToolkit.processEventsToIdle()
+xInputWin = xCalcDoc.getChild("sc_input_window")
 self.assertEqual(get_state_as_dict(xInputWin)["Text"], "1")
 
 # vim: set shiftwidth=4 softtabstop=4 expandtab:
diff --git a/sc/qa/uitest/inputLine/tdf54197.py 
b/sc/qa/uitest/inputLine/tdf54197.py
index ba86982d91ca..a7a82d5c338f 100644
--- a/sc/qa/uitest/inputLine/tdf54197.py
+++ b/sc/qa/uitest/inputLine/tdf54197.py
@@ -29,6 +29,8 @@ class tdf54197(UITestCase):
 # The same text as above is displayed at cell A2, BUT input line 
is still blank
 self.assertEqual(get_cell_by_position(document, 0, 0, 
0).getString(), "t")
 self.assertEqual(get_cell_by_position(document, 0, 0, 
1).getString(), "t")
+xToolkit = 
self.xContext.ServiceManager.createInstance('com.sun.star.awt.Toolkit')
+xToolkit.processEventsToIdle()
 xInputWin = xCalcDoc.getChild("sc_input_window")
 self.assertEqual(get_state_as_dict(xInputWin)["Text"], "t")
 
diff --git a/sc/qa/uitest/inputLine/tdf67346.py 
b/sc/qa/uitest/inputLine/tdf67346.py
index 5d5ff32cefb5..cc9dc14d628c 100644
--- a/sc/qa/uitest/inputLine/tdf67346.py
+++ b/sc/qa/uitest/inputLine/tdf67346.py
@@ -26,6 +26,8 @@ class tdf67346(UITestCase):
 # Move focus to input window and copy the text from there
 gridwin.executeAction("TYPE", 
mkPropertyValues({"KEYCODE":"CTRL+SHIFT+F2"}))
 
+xToolkit = 
self.xContext.ServiceManager.createInstance('com.sun.star.awt.Toolkit')
+xToolkit.processEventsToIdle()
 xInputWin = xCalcDoc.getChild("sc_input_window")
 self.assertEqual('true', get_state_as_dict(xInputWin)["HasFocus"])
 


[Libreoffice-bugs] [Bug 150550] After drawing a shape, the shape tool should be unselected from the sidebar

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

--- Comment #1 from Rafael Lima  ---
By the way, while testing this bug I came across a (possibly undocumented)
feature. If you click the tool twice, it will remain enabled and you can draw
as many instances of the same shape as you like.

In such case, the highlight should remain visible.

Is this indeed a feature? I could not find documentation for it. If it is
indeed a feature, it should be documented in the help [1] via a tip or note.

[1] https://help.libreoffice.org/latest/en-US/text/shared/02/0114.html

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

[Libreoffice-bugs] [Bug 150550] New: After drawing a shape, the shape tool should be unselected from the sidebar

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

Bug ID: 150550
   Summary: After drawing a shape, the shape tool should be
unselected from the sidebar
   Product: LibreOffice
   Version: 7.3.5.2 release
  Hardware: All
OS: All
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Impress
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: rafael.palma.l...@gmail.com

Created attachment 181955
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181955=edit
Video showing the problem

After drawing a shape in Impress the shape tool should no longer be selected
(highlighted) because it is no longer enabled.

See attached video for more information.

Steps to reproduce:
1) Create a new Impress document
2) Open the "Shapes" sidebar
3) Click once in any of the shapes and draw it onto the canvas
4) Right after finishing to draw it, try to draw again; notice that it wont
draw a new shape, hence the tool should no longer be highlighted

The expected behavior would be to automatically remove the tool highlight from
the sidebar after finishing drawing the shape.

System info

Version: 7.3.5.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 12; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: pt-BR (pt_BR.UTF-8); UI: en-US
Ubuntu package version: 1:7.3.5-0ubuntu0.22.04.1
Calc: threaded

Also repro in

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

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

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

2022-08-22 Thread Caolán McNamara (via logerrit)
 sc/source/ui/navipi/navipi.cxx |   19 +--
 1 file changed, 9 insertions(+), 10 deletions(-)

New commits:
commit 6b68ac7d989dd34071c12ee64fe7c4cb99e37c09
Author: Caolán McNamara 
AuthorDate: Mon Aug 22 16:38:01 2022 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 22 21:52:50 2022 +0200

cid#1509273 Unchecked dynamic_cast

get ScDocument the way the rest of ScNavigatorDlg does

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

diff --git a/sc/source/ui/navipi/navipi.cxx b/sc/source/ui/navipi/navipi.cxx
index 44ba8fee2c26..6340c1a80f33 100644
--- a/sc/source/ui/navipi/navipi.cxx
+++ b/sc/source/ui/navipi/navipi.cxx
@@ -126,23 +126,22 @@ namespace
 
 IMPL_LINK(ScNavigatorDlg, ParseRowInputHdl, int*, result, bool)
 {
-SCCOL nCol;
+SCCOL nCol(0);
 
 OUString aStrCol = m_xEdCol->get_text();
 
 if (!aStrCol.isEmpty())
 {
-//  nKeyGroup is no longer set at VCL, in cause of lack of keyinput
+if (ScViewData* pData = GetViewData())
+{
+ScDocument& rDoc = pData->GetDocument();
 
-ScTabViewShell* pViewSh = dynamic_cast( 
SfxViewShell::Current() );
-auto& rDoc = pViewSh->GetViewData().GetDocument();
-if ( CharClass::isAsciiNumeric(aStrCol) )
-nCol = NumStrToAlpha( rDoc.GetSheetLimits(), aStrCol );
-else
-nCol = AlphaToNum( rDoc, aStrCol );
+if ( CharClass::isAsciiNumeric(aStrCol) )
+nCol = NumStrToAlpha( rDoc.GetSheetLimits(), aStrCol );
+else
+nCol = AlphaToNum( rDoc, aStrCol );
+}
 }
-else
-nCol = 0;
 
 *result = nCol;
 return true;


[Libreoffice-bugs] [Bug 150549] Libreoffice leaves "zombie process" running after closing any LibreOffice component, preventing new components from opening

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

--- Comment #2 from Dan  ---
Okay... after some more investigation, although safemode crashes for whatever
reason, running with " soffice.com
-env:UserInstallation=file:///c:/my-test-profile" yields the expected result
and renaming my actual profile so the lo creates a new profile, does fix the
issue. so it looks like "corrupt user profile" is to blame for this issue.

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

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

2022-08-22 Thread Caolán McNamara (via logerrit)
 formula/source/core/api/FormulaCompiler.cxx |1 +
 1 file changed, 1 insertion(+)

New commits:
commit cd816c0b4e698386a482f20c975281a0f3a7d332
Author: Caolán McNamara 
AuthorDate: Mon Aug 22 16:21:21 2022 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 22 21:23:50 2022 +0200

cid#1509296 silence Use after free

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

diff --git a/formula/source/core/api/FormulaCompiler.cxx 
b/formula/source/core/api/FormulaCompiler.cxx
index 591b28744289..55e65ae9fd7c 100644
--- a/formula/source/core/api/FormulaCompiler.cxx
+++ b/formula/source/core/api/FormulaCompiler.cxx
@@ -2053,6 +2053,7 @@ void FormulaCompiler::IntersectionLine()
 FormulaTokenRef pIntersect( new FormulaByteToken( 
ocIntersect));
 // Replace ocSpaces with ocIntersect so that when switching
 // formula syntax the correct operator string is created.
+// coverity[freed_arg : FALSE] - FormulaTokenRef has a ref so 
ReplaceToken won't delete pIntersect
 pArr->ReplaceToken( nCodeIndex, pIntersect.get(), 
FormulaTokenArray::ReplaceMode::CODE_ONLY);
 PutCode( pIntersect);
 }


[Libreoffice-bugs] [Bug 150549] Libreoffice leaves "zombie process" running after closing any LibreOffice component, preventing new components from opening

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

--- Comment #1 from Dan  ---
In case the "Safe-mode" crashing is related, here is the event log export for
that crash.

Faulting application name: soffice.bin, version: 7.4.0.3, time stamp:
0x62f401e3
Faulting module name: ucrtbase.dll, version: 10.0.22621.436, time stamp:
0xf5fc15a3
Exception code: 0xc409
Fault offset: 0x0007f61e
Faulting process id: 0x0x22A4
Faulting application start time: 0x0x1D8B659A5ECE055
Faulting application path: C:\Program Files\LibreOffice\program\soffice.bin
Faulting module path: C:\WINDOWS\System32\ucrtbase.dll
Report Id: c27f6d3d-33ac-4b0b-be9b-57e646074662
Faulting package full name: 
Faulting package-relative application ID:

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

Fwd: [core] tdf#148251 Use std::swap instead of using temporary values

2022-08-22 Thread Hao Liu
Which reviewer should I add to merge this patch?
https://gerrit.libreoffice.org/c/core/+/138256

-- Forwarded message -
发件人: Paris Oplopoios (via Code Review) 
Date: 2022年8月23日周二 00:30
Subject: [core] tdf#148251 Use std::swap instead of using temporary values
To: Hao Liu 


Attention is currently required from: Hao Liu.

Paris Oplopoios has posted comments on this change. (
https://gerrit.libreoffice.org/c/core/+/138256 )

Change subject: tdf#148251 Use std::swap instead of using temporary values
..


Patch Set 3: Code-Review+1

(1 comment)

Patchset:

PS3:
Looks good to me but don't have merge privileges so you're gonna have to
find a different reviewer



-- 
To view, visit https://gerrit.libreoffice.org/c/core/+/138256
To unsubscribe, or for help writing mail filters, visit
https://gerrit.libreoffice.org/settings

Gerrit-Project: core
Gerrit-Branch: master
Gerrit-Change-Id: Ibbc46bfb17d4d27dc40e63c65190b3f3a5f2d9c4
Gerrit-Change-Number: 138256
Gerrit-PatchSet: 3
Gerrit-Owner: Hao Liu 
Gerrit-Reviewer: Jenkins
Gerrit-Reviewer: Paris Oplopoios 
Gerrit-Attention: Hao Liu 
Gerrit-Comment-Date: Mon, 22 Aug 2022 16:29:47 +
Gerrit-HasComments: Yes
Gerrit-Has-Labels: Yes
Gerrit-MessageType: comment


[Libreoffice-bugs] [Bug 150531] Incorrectly rendered SVG images with Skia enabled

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

--- Comment #7 from gibt...@gmx.de ---
Sorry, forgot to add in my last comment: Switching the different Skia options
does not change anything, the effect is the same.

Currently, the issue is mostly visible in the full screen mode of Impress, i.e.
it is actually impossible to compare this to Draw. Still, I will perform
further tests to reproduce the issue in editing mode and compare this to Draw.
If I succeed in reproducing the issue with Draw, I will add another comment.

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

[Libreoffice-bugs] [Bug 150549] New: Libreoffice leaves "zombie process" running after closing any LibreOffice component, preventing new components from opening

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

Bug ID: 150549
   Summary: Libreoffice leaves "zombie process" running after
closing any LibreOffice component, preventing new
components from opening
   Product: LibreOffice
   Version: 7.4.0.3 release
  Hardware: x86-64 (AMD64)
OS: Windows (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: LibreOffice
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: d...@leeprecision.com

Description:
I can open a LibreOffice component, calc or writer, then close it (using [CTRL
+ W] or the generic close button in the top right) and a "Zombie" LibreOffice
process is opened and does not close, as well as the original LibreOffice and
Calc processes staying alive.

This new "Zombie" process consumes an odd amount of CPU resources, anywhere
from 5%-15% of CPU, and ~115MB of RAM, while the original LibreOffice process
consumes exactly 0 CPU and 0.9MB RAM, and the original Calc process consumes 0%
CPU and 0.5MB of RAM.

The "Zombie" process will run forever, bouncing between 5%-15% CPU utilization
while i keep it alive. It can only be killed with the task manager, and once
this process is killed, the other two original processes also close.

While the zombie process is alive, no new LibreOffice processes are allowed to
open, of any kind. They will "start" and can be seen in the Task manager, but
will not actually open the UI until the zombie process is killed.

This bug occurs in both windows 10 and 11 environments and on several different
machines.
I experienced it in 7.2, 7.3 and now in 7.4.

If I can provide some debug report from LibreOffice, let me know.

However, it does not occur on ALL of my machine running LibreOffice. It seems
that at some point something in the config gets corrupt or something and then
it occurs every time LibreOffice is run.

Steps to Reproduce:
1. Open any LibreOffice component.
2. Close same component.


Actual Results:
New zombie process started and original processes still running, no new
components are allowed to be started.

Expected Results:
Application closes completely. New components are able to be opened without
issue.


Reproducible: Always


User Profile Reset: Yes


OpenGL enabled: Yes

Additional Info:
Version: 7.4.0.3 (x64) / LibreOffice Community
Build ID: f85e47c08ddd19c015c0114a68350214f7066f5a
CPU threads: 12; OS: Windows 10.0 Build 22622; UI render: Skia/Raster; VCL: win
Locale: en-US (en_US); UI: en-US
Calc: CL

Tried launching in safe mode. In safe mode, the application starts and
crashes...so it made it worse. Disabling safe mode allows application to start.

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

[Libreoffice-bugs] [Bug 128801] switching from 4:3 to 16:9 aspect ratio warps images and formulas

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

--- Comment #5 from Faisal  ---
Still reproducible with Version: 7.3.5.2 / LibreOffice Community
Build ID: 30(Build:2)
CPU threads: 4; OS: Linux 5.15; UI render: default; VCL: kf5 (cairo+xcb)
Locale: en-US (en_US); UI: en-US
7.3.5-3
Calc: threaded

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

[Libreoffice-bugs] [Bug 135569] FILEOPEN DOCX Inner margins are rotated with textbox

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

--- Comment #9 from Commit Notification 
 ---
Miklos Vajna committed a patch related to this issue.
It has been pushed to "libreoffice-7-4":

https://git.libreoffice.org/core/commit/9ad83843742354312cd543bfbed286c9f52baf1a

crashtesting: fix DOCX export of tdf135569-1.docx

It will be available in 7.4.1.

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

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

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

[Libreoffice-bugs] [Bug 135569] FILEOPEN DOCX Inner margins are rotated with textbox

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

Commit Notification  changed:

   What|Removed |Added

 Whiteboard|target:7.5.0|target:7.5.0 target:7.4.1

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

[Libreoffice-bugs] [Bug 150531] Incorrectly rendered SVG images with Skia enabled

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

--- Comment #6 from V Stuart Foote  ---
(In reply to Gibtnix from comment #5)
>...
> Hope this helps. Thanks for responding and let me know in case you need any
> further information.

Yes thank you.

And just to clarify--the bad SVG rendering is with Vulkan accelerated
rendering, or *also* occurs with Skia software rendering?

And, is this only with Impress? Or are the other modules also affected--Draw,
Writer, or Calc?

Thanks!

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

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

2022-08-22 Thread Miklos Vajna (via logerrit)
 sw/qa/extras/ooxmlexport/data/content-control-shape.docx |binary
 sw/qa/extras/ooxmlexport/ooxmlfieldexport.cxx|   12 
 2 files changed, 12 insertions(+)

New commits:
commit 9ad83843742354312cd543bfbed286c9f52baf1a
Author: Miklos Vajna 
AuthorDate: Mon Aug 22 14:37:56 2022 +0200
Commit: Xisco Fauli 
CommitDate: Mon Aug 22 21:00:46 2022 +0200

crashtesting: fix DOCX export of tdf135569-1.docx

This went wrong in commit 9700c1b2170ad04453a361ed5647937833ac3c18 (sw
content controls, plain text: add DOCX import, 2022-07-25), which
conditionally disabled the DOCX export of content control starts.

The trouble is that the matching content control end is still written
unconditionally, leading to a not-well-formed XML output.

Reading the commit message, the motivation for the conditional start was
testSdtAndShapeOverlapping in CppunitTest_sw_ooxmlexport7, but it seems
that is no longer necessary since we try it harder to not pollute the
doc model with never-read grab-bags in commit
7ad44f3dd271a591529b048212c4391d8b38ed9d (crashtesting: fix DOCX export
of forum-mso-de-99522.docx, 2022-08-16).

Fix the problem by just reverting the problematic hunk, seeing it's no
longer needed for the testSdtAndShapeOverlapping scenario and it causes
a problem here.

Change-Id: I5cc94ee00dfdf55de1fffe1c28242388549862ac
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138685
Reviewed-by: Miklos Vajna 
Tested-by: Jenkins
(cherry picked from commit 223a43537559efc96b4b5bca32d93211ed849ae5)
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138544
Reviewed-by: Xisco Fauli 

diff --git a/sw/qa/extras/ooxmlexport/data/content-control-shape.docx 
b/sw/qa/extras/ooxmlexport/data/content-control-shape.docx
new file mode 100644
index ..ab5be6cb694a
Binary files /dev/null and 
b/sw/qa/extras/ooxmlexport/data/content-control-shape.docx differ
diff --git a/sw/qa/extras/ooxmlexport/ooxmlfieldexport.cxx 
b/sw/qa/extras/ooxmlexport/ooxmlfieldexport.cxx
index d624a2de0469..952f397e5cf6 100644
--- a/sw/qa/extras/ooxmlexport/ooxmlfieldexport.cxx
+++ b/sw/qa/extras/ooxmlexport/ooxmlfieldexport.cxx
@@ -881,6 +881,18 @@ CPPUNIT_TEST_FIXTURE(Test, testContentControlGrabBag)
 save("Office Open XML Text", maTempFile);
 }
 
+CPPUNIT_TEST_FIXTURE(Test, testContentControlShape)
+{
+// Given a document with a  tag:
+OUString aURL = m_directories.getURLFromSrc(DATA_DIRECTORY) + 
"content-control-shape.docx";
+loadURL(aURL, nullptr);
+
+// When exporting that document back to DOCX:
+// Then make sure that completes without an assertion failure, which would 
mean not-well-formed
+// output was produced, since the  was conditional but the  
was unconditional:
+save("Office Open XML Text", maTempFile);
+}
+
 CPPUNIT_TEST_FIXTURE(Test, testTdf104823)
 {
 // Test how we can roundtrip sdt plain text with databindings support


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

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

Bug 150507 Summary: VIEWING: Text in Writer 7.4.0 appears smeared
https://bugs.documentfoundation.org/show_bug.cgi?id=150507

   What|Removed |Added

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

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

[Libreoffice-bugs] [Bug 150515] Document font rendering looks (very) ugly but fine when placed in text boxes

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

Timur  changed:

   What|Removed |Added

 Ever confirmed|0   |1
 Status|UNCONFIRMED |NEEDINFO

--- Comment #6 from Timur  ---
please see if a duplicate of bug 150507. 
There was a fix trial there today so please test daily master with that fix.

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

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

2022-08-22 Thread Caolán McNamara (via logerrit)
 svx/source/tbxctrls/tbcontrl.cxx   |3 +++
 unotools/source/i18n/localedatawrapper.cxx |   20 +++-
 2 files changed, 10 insertions(+), 13 deletions(-)

New commits:
commit 37494dc5457f5feb44465d2eb508a5f77fec74cd
Author: Caolán McNamara 
AuthorDate: Mon Aug 22 14:44:43 2022 +0100
Commit: Caolán McNamara 
CommitDate: Mon Aug 22 20:30:32 2022 +0200

I see this empty entry for the separator in the styles dropdown

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

diff --git a/svx/source/tbxctrls/tbcontrl.cxx b/svx/source/tbxctrls/tbcontrl.cxx
index 610bb630e55e..db147949c306 100644
--- a/svx/source/tbxctrls/tbcontrl.cxx
+++ b/svx/source/tbxctrls/tbcontrl.cxx
@@ -1575,6 +1575,9 @@ void 
SvxStyleBox_Base::CalcOptimalExtraUserWidth(vcl::RenderContext& rRenderCont
 {
 OUString sStyleName(get_text(i));
 
+if (sStyleName.isEmpty())
+continue;
+
 rRenderContext.Push(vcl::PushFlags::FILLCOLOR | vcl::PushFlags::FONT | 
vcl::PushFlags::TEXTCOLOR);
 SetupEntry(rRenderContext, i, tools::Rectangle(0, 0, RECT_MAX, 
ITEM_HEIGHT), sStyleName, true);
 auto aScriptChanges = CheckScript(sStyleName);
commit b759f6720b50487200740a6202e21774897b44b2
Author: Eike Rathke 
AuthorDate: Mon Aug 22 16:16:08 2022 +0200
Commit: Eike Rathke 
CommitDate: Mon Aug 22 20:30:20 2022 +0200

Resolves: tdf#150288 Do not prepend the locale's full date acceptance 
pattern

... if it wasn't present as first, but take the date acceptance
patterns as specified by the user.

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

diff --git a/unotools/source/i18n/localedatawrapper.cxx 
b/unotools/source/i18n/localedatawrapper.cxx
index 54aa49f44877..9e6fe8990979 100644
--- a/unotools/source/i18n/localedatawrapper.cxx
+++ b/unotools/source/i18n/localedatawrapper.cxx
@@ -1550,19 +1550,13 @@ void LocaleDataWrapper::loadDateAcceptancePatterns(
 }
 }
 
-// Never overwrite the locale's full date pattern! The first.
-if (std::as_const(aDateAcceptancePatterns)[0] == rPatterns[0])
-aDateAcceptancePatterns = comphelper::containerToSequence(rPatterns);  
  // sane
-else
-{
-// Copy existing full date pattern and append the sequence passed.
-/* TODO: could check for duplicates and shrink target sequence */
-Sequence< OUString > aTmp( rPatterns.size() + 1 );
-auto it = aTmp.getArray();
-*it = std::as_const(aDateAcceptancePatterns)[0];
-std::copy(rPatterns.begin(), rPatterns.end(), std::next(it));
-aDateAcceptancePatterns = aTmp;
-}
+// Earlier versions checked for presence of the full date pattern with
+// aDateAcceptancePatterns[0] == rPatterns[0] and prepended that if not.
+// This lead to confusion if the patterns were intentionally specified
+// without, giving entirely a different DMY order, see tdf#150288.
+// Not checking this and accepting the given patterns as is may result in
+// the user shooting themself in the foot, but we can't have both.
+aDateAcceptancePatterns = comphelper::containerToSequence(rPatterns);
 }
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */


[Libreoffice-bugs] [Bug 149506] Slow start-up and opening of files since 7.3 release

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

--- Comment #9 from Timur  ---
Please turn off proxy, to check if bug 135754. 
Joe, please see Ubuntu bug 150479.

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

[Libreoffice-bugs] [Bug 143651] Calc fails to fill-in default value from prior rows

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

Buovjaga  changed:

   What|Removed |Added

 Status|NEEDINFO|UNCONFIRMED
 Ever confirmed|1   |0

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

[Libreoffice-bugs] [Bug 150531] Incorrectly rendered SVG images with Skia enabled

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

--- Comment #5 from gibt...@gmx.de ---
Thanks for the clarification. I did not know that there is a significant
difference in using the force skia for rendering checkbox. I unchecked it and
the skia.log now contains the following content:

RenderMethod: vulkan
Vendor: 0x8086
Device: 0x9b41
API: 1.2.196
Driver: 0.404.1122
DeviceType: integrated
DeviceName: Intel(R) UHD Graphics
Denylisted: no

Hope this helps. Thanks for responding and let me know in case you need any
further information.

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

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

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

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

   What|Removed |Added

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

--- Comment #6 from Roman Kuznetsov <79045_79...@mail.ru> ---
So, Karl, I close your bug report as WORKSFORME then.

If you have some ideas to improve the dialog or other any things in
LibreOffice, just file another report. But please file one report per problem.
Thank you

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

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

2022-08-22 Thread Tomaž Vajngerl (via logerrit)
 chart2/qa/extras/chart2import2.cxx 
 |   16 ++
 
chart2/qa/extras/data/xlsx/DataTable-MultipleLegendEntriesForOneDataSeries.xlsx 
|binary
 chart2/source/view/charttypes/VSeriesPlotter.cxx   
 |   78 --
 chart2/source/view/main/DataTableView.cxx  
 |   26 +--
 4 files changed, 38 insertions(+), 82 deletions(-)

New commits:
commit e5583150a3953f08782865e0f90d41f5c35cc6c5
Author: Tomaž Vajngerl 
AuthorDate: Mon Aug 22 16:46:41 2022 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 19:50:28 2022 +0200

chart2: always use the series for the keys (symbols) in data table

In special case the legend keys are not series, but can be each
value of a series. This can't be done for the data table where so
this has been disabled.

This caused a crash when positioning the symbols, as in some cases
we got 0 symbols (due to another bug that didn't put the symbols
into the collection) or more symbols that there were series (and
the symbols couldn't be positioned correctly). With this change
we now get the correct number of symbols for the data table.

Also added the test document and test to open the document, to
make sure we don't crash anymore.

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

diff --git a/chart2/qa/extras/chart2import2.cxx 
b/chart2/qa/extras/chart2import2.cxx
index eb983f7687f1..81e274400c67 100644
--- a/chart2/qa/extras/chart2import2.cxx
+++ b/chart2/qa/extras/chart2import2.cxx
@@ -61,6 +61,7 @@ public:
 void testTdf139658();
 void testTdf146066();
 void testTdf150434();
+void testChartDataTableWithMultipleLegendEntriesForOneDataSeries();
 
 CPPUNIT_TEST_SUITE(Chart2ImportTest2);
 
@@ -103,6 +104,7 @@ public:
 CPPUNIT_TEST(testTdf139658);
 CPPUNIT_TEST(testTdf146066);
 CPPUNIT_TEST(testTdf150434);
+CPPUNIT_TEST(testChartDataTableWithMultipleLegendEntriesForOneDataSeries);
 
 CPPUNIT_TEST_SUITE_END();
 };
@@ -948,6 +950,20 @@ void Chart2ImportTest2::testTdf150434()
 CPPUNIT_ASSERT_GREATEREQUAL(static_cast(0), aPosition.Y);
 }
 
+void 
Chart2ImportTest2::testChartDataTableWithMultipleLegendEntriesForOneDataSeries()
+{
+load(u"/chart2/qa/extras/data/xlsx/", 
u"DataTable-MultipleLegendEntriesForOneDataSeries.xlsx");
+// Loading this file caused a crash in the data table code
+
+Reference xChartDoc(getChartDocFromSheet(0, 
mxComponent),
+   UNO_QUERY_THROW);
+Reference xDrawPageSupplier(xChartDoc, 
UNO_QUERY_THROW);
+Reference xDrawPage(xDrawPageSupplier->getDrawPage(), 
UNO_SET_THROW);
+Reference xShapes(xDrawPage->getByIndex(0), 
UNO_QUERY_THROW);
+Reference xDataTableShape = getShapeByName(xShapes, 
"CID/D=0:DataTable=");
+CPPUNIT_ASSERT(xDataTableShape.is());
+}
+
 CPPUNIT_TEST_SUITE_REGISTRATION(Chart2ImportTest2);
 
 CPPUNIT_PLUGIN_IMPLEMENT();
diff --git 
a/chart2/qa/extras/data/xlsx/DataTable-MultipleLegendEntriesForOneDataSeries.xlsx
 
b/chart2/qa/extras/data/xlsx/DataTable-MultipleLegendEntriesForOneDataSeries.xlsx
new file mode 100644
index ..b077fd2e0315
Binary files /dev/null and 
b/chart2/qa/extras/data/xlsx/DataTable-MultipleLegendEntriesForOneDataSeries.xlsx
 differ
diff --git a/chart2/source/view/charttypes/VSeriesPlotter.cxx 
b/chart2/source/view/charttypes/VSeriesPlotter.cxx
index e34606cd1dc9..65d404fb0be8 100644
--- a/chart2/source/view/charttypes/VSeriesPlotter.cxx
+++ b/chart2/source/view/charttypes/VSeriesPlotter.cxx
@@ -2516,11 +2516,6 @@ std::vector 
VSeriesPlotter::createSymbols(const awt::Size& rEn
 if (!pSeries)
 continue;
 
-if 
(!pSeries->getPropertiesOfSeries()->getPropertyValue("ShowLegendEntry").get())
-{
-continue;
-}
-
 std::vector aSeriesSymbols = 
createSymbolsForSeries(rEntryKeyAspectRatio, *pSeries, xTarget, xContext);
 
 //add series entries to the result now
@@ -2887,75 +2882,16 @@ std::vector 
VSeriesPlotter::createSymbolsForSeries(
 try
 {
 ViewLegendSymbol aEntry;
-bool bVaryColorsByPoint = rSeries.isVaryColorsByPoint();
-bool bIsPie = 
m_xChartTypeModel->getChartType().equalsIgnoreAsciiCase(CHART2_SERVICE_NAME_CHARTTYPE_PIE);
-try
-{
-if (bIsPie)
-{
-bool bDonut = false;
-if ((m_xChartTypeModel->getPropertyValue("UseRings") >>= 
bDonut) && bDonut)
-bIsPie = false;
-}
-}
-catch (const uno::Exception&)
-{
-}
-
-if (bVaryColorsByPoint || bIsPie)
-{
-Sequence< 

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

2022-08-22 Thread Tomaž Vajngerl (via logerrit)
 chart2/source/controller/main/ChartController_Window.cxx |1 -
 1 file changed, 1 deletion(-)

New commits:
commit 4c789414bf53c4df2b3c798c379adcb493a9e84a
Author: Tomaž Vajngerl 
AuthorDate: Mon Aug 22 12:12:01 2022 +0200
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 19:50:09 2022 +0200

chart2: remove duplicate eObjectType

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

diff --git a/chart2/source/controller/main/ChartController_Window.cxx 
b/chart2/source/controller/main/ChartController_Window.cxx
index cc8125057a09..435f4b1474f3 100644
--- a/chart2/source/controller/main/ChartController_Window.cxx
+++ b/chart2/source/controller/main/ChartController_Window.cxx
@@ -1035,7 +1035,6 @@ void ChartController::execute_Command( const 
CommandEvent& rCEvt )
 xPopupMenu->insertSeparator( -1 );
 }
 
-ObjectType eObjectType = ObjectIdentifier::getObjectType( 
m_aSelection.getSelectedCID() );
 Reference< XDiagram > xDiagram = 
ChartModelHelper::findDiagram( getModel() );
 
 OUString aFormatCommand( lcl_getFormatCommandForObjectCID( 
m_aSelection.getSelectedCID() ) );


[Libreoffice-bugs] [Bug 143651] Calc fails to fill-in default value from prior rows

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

--- Comment #13 from MR Zenwiz  ---
(In reply to Buovjaga from comment #12)
:
> 
> Is Tools - AutoInput checked?
> 
Yes.

> Note also that in 7.3 there was a change in how the suggestions appear:
> https://wiki.documentfoundation.org/ReleaseNotes/7.3#General_improvements_2
> "Bash like autocompletion for Calc autoinput was implemented"

I appreciate that. I was not aware of it and will explore this. However, that
does not really apply here - when I type in the first character of a prior
entry, nothing is prompted at all, and adding more promptable characters also
has no effect.

Thank you.

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

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

2022-08-22 Thread Michael Stahl (via logerrit)
 sw/source/ui/misc/bookmark.cxx |2 +-
 sw/source/uibase/utlui/content.cxx |6 --
 2 files changed, 5 insertions(+), 3 deletions(-)

New commits:
commit 5c3c49bbf768f88b3d9a45d1ffc88c44ad2870c1
Author: Michael Stahl 
AuthorDate: Mon Aug 22 15:12:54 2022 +0200
Commit: Michael Stahl 
CommitDate: Mon Aug 22 19:09:26 2022 +0200

sw: Navigator: Protect Bookmarks should allow editing the text ...

... in the bookmark.  Only deleting/renaming it is forbidden.

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

diff --git a/sw/source/ui/misc/bookmark.cxx b/sw/source/ui/misc/bookmark.cxx
index 89d353c20245..363b861653a4 100644
--- a/sw/source/ui/misc/bookmark.cxx
+++ b/sw/source/ui/misc/bookmark.cxx
@@ -396,7 +396,7 @@ SwInsertBookmarkDlg::SwInsertBookmarkDlg(weld::Window* 
pParent, SwWrtShell& rS,
 m_xEditTextBtn->set_sensitive(false);
 m_xRenameBtn->set_sensitive(false);
 
-// select 3rd column, otherwise it'll pick 1st one FIXME doesn't work with 
gtk?
+// select 3rd column, otherwise it'll pick 1st one
 m_xBookmarksBox->set_column_editables({ false, false, true, false, false 
});
 
 PopulateTable();
diff --git a/sw/source/uibase/utlui/content.cxx 
b/sw/source/uibase/utlui/content.cxx
index dfba5294a35e..7922655252fa 100644
--- a/sw/source/uibase/utlui/content.cxx
+++ b/sw/source/uibase/utlui/content.cxx
@@ -1680,7 +1680,7 @@ IMPL_LINK(SwContentTree, CommandHdl, const CommandEvent&, 
rCEvt, bool)
 const bool bProtectBM = (ContentTypeId::BOOKMARK == nContentType)
 && 
m_pActiveShell->getIDocumentSettingAccess().get(DocumentSettingId::PROTECT_BOOKMARKS);
 const bool bEditable = pType->IsEditable() &&
-((bVisible && !bProtected && !bProtectBM) || 
ContentTypeId::REGION == nContentType);
+((bVisible && !bProtected) || ContentTypeId::REGION == 
nContentType);
 const bool bDeletable = pType->IsDeletable() &&
 ((bVisible && !bProtected && !bProtectBM) || 
ContentTypeId::REGION == nContentType);
 const bool bRenamable = bEditable && !bReadonly &&
@@ -4911,20 +4911,22 @@ void SwContentTree::EditEntry(const weld::TreeIter& 
rEntry, EditEntryMode nMode)
 nSlot = FN_FORMAT_FRAME_DLG;
 break;
 case ContentTypeId::BOOKMARK  :
-
assert(!m_pActiveShell->getIDocumentSettingAccess().get(DocumentSettingId::PROTECT_BOOKMARKS));
 if(nMode == EditEntryMode::DELETE)
 {
+
assert(!m_pActiveShell->getIDocumentSettingAccess().get(DocumentSettingId::PROTECT_BOOKMARKS));
 IDocumentMarkAccess* const pMarkAccess = 
m_pActiveShell->getIDocumentMarkAccess();
 
pMarkAccess->deleteMark(pMarkAccess->findMark(pCnt->GetName()), false);
 }
 else if(nMode == EditEntryMode::RENAME)
 {
+
assert(!m_pActiveShell->getIDocumentSettingAccess().get(DocumentSettingId::PROTECT_BOOKMARKS));
 uno::Reference< frame::XModel >  xModel = 
m_pActiveShell->GetView().GetDocShell()->GetBaseModel();
 uno::Reference< text::XBookmarksSupplier >  xBkms(xModel, 
uno::UNO_QUERY);
 xNameAccess = xBkms->getBookmarks();
 }
 else
 {
+// allowed despite PROTECT_BOOKMARKS: the dialog itself 
enforces it
 SfxStringItem const name(FN_EDIT_BOOKMARK, pCnt->GetName());
 SfxPoolItem const* args[2] = { , nullptr };
 m_pActiveShell->GetView().GetViewFrame()->


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

2022-08-22 Thread Andreas Heinisch (via logerrit)
 basic/inc/sbxbase.hxx|1 +
 basic/source/classes/sb.cxx  |4 ++--
 basic/source/sbx/sbxbase.cxx |   17 +
 basic/source/sbx/sbxexec.cxx |2 +-
 basic/source/sbx/sbxobj.cxx  |2 +-
 include/basic/sbxcore.hxx|2 ++
 6 files changed, 24 insertions(+), 4 deletions(-)

New commits:
commit 7e2a8a9869e362ae181ad92f50ea287e9abf85bb
Author: Andreas Heinisch 
AuthorDate: Tue Aug 9 18:42:40 2022 +0200
Commit: Andreas Heinisch 
CommitDate: Mon Aug 22 19:03:42 2022 +0200

tdf#150139 - Add a concrete name to the error message

Add a concrete name to the "Property or method not found: $(ARG1)" error 
message when creating a listener.

Change-Id: I76c904793a96082c49f8aa2d45c76c50a453d892
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/138061
Tested-by: Jenkins
Reviewed-by: Andreas Heinisch 

diff --git a/basic/inc/sbxbase.hxx b/basic/inc/sbxbase.hxx
index 80552c2fbf08..e06d76161dec 100644
--- a/basic/inc/sbxbase.hxx
+++ b/basic/inc/sbxbase.hxx
@@ -36,6 +36,7 @@ class SbxBasicFormater;
 struct SbxAppData
 {
 ErrCode eErrCode; // Error code
+OUString aErrorMsg; // Error message for $ARG
 SbxVariableRef m_aGlobErr; // Global error object
 std::vector m_Factories; // these are owned by fields in 
SbiGlobals
 tools::SvRef mrImplRepository;
diff --git a/basic/source/classes/sb.cxx b/basic/source/classes/sb.cxx
index 8d87330e469b..f1ab6dd1da0c 100644
--- a/basic/source/classes/sb.cxx
+++ b/basic/source/classes/sb.cxx
@@ -1325,11 +1325,11 @@ bool StarBASIC::Call( const OUString& rName, SbxArray* 
pParam )
 if( !bRes )
 {
 ErrCode eErr = SbxBase::GetError();
-SbxBase::ResetError();
 if( eErr != ERRCODE_NONE )
 {
-RTError( eErr, OUString(), 0, 0, 0 );
+RTError(eErr, SbxBase::GetErrorMsg(), 0, 0, 0);
 }
+SbxBase::ResetError();
 }
 return bRes;
 }
diff --git a/basic/source/sbx/sbxbase.cxx b/basic/source/sbx/sbxbase.cxx
index 3b70307ec3e7..9d2129e68a1e 100644
--- a/basic/source/sbx/sbxbase.cxx
+++ b/basic/source/sbx/sbxbase.cxx
@@ -37,6 +37,7 @@
 
 SbxAppData::SbxAppData()
 : eErrCode(ERRCODE_NONE)
+, aErrorMsg(OUString())
 , eBasicFormaterLangType(LANGUAGE_DONTKNOW)
 {
 }
@@ -96,6 +97,21 @@ ErrCode const & SbxBase::GetError()
 return GetSbxData_Impl().eErrCode;
 }
 
+OUString const & SbxBase::GetErrorMsg()
+{
+return GetSbxData_Impl().aErrorMsg;
+}
+
+void SbxBase::SetError(ErrCode e, const OUString& rMsg)
+{
+SbxAppData& r = GetSbxData_Impl();
+if (e && r.eErrCode == ERRCODE_NONE)
+{
+r.eErrCode = e;
+r.aErrorMsg = rMsg;
+}
+}
+
 void SbxBase::SetError( ErrCode e )
 {
 SbxAppData& r = GetSbxData_Impl();
@@ -111,6 +127,7 @@ bool SbxBase::IsError()
 void SbxBase::ResetError()
 {
 GetSbxData_Impl().eErrCode = ERRCODE_NONE;
+GetSbxData_Impl().aErrorMsg = OUString();
 }
 
 void SbxBase::AddFactory( SbxFactory* pFac )
diff --git a/basic/source/sbx/sbxexec.cxx b/basic/source/sbx/sbxexec.cxx
index d830061f071d..f3abca766d35 100644
--- a/basic/source/sbx/sbxexec.cxx
+++ b/basic/source/sbx/sbxexec.cxx
@@ -323,7 +323,7 @@ static SbxVariableRef Element
 }
 }
 else
-SbxBase::SetError( ERRCODE_BASIC_NO_METHOD );
+SbxBase::SetError( ERRCODE_BASIC_NO_METHOD, aSym );
 }
 *ppBuf = p;
 return refVar;
diff --git a/basic/source/sbx/sbxobj.cxx b/basic/source/sbx/sbxobj.cxx
index f83324a26bb8..4dd054959823 100644
--- a/basic/source/sbx/sbxobj.cxx
+++ b/basic/source/sbx/sbxobj.cxx
@@ -275,7 +275,7 @@ bool SbxObject::Call( const OUString& rName, SbxArray* 
pParam )
 pMeth->SetParameters( nullptr );
 return true;
 }
-SetError( ERRCODE_BASIC_NO_METHOD );
+SetError( ERRCODE_BASIC_NO_METHOD, rName );
 return false;
 }
 
diff --git a/include/basic/sbxcore.hxx b/include/basic/sbxcore.hxx
index e5f726e8d016..477858d9656c 100644
--- a/include/basic/sbxcore.hxx
+++ b/include/basic/sbxcore.hxx
@@ -85,7 +85,9 @@ public:
 virtual boolLoadCompleted();
 
 static ErrCode const & GetError();
+static OUString const& GetErrorMsg();
 static void SetError( ErrCode );
+static void SetError(ErrCode, const OUString&);
 static bool IsError();
 static void ResetError();
 


[Libreoffice-bugs] [Bug 142031] Formula bar suggestion tooltips don't like accents

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

Andreas Heinisch  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|libreoffice-b...@lists.free |andreas.heini...@yahoo.de
   |desktop.org |

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

[Libreoffice-bugs] [Bug 150548] Spreadsheet shows useful information after FILEOPEN in MSO but in Libre Calc 7.3.5.2 no useful information can be seen

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

tom1willi...@yandex.com changed:

   What|Removed |Added

 OS|Linux (All) |All

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

[Libreoffice-bugs] [Bug 150548] Spreadsheet shows useful information after FILEOPEN in MSO but in Libre Calc 7.3.5.2 no useful information can be seen

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

--- Comment #1 from tom1willi...@yandex.com ---
Created attachment 181954
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181954=edit
an .XLS file from a reputable source on the internet

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

[Libreoffice-bugs] [Bug 150548] New: Spreadsheet shows useful information after FILEOPEN in MSO but in Libre Calc 7.3.5.2 no useful information can be seen

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

Bug ID: 150548
   Summary: Spreadsheet shows useful information after FILEOPEN in
MSO but in Libre Calc 7.3.5.2 no useful information
can be seen
   Product: LibreOffice
   Version: 7.3.5.2 release
  Hardware: All
OS: Linux (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: Calc
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: tom1willi...@yandex.com

Description:
Spreadsheet shows useful information after FILEOPEN in MSO but in Libre Calc
7.3.5.2 no useful information can be seen.

Steps to Reproduce:
1.Open the file in Libre Calc 7.3.5.2
2. No useful data can be seen
3. Open the file in MSO and the data can be easily seen

Actual Results:
no data visible except for the names of individual sheets

Expected Results:
Expected to see an income statement, a balance sheet and other financial
statements 


Reproducible: Always


User Profile Reset: No



Additional Info:
This file comes from the US SEC EDGAR system. That seems like an important
source for data and a file format that Libre should open as well as MS Excel
does.

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

[Libreoffice-bugs] [Bug 150547] New: Unusable documentation for scripting

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

Bug ID: 150547
   Summary: Unusable documentation for scripting
   Product: LibreOffice
   Version: 6.4.7.2 release
  Hardware: x86-64 (AMD64)
OS: Linux (All)
Status: UNCONFIRMED
  Severity: normal
  Priority: medium
 Component: BASIC
  Assignee: libreoffice-bugs@lists.freedesktop.org
  Reporter: fsd...@gmail.com

Description:
I try to find the Sheet in a Basic script for more than a week.
The documentation on the net is for OpenOffice.
ThisComponent which returns the actual document fits here but there is no
Sheets method nor getSheetByName nor getSheetByIndex.
The documentation for ThisComponent says that the availble methods of the
object returned depends on the document type (what makes sense).
There is no way to figure out which methods ARE available tough using:
https://help.libreoffice.org/latest/gu/text/sbasic/shared/

Steps to Reproduce:
1. From a Spreadsheet open the Macro Editor
2. Get the actual document using ThisComponent
3. Try to find the Sheets, Cells, Rows, Columns, ...

Actual Results:
well, nothing

Expected Results:
Documentation which makes it possible to find the required information


Reproducible: Always


User Profile Reset: No



Additional Info:
Structuring the documentation as if it was an API documentation would make
sense.
Object/Service and in this Methods, Properties etc.
Doc = ThisComponent  # which is a spreadsheet at the moment
Sheet = Doc.getSheets()
fails
There should a section "SpreadsheetDocumetObject" with the methods and
properties listed
Doc = ThisComponent
Sheet = Doc.theRightMethodFromTheDocumentationToGetTheSheets()
should work

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

[Libreoffice-bugs] [Bug 150533] Font and font size fields overwrite field rather than clear first creating illegible gibberish

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

Ken  changed:

   What|Removed |Added

 Ever confirmed|1   |0
 Status|NEEDINFO|UNCONFIRMED

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

[Libreoffice-bugs] [Bug 150533] Font and font size fields overwrite field rather than clear first creating illegible gibberish

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

--- Comment #4 from Ken  ---
Created attachment 181953
  --> https://bugs.documentfoundation.org/attachment.cgi?id=181953=edit
Facsimile of document that presented bug

change some fonts or font sizes

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

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

2022-08-22 Thread Noel Grandin (via logerrit)
 sw/source/core/doc/DocumentRedlineManager.cxx |   48 +-
 1 file changed, 24 insertions(+), 24 deletions(-)

New commits:
commit 0bda4401eda5b19b1ebfeb712cab6af8c8d1f59b
Author: Noel Grandin 
AuthorDate: Mon Aug 22 15:27:21 2022 +0200
Commit: Noel Grandin 
CommitDate: Mon Aug 22 18:08:50 2022 +0200

use more SwPosition::Assign

as part of the attempt to keep the internals of SwPosition in sync with
each other

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

diff --git a/sw/source/core/doc/DocumentRedlineManager.cxx 
b/sw/source/core/doc/DocumentRedlineManager.cxx
index 6670aca270ed..cc51a0dc9d55 100644
--- a/sw/source/core/doc/DocumentRedlineManager.cxx
+++ b/sw/source/core/doc/DocumentRedlineManager.cxx
@@ -567,7 +567,7 @@ namespace
 case SwComparePosition::Equal:
 {
 bool bInsert = RedlineType::Insert == pRedl->GetType();
-SwPosition aPos(pRedl->Start()->nNode);
+SwPosition aPos(pRedl->Start()->GetNode());
 rArr.DeleteAndDestroy( rPos-- );
 
 // remove tracking of the table row, if needed
@@ -778,7 +778,7 @@ namespace
 aPam.GetBound( false ).nContent.Assign( nullptr, 0 
);
 if (aPam.End()->GetNode().IsStartNode())
 {   // end node will be deleted too! see 
nNodeDiff+1
---aPam.End()->nNode;
+aPam.End()->Adjust(SwNodeOffset(-1));
 }
 assert(!aPam.End()->GetNode().IsStartNode());
 rDoc.getIDocumentContentOperations().DelFullPara( 
aPam );
@@ -3141,7 +3141,7 @@ const SwRangeRedline* 
DocumentRedlineManager::SelNextRedline( SwPaM& rPam ) cons
 {
 SwNodeIndex aTmp( pEnd->GetNode() );
 SwContentNode* pCNd = SwNodes::GoPrevSection(  );
-if( !pCNd || ( aTmp == rSttPos.nNode &&
+if( !pCNd || ( aTmp == rSttPos.GetNode() &&
 pCNd->Len() == rSttPos.GetContentIndex() ))
 pFnd = nullptr;
 }
@@ -3196,14 +3196,14 @@ const SwRangeRedline* 
DocumentRedlineManager::SelNextRedline( SwPaM& rPam ) cons
 const SwRangeRedline* pSaveFnd = pFnd;
 
 SwContentNode* pCNd;
-SwNodeIndex* pIdx = ()->nNode;
-if( !pIdx->GetNode().IsContentNode() )
+SwPosition* pPos = rPam.GetMark();
+if( !pPos->GetNode().IsContentNode() )
 {
-pCNd = m_rDoc.GetNodes().GoNextSection( pIdx );
+pCNd = m_rDoc.GetNodes().GoNextSection( pPos );
 if( pCNd )
 {
-if( *pIdx <= rPam.GetPoint()->GetNode() )
-rPam.GetMark()->nContent.Assign( pCNd, 0 );
+if( pPos->GetNode() <= rPam.GetPoint()->GetNode() )
+pPos->Assign( *pCNd, 0 );
 else
 pFnd = nullptr;
 }
@@ -3211,14 +3211,14 @@ const SwRangeRedline* 
DocumentRedlineManager::SelNextRedline( SwPaM& rPam ) cons
 
 if( pFnd )
 {
-pIdx = ()->nNode;
-if( !pIdx->GetNode().IsContentNode() )
+pPos = rPam.GetPoint();
+if( !pPos->GetNode().IsContentNode() )
 {
-pCNd = SwNodes::GoPrevSection( pIdx );
+pCNd = SwNodes::GoPrevSection( pPos );
 if( pCNd )
 {
-if( *pIdx >= rPam.GetMark()->nNode )
-rPam.GetPoint()->nContent.Assign( pCNd, 
pCNd->Len() );
+if( pPos->GetNode() >= rPam.GetMark()->GetNode() )
+pPos->Assign( *pCNd, pCNd->Len() );
 else
 pFnd = nullptr;
 }
@@ -3267,7 +3267,7 @@ const SwRangeRedline* 
DocumentRedlineManager::SelPrevRedline( SwPaM& rPam ) cons
 {
 SwNodeIndex aTmp( pStt->GetNode() );
 SwContentNode* pCNd = m_rDoc.GetNodes().GoNextSection(  );
-if( !pCNd || ( aTmp == rSttPos.nNode &&
+if( !pCNd || ( aTmp == rSttPos.GetNode() &&
 !rSttPos.GetContentIndex() ))
 pFnd = nullptr;
 }
@@ -3323,14 +3323,14 @@ const SwRangeRedline* 
DocumentRedlineManager::SelPrevRedline( SwPaM& rPam ) cons
 const SwRangeRedline* pSaveFnd = pFnd;
 
 SwContentNode* pCNd;
-SwNodeIndex* pIdx = ()->nNode;
-if( !pIdx->GetNode().IsContentNode() )
+SwPosition* pPos = rPam.GetMark();
+

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

2022-08-22 Thread offtkp (via logerrit)
 vcl/source/filter/GraphicFormatDetector.cxx |   40 +-
 vcl/source/filter/graphicfilter2.cxx|   50 ++--
 2 files changed, 42 insertions(+), 48 deletions(-)

New commits:
commit 86d49f41d46519235b55918ac375bc1d2b99552b
Author: offtkp 
AuthorDate: Sat Aug 13 12:52:26 2022 +0300
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 18:05:29 2022 +0200

Remove code duplication in GraphicDescriptor for PSD

GraphicFormatDetector and GraphicDescriptor have duplicate format
detection code so now GraphicDescriptor uses GraphicFormatDetector
functions instead to detect the format for PSD files

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

diff --git a/vcl/source/filter/GraphicFormatDetector.cxx 
b/vcl/source/filter/GraphicFormatDetector.cxx
index 5b5ccf33d004..8956c3cd9530 100644
--- a/vcl/source/filter/GraphicFormatDetector.cxx
+++ b/vcl/source/filter/GraphicFormatDetector.cxx
@@ -792,12 +792,48 @@ bool GraphicFormatDetector::checkPCD()
 
 bool GraphicFormatDetector::checkPSD()
 {
+bool bRet = false;
 if ((mnFirstLong == 0x38425053) && ((mnSecondLong >> 16) == 1))
 {
 maMetadata.mnFormat = GraphicFileFormat::PSD;
-return true;
+bRet = true;
+if (mbExtendedInfo)
+{
+sal_uInt16 nChannels = 0;
+sal_uInt32 nRows = 0;
+sal_uInt32 nColumns = 0;
+sal_uInt16 nDepth = 0;
+sal_uInt16 nMode = 0;
+mrStream.Seek(mnStreamPosition + 6);
+mrStream.SeekRel(6); // Pad
+mrStream.ReadUInt16(nChannels)
+.ReadUInt32(nRows)
+.ReadUInt32(nColumns)
+.ReadUInt16(nDepth)
+.ReadUInt16(nMode);
+if ((nDepth == 1) || (nDepth == 8) || (nDepth == 16))
+{
+maMetadata.mnBitsPerPixel = (nDepth == 16) ? 8 : nDepth;
+switch (nChannels)
+{
+case 4:
+case 3:
+maMetadata.mnBitsPerPixel = 24;
+[[fallthrough]];
+case 2:
+case 1:
+maMetadata.maPixSize.setWidth(nColumns);
+maMetadata.maPixSize.setHeight(nRows);
+break;
+default:
+bRet = false;
+}
+}
+else
+bRet = false;
+}
 }
-return false;
+return bRet;
 }
 
 bool GraphicFormatDetector::checkEPS()
diff --git a/vcl/source/filter/graphicfilter2.cxx 
b/vcl/source/filter/graphicfilter2.cxx
index 4327ef5e2f59..1117b2cc9cbb 100644
--- a/vcl/source/filter/graphicfilter2.cxx
+++ b/vcl/source/filter/graphicfilter2.cxx
@@ -683,54 +683,12 @@ bool GraphicDescriptor::ImpDetectTGA( SvStream& rStm, 
bool )
 
 bool GraphicDescriptor::ImpDetectPSD( SvStream& rStm, bool bExtendedInfo )
 {
-bool bRet = false;
-
-sal_uInt32  nMagicNumber = 0;
 sal_Int32 nStmPos = rStm.Tell();
-rStm.SetEndian( SvStreamEndian::BIG );
-rStm.ReadUInt32( nMagicNumber );
-if ( nMagicNumber == 0x38425053 )
-{
-sal_uInt16 nVersion = 0;
-rStm.ReadUInt16( nVersion );
-if ( nVersion == 1 )
-{
-bRet = true;
-if ( bExtendedInfo )
-{
-sal_uInt16 nChannels = 0;
-sal_uInt32 nRows = 0;
-sal_uInt32 nColumns = 0;
-sal_uInt16 nDepth = 0;
-sal_uInt16 nMode = 0;
-rStm.SeekRel( 6 );  // Pad
-rStm.ReadUInt16( nChannels ).ReadUInt32( nRows ).ReadUInt32( 
nColumns ).ReadUInt16( nDepth ).ReadUInt16( nMode );
-if ( ( nDepth == 1 ) || ( nDepth == 8 ) || ( nDepth == 16 ) )
-{
-aMetadata.mnBitsPerPixel = ( nDepth == 16 ) ? 8 : nDepth;
-switch ( nChannels )
-{
-case 4 :
-case 3 :
-aMetadata.mnBitsPerPixel = 24;
-[[fallthrough]];
-case 2 :
-case 1 :
-aMetadata.maPixSize.setWidth( nColumns );
-aMetadata.maPixSize.setHeight( nRows );
-break;
-default:
-bRet = false;
-}
-}
-else
-bRet = false;
-}
-}
-}
-
+vcl::GraphicFormatDetector aDetector( rStm, aPathExt, bExtendedInfo );
+bool bRet = aDetector.detect();
+bRet &= aDetector.checkPSD();
 if ( bRet )
-aMetadata.mnFormat = 

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

2022-08-22 Thread offtkp (via logerrit)
 vcl/source/filter/GraphicFormatDetector.cxx |  172 +++-
 vcl/source/filter/graphicfilter2.cxx|  199 +---
 2 files changed, 180 insertions(+), 191 deletions(-)

New commits:
commit 6a2fd2d07ed1cb7713776b32ff175b300ee732e9
Author: offtkp 
AuthorDate: Sat Aug 13 12:02:01 2022 +0300
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 18:04:54 2022 +0200

Remove code duplication in GraphicDescriptor for PNG

GraphicFormatDetector and GraphicDescriptor have duplicate format
detection code so now GraphicDescriptor uses GraphicFormatDetector
functions instead to detect the format for PNG files

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

diff --git a/vcl/source/filter/GraphicFormatDetector.cxx 
b/vcl/source/filter/GraphicFormatDetector.cxx
index 43e403e59936..5b5ccf33d004 100644
--- a/vcl/source/filter/GraphicFormatDetector.cxx
+++ b/vcl/source/filter/GraphicFormatDetector.cxx
@@ -645,6 +645,102 @@ bool GraphicFormatDetector::checkPNG()
 if (mnFirstLong == 0x89504e47 && mnSecondLong == 0x0d0a1a0a)
 {
 maMetadata.mnFormat = GraphicFileFormat::PNG;
+if (mbExtendedInfo)
+{
+sal_uInt32 nTemp32;
+mrStream.Seek(mnStreamPosition + 8);
+do
+{
+sal_uInt8 cByte = 0;
+
+// IHDR-Chunk
+mrStream.SeekRel(8);
+
+// width
+mrStream.ReadUInt32(nTemp32);
+if (!mrStream.good())
+break;
+maMetadata.maPixSize.setWidth(nTemp32);
+
+// height
+mrStream.ReadUInt32(nTemp32);
+if (!mrStream.good())
+break;
+maMetadata.maPixSize.setHeight(nTemp32);
+
+// Bits/Pixel
+mrStream.ReadUChar(cByte);
+if (!mrStream.good())
+break;
+maMetadata.mnBitsPerPixel = cByte;
+
+// Colour type - check whether it supports alpha values
+sal_uInt8 cColType = 0;
+mrStream.ReadUChar(cColType);
+if (!mrStream.good())
+break;
+maMetadata.mbIsAlpha = maMetadata.mbIsTransparent
+= (cColType == 4 || cColType == 6);
+
+// Planes always 1;
+// compression always
+maMetadata.mnPlanes = 1;
+
+sal_uInt32 nLen32 = 0;
+nTemp32 = 0;
+
+mrStream.SeekRel(7);
+
+// read up to the start of the image
+mrStream.ReadUInt32(nLen32);
+mrStream.ReadUInt32(nTemp32);
+while (mrStream.good() && nTemp32 != 0x49444154)
+{
+if (nTemp32 == 0x70485973) // physical pixel dimensions
+{
+sal_uLong nXRes;
+sal_uLong nYRes;
+
+// horizontal resolution
+nTemp32 = 0;
+mrStream.ReadUInt32(nTemp32);
+nXRes = nTemp32;
+
+// vertical resolution
+nTemp32 = 0;
+mrStream.ReadUInt32(nTemp32);
+nYRes = nTemp32;
+
+// unit
+cByte = 0;
+mrStream.ReadUChar(cByte);
+
+if (cByte)
+{
+if (nXRes)
+maMetadata.maLogSize.setWidth(
+(maMetadata.maPixSize.Width() * 10) / 
nXRes);
+
+if (nYRes)
+maMetadata.maLogSize.setHeight(
+(maMetadata.maPixSize.Height() * 10) / 
nYRes);
+}
+
+nLen32 -= 9;
+}
+else if (nTemp32 == 0x74524e53) // transparency
+{
+maMetadata.mbIsTransparent = true;
+maMetadata.mbIsAlpha = (cColType != 0 && cColType != 
2);
+}
+
+// skip forward to next chunk
+mrStream.SeekRel(4 + nLen32);
+mrStream.ReadUInt32(nLen32);
+mrStream.ReadUInt32(nTemp32);
+}
+} while (false);
+}
 return true;
 }
 return false;
diff --git a/vcl/source/filter/graphicfilter2.cxx 
b/vcl/source/filter/graphicfilter2.cxx
index 0ef05ef1db9d..4327ef5e2f59 100644
--- a/vcl/source/filter/graphicfilter2.cxx
+++ b/vcl/source/filter/graphicfilter2.cxx
@@ -427,113 +427,12 @@ 

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

2022-08-22 Thread offtkp (via logerrit)
 vcl/source/filter/GraphicFormatDetector.cxx |8 +
 vcl/source/filter/graphicfilter2.cxx|   43 ++--
 2 files changed, 18 insertions(+), 33 deletions(-)

New commits:
commit c24b88efc4ac80d25d79b9764e6718d0392f718a
Author: offtkp 
AuthorDate: Fri Aug 12 22:04:35 2022 +0300
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 18:04:02 2022 +0200

Remove code duplication in GraphicDescriptor for RAS

GraphicFormatDetector and GraphicDescriptor have duplicate format
detection code so now GraphicDescriptor uses GraphicFormatDetector
functions instead to detect the format for RAS files

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

diff --git a/vcl/source/filter/graphicfilter2.cxx 
b/vcl/source/filter/graphicfilter2.cxx
index 0706e1b2301c..e622901767e8 100644
--- a/vcl/source/filter/graphicfilter2.cxx
+++ b/vcl/source/filter/graphicfilter2.cxx
@@ -838,16 +838,12 @@ bool GraphicDescriptor::ImpDetectPPM( SvStream& rStm, 
bool )
 
 bool GraphicDescriptor::ImpDetectRAS( SvStream& rStm, bool )
 {
-sal_uInt32 nMagicNumber = 0;
-bool bRet = false;
 sal_Int32 nStmPos = rStm.Tell();
-rStm.SetEndian( SvStreamEndian::BIG );
-rStm.ReadUInt32( nMagicNumber );
-if ( nMagicNumber == 0x59a66a95 )
-{
-aMetadata.mnFormat = GraphicFileFormat::RAS;
-bRet = true;
-}
+vcl::GraphicFormatDetector aDetector( rStm, aPathExt, false /* 
bExtendedInfo */ );
+bool bRet = aDetector.detect();
+bRet &= aDetector.checkRAS();
+if ( bRet )
+aMetadata = aDetector.getMetadata();
 rStm.Seek( nStmPos );
 return bRet;
 }
commit 0853187cbc66f17620c2951d24d1f71c9ea9f81a
Author: offtkp 
AuthorDate: Fri Aug 12 22:01:58 2022 +0300
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 18:03:53 2022 +0200

Remove code duplication in GraphicDescriptor for WEBP

GraphicFormatDetector and GraphicDescriptor have duplicate format
detection code so now GraphicDescriptor uses GraphicFormatDetector
functions instead to detect the format for WEBP files

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

diff --git a/vcl/source/filter/GraphicFormatDetector.cxx 
b/vcl/source/filter/GraphicFormatDetector.cxx
index 684298ac773a..0bd3394bc99d 100644
--- a/vcl/source/filter/GraphicFormatDetector.cxx
+++ b/vcl/source/filter/GraphicFormatDetector.cxx
@@ -25,6 +25,7 @@
 #include 
 #include 
 #include 
+#include 
 #include 
 
 constexpr sal_uInt32 SVG_CHECK_SIZE = 2048;
@@ -884,6 +885,13 @@ bool GraphicFormatDetector::checkWEBP()
 && maFirstBytes[10] == 'B' && maFirstBytes[11] == 'P')
 {
 maMetadata.mnFormat = GraphicFileFormat::WEBP;
+if (mbExtendedInfo)
+{
+mrStream.Seek(mnStreamPosition);
+ReadWebpInfo(mrStream, maMetadata.maPixSize, 
maMetadata.mnBitsPerPixel,
+ maMetadata.mbIsAlpha);
+maMetadata.mbIsTransparent = maMetadata.mbIsAlpha;
+}
 return true;
 }
 return false;
diff --git a/vcl/source/filter/graphicfilter2.cxx 
b/vcl/source/filter/graphicfilter2.cxx
index 93588c40c229..0706e1b2301c 100644
--- a/vcl/source/filter/graphicfilter2.cxx
+++ b/vcl/source/filter/graphicfilter2.cxx
@@ -26,7 +26,6 @@
 #include 
 #include 
 #include 
-#include 
 #include 
 #include "graphicfilter_internal.hxx"
 
@@ -1081,30 +1080,12 @@ bool GraphicDescriptor::ImpDetectSVG( SvStream& rStm, 
bool /*bExtendedInfo*/ )
 
 bool GraphicDescriptor::ImpDetectWEBP( SvStream& rStm, bool bExtendedInfo )
 {
-sal_uInt32  nTemp32 = 0;
-boolbRet = false;
-
 sal_Int32 nStmPos = rStm.Tell();
-rStm.SetEndian( SvStreamEndian::BIG );
-rStm.ReadUInt32( nTemp32 );
-
-if ( nTemp32 == 0x52494646 )
-{
-rStm.ReadUInt32( nTemp32 ); // skip
-rStm.ReadUInt32( nTemp32 );
-if ( nTemp32 == 0x57454250 )
-{
-aMetadata.mnFormat = GraphicFileFormat::WEBP;
-bRet = true;
-
-if ( bExtendedInfo )
-{
-rStm.Seek(nStmPos);
-ReadWebpInfo(rStm, aMetadata.maPixSize, 
aMetadata.mnBitsPerPixel, aMetadata.mbIsAlpha );
-aMetadata.mbIsTransparent = aMetadata.mbIsAlpha;
-}
-}
-}
+vcl::GraphicFormatDetector aDetector( rStm, aPathExt, bExtendedInfo );
+bool bRet = aDetector.detect();
+bRet &= aDetector.checkWEBP();
+if ( bRet )
+aMetadata = aDetector.getMetadata();
 rStm.Seek( nStmPos );
 return bRet;
 }


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

2022-08-22 Thread offtkp (via logerrit)
 vcl/source/filter/graphicfilter2.cxx |   26 --
 1 file changed, 16 insertions(+), 10 deletions(-)

New commits:
commit 3682ef9da1ba3b6479e6bd6bf6d9705c784b4fd4
Author: offtkp 
AuthorDate: Fri Aug 12 21:35:25 2022 +0300
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 18:03:17 2022 +0200

Remove code duplication in GraphicDescriptor for MET

GraphicFormatDetector and GraphicDescriptor have duplicate format
detection code so now GraphicDescriptor uses GraphicFormatDetector
functions instead to detect the format for MET files

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

diff --git a/vcl/source/filter/graphicfilter2.cxx 
b/vcl/source/filter/graphicfilter2.cxx
index 59b468a5317e..93588c40c229 100644
--- a/vcl/source/filter/graphicfilter2.cxx
+++ b/vcl/source/filter/graphicfilter2.cxx
@@ -943,12 +943,15 @@ bool GraphicDescriptor::ImpDetectDXF( SvStream& rStm, 
bool )
 return bRet;
 }
 
-bool GraphicDescriptor::ImpDetectMET( SvStream&, bool )
+bool GraphicDescriptor::ImpDetectMET( SvStream& rStm, bool )
 {
-bool bRet = aPathExt.startsWith( "met" );
-if (bRet)
-aMetadata.mnFormat = GraphicFileFormat::MET;
-
+sal_Int32 nStmPos = rStm.Tell();
+vcl::GraphicFormatDetector aDetector( rStm, aPathExt, false 
/*bExtendedInfo*/ );
+bool bRet = aDetector.detect();
+bRet &= aDetector.checkMET();
+if ( bRet )
+aMetadata = aDetector.getMetadata();
+rStm.Seek( nStmPos );
 return bRet;
 }
 
commit a1b2a8370e61ada2690a29a623d55b38bd7a7cf3
Author: offtkp 
AuthorDate: Fri Aug 12 21:30:19 2022 +0300
Commit: Tomaž Vajngerl 
CommitDate: Mon Aug 22 18:03:05 2022 +0200

Remove code duplication in GraphicDescriptor for TGA

GraphicFormatDetector and GraphicDescriptor have duplicate format
detection code so now GraphicDescriptor uses GraphicFormatDetector
functions instead to detect the format for TGA files

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

diff --git a/vcl/source/filter/graphicfilter2.cxx 
b/vcl/source/filter/graphicfilter2.cxx
index 01741afa7ec2..59b468a5317e 100644
--- a/vcl/source/filter/graphicfilter2.cxx
+++ b/vcl/source/filter/graphicfilter2.cxx
@@ -853,12 +853,15 @@ bool GraphicDescriptor::ImpDetectRAS( SvStream& rStm, 
bool )
 return bRet;
 }
 
-bool GraphicDescriptor::ImpDetectTGA( SvStream&, bool )
+bool GraphicDescriptor::ImpDetectTGA( SvStream& rStm, bool )
 {
-bool bRet = aPathExt.startsWith( "tga" );
-if (bRet)
-aMetadata.mnFormat = GraphicFileFormat::TGA;
-
+sal_Int32 nStmPos = rStm.Tell();
+vcl::GraphicFormatDetector aDetector( rStm, aPathExt, false /* 
bExtendedInfo */ );
+bool bRet = aDetector.detect();
+bRet &= aDetector.checkTGA();
+if ( bRet )
+aMetadata = aDetector.getMetadata();
+rStm.Seek( nStmPos );
 return bRet;
 }
 


  1   2   3   >