From 28cd5d7cc19e00f73a50ddd7a1fd7f5e1a0d612c Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Jonathan=20Beli=C3=ABn?= <jbe@geo6.be>
Date: Wed, 28 Aug 2019 17:12:55 +0200
Subject: [PATCH] [WIP] Display links to files

Related to #17
---
 config/autoload/templates.global.php |  2 +
 src/App/Twig/FileExtension.php       | 88 ++++++++++++++++++++++++++++
 src/App/Twig/ValueExtension.php      | 30 +++++++---
 templates/app/table/body.html.twig   | 22 ++++---
 templates/app/table/head.html.twig   | 22 ++++++-
 5 files changed, 145 insertions(+), 19 deletions(-)
 create mode 100644 src/App/Twig/FileExtension.php

diff --git a/config/autoload/templates.global.php b/config/autoload/templates.global.php
index 2bca220e..b9ea123c 100644
--- a/config/autoload/templates.global.php
+++ b/config/autoload/templates.global.php
@@ -5,11 +5,13 @@
 return [
     'dependencies' => [
         'factories'  => [
+            App\Twig\FileExtension::class => Zend\ServiceManager\AbstractFactory\ReflectionBasedAbstractFactory::class,
             App\Twig\ValueExtension::class => Zend\ServiceManager\AbstractFactory\ReflectionBasedAbstractFactory::class,
         ],
     ],
     'twig' => [
         'extensions' => [
+            App\Twig\FileExtension::class,
             App\Twig\ValueExtension::class,
         ],
     ],
diff --git a/src/App/Twig/FileExtension.php b/src/App/Twig/FileExtension.php
new file mode 100644
index 00000000..35fb9b33
--- /dev/null
+++ b/src/App/Twig/FileExtension.php
@@ -0,0 +1,88 @@
+<?php
+
+declare(strict_types=1);
+
+namespace App\Twig;
+
+use Twig\Extension\AbstractExtension;
+use Twig\TwigFunction;
+
+class FileExtension extends AbstractExtension
+{
+    public function getFunctions(): array
+    {
+        return [
+        new TwigFunction('file', [$this, 'file'], ['is_safe' => ['html']]),
+        ];
+    }
+
+    public function file($path, bool $preview, bool $download): string
+    {
+        if (!is_null($path)) {
+            $path = trim($path);
+        }
+
+        if (is_null($path) || strlen($path) === 0 || !file_exists($path)) {
+            $output = '<td';
+            $output .= ' class="text-nowrap"';
+            $output .= ' colspan="2"';
+            $output .= '>';
+
+            if (is_null($path)) {
+                $output .= ValueExtension::null();
+            } elseif (strlen($path) === 0) {
+                $output .= '';
+            } elseif (!file_exists($path)) {
+                $output .= self::notexists($path);
+            }
+
+            $output .= '</td>';
+        } else {
+            $output = '';
+
+            if ($preview === true) {
+                $output = '<td ';
+                $output .= ' class="text-nowrap"';
+                $output .= $download !== true ? ' colspan="2"' : '';
+                $output .= '>';
+                $output .= self::preview($path, true);
+                $output .= '</td>';
+            }
+
+            if ($download === true) {
+                $output .= '<td ';
+                $output .= ' class="text-nowrap"';
+                $output .= $preview !== true ? ' colspan="2"' : '';
+                $output .= '>';
+                $output .= self::download($path, $preview && $download ? false : true);
+                $output .= '</td>';
+            }
+        }
+
+        return $output;
+    }
+
+    private function notexists($path) : string
+    {
+        return '<span class="text-muted" title="File does not exists." style="cursor: help; font-style: italic;">'
+            .'<i class="fas fa-fw fa-exclamation-circle"></i>'
+            .' '.basename($path)
+            .'</span>';
+    }
+
+    private function preview($path, bool $displayFilename) : string
+    {
+        return '<a href="#" style="text-decoration: none;">'
+            .'<i class="far fa-fw fa-eye"></i> '
+            .($displayFilename ? ' '.basename($path) : '')
+            .'</a>';
+    }
+
+    private function download($path, bool $displayFilename) : string
+    {
+        return '<a href="#" style="text-decoration: none;">'
+            .'<i class="fas fa-fw fa-file-download"></i> '
+            .($displayFilename ? ' '.basename($path) : '')
+            .'</a>';
+    }
+}
diff --git a/src/App/Twig/ValueExtension.php b/src/App/Twig/ValueExtension.php
index c0b488e5..9e7e45e7 100644
--- a/src/App/Twig/ValueExtension.php
+++ b/src/App/Twig/ValueExtension.php
@@ -18,22 +18,34 @@ public function getFunctions(): array
 
     public function value($value, ?string $datatype = null): string
     {
-        if (is_null($value)) {
-            return self::null();
+        $textAlign = 'left';
+        if ($datatype === 'integer') {
+            $textAlign = 'right';
+        } elseif ($datatype === 'boolean') {
+            $textAlign = 'center';
         }
 
-        if ($datatype === 'boolean') {
-            return self::boolean($value);
-        }
+        $output = '<td';
+        $output .= sprintf(' class="text-nowrap text-%s"', $textAlign);
+        $output .= sprintf(' data-datatype="%s"', (string) $datatype);
+        $output .= '>';
 
-        if (is_string($value) && filter_var($value, FILTER_VALIDATE_URL) !== false) {
-            return self::varcharLink($value);
+        if (is_null($value)) {
+            $output .= self::null();
+        } elseif ($datatype === 'boolean') {
+            $output .= self::boolean($value);
+        } elseif (is_string($value) && filter_var($value, FILTER_VALIDATE_URL) !== false) {
+            $output .= self::varcharLink($value);
+        } else {
+            $output .= (string) $value;
         }
 
-        return (string) $value;
+        $output .= '</td>';
+
+        return $output;
     }
 
-    private static function null(): string
+    public static function null(): string
     {
         return '<span class="text-muted font-italic">NULL</span>';
     }
diff --git a/templates/app/table/body.html.twig b/templates/app/table/body.html.twig
index 50847710..0c1baa55 100644
--- a/templates/app/table/body.html.twig
+++ b/templates/app/table/body.html.twig
@@ -16,10 +16,11 @@
         </td>
         {% endif %}
 
-        <td class="text-nowrap{{ column.datatype == 'integer' ? ' text-right' : column.datatype == 'boolean' ? ' text-center' }}"
-            data-datatype="{{ column.datatype }}">
-            {{ value(value, column.datatype) }}
-        </td>
+        {% if column.config.preview == true or column.config.download == true %}
+        {{ file(value, column.config.preview, column.config.download) }}
+        {% else %}
+        {{ value(value, column.datatype) }}
+        {% endif %}
 
         {% endfor %}
 
@@ -29,10 +30,15 @@
         {% set fcolumns = foreign.columns | filter(c => c.name in fselectColumns) %}
 
         {% for fcolumn in fcolumns %}
-        <td
-            class="table-info text-nowrap{{ fcolumn.datatype == 'integer' ? ' text-right' : fcolumn.datatype == 'boolean' ? ' text-center' }}">
-            {{ value(attribute(record.properties, foreign.name ~ '.' ~ fcolumn.name), fcolumn.datatype) }}
-        </td>
+
+        {% set value = attribute(record.properties, foreign.name ~ '.' ~ fcolumn.name) %}
+
+        {% if fcolumn.config.preview == true or fcolumn.config.download == true %}
+        {{ file(value, fcolumn.config.preview, fcolumn.config.download) }}
+        {% else %}
+        {{ value(value, fcolumn.datatype) }}
+        {% endif %}
+
         {% endfor %}
 
         {% endfor %}
diff --git a/templates/app/table/head.html.twig b/templates/app/table/head.html.twig
index 40385231..54f4d00d 100644
--- a/templates/app/table/head.html.twig
+++ b/templates/app/table/head.html.twig
@@ -11,13 +11,23 @@
     </tr>
     <tr>
         {% for column in columns %}
+
         {% set columnName = table.name ~ '.' ~ column.name %}
-        <th scope="col" {{ thematic.column == column.name ? ' colspan="2"'}}>
+
+        {% set colspan = 1 %}
+        {% if thematic.column == column.name %}
+        {% set colspan = 2 %}
+        {% elseif column.config.preview == true or column.config.download == true %}
+        {% set colspan = 2 %}
+        {% endif %}
+
+        <th scope="col" colspan="{{ colspan }}">
             <a href="{{ path('table', {'config': configId, 'offset': 0}) }}?sort={{ columnName }}&amp;order={{ sort == columnName and order == 'asc' ? 'desc' : 'asc' }}"
                 class="d-flex justify-content-between align-items-center">
                 <span class="text-nowrap">
                     {{ thematic.column == column.name ? '<i class="fas fa-palette"></i>' }}
                     {{ column.isForeignKey == true ? '<i class="fas fa-link"></i>' }}
+                    {{ column.config.preview == true or column.config.download == true ? '<i class="fas fa-file"></i>' }}
                     {{ column.name }}
                 </span>
                 {% if sort == columnName and order == 'asc' %}
@@ -37,11 +47,18 @@
         {% set fcolumns = foreign.columns | filter(c => c.name in fselectColumns) %}
 
         {% for fcolumn in fcolumns %}
-        <th scope="col">
+
+        {% set colspan = 1 %}
+        {% if fcolumn.config.preview == true or fcolumn.config.download == true %}
+        {% set colspan = 2 %}
+        {% endif %}
+
+        <th scope="col" colspan="{{ colspan }}">
             {% set columnName = foreign.name ~ '.' ~ fcolumn.name %}
             <a href="{{ path('table', {'config': configId, 'offset': 0}) }}?sort={{ columnName }}&amp;order={{ sort == columnName and order == 'asc' ? 'desc' : 'asc' }}"
                 class="d-flex justify-content-between align-items-center">
                 <span class="text-nowrap">
+                    {{ fcolumn.config.preview == true or fcolumn.config.download == true ? '<i class="fas fa-file"></i>' }}
                     {{ fcolumn.name }}
                 </span>
                 {% if sort == columnName and order == 'asc' %}
@@ -53,6 +70,7 @@
                 {% endif %}
             </a>
         </th>
+
         {% endfor %}
 
         {% endfor %}