Foren / National / Russian / Siena 0.9.4

Trustmaster
#30785 21. September 2011, 07:25

Большая просьба всем тестирующим проверить наличие указанных выше багов в текущем срезе и создать для ещё не закрытых esclkm'ом багов тикеты здесь (можно даже на русском, переведу если надо), указав линк на сообщение в этой теме. Иначе, боюсь, потеряемся в таком потоке. И для вновь добавляемых багов тоже карточки создавайте.

Alex300, по поводу ЧПУ в своих модулях: по умолчанию система расценивает параметр 'с' в любых ссылках как категорию в структуре (то есть берёт $structure['имя_модуля'][$c]['path'] для этой категории), параметр 'al' как алиас, а параметр 'id' как числовой идентификатор. То есть если сделать

PHP
1
cot_url('my_module', 'c=mycat&al=supername')

то на выходе будет что-то вроде http://example.com/path/to/mycat/supername.

К слешам на конце новая система не чувствительна, поскольку категории проверяются на наличие в $structure, а не по слешу. Суффикс .html для страниц добавлен не будет, поскольку это усложняет написание ссылок в адресной строке. Сравните: http://сайт.рф/товарищ_сталин и http://сайт.рф/товарищ_сталин.html.

По остальному смогу отписаться только после того, как будет ясно, что уже пофиксили, а что нет.

Added 10 hours later:

#30753 Macik:
> #30751 esclkm:

А что мешает самому проверять в RC переменную $e

Да собственно ничего (это и не баг в чистом виде без которого система не работает),

но образуется избыточность кода. Вставлять таие блоки в каждай плагин:

PHP
1
2
3
4
5
6
7
8
9
if (empty($_GET['e']) && !empty($_GET['r']))
{
    $_GET['e'] = $_GET['r'];
}
if (empty($_GET['e']) && !empty($_GET['o']))
{
    $_GET['e'] = $_GET['o'];
}
 $env['ext'] = $_GET['e'];

как-то нелепо. Тем более что этот код есть в самой CMS.

Да и чисто логически переменная $env['ext'] для того и была заведена, чтобы на каждый "чих" избавиться от рутины проверки входных параметров на предет того, какой модуль запущен.

Развивая мысль: в прошлах версиях в ajax вызовах плагина (Hooks=ajax) не было автоматической (средствами CMS) подгрузки языковых файлов плагина, приходилось именно так и поступать - копировать 10 строк кода из common и вставлять в каждый plug.ajax.php файл. Сайчас в этом необходимости нет, и это гораздо удобнее. Суть - это дать разработчику сосредоточится на цели создания кода (плагина), а не отвлекать необходимостью делать такого рода подпорки, повторяя уже заложенный в CMS функционал.

Используй просто $_GET['e']. r и o тебе не нужны, поскольку в ajax (r) нет header-части, а popup (o) сейчас мало где используется.

May the Source be with you!

Dieser Beitrag wurde von Trustmaster (am 21. September 2011, 17:39, vor 13 Jahre) bearbeitet