afterBuildMethod is not called on decorator, but only on base instance about di52

[lucatume]

Hi @netdust,

thanks for taking the time to open an issue.

Could you provide a minimal example to reproduce the issue?

[netdust]

<?php

use lucatume\DI52\Container;

$container = new Container();

// Decorators are built left to right, outer decorators are listed first.
 $container->bindDecorators(PostEndpoint::class, [
		    LoggingEndpoint::class,
                    CachingEndpoint::class,
		    BaseEndpoint::class
	    ],
           ['add_actions']
 );

$container->get(PostEndpoint::class);

This should call the add_action method on LoggingEndpoint first, then from there all the way to BaseEndPoint. But instead it calls the method directly on BaseEndpoint, skipping the decorators.

[lucatume]

Thanks for providing an example.

Since we're talking about the decorator pattern here, the add_actions after-build method should be called on the outer decorator only; i.e. the LoggingEndpoint one.
It should be, then, up to the LoggingEndpoint to call CachingEndpoint::add_actions that should, in turn, call the BaseEndpoint::add_actions method.

I will look into this with this frame of mind: the add_actions method should be called after the PostEndpoint is built, whatever is the outer decorator.

[netdust]

👌thx!