James Cammarata wrote:
I was looking through the code last night and noticed you'd
commented out
the modify_list function. This is the function that handles all the paging
functions like # of items per page, current page, etc. I've noticed
previously that it was functionally very similar to the modify_filter
function, so I went ahead and merged the filter stuff into the
modify_list() function, and modified the urls and template to use the new
method.
modify_filter() is still in the code, but is no longer being used, but I
left it for now. I added a header comment to the function explaining what
it does, and also added a few comments in the code to clarify things.
Pushed to my github:
commit 3903476bf2955f901487dce0d1916ad93de58e5f
Author: James Cammarata <jimi(a)sngx.net>
Date: Wed May 20 06:33:36 2009 -0500
Fixing modify_list function and merging it with the modify_filter
function
Merged, thanks.
Everyone else --you're not forgotten, I'll merge the rest of the queue a
bit later as we get further along on the web app. This is taking most of
my attention ATM.
--Michael