On Tue, Dec 14, 2010 at 8:39 PM, Naveed Massjouni <naveedm9@gmail.com> wrote:
On Tue, Dec 14, 2010 at 6:03 PM, Gabor Szabo <szabgab@gmail.com> wrote:
After a misunderstanding in the environments/development.yml I changed the value of "log" to
log: "console"
which is incorrect (as this value belongs to logger) but I have not seen any complaints about it.
Shouldn't DAncer check if the config params are valid?
Yes, this makes sense. For example, maybe there should be a warning if not grep config->{log} eq $_, qw(core, debug, warning, error).
Ooops! I wrote bad syntax. Should be: if not grep config->{log} eq $_, qw(core debug warning error) You get the idea :) -Naveed
BTW I also tried to add a configuration parameter that does not exists and it was silently disregarded. As it could be the result of a typo or a misunderstanding I think that should be reported as well.
I don't agree with this. I put my own app specific params in the config. Like max_widgets: 100
-Naveed
Gabor
-- Gabor Szabo http://szabgab.com/ _______________________________________________ Dancer-users mailing list Dancer-users@perldancer.org http://www.backup-manager.org/cgi-bin/listinfo/dancer-users