Re: Re: [PEPr] +1 for Web Services::Services_GeoNames

Christian Weiske a écrit :

Hello David,

Nicely small and clean code. I'd like to see @method tags in the class docblock since you use __call.

I thought about adding the @method tags but imho it would be just duplicating the geonames documentation, and it would be harder to maintain/keep sync with their API, that's why I just put an @link tag pointing to the official documentation.

But IDEs with code completion could at least list the available methods .)

hmm... not sure <insert_your_fav_ide_here> completion take care of phpdoc @method :) Fair enough tho, I'll add the @method tags. -- David