From 09e4bb8cad52243fe4c32ad8593c8060eda253b8 Mon Sep 17 00:00:00 2001
From: Chris Hunt <chris.hunt@kasmweb.com>
Date: Mon, 19 Feb 2024 10:03:03 +0000
Subject: [PATCH] Fix required fields and default items and tags to pinned

---
 resources/views/items/form.blade.php | 12 ++++++------
 resources/views/tags/form.blade.php  |  8 ++++----
 2 files changed, 10 insertions(+), 10 deletions(-)

diff --git a/resources/views/items/form.blade.php b/resources/views/items/form.blade.php
index 638986bd..8d24a7f1 100644
--- a/resources/views/items/form.blade.php
+++ b/resources/views/items/form.blade.php
@@ -7,8 +7,8 @@
                 {!! Form::hidden('pinned', '0') !!}
                 <label class="switch">
                     <?php
-                    $checked = false;
-                    if(isset($item->pinned) && (bool)$item->pinned === true) $checked = true;
+                    $checked = true;
+                    if(isset($item->pinned) && (bool)$item->pinned !== true) $checked = false;
                     $set_checked = ($checked) ? ' checked="checked"' : '';
                     ?>                   
                     <input type="checkbox" name="pinned" value="1"<?php echo $set_checked;?> />
@@ -57,17 +57,17 @@
             {!! csrf_field() !!}
             <div class="input">
                 <label>{{ __('app.apps.application_name') }} *</label>
-                {!! Form::text('title', null, array('placeholder' => __('app.apps.title'), 'id' => 'appname', 'class' => 'form-control')) !!}
+                {!! Form::text('title', null, array('placeholder' => __('app.apps.title'), 'id' => 'appname', 'class' => 'form-control', 'required')) !!}
             </div>
 
             <div class="input">
-                <label>{{ __('app.apps.colour') }} *</label>
+                <label>{{ __('app.apps.colour') }}</label>
                 {!! Form::text('colour', $item->colour ?? '#161b1f', array('placeholder' => __('app.apps.hex'), 'id' => 'appcolour', 'class' => 'form-control color-picker set-bg-elem')) !!}
             </div>
 
             <div class="input">
-                <label>{{ strtoupper(__('app.url')) }}</label>
-                {!! Form::text('url', $item->url ?? null, array('placeholder' => __('app.url'), 'id' => 'appurl', 'class' => 'form-control')) !!}
+                <label>{{ strtoupper(__('app.url')) }} *</label>
+                {!! Form::text('url', $item->url ?? null, array('placeholder' => __('app.url'), 'id' => 'appurl', 'class' => 'form-control', 'required')) !!}
                 <small class="help">Don't forget http(s)://</small>
             </div>
 
diff --git a/resources/views/tags/form.blade.php b/resources/views/tags/form.blade.php
index 90d2ed76..634632d0 100644
--- a/resources/views/tags/form.blade.php
+++ b/resources/views/tags/form.blade.php
@@ -11,14 +11,14 @@
 
             <div class="input">
                 <label>{{ __('app.apps.tag_name') }} *</label>
-                {!! Form::text('title', null, array('placeholder' => __('app.apps.title'), 'class' => 'form-control')) !!}
+                {!! Form::text('title', null, array('placeholder' => __('app.apps.title'), 'class' => 'form-control', 'required')) !!}
                 <hr />
                 <label>{{ __('app.apps.pinned') }}</label>
                 {!! Form::hidden('pinned', '0') !!}
                 <label class="switch">
                     <?php
-                    $checked = false;
-                    if(isset($item->pinned) && (bool)$item->pinned === true) $checked = true;
+                    $checked = true;
+                    if(isset($item->pinned) && (bool)$item->pinned !== true) $checked = false;
                     $set_checked = ($checked) ? ' checked="checked"' : '';
                     ?>                   
                     <input type="checkbox" name="pinned" value="1"<?php echo $set_checked;?> />
@@ -26,7 +26,7 @@
                 </label>
             </div>
             <div class="input">
-                <label>{{ __('app.apps.colour') }} *</label>
+                <label>{{ __('app.apps.colour') }}</label>
                 {!! Form::text('colour', null, array('placeholder' => __('app.apps.hex'),'class' => 'form-control color-picker')) !!}
                 <hr />
             </div>