b4039 Moisture recent charts-a-select without unit

[Dador]

Hi,

after updating to the latest version, I set up the soil moisture data from Davis and Ecowitt on one graph. The Recent chart displays the Davis (cb) and Ecowitt (%) units fine, but the Recent select-a-chart returns the unit as undefined.

I have the same problem even at the Ecowitt station, where I only use the Ecowitt moisture sensor - WH51.

[The Dalek Hunter]

IMG_0234.jpeg

Same for me on my Ecowitt Station

[mcrossley]

I've double checked and it is working on my test install use Ecowitt.

Can you confirm that line 1425 of your /interface/js/chartscompare.js is like this...

Code: Select all

                    valueSuffix: ' ' + config.soilmoisture.units[unitIdx]

If so, have you tried clearing the browser cache - or Ctrl-F5 the page?

[The Dalek Hunter]

Yes cache cleared.

Tried it on different browsers and it says undefined on them all


And yes that line is present exactly as you posted

[Dador]

I have the same line of code. Memory cleared. Unit missing on all browsers.

This, of course, applies to Recent select-a-chart and Select-a-period charts.

[mcrossley]

Ok. I'll take another look later ...

[mcrossley]

Ah, wait, is this in Neils AI2 rather than the standard dashboard?

[The Dalek Hunter]

No - this is on the Web Page
https://www.longsutton-weather.co.uk/selectachart.htm

[mcrossley]

Ok that helps, thanks. I'll make a fix available later today...

[mcrossley]

OK, please try replacing the js scripts on your web site with these...

[deleted]

[Dador]

I noticed this change in the code selectachart.js, .

Code: Select all

max: config.soilmoisture.units.includes('cb') ? 200 : 100, // Davis 0-200 cb, Ecowitt 0-100%

but it didn't help. It's still as it was.

[mcrossley]

OK, I debugged your web site, and I see what the problem is. It is more complex than I first thought. I'll have a solution soon...

[mcrossley]

Try this version...

[deleted]

[Dador]

Nope..

When using a new file, it does not load soil moisture charts at all. The rest works fine.
Browser memory cleared on an ongoing basis.

[mcrossley]

Sorry, the names are inconsistent in the JSON (bad me!)

I corrected the script to account for that....