[Python] [pythonisti] aggiornamento

[Manlio Perillo]

Lawrence Oluyede ha scritto:
> Ho sistemato il problema dei campi richiesti nel profilo
> 
>> L'errore è appunto quando provo ad aggiungere una GeoLocation, dalla
>> pagina di creazione del profilo:
> 
> Si si è dovuto a editable=False, l'ho tolto ma rimane un ma:
> 
>> In questo modo non posso testare se la mappa funziona.
> 
> Puoi, provandolo da shell. Bisognerebbe aggiungere il supporto a
> google.py dentro l'admin. Io sono ancora contrario a metterlo nel
> save() perché quando stavo sistemando i model mi sono accorto che
> geo_location aveva bisogno di user (per l'indirizzo) e user di
> geo_location e mi pare una dipendenza stupida.

geo_location ha bisogno di user soloperchè user_id è la sua primary key.
Comunque non mi è chiaro cosa vuoi dire.

> Inoltre a noi non serve
> memorizzare l'indirizzo dentro un campo address in GeoLocation perché
> tanto viene costruito a partire dal profilo dell'utente (per questo la
> dipendenza è controproducente).
> 

Si, questo può essere tolto, lo avevo messo per completezza.

> Io lascerei cosi per ora, più avanti integriamo google.py anche dentro 
> l'admin
>

Non mi piace molto l'idea di non essere più in grado di inizializzare 
tutti i dati via l'interfaccia di amministrazione.

Infine, nella patch, non mi convince questo:

apps/geo/views.py
===================================================================
--- apps/geo/views.py	(revision 271)
+++ apps/geo/views.py	(working copy)
@@ -10,10 +10,11 @@
      locations = models.GeoLocation.objects.all()

      content = []
-    for item in locations:
+    for location in locations:
          content.append({
-                'info': item.username.username,
-                'location': simplejson.loads(item.geolocation)
+                # XXX: to fix
+                'info': location.users.all() or "anonymous",
+                'location': simplejson.loads(location.geo_location)
          })


Innanzitutto solo gli iscritti possono creare il profilo e quindi 
inserire la locazione geografica.

Inoltre nel campo location vuoi inserire tutto il dettaglio del 
geocoder? Non bastano solo le coordinate?



Saluti  Manlio Perillo