[Python] [pythonisti] aggiornamento

[Manlio Perillo]

Lawrence Oluyede ha scritto:
>> > 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.
> 
> Io voglio semplicemente evitare la dipendenza circolare.
> Un utente si registra con i suoi dati, noi computiamo la geolocazione,
> la creiamo, la assegnamo al suo profilo e poi memorizziamo nel db il
> tutto.
> 

Va anche bene così.
Io lo avevo pensato in modo da essere un componente riutilizzabile.

> 
>> > 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.
> 
> C'è anceh da dire che la geolocazione ha uno strano formato
> a-la-google quindi non ti metteresti mai a mano a inserire tutto quel
> malloppone di informazioni (né ad inserire coordinate a caso)
> 

Infatti come è adesso, devi solo inserire un indirizzo, tutto il resto 
viene fatto da save.

Purtroppo qui è anche colpa di Django che accoppia la struttura del 
database con il layer successivo.

Tanto per fare un esempio, io con SQLAlchemy ho:
1° livello: definizione struttura database
2° livello: definizione mapper
3° livello: definizione model (che si occupa di gestire i dati inseriti
             dall'utente)

>> Infine, nella patch, non mi convince questo:
> 
> Si quelle view son tutte da riscrivere, era una monkey patch al volo
> per far funzionare il sito.
> 
>> Inoltre nel campo location vuoi inserire tutto il dettaglio del
>> geocoder? Non bastano solo le coordinate?
> 
> Appunto, bisogna riscrivere :-)
> 

Se ce la fai, prova a creare una patch alla volta, la prima per 
sistemare geolocation e l'altra per il profilo.

Altrimenti fai il commit di quello che c'è, lo correggiamo in seguito.
La mia obiezione era che il codice attuale c'è e funziona.




Saluti  Manlio Perillo