Refactor getColumnTypes method to improve logic for sheet availabilit…

…y and initialization.
przemek83 · Jan 2, 2025 · da5ee19 · da5ee19
1 parent 7225a43
commit da5ee19
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/ImportOds.cpp b/src/ImportOds.cpp
@@ -33,10 +33,10 @@ std::pair<bool, QVector<ColumnType>> ImportOds::getColumnTypes(
         it != columnTypes_.constEnd())
         return {true, *it};
 
-    if (!isSheetAvailable(sheetName) || !initializeColumnTypes(sheetName))
-        return {false, {}};
+    if (isSheetAvailable(sheetName) && initializeColumnTypes(sheetName))
+        return {true, columnTypes_.value(sheetName)};
 
-    return {true, columnTypes_.value(sheetName)};
+    return {false, {}};
 }
 
 std::pair<bool, QStringList> ImportOds::getColumnNames(const QString& sheetName)