From 3ed705b1e2ae374b500d4225d26102eb8d937780 Mon Sep 17 00:00:00 2001
From: Vincent Vanwaelscappel <vincent@cubedesigners.com>
Date: Tue, 30 Aug 2022 19:44:47 +0200
Subject: [PATCH] wip #4216 @1

---
 app/Fields/FluidbookThemeImage.php            | 12 ++++++++++
 .../FluidbookPublication/PreviewOperation.php | 10 ++++----
 app/Jobs/FluidbookCompiler.php                | 16 ++++++++++---
 app/Models/FluidbookTheme.php                 | 23 ++++++++-----------
 4 files changed, 38 insertions(+), 23 deletions(-)
 create mode 100644 app/Fields/FluidbookThemeImage.php

diff --git a/app/Fields/FluidbookThemeImage.php b/app/Fields/FluidbookThemeImage.php
new file mode 100644
index 000000000..a726fac3a
--- /dev/null
+++ b/app/Fields/FluidbookThemeImage.php
@@ -0,0 +1,12 @@
+<?php
+
+namespace App\Fields;
+
+use Cubist\Backpack\Magic\Fields\Images;
+
+class FluidbookThemeImage extends Images
+{
+    protected $_translatable = false;
+    protected $_nameAsCollection=false;
+    protected $_renameCollectionOnChange=true;
+}
diff --git a/app/Http/Controllers/Admin/Operations/FluidbookPublication/PreviewOperation.php b/app/Http/Controllers/Admin/Operations/FluidbookPublication/PreviewOperation.php
index 9d6640813..107205a8f 100644
--- a/app/Http/Controllers/Admin/Operations/FluidbookPublication/PreviewOperation.php
+++ b/app/Http/Controllers/Admin/Operations/FluidbookPublication/PreviewOperation.php
@@ -174,12 +174,10 @@ html,body{height:100%;cursor: wait;font-family: "Open Sans", Arial;background-co
     protected static function _getFluidbookAndTheme($id, $hash, &$fluidbook, &$theme)
     {
         $ee = explode('-', $id);
-        if (count($ee) === 1) {
-            $forceThemeData = request('theme', false);
-            if ($forceThemeData) {
-                $theme = FluidbookTheme::fromArray(json_decode($forceThemeData, true));
-            }
-        } else {
+        $forceThemeData = request('theme', false);
+        if ($forceThemeData) {
+            $theme = FluidbookTheme::fromArray(json_decode($forceThemeData, true));
+        } else if (count($ee) == 2) {
             $theme = FluidbookTheme::find($ee[1]);
             $id = $ee[0];
         }
diff --git a/app/Jobs/FluidbookCompiler.php b/app/Jobs/FluidbookCompiler.php
index 3eb75d475..864d06616 100644
--- a/app/Jobs/FluidbookCompiler.php
+++ b/app/Jobs/FluidbookCompiler.php
@@ -406,11 +406,21 @@ class FluidbookCompiler extends Base implements CompilerInterface
         if ($collection) {
             $path = $this->theme->getFirstMediaPath($collection);
             if (!$path || !file_exists($path)) {
-                return false;
+                $res = false;
+            } else {
+                $res = new SplFileInfo($path);
             }
-            return new SplFileInfo($path);
+        } else {
+            $res = false;
         }
-        return false;
+        if (null !== $this->config) {
+            if ($res instanceof SplFileInfo) {
+                $this->config->set($key, $res->getFilename());
+            } else {
+                $this->config->set($key, '');
+            }
+        }
+        return $res;
     }
 
     /**
diff --git a/app/Models/FluidbookTheme.php b/app/Models/FluidbookTheme.php
index 838c72e95..536817cac 100644
--- a/app/Models/FluidbookTheme.php
+++ b/app/Models/FluidbookTheme.php
@@ -3,6 +3,7 @@
 
 namespace App\Models;
 
+use App\Fields\FluidbookThemeImage;
 use App\Jobs\GenerateThemePreview;
 use App\Jobs\UpdateWS2ThemeTable;
 use App\Models\Base\ToolboxSettingsModel;
@@ -225,8 +226,8 @@ class FluidbookTheme extends ToolboxSettingsModel
 
         $this->addField([
             'name' => 'symbols',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
+            'accept' => '.svg',
             'label' => __('Remplacer des icônes (symboles svg)'),
             'translatable' => false,
         ]);
@@ -417,8 +418,7 @@ class FluidbookTheme extends ToolboxSettingsModel
         ]);
         $this->addField([
             'name' => 'backgroundImage',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
             'label' => __('Image de fond'),
             'translatable' => false,
         ]);
@@ -495,8 +495,7 @@ class FluidbookTheme extends ToolboxSettingsModel
 
         $this->addField([
             'name' => 'menuImage',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
             'label' => __('Image de fond'),
             'translatable' => false,
         ]);
@@ -510,8 +509,7 @@ class FluidbookTheme extends ToolboxSettingsModel
         ]);
         $this->addField([
             'name' => 'afterSearch',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
             'label' => __('Image supplémentaire'),
             'hint' => __('Image disposée à droite des icônes'),
             'translatable' => false,
@@ -523,8 +521,7 @@ class FluidbookTheme extends ToolboxSettingsModel
         ]);
         $this->addField([
             'name' => 'logo',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
             'label' => __('Logo'),
             'translatable' => false,
         ]);
@@ -540,8 +537,7 @@ class FluidbookTheme extends ToolboxSettingsModel
         ]);
         $this->addField([
             'name' => 'favicon',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
             'label' => __('Icone (pour favicon et CD-ROM)'),
             'hint' => __('Image PNG de 256x256'),
             'translatable' => false,
@@ -580,8 +576,7 @@ class FluidbookTheme extends ToolboxSettingsModel
         ]);
         $this->addField([
             'name' => 'logoLoader',
-            'name_as_collection' => true,
-            'type' => 'Images',
+            'type' => FluidbookThemeImage::class,
             'label' => __('Logo affiché au chargement'),
             'translatable' => false,
         ]);
-- 
2.39.5