Merge pull request #1834 from bolt/feature/upload-location

Use configured `upload_location` for images and files in ContentTypes
bolt · Sep 10, 2020 · 414064c · 414064c
2 parents b5201aa + 6a07824
commit 414064c
Show file tree

Hide file tree

Showing 11 changed files with 64 additions and 40 deletions.
diff --git a/config/bolt/config.yaml b/config/bolt/config.yaml
@@ -180,6 +180,8 @@ accept_media_types: [ gif, jpg, jpeg, png, svg, pdf, mp3, tiff ]
 #`post_max_size` and `upload_max_filesize` in `php.ini`.
 accept_upload_size: 8M
 
+upload_location: "{year}/{month}/{contenttype}/"
+
 # Options to use with curl requests.
 # For all options, check the official curl documentation here https://curl.haxx.se/libcurl/c/curl_easy_setopt.html
 curl_options:

diff --git a/src/Configuration/Parser/ContentTypesParser.php b/src/Configuration/Parser/ContentTypesParser.php
@@ -274,23 +274,28 @@ private function parseField($key, &$field, $acceptFileTypes, &$currentGroup): vo
             throw new ConfigurationException($error);
         }
 
-        // If field is a "file" type, make sure the 'extensions' are set, and it's an array.
+        // If field is a "file" type, make sure the 'extensions' are set.
         if ($field['type'] === 'file' || $field['type'] === 'filelist') {
             if (empty($field['extensions'])) {
                 $field['extensions'] = $acceptFileTypes;
             }
-
-            $field['extensions'] = (array) $field['extensions'];
         }
 
-        // If field is an "image" type, make sure the 'extensions' are set, and it's an array.
+        // If field is an "image" type, make sure the 'extensions' are set.
         if ($field['type'] === 'image' || $field['type'] === 'imagelist') {
             if (empty($field['extensions'])) {
                 $extensions = new Collection(['gif', 'jpg', 'jpeg', 'png', 'svg']);
                 $field['extensions'] = $extensions->intersect($acceptFileTypes)->toArray();
             }
+        }
 
+        // Image and File fields should have 'extensions' as an array, and a defined upload location
+        if (in_array($field['type'], ['file', 'filelist', 'image', 'imagelist'], true)) {
             $field['extensions'] = (array) $field['extensions'];
+
+            if (empty($field['upload'])) {
+                $field['upload'] = $this->generalConfig->get('upload_location');
+            }
         }
 
         // Make indexed arrays into associative for select fields

diff --git a/src/Configuration/Parser/GeneralParser.php b/src/Configuration/Parser/GeneralParser.php
@@ -132,6 +132,7 @@ protected function getDefaultConfig(): array
             'accept_file_types' => explode(',', 'twig,html,js,css,scss,gif,jpg,jpeg,png,ico,zip,tgz,txt,md,doc,docx,pdf,epub,xls,xlsx,csv,ppt,pptx,mp3,ogg,wav,m4a,mp4,m4v,ogv,wmv,avi,webm,svg'),
             'accept_media_types' => explode(',', 'gif,jpg,jpeg,png,svg,pdf,mp3,tiff'),
             'accept_upload_size' => '8M',
+            'upload_location' => '{year}/{month}/{contenttype}/',
             'branding' => [
                 'name' => 'Bolt',
                 'path' => '/bolt',

diff --git a/src/Controller/Backend/Async/UploadController.php b/src/Controller/Backend/Async/UploadController.php
@@ -76,7 +76,7 @@ public function handleUpload(Request $request): JsonResponse
 
         $uploadHandler = new Handler($target, [
             Handler::OPTION_AUTOCONFIRM => true,
-            Handler::OPTION_OVERWRITE => true,
+            Handler::OPTION_OVERWRITE => false,
         ]);
 
         $acceptedFileTypes = array_merge($this->config->getMediaTypes()->toArray(), $this->config->getFileTypes()->toArray());

diff --git a/src/Twig/HtmlExtension.php b/src/Twig/HtmlExtension.php
@@ -58,6 +58,7 @@ public function getFilters(): array
         return [
             new TwigFilter('markdown', [$this, 'markdown'], $safe),
             new TwigFilter('shy', [$this, 'shy'], $safe),
+            new TwigFilter('placeholders', [$this, 'placeholders'], $safe),
         ];
     }
 
