[Libreoffice-commits] core.git: Branch 'libreoffice-6-4' - sc/source

[Julien Nabet (via logerrit)]

 sc/source/core/tool/interpr8.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit f075eaf3e6e4244cb7b92420e4286848bf22ed91
Author:     Julien Nabet <serval2412 at yahoo.fr>
AuthorDate: Thu Apr 16 12:47:46 2020 +0200
Commit:     Eike Rathke <erack at redhat.com>
CommitDate: Fri Apr 17 21:23:38 2020 +0200

    tdf#130583: fix Switch in Calc
    
    Don't set bFinished to true too soon if there are still some params to take into account
    
    Change-Id: Ie7bdd6f316c08342d379d55bf3853774dd738e7d
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92345
    Tested-by: Jenkins
    Reviewed-by: Eike Rathke <erack at redhat.com>
    (cherry picked from commit bb3ea0ae51bb943f7bb3dca80eee153f9c55c03e)
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/92432

diff --git a/sc/source/core/tool/interpr8.cxx b/sc/source/core/tool/interpr8.cxx
index 18a65e86f86c..71269e99eb24 100644
--- a/sc/source/core/tool/interpr8.cxx
+++ b/sc/source/core/tool/interpr8.cxx
@@ -1965,7 +1965,7 @@ void ScInterpreter::ScSwitch_MS()
         else
             aStr = GetString();
         nParamCount--;
-        if ( nGlobalError != FormulaError::NONE || (( isValue && rtl::math::approxEqual( fRefVal, fVal ) ) ||
+        if ( (nGlobalError != FormulaError::NONE && nParamCount < 2) || (( isValue && rtl::math::approxEqual( fRefVal, fVal ) ) ||
              ( !isValue && aRefStr.getDataIgnoreCase() == aStr.getDataIgnoreCase() )) )
         {
             // TRUE