Tools

Samuel said

Samuel  

Geo bug: commas in coordinates

12 comments

Samuel posted to #Qaiku-api somewhere 07.12.2009 (en)

12 comments

Bottom

Samuel  

APi returns for example:

"geo":{"type":"Point","coordinates":[25,022301,60,225899]}

Should be [25.022301,60.225899], right? (Dots instead of commas!)

Samuel commented on posted to #Qaiku-api somewhere 07.12.2009 (en)

Samuel  

At first I actually thought that an array of four members is some funky way of returning coordinates in JSON, but 022301 is not a valid numeric value in JSON, is it. (Invalid due to leading zero.)

Samuel commented on posted to #Qaiku-api somewhere 07.12.2009 (en)

bergie  

Ah, yes. This is a PHP json_encode bug, but we must work around it somehow

bergie commented on posted to #Qaiku-api Museokatu 35, Helsinki 08.12.2009 (en)

Samuel  

@bergie Are you sure you're feeding the coordinates to json_encode in the correct format (with dots, not commas)?

At least the following works with me:
<?php
$test = array(25.022301,60.225899);
echo json_encode($test);
?>

returns: [25.022301,60.225899]

Samuel commented on posted to #Qaiku-api somewhere 08.12.2009 (en)

bergie  

It is a normal PHP float type. But I think this probably has to do with locales, so have to add some setlocale magic there. Midgard ticket 1519, sent patch to @tepheikk

bergie commented on posted to #Qaiku-api Museokatu 35, Helsinki 08.12.2009 (en)

indeyets  

@bergie this bug? fixed in php-5.2.5
2 years ago… good reason for upgrade

indeyets commented on posted to #Qaiku-api Saint-Petersburg, Russia 08.12.2009 (en)

bergie  

@indeyets thanks, I'll mention that in the ticket. Qaiku is anyway using whatever PHP comes in Ubuntu LTS (now 5.2.4)

bergie commented on posted to #Qaiku-api Museokatu 35, Helsinki 08.12.2009 (en)

indeyets  

@bergie yup… I guess that's just another argument about problems of version-freeze policies in linux distros.
point-version updates in upstream are there for a reason and the reason is "bug fixing". so, LTS distro should include all these updates (after proper testing)
otherwise, it just doesn't make sense to use it. you are left with old and buggy software

indeyets commented on posted to #Qaiku-api Saint-Petersburg, Russia 08.12.2009 (en) Posted with qaikubot

tepheikk  

@indeyets I would be handy if you could get these packages and their proper working extensions as packages if you choose to. Self compiling PHP is pure madness. After each update you basically need to recompile all extensions you use.

tepheikk commented on posted to #Qaiku-api 08.12.2009 (en)

indeyets  

@tepheikk I am not saying that binary distributions are evil. I just say, that not updating to new versions is evil.

In my opinion, LTS-distributions need to include all upstream point-updates (no major updates, of course). So, ubuntu-8.04 should have 5.2.11 by now. the difference between 5.2.4 and 5.2.11 is 2 years of bug-fixes

indeyets commented on posted to #Qaiku-api Saint-Petersburg, Russia 08.12.2009 (en)

tepheikk  

@indeyets Don't they even patch part of the code? Really the problem here is that if there would be some "official" and reliable source that could provide those PHP-versions and extensions as packages. It would make maintaining a server farm easier.

tepheikk commented on posted to #Qaiku-api 08.12.2009 (en)

indeyets  

@tepheikk they only apply "security" patches as far as I understand. I doubt that approach works, though. Each of php-releases has a lot of security fixes and I doubt LTS distributions backport all of those.

Actually, I think, paid customers can "ask" to patch specific issues using support-channels… but I am not sure even about that

indeyets commented on posted to #Qaiku-api Saint-Petersburg, Russia 08.12.2009 (en)

Login or register to leave a comment

Publicity
These messages are public and can be seen by anyone.