From a467b0289a57e4f5731bc9fe41192575c5fd8dcb Mon Sep 17 00:00:00 2001
From: Denis Brumann <denis.brumann@sensiolabs.de>
Date: Sun, 4 Feb 2018 16:34:45 +0100
Subject: [PATCH] Add TemplateController for even more effortless templating.

---
 README.md                                   | 43 +++++++++++-
 composer.json                               |  3 +-
 composer.lock                               |  6 +-
 src/Controller/TemplateController.php       | 42 ++++++++++++
 tests/Controller/TemplateControllerTest.php | 74 +++++++++++++++++++++
 5 files changed, 163 insertions(+), 5 deletions(-)
 create mode 100644 src/Controller/TemplateController.php
 create mode 100644 tests/Controller/TemplateControllerTest.php

diff --git a/README.md b/README.md
index ff3f6e0..cff255a 100644
--- a/README.md
+++ b/README.md
@@ -27,8 +27,12 @@ The bundle can be installed via Composer:
 
     composer require jaem3l/template-bundle
 
-Usage Examples
---------------
+When using this bundle inside a Symfony 4 application bootstrapped with Flex,
+make sure you have installed the twig-bundle and set it up with its recipe, e.g.
+by using `composer req twig`.
+
+Usage examples for Route-annotation
+-----------------------------------
 
     <?php
 
@@ -66,3 +70,38 @@ Usage Examples
             ];
         }
     }
+
+Usage examples for TemplateController
+-------------------------------------
+
+In your routing you can now rely on this controller for directly rendering a
+provided template:
+
+```
+# SF3: app/config/routing.yml
+# SF4: config/routes.yaml
+
+hello_world:
+    path: /hello-world
+    controller: jæm3l\TemplateBundle\Controller\TemplateController::template
+    defaults:
+        template: 'Hello World!'
+        
+twig_expression:
+    path: /twig-expression
+    controller: jæm3l\TemplateBundle\Controller\TemplateController::template
+
+    defaults:
+        template: '{{ 1 + 2 }}'
+        
+advanced_template:
+    path: /advanced-template
+    controller: jæm3l\TemplateBundle\Controller\TemplateController
+    defaults:
+        template: |
+            {% extends 'base.html.twig' %}
+
+            {% block body %}
+            Hello World!
+            {% endblock %}
+```
diff --git a/composer.json b/composer.json
index 1b7aa2e..771d9c9 100644
--- a/composer.json
+++ b/composer.json
@@ -21,7 +21,8 @@
     },
     "require": {
         "twig/twig": "^2.4",
-        "sensio/framework-extra-bundle": "^5.1"
+        "sensio/framework-extra-bundle": "^5.1",
+        "php": "^7.1"
     },
     "require-dev": {
         "phpunit/phpunit": "^6.5"
diff --git a/composer.lock b/composer.lock
index 0681368..c966de0 100644
--- a/composer.lock
+++ b/composer.lock
@@ -4,7 +4,7 @@
         "Read more about it at https://getcomposer.org/doc/01-basic-usage.md#composer-lock-the-lock-file",
         "This file is @generated automatically"
     ],
-    "content-hash": "727427adc87026ccc53822e6c0eaeffe",
+    "content-hash": "e7b4613987785d15cd3ceb51cd588ca1",
     "packages": [
         {
             "name": "doctrine/annotations",
@@ -3006,6 +3006,8 @@
     "stability-flags": [],
     "prefer-stable": false,
     "prefer-lowest": false,
-    "platform": [],
+    "platform": {
+        "php": "^7.1"
+    },
     "platform-dev": []
 }
diff --git a/src/Controller/TemplateController.php b/src/Controller/TemplateController.php
new file mode 100644
index 0000000..b43e134
--- /dev/null
+++ b/src/Controller/TemplateController.php
@@ -0,0 +1,42 @@
+<?php declare(strict_types = 1);
+
+namespace jæm3l\TemplateBundle\Controller;
+
+use Symfony\Component\HttpFoundation\Response;
+use Twig\Environment;
+
+final class TemplateController
+{
+    private $twig;
+
+    public function __construct(Environment $twig)
+    {
+        $this->twig = $twig;
+    }
+
+    public function templateAction(string $template, ?int $maxAge, ?int $sharedMaxAge, ?bool $private): Response
+    {
+        $twigTemplate = $this->twig->createTemplate($template);
+
+        $response = new Response($twigTemplate->render([]));
+
+        if ($maxAge) {
+            $response->setMaxAge($maxAge);
+        }
+        if ($sharedMaxAge) {
+            $response->setSharedMaxAge($sharedMaxAge);
+        }
+        if ($private) {
+            $response->setPrivate();
+        } elseif (false === $private || (null === $private && ($maxAge || $sharedMaxAge))) {
+            $response->setPublic();
+        }
+
+        return $response;
+    }
+
+    public function __invoke(string $template, ?int $maxAge, ?int $sharedMaxAge, ?bool $private): Response
+    {
+        return $this->templateAction($template, $maxAge, $sharedMaxAge, $private);
+    }
+}
diff --git a/tests/Controller/TemplateControllerTest.php b/tests/Controller/TemplateControllerTest.php
new file mode 100644
index 0000000..a59842a
--- /dev/null
+++ b/tests/Controller/TemplateControllerTest.php
@@ -0,0 +1,74 @@
+<?php declare(strict_types = 1);
+
+namespace jæm3l\TemplateBundle\Tests\Controller;
+
+use jæm3l\TemplateBundle\Controller\TemplateController;
+use PHPUnit\Framework\TestCase;
+use Twig\Environment;
+use Twig\Loader\ArrayLoader;
+
+class TemplateControllerTest extends TestCase
+{
+    /**
+     * @var TemplateController
+     */
+    private $controller;
+
+    protected function setUp()
+    {
+        parent::setUp();
+
+        $loader = new ArrayLoader([]);
+        $twig = new Environment($loader);
+
+        $this->controller = new TemplateController($twig);
+    }
+
+    public function test_it_renders_basic_template()
+    {
+        $template = 'Hello World!';
+
+        $response = $this->controller->templateAction($template, null, null, null);
+
+        $this->assertSame($template, $response->getContent());
+        $this->assertSame('no-cache, private', $response->headers->get('Cache-Control'));
+    }
+
+    public function test_it_sets_max_age_on_response()
+    {
+        $template = 'Hello World!';
+
+        $response = $this->controller->templateAction($template, 360, null, null);
+
+        $this->assertContains('max-age=360', $response->headers->get('Cache-Control'));
+        $this->assertContains('public', $response->headers->get('Cache-Control'));
+    }
+
+    public function test_it_sets_shared_max_age_on_response()
+    {
+        $template = 'Hello World!';
+
+        $response = $this->controller->templateAction($template, null, 360, null);
+
+        $this->assertContains('s-maxage=360', $response->headers->get('Cache-Control'));
+        $this->assertContains('public', $response->headers->get('Cache-Control'));
+    }
+
+    public function test_it_sets_cache_private_on_response()
+    {
+        $template = 'Hello World!';
+
+        $response = $this->controller->templateAction($template, null, null, true);
+
+        $this->assertContains('private', $response->headers->get('Cache-Control'));
+    }
+
+    public function test_it_sets_cache_public_on_response()
+    {
+        $template = 'Hello World!';
+
+        $response = $this->controller->templateAction($template, null, null, false);
+
+        $this->assertContains('public', $response->headers->get('Cache-Control'));
+    }
+}