search

disqus / disqus-php

Disqus API bindings for PHP
http://disqus.com/api/
Apache License 2.0
149 stars 73 forks source link

Minimum required PHP version #14

Open borfast opened 12 years ago

borfast commented 12 years ago

README.rst mentions PHP 5.3 as the minimum required version but from a quick code analysis with this neat little tool, it's actually 5.2 and just for json_decode().

For all the rest you only need PHP 5.1. Could you please confirm this and, if correct, change the docs accordingly?

Thanks! :)

dcramer commented 12 years ago

5.3 might have actually been a guess. Will try to review and confirm.

Neat tool :)

borfast commented 12 years ago

Thanks for looking into it :)

I forgot to mention this but since you provide an alternative json implementation, the minimum required version is effectively 5.1.

I also went through the code manually to confirm the tool's result and I think it's right, I don't see anything that requires PHP over 5.1 (or 5.2 for json_decode()).