diff --git a/app/Http/Controllers/Assets/AssetsController.php b/app/Http/Controllers/Assets/AssetsController.php index 65b64ef0da..83c7a08ea1 100755 --- a/app/Http/Controllers/Assets/AssetsController.php +++ b/app/Http/Controllers/Assets/AssetsController.php @@ -398,6 +398,7 @@ class AssetsController extends Controller $asset->assigned_to = null; $asset->assigned_type = null; $asset->accepted = null; + $asset->last_checkin = now(); event(new CheckoutableCheckedIn($asset, $target, auth()->user(), 'Checkin on asset update with '.$status->getStatuslabelType().' status', date('Y-m-d H:i:s'), $originalValues)); } diff --git a/resources/macros/macros.php b/resources/macros/macros.php index b048c407fa..b23661dcf9 100644 --- a/resources/macros/macros.php +++ b/resources/macros/macros.php @@ -104,23 +104,6 @@ Form::macro('digit_separator', function ($name = 'digit_separator', $selected = return $select; }); - -Form::macro('name_display_format', function ($name = 'name_display_format', $selected = null, $class = null) { - $formats = [ - 'first_last' => trans('general.firstname_lastname_display'), - 'last_first' => trans('general.lastname_firstname_display'), - ]; - - $select = ''; - - return $select; -}); - /** * Barcode macro * Generates the dropdown menu of available 1D barcodes @@ -168,32 +151,6 @@ Form::macro('barcode_types', function ($name = 'barcode_type', $selected = null, return $select; }); -Form::macro('email_format', function ($name = 'email_format', $selected = null, $class = null) { - $formats = [ - 'firstname.lastname' => trans('admin/settings/general.email_formats.firstname_lastname_format'), - 'firstname' => trans('admin/settings/general.email_formats.first_name_format'), - 'lastname' => trans('admin/settings/general.email_formats.last_name_format'), - 'filastname' => trans('admin/settings/general.email_formats.filastname_format'), - 'lastnamefirstinitial' => trans('admin/settings/general.email_formats.lastnamefirstinitial_format'), - 'firstname_lastname' => trans('admin/settings/general.email_formats.firstname_lastname_underscore_format'), - 'firstinitial.lastname' => trans('admin/settings/general.email_formats.firstinitial_lastname'), - 'lastname_firstinitial' => trans('admin/settings/general.email_formats.lastname_firstinitial'), - 'lastname.firstinitial' => trans('admin/settings/general.email_formats.lastname_dot_firstinitial_format'), - 'firstnamelastname' => trans('admin/settings/general.email_formats.firstnamelastname'), - 'firstnamelastinitial' => trans('admin/settings/general.email_formats.firstnamelastinitial'), - 'lastname.firstname' => trans('admin/settings/general.email_formats.lastnamefirstname'), - ]; - - $select = ''; - - return $select; -}); - Form::macro('username_format', function ($name = 'username_format', $selected = null, $class = null) { $formats = [ 'firstname.lastname' => trans('admin/settings/general.username_formats.firstname_lastname_format'), diff --git a/resources/views/blade/input/email-format-select.blade.php b/resources/views/blade/input/email-format-select.blade.php new file mode 100644 index 0000000000..fcb750d10c --- /dev/null +++ b/resources/views/blade/input/email-format-select.blade.php @@ -0,0 +1,27 @@ +@php + $formats = [ + 'firstname.lastname' => trans('admin/settings/general.email_formats.firstname_lastname_format'), + 'firstname' => trans('admin/settings/general.email_formats.first_name_format'), + 'lastname' => trans('admin/settings/general.email_formats.last_name_format'), + 'filastname' => trans('admin/settings/general.email_formats.filastname_format'), + 'lastnamefirstinitial' => trans('admin/settings/general.email_formats.lastnamefirstinitial_format'), + 'firstname_lastname' => trans('admin/settings/general.email_formats.firstname_lastname_underscore_format'), + 'firstinitial.lastname' => trans('admin/settings/general.email_formats.firstinitial_lastname'), + 'lastname_firstinitial' => trans('admin/settings/general.email_formats.lastname_firstinitial'), + 'lastname.firstinitial' => trans('admin/settings/general.email_formats.lastname_dot_firstinitial_format'), + 'firstnamelastname' => trans('admin/settings/general.email_formats.firstnamelastname'), + 'firstnamelastinitial' => trans('admin/settings/general.email_formats.firstnamelastinitial'), + 'lastname.firstname' => trans('admin/settings/general.email_formats.lastnamefirstname'), + ]; +@endphp + + + @foreach($formats as $format => $label) + + @endforeach + diff --git a/resources/views/partials/forms/edit/image-upload.blade.php b/resources/views/partials/forms/edit/image-upload.blade.php index 61c961c5bb..c6fdffdd5d 100644 --- a/resources/views/partials/forms/edit/image-upload.blade.php +++ b/resources/views/partials/forms/edit/image-upload.blade.php @@ -38,8 +38,8 @@ @elseif (isset($item) && (isset($item->model)) && ($item->model->image != ''))
-

- {{ trans('general.use_cloned_no_image_help') }} +

+ {{ trans('general.use_cloned_no_image_help') }}

diff --git a/resources/views/reports/licenses.blade.php b/resources/views/reports/licenses.blade.php index 3c0b11d5e6..8897bc0543 100644 --- a/resources/views/reports/licenses.blade.php +++ b/resources/views/reports/licenses.blade.php @@ -31,6 +31,7 @@ {{ trans('admin/licenses/form.seats') }} {{ trans('admin/licenses/form.remaining_seats') }} {{ trans('admin/licenses/form.expiration') }} + {{ trans('admin/licenses/form.termination_date') }} {{ trans('general.purchase_date') }} {{ trans('general.purchase_cost') }} {{ trans('general.depreciation') }} @@ -54,6 +55,12 @@ {{ $license->seats }} {{ $license->remaincount() }} {{ $license->expiration_date }} + {{ $license->termination_date }} + @if ($license->isTerminated()) + + + + @endif {{ $license->purchase_date }} {{ $snipeSettings->default_currency }}{{ Helper::formatCurrencyOutput($license->purchase_cost) }} diff --git a/resources/views/settings/general.blade.php b/resources/views/settings/general.blade.php index 43ce9aa1aa..ac3330be73 100644 --- a/resources/views/settings/general.blade.php +++ b/resources/views/settings/general.blade.php @@ -87,7 +87,12 @@
- {!! Form::email_format('email_format', old('email_format', $setting->email_format), 'select2') !!} + {!! $errors->first('email_format', '') !!}
diff --git a/resources/views/settings/localization.blade.php b/resources/views/settings/localization.blade.php index c358350cd6..63cfdd54ce 100644 --- a/resources/views/settings/localization.blade.php +++ b/resources/views/settings/localization.blade.php @@ -58,8 +58,12 @@
- {!! Form::name_display_format('name_display_format', old('name_display_format', $setting->name_display_format), 'select2') !!} - + {!! $errors->first('name_display_format', '') !!}
diff --git a/resources/views/users/edit.blade.php b/resources/views/users/edit.blade.php index 9cf4eb6a30..feeffce485 100755 --- a/resources/views/users/edit.blade.php +++ b/resources/views/users/edit.blade.php @@ -295,7 +295,14 @@
diff --git a/tests/Feature/Assets/Ui/EditAssetTest.php b/tests/Feature/Assets/Ui/EditAssetTest.php index 0fe29c6936..0e1432e893 100644 --- a/tests/Feature/Assets/Ui/EditAssetTest.php +++ b/tests/Feature/Assets/Ui/EditAssetTest.php @@ -75,7 +75,10 @@ class EditAssetTest extends TestCase $user = User::factory()->create(); $deployable_status = Statuslabel::factory()->rtd()->create(); $achived_status = Statuslabel::factory()->archived()->create(); - $asset = Asset::factory()->assignedToUser($user)->create(['status_id' => $deployable_status->id]); + $asset = Asset::factory()->assignedToUser($user)->create([ + 'status_id' => $deployable_status->id, + 'last_checkin' => null, + ]); $this->assertTrue($asset->assignedTo->is($user)); $currentTimestamp = now(); @@ -96,6 +99,7 @@ class EditAssetTest extends TestCase $this->assertNull($asset->assigned_to); $this->assertNull($asset->assigned_type); $this->assertEquals($achived_status->id, $asset->status_id); + $this->assertNotNull($asset->last_checkin); Event::assertDispatched(function (CheckoutableCheckedIn $event) use ($currentTimestamp) { return (int) Carbon::parse($event->action_date)->diffInSeconds($currentTimestamp, true) < 2;