[3.1][3.2] qrLogin - вход на форум с помощью телефона

[alex_qrlogin]

Установка - стандарт: распаковать архив в ext и затем в панели включить.
Дальше залогиниться на форуме и в Личном разделе во вкладке qrLogin отсканировать qrcode аккаунта и можно входить))

Репо: https://github.com/qrLogin/phpbb_ext

[Татьяна5]

На этом форуме вы уже выложили
В файловом архиве раздел давно не пополняют

Отправлено спустя 1 минуту 8 секунд:
Для расширений его даже нет. Только для модов

[alex_qrlogin]

Татьяна5, имел в виду, переместить в раздел раширений из этого

Отправлено спустя 1 минуту 6 секунд:
я так понимаю, у Вас два раздела - один для своих расширений, второй для сторонних разработчиков, а этот для бета-версий

[alex_qrlogin]

выложил на гит: https://github.com/qrLogin/phpbb_ext
правда без библиотеки qrcode - на нее тревис сильно ругается за пробелы и тп... а она не моя что бы править

вот такие результаты у тревиса:
в 1 тесте только варнинги и файлед(((
а 12 тест вообще не понимаю - чего он не выполняет там ничего?
пишет

HHVM is no longer supported on Ubuntu Precise. Please consider using Trusty with `dist: trusty`.

[Sheer]

Начнем с того, что все файлы и папки расширения на github должны быть размещены в корневой папке репозитория, а не в qrlogin/qrlogin, например как тут https://github.com/AlexSheer/phpbb3.1-Request-Pattern

[alex_qrlogin]

Sheer,изначально так и сделал, но тогда тревис ругался на структуру
Да и так как сейчас при скачивании архив получается с папками и его просто распаковать нужно и все

Отправлено спустя 52 секунды:
Вернуть не долго)) но вернется и ошибка

[Татьяна5]

Не ругается travis на структуру, когда нет папок vendor и extname
А остальные ошибки исправляйте. Ясно же пишет, что константа IN_PHPBB не задана, а exit использовать в тех файлах нельзя

[alex_qrlogin]

Татьяна5,
Вот что он писал насчет структуры:

Error: Packaging structure doesn't meet the extension DB policies.
Expected: qrlogin/qrlogin

А насчет варнингов -

Warning: IN_PHPBB is not defined in /qrlogin/qrlogin/event/listener.php

Разве в листенере нужно? Нигде нет, как и в модулях асп и юсп

Насчет exit - ну что за требование тянуть if если после условия ничего делать не нужно
И уж один exit мне точно нужен - надо как то сообщить жаваскрипту что необходимо релоад страницы сделать, echo он вообще кричал ошибка)

[rxu]

Вот что он писал насчет структуры:

Это можно игнорировать, на валидацию не влияет.

Разве в листенере нужно?

У вас там есть код до объявления класса. Если кроме класса есть просто исполняемый код, то нужно. Хотя с таким валидацию вряд ли пройдет.

[Татьяна5]

Разве в листенере нужно?

Не нужно
А вот что там делает инклуд вне класса - это вопрос

ну что за требование тянуть if если после условия ничего делать не нужно

Нет такого требования
На месте первого exit'а надо возвращать ошибку, а второй там вообще не нужен

[rxu]

Не нужно

Если кроме класса есть просто исполняемый код, то нужно.

[Татьяна5]

Хотя с таким валидацию вряд ли пройдет

Так что - не нужно. Нужно убрать тот код

[alex_qrlogin]

Татьяна5,
а куда инклуд? в конструктор?
насчет exit - как вернуть ошибку?
а второй как раз мне заменяет echo

И уж один exit мне точно нужен - надо как то сообщить жаваскрипту что необходимо релоад страницы сделать, echo он вообще кричал ошибка)

[Татьяна5]

а куда инклуд? в конструктор?

По-моему в какой-то метод лучше

насчет exit - как вернуть ошибку?

Лучше исключением, наверное - http://php.net/manual/ru/language.exceptions.php

а второй как раз мне заменяет echo

Почему не возвращать обычный json?

[rxu]

а куда инклуд?

По-хорошему, сделать пакет зависимостью в composer.json, после установки зависимостей она попадет в папку vendor, и оттуда уже использовать как сервис.

[alex_qrlogin]

Почему не возвращать обычный json?

там совершенно все равно что вернуть - лишь бы что то - загляните в жава скрипт - проверка на не пустой ответ
возвращал echo '!' - вот поменял на exit('!') - warning вместо error теперь))