@@ -101,4 +102,19 @@ public function absoluteLink(string $link): string
     {
         return Html::makeAbsoluteLink($link);
     }
+
+    public function placeholders(string $string, array $replacements = []): string
+    {
+        $baseReplacements = [
+            'year' => date('Y'),
+            'month' => date('m'),
+            'day' => date('D'),
+            'date' => date('Y-m-D'),
+            'random' => bin2hex(random_bytes(4)),
+        ];
+
+        $replacements = array_merge($baseReplacements, $replacements);
+
+        return Str::placeholders($string, $replacements, true);
+    }
 }
diff --git a/templates/_macro/_macro.html.twig b/templates/_macro/_macro.html.twig
@@ -39,7 +39,7 @@
 
 {% endapply %}{% endmacro %}
 
-{% macro generate_collection_fields(collectionField, fields, compileTemplates) %}{% apply spaceless %}
+{% macro generate_collection_fields(collectionField, fields, record, compileTemplates) %}{% apply spaceless %}
     {% set fieldsHtml = [] %}
     {% for item_field in fields %}
         {% set collectionItemName = 'collections[' ~ collectionField.name ~ '][' ~ item_field.definition.name ~ ']' %}
@@ -60,6 +60,7 @@
                 'collection_name': collectionField.name,
                 'collection_label': collectionField.definition.label,
                 'hash': hash,
+                'record': record
             } %}
 
             {% if item_field.type != 'set' %}

