Hooks inside hooks possible?

[bernhard]

-- Update -- The hooks in my module work! Some things to pay attention to:

• Tracy's bd() call will maybe not be the best option for debugging, because you might have some session redirects going an and that might wipe your dump. So it could be possible that the hook is actually triggered but you will just not see the result.
• First, I tried using $this->log('...') and saw some more output than using bd(), but the best option (another new learning using TracyDebugger) is this:
• Using the log() funktion of Tracy is great:
  

This is my setup:

// in site/init.php
$this->addHookBefore("RockCI::restoreRemote", function($event) {
  l('restore hooked in site/init.php');
});

// in site/ready.php
$this->addHookBefore("RockCI::restoreRemote", function($event) {
  l('restore hooked in site/ready.php');
});

// in RockCI module
$this->addHookBefore("restoreRemote", function($event) {
  l('restore hooked in RockCI module');
});

So everything here works as expected. Note that the hook in site/ready.php does never get triggered! I'm not 100% sure WHY, and that's why I wish something like this would exist: https://processwire.com/talk/topic/10857-making-hooks-visible-hookrecorder/

Also note that I'm using addHookBefore as the $session->redirect() might prevent any execution of hooks added after that method!

Does that help you?

[Gadgetto]

  On 3/20/2020 at 10:47 AM, bernhard said:

So everything here works as expected. Note that the hook in site/ready.php does never get triggered!

Expand  

Good hint with Tracy log()!

But I don't understand what you tried to say: You say everything works like expected but then the hook in ready() isn't triggered. So you are having the same problem as I do?

[Gadgetto]

Just placed the hook in init.php and now it all works! This is mysterious ...

[bernhard]

It's getting even more confusing - now it works even in /site/ready.php ?

<?php namespace ProcessWire;
class RockHook extends WireData implements Module {

  public static function getModuleInfo() {
    return [
      'title' => 'RockHook',
      'version' => '0.0.1',
      'summary' => 'RockHook',
      'autoload' => true,
      'singular' => true,
      'icon' => 'bolt',
      'requires' => [],
      'installs' => [],
    ];
  }

  public function init() {
    bd('init()');
    bd(1);
    bd(2);
    bd(3);

    bd('add handleWebhook');
    $this->addHookAfter("ProcessPageView::execute", $this, "handleWebhook");

    bd('add hello-hook in init()');
    $this->addHookAfter('hello', function(HookEvent $event) {
      bd('hello-hook fired in init()');
    });
  }

  public function ready() {
    bd('ready()');
    
    bd('add hello-hook in ready()');
    $this->addHookAfter('hello', function(HookEvent $event) {
      bd('hello-hook fired in ready()');
    });
  }

  public function handleWebhook($event) {
    bd('handlewebhook fired');
    bd('trigger $this->hello()');
    $this->hello("trigger hello() in execute() hook");
  }

  public function ___hello($where) {
    bd('hello!', $where);
  }
}

 

[adrian]

  On 3/20/2020 at 10:47 AM, bernhard said:

Tracy's bd() call will maybe not be the best option for debugging, because you might have some session redirects going an and that might wipe your dump.

Expand  

That's when it's time to turn on the Dumps Recorder panel ?

[Gadgetto]

  On 4/6/2020 at 11:44 PM, adrian said:

That's when it's time to turn on the Dumps Recorder panel ?

Expand  

This is great, didn't even know this exists!

I always have problems with bd() within modals or panels and when a redirect is involved!

[adrian]

  On 4/7/2020 at 10:00 AM, Gadgetto said:

I always have problems with bd() within modals or panels

Expand  

Do you have Tracy enabled for inside modals, panels, iframes? There is a config setting for disabling it in those to make things cleaner, but in most cases you probably want them on.

[Gadgetto]

  On 4/7/2020 at 2:08 PM, adrian said:

Do you have Tracy enabled for inside modals, panels, iframes? There is a config setting for disabling it in those to make things cleaner, but in most cases you probably want them on.

Expand  

Yep, is enabled! On smaller displays (my MacBook) the 2 bars are overlapping sometimes, but it's great to have this possibility!