Implementazione ActionListViews

[joke2k]

dentro openaction/action/views.py ho creato 3 semplici view per proseguire lo sviluppo dei template,

ActionListView: generica, riutilizzabile dalle altre

ActionByCategoryListView: actions per una categoria selezionata ActionByGeonameListView: actions per un territorio selezionato

questi sono due url che ho aggiunto:

#categories navigation
url(r'^argument/(?P<pk>\d+)/$', action_views.ActionByCategoryListView.as_view(),
    name='category-action-list'
),
#geonames navigation
url(r'^location/(?P<pk>\d+)/$', action_views.ActionByGeonameListView.as_view(),
   name='geoname-action-list'
),

successivamente sarà necessaria anche una ActionSearchView(ActionListView) che avrà dei filtri ancora più avanzati.

[feroda]

Siamo sul pezzo.

In mattinata pusheremo il codice.

Sono un po' perplesso dal tuo tipo di uso del "get_absolute_url". Usato in questo modo è solo contestualizzato nell'oggetto Action, non esiste "a prescindere". Invece i modelli ActionCategory e Geonames sono indipendenti e quindi la pagina di riferimento in teoria sarebbe quella della descrizione della categoria, o del territorio (ci sta bene una mappa).

Secondo me non vale la pena sacrificare il "get_absolute_url" per questa semplificazione, che ne pensi?

Riprendiamo il merge

[feroda]

Ho appena pushato sul ramo befair-dev: https://github.com/openpolis/open-action/commit/b1e5d8cfe2750c4189af4852b182c34e2f3772b7 https://github.com/openpolis/open-action/commit/003ffedd3fba9336ce9e8fb10876be080d31aff3

• c'è una vista generica FilteredActionListView che è da intendersi come API, si basa sul passaggio di parametri via GET HTTP e consente di fare filtro per categorie, territori e politici e consente anche l'ordinamento per territori e politici. Come da implementazione tue viste vengono ritornati anche gli oggetti su cui è stato richiesto il filtraggio ('filter_geonames', 'filter_categories', ...)
• ci sono le tue viste leggermente aggiustate.
• il nucleo dei filtraggi è implementato nell'ActionQueryset (metodi chiamati anche da ActionManager)

Ora puoi scegliere a quale livello di dettaglio usare le viste:

• o usi la vista generica e puoi fare "di tutto"
• o usi le viste specifiche e hai più comodi gli url. Facci sapere le viste esatte che ti servono che Matteo evolve le tue viste oppure ne implementa altre su misura per facilitarti al massimo la scrittura dei template.

Spero sia stato chiaro, vedi anche commento qui sopra sul "get_absolute_url" che io non utilizzerei in quel modo.

Passatevi i numeri di telefono che può far sempre comodo (anche a me che se sono sul pezzo e ho un dubbio ti chiamo)

Grazie, a presto Luca

[joke2k]

Non ho capito a quale uso di get_absolute_url ti riferisci ;)

sto facendo il merge con le modifiche che avete messo sul befair-dev, e mi da alcuni problemi:

1. riga 71:

url(r'^filter/$', action_views.ActionListView.as_view(),
        name='actions-filter'
    ),

non trova la ActionListView, credo sia un refuso, ma lo riporto per ogni evenienza.

1. Se provo a vedere una pagina di un argomento (http://127.0.0.1:8000/action/argument/1/ con argomento 1 inserito dal backend) mi da:

TypeError at /action/argument/1/
int() argument must be a string or a number, not 'tuple'
...
/dev/open_action_web/action/query.py in by_categories
        return self.filter(category_set=categories) 

il problema si ripercuote anche sulla ActionByGeonameListView.

[matteo88]

Il primo problema scaturisce dal fatto che nell'ultima push mancano alcune piccole modifiche (tra cui il refactoring dell'url che ti da problemi), che andavano riviste prima di essere integrate.

Il type error, invece, dipende da un bug contenuto nel manager del modello Action, e che ora è stato fixato nell'ultimo commit.

Integreremo queste modifiche il prima possibile.

[joke2k]

Sono un po' perplesso dal tuo tipo di uso del "get_absolute_url". Usato in questo modo è solo contestualizzato nell'oggetto Action, non esiste "a prescindere". Invece i modelli ActionCategory e Geonames sono indipendenti e quindi la pagina di riferimento in teoria sarebbe quella della descrizione della categoria, o del territorio (ci sta bene una mappa).

Secondo me non vale la pena sacrificare il "get_absolute_url" per questa semplificazione, che ne pensi?

sono d'accordo sui modelli ActionCategory e Geonames, cioè che la pagina di riferimento debba essere quella del Dettaglio dei singoli (il mio approccio era volontariamente temporaneo in quanto volevo lasciarvi libertà di implementare la cosa). ma ancora non ho capito quale uso di get_absolute_url non ti convince. ^_^