diff --git a/templates/_partials/fields/collection.html.twig b/templates/_partials/fields/collection.html.twig
@@ -12,12 +12,12 @@
     {% set limit = field.definition.get('limit')|default(200) %}
 
     {# get the html for all collection field already in the database #}
-    {% set existing_fields %}{{ macro.generate_collection_fields(field, field.value, false) }}{% endset %}
+    {% set existing_fields %}{{ macro.generate_collection_fields(field, field.value, record, false) }}{% endset %}
 
     {# get the html template for the collection fields defined in the field definition #}
     {% set templated_fields = "" %}
     {% if field.templates is defined %}
-        {% set templated_fields %}{{  macro.generate_collection_fields(field, field.templates, true) }}{% endset %}
+        {% set templated_fields %}{{  macro.generate_collection_fields(field, field.templates, record, true) }}{% endset %}
     {% endif %}
 
     <editor-collection

diff --git a/templates/_partials/fields/file.html.twig b/templates/_partials/fields/file.html.twig
@@ -7,7 +7,7 @@
 {% endset %}
 
 {% block field %}
-    {% set setPath = field.definition.get('upload')|default('') %}
+    {% set setPath = field.definition.get('upload')|placeholders({'random': 4, 'contenttype': record.contenttype}) %}
     {% set directory = path('bolt_async_upload', {'location': location|default('files'), 'path': setPath}) %}
     {% set filelist = path('bolt_async_filelisting', {'location': location|default('files') }) %}
     {% set labels = {

diff --git a/templates/_partials/fields/filelist.html.twig b/templates/_partials/fields/filelist.html.twig
@@ -7,7 +7,7 @@
 {% endset %}
 
 {% block field %}
-    {% set setPath = field.definition.get('upload')|default('') %}
+    {% set setPath = field.definition.get('upload')|placeholders({'random': 4, 'contenttype': record.contenttype}) %}
     {% set directory = path('bolt_async_upload', {'location': location|default('files'), 'path': setPath}) %}
     {% set filelist = path('bolt_async_filelisting', {'location': location|default('files') }) %}
     {% set labels = {

diff --git a/templates/_partials/fields/image.html.twig b/templates/_partials/fields/image.html.twig
@@ -8,7 +8,7 @@
 
 {% block field %}
 
-    {% set setPath = field.definition.get('upload')|default('') %}
+    {% set setPath = field.definition.get('upload')|placeholders({'random': 4, 'contenttype': record.contenttype}) %}
     {% set directory = path('bolt_async_upload', {'location': location|default('files'), 'path': setPath}) %}
     {% set filelist = path('bolt_async_filelisting', {'location': location|default('files'), 'type': 'images' }) %}
     {% set labels = {

diff --git a/templates/_partials/fields/imagelist.html.twig b/templates/_partials/fields/imagelist.html.twig
@@ -7,35 +7,34 @@
 {% endset %}
 
 {% block field %}
+    {% set setPath = field.definition.get('upload')|placeholders({'random': 4, 'contenttype': record.contenttype}) %}
+    {% set directory = path('bolt_async_upload', {'location': location|default('files'), 'path': setPath}) %}
+    {% set filelist = path('bolt_async_filelisting', {'location': location|default('files'), 'type': 'images' }) %}
+    {% set labels = {
+        'button_upload': 'image.button_upload'|trans,
+        'button_from_library': 'image.button_from_library'|trans,
+        'placeholder_filename': 'image.placeholder_filename'|trans,
+        'placeholder_alt_text': 'image.placeholder_alt_text'|trans,
+        'placeholder_title': 'image.placeholder_title'|trans,
+        'button_remove': 'image.button_remove'|trans,
+        'button_edit_attributes': 'image.button_edit_attributes'|trans,
+        'button_move_up': 'image.button_up'|trans,
+        'button_move_down': 'image.button_down'|trans,
+        'add_new_image': 'image.add_new_image'|trans,
+    }|json_encode %}
+    {% set limit = field.definition.get('limit')|default(200) %}
 
-   {% set setPath = field.definition.get('upload')|default('') %}
-   {% set directory = path('bolt_async_upload', {'location': location|default('files'), 'path': setPath}) %}
-   {% set filelist = path('bolt_async_filelisting', {'location': location|default('files'), 'type': 'images' }) %}
-   {% set labels = {
-       'button_upload': 'image.button_upload'|trans,
-       'button_from_library': 'image.button_from_library'|trans,
-       'placeholder_filename': 'image.placeholder_filename'|trans,
-       'placeholder_alt_text': 'image.placeholder_alt_text'|trans,
-       'placeholder_title': 'image.placeholder_title'|trans,
-       'button_remove': 'image.button_remove'|trans,
-       'button_edit_attributes': 'image.button_edit_attributes'|trans,
-       'button_move_up': 'image.button_up'|trans,
-       'button_move_down': 'image.button_down'|trans,
-       'add_new_image': 'image.add_new_image'|trans,
-   }|json_encode %}
-   {% set limit = field.definition.get('limit')|default(200) %}
-
-   <editor-imagelist
-       :images='{{ field.jsonvalue }}'
-       :name='{{ name|json_encode }}'
-       :directory='{{ directory|json_encode }}'
-       :filelist='{{ filelist|json_encode }}'
-       :csrf-token='{{ csrf_token('upload')|json_encode }}'
-       :labels='{{ labels }}'
-       :extensions='{{  extensions|json_encode }}'
-       :attributes-link='{{ path('bolt_media_new')|json_encode }}'
-       :limit='{{ limit|json_encode }}'
-       :readonly='{{ readonly|json_encode }}'
-   ></editor-imagelist>
+    <editor-imagelist
+        :images='{{ field.jsonvalue }}'
+        :name='{{ name|json_encode }}'
+        :directory='{{ directory|json_encode }}'
+        :filelist='{{ filelist|json_encode }}'
+        :csrf-token='{{ csrf_token('upload')|json_encode }}'
+        :labels='{{ labels }}'
+        :extensions='{{  extensions|json_encode }}'
+        :attributes-link='{{ path('bolt_media_new')|json_encode }}'
+        :limit='{{ limit|json_encode }}'
+        :readonly='{{ readonly|json_encode }}'
+    ></editor-imagelist>
 
 {% endblock %}