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

[alex_qrlogin]

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

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

[Татьяна5]

кроме временных файлов и бд

Куки в браузере

[Sheer]

/qrlogin/qrlogin - так и должно быть

Так не должно быть. Это вводит в заблуждение.

разве он не связан с services.yml

Естественно. При изменении списка параметров в конструкторе самом собой разумеется нужно изменять их и в сервисах.

из работающего расширения и оно работает

Но это не означает, что так правильно.

[alex_qrlogin]

Татьяна5, гм... может я чего то не понимаю... при чем куки? мне нужно передать логин-пароль из сессии с телефона в сессию с компа...

Отправлено спустя 1 минуту 5 секунд:
Sheer, кого ввводит в заблуждение?))
а насчет причесывания кода - я им займусь поближе к финишу))

[Sheer]

Аааа!!! Вот это ваще жесть!

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

	public function __construct()
	{
		global $user, $template, $config;
		
		$this->user			= $user;
		$this->template		= $template;
		$this->config		= $config;

[alex_qrlogin]

Sheer, это из ucp) для него тоже можно в сервисах прописать? я реально куски из разных расширений дергал - там так было))

Отправлено спустя 39 секунд:
и в acp так же - и там тоже можно?

[Татьяна5]

из сессии с телефона в сессию с компа

В таком случае остаются, разве что, сокеты. Но они не подходят под задачу (плюс мало где доступны)

[alex_qrlogin]

Sheer, в ajaxchat такие конструкции) мне его как пример посоветовали))(Татьяна5)

Отправлено спустя 1 минуту 56 секунд:
Татьяна5, успокоили)) а то я морочусь ищу ищу... значит или файл или бд... а к чему разработчики phpbb благосклоннее? как то мне лень свою временную табличку делать)) может в settings можно кидать?))) на каждый логин через них передавать))

[Sheer]

это из ucp)

Достаточно глобальных объектов функции main()
Например так выглядит "верхушка модуля

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

class report_module
{
	var $p_master;
	var $u_action;

	function __construct(&$p_master)
	{
		$this->p_master = &$p_master;
	}

	function main($id, $mode)
	{
		/**
		* Setup global variables such as $db
		*/
		global $config, $db, $user, $template, $cache, $phpbb_container, $phpbb_root_path, $phpEx, $request;

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

'qrlogin_qrcode_HEADER'                 => 'Параметры QRCODE',

В phpBB не принято использовать символы нижнего регистра для "языковых переменных", а тем более смешивать символы в разных регистрах.

[alex_qrlogin]

Sheer, т.е. в листенере вы требуете получить как параметры в конструктор глобальные переменные и присвоить их локальным, а в модулях считаете не правильным их присваивать в конструкторе локальным?))) где логика?)) две прямо противоположные рекомендации))

Отправлено спустя 1 минуту 48 секунд:
про то что qrlogin_qrcode_HEADER нужно будет переименовать в QRLOGIN_QRCODE_HEADER я в курсе... ну принято так, так принято)) дело привычки)) но сначала РАБОТОСПОСОБНОСТЬ, а потом уже плюшки и красота кода тем более))

[Татьяна5]

Достаточно глобальных объектов функции main()

Не вижу ничего плохого, если вынести их в __construct (без дубляжа где-либо)
Как "задел на будущее", когда нормальное получение объектов в модулях всё-таки сделают, это по-моему удобней

alex_qrlogin, а вот что в конструкторе делает инклуд мне непонятно

но сначала РАБОТОСПОСОБНОСТЬ, а потом уже плюшки и красота кода тем более

С плюшками верно, с красотой кода нет. При приведении в порядок кода "потом" работоспособность может пропасть (это вплоть до необходимости переписывать всё)

[alex_qrlogin]

Татьяна5, инклуд подключает библиотеку))

Отправлено спустя 3 минуты 40 секунд:
согласен)) вынес инклуд в шапку))
сори - реально на пхп вторая неделя)))

Отправлено спустя 1 минуту 14 секунд:
Татьяна5, работоспособность не пропадет от изменения регистра переменных))

[Татьяна5]

от изменения регистра переменных

И от несоответствия этого регистра где-нибудь на части серверов работоспособность пропадёт

[alex_qrlogin]

Татьяна5, уговорили)) пойду менять)) работы на всю ночь)))

Отправлено спустя 2 минуты 33 секунды:
а насчет функционала комментариев нету?
мне например важно мнение насчет размещения qrcode для логина.... в шапке, в меню и фиксированное в любом месте ... на выбор ...
может есть какие то соображения?
и по event-ам - те ли или есть получше места?))

[Татьяна5]

Для разбора кода сам код для начала должен быть нормальным

[alex_qrlogin]

Татьяна5, т.е. только код посмотрели?)) ставить не ставили и не пробовали))
тогда ясно почему вопросы по стилю)))