Core\Models\User\UserRepository

Complexity

Total Complexity

9

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	find()	0	4	1
A	findById()	0	8	2
A	findMany()	0	4	1
A	delete()	0	4	1
A	save()	0	14	3
A	build()	0	3	1

<?php

namespace Core\Models\User;

use Propel\Runtime\ActiveRecord\ActiveRecordInterface;

use Core\Models\User\User;
use Core\Models\User\UserQuery;

/**
 * Description of UserRepo
 *
 * @author Dmitriy
 */
class UserRepository implements \Frameworkless\CrudInterface
{   

The opening class brace should be on a newline by itself.

    
    public function find(array $conditions = [])
    {
        return UserQuery::create()->findOneByArray($conditions);  
    }

    /**
     * Поиск по id
     * @param type $id
     * @return \Models\User\User $User
     * @throws \InvalidArgumentException
     */
    public function findById($id)
    {
        if (!$User = UserQuery::create()->findPk($id)) {
            throw new \InvalidArgumentException(sprintf('User with ID %d does not exist',$id));
        }

        return $User;
    }

    public function findMany(array $conditions = [])
    {
        return UserQuery::create()->findByArray($conditions);    
    }

    public function delete(ActiveRecordInterface $Model)
    {
        
    }

    /**
     * Сохраняем
     * @param ActiveRecordInterface $Model
     * @return boolean
     * @throws \DomainException
     */
    public function save(ActiveRecordInterface $Model)
    {
        if (!$Model->validate()) {

It seems like you code against a concrete implementation and not the interface Propel\Runtime\ActiveRecord\ActiveRecordInterface as the method validate() does only exist in the following implementations of said interface: Core\Models\User\Base\User, Core\Models\User\User.

            
            throw new \Frameworkless\Exceptions\ValidateException("User not validate", $Model->getValidationFailures());

It seems like you code against a concrete implementation and not the interface Propel\Runtime\ActiveRecord\ActiveRecordInterface as the method getValidationFailures() does only exist in the following implementations of said interface: Core\Models\User\Base\User, Core\Models\User\User.

The call to ValidateException::__construct() has too many arguments starting with 'User not validate'.

        }        
        
        
        if (!$Model->save()) {
            throw new \DomainException(sprintf('User not save',$id));
        }

        return true;        
    }
    
    /**
     * 
     * @return \Models\User\Models\User\User $User
     */
    public function build(){
        return new User;
    }

}