[INZ] SMS Two-step verification

[inz]

Громко и глупо.

public function meetsRequirements(\XF\Entity\User $user, &$error)
{
    if ($user->user_state != 'valid' || empty($this->getUserPhone($user->user_id)))
    {
        $error = \XF::phrase('you_must_have_valid_email_account_confirmed');
        return false;
    }
  
    return true;
}

Если не валидный телефон, то выдавать ошибку с тем, что мыло не валидно ? Каво ?
--------
Вопрос второй.
$this->getUserPhone - нахрена, а главное зачем ?

Первое поправлю, а во второе для читабельности кода. Твой аддон ничего сверхъестественного не делает по сравнению с этим.

 

[25517]

root, читабельность заключается в лишнем запросе ?

 

[inz]

root, читабельность заключается в лишнем запросе ?

Скажи мне, ты дурачек? Какая разница, что бы я фетчил этот запрос в самой переменной, или в отдельной функции?

 

[25517]

Твой аддон ничего сверхъестественного не делает по сравнению с этим

 

[inz]

Посмотреть вложение 125478

OFFTOP

Когда сделали аналог твоего плагина за 7.50$ и ты пытаешься найти каждую загвоздку))

 

[25517]

root,

OFFTOP

Мне то похер, живи дебилом и создавай лишние запросы.

 

[inz]

root,

OFFTOP

Мне то похер, живи дебилом и создавай лишние запросы.

OFFTOP

ахахах, кто бы говорил