Do not render helper output

[thekid]

The following code produces the output YEHUDA Katz by interpolating the string returned from the helper:

use com\handlebarsjs\HandlebarsEngine;

$engine= new HandlebarsEngine();
$engine->withHelper('full', function($node, $context, $options) {
  return strtoupper($options[0]).' {{lastname}}';
});

echo $engine->render('{{full firstname}}', [
  'firstname' => 'Yehuda',
  'lastname'  => 'Katz',
]);

However, the official Handlebars engine does not do this:

This is due to how the Mustache library is implemented. It even has a unittest verifying this behavior.

[thekid]

One idea would be to decouple this library from the Mustache library. This would mean quite a bit of changes!

[thekid]

The other idea would be to overwrite the asRendering() method as follows:

diff --git a/src/main/php/com/handlebarsjs/DefaultContext.class.php b/src/main/php/com/handlebarsjs/DefaultContext.class.php
index ae1dfb4..1ea8308 100755
--- a/src/main/php/com/handlebarsjs/DefaultContext.class.php
+++ b/src/main/php/com/handlebarsjs/DefaultContext.class.php
@@ -36,6 +36,24 @@ class DefaultContext extends DataContext {
     return new self($result, $parent ?: $this);
   }
 
+  /**
+   * Returns a rendering of a given helper closure
+   *
+   * @param  var $closure
+   * @param  com.github.mustache.Node $node
+   * @param  string[] $options
+   * @param  string $start
+   * @param  string $end
+   * @return string
+   */
+  public function asRendering($closure, $node, $options= [], $start= '{{', $end= '}}') {
+    $pass= [];
+    foreach ($options as $key => $option) {
+      $pass[$key]= $this->isCallable($option) ? $option($node, $this, $pass) : $option;
+    }
+    return $closure($node, $this, $pass);
+  }
+
   /**
    * Parse input into segments. Handles literal segments including quoted
    * strings, e.g. `input.["item-class"]`.

[thekid]

Released in https://github.com/xp-forge/handlebars/releases/tag/v8.1.0