Office\user

Complexity

Total Complexity

3

Size/Duplication

Total Lines	36
Duplicated Lines	0 %

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getOffices()	0	7	2
A	__construct()	0	5	1

<?php

namespace Office;

class user extends \User\user
{
  /**
   * PDO instance.
   *
   * @var \DB\pdo
   */
  private $pdo;
  /**
   * User meta instance.
   *
   * @var \User\meta
   */
  public $usermeta;

  public function __construct(\DB\pdo $pdo)
  {
    $this->pdo = $pdo;
    $this->pdo_instance($pdo);
    $this->usermeta = new \User\meta($pdo);
  }

  /**
   * Get office list of current logged in user.
   *
   * @requires `\User\user::is_login` check login before doing this
   *
   * @return array
   */
  public function getOffices()
  {
    $meta = $this->usermeta->get($this->userdata('id'), 'office');

Are you sure the assignment to $meta is correct as $this->usermeta->get($this->userdata('id'), 'office') targeting User\meta::get() seems to always return null.

    if (isset($meta['value'])) {
      return $meta['value'];
    }
    return [];
  }
}