Phalcon Logger Interpolation does not work

[gnikolopoulos]

Hi all,

I am using the code from here in order to inject additional, dynamic data into my logging process while calling an external API, but I only get the placeholders themselves instead of the context values.

I am using Phalcon 5.2.1 and even tried the code on the example as is, to no avail.

[s-ohnishi]

I tested it and got the same result.
I checked the source of .zep and finally found this.

Phalcon\Support\Helper\Str\AbstractStr::toInterpolate(
    string input,
    array context = [],
    string left = "%",
    string right = "%"
)

And neither left and right are specified when calling, so the description in the manual is wrong.
Following this will give you the expected result:

$message = '%framework% executed the "Hello World" test in %secs% second(s)';

[s-ohnishi]

An inconsistency was found in the HelperFactory.zep description.
Show only the problem part.

  * @method bool between(int $value, int $start, int $end)

and

  "filter" : "Phalcon\\Support\\Helper\\Arr\\Filter",
  "isBetween" : "Phalcon\\Support\\Helper\\Number\\IsBetween",
  "dynamic" : "Phalcon\\Support\\Helper\\Str\\Dynamic",
  "interpolate" : "Phalcon\\Support\\Helper\\Str\\Interpolate",

"isBetween" is "between" in PHPDoc, and nothing else exists.

[niden]

Thank you @s-ohnishi I fixed that in the latest PR

[s-ohnishi]

The test I was taught was "not found".
Is it something that hasn't been published or pushed?

I think it's a problem that it can't be referenced properly in the IDE, so I hope the PR will be merged soon.

[niden]

The PR was merged and available with the new release. Can you check if you don't mind?

[s-ohnishi]

I only have a rough understanding of Zephir, so even if I look at the commit, I don't understand it very well, so I decline.
However, it seems that the comment part affects ide-stubs, so I would like you to correct the comment in HelperFactory.zep.

[Jeckerson]

Issue created #16427