include/xmloff/shapeexport.hxx | 114 +++++++------- xmloff/source/draw/shapeexport.cxx | 292 ++++++++++++++++++------------------- xmloff/source/draw/ximpshap.cxx | 16 +- 3 files changed, 211 insertions(+), 211 deletions(-)
New commits: commit ac30e6bfb653490003d04f30bf2d5f8f857e572b Author: Noel Grandin <noelgran...@gmail.com> AuthorDate: Mon Apr 17 19:11:29 2023 +0200 Commit: Noel Grandin <noel.gran...@collabora.co.uk> CommitDate: Tue Apr 18 09:43:04 2023 +0200 convert XmlShapeType to scoped enum Change-Id: I1fd3d995508eed507d07de11c4f60f05d075b84f Reviewed-on: https://gerrit.libreoffice.org/c/core/+/150528 Tested-by: Jenkins Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk> diff --git a/include/xmloff/shapeexport.hxx b/include/xmloff/shapeexport.hxx index 34abaf37547e..5313fe9e48f9 100644 --- a/include/xmloff/shapeexport.hxx +++ b/include/xmloff/shapeexport.hxx @@ -67,63 +67,63 @@ namespace o3tl #define SEF_DEFAULT XMLShapeExportFlags::POSITION|XMLShapeExportFlags::SIZE -enum XmlShapeType +enum class XmlShapeType { - XmlShapeTypeUnknown, // not known - - XmlShapeTypeDrawRectangleShape, // "com.sun.star.drawing.RectangleShape" - XmlShapeTypeDrawEllipseShape, // "com.sun.star.drawing.EllipseShape" - XmlShapeTypeDrawControlShape, // "com.sun.star.drawing.ControlShape" - XmlShapeTypeDrawConnectorShape, // "com.sun.star.drawing.ConnectorShape" - XmlShapeTypeDrawMeasureShape, // "com.sun.star.drawing.MeasureShape" - XmlShapeTypeDrawLineShape, // "com.sun.star.drawing.LineShape" - XmlShapeTypeDrawPolyPolygonShape, // "com.sun.star.drawing.PolyPolygonShape" - XmlShapeTypeDrawPolyLineShape, // "com.sun.star.drawing.PolyLineShape" - XmlShapeTypeDrawOpenBezierShape, // "com.sun.star.drawing.OpenBezierShape" - XmlShapeTypeDrawClosedBezierShape, // "com.sun.star.drawing.ClosedBezierShape" - XmlShapeTypeDrawGraphicObjectShape, // "com.sun.star.drawing.GraphicObjectShape" - XmlShapeTypeDrawGroupShape, // "com.sun.star.drawing.GroupShape" - XmlShapeTypeDrawTextShape, // "com.sun.star.drawing.TextShape" - XmlShapeTypeDrawOLE2Shape, // "com.sun.star.drawing.OLE2Shape" - XmlShapeTypeDrawChartShape, // embedded com.sun.star.chart - XmlShapeTypeDrawSheetShape, // embedded com.sun.star.sheet - XmlShapeTypeDrawPageShape, // "com.sun.star.drawing.PageShape" - XmlShapeTypeDrawFrameShape, // "com.sun.star.drawing.FrameShape" - XmlShapeTypeDrawCaptionShape, // "com.sun.star.drawing.CaptionShape" - XmlShapeTypeDrawAppletShape, // "com.sun.star.drawing.AppletShape" - XmlShapeTypeDrawPluginShape, // "com.sun.star.drawing.PlugginShape" - - XmlShapeTypeDraw3DSceneObject, // "com.sun.star.drawing.Shape3DSceneObject" - XmlShapeTypeDraw3DCubeObject, // "com.sun.star.drawing.Shape3DCubeObject" - XmlShapeTypeDraw3DSphereObject, // "com.sun.star.drawing.Shape3DSphereObject" - XmlShapeTypeDraw3DLatheObject, // "com.sun.star.drawing.Shape3DLatheObject" - XmlShapeTypeDraw3DExtrudeObject, // "com.sun.star.drawing.Shape3DExtrudeObject" - - XmlShapeTypePresTitleTextShape, // "com.sun.star.presentation.TitleTextShape" - XmlShapeTypePresOutlinerShape, // "com.sun.star.presentation.OutlinerShape" - XmlShapeTypePresSubtitleShape, // "com.sun.star.presentation.SubtitleShape" - XmlShapeTypePresGraphicObjectShape, // "com.sun.star.presentation.GraphicObjectShape" - XmlShapeTypePresPageShape, // "com.sun.star.presentation.PageShape" - XmlShapeTypePresOLE2Shape, // "com.sun.star.presentation.OLE2Shape" - XmlShapeTypePresChartShape, // "com.sun.star.presentation.ChartShape" - XmlShapeTypePresSheetShape, // "com.sun.star.presentation.CalcShape" - XmlShapeTypePresTableShape, // "com.sun.star.presentation.TableShape" - XmlShapeTypePresOrgChartShape, // "com.sun.star.presentation.OrgChartShape" - XmlShapeTypePresNotesShape, // "com.sun.star.presentation.NotesShape" - XmlShapeTypeHandoutShape, // "com.sun.star.presentation.HandoutShape" - - XmlShapeTypePresHeaderShape, // "com.sun.star.presentation.HeaderShape" - XmlShapeTypePresFooterShape, // "com.sun.star.presentation.FooterShape" - XmlShapeTypePresSlideNumberShape, // "com.sun.star.presentation.SlideNumberShape" - XmlShapeTypePresDateTimeShape, // "com.sun.star.presentation.DateTimeShape" - - XmlShapeTypeDrawCustomShape, // "com.sun.star.drawing.CustomShape" - XmlShapeTypeDrawMediaShape, // "com.sun.star.drawing.MediaShape" - XmlShapeTypePresMediaShape, // "com.sun.star.presentation.MediaShape" - - XmlShapeTypeDrawTableShape, // "com.sun.star.drawing.TableShape" - - XmlShapeTypeNotYetSet + Unknown, // not known + + DrawRectangleShape, // "com.sun.star.drawing.RectangleShape" + DrawEllipseShape, // "com.sun.star.drawing.EllipseShape" + DrawControlShape, // "com.sun.star.drawing.ControlShape" + DrawConnectorShape, // "com.sun.star.drawing.ConnectorShape" + DrawMeasureShape, // "com.sun.star.drawing.MeasureShape" + DrawLineShape, // "com.sun.star.drawing.LineShape" + DrawPolyPolygonShape, // "com.sun.star.drawing.PolyPolygonShape" + DrawPolyLineShape, // "com.sun.star.drawing.PolyLineShape" + DrawOpenBezierShape, // "com.sun.star.drawing.OpenBezierShape" + DrawClosedBezierShape, // "com.sun.star.drawing.ClosedBezierShape" + DrawGraphicObjectShape, // "com.sun.star.drawing.GraphicObjectShape" + DrawGroupShape, // "com.sun.star.drawing.GroupShape" + DrawTextShape, // "com.sun.star.drawing.TextShape" + DrawOLE2Shape, // "com.sun.star.drawing.OLE2Shape" + DrawChartShape, // embedded com.sun.star.chart + DrawSheetShape, // embedded com.sun.star.sheet + DrawPageShape, // "com.sun.star.drawing.PageShape" + DrawFrameShape, // "com.sun.star.drawing.FrameShape" + DrawCaptionShape, // "com.sun.star.drawing.CaptionShape" + DrawAppletShape, // "com.sun.star.drawing.AppletShape" + DrawPluginShape, // "com.sun.star.drawing.PlugginShape" + + Draw3DSceneObject, // "com.sun.star.drawing.Shape3DSceneObject" + Draw3DCubeObject, // "com.sun.star.drawing.Shape3DCubeObject" + Draw3DSphereObject, // "com.sun.star.drawing.Shape3DSphereObject" + Draw3DLatheObject, // "com.sun.star.drawing.Shape3DLatheObject" + Draw3DExtrudeObject, // "com.sun.star.drawing.Shape3DExtrudeObject" + + PresTitleTextShape, // "com.sun.star.presentation.TitleTextShape" + PresOutlinerShape, // "com.sun.star.presentation.OutlinerShape" + PresSubtitleShape, // "com.sun.star.presentation.SubtitleShape" + PresGraphicObjectShape, // "com.sun.star.presentation.GraphicObjectShape" + PresPageShape, // "com.sun.star.presentation.PageShape" + PresOLE2Shape, // "com.sun.star.presentation.OLE2Shape" + PresChartShape, // "com.sun.star.presentation.ChartShape" + PresSheetShape, // "com.sun.star.presentation.CalcShape" + PresTableShape, // "com.sun.star.presentation.TableShape" + PresOrgChartShape, // "com.sun.star.presentation.OrgChartShape" + PresNotesShape, // "com.sun.star.presentation.NotesShape" + HandoutShape, // "com.sun.star.presentation.HandoutShape" + + PresHeaderShape, // "com.sun.star.presentation.HeaderShape" + PresFooterShape, // "com.sun.star.presentation.FooterShape" + PresSlideNumberShape, // "com.sun.star.presentation.SlideNumberShape" + PresDateTimeShape, // "com.sun.star.presentation.DateTimeShape" + + DrawCustomShape, // "com.sun.star.drawing.CustomShape" + DrawMediaShape, // "com.sun.star.drawing.MediaShape" + PresMediaShape, // "com.sun.star.presentation.MediaShape" + + DrawTableShape, // "com.sun.star.drawing.TableShape" + + NotYetSet }; /** caches style and type info after a collectShapeAutostyle for later use in exportShape */ @@ -136,7 +136,7 @@ struct ImplXMLShapeExportInfo css::uno::Reference< css::drawing::XShape > xCustomShapeReplacement; - ImplXMLShapeExportInfo() : mnFamily( XmlStyleFamily::SD_GRAPHICS_ID ), meShapeType( XmlShapeTypeNotYetSet ) {} + ImplXMLShapeExportInfo() : mnFamily( XmlStyleFamily::SD_GRAPHICS_ID ), meShapeType( XmlShapeType::NotYetSet ) {} }; /** a vector for shape style and type cache information */ diff --git a/xmloff/source/draw/shapeexport.cxx b/xmloff/source/draw/shapeexport.cxx index 94b7ad8e78a0..d9661a531986 100644 --- a/xmloff/source/draw/shapeexport.cxx +++ b/xmloff/source/draw/shapeexport.cxx @@ -137,18 +137,18 @@ namespace { bool supportsText(XmlShapeType eShapeType) { - return eShapeType != XmlShapeTypePresChartShape && - eShapeType != XmlShapeTypePresOLE2Shape && - eShapeType != XmlShapeTypeDrawSheetShape && - eShapeType != XmlShapeTypePresSheetShape && - eShapeType != XmlShapeTypeDraw3DSceneObject && - eShapeType != XmlShapeTypeDraw3DCubeObject && - eShapeType != XmlShapeTypeDraw3DSphereObject && - eShapeType != XmlShapeTypeDraw3DLatheObject && - eShapeType != XmlShapeTypeDraw3DExtrudeObject && - eShapeType != XmlShapeTypeDrawPageShape && - eShapeType != XmlShapeTypePresPageShape && - eShapeType != XmlShapeTypeDrawGroupShape; + return eShapeType != XmlShapeType::PresChartShape && + eShapeType != XmlShapeType::PresOLE2Shape && + eShapeType != XmlShapeType::DrawSheetShape && + eShapeType != XmlShapeType::PresSheetShape && + eShapeType != XmlShapeType::Draw3DSceneObject && + eShapeType != XmlShapeType::Draw3DCubeObject && + eShapeType != XmlShapeType::Draw3DSphereObject && + eShapeType != XmlShapeType::Draw3DLatheObject && + eShapeType != XmlShapeType::Draw3DExtrudeObject && + eShapeType != XmlShapeType::DrawPageShape && + eShapeType != XmlShapeType::PresPageShape && + eShapeType != XmlShapeType::DrawGroupShape; } @@ -291,13 +291,13 @@ void XMLShapeExport::collectShapeAutoStyles(const uno::Reference< drawing::XShap // first compute the shapes type ImpCalcShapeType(xShape, aShapeInfo.meShapeType); - // #i118485# enabled XmlShapeTypeDrawChartShape and XmlShapeTypeDrawOLE2Shape + // #i118485# enabled XmlShapeType::DrawChartShape and XmlShapeType::DrawOLE2Shape // to have text const bool bObjSupportsText = supportsText(aShapeInfo.meShapeType); const bool bObjSupportsStyle = - aShapeInfo.meShapeType != XmlShapeTypeDrawGroupShape; + aShapeInfo.meShapeType != XmlShapeType::DrawGroupShape; bool bIsEmptyPresObj = false; @@ -393,11 +393,11 @@ void XMLShapeExport::collectShapeAutoStyles(const uno::Reference< drawing::XShap std::vector< XMLPropertyState > aPropStates; sal_Int32 nCount = 0; - if( !bIsEmptyPresObj || (aShapeInfo.meShapeType != XmlShapeTypePresPageShape) ) + if( !bIsEmptyPresObj || (aShapeInfo.meShapeType != XmlShapeType::PresPageShape) ) { aPropStates = GetPropertySetMapper()->Filter(mrExport, xPropSet); - if (XmlShapeTypeDrawControlShape == aShapeInfo.meShapeType) + if (XmlShapeType::DrawControlShape == aShapeInfo.meShapeType) { // for control shapes, we additionally need the number format style (if any) uno::Reference< drawing::XControlShape > xControl(xShape, uno::UNO_QUERY); @@ -444,12 +444,12 @@ void XMLShapeExport::collectShapeAutoStyles(const uno::Reference< drawing::XShap } // optionally generate auto style for text attributes - if( (!bIsEmptyPresObj || (aShapeInfo.meShapeType != XmlShapeTypePresPageShape)) && bObjSupportsText ) + if( (!bIsEmptyPresObj || (aShapeInfo.meShapeType != XmlShapeType::PresPageShape)) && bObjSupportsText ) { aPropStates = GetExport().GetTextParagraphExport()->GetParagraphPropertyMapper()->Filter(mrExport, xPropSet); // yet more additionally, we need to care for the ParaAdjust property - if ( XmlShapeTypeDrawControlShape == aShapeInfo.meShapeType ) + if ( XmlShapeType::DrawControlShape == aShapeInfo.meShapeType ) { uno::Reference< beans::XPropertySetInfo > xPropSetInfo( xPropSet->getPropertySetInfo() ); uno::Reference< beans::XPropertyState > xPropState( xPropSet, uno::UNO_QUERY ); @@ -502,7 +502,7 @@ void XMLShapeExport::collectShapeAutoStyles(const uno::Reference< drawing::XShap switch( aShapeInfo.meShapeType ) { - case XmlShapeTypeDrawConnectorShape: + case XmlShapeType::DrawConnectorShape: { uno::Reference< uno::XInterface > xConnection; @@ -516,8 +516,8 @@ void XMLShapeExport::collectShapeAutoStyles(const uno::Reference< drawing::XShap mrExport.getInterfaceToIdentifierMapper().registerReference( xConnection ); break; } - case XmlShapeTypePresTableShape: - case XmlShapeTypeDrawTableShape: + case XmlShapeType::PresTableShape: + case XmlShapeType::DrawTableShape: { try { @@ -661,7 +661,7 @@ void XMLShapeExport::exportShape(const uno::Reference< drawing::XShape >& xShape // first compute the shapes type { - XmlShapeType eShapeType(XmlShapeTypeNotYetSet); + XmlShapeType eShapeType(XmlShapeType::NotYetSet); ImpCalcShapeType(xShape, eShapeType); SAL_WARN_IF( eShapeType != aShapeInfo.meShapeType, "xmloff", "exportShape callings do not correspond to collectShapeAutoStyles calls!: " << xShape->getShapeType() ); @@ -683,8 +683,8 @@ void XMLShapeExport::exportShape(const uno::Reference< drawing::XShape >& xShape GetExport().GetModelType() != SvtModuleOptions::EFactory::WRITERWEB && GetExport().GetModelType() != SvtModuleOptions::EFactory::WRITERGLOBAL ) || ( GetExport().getExportFlags() & SvXMLExportFlags::OASIS ) || - aShapeInfo.meShapeType == XmlShapeTypeDrawGroupShape || - ( aShapeInfo.meShapeType == XmlShapeTypeDrawCustomShape && + aShapeInfo.meShapeType == XmlShapeType::DrawGroupShape || + ( aShapeInfo.meShapeType == XmlShapeType::DrawCustomShape && aShapeInfo.xCustomShapeReplacement.is() ) ) { uno::Reference< container::XNamed > xNamed( xShape, uno::UNO_QUERY ); @@ -747,8 +747,8 @@ void XMLShapeExport::exportShape(const uno::Reference< drawing::XShape >& xShape // export draw:display (do not export in ODF 1.3 or older) if (xSet.is() && (mrExport.getSaneDefaultVersion() & SvtSaveOptions::ODFSVER_EXTENDED)) { - if( aShapeInfo.meShapeType != XmlShapeTypeDrawPageShape && aShapeInfo.meShapeType != XmlShapeTypePresPageShape && - aShapeInfo.meShapeType != XmlShapeTypeHandoutShape && aShapeInfo.meShapeType != XmlShapeTypeDrawChartShape ) + if( aShapeInfo.meShapeType != XmlShapeType::DrawPageShape && aShapeInfo.meShapeType != XmlShapeType::PresPageShape && + aShapeInfo.meShapeType != XmlShapeType::HandoutShape && aShapeInfo.meShapeType != XmlShapeType::DrawChartShape ) try { bool bVisible = true; @@ -788,147 +788,147 @@ void XMLShapeExport::exportShape(const uno::Reference< drawing::XShape >& xShape // export shape element switch(aShapeInfo.meShapeType) { - case XmlShapeTypeDrawRectangleShape: + case XmlShapeType::DrawRectangleShape: { ImpExportRectangleShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawEllipseShape: + case XmlShapeType::DrawEllipseShape: { ImpExportEllipseShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawLineShape: + case XmlShapeType::DrawLineShape: { ImpExportLineShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawPolyPolygonShape: // closed PolyPolygon - case XmlShapeTypeDrawPolyLineShape: // open PolyPolygon - case XmlShapeTypeDrawClosedBezierShape: // closed tools::PolyPolygon containing curves - case XmlShapeTypeDrawOpenBezierShape: // open tools::PolyPolygon containing curves + case XmlShapeType::DrawPolyPolygonShape: // closed PolyPolygon + case XmlShapeType::DrawPolyLineShape: // open PolyPolygon + case XmlShapeType::DrawClosedBezierShape: // closed tools::PolyPolygon containing curves + case XmlShapeType::DrawOpenBezierShape: // open tools::PolyPolygon containing curves { ImpExportPolygonShape(xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawTextShape: - case XmlShapeTypePresTitleTextShape: - case XmlShapeTypePresOutlinerShape: - case XmlShapeTypePresSubtitleShape: - case XmlShapeTypePresNotesShape: - case XmlShapeTypePresHeaderShape: - case XmlShapeTypePresFooterShape: - case XmlShapeTypePresSlideNumberShape: - case XmlShapeTypePresDateTimeShape: + case XmlShapeType::DrawTextShape: + case XmlShapeType::PresTitleTextShape: + case XmlShapeType::PresOutlinerShape: + case XmlShapeType::PresSubtitleShape: + case XmlShapeType::PresNotesShape: + case XmlShapeType::PresHeaderShape: + case XmlShapeType::PresFooterShape: + case XmlShapeType::PresSlideNumberShape: + case XmlShapeType::PresDateTimeShape: { ImpExportTextBoxShape(xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawGraphicObjectShape: - case XmlShapeTypePresGraphicObjectShape: + case XmlShapeType::DrawGraphicObjectShape: + case XmlShapeType::PresGraphicObjectShape: { ImpExportGraphicObjectShape(xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawChartShape: - case XmlShapeTypePresChartShape: + case XmlShapeType::DrawChartShape: + case XmlShapeType::PresChartShape: { ImpExportChartShape(xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint, pAttrList ); break; } - case XmlShapeTypeDrawControlShape: + case XmlShapeType::DrawControlShape: { ImpExportControlShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawConnectorShape: + case XmlShapeType::DrawConnectorShape: { ImpExportConnectorShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawMeasureShape: + case XmlShapeType::DrawMeasureShape: { ImpExportMeasureShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawOLE2Shape: - case XmlShapeTypePresOLE2Shape: - case XmlShapeTypeDrawSheetShape: - case XmlShapeTypePresSheetShape: + case XmlShapeType::DrawOLE2Shape: + case XmlShapeType::PresOLE2Shape: + case XmlShapeType::DrawSheetShape: + case XmlShapeType::PresSheetShape: { ImpExportOLE2Shape(xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypePresTableShape: - case XmlShapeTypeDrawTableShape: + case XmlShapeType::PresTableShape: + case XmlShapeType::DrawTableShape: { ImpExportTableShape( xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawPageShape: - case XmlShapeTypePresPageShape: - case XmlShapeTypeHandoutShape: + case XmlShapeType::DrawPageShape: + case XmlShapeType::PresPageShape: + case XmlShapeType::HandoutShape: { ImpExportPageShape(xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawCaptionShape: + case XmlShapeType::DrawCaptionShape: { ImpExportCaptionShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDraw3DCubeObject: - case XmlShapeTypeDraw3DSphereObject: - case XmlShapeTypeDraw3DLatheObject: - case XmlShapeTypeDraw3DExtrudeObject: + case XmlShapeType::Draw3DCubeObject: + case XmlShapeType::Draw3DSphereObject: + case XmlShapeType::Draw3DLatheObject: + case XmlShapeType::Draw3DExtrudeObject: { ImpExport3DShape(xShape, aShapeInfo.meShapeType); break; } - case XmlShapeTypeDraw3DSceneObject: + case XmlShapeType::Draw3DSceneObject: { ImpExport3DSceneShape( xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawGroupShape: + case XmlShapeType::DrawGroupShape: { // empty group ImpExportGroupShape( xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawFrameShape: + case XmlShapeType::DrawFrameShape: { ImpExportFrameShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawAppletShape: + case XmlShapeType::DrawAppletShape: { ImpExportAppletShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawPluginShape: + case XmlShapeType::DrawPluginShape: { ImpExportPluginShape(xShape, nFeatures, pRefPoint ); break; } - case XmlShapeTypeDrawCustomShape: + case XmlShapeType::DrawCustomShape: { if ( aShapeInfo.xCustomShapeReplacement.is() ) ImpExportGroupShape( aShapeInfo.xCustomShapeReplacement, nFeatures, pRefPoint ); @@ -937,16 +937,16 @@ void XMLShapeExport::exportShape(const uno::Reference< drawing::XShape >& xShape break; } - case XmlShapeTypePresMediaShape: - case XmlShapeTypeDrawMediaShape: + case XmlShapeType::PresMediaShape: + case XmlShapeType::DrawMediaShape: { ImpExportMediaShape( xShape, aShapeInfo.meShapeType, nFeatures, pRefPoint ); break; } - case XmlShapeTypePresOrgChartShape: - case XmlShapeTypeUnknown: - case XmlShapeTypeNotYetSet: + case XmlShapeType::PresOrgChartShape: + case XmlShapeType::Unknown: + case XmlShapeType::NotYetSet: default: { // this should never happen and is an error @@ -1143,7 +1143,7 @@ void XMLShapeExport::ImpCalcShapeType(const uno::Reference< drawing::XShape >& x XmlShapeType& eShapeType) { // set in every case, so init here - eShapeType = XmlShapeTypeUnknown; + eShapeType = XmlShapeType::Unknown; if(!xShape.is()) return; @@ -1156,38 +1156,38 @@ void XMLShapeExport::ImpCalcShapeType(const uno::Reference< drawing::XShape >& x if(aType.match("drawing.", 13)) { // drawing shapes - if (aType.match("Rectangle", 21)) { eShapeType = XmlShapeTypeDrawRectangleShape; } + if (aType.match("Rectangle", 21)) { eShapeType = XmlShapeType::DrawRectangleShape; } // #i72177# Note: Correcting CustomShape, CustomShape->Custom, len from 9 (was wrong anyways) to 6. // As can be seen at the other compares, the appendix "Shape" is left out of the comparison. - else if(aType.match("Custom", 21)) { eShapeType = XmlShapeTypeDrawCustomShape; } + else if(aType.match("Custom", 21)) { eShapeType = XmlShapeType::DrawCustomShape; } - else if(aType.match("Ellipse", 21)) { eShapeType = XmlShapeTypeDrawEllipseShape; } - else if(aType.match("Control", 21)) { eShapeType = XmlShapeTypeDrawControlShape; } - else if(aType.match("Connector", 21)) { eShapeType = XmlShapeTypeDrawConnectorShape; } - else if(aType.match("Measure", 21)) { eShapeType = XmlShapeTypeDrawMeasureShape; } - else if(aType.match("Line", 21)) { eShapeType = XmlShapeTypeDrawLineShape; } + else if(aType.match("Ellipse", 21)) { eShapeType = XmlShapeType::DrawEllipseShape; } + else if(aType.match("Control", 21)) { eShapeType = XmlShapeType::DrawControlShape; } + else if(aType.match("Connector", 21)) { eShapeType = XmlShapeType::DrawConnectorShape; } + else if(aType.match("Measure", 21)) { eShapeType = XmlShapeType::DrawMeasureShape; } + else if(aType.match("Line", 21)) { eShapeType = XmlShapeType::DrawLineShape; } // #i72177# Note: This covers two types by purpose, PolyPolygonShape and PolyPolygonPathShape - else if(aType.match("PolyPolygon", 21)) { eShapeType = XmlShapeTypeDrawPolyPolygonShape; } + else if(aType.match("PolyPolygon", 21)) { eShapeType = XmlShapeType::DrawPolyPolygonShape; } // #i72177# Note: This covers two types by purpose, PolyLineShape and PolyLinePathShape - else if(aType.match("PolyLine", 21)) { eShapeType = XmlShapeTypeDrawPolyLineShape; } + else if(aType.match("PolyLine", 21)) { eShapeType = XmlShapeType::DrawPolyLineShape; } - else if(aType.match("OpenBezier", 21)) { eShapeType = XmlShapeTypeDrawOpenBezierShape; } - else if(aType.match("ClosedBezier", 21)) { eShapeType = XmlShapeTypeDrawClosedBezierShape; } + else if(aType.match("OpenBezier", 21)) { eShapeType = XmlShapeType::DrawOpenBezierShape; } + else if(aType.match("ClosedBezier", 21)) { eShapeType = XmlShapeType::DrawClosedBezierShape; } // #i72177# FreeHand (opened and closed) now supports the types OpenFreeHandShape and // ClosedFreeHandShape respectively. Represent them as bezier shapes - else if(aType.match("OpenFreeHand", 21)) { eShapeType = XmlShapeTypeDrawOpenBezierShape; } - else if(aType.match("ClosedFreeHand", 21)) { eShapeType = XmlShapeTypeDrawClosedBezierShape; } + else if(aType.match("OpenFreeHand", 21)) { eShapeType = XmlShapeType::DrawOpenBezierShape; } + else if(aType.match("ClosedFreeHand", 21)) { eShapeType = XmlShapeType::DrawClosedBezierShape; } - else if(aType.match("GraphicObject", 21)) { eShapeType = XmlShapeTypeDrawGraphicObjectShape; } - else if(aType.match("Group", 21)) { eShapeType = XmlShapeTypeDrawGroupShape; } - else if(aType.match("Text", 21)) { eShapeType = XmlShapeTypeDrawTextShape; } + else if(aType.match("GraphicObject", 21)) { eShapeType = XmlShapeType::DrawGraphicObjectShape; } + else if(aType.match("Group", 21)) { eShapeType = XmlShapeType::DrawGroupShape; } + else if(aType.match("Text", 21)) { eShapeType = XmlShapeType::DrawTextShape; } else if(aType.match("OLE2", 21)) { - eShapeType = XmlShapeTypeDrawOLE2Shape; + eShapeType = XmlShapeType::DrawOLE2Shape; // get info about presentation shape uno::Reference <beans::XPropertySet> xPropSet(xShape, uno::UNO_QUERY); @@ -1207,11 +1207,11 @@ void XMLShapeExport::ImpCalcShapeType(const uno::Reference< drawing::XShape >& x #endif sCLSID == SvGlobalName( SO3_RPTCH_CLASSID ).GetHexName() ) { - eShapeType = XmlShapeTypeDrawChartShape; + eShapeType = XmlShapeType::DrawChartShape; } else if (sCLSID == SvGlobalName( SO3_SC_CLASSID ).GetHexName() ) { - eShapeType = XmlShapeTypeDrawSheetShape; + eShapeType = XmlShapeType::DrawSheetShape; } else { @@ -1220,32 +1220,32 @@ void XMLShapeExport::ImpCalcShapeType(const uno::Reference< drawing::XShape >& x } } } - else if(aType.match("Page", 21)) { eShapeType = XmlShapeTypeDrawPageShape; } - else if(aType.match("Frame", 21)) { eShapeType = XmlShapeTypeDrawFrameShape; } - else if(aType.match("Caption", 21)) { eShapeType = XmlShapeTypeDrawCaptionShape; } - else if(aType.match("Plugin", 21)) { eShapeType = XmlShapeTypeDrawPluginShape; } - else if(aType.match("Applet", 21)) { eShapeType = XmlShapeTypeDrawAppletShape; } - else if(aType.match("MediaShape", 21)) { eShapeType = XmlShapeTypeDrawMediaShape; } - else if(aType.match("TableShape", 21)) { eShapeType = XmlShapeTypeDrawTableShape; } + else if(aType.match("Page", 21)) { eShapeType = XmlShapeType::DrawPageShape; } + else if(aType.match("Frame", 21)) { eShapeType = XmlShapeType::DrawFrameShape; } + else if(aType.match("Caption", 21)) { eShapeType = XmlShapeType::DrawCaptionShape; } + else if(aType.match("Plugin", 21)) { eShapeType = XmlShapeType::DrawPluginShape; } + else if(aType.match("Applet", 21)) { eShapeType = XmlShapeType::DrawAppletShape; } + else if(aType.match("MediaShape", 21)) { eShapeType = XmlShapeType::DrawMediaShape; } + else if(aType.match("TableShape", 21)) { eShapeType = XmlShapeType::DrawTableShape; } // 3D shapes - else if(aType.match("Scene", 21 + 7)) { eShapeType = XmlShapeTypeDraw3DSceneObject; } - else if(aType.match("Cube", 21 + 7)) { eShapeType = XmlShapeTypeDraw3DCubeObject; } - else if(aType.match("Sphere", 21 + 7)) { eShapeType = XmlShapeTypeDraw3DSphereObject; } - else if(aType.match("Lathe", 21 + 7)) { eShapeType = XmlShapeTypeDraw3DLatheObject; } - else if(aType.match("Extrude", 21 + 7)) { eShapeType = XmlShapeTypeDraw3DExtrudeObject; } + else if(aType.match("Scene", 21 + 7)) { eShapeType = XmlShapeType::Draw3DSceneObject; } + else if(aType.match("Cube", 21 + 7)) { eShapeType = XmlShapeType::Draw3DCubeObject; } + else if(aType.match("Sphere", 21 + 7)) { eShapeType = XmlShapeType::Draw3DSphereObject; } + else if(aType.match("Lathe", 21 + 7)) { eShapeType = XmlShapeType::Draw3DLatheObject; } + else if(aType.match("Extrude", 21 + 7)) { eShapeType = XmlShapeType::Draw3DExtrudeObject; } } else if(aType.match("presentation.", 13)) { // presentation shapes - if (aType.match("TitleText", 26)) { eShapeType = XmlShapeTypePresTitleTextShape; } - else if(aType.match("Outliner", 26)) { eShapeType = XmlShapeTypePresOutlinerShape; } - else if(aType.match("Subtitle", 26)) { eShapeType = XmlShapeTypePresSubtitleShape; } - else if(aType.match("GraphicObject", 26)) { eShapeType = XmlShapeTypePresGraphicObjectShape; } - else if(aType.match("Page", 26)) { eShapeType = XmlShapeTypePresPageShape; } + if (aType.match("TitleText", 26)) { eShapeType = XmlShapeType::PresTitleTextShape; } + else if(aType.match("Outliner", 26)) { eShapeType = XmlShapeType::PresOutlinerShape; } + else if(aType.match("Subtitle", 26)) { eShapeType = XmlShapeType::PresSubtitleShape; } + else if(aType.match("GraphicObject", 26)) { eShapeType = XmlShapeType::PresGraphicObjectShape; } + else if(aType.match("Page", 26)) { eShapeType = XmlShapeType::PresPageShape; } else if(aType.match("OLE2", 26)) { - eShapeType = XmlShapeTypePresOLE2Shape; + eShapeType = XmlShapeType::PresOLE2Shape; // get info about presentation shape uno::Reference <beans::XPropertySet> xPropSet(xShape, uno::UNO_QUERY); @@ -1257,7 +1257,7 @@ void XMLShapeExport::ImpCalcShapeType(const uno::Reference< drawing::XShape >& x { if( sCLSID == SvGlobalName( SO3_SC_CLASSID ).GetHexName() ) { - eShapeType = XmlShapeTypePresSheetShape; + eShapeType = XmlShapeType::PresSheetShape; } } } @@ -1266,17 +1266,17 @@ void XMLShapeExport::ImpCalcShapeType(const uno::Reference< drawing::XShape >& x SAL_WARN( "xmloff", "XMLShapeExport::ImpCalcShapeType(), expected ole shape to have the CLSID property?" ); } } - else if(aType.match("Chart", 26)) { eShapeType = XmlShapeTypePresChartShape; } - else if(aType.match("OrgChart", 26)) { eShapeType = XmlShapeTypePresOrgChartShape; } - else if(aType.match("CalcShape", 26)) { eShapeType = XmlShapeTypePresSheetShape; } - else if(aType.match("TableShape", 26)) { eShapeType = XmlShapeTypePresTableShape; } - else if(aType.match("Notes", 26)) { eShapeType = XmlShapeTypePresNotesShape; } - else if(aType.match("HandoutShape", 26)) { eShapeType = XmlShapeTypeHandoutShape; } - else if(aType.match("HeaderShape", 26)) { eShapeType = XmlShapeTypePresHeaderShape; } - else if(aType.match("FooterShape", 26)) { eShapeType = XmlShapeTypePresFooterShape; } - else if(aType.match("SlideNumberShape", 26)) { eShapeType = XmlShapeTypePresSlideNumberShape; } - else if(aType.match("DateTimeShape", 26)) { eShapeType = XmlShapeTypePresDateTimeShape; } - else if(aType.match("MediaShape", 26)) { eShapeType = XmlShapeTypePresMediaShape; } + else if(aType.match("Chart", 26)) { eShapeType = XmlShapeType::PresChartShape; } + else if(aType.match("OrgChart", 26)) { eShapeType = XmlShapeType::PresOrgChartShape; } + else if(aType.match("CalcShape", 26)) { eShapeType = XmlShapeType::PresSheetShape; } + else if(aType.match("TableShape", 26)) { eShapeType = XmlShapeType::PresTableShape; } + else if(aType.match("Notes", 26)) { eShapeType = XmlShapeType::PresNotesShape; } + else if(aType.match("HandoutShape", 26)) { eShapeType = XmlShapeType::HandoutShape; } + else if(aType.match("HeaderShape", 26)) { eShapeType = XmlShapeType::PresHeaderShape; } + else if(aType.match("FooterShape", 26)) { eShapeType = XmlShapeType::PresFooterShape; } + else if(aType.match("SlideNumberShape", 26)) { eShapeType = XmlShapeType::PresSlideNumberShape; } + else if(aType.match("DateTimeShape", 26)) { eShapeType = XmlShapeType::PresDateTimeShape; } + else if(aType.match("MediaShape", 26)) { eShapeType = XmlShapeType::PresMediaShape; } } } @@ -2022,49 +2022,49 @@ void XMLShapeExport::ImpExportTextBoxShape( switch(eShapeType) { - case XmlShapeTypePresSubtitleShape: + case XmlShapeType::PresSubtitleShape: { aStr = GetXMLToken(XML_SUBTITLE); bIsPresShape = true; break; } - case XmlShapeTypePresTitleTextShape: + case XmlShapeType::PresTitleTextShape: { aStr = GetXMLToken(XML_TITLE); bIsPresShape = true; break; } - case XmlShapeTypePresOutlinerShape: + case XmlShapeType::PresOutlinerShape: { aStr = GetXMLToken(XML_PRESENTATION_OUTLINE); bIsPresShape = true; break; } - case XmlShapeTypePresNotesShape: + case XmlShapeType::PresNotesShape: { aStr = GetXMLToken(XML_NOTES); bIsPresShape = true; break; } - case XmlShapeTypePresHeaderShape: + case XmlShapeType::PresHeaderShape: { aStr = GetXMLToken(XML_HEADER); bIsPresShape = true; break; } - case XmlShapeTypePresFooterShape: + case XmlShapeType::PresFooterShape: { aStr = GetXMLToken(XML_FOOTER); bIsPresShape = true; break; } - case XmlShapeTypePresSlideNumberShape: + case XmlShapeType::PresSlideNumberShape: { aStr = GetXMLToken(XML_PAGE_NUMBER); bIsPresShape = true; break; } - case XmlShapeTypePresDateTimeShape: + case XmlShapeType::PresDateTimeShape: { aStr = GetXMLToken(XML_DATE_TIME); bIsPresShape = true; @@ -2308,8 +2308,8 @@ void XMLShapeExport::ImpExportPolygonShape( if(!xPropSet.is()) return; - bool bBezier(eShapeType == XmlShapeTypeDrawClosedBezierShape - || eShapeType == XmlShapeTypeDrawOpenBezierShape); + bool bBezier(eShapeType == XmlShapeType::DrawClosedBezierShape + || eShapeType == XmlShapeType::DrawOpenBezierShape); // get matrix ::basegfx::B2DHomMatrix aMatrix; @@ -2447,7 +2447,7 @@ void XMLShapeExport::ImpExportGraphicObjectShape( // Transformation ImpExportNewTrans(xPropSet, nFeatures, pRefPoint); - if(eShapeType == XmlShapeTypePresGraphicObjectShape) + if(eShapeType == XmlShapeType::PresGraphicObjectShape) bIsEmptyPresObj = ImpExportPresentationAttributes( xPropSet, GetXMLToken(XML_GRAPHIC) ); bool bCreateNewline( (nFeatures & XMLShapeExportFlags::NO_WS) == XMLShapeExportFlags::NONE ); // #86116#/#92210# @@ -2946,11 +2946,11 @@ void XMLShapeExport::ImpExportOLE2Shape( bool bIsEmptyPresObj = false; // presentation settings - if(eShapeType == XmlShapeTypePresOLE2Shape) + if(eShapeType == XmlShapeType::PresOLE2Shape) bIsEmptyPresObj = ImpExportPresentationAttributes( xPropSet, GetXMLToken(XML_OBJECT) ); - else if(eShapeType == XmlShapeTypePresChartShape) + else if(eShapeType == XmlShapeType::PresChartShape) bIsEmptyPresObj = ImpExportPresentationAttributes( xPropSet, GetXMLToken(XML_CHART) ); - else if(eShapeType == XmlShapeTypePresSheetShape) + else if(eShapeType == XmlShapeType::PresSheetShape) bIsEmptyPresObj = ImpExportPresentationAttributes( xPropSet, GetXMLToken(XML_TABLE) ); bool bCreateNewline( (nFeatures & XMLShapeExportFlags::NO_WS) == XMLShapeExportFlags::NONE ); // #86116#/#92210# @@ -3141,7 +3141,7 @@ void XMLShapeExport::ImpExportPageShape( // a presentation page shape, normally used on notes pages only. If // it is used not as presentation shape, it may have been created with // copy-paste exchange between draw and impress (this IS possible...) - if(eShapeType == XmlShapeTypePresPageShape) + if(eShapeType == XmlShapeType::PresPageShape) { mrExport.AddAttribute(XML_NAMESPACE_PRESENTATION, XML_CLASS, XML_PAGE); @@ -3497,7 +3497,7 @@ void XMLShapeExport::ImpExportMediaShape( // Transformation ImpExportNewTrans(xPropSet, nFeatures, pRefPoint); - if(eShapeType == XmlShapeTypePresMediaShape) + if(eShapeType == XmlShapeType::PresMediaShape) { (void)ImpExportPresentationAttributes( xPropSet, GetXMLToken(XML_OBJECT) ); } @@ -3652,7 +3652,7 @@ void XMLShapeExport::ImpExport3DShape( switch(eShapeType) { - case XmlShapeTypeDraw3DCubeObject: + case XmlShapeType::Draw3DCubeObject: { // minEdge aAny = xPropSet->getPropertyValue("D3DPosition"); @@ -3692,7 +3692,7 @@ void XMLShapeExport::ImpExport3DShape( break; } - case XmlShapeTypeDraw3DSphereObject: + case XmlShapeType::Draw3DSphereObject: { // Center aAny = xPropSet->getPropertyValue("D3DPosition"); @@ -3729,8 +3729,8 @@ void XMLShapeExport::ImpExport3DShape( break; } - case XmlShapeTypeDraw3DLatheObject: - case XmlShapeTypeDraw3DExtrudeObject: + case XmlShapeType::Draw3DLatheObject: + case XmlShapeType::Draw3DExtrudeObject: { // write special 3DLathe/3DExtrude attributes, get 3D tools::PolyPolygon as drawing::PolyPolygonShape3D aAny = xPropSet->getPropertyValue("D3DPolyPolygon3D"); @@ -3772,7 +3772,7 @@ void XMLShapeExport::ImpExport3DShape( // write point array mrExport.AddAttribute(XML_NAMESPACE_SVG, XML_D, aPolygonString); - if(eShapeType == XmlShapeTypeDraw3DLatheObject) + if(eShapeType == XmlShapeType::Draw3DLatheObject) { // write 3DLathe shape SvXMLElementExport aOBJ(mrExport, XML_NAMESPACE_DR3D, XML_ROTATE, true, true); @@ -5125,7 +5125,7 @@ void XMLShapeExport::ImpExportTableShape( const uno::Reference< drawing::XShape bool bIsEmptyPresObj = false; // presentation settings - if(eShapeType == XmlShapeTypePresTableShape) + if(eShapeType == XmlShapeType::PresTableShape) bIsEmptyPresObj = ImpExportPresentationAttributes( xPropSet, GetXMLToken(XML_TABLE) ); const bool bCreateNewline( (nFeatures & XMLShapeExportFlags::NO_WS) == XMLShapeExportFlags::NONE ); diff --git a/xmloff/source/draw/ximpshap.cxx b/xmloff/source/draw/ximpshap.cxx index 0e7ce7a1619f..5b671ab1caf4 100644 --- a/xmloff/source/draw/ximpshap.cxx +++ b/xmloff/source/draw/ximpshap.cxx @@ -1532,46 +1532,46 @@ void SdXMLTextBoxShapeContext::startFastElement (sal_Int32 nElement, { if( IsXMLToken( maPresentationClass, XML_SUBTITLE )) { - // XmlShapeTypePresSubtitleShape + // XmlShapeType::PresSubtitleShape service = "com.sun.star.presentation.SubtitleShape"; } else if( IsXMLToken( maPresentationClass, XML_PRESENTATION_OUTLINE ) ) { - // XmlShapeTypePresOutlinerShape + // XmlShapeType::PresOutlinerShape service = "com.sun.star.presentation.OutlinerShape"; } else if( IsXMLToken( maPresentationClass, XML_NOTES ) ) { - // XmlShapeTypePresNotesShape + // XmlShapeType::PresNotesShape service = "com.sun.star.presentation.NotesShape"; } else if( IsXMLToken( maPresentationClass, XML_HEADER ) ) { - // XmlShapeTypePresHeaderShape + // XmlShapeType::PresHeaderShape service = "com.sun.star.presentation.HeaderShape"; bClearText = true; } else if( IsXMLToken( maPresentationClass, XML_FOOTER ) ) { - // XmlShapeTypePresFooterShape + // XmlShapeType::PresFooterShape service = "com.sun.star.presentation.FooterShape"; bClearText = true; } else if( IsXMLToken( maPresentationClass, XML_PAGE_NUMBER ) ) { - // XmlShapeTypePresSlideNumberShape + // XmlShapeType::PresSlideNumberShape service = "com.sun.star.presentation.SlideNumberShape"; bClearText = true; } else if( IsXMLToken( maPresentationClass, XML_DATE_TIME ) ) { - // XmlShapeTypePresDateTimeShape + // XmlShapeType::PresDateTimeShape service = "com.sun.star.presentation.DateTimeShape"; bClearText = true; } else // IsXMLToken( maPresentationClass, XML_TITLE ) ) { - // XmlShapeTypePresTitleTextShape + // XmlShapeType::PresTitleTextShape service = "com.sun.star.presentation.TitleTextShape"; } bIsPresShape = true;