steve71 / RasPiBrew

Raspberry Pi Temperature Controller for homebrewing and sous vide cooking
http://steve71.github.io/RasPiBrew/
MIT License
188 stars 75 forks source link

Invalid left hand assignment raspibrew_bootstrap.js:409 #8

Closed archonic closed 10 years ago

archonic commented 10 years ago

Hi Steve! Running 35.0.1916.153 m of Chrome (up to date as of writing this), a JS error stops any of the temperatures or graphs from initializing on the web interface. Firefox works fine.

archonic commented 10 years ago

Looks like FF is the only one that supports that syntax on line 409. Replacing it with this:

var LS = findLS(selected_start, selected_end, tempDataArray[0]);
pointArray = LS[0]; m = LS[1]; b = LS[2];

And getting rid of the last }) on line 566 gets chrome working.

steve71 commented 10 years ago

Thanks for the fix for Chrome!. Tested and works.  Committed and pushed to raspibrew github.

On Saturday, June 28, 2014 8:57 PM, Josh notifications@github.com wrote:

Looks like FF is the only one that supports that syntax on line 409. Replacing it with this: var LS = findLS(selected_start, selected_end, tempDataArray[0]); pointArray = LS[0]; m = LS[1]; b = LS[2]; And getting rid of the last }) on line 566 gets chrome working. — Reply to this email directly or view it on GitHub.

steve71 commented 10 years ago

Also works for Internet Explorer 11

On Friday, July 4, 2014 9:54 PM, Steve steve028@yahoo.com wrote:

Thanks for the fix for Chrome!. Tested and works.  Committed and pushed to raspibrew github.

On Saturday, June 28, 2014 8:57 PM, Josh notifications@github.com wrote:

Looks like FF is the only one that supports that syntax on line 409. Replacing it with this: var LS = findLS(selected_start, selected_end, tempDataArray[0]); pointArray = LS[0]; m = LS[1]; b = LS[2]; And getting rid of the last }) on line 566 gets chrome working. — Reply to this email directly or view it on GitHub.

archonic commented 10 years ago

Wicked. Closing!