CPAN-API/metacpan-web — pull request #1123 by oalders (2014-03-13 23:23:06+00:00)
Title: oalders/all author releases

Before this can be merged, I wanted to raise a couple of points.

1) The way I'm handling the routes in Catalyst is terrible. If someone could show me the "right" way to do this, I'd be grateful.

2) I've decided to handle pagination in the Controller rather than the View. I was trying to troubleshoot the pagination we're currently using and having all that in the template directly was driving me bananas. I can see why it might be done that way, but it's a huge pain for debugging. I'd really like to see that code generally live in the Controller. That seems to be the least surprising way of handling pagination and the one which offers less friction when it comes to dumping objects etc. I know you can do this in the template too, but this just seems *easier*. I'm happy to convert the existing pagination to this method if nobody has any objections. For now, there are 2 pagination approaches in root/inc/pager.html

3) I haven't added any tests for the model. We're pretty short on model tests in this repo, but we should think about how we best want to handle this.