Undefined units in trends.htm

[laulau]

I have (undefined) as units in trends.htm, it's ok in charts.html.
Is something missing somewhere ?

CaptCMX.JPG

[steve]

The javascript reads the units from the units.json file that MX uploads. The contents of that file downloaded from your site look OK, but your web server is returning it with Content-Type application/octet-stream, unlike my web server, which returns it as application/json. Perhaps this stops the javascript being able to interpret it correctly.

[jpsc]

I see the same 3009 and 3010, Windows 7 serving to Chrome

[mcrossley]

I have a nagging memory that this is why I stuck with .txt for the ss gauges, too many web servers don't have the mime types up to date. Though using .txt also has its own problems.

[laulau]

I think I'll do a test on my main website (paying hosting with OVH.com) rather than Chez.com that is a free hosting (but with less capable services)

[laulau]

http://laurentmey.fr/test/trends.htm
(not updated for the moment)

[steve]

Currently, the code does this:

Code: Select all

$.ajax({url: "units.json", success: function (result) {
            createTemp(result.temp);

And the problem is that 'result' is not being given to the script as an object but a string.

If I change it to use JSON.parse() instead, would that get around the problem? But would it break the case where it's working already?

[mcrossley]

Steve, you could tell jQuery that it is a json query by adding the parameter

dataType: "json",

Or switch to using the jQuery $.getJSON() function rather than the $.ajax()

http://api.jquery.com/jquery.getjson/

[steve]

Thanks Mark, I'll give that a try.

[steve]

I tried adding "dataType: "json"," (using the Chrome developer tools) on Laurent's site and it works.

[mcrossley]

Good, I prefer to add the parameter rather than use $.getJSON() - which is just a wrapper to $.ajax anyway.

iirc, adding that parameter also auto-enables appending a browser cache buster string to the query?

[steve]

It looks like it might have changed the cache headers to no-cache?

[steve]

I've done this change to the cumuluscharts.js file in build 3011 - you'll need to upload it to the web site, of course.

[laulau]

Fixed with Build 3011