search

opserver / Opserver

Stack Exchange's Monitoring System
https://opserver.github.io/Opserver/
MIT License
4.51k stars 827 forks source link

Fixed a javascript error when server culture uses comma as separator #188

Closed smurariu closed 8 years ago

smurariu commented 8 years ago

Naive fix, i know but Nullable.ToString does not take any arguments hence it was not possible to send InvariantCulture without complicating the code a lot

latop2604 commented 8 years ago

You should use ?.ToString(CultureInfo.InvariantCulture) instead of .ToString().Replace(',','.')

latop2604 commented 8 years ago

Same issue in /Views/Dashboard/Node.Stats.cshtml

smurariu commented 8 years ago

thank you. updated.

NickCraver commented 8 years ago

thanks! will fix up these includes in another commit since it's in a few places now.

smurariu commented 8 years ago

glad i could help