core.git: chart2/source

[Bogdan Buzea (via logerrit)]

 chart2/source/controller/main/ShapeController.cxx |    9 ++-------
 1 file changed, 2 insertions(+), 7 deletions(-)

New commits:
commit 61717e306b1c63dc4944efe21bc2ba863b8990aa
Author:     Bogdan Buzea <buzea.bog...@libreoffice.org>
AuthorDate: Fri Oct 18 06:55:56 2024 +0200
Commit:     Noel Grandin <noel.gran...@collabora.co.uk>
CommitDate: Fri Nov 15 14:56:17 2024 +0100

    tdf#163486: PVS: Identical branches
    
    Change-Id: I71c6cd9b5dbcb3126da1919108be009f6512b0d0
    V1037: Two or more case-branches perform the same actions. Check lines: 
100, 122
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/175054
    Reviewed-by: Noel Grandin <noel.gran...@collabora.co.uk>
    Tested-by: Jenkins

diff --git a/chart2/source/controller/main/ShapeController.cxx 
b/chart2/source/controller/main/ShapeController.cxx
index 7e3e7c15db6b..f9d262c6e3f3 100644
--- a/chart2/source/controller/main/ShapeController.cxx
+++ b/chart2/source/controller/main/ShapeController.cxx
@@ -96,6 +96,8 @@ FeatureState ShapeController::getState( const OUString& 
rCommand )
             case ChartCommandID::ShapeTransformDialog:
             case ChartCommandID::ShapeObjectTitleDescription:
             case ChartCommandID::ShapeRenameObject:
+            case ChartCommandID::ShapeFontDialog:
+            case ChartCommandID::ShapeParagraphDialog:
                 {
                     aReturn.bEnabled = bWritable;
                     aReturn.aState <<= false;
@@ -116,13 +118,6 @@ FeatureState ShapeController::getState( const OUString& 
rCommand )
                     aReturn.aState <<= false;
                 }
                 break;
-            case ChartCommandID::ShapeFontDialog:
-            case ChartCommandID::ShapeParagraphDialog:
-                {
-                    aReturn.bEnabled = bWritable;
-                    aReturn.aState <<= false;
-                }
-                break;
             default:
                 {
                     aReturn.bEnabled = false;