| Subject: | api for Catalyst::Helper->mk_app odd and inconsistent with docs |
| Date: | Tue, 23 Nov 2010 17:35:25 -0600 |
| To: | bugs-catalyst-devel [...] rt.cpan.org |
| From: | Caleb Cushing <xenoterracide [...] gmail.com> |
I think the API is poorly designed to use ->mk_app( $name ); this is
also undocumented. I wrote a patch that preserves $self->{name} and
allows for a cleaner API as well as one consistent with the
documentation
. it's in the branch topic/helper-preserve-name of the repo, hoping
someone will have time to review it and merge it. It supports
backwards compatibility. It should apply to the HEAD of master, if it
doesn't let me know I should be able to adapt it.
( also hoping that a new release will follow as I'd like to be able to
use my API fixups soon as I've a few already merged )
--
Caleb Cushing
http://xenoterracide.com