site\models\DeleteAccountForm

Complexity

Total Complexity

6

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	3	1
A	deleteAccount()	0	9	3
A	rules()	0	5	1
A	attributeLabels()	0	3	1

<?php
namespace site\models;

use common\models\User;
use yii\base\Model;
use Yii;

/**
 * edit profile form
 */
class DeleteAccountForm extends Model
{
  public $password;
  private $user;

  /**
   * Creates a form model
   *
   * @param  object                          $user
   * @param  array                           $config name-value pairs that will be used to initialize the object properties
   */
  public function __construct(\common\interfaces\UserInterface $user, $config = []) {
    $this->user = $user;
    parent::__construct($config);
  }

  /**
   * @inheritdoc
   */
  public function rules()
  {
    return [
      ['password', 'required'],
      ['password', 'string', 'min' => 6],
    ];
  }

  public function attributeLabels() {
    return [
      'password' => 'Password'
    ];
  }

  /**
   * saves user's profile info.
   *
   * @return User|null the saved model or null if saving fails
   */
  public function deleteAccount()
  {
    if ($this->validate() && $this->user->validatePassword($this->password)) {

The method validatePassword() does not exist on common\interfaces\UserInterface. Since it exists in all sub-types, consider adding an abstract or default implementation to common\interfaces\UserInterface.

      $this->user->sendDeleteNotificationEmail();

The method sendDeleteNotificationEmail() does not exist on common\interfaces\UserInterface. It seems like you code against a sub-type of common\interfaces\UserInterface such as common\models\User.

      $this->user->delete();
      return true;

The expression return true returns the type true which is incompatible with the documented return type common\models\User|null.

    }

    return false;

The expression return false returns the type false which is incompatible with the documented return type common\models\User|null.

  }
}