On Tue, May 12, 2009 at 14:07, Tomas Doran <[email protected]> wrote:
> Roland Lammel wrote: > >> Here is the very simplistic patch, which only renames the "actions" >> attribute to "_controller_actions" in Catalyst::Controller. Test suite still >> passes with the patch and my app that originally showed that error, is now >> also working like a charm. >> > > Great stuff, change looks perfect. > > Could you also add an action named 'actions' to one of the TestApp > controllers in the test suite, so that there is a regression test and > evidence in the log/tests of _why_ this got changed? > > This should be fairly trivial, as adding just an empty method should cause > everything to explode, right? I actually did that in intial patch, that just showed the erronous behaviour, I'll add that together and repost to the list, with a note of why that got changed in the test itself. > > > TIA > t0m > > > _______________________________________________ > 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/ > -- Roland Lammel QuikIT - IT Lösungen - flexibel und schnell Web: http://www.quikit.at Email: [email protected] "Enjoy your job, make lots of money, work within the law. Choose any two."
_______________________________________________ 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/
