App\Controllers\Admin\User\Traits\Show

Complexity

Total Complexity

2

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	1

2 Methods

Rating	Name	Duplication	Size	Complexity
A	row()	0	21	1
A	index()	0	11	1

<?php

namespace App\Controllers\Admin\User\Traits;


trait Show
{
  public function index()
  {
    $users = $this->load->model('User')->users();
    $usersformatted = $this->formatUsers($users);

It seems like formatUsers() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    $countries = $this->countries('all', 'name');

It seems like countries() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    $context = [
      'users' => $usersformatted,
      'countries' => $countries,
    ];
    return $this->view->render('admin/pages/users/users', $context);
  }

  public function row()
  {
    $id = userId();
    $model = $this->load->model('User');
    $user = $model->user($id);

    $user->new = $this->isUserNew($user->registration);

It seems like isUserNew() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    $user->registration = $this->changeFormatDate($user->registration);

It seems like changeFormatDate() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    $user->last_login = $this->changeFormatDate($user->last_login);
    $user->last_logout = $this->changeFormatDate($user->last_logout);
    $user->birthday = $this->changeFormatDate($user->birthday, ['Y-m-d', 'd M Y']);
    $user->country_icon = $this->countries($user->country);

    $countries = $this->countries('all', 'name');
    $countries_options = implode(',', $countries);

    $context = [
      'user' => $user,
      'countries_options' => $countries_options,
    ];
    return $this->view->render('admin/pages/users/user', $context);
  }
}