Major bug: cell differences ignored fixed

This commit is contained in:
2026-02-21 17:08:57 +01:00
parent 16ef02b533
commit c12444c0d6
3 changed files with 13 additions and 13 deletions

View File

@@ -272,11 +272,15 @@ public class FileActions extends JFrame {
WorkbookComparisonResult compareResult = excel1.compareTo(excel2); WorkbookComparisonResult compareResult = excel1.compareTo(excel2);
if (compareResult.tablesAreEquals()) {
System.out.printf("RESULT: Tables %s and %s are equal.\n", resultFilePath, pClientFilePath);
} else {
ExcelWorkbook integratedTable = (ExcelWorkbook) compareResult.integrateDifferencesIntoMaster(); ExcelWorkbook integratedTable = (ExcelWorkbook) compareResult.integrateDifferencesIntoMaster();
integratedTable.save(); integratedTable.save();
Desktop.getDesktop().open(resultFilePath.toFile()); Desktop.getDesktop().open(resultFilePath.toFile());
}
} catch (IOException ex) { } catch (IOException ex) {
JOptionPane.showMessageDialog(this, "Cannot open comparison result file.", "Error", JOptionPane.showMessageDialog(this, "Cannot open comparison result file.", "Error",
JOptionPane.ERROR_MESSAGE); JOptionPane.ERROR_MESSAGE);

View File

@@ -1180,10 +1180,6 @@ public abstract class AbstractTable implements Table {
} else if (cmpValue instanceof String && !(thisValue instanceof String)) { } else if (cmpValue instanceof String && !(thisValue instanceof String)) {
thisValue = thisValue.toString(); thisValue = thisValue.toString();
} }
} else {
result.addCellDifference(thisKey, n, colName, cmpRowNo, colName);
rowsAreEqual = false;
continue;
} }
if (!Objects.deepEquals(thisValue, cmpValue)) { if (!Objects.deepEquals(thisValue, cmpValue)) {

View File

@@ -85,7 +85,7 @@ public class WorkbookComparisonResult {
return result; return result;
} }
boolean tablesAreEquals() { public boolean tablesAreEquals() {
boolean result = masterIncludesClientTables() && tablesToDeleteFromMaster.isEmpty(); boolean result = masterIncludesClientTables() && tablesToDeleteFromMaster.isEmpty();
return result; return result;