fix(Import): increase error again, when there was a gap in column headers

Signed-off-by: Arthur Schiwon <[email protected]>

Author: blizzz

lib/Service/ImportService.php

@@ -457,6 +457,8 @@ private function getColumns(Row $firstRow, Row $secondRow): void {
 		$index = 0;
 		$countMatchingColumnsFromConfig = 0;
 		$countCreatedColumnsFromConfig = 0;
+		$lastCellWasEmpty = false;
+		$hasGapInTitles = false;
 		foreach ($cellIterator as $cell) {
 			if ($cell && $cell->getValue() !== null && $cell->getValue() !== '') {
 				$title = $cell->getValue();
@@ -480,12 +482,16 @@ private function getColumns(Row $firstRow, Row $secondRow): void {
 
 				// Convert data type to our data type
 				$dataTypes[] = $this->parseColumnDataType($secondRowCellIterator->current());
+				if ($lastCellWasEmpty) {
+					$hasGapInTitles = true;
+				}
+				$lastCellWasEmpty = false;
 			} else {
 				$this->logger->debug('No cell given or cellValue is empty while loading columns for importing');
 				if ($cell->getDataType() === 'null') {
 					// LibreOffice generated XLSX doc may have more empty columns in the first row.
-					// Continue without increasing error count.
-					// Question: What about tables where a column does not have a heading?
+					// Continue without increasing error count, but leave a marker to detect gaps in titles.
+					$lastCellWasEmpty = true;
 					continue;
 				}
 				$this->countErrors++;
@@ -494,6 +500,11 @@ private function getColumns(Row $firstRow, Row $secondRow): void {
 			$index++;
 		}
 
+		if ($hasGapInTitles) {
+			$this->logger->info('Imported table is having a gap in column titles');
+			$this->countErrors++;
+		}
+
 		$this->rawColumnTitles = $titles;
 		$this->rawColumnDataTypes = $dataTypes;