twigplugin.php

<?php 

kirby::$handlers['template'] = function($kirby, $page, $data = array()) {

  require_once(__DIR__ . DS . 'vendor' . DS . 'autoload.php');  

  $loader = new Twig_Loader_Filesystem($kirby->roots()->templates());
  $twig   = new Twig_Environment($loader, array(
    'cache' => false
  ));

  $data = array_merge(tpl::$data, array(
    'kirby' => $kirby,
    'site'  => $kirby->site(),
    'pages' => $kirby->site()->children(),
    'page'  => $page
  ), $page->templateData(), $data, $kirby->controller($page, $data));

  return $twig->render(basename($page->templateFile()), $data);

};

@shiftsave It sucks so badly that I didn't get any notifcation about your comment. I'm about to add the handlers api to the next version of Kirby, which makes this kind of plugins super easy. I'm not actually planning to be the main developer of a twit plugin, but I'd love to through a first version out there for a real twig expert to extend it. The same could happen for Jade.

This looks good, but why not leave the creation of the $data object to the caller? This will probably always be the same anyway, and is already defined here:

https://github.com/getkirby/kirby/blob/master/kirby.php#L666

Also, why is tpl::$data overridden on each call to $kirby->template()?

@shiftsave You might not need it anymore after close to one year, but for the record I made a Twig template component for Kirby 2.3: https://github.com/fvsch/kirby-twig
(And for other PHP templating engines, I guess it’s an okay enough example to help get people started.)