lunax/snipe_it
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity Actions

Merge pull request #15525 from akemidx/updated_ad_hoc_notes

Browse source
This commit is contained in:
snipe 2025-01-14 11:33:21 +00:00 • committed by GitHub
commit 1c8d94c953
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
12 changed files with 256 additions and 9 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

28
app/Events/NoteAdded.php Normal file
View file

@ -0,0 +1,28 @@
<?php
namespace App\Events;
use App\Models\User;
use Illuminate\Foundation\Events\Dispatchable;
use Illuminate\Queue\SerializesModels;
class NoteAdded
{
use Dispatchable, SerializesModels;
public $itemNoteAddedOn;
public $note;
public $noteAddedBy;
/**
* Create a new event instance.
*
* @return void
*/
public function __construct($itemNoteAddedOn, User $noteAddedBy, $note)
{
$this->itemNoteAddedOn = $itemNoteAddedOn;
$this->note = $note;
$this->noteAddedBy = $noteAddedBy;
}
}

4
app/Helpers/IconHelper.php
View file

@ -184,7 +184,9 @@ class IconHelper
return 'fa-regular fa-id-card';
case 'department' :
return 'fa-solid fa-building-user';
case 'note':
case 'notes':
return 'fas fa-sticky-note';
}
}
}

43
app/Http/Controllers/Api/NotesController.php Normal file
View file

@ -0,0 +1,43 @@
<?php
namespace App\Http\Controllers\Api;
use App\Events\NoteAdded;
use App\Helpers\Helper;
use App\Http\Controllers\Controller;
use App\Models\Asset;
use Illuminate\Http\Request;
use Illuminate\Support\Facades\Auth;
use Illuminate\Validation\Rule;
class NotesController extends Controller
{
public function store(Request $request)
{
$validated = $request->validate([
'note' => 'required|string|max:500',
'type' => [
'required',
Rule::in(['asset']),
],
]);
// This can be made dynamic by using $request->input('type') to determine which model type to add the note to.
// For now, we are only placing this on Assets
$item = Asset::findOrFail($request->input("id"));
$this->authorize('update', $item);
event(new NoteAdded($item, Auth::user(), $validated['note']));
return response()->json(Helper::formatStandardApiResponse('success'));
}
public function update(Request $request)
{
}
public function destroy(Request $request)
{
}
}

3
app/Http/Controllers/ModalController.php
View file

@ -32,7 +32,8 @@ class ModalController extends Controller
'statuslabel',
'supplier',
'upload-file',
'user',
'user',
'add-note',
];

19
app/Listeners/LogListener.php
View file

@ -17,6 +17,7 @@ use App\Events\ItemAccepted;
use App\Events\ItemDeclined;
use App\Events\LicenseCheckedIn;
use App\Events\LicenseCheckedOut;
use App\Events\NoteAdded;
use App\Models\Actionlog;
use App\Models\User;
use App\Models\LicenseSeat;
@ -128,6 +129,23 @@ class LogListener
}
/**
* Note is added to action log
*
*/
public function onNoteAdded(NoteAdded $event)
{
$logaction = new Actionlog();
$logaction->item_id = $event->itemNoteAddedOn->id;
$logaction->item_type = get_class($event->itemNoteAddedOn);
$logaction->note = $event->note; //this is the received alphanumeric text from the box
$logaction->created_by = $event->noteAddedBy->id;
$logaction->action_type = 'note_added';
$logaction->save();
}
/**
* Register the listeners for the subscriber.
*
@ -141,6 +159,7 @@ class LogListener
'CheckoutAccepted',
'CheckoutDeclined',
'UserMerged',
'NoteAdded',
];
foreach ($list as $event) {

15
resources/assets/js/snipeit_modals.js
View file

@ -28,7 +28,7 @@ $(function () {
var baseUrl = $('meta[name="baseUrl"]').attr('content');
//handle modal-add-interstitial calls
var model, select, refreshSelector;
var model, select, refreshSelector, hasnopayload;
if($('#createModal').length == 0) {
$('body').append('<div class="modal fade" id="createModal"></div><!-- /.modal -->');
@ -40,6 +40,7 @@ $(function () {
select = link.data("select");
refreshSelector = link.data("refresh");
hasnopayload = link.data("hasnopayload");
$('#createModal').load(link.attr('href'),function () {
@ -122,11 +123,13 @@ $(function () {
$('#modal_error_msg').html(error_message).show();
return false;
}
var id = result.payload.id;
var name = result.payload.name || (result.payload.first_name + " " + result.payload.last_name);
if(!id || !name) {
console.error("Could not find resulting name or ID from modal-create. Name: "+name+", id: "+id);
return false;
if(!hasnopayload) {
var id = result.payload.id;
var name = result.payload.name || (result.payload.first_name + " " + result.payload.last_name);
if (!id || !name) {
console.error("Could not find resulting name or ID from modal-create. Name: " + name + ", id: " + id);
return false;
}
}
$('#createModal').modal('hide');
$('#createModal').html("");

6
resources/lang/en-US/general.php
View file

@ -216,6 +216,12 @@ return [
'no_results' => 'No Results.',
'no' => 'No',
'notes' => 'Notes',
'note_added' => 'Note Added',
'add_note' => 'Add Note',
'note_edited' => 'Note Edited',
'edit_note' => 'Edit Note',
'note_deleted' => 'Note Deleted',
'delete_note' => 'Delete Note',
'order_number' => 'Order Number',
'only_deleted' => 'Only Deleted Assets',
'page_menu' => 'Showing _MENU_ items',

10
resources/views/hardware/view.blade.php
View file

@ -253,6 +253,15 @@
{{ Form::close() }}
</div>
<!-- Add notes -->
@can('update', \App\Models\Asset::class)
<div class="col-md-12 hidden-print" style="padding-top: 5px;">
<a href='{{ route('modal.show', 'add-note') }}?type=asset&id={{$asset->id}}' style="width: 100%" data-toggle="modal" data-target="#createModal" data-select='add-note_select_id' data-refresh="assetHistory" data-hasnopayload="true" class="btn btn-sm btn-primary btn-block btn-social hidden-print">
<x-icon type="note" />
{{ trans('general.add_note') }}</a>
</div>
@endcan
@can('delete', $asset)
<div class="col-md-12 hidden-print" style="padding-top: 30px; padding-bottom: 30px;">
@ -1394,7 +1403,6 @@
@include ('modals.upload-file', ['item_type' => 'asset', 'item_id' => $asset->id])
@endcan
@stop
@section('moar_scripts')
<script>

28
resources/views/modals/add-note.blade.php Normal file
View file

@ -0,0 +1,28 @@
{{-- See snipeit_modals.js for what powers this --}}
<div class="modal-dialog">
<div class="modal-content">
<div class="modal-header">
<button type="button" class="close" data-dismiss="modal" aria-label="Close"><span aria-hidden="true">&times;</span></button>
<h2 class="modal-title">{{ trans('general.add_note') }}</h2>
</div>
<div class="modal-body">
<form action="{{ route('api.notes.store') }}" onsubmit="return false">
<input type="hidden" name="type" value="{{request("type")}}"/>
<input type="hidden" name="id" value="{{request("id")}}"/>
<div class="alert alert-danger" id="modal_error_msg" style="display:none"></div>
<div class="row">
<div class="col-md-12">
<textarea class="form-control" id="note" name="note">{{ old('note') }}</textarea>
{!! $errors->first('note', '<span class="alert-msg" aria-hidden="true"><i class="fas fa-times" aria-hidden="true"></i> :message</span>') !!}
</div>
</div>
</form>
</div>
<div class="modal-footer">
<button type="button" class="btn btn-default pull-left" data-dismiss="modal">{{ trans('button.cancel') }}</button>
<button type="button" class="btn btn-primary pull-right" id="modal-save">{{ trans('general.save') }}</button>
</div>
</div><!-- /.modal-content -->
</div><!-- /.modal-dialog -->

16
routes/api.php
View file

@ -1303,6 +1303,22 @@ Route::group(['prefix' => 'v1', 'middleware' => ['api', 'throttle:api']], functi
}); // end reports api routes
/**
* Notes API routes
*/
Route::group(['prefix' => 'notes'], function () {
Route::post(
'/',
[ Api\NotesController::class,
'store'
]
)->name('api.notes.store');
}); // end notes api routes
/**
* Version API routes
*/

76
tests/Feature/Notes/AssetNotesTest.php Normal file
View file

@ -0,0 +1,76 @@
<?php
namespace Tests\Feature\Notes;
use App\Events\NoteAdded;
use App\Models\Asset;
use App\Models\User;
use Illuminate\Support\Facades\Event;
use Tests\TestCase;
class AssetNotesTest extends TestCase
{
public function testRequiresPermission()
{
$asset = Asset::factory()->create();
$this->actingAsForApi(User::factory()->create())
->postJson(route('api.notes.store'), [
'note' => 'New Note!',
'type' => 'asset',
'id' => $asset->id,
])
->assertForbidden();
}
public function testValidation()
{
$asset = Asset::factory()->create();
$this->actingAsForApi(User::factory()->editAssets()->create())
->postJson(route('api.notes.store'), [
// 'note' => '',
'type' => 'a_type_not_asset',
'id' => $asset->id,
])
->assertOk()
->assertStatusMessageIs('error')
->assertJsonValidationErrors(['note', 'type'], 'messages');
}
public function testRequiresExistingAsset()
{
$this->actingAsForApi(User::factory()->editAssets()->create())
->postJson(route('api.notes.store'), [
'note' => 'New Note!',
'type' => 'asset',
'id' => 999_999,
])
->assertStatusMessageIs('error')
->assertMessagesAre('Asset not found');
}
public function testCanAddNoteToAsset()
{
Event::fake([NoteAdded::class]);
$asset = Asset::factory()->create();
$user = User::factory()->editAssets()->create();
$this->actingAsForApi($user)
->postJson(route('api.notes.store'), [
'note' => 'New Note!',
'type' => 'asset',
'id' => $asset->id,
])
->assertOk()
->assertStatusMessageIs('success');
Event::assertDispatchedTimes(NoteAdded::class, 1);
Event::assertDispatched(NoteAdded::class, function (NoteAdded $event) use ($asset, $user) {
return $event->itemNoteAddedOn->is($asset)
&& $event->note === 'New Note!'
&& $event->noteAddedBy->is($user);
});
}
}

17
tests/Unit/Listeners/LogListenerTest.php
View file

@ -3,6 +3,7 @@
namespace Tests\Unit\Listeners;
use App\Events\CheckoutableCheckedOut;
use App\Events\NoteAdded;
use App\Listeners\LogListener;
use App\Models\Asset;
use App\Models\User;
@ -36,4 +37,20 @@ class LogListenerTest extends TestCase
'note' => 'A simple note...',
]);
}
public function testLogsEntryOnAssetNoteCreation()
{
$asset = Asset::factory()->create();
$noteAddedBy = User::factory()->create();
event(new NoteAdded($asset, $noteAddedBy, 'My Cool Note!'));
$this->assertDatabaseHas('action_logs', [
'action_type' => 'note_added',
'created_by' => $noteAddedBy->id,
'item_id' => $asset->id,
'item_type' => Asset::class,
'note' => 'My Cool Note!',
]);
}
}