Application\Media\Table

Complexity

Total Complexity

3

Size/Duplication

Total Lines	45
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	4

Test Coverage

Coverage

100%

Importance

Changes	1
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	getImagesByUserId()	0	7	1
A	getImages()	0	9	2

<?php
/**
 * @copyright Bluz PHP Team
 * @link https://github.com/bluzphp/skeleton
 */

/**
 * @namespace
 */
namespace Application\Media;

use Application\Exception;
use Bluz\Proxy\Auth;

/**
 * Table
 *
 * @package  Application\Media
 *
 * @method   static Row findRow($primaryKey)
 * @method   static Row findRowWhere($whereList)
 */
class Table extends \Bluz\Db\Table
{
    /**
     * Table
     *
     * @var string
     */
    protected $table = 'media';

    /**
     * Primary key(s)
     * @var array
     */
    protected $primary = array('id');

    /**
     * Get images of current user
     *
     * @return Row[]
     * @throws Exception
     */
    public function getImages()
    {
        /** @var Row $user */
        if (!$user = Auth::getIdentity()) {
            throw new Exception('User not found');
        }

        return $this->getImagesByUserId($user->id);

Accessing id on the interface Bluz\Auth\EntityInterface suggest that you code against a concrete implementation. How about adding an instanceof check?

    }

    /**
     * Get images by owner
     *
     * @param  integer $id
     * @return Row[]
     */
    public function getImagesByUserId($id)
    {
        return self::select()
            ->where('type LIKE (?)', 'image/%')
            ->andWhere('userId = ?', $id)
            ->execute();
    }
}