Produits Entr'ouvert » w.c.s.

pub.site_options.set('options', 'nominatim_search_url', 'http://search.openstreetmap.org')

Dans dix minutes tout le monde aura oublié pourquoi deux paramètres et quelqu'un voudra aller à cette adresse en se disant que peut-être l'API y est différente et le nom de domaine n'existera pas et la perplexité grandira encore.

Ce qu'il faut plutôt c'est avoir du côté du publisher get_geocoding_service_url() et get_reverse_geocoding_service_url() et chacune regarderait d'abord si (reverse_)geocoding_service_url existe dans le site-options.cfg et sinon construirait l'URL à partir de nominatim_url, dans ce genre :

def get_reverse_geocoding_service_url(self):
    url = self.get_site_option('reverse_geocoding_service_url')
    if url:
        return url
    nominatim_url = self.get_site_option('nominatim_url') or 'http://nominatim.openstreetmap.org'
    return nominatim_url + 'reverse'

Le même principe doit être suivi aussi pour les paramètres liés à nominatim, comme nominatim_key, nominatim_reverse_zoom_level, imo.

def get_geocoding_key(self):

Si ça doit arriver ça doit être en permettant aux URL de géocodage (inversé) de contenir une query string arbitraire.

def get_geocoding_reverse_zoom_level(self):

Si ça devait arriver ça bénéficierait aussi du point précédent. Ne pas ajouter cette méthode, continuer avec l'ancienne option uniquement en attendant.

soudainement deux lignes blanches entre deux méthodes

Non.

pub.site_options.set('options', 'custom_geocode_key', 'geocoding_secret')

Nope je me suis mal fait comprendre, je ne veux pas de nouvelle option sur le sujet; je veux que geocoding_service_url puisse être https://whatever/search?key=toto&foo=bar qu'il n'y ait pas besoin d'un site_options par paramètre.

Sur ce passage :

  url = url % (urllib2.quote(address))

pour la beauté de la chose, éviter de faire le % alors qu'on ne sait pas vraiment ce qu'il y a dans url (s'il y a un autre %truc, ça va crasher salement). Plutôt :

        if '?' in url:
            url += '&'
        else:
            url += '?'
        url += 'q=%s' % urllib2.quote(address)

Je pourrais faire mon pénible relou en disant que tu pouvais laisser la ligne

url += '&accept-language=%s' % (get_publisher().get_site_language() or 'en')

Si tu peux/veux attendre Frédéric qui est en début de relecture, c'est bien, sinon, tu pousses.

Pour moi les options nominatim_* devraient servir, vu leur nom, uniquement pour nominatim, à intégrer en bout de get_reverse_geocoding_service_url et get_geocoding_service_url.