Inverter Config UI corrupts Inverter List on Serial Update about opendtu HOT 2 CLOSED

also this seems to mess up live data view

from opendtu.

I think there is some mixup when the inverter list gets modified in any way after initial config read. This stems from e.g. code segments like this (WebApi_inverter.cpp)

code to display the list

for (uint8_t i = 0; i < INV_MAX_COUNT; i++) { if (config.Inverter[i].Serial > 0) { JsonObject obj = data.createNestedObject(); obj[F("id")] = i; ..........

code to edit an entry
if (inv != nullptr && inv->serial() != inverter.Serial) { // Serial was changed Hoymiles.removeInverterByPos(root[F("id")].as<uint64_t>()); inv = Hoymiles.addInverter(inverter.Name, inverter.Serial); } else if (inv == nullptr) { inv = Hoymiles.addInverter(inverter.Name, inverter.Serial); }

where the position of the inverter in the Hoymiles vector and the id in the configuration start to diverge

from opendtu.