Qualify `call_user_func_array`
j-applese3d opened this issue ยท 2 comments
This feels like a PHP bug, so feel free to close as off-topic/won't fix. ๐
Basically, the stack trace - as returned from debug_backtrace()
- is messed up when calling call_user_func_array()
from within a namespace.
Simple PHP example:
<?php
namespace SomeTest;
function printTrace()
{
print_r(debug_backtrace(DEBUG_BACKTRACE_IGNORE_ARGS));
}
call_user_func_array('\SomeTest\printTrace', []);
The output is:
Array
(
[0] => Array
(
[function] => SomeTest\printTrace
)
[1] => Array
(
[file] => /var/www/***
[line] => 10
[function] => call_user_func_array
)
)
But, if you Fully-Qualify the function used, then you'll find the trace is "normal".
You can qualify the function by either prepending a \
like \call_user_func_array(...
,
or you can add a use function call_user_func_array;
statement at the top
Either method will fix the trace, and will have the added [alleged] benefit of being faster (as it doesn't have to check for a namespaced function):
Array
(
[0] => Array
(
[file] => /var/www/***
[line] => 10
[function] => SomeTest\printTrace
)
)
How does this affect Smarty?
Smarty makes use of call_user_func_array
. Ex: in Extension\CallbackWrapper.php.
smarty/src/Extension/CallbackWrapper.php
Line 29 in a1b4c9c
This could be changed to either Fully Qualify the function (as explained above), or call_user_func_array could simply be dropped and use the splat operator like so:
return ($this->callback)(...$params);
But like I mentioned, the entire issue is somewhat beyond the scope of Smarty.
Just figured it may be of some interest.
Sounds good. Just to be sure: you are saying it would be better to add a backslash before each time Smarty uses call_user_func_array
from a namespaced file?
Yes, that is correct.
Either that (add \
in front of every call_user_func_array
),
Or add use function call_user_func_array;
Or don't use call_user_func_array at all and simply execute the callable: ($this->callback)(...$params);
But again, it's not a "smarty problem" so if you do not wish to add \
everywhere, I can also understand and do not mind. ๐