[Dancer-users] Invalid value for log config

[Naveed Massjouni]

On Tue, Dec 14, 2010 at 8:39 PM, Naveed Massjouni <naveedm9 at gmail.com> wrote:
> On Tue, Dec 14, 2010 at 6:03 PM, Gabor Szabo <szabgab at 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 at perldancer.org
>> http://www.backup-manager.org/cgi-bin/listinfo/dancer-users
>>
>