Merge PR #1024: Fix new charts for older Qt versions

This commit is contained in:
jenkins 2023-05-10 23:19:50 +02:00
commit 1c6c1f7fc5
4 changed files with 27 additions and 22 deletions

View File

@ -13,6 +13,10 @@ Q_DECLARE_LOGGING_CATEGORY(dcLogEngine)
NewLogsModel::NewLogsModel(QObject *parent) NewLogsModel::NewLogsModel(QObject *parent)
: QAbstractListModel{parent} : QAbstractListModel{parent}
{ {
// Workaround for older Qt versions (5.12 and older) which can't deal with the QList<EnergyLogEntry*> argument
connect(this, &NewLogsModel::entriesAdded, this, [this](int index, const QList<NewLogEntry*> &entries){
emit entriesAddedIdx(index, entries.count());
});
} }

View File

@ -119,6 +119,7 @@ signals:
void fetchBlockSizeChanged(); void fetchBlockSizeChanged();
void entriesAdded(int index, const QList<NewLogEntry*> &entries); void entriesAdded(int index, const QList<NewLogEntry*> &entries);
void entriesAddedIdx(int index, int count);
void entriesRemoved(int index, int count); void entriesRemoved(int index, int count);
private slots: private slots:

View File

@ -69,10 +69,10 @@ Item {
property double minValue property double minValue
property double maxValue property double maxValue
onEntriesAdded: { onEntriesAddedIdx: {
print("**** entries added", index, entries.length, "entries in series:", valueSeries.count, "in model", logsModel.count) print("**** entries added", index, count, "entries in series:", valueSeries.count, "in model", logsModel.count)
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
// print("entry", entry.timestamp, entry.source, JSON.stringify(entry.values)) // print("entry", entry.timestamp, entry.source, JSON.stringify(entry.values))
zeroSeries.ensureValue(entry.timestamp) zeroSeries.ensureValue(entry.timestamp)

View File

@ -323,9 +323,9 @@ Page {
} }
} }
onEntriesAdded: { onEntriesAddedIdx: {
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
var value = entry.values["temperature"] var value = entry.values["temperature"]
if (value == null) { if (value == null) {
value = 0; value = 0;
@ -380,9 +380,9 @@ Page {
} }
} }
onEntriesAdded: { onEntriesAddedIdx: {
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
var value = entry.values["temperature"] var value = entry.values["temperature"]
if (value == null) { if (value == null) {
value = 0; value = 0;
@ -438,9 +438,9 @@ Page {
} }
} }
onEntriesAdded: { onEntriesAddedIdx: {
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
var value = entry.values["humidity"] var value = entry.values["humidity"]
if (value == null) { if (value == null) {
value = 0; value = 0;
@ -491,9 +491,9 @@ Page {
chartView.busyCounter-- chartView.busyCounter--
} }
} }
onEntriesAdded: { onEntriesAddedIdx: {
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
var value = entry.values["voc"] var value = entry.values["voc"]
if (value == null) { if (value == null) {
value = 0; value = 0;
@ -580,9 +580,9 @@ Page {
chartView.busyCounter-- chartView.busyCounter--
} }
} }
onEntriesAdded: { onEntriesAddedIdx: {
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
var value = entry.values["closed"] var value = entry.values["closed"]
if (value == null) { if (value == null) {
value = false; value = false;
@ -690,9 +690,9 @@ Page {
chartView.busyCounter-- chartView.busyCounter--
} }
} }
onEntriesAdded: { onEntriesAddedIdx: {
for (var i = 0; i < entries.length; i++) { for (var i = 0; i < count; i++) {
var entry = entries[i] var entry = logsModel.get(i)
var value = entry.values["heatingOn"] var value = entry.values["heatingOn"]
if (value == null) { if (value == null) {
value = false; value = false;