Yii Framework

Привет всем. Понадобилось использование Google ReCaptcha 2.0 на одном из проектов. Существующие расширения не устроили полной непригодностью при выключенном JS. Да и городили собственные реализации работы с API Google.
Посему на коленке сделал расширение, которое:

Работает при выключенном JS;
Поддерживает несколько виджетов на странице;
Работает с API через библиотеку от Google.

https://github.com/brussens/yii2-recaptcha
Предложения по улучшению приветствуются.

Прежде всего: Отличное расширение, спасибо.

Для ключа API лучше не делать компонент и не привязываться к конкретному названию Yii::$app->recaptcha. А запрашивать ключи в конструкторах валидатора и виджета и внедрять их через контейнер используя callback. Ну или сделать объект с конфигурацией и внедрять его, зарегистрировав в контейнере.

Класс ReCaptcha тоже надо внедрять в валидатор, по идее через конструктор, иначе вы себя лишаете возможности написать модульный тест.

Могу сделать PR.

P.S. В Yii DI конечно очень странно реализовано, но что есть то есть.

Как ее обойти в acceptance тестах?

satoved писал(а): ↑2017.02.09, 14:39 Как ее обойти в acceptance тестах?

Заменить валидатор моком через DI)

anton_z писал(а): ↑2017.02.08, 03:03 Для ключа API лучше не делать компонент и не привязываться к конкретному названию Yii::$app->recaptcha. А запрашивать ключи в конструкторах валидатора и виджета и внедрять их через контейнер используя callback. Ну или сделать объект с конфигурацией и внедрять его, зарегистрировав в контейнере.

Спасибо, об этом думал уже. Такая реализация, через компонент, потому что надо было в резком темпе сделать что то работающее.

anton_z писал(а): ↑2017.02.08, 03:03 Класс ReCaptcha тоже надо внедрять в валидатор, по идее через конструктор, иначе вы себя лишаете возможности написать модульный тест.

Спасибо, думаю тоже будет не лишним.

anton_z писал(а): ↑2017.02.08, 03:03 Могу сделать PR.

Было бы не плохо, конечно присылайте)

А в общем спасибо за конструктивную критику) Так потихоньку расширения Yii становятся лучше)

попытался подключить и получил "банан":

PHP Warning – yii\base\ErrorException
Invalid argument supplied for foreach()

при вызове из:

Код: Выделить всё


2. in C:\WebServers\home\site.local\common\config\main.php at line 164 – yii\base\Object::__construct('config' => '', 1 => '', 2 => '')
158159160161162163164165166167168169170

            ],
        ],
    ],
    'container' => [
        'definitions' => [
            \brussens\yii2\extensions\recaptcha\Widget::class => function($container, $params, $config) {
                return new \brussens\yii2\extensions\recaptcha\Widget('***************************', \Yii::$app->language, $config);
            }
        ],
        'singletons' => [
            \ReCaptcha\ReCaptcha::class => function($container, $params, $config) {
                return new \ReCaptcha\ReCaptcha('*******************************');

shimbo писал(а): ↑2017.06.26, 18:39 попытался подключить и получил "банан":

Было бы прекрасно, если бы описали, как появилась ошибка (фрагменты подключаемого кода и т.п.).

так все по схеме.
поставил composer-ом
шаблон advanced.
кусок из common/config/main.php:

Код: Выделить всё

    'container' => [
        'definitions' => [
            \brussens\yii2\extensions\recaptcha\Widget::class => function($container, $params, $config) {
                return new \brussens\yii2\extensions\recaptcha\Widget('********************', \Yii::$app->language, $config);
            }
        ],
        'singletons' => [
            \ReCaptcha\ReCaptcha::class => function($container, $params, $config) {
                return new \ReCaptcha\ReCaptcha('***********************');
            }
        ]
    ],

есть 2 модели:

Код: Выделить всё

class CallBack extends Model

и

Код: Выделить всё

class ContactUs extends Model

в каждой из моделей есть атрибут

Код: Выделить всё

    /**
     * for captcha
     *
     * @var string
     */
    public $verifyCode;

и соответственно прописаны одинаковые валидаторы:

Код: Выделить всё


use brussens\yii2\extensions\recaptcha\Validator as ReCaptcha;

    public function rules()
    {
        return [
            [
                [
                    'phone',
                    'message',
                ],
                'required',
            ],
            [['phone'], 'string', 'length' => 12],
            [['message'], 'string'],
            ['verifyCode', ReCaptcha::className()],
            [
                [
                    'message',
                    'phone',
                ],
                'safe',
            ],
        ];
    }

контроллер вызывает вьюху, в которой прописан вызов виджета:

Код: Выделить всё

<?php
?>

<?=\common\widgets\Feedback::widget();?>

ну в самом виджете таб и 2 формы на каждой из вкладок
внутри каждой из форм капча:

Код: Выделить всё

use brussens\yii2\extensions\recaptcha\Widget as ReCaptcha;

                                        <?=$form->field($contactUs, 'verifyCode')->widget(ReCaptcha::className(), [
                                            'options' => [
                                                'id' => 'insert-unique-widget-id-1',
                                            ],
                                        ]);?>

и такой же только для другой модели. но до него даже и не доходит. валится на первом же вызове.

Yii 2.0.12
PHP 5.6.30

падает тут:
\yii\BaseYii::configure

Код: Выделить всё

    /**
     * Configures an object with the initial property values.
     * @param object $object the object to be configured
     * @param array $properties the property initial values given in terms of name-value pairs.
     * @return object the object itself
     */
    public static function configure($object, $properties)
    {
        foreach ($properties as $name => $value) {
            $object->$name = $value;
        }

        return $object;
    }

потому что в $properties приходит текстовый ключ, а он пытается обратиться к нему как к массиву.

сам себе и отвечу

используйте

php composer.phar require --prefer-dist brussens/yii2-recaptcha "@dev"

а не

php composer.phar require --prefer-dist brussens/yii2-recaptcha "*"

потому как после обновления до актуальной версии все заработало.

shimbo писал(а): ↑2017.06.26, 20:09 сам себе и отвечу
используйте
php composer.phar require --prefer-dist brussens/yii2-recaptcha "@dev"
а не

php composer.phar require --prefer-dist brussens/yii2-recaptcha "*"
потому как после обновления до актуальной версии все заработало.

Да, именно из-за того, что релиз по другому немного работает. Немного допилю и тогда уже сделаю релиз, пока надо убедиться, что всё будет работать, как надо.

Всем привет, пытаюсь установить, делаю все по инструкции, но выдает ошибку:

PHP Warning – yii\base\ErrorException
Invalid argument supplied for foreach()

\yii\BaseYii::configure at 528

Yii 2.0.12
PHP 7.1.0
Yii2-recaptcha 1.0.0

akkman писал(а): ↑2017.09.18, 09:29 Всем привет, пытаюсь установить, делаю все по инструкции, но выдает ошибку:

PHP Warning – yii\base\ErrorException
Invalid argument supplied for foreach()

\yii\BaseYii::configure at 528
Yii 2.0.12
PHP 7.1.0
Yii2-recaptcha 1.0.0

Как именно делаете? Вот прям всё в точности, как в инструкции? Если так, то ошибки не должно быть. Код свой покажите

urichalex писал(а): ↑2017.09.18, 13:38 Код свой покажите

bootstrap:

Код: Выделить всё

<?php

namespace app\bootstrap;

use brussens\yii2\extensions\recaptcha\Widget;
use ReCaptcha\ReCaptcha;
use yii\base\BootstrapInterface;
use Yii;

class ReqBootstrap implements BootstrapInterface
{
    public function bootstrap($app)
    {
        $container = Yii::$container;

        $container->setSingleton(ReCaptcha::class, function($container)
        {
            return new ReCaptcha(RECAPTCHA_SECRET);
        });

        $container->set(Widget::class, function($container, $params, $config)
        {
            return new Widget(RECAPTCHA_KEY, Yii::$app->language, $config);
        });
    }
}

model:

Код: Выделить всё

<?php

namespace app\req\forms;

use app\req\ReqModel;
use brussens\yii2\extensions\recaptcha\Validator;

class test extends ReqModel
{
    public $verifyCode;

    public function rules()
    {
        return [
            ['verifyCode', Validator::className()]
        ];
    }
}

ReqModel:

Код: Выделить всё

use yii\base\Model;
class ReqModel extends Model

views:

Код: Выделить всё

<?= $form->field($model, 'verifyCode')
    ->widget(\brussens\yii2\extensions\recaptcha\Widget::className()) ?>

И полный выхлоп ошибки

urichalex писал(а): ↑2017.09.18, 17:19 И полный выхлоп ошибки

Проблема решилась, после того как взял файл Widget.php с ветки `master`

в релизе в файле Widget.php отсутствовал конструктор

Код: Выделить всё

__construct($siteKey, $language, $config = [])

update: валидация тоже работала некорректно, заработала после того, как заменил файл Validator.php из ветки `master`

akkman писал(а): ↑2017.09.18, 19:04 Проблема решилась, после того как взял файл Widget.php с ветки `master`

в релизе в файле Widget.php отсутствовал конструктор
Код: Выделить всё
__construct($siteKey, $language, $config = [])
update: валидация тоже работала некорректно, заработала после того, как заменил файл Validator.php из ветки `master`

Пока что не хватает времени допилить, что бы релизнуть. Постараюсь как можно быстрее запилить релиз.

Yii Framework

Yii2-recaptcha

Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha

Re: Yii2-recaptcha