проект статьи
| Trustmaster |
|
|---|---|
|
На том и порешили. Тикет добавлю, изменения внесу.
May the Source be with you!
|
| Asmo |
|
|---|---|
|
И по хорошему установочные файлы защищены должны быть.
if (!defined('SED_CODE')) { die('Wrong URL.'); }А то посмотрел тот же markitup, инсталяционные файлы не защищены и если бы код не был вынесен в отдельную функцию, то можно было бы через прямой доступ к файлу унинсталить плагин. В случае с редактором вреда это принесет не много, а вот грохнутые таким образом таблицы какой нибудь доски объявлений - потеря ощутимая. |
| Trustmaster |
|
|---|---|
|
Да заметил это. Исправлено.
May the Source be with you!
|
| dervan |
|
|---|---|
|
Предлагаю сделать контроль ошибок при инсталляции плагина
Если при инсталляции плагина произошёл критический сбой, тогда из секции инсталляции возвращать строку с сообщением об ошибке, при получении которой прерывать установку плагина. Т.е. код в system/core/admin/admin.plug.inc.php
$action = 'install';
include_once($extplugin_info);
$adminmain .= "Done!<br />";
...
$action = 'install';
$installerror = include_once($extplugin_info);
if (!empty($installerror))
{
// error handling
...
}
else
{
$adminmain .= "Done!<br />";
...
При такой реализации существующие плагины, в которых нет return(), переделывать не придётся. Про интернационализацию сообщений об ошибках. Можно возвращать не само сообщение об ошибке, а отличный от 0 номер ошибки. |
| esclkm |
|
|---|---|
|
в плагинах есть преверка на плагины от лду - о имхо она какаято грубая.может убрать эту проверку и делаеть более современую? (а то както глупа если есть p то это лду.)
littledev.ru - мой маленький зарождающийся блог о котонти.
снижение стоимости программирования и снижение стоимости производства разные вещи. Первое можно скорее сравнить с раздачей работникам дешевых инструментов, чем со снижением зарплаты |
| Fagot |
|
|---|---|
|
предлагаю в файл plug.inc.php в строки определяющие лэнг файл добавить следующие:
$path_lang_chr = $cfg['plugins_dir']."/$e/lang/$e.$lang."(".$cfg['charset'].").lang.php";
if (file_exists($path_lang_chr))
{ require_once($path_lang_chr); }
это позволит разработчикам русскоязычных плагинов заранее готовить языковые файлы вида: gallery.ru.(UTF-8).lang.php gallery.ru.(windows-1251).lang.php для различных кодировок, а пользователям не париться с переименованиями и перекодированиями. |
| Sergeich |
|
|---|---|
|
да вроде как решили - UTF и точка. С моей точки зрения - это правильно, чтобы не возникал (кроме всего прочего) зоопарк с кодировками.
|
