OpenRepairPlatform icon indicating copy to clipboard operation
OpenRepairPlatform copied to clipboard

Inversion Latitude Longitude

Open Skrattoune opened this issue 3 years ago • 2 comments

Selon les conventions, les geolocalisations sont listées dans l'ordre latitude, longitude or, aussi bien dans le model que dans le formulaire, ils ont été listés dans l'ordre inverse.

Du coup, quand j'ai créé mon premier lieu, j'ai cru qu'il y avait un problème car votre super système de cartes affichait ... rien.

serait-il possible d'inverser dans le model comme dans

class PlaceForm(ModelForm):
    longitude = forms.CharField(widget=forms.HiddenInput)
    latitude = forms.CharField(widget=forms.HiddenInput)

l'ordre des deux fields ?

Si vous voulez, je peux aussi le faire directement

Skrattoune avatar Jan 21 '22 16:01 Skrattoune

Il y a un changement dans le model location entre dev et dev_CH:

location = models.PointField(null=True, blank=True, geography=True)

@property
    def latitude(self):
        if self.location:
            return self.location.y
        return None

    @property
    def longitude(self):
        if self.location:
            return self.location.x
        return None

A part le coté historique interne à l'application, y a-t-il une raison de garder location = (longitude, latitude) plutôt que location = (latitude, longitude) ?

cf aussi le form: instance.location = Point(longitude, latitude)

Je pense que ça pourrait être bien de rajoutter un help_text='instance.location = Point(longitude, latitude)' dans la definition du model: location = models.PointField(null=True, blank=True, geography=True, help_text='instance.location = Point(longitude, latitude)')

Skrattoune avatar Jan 23 '22 09:01 Skrattoune

Alors, c'est normal.

On utilise GeoDjango et en effet, le Point() est défini dans cet ordre. Voir la documentation : https://docs.djangoproject.com/fr/4.0/ref/contrib/gis/tutorial/#id13

Dans tous les cas, oui il y a des différence entre notre branch dev_branch et la branche dev_fr_suisse. Nous avons développé la fonctionnalité d'entrée sur la plateforme par localisation sur dev_branch. Or, on utilise l'API du gouvernement Française et ce n'est pas utilisable chez vous dans l'état. C'est pour ça que nous avons créé cette branche commune dev_fr_suisse.

Je l'avais mentionné à Nico mais il te n'a peut être pas fait passé l'info. @LucienD

ClemsP avatar Jan 23 '22 14:01 ClemsP