Re^2: Right place to put common execute code in App::Cmd


Your skill will accomplish what the force of many cannot
	PerlMonks

Re^2: Right place to put common execute code in App::Cmd

by McA (Priest)

on Dec 10, 2012 at 15:06 UTC ( [id://1008112]=note: print w/replies, xml )

Need Help??

in reply to Re: Right place to put common execute code in App::Cmd
in thread Right place to put common execute code in App::Cmd

Hi tobyink,

thank you for answering that fast. I have to admit that I hoped it. ;-) (a ++ for that)

The first suggestion is not an option for me right now. The second approach is which I'm doing right now. But I wanted to avoid the call of before_execute in every execute method. I really hoped that there is another callback which I haven't found.

Once again. Thank you.

Best regards
McA

Comment on Re^2: Right place to put common execute code in App::Cmd Select or Download Code

Replies are listed 'Best First'.

Re^3: Right place to put common execute code in App::Cmd
by tobyink (Canon) on Dec 10, 2012 at 16:43 UTC

"I really hoped that there is another callback which I haven't found."

The relevant code is App::Cmd::execute_command. As you can see, nothing happens between validate_args and execute that you could possibly hook onto.

One alternative approach could be to subclass App::Cmd, overriding that method with your own implementation that adds extra hooks between validate_args and execute:

{
  package My::App::Cmd;
  use base 'App::Cmd';
  sub execute_command {
    my ($self, $cmd, $opt, @args) = @_;
    local our $active_cmd = $cmd;
    $cmd->validate_args($opt, \@args);
    $cmd->before_execute($opt, \@args);
    $cmd->execute($opt, \@args);
  }
}
[download]

Then use My::App::Cmd instead of App::Cmd.

perl -E'sub Monkey::do{say$_,for@_,do{($monkey=[caller(0)]->[3])=~s{::}{ }and$monkey}}"Monkey say"->Monkey::do'

[reply]
[d/l]
[select]

In Section Seekers of Perl Wisdom