Форумы / National / Russian / Siena 0.9.4

Macik
#30753 18.09.2011 22:00
#30751 esclkm:

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

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

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

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 функционал.

https://github.com/macik
правильный хостинг — https://goo.gl/fjCa1F