Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add set_min_level() method #28

Open
autarch opened this issue Nov 14, 2016 · 8 comments
Open

Add set_min_level() method #28

autarch opened this issue Nov 14, 2016 · 8 comments
Labels

Comments

@autarch
Copy link
Member

autarch commented Nov 14, 2016

Migrated from rt.cpan.org #22901 (status was 'open')

Requestors:

Attachments:

From [email protected] on 2006-11-08 15:21:09:

( I may submit a patch for this later as tuits permit ).

In a large system it can be useful to have "debug" turned on for one
part, but not the whole thing.

Therefore, it's useful to be able set a global "min_level" through the
constructor, but override that later, at least temporarily.

A simple method like set_min_level($level) could do this.

It would be neat if there was an option for this declaration to go out
of scope and automatically revert to the previous value, but I don't see
that feature as required.

Mark

@autarch
Copy link
Member Author

autarch commented Nov 14, 2016

From [email protected] on 2006-11-14 22:48:42:

On Wed Nov 08 10:21:09 2006, MARKSTOS wrote:

( I may submit a patch for this later as tuits permit ).

In a large system it can be useful to have "debug" turned on for one
part, but not the whole thing.

Therefore, it's useful to be able set a global "min_level" through the
constructor, but override that later, at least temporarily.

A simple method like set_min_level($level) could do this.

A patch is attached for this, which includes code, test and doc updates.

Mark

@autarch
Copy link
Member Author

autarch commented Nov 14, 2016

From [email protected] on 2006-11-15 19:31:12:

Hmm, while I successfully added a "min_level()" method to the
Log::Dispatch::Output, I really meant to add one to Log::Dispatch. It's
less clear to me how to do that in a way that is "clean" within the
overall design. I'm going to keep thinking about the best way to solve
the case I've run into.

Mark

@autarch
Copy link
Member Author

autarch commented Nov 14, 2016

From [email protected] (@karenetheridge) on 2012-05-14 20:29:32:

min_level now exists; this ticket should be closed.

@autarch
Copy link
Member Author

autarch commented Nov 14, 2016

From [email protected] (@autarch) on 2012-07-26 14:25:03:

On Mon May 14 16:29:32 2012, ETHER wrote:

min_level now exists; this ticket should be closed.

The request was for a setter. All that exists now is a getter.

@autarch autarch changed the title wish: add set_min_level() method Add set_min_level() method Nov 14, 2016
@markstos
Copy link

Hello, I was the original bug reporter 10 years ago. I got a new job that doesn't use much Perl and don't use this module so feel free to close this feature request since no else added a "me too" comment in the last 10 years.

@martinvonwittich
Copy link

I'm looking for exactly this, and now I'm disappointed that it's not implemented :/

@autarch
Copy link
Member Author

autarch commented Jun 7, 2018

@martinvonwittich If you want to turn the patch into a PR I can take a look.

@martinvonwittich
Copy link

@autarch sorry, ain't nobody got time for that :/

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Development

No branches or pull requests

3 participants