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

[Libreoffice Gerrit user]

 reportdesign/source/ui/report/ReportController.cxx |    4 ++++
 1 file changed, 4 insertions(+)

New commits:
commit c60ef370a39e91974a242c7f780d7d8725fea7da
Author:     Julien Nabet <serval2412 at yahoo.fr>
AuthorDate: Thu Aug 16 07:30:39 2018 +0200
Commit:     Noel Grandin <noel.grandin at collabora.co.uk>
CommitDate: Thu Aug 16 08:57:57 2018 +0200

    Related tdf#117795: comment fix
    
    Change-Id: Icf3bb150d0009b50295002a781e71a70a77521fa
    Reviewed-on: https://gerrit.libreoffice.org/59147
    Tested-by: Jenkins
    Reviewed-by: Noel Grandin <noel.grandin at collabora.co.uk>

diff --git a/reportdesign/source/ui/report/ReportController.cxx b/reportdesign/source/ui/report/ReportController.cxx
index 1a4964a2ed2c..57c38e54109a 100644
--- a/reportdesign/source/ui/report/ReportController.cxx
+++ b/reportdesign/source/ui/report/ReportController.cxx
@@ -4175,6 +4175,10 @@ bool OReportController::impl_setPropertyAtControls_throw(const char* pUndoResId,
     {
         const uno::Reference< beans::XPropertySet > xControlModel(*aIter,uno::UNO_QUERY);
         if ( xControlModel.is() )
+            // tdf#117795: some elements may have not some property
+            // eg class "OFixedLine" doesn't have property "CharFontName"
+            // so in this case, instead of crashing when selecting all and changing font
+            // just display a warning
             try
             {
                 xControlModel->setPropertyValue(_sProperty,_aValue);