Adding missing icons type in the backend and db enums

Author: Michael-Schaer

app/Http/Controllers/ImageController.php

@@ -148,7 +148,7 @@ public function update(Request $request, Image $image)
             'background'  => [
                 'sometimes',
                 'required',
-                'in:'.Image::BG_CUSTOM.','.Image::BG_TRANSPARENT.','.Image::BG_GRADIENT,
+                'in:'.Image::BG_CUSTOM.','.Image::BG_TRANSPARENT.','.Image::BG_GRADIENT.','.Image::BG_ICONS,
                 new ImageBackgroundRule($image),
             ],
             // if mutable we would have to check the legal etc
@@ -215,7 +215,7 @@ public function store(Request $request, Image $image)
             ],
             'background'  => [
                 'required',
-                'in:'.Image::BG_CUSTOM.','.Image::BG_TRANSPARENT.','.Image::BG_GRADIENT,
+                'in:'.Image::BG_CUSTOM.','.Image::BG_TRANSPARENT.','.Image::BG_GRADIENT.','.Image::BG_ICONS,
                 new ImageBackgroundRule($image),
             ],
             'type'        => ['required', 'in:'.Image::TYPE_RAW.','.Image::TYPE_FINAL],

app/Image.php

@@ -70,6 +70,7 @@ class Image extends Model
     public const BG_GRADIENT = 'gradient';
     public const BG_TRANSPARENT = 'transparent';
     public const BG_CUSTOM = 'custom';
+    public const BG_ICONS = 'icons';
 
     /**
      * The attributes that should be hidden for arrays.

database/migrations/2025_04_14_171232_add_icons_background_to_images_table.php

@@ -0,0 +1,32 @@
+<?php
+
+use App\Image;
+use Illuminate\Support\Facades\Schema;
+use Illuminate\Database\Schema\Blueprint;
+use Illuminate\Database\Migrations\Migration;
+use Illuminate\Support\Facades\DB;
+
+class AddIconsBackgroundToImagesTable extends Migration
+{
+    /**
+     * Run the migrations.
+     *
+     * @return void
+     */
+    public function up()
+    {
+        DB::statement("ALTER TABLE images MODIFY COLUMN background ENUM('gradient', 'transparent', 'custom', 'icons') NOT NULL");
+    }
+
+    /**
+     * Reverse the migrations.
+     *
+     * @return void
+     */
+    public function down()
+    {
+        // ensure no rows use the 'icons' value
+        DB::table('images')->where('background', 'icons')->update(['background' => 'gradient']);
+        DB::statement("ALTER TABLE images MODIFY COLUMN background ENUM('gradient', 'transparent', 'custom') NOT NULL");
+    }
+}

resources/js/components/organisms/OImageDialog.vue

@@ -183,6 +183,10 @@ import {
             },
 
             downloadLink() {
+                if (!this.finalImageSrc) {
+                    return null;
+                }
+
                 const url = new URL(this.finalImageSrc);
                 url.searchParams.append('format', this.downloadImageFileExtension);
                 url.searchParams.append('color_profile', this.colorEncoding);
@@ -334,6 +338,10 @@ import {
             },
 
             processImage() {
+                if (!this.finalImageSrc) {
+                    return Promise.reject(new Error('Image source not available yet'));
+                }
+
                 // trigger a download to process the image on the server
                 // it will cache processed image server sides and dramatically
                 // increase ux when clicking the download button.