DRCIT/snipe-it
1
0
Fork 0
mirror of https://github.com/snipe/snipe-it.git synced 2025-02-21 03:15:45 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Fixed #6113 - use $asset->fill vs filled() to allow blanking values via API

Browse source
This commit is contained in:
snipe 2019-02-12 22:15:32 -08:00
parent 3f7d2aebc7
commit 8e358faebc
2 changed files with 5 additions and 32 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

35
app/Http/Controllers/Api/AssetsController.php
View file

@ -476,39 +476,12 @@ class AssetsController extends Controller
{ {
$this->authorize('update', Asset::class); $this->authorize('update', Asset::class);
$asset->fill($request->all());
if ($asset = Asset::find($id)) { if ($asset = Asset::find($id)) {
($request->filled('model_id')) ? ($request->filled('model_id')) ?
$asset->model()->associate(AssetModel::find($request->get('model_id'))) : ''; $asset->model()->associate(AssetModel::find($request->get('model_id'))) : null;
($request->filled('name')) ?
$asset->name = $request->get('name') : '';
($request->filled('serial')) ?
$asset->serial = $request->get('serial') : '';
($request->filled('model_id')) ?
$asset->model_id = $request->get('model_id') : '';
($request->filled('order_number')) ?
$asset->order_number = $request->get('order_number') : '';
($request->filled('notes')) ?
$asset->notes = $request->get('notes') : '';
($request->filled('asset_tag')) ?
$asset->asset_tag = $request->get('asset_tag') : '';
($request->filled('archived')) ?
$asset->archived = $request->get('archived') : '';
($request->filled('status_id')) ?
$asset->status_id = $request->get('status_id') : '';
($request->filled('warranty_months')) ?
$asset->warranty_months = $request->get('warranty_months') : '';
($request->filled('purchase_cost')) ?
$asset->purchase_cost = Helper::ParseFloat($request->get('purchase_cost')) : '';
($request->filled('purchase_date')) ?
$asset->purchase_date = $request->get('purchase_date') : '';
($request->filled('assigned_to')) ?
$asset->assigned_to = $request->get('assigned_to') : '';
($request->filled('supplier_id')) ?
$asset->supplier_id = $request->get('supplier_id') : '';
($request->filled('requestable')) ?
$asset->requestable = $request->get('requestable') : '';
($request->filled('rtd_location_id')) ?
$asset->rtd_location_id = $request->get('rtd_location_id') : '';
($request->filled('rtd_location_id')) ? ($request->filled('rtd_location_id')) ?
$asset->location_id = $request->get('rtd_location_id') : ''; $asset->location_id = $request->get('rtd_location_id') : '';
($request->filled('company_id')) ? ($request->filled('company_id')) ?

2
app/Http/Kernel.php
View file

@ -25,7 +25,7 @@ class Kernel extends HttpKernel
\Fideloper\Proxy\TrustProxies::class, \Fideloper\Proxy\TrustProxies::class,
\App\Http\Middleware\CheckForSetup::class, \App\Http\Middleware\CheckForSetup::class,
\App\Http\Middleware\CheckForDebug::class, \App\Http\Middleware\CheckForDebug::class,
// \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class, \Illuminate\Foundation\Http\Middleware\ConvertEmptyStringsToNull::class,
]; ];
/** /**