Foren / National / Russian / Тех. поддержка / Баги 097

#33539 Eugene:

Что бы в самописных плагинах и модулях использовать их на ряду с $db_pages и пр.

Я наткнулся на эту проблему на прошлой неделе - в процессе конвертации MyFiles (от ez) под Siena. Оказалось, что эти 2 переменные для базы сейчас в конфиге самого модуля PFS. Это вроде логично - включил модуль и переменные для 2 таблиц стали активными - вот только scope их использования = (((( 

Сторонний плагин действительно не имеет к этим переменным доступа...

Поэтому плагины правильно писать используя конструкцию вида (на примере того же модуля page)

if (cot_module_active('page'))
{
	require_once cot_incfile('page',  'module');
}

Которая проинклудит файл /modules/page/inc/page.functions.php в котором есть 

// Global variables
global $cot_extrafields, $db_pages, $db_x;
$db_pages = (isset($db_pages)) ? $db_pages : $db_x . 'pages';

И всего делов то - правильно пишем плагины под Сиену

Это защита от hotlinking'а

Это просто отлично, но многие юзеры открывают сразу несколько вкладок. Как я понимаю, нельзя сделать возможность скачивания файла из нескольких вкладок, без отключения защиты от хотлинка?

#33543 Dayver:

И всего делов то - правильно пишем плагины под Сиену

Согласен... для меня больше это было новостью, чем багом...

ещё учиться и учиться... )))  Поднимем уровень, сиеноплагинописательства на новые высоты, товарищи!

В последней версии сиены появился хук urleditor.rewrite.first позволяющий модулям обрабатывать свое ЧПУ, не описанное в пресетах по-умолчанию. Но код, следующий за ним его "перекрывает", что делает невозмжным использование этого хука.

Сам хук вызывается в функции cot_apply_rwr(), но в силу особенностей php, если мой модуль после обработки ЧПУ не может прервать выполнение этой функции т.к. "return" в приинклуденном файле прерывает выполнение этого самого файла и передает управление назад после вызвавшей его строки.

Предлагаю в этой функции после кода

      /* === Hook === */
		foreach (cot_getextplugins('urleditor.rewrite.first') as $pl)
		{
			include $pl;
		}
		/* ===== */

Добавить проверку необходимости дальнейшей обработки ЧПУ.

Я смог реализовать обработку ЧПУ при помощи хука input. Но правильней все-таки было бы использовать urleditor.rewrite.first

#get;e=i18n&m=structure&l=ru (перед последним парамтром &d=2) - лишняя часть ссылки. Надо бы избавиться.