FsaController::actionRemoveOldUnconfirmedAccounts() - Code Metrics - Inspection of "Creates cli function for removing old accounts" - CorWatts/fasterscale - Measure and Improve Code Quality continuously with Scrutinizer

Passed

Pull Request — master (#191)

by Corey

created 2020-12-18 19:39 UTC

actionRemoveOldUnconfirmedAccounts() A

↳ Parent: FsaController

Complexity

Conditions	1
Paths	1

Size

Total Lines	14
Code Lines	8

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
eloc	8
c	1
b	0
f	0
dl	0
loc	14
rs	10
cc	1
nc	1
nop	0

<?php
namespace console\controllers;

use \yii\helpers\BaseConsole as Console;
use \common\models\User;

/**
 * Faster Scale App CLI Helpers
 */
class FsaController extends \yii\console\Controller
{
  public $defaultAction = null;

  public $ageThreshold = 7; // in days

  /**
   * Deletes unconfirmed accounts that are older than the lifetime of the
   * verification token
   */
  public function actionRemoveOldUnconfirmedAccounts(): void
  {
    $this->stdout("Removing accounts older than {$this->ageThreshold} days...\n", Console::FG_YELLOW);
    $count = User::deleteAll([
      'and',
        ['or',
          ['not', ['verify_email_token' => null]],
          // we have to escape the '_' at the start of User::CONFIRMED_STRING
          ['not like', 'verify_email_token', '%\\'.User::CONFIRMED_STRING, false]
        ],
        ['<', 'created_at', $this->getTimeThreshold()],
    ]);

    $this->stdout("Removed $count accounts", Console::FG_GREEN);
  }

  /**
   * Returns the cut-off time threshold for unconfirmed users we should delete
   *
   * @return integer
   */
  public function getTimeThreshold(): int
  {
        $expire = \Yii::$app->params['user.verifyAccountTokenExpire'];
        return time() - $expire;
  }

}


1			<?php
2			namespace console\controllers;
3
4			use \yii\helpers\BaseConsole as Console;
5			use \common\models\User;
6
7			/**
8			* Faster Scale App CLI Helpers
9			*/
10			class FsaController extends \yii\console\Controller
11			{
12			public $defaultAction = null;
13
14			public $ageThreshold = 7; // in days
15
16			/**
17			* Deletes unconfirmed accounts that are older than the lifetime of the
18			* verification token
19			*/
20			public function actionRemoveOldUnconfirmedAccounts(): void
21			{
22			$this->stdout("Removing accounts older than {$this->ageThreshold} days...\n", Console::FG_YELLOW);
23			$count = User::deleteAll([
24			'and',
25			['or',
26			['not', ['verify_email_token' => null]],
27			// we have to escape the '_' at the start of User::CONFIRMED_STRING
28			['not like', 'verify_email_token', '%\\'.User::CONFIRMED_STRING, false]
29			],
30			['<', 'created_at', $this->getTimeThreshold()],
31			]);
32
33			$this->stdout("Removed $count accounts", Console::FG_GREEN);
34			}
35
36			/**
37			* Returns the cut-off time threshold for unconfirmed users we should delete
38			*
39			* @return integer
40			*/
41			public function getTimeThreshold(): int
42			{
43			$expire = \Yii::$app->params['user.verifyAccountTokenExpire'];
44			return time() - $expire;
45			}
46
47			}
48

CorWatts / fasterscale

Pull Request — master (#191)

actionRemoveOldUnconfirmedAccounts() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like