tdf#129056: No need for catastrophe
We can continue working without hurting others in case of an invalid reference. Putting in a warning so that it doesn't become a black hole which silences errors. Change-Id: I6d75edeba227a4293b0ae16217811e49ae336a6b Reviewed-on: https://gerrit.libreoffice.org/85111 Tested-by: Jenkins Reviewed-by: Muhammet Kara <muhammet.kara@collabora.com>
This commit is contained in:
@@ -72,8 +72,19 @@ ChartColorWrapper::ChartColorWrapper(
|
||||
|
||||
void ChartColorWrapper::operator()(const OUString& , const NamedColor& rColor)
|
||||
{
|
||||
if (!mxModel.is())
|
||||
{
|
||||
SAL_WARN("chart2", "Invalid reference to mxModel");
|
||||
return;
|
||||
}
|
||||
|
||||
css::uno::Reference<css::beans::XPropertySet> xPropSet = getPropSet(mxModel);
|
||||
assert(xPropSet.is());
|
||||
|
||||
if (!xPropSet.is())
|
||||
{
|
||||
SAL_WARN("chart2", "Invalid reference to xPropSet");
|
||||
return;
|
||||
}
|
||||
|
||||
xPropSet->setPropertyValue(maPropertyName, css::uno::makeAny(rColor.first));
|
||||
}
|
||||
|
||||
Reference in New Issue
Block a user