diff --git a/app/Http/Controllers/ReportsController.php b/app/Http/Controllers/ReportsController.php index b65d5ba2bf..32374a1aff 100644 --- a/app/Http/Controllers/ReportsController.php +++ b/app/Http/Controllers/ReportsController.php @@ -628,15 +628,13 @@ class ReportsController extends Controller $row[] = ''; } } + if (e(Input::get('location')) == '1') { $show_loc = ''; - if (( $asset->assigned_to > 0 ) && ( $asset->assigneduser->location_id !='' )) { - $location = Location::find($asset->assigneduser->location_id); - if ($location) { - $show_loc .= '"' .e($location->name). '"'; - } else { - $show_loc .= 'User location '.$asset->assigneduser->location_id.' is invalid'; - } + + + if (($asset->assigned_to > 0) && ($asset->assigneduser) && ($asset->assigneduser->location)) { + $show_loc .= '"' .e($asset->assigneduser->location->name). '"'; } elseif ($asset->rtd_location_id!='') { $location = Location::find($asset->rtd_location_id); if ($location) { @@ -649,19 +647,19 @@ class ReportsController extends Controller $row[] = $show_loc; } + + if (e(Input::get('assigned_to')) == '1') { - if ($asset->assigned_to > 0) { - $user = User::find($asset->assigned_to); - $row[] = '"' .e($user->fullName()). '"'; + if ($asset->assigneduser) { + $row[] = '"' .e($asset->assigneduser->fullName()). '"'; } else { $row[] = ''; // Empty string if unassigned } } if (e(Input::get('username')) == '1') { - if ($asset->assigned_to > 0) { - $user = User::find($asset->assigned_to); - $row[] = '"' .e($user->username). '"'; + if ($asset->assigneduser) { + $row[] = '"' .e($asset->assigneduser->username). '"'; } else { $row[] = ''; // Empty string if unassigned } diff --git a/database/migrations/2016_10_19_145520_fix_order_number_in_components_to_string.php b/database/migrations/2016_10_19_145520_fix_order_number_in_components_to_string.php new file mode 100644 index 0000000000..c08131d922 --- /dev/null +++ b/database/migrations/2016_10_19_145520_fix_order_number_in_components_to_string.php @@ -0,0 +1,31 @@ +string('order_number')->nullable()->default(null)->change(); + }); + } + + /** + * Reverse the migrations. + * + * @return void + */ + public function down() + { + Schema::table('components', function ($table) { + $table->integer('order_number')->nullable()->default(null)->change(); + }); + } +}