} else if (d->strategy != OnManualSubmit) {
// historical bug: this could all be simple like OnManualSubmit, but isn't
const QModelIndex cIndex = createIndex(row, idx);
- // historical bug: comparing EditRole with DisplayRole values here
- const QVariant oldValue = data(cIndex);
- const QVariant value = record.value(i);
- // historical bug: it's a bad idea to check for change here
- // historical bug: should test oldValue.isNull() != value.isNull()
- if (oldValue.isNull() || oldValue != value) {
- mrow.setValue(idx, record.value(i));
- emit dataChanged(cIndex, cIndex);
- }
+ mrow.setValue(idx, record.value(i));
+ emit dataChanged(cIndex, cIndex);
} else {
mrow.setValue(idx, record.value(i));
}
model.submit();
else {
// dataChanged() is not emitted when submitAll() is called
- QCOMPARE(spy.count(), 2);
+ QCOMPARE(spy.count(), model.columnCount());
QCOMPARE(spy.at(0).count(), 2);
- QCOMPARE(qvariant_cast<QModelIndex>(spy.at(0).at(0)), model.index(i, 1));
- QCOMPARE(qvariant_cast<QModelIndex>(spy.at(0).at(1)), model.index(i, 1));
+ QCOMPARE(qvariant_cast<QModelIndex>(spy.at(1).at(0)), model.index(i, 1));
+ QCOMPARE(qvariant_cast<QModelIndex>(spy.at(1).at(1)), model.index(i, 1));
}
}