Contact force verification #2
Conversation
bladeroot
commented
Jan 6, 2017
bladeroot
commented
- ForceVerificationBlock widget
- use ForceVerificationBlock in Contact and Client View
| <?php $box->end() ?> | ||
| <?php endif ?> | ||
| <?= ForceVerificationBlock::widget([ | ||
| 'model' => $model, |
There was a problem hiding this comment.
у тебя тут отступ 8 пробелов - а у нас везде 4
проверь свои настройки что было 4
There was a problem hiding this comment.
правильно. виджет находится между дивами. таб = 4 пробела
| @@ -48,6 +50,10 @@ | |||
| <?= ClientDetailMenu::widget(['model' => $model]) ?> | |||
| </div> | |||
| <?php Box::end() ?> | |||
|
|
|||
| <?= ForceVerificationBlock::widget([ | |||
| 'model' => $model->contact | |||
There was a problem hiding this comment.
закидывай запятую в массивах, потом легче докидывать новые элементы
| <?= Verification::widget([ | ||
| 'model' => $model->getVerification($attribute), | ||
| 'scenario' => $scenario, | ||
| 'submitUrl' => $submitUrl |
| use yii\helpers\Html; | ||
| use hipanel\widgets\Box; | ||
| use hipanel\modules\client\widgets\Verification; | ||
|
|
There was a problem hiding this comment.
В блоке PHPDoc мы описываем переменные, которые приходят извне.
Смотри пример тут https://github.com/hiqdev/hipanel-module-mailing/blob/master/src/views/prepare/index.php#L6-L14
|
|
||
| if ($this->title === null) { | ||
| $this->title = Yii::t('hipanel:client', 'Verification level'); | ||
|
|
| * | ||
| * @var array | ||
| */ | ||
| public $options = []; |
There was a problem hiding this comment.
кажется, это свойство не используется
| 'scenario' => $this->scenario, | ||
| 'submitUrl' => $this->submitUrl | ||
| ]); | ||
| } |
There was a problem hiding this comment.
Методы должны иметь явно описанные возвращаемые данные. Предпочтительно использование условий для прерывания выполнения, вместо описания полезного кода внутри if. Это упрощает чтение кода.
if (!Yii::$app->user->can('contact.force-verify')) {
return;
}
return $this->render((new \ReflectionClass($this))->getShortName(), [
'attributes' => $this->attributes,
'model' => $this->model,
'title' => $this->title,
'scenario' => $this->scenario,
'submitUrl' => $this->submitUrl
]);
}
