search

mst / whatsupcoming

2 stars 0 forks source link

Distance Calculation broken #17

Open koeulr opened 12 years ago

koeulr commented 12 years ago

Seems like endless pagination broke distance calculation. Anyway.... distance calculation should be moved from the view to Event.objects.filter e.g.

mst commented 12 years ago

I agree that we probably should use a custom query method for that. You can add an implementation of a "Manager" (or something like that) which contains custom methods that you can use like that: Event.objects.custom_query()....

However, it doesn't feel right for a query method to add an attribute to the model object.

Any ideas?

2012/5/14 koeulr reply@reply.github.com:

Seems like endless pagination broke distance calculation. Anyway.... distance calculation should be moved from the view to Event.objects.filter e.g.

Reply to this email directly or view it on GitHub: https://github.com/marcelst/whatsupcoming/issues/17