app\commands\AdminController

Complexity

Total Complexity

22

Size/Duplication

Total Lines	142
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	18

Importance

Changes	2
Bugs	0	Features	1

6 Methods

Rating	Name	Duplication	Size	Complexity
A	actions()	0	8	1
B	actionThumbnails()	0	28	5
B	actionClearTests()	0	24	1
A	actionTestMail()	0	18	2
B	createStructure()	0	18	9
A	actionCreateTheme()	0	21	4

<?php

namespace app\commands;

use Imagine\Image\ManipulatorInterface;
use app\backend\actions\FlushCacheConsoleAction;
use Yii;
use yii\console\Controller;
use app;
use app\modules\image\models\Image;

/**
 * Admin commands
 * @package app\commands
 */
class AdminController extends Controller
{

    public function actions()
    {
        return [
            'flush-cache' => [
                'class' => FlushCacheConsoleAction::className(),
            ],
        ];
    }

    /**
     * Generate thumbnails for Image model
     * @param bool $updateThumbnailSrc
     * @param bool $deleteIfNotExists
     * @throws \Exception
     */
    public function actionThumbnails($updateThumbnailSrc = false, $deleteIfNotExists = false)
    {
        $images = Image::find()->all();
        /** @var $images Image[] */
        foreach ($images as $image) {
            $dir = '@webroot' . mb_substr($image->filename, 0, mb_strrpos($image->filename, '/')) . '/';
            $filename = \Yii::getAlias($dir . $image->filename);
            if (!file_exists($filename)) {
                echo "File not found: " . $filename . "\n";
                if ($deleteIfNotExists) {
                    $image->delete();
                }
                continue;
            }
            $img = \yii\imagine\Image::thumbnail(
                $filename,

It seems like $filename defined by \Yii::getAlias($dir . $image->filename) on line 40 can also be of type boolean; however, yii\imagine\BaseImage::thumbnail() does only seem to accept string, maybe add an additional type check?

                80,
                80,
                ManipulatorInterface::THUMBNAIL_INSET
            );
            $img->save(\Yii::getAlias($dir . 'small-' . $image->filename));

It seems like \Yii::getAlias($dir . 'small-' . $image->filename) targeting yii\BaseYii::getAlias() can also be of type boolean; however, Imagine\Image\ManipulatorInterface::save() does only seem to accept string, maybe add an additional type check?

            if ($updateThumbnailSrc) {
                $image->thumbnail_src = mb_substr($image->filename, 0, mb_strrpos($image->filename, '/'))

The property thumbnail_src does not exist on object<app\modules\image\models\Image>. Since you implemented __set, maybe consider adding a @property annotation.

                    . '/small-' . $image->filename;
                $image->save(true, ['thumbnail_src']);
            }
        }
    }

    /**
     * Clear test review, orders and other information
     * @throws \yii\db\Exception
     */
    public function actionClearTests()
    {
        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\models\ErrorLog::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\models\ErrorUrl::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\backend\models\Notification::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\modules\shop\models\Order::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\modules\shop\models\OrderItem::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\modules\shop\models\OrderTransaction::tableName())->execute();

This line exceeds maximum limit of 120 characters; contains 123 characters

        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\backend\models\OrderChat::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE {{%order_category}}')->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE {{%order_eav}}')->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE {{%order_property}}')->execute();
//        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\reviews\models\Review::tableName())->execute(); @todo need to implement correct reviews deleting

Comments for TODO tasks are often forgotten in the code; it might be better to use a dedicated issue tracker.

37% of this comment could be valid code. Did you maybe forget this after debugging?

This line exceeds maximum limit of 120 characters; contains 159 characters

        Yii::$app->db->createCommand('TRUNCATE TABLE ' . app\models\Submission::tableName())->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE {{%submission_category}}')->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE {{%submission_eav}}')->execute();
        Yii::$app->db->createCommand('TRUNCATE TABLE {{%submission_property}}')->execute();
        Yii::$app->db->createCommand('DELETE FROM ' . app\models\ObjectPropertyGroup::tableName()
            . ' WHERE `object_id` IN (SELECT `id` FROM ' . app\models\Object::tableName()
            . ' WHERE `object_class` IN (\'app\\models\\Submission\', \'app\\models\\Order\'))')->execute();
        Yii::$app->db->createCommand('DELETE FROM ' . app\models\ObjectStaticValues::tableName()
            . ' WHERE `object_id` IN (SELECT `id` FROM ' . app\models\Object::tableName()
            . ' WHERE `object_class` IN (\'app\\models\\Submission\', \'app\\models\\Order\'))')->execute();
    }

    public function actionTestMail($email)
    {

        $mailComponent = Yii::$app->mail;

        $status = $mailComponent->compose('test')
            ->setFrom(Yii::$app->getModule('core')->emailConfig['mailFrom'])
            ->setTo($email)
            ->setSubject(
                Yii::t(
                    'app', 'subject test mail on {site}',
                    [
                        'site' => 'Site'
                    ]
                )
            )->send();
        echo $status ? "OK\n" : "NOT OK\n";
    }


    protected function createStructure($path, $children, $themeName)
    {
        foreach ($children as $child) {
            $newPath = $path . DIRECTORY_SEPARATOR . $child['name'];
            if (isset($child['type']) && $child['type'] == 'dir') {
                mkdir($newPath);
                chmod($newPath, isset($child['writable']) && $child['writable'] ? 0777 : 0775);
                if (isset($child['children']) && !empty($child['children'])) {
                    $this->createStructure($newPath, $child['children'], $themeName);
                } else {
                    file_put_contents($newPath . DIRECTORY_SEPARATOR . '.keep', '');
                }
            } else {
                file_put_contents($newPath,
                    isset($child['content']) ? str_replace('{%theme-name%}', $themeName, $child['content']) : '');
            }
        }
    }

    /**
     * Create theme structure
     * @param string $name
     * @throws \yii\base\ExitException
     */
    public function actionCreateTheme($name = 'theme')
    {
        $name = trim($name);
        if (preg_match('#^[a-z]+$#i', $name) == 0) {
            echo "Bad theme name" . PHP_EOL;
            Yii::$app->end();
        }
        $themeRoot = Yii::getAlias('@webroot/' . $name);
        if (file_exists($themeRoot)) {
            echo "Directory \"{$themeRoot}\" already exists: " . PHP_EOL;
            Yii::$app->end();
        }
        $structure = include __DIR__ . DIRECTORY_SEPARATOR . 'data' . DIRECTORY_SEPARATOR . 'theme-structure.php';
        mkdir($themeRoot);
        $this->createStructure($themeRoot, $structure, $name);
        echo "Theme \"{$name}\" has been created" . PHP_EOL;
        if (exec("which npm")) {
            echo "Installing node.js dependencies" . PHP_EOL;
            exec("cd $themeRoot ; npm install");
        }
    }
}