rhosocial / yii2-user

UserProfileSearchTrait.php

<?php

/**
 *  _   __ __ _____ _____ ___  ____  _____
 * | | / // // ___//_  _//   ||  __||_   _|
 * | |/ // /(__  )  / / / /| || |     | |
 * |___//_//____/  /_/ /_/ |_||_|     |_|
 * @link https://vistart.me/
 * @copyright Copyright (c) 2016 - 2017 vistart
 * @license https://vistart.me/license/
 */

namespace rhosocial\user;

use Yii;
use yii\base\Model;
use yii\data\ActiveDataProvider;

/**
 * @version 1.0
 * @author vistart <i@vistart.me>
 */
trait UserProfileSearchTrait
{
    public $createdFrom;
    protected $createdFromInUtc;
    public $createdTo;
    protected $createdToInUtc;
    /**
     * 
     * @return type
     */
    public function rules()
    {
        return [
            ['id', 'integer'],
            ['nickname', 'string'],
            [['createdFrom', 'createdTo'], 'datetime', 'format' => 'yyyy-mm-dd HH:mm:ss'],
            [['createdFrom', 'createdTo'], 'gmdate'],
        ];
    }

    public function gmdate($attribute, $params, $validator)

The parameter $params is not used and could be removed.

The parameter $validator is not used and could be removed.

    {
        if (isset($this->$attribute)) {
            $timestamp = strtotime($this->$attribute);
            $this->{$attribute . 'InUtc'} = gmdate('Y-m-d H:i:s', $timestamp);
        }
    }

    /**
     * 
     * @return type
     */
    public function scenarios()
    {
        // bypass scenarios() implementation in the parent class
        return Model::scenarios();
    }

    /**
     * 
     * @param type $params
     * @return ActiveDataProvider
     */
    public function search($params)
    {
        $query = static::find();
        $dataProvider = new ActiveDataProvider([
            'query' => $query,
            'pagination' => [
                'pageParam' => 'user-page',
                'defaultPageSize' => 20,
                'pageSizeParam' => 'user-per-page',
            ],
            'sort' => [
                'sortParam' => 'user-sort',
            ],
        ]);

        if (!($this->load($params) && $this->validate())) {

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

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

            return $dataProvider;
        }

        $query = $query->andFilterWhere([
            'LIKE', 'id', $this->id,

The property id does not exist. Did you maybe forget to declare it?

        ])->andFilterWhere([
            'LIKE', 'nickname', $this->nickname,

The property nickname does not exist. Did you maybe forget to declare it?

        ])->andFilterWhere([
            '>=', 'created_at', $this->createdFromInUtc,
        ])->andFilterWhere([
            '<=', 'created_at', $this->createdToInUtc,
        ]);
        $dataProvider->query = $query;
        return $dataProvider;
    }

    public function attributeLabels()
    {
        $attributeLabels = parent::attributeLabels();
        $attributeLabels['createdFrom'] = Yii::t('user', 'From');
        $attributeLabels['createdTo'] = Yii::t('user', 'To');
        return $attributeLabels;
    }
}