License importer, small tweaks to asset importer

Author: snipe

app/commands/ImportCommand.php renamed to app/commands/AssetImportCommand.php

@@ -5,21 +5,21 @@
 use Symfony\Component\Console\Input\InputArgument;
 use League\Csv\Reader;
 
-class ImportCommand extends Command {
+class AssetImportCommand extends Command {
 
 	/**
 	 * The console command name.
 	 *
 	 * @var string
 	 */
-	protected $name = 'import:csv';
+	protected $name = 'asset-import:csv';
 
 	/**
 	 * The console command description.
 	 *
 	 * @var string
 	 */
-	protected $description = 'Import from CSV';
+	protected $description = 'Import Assets from CSV';
 
 	/**
 	 * Create a new command instance.
@@ -42,9 +42,9 @@ public function fire()
 
 
 		if (!$this->option('testrun')=='true') {
-			$this->comment('======= Importing '.$filename.' =========');
+			$this->comment('======= Importing Assets from '.$filename.' =========');
 		} else {
-			$this->comment('====== TEST ONLY Import for '.$filename.' ====');
+			$this->comment('====== TEST ONLY Asset Import for '.$filename.' ====');
 			$this->comment('============== NO DATA WILL BE WRITTEN ==============');
 		}
 
@@ -63,66 +63,77 @@ public function fire()
 
 			// Let's just map some of these entries to more user friendly words
 
+			// User's name
 			if (array_key_exists('0',$row)) {
 				$user_name = trim($row[0]);
 			} else {
 				$user_name = '';
 			}
 
+			// User's email
 			if (array_key_exists('1',$row)) {
 				$user_email = trim($row[1]);
 			} else {
 				$user_email = '';
 			}
 
+			// Asset Category
 			if (array_key_exists('2',$row)) {
 				$user_asset_category = trim($row[2]);
 			} else {
 				$user_asset_category = '';
 			}
 
+			// Asset Name
 			if (array_key_exists('3',$row)) {
 				$user_asset_name = trim($row[3]);
 			} else {
 				$user_asset_name = '';
 			}
 
+			// Asset Manufacturer
 			if (array_key_exists('4',$row)) {
 				$user_asset_mfgr = trim($row[4]);
 			} else {
 				$user_asset_mfgr = '';
 			}
 
+			// Asset model number
 			if (array_key_exists('5',$row)) {
 				$user_asset_modelno = trim($row[5]);
 			} else {
 				$user_asset_modelno = '';
 			}
 
+			// Asset serial number
 			if (array_key_exists('6',$row)) {
 				$user_asset_serial = trim($row[6]);
 			} else {
 				$user_asset_serial = '';
 			}
 
+			// Asset tag
 			if (array_key_exists('7',$row)) {
 				$user_asset_tag = trim($row[7]);
 			} else {
 				$user_asset_tag = '';
 			}
 
+			// Asset location
 			if (array_key_exists('8',$row)) {
 				$user_asset_location = trim($row[8]);
 			} else {
 				$user_asset_location = '';
 			}
 
+			// Asset notes
 			if (array_key_exists('9',$row)) {
 				$user_asset_notes = trim($row[9]);
 			} else {
 				$user_asset_notes = '';
 			}
 
+			// Asset purchase date
 			if (array_key_exists('10',$row)) {
 				if ($row[10]!='') {
 					$user_asset_purchase_date = date("Y-m-d 00:00:01", strtotime($row[10]));
@@ -137,8 +148,8 @@ public function fire()
 			if (is_numeric($user_name)) {
 				$this->comment('User '.$user_name.' is not a name - assume this user already exists');
 				$user_username = '';
-			// No name was given
 
+			// No name was given
 			} elseif ($user_name=='') {
 				$this->comment('No user data provided - skipping user creation, just adding asset');
 				$first_name = '';
@@ -221,7 +232,7 @@ public function fire()
 							'admin' => 0,
 							'user'  => 1,
 						),
-						'notes'         => 'Imported user'
+						'notes'         => 'User imported through asset importer'
 					));
 
 					// Find the group using the group id