Hi

 

I’ve tried your patch, but I got three problems with it

·         The link is not placed nicely on the page. Sort Alpha is on the 
right, Sort by number of task on the left below the title of the group. Perhaps 
you can integrate both links, i.e. put “Sort Alphabetically | By number of 
tasks” on the right side?

·         For me the sorting does not work. I get a random sorted list of 
projects (or so it seems…). I’m not sure where things go wrong though.

·         The is a lot of SQL going on to find the right order, see your log. 
Every x.todos.count results in a separate database query. 
Perhaps you can let the database do the sorting by a specific SQL? Something 
like SELECT id, count(*) as p_count FROM projects WHERE user_id = … AND 
state=’active’ ORDER BY p_count and then map the sorted id?

 

I’ll apply your patch if you can fix these issues…

 

Reinier

 

Van: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED] Namens Walter Cruz
Verzonden: vrijdag 24 oktober 2008 14:35
Aan: [email protected]
Onderwerp: [Tracks-discuss] sort projects by the number of todos

 

I've added this to my github branch.

http://github.com/waltercruz/tracks/tree/sort_projects_by_todos

Can someone test, and, if possible, apply to the master, or guide me to 
strenght this code?

-- 
[]'
- Walter
waltercruz.com

_______________________________________________
Tracks-discuss mailing list
[email protected]
http://lists.rousette.org.uk/mailman/listinfo/tracks-discuss

Reply via email to