search

theorchard / monolog-cascade

Configure multiple loggers and handlers in the blink of an eye
MIT License
145 stars 62 forks source link

When an optional parameter has no devault value available, set it required #40

Closed kafene closed 8 years ago

kafene commented 8 years ago

resolves #37

kafene commented 8 years ago

replacement for #38 - check that a default value is available when setting the default value, otherwise set the param required.

@rantonmattei - I think you're right, this is the simplest and cleanest way to do it. If the user doesn't fill in all the defaults for an internal class/method, it will fail in a consistent way that makes sense (missing params).

rantonmattei commented 8 years ago

That's great. Thanks!