On Fri, Dec 20, 2013 at 8:34 PM, neil.lunn <[email protected]> wrote:
> .... > My article today actually (http://www.catalystframework.org/calendar/2013/21), > even though I'm actually talking here about the above case. > Just a note on the Advent article. Thanks for writing that up. It's a well-written article and clearly explains the issue I was facing and the fix you provided to me. One thing I really like about the Advent articles is that I learn new ways to do things. For example, I wasn't aware of namespace::sweep and never thought about overriding the -X filetests. I just set the content length manually now in the Controller along with the body. I'm was also very happy to see you building this into a model at the end. I sometimes wonder if that is not stressed enough when learning Catalyst. I see a lot of code written into controllers at work that should really be models. I will pass the link to the Advent article around. In my specific situation for this problem I already had a model. The gzipped files are stored on a distributed file system and I already had a model class for fetching files. I just extended that to handle these gzipped files. But, I think your solution is a bit more elegant and, well, more correct because it can be used more widely. Thanks,
_______________________________________________ List: [email protected] Listinfo: http://lists.scsys.co.uk/cgi-bin/mailman/listinfo/catalyst Searchable archive: http://www.mail-archive.com/[email protected]/ Dev site: http://dev.catalyst.perl.org/
