DRCIT/snipe-it
1
0
Fork 0
mirror of https://github.com/snipe/snipe-it.git synced 2024-12-23 12:44:12 -08:00
Code Issues Actions 1 Packages Projects Releases Wiki Activity

Pass user id to importer. This shows items imported through web interface as created by the appropriate user. Also save warranty_months to item, not just read it from csv. Fixes #2175

Browse source
This commit is contained in:
Daniel Meltzer 2016-06-19 23:06:54 -04:00
parent 6165d804f9
commit 4715cc6447
2 changed files with 16 additions and 11 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

21
app/Console/Commands/ObjectImportCommand.php
View file

@ -321,7 +321,7 @@ class ObjectImportCommand extends Command {
$asset_model->manufacturer_id = $manufacturer->id;
$asset_model->modelno = $asset_modelno;
$asset_model->category_id = $category->id;
$asset_model->user_id = 1;
$asset_model->user_id = $this->option('user_id');
if(!$this->option('testrun')) {
@ -367,7 +367,7 @@ class ObjectImportCommand extends Command {
$category->name = $asset_category;
$category->category_type = $item_type;
$category->user_id = 1;
$category->user_id = $this->option('user_id');
if(!$this->option('testrun')) {
@ -491,7 +491,7 @@ class ObjectImportCommand extends Command {
$manufacturer = new Manufacturer();
$manufacturer->name = $asset_mfgr;
$manufacturer->user_id = 1;
$manufacturer->user_id = $this->option('user_id');
if (!$this->option('testrun')) {
if ($manufacturer->save()) {
@ -538,7 +538,7 @@ class ObjectImportCommand extends Command {
$location->city = '';
$location->state = '';
$location->country = '';
$location->user_id = 1;
$location->user_id = $this->option('user_id');
if (!$this->option('testrun')) {
if ($location->save()) {
@ -584,7 +584,7 @@ class ObjectImportCommand extends Command {
$supplier = new Supplier();
$supplier->name = $supplier_name;
$supplier->user_id = 1;
$supplier->user_id = $this->option('user_id');
if(!$this->option('testrun')) {
if ($supplier->save()) {
@ -721,6 +721,7 @@ class ObjectImportCommand extends Command {
$this->log('Serial No: '.$asset_serial);
$this->log('Asset Tag: '.$asset_tag);
$this->log('Notes: '.$item["notes"]);
$this->log('Warranty Months: ' . $asset_warranty_months);
foreach ($this->assets as $tempasset) {
if (strcasecmp($tempasset->asset_tag, $asset_tag ) == 0 ) {
@ -761,6 +762,7 @@ class ObjectImportCommand extends Command {
}
$asset->serial = $asset_serial;
$asset->asset_tag = $asset_tag;
$asset->warranty_months = $asset_warranty_months;
if($asset_model)
$asset->model_id = $asset_model->id;
@ -768,7 +770,7 @@ class ObjectImportCommand extends Command {
$asset->assigned_to = $item["user"]->id;
if($item["location"])
$asset->rtd_location_id = $item["location"]->id;
$asset->user_id = 1;
$asset->user_id = $this->option('user_id');
$this->log("status_id: " . $status_id);
$asset->status_id = $status_id;
if($item["company"])
@ -827,7 +829,7 @@ class ObjectImportCommand extends Command {
}
if($item["location"])
$accessory->location_id = $item["location"]->id;
$accessory->user_id = 1;
$accessory->user_id = $this->option('user_id');
if($item["company"])
$accessory->company_id = $item["company"]->id;
$accessory->order_number = $item["order_number"];
@ -893,7 +895,7 @@ class ObjectImportCommand extends Command {
$consumable->purchase_cost = 0.00;
}
$consumable->location_id = $item["location"]->id;
$consumable->user_id = 1; // TODO: What user_id should we use for imports?
$consumable->user_id = $this->option('user_id');
$consumable->company_id = $item["company"]->id;
$consumable->order_number = $item["order_number"];
$consumable->category_id = $item["category"]->id;
@ -950,7 +952,8 @@ class ObjectImportCommand extends Command {
array('testrun', null, InputOption::VALUE_NONE, 'If set, will parse and output data without adding to database', null),
array('logfile', null, InputOption::VALUE_REQUIRED, 'The path to log output to. storage/logs/importer.log by default', storage_path('logs/importer.log') ),
array('item-type', null, InputOption::VALUE_REQUIRED, 'Item Type To import. Valid Options are Asset, Consumable, Or Accessory', 'Asset'),
array('web-importer', null, InputOption::VALUE_NONE, 'Internal: packages output for use with the web importer')
array('web-importer', null, InputOption::VALUE_NONE, 'Internal: packages output for use with the web importer'),
array('user_id', null, InputOption::VALUE_REQUIRED, 'ID of user creating items', 1)
);
}

6
app/Http/Controllers/AssetsController.php
View file

@ -878,15 +878,17 @@ class AssetsController extends Controller
['filename'=> config('app.private_uploads').'/imports/assets/'.$filename,
'--email_format'=>'firstname.lastname',
'--username_format'=>'firstname.lastname',
'--web-importer' => true
'--web-importer' => true,
'--user_id' => Auth::user()->id
]);
$display_output = Artisan::output();
$file = config('app.private_uploads').'/imports/assets/'.str_replace('.csv', '', $filename).'-output-'.date("Y-m-d-his").'.txt';
file_put_contents($file, $display_output);
if( $return === 0) //Success
return redirect()->to('hardware')->with('success', trans('admin/hardware/message.import.success'));
else if( $return === 1) // Failure
else if( $return === 1) { // Failure
return redirect()->back()->with('import_errors', json_decode($display_output))->with('error', trans('admin/hardware/message.import.error'));
}
dd("Shouldn't be here");
}