I have allowed non-locale-compliant numeric parsing code to creep back into the code base, specifically in:
However, I can see why I missed this. It's a different scenario this time. In the past we systematically fixed all UI code to handle locale specific numeric input. The common code used by APASS/BSM above does this correctly. However, what it fails to do correctly (locale-wise) is to construct a URL in the correct locale! The problem in this case is that unlike for UI code, we want to ignore the default locale (the one used by the rest of VStar) and generate the one required by the PHP APASS/BSM server.
In getUrls(), we have:
String params = String.format("radeg=%f&decdeg=%f&raddeg=%f",
raDegs, decDegs, radiusDegs);
The locale-specific version of this method needs to be used instead to avoid generating numbers with commas, for example. The plugins will need to be rebuilt with this and re-released.
A new VStar release will be required because the bug is in common code not plugin code.
Log in to post a comment.