Merge pull request #15653 from snipe/15651_admin_user_on_maintenances

Fixed #15651 - admin user now displaying on maintenances page

Author: snipe

app/Http/Transformers/AssetMaintenancesTransformer.php

@@ -66,7 +66,7 @@ public function transformAssetMaintenance(AssetMaintenance $assetmaintenance)
             'completion_date'     => Helper::getFormattedDateObject($assetmaintenance->completion_date, 'date'),
             'user_id'    => ($assetmaintenance->adminuser) ? [
                 'id' => $assetmaintenance->adminuser->id,
-                'name'=> e($assetmaintenance->admin->getFullNameAttribute())
+                'name'=> e($assetmaintenance->adminuser->present()->fullName())
             ] : null, // legacy to not change the shape of the API
             'created_by' => ($assetmaintenance->adminuser) ? [
                 'id' => (int) $assetmaintenance->adminuser->id,

app/Models/AssetMaintenance.php

@@ -176,7 +176,7 @@ public function asset()
      */
     public function adminuser()
     {
-        return $this->belongsTo(\App\Models\User::class, 'user_id')
+        return $this->belongsTo(\App\Models\User::class, 'created_by')
             ->withTrashed();
     }
 

app/Presenters/AssetMaintenancesPresenter.php

@@ -116,12 +116,6 @@ public static function dataTableLayout()
                 'sortable' => true,
                 'title' => trans('admin/asset_maintenances/form.cost'),
                 'class' => 'text-right',
-            ], [
-                'field' => 'user_id',
-                'searchable' => true,
-                'sortable' => true,
-                'title' => trans('general.admin'),
-                'formatter' => 'usersLinkObjFormatter',
             ], [
                 'field' => 'created_by',
                 'searchable' => false,