Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Inversion Latitude Longitude #144

Open
Skrattoune opened this issue Jan 21, 2022 · 2 comments
Open

Inversion Latitude Longitude #144

Skrattoune opened this issue Jan 21, 2022 · 2 comments

Comments

@Skrattoune
Copy link

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 added a commit to Skrattoune/OpenRepairPlatform that referenced this issue Jan 23, 2022
Skrattoune added a commit to Skrattoune/OpenRepairPlatform that referenced this issue Jan 23, 2022
@Skrattoune
Copy link
Author

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)')

@ClemsP
Copy link
Collaborator

ClemsP commented Jan 23, 2022

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants