app\components\stats\AccountDaily::initDiff()

Complexity

Conditions	4
Paths	2

Size

Total Lines

35

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function initDiff($olderDate, $newerDate = null)
    {
        $olderDate = (new Carbon($olderDate))->startOfDay()->toDateTimeString();
        $newerDate = (new Carbon($newerDate))->endOfDay()->toDateTimeString();

        if (empty(static::$cache)) {

Since $cache is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $cache to at least protected.

            //group by DATE Y-m-d
            $ids = AccountStats::find()
                ->select(new Expression('MAX(id) as id'))
                ->andWhere(['account_id' => $this->model->id])
                ->andWhere(['>=', 'created_at', $olderDate])
                ->andWhere(['<=', 'created_at', $newerDate])
                ->groupBy(new Expression('DATE(created_at)'))
                ->column();

            $stats = AccountStats::find()
                ->select([
                    'account_stats.*',
                    new Expression('DATE(created_at) as created_at'),
                ])
                ->andWhere(['account_id' => $this->model->id])
                ->andWhere(['id' => $ids])
                ->orderBy('id ASC')
                ->asArray()
                ->all();

            foreach ($stats as $stat) {
                foreach ($this->statsAttributes as $statsAttribute) {
                    $value = ArrayHelper::getValue($stat, $statsAttribute, 0);
                    static::$cache[$stat['created_at']][$statsAttribute] = $value;
                }
            }
        }

        return $this;
    }
}