Информация Мини-аудит кода

[Gameer]

Увы, это не очень красиво для пользователей

Со всем уважением, причем тут пользователи? Как они знают что ты используешь а что нет? Для них нужно получить то что они хотят а еще лучше что бы при этом ничего делать не нужно. Наоборот чем меньше ты используешь что-то от движка - тем лучше. Тебе не нужно будет ввести поддержку, в случае если какой-то инструмент движка удалят в новой версии / изменят / заменят. Ты будешь сосредоточен только на своем продукте.

 

[Горбушка]

Gameer, они переходят в другой дизайн... Это воспринимается как внешняя система. По опыту работы с крупными заказчиками - большим дядям это не нравятся. Они хотят видеть 1 систему, а не сборище отдельных систем...

 

[bykaVBS]

Меня позабавила система аплоада. В категории есть поле для иконки. Туда нужно пихать урл картинки. (лол?) Хотя в dle есть вроде как аплоадер (например для миниатюр в новости, xfields ). Захотел привязать этот аплоадер к иконке категории, показалось что не должно быть ничего сложного. Но когда начал ковырять этот аплоадер - оказалось везде где только можно идет привязка к "новостям"

/engine/ajax/upload.php

$row = $db->super_query( "SELECT id, onserver FROM " . PREFIX . "_files WHERE author = '$author' AND news_id = '$news_id' AND id='$file'" );

/engine/classes/upload.class

$row = $db->super_query( "SELECT COUNT(*) as count  FROM " . PREFIX . "_files WHERE author = '{$this->author}' AND news_id = '{$this->news_id}'" );

итд итп и вообще нет возможности юзать подсистему аплоада для каких-то других нужд... И вообще это сбор такого матерого говнокода... что проще было просто взять и привязать к полю левый аплоадер

В этой системе кстати кроме этого еще куча разных забавных вещей.
Да там даже орфографические ошибки присутствуют. $row['autor']

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

 

[PirateGod]

Меня позабавила система аплоада. В категории есть поле для иконки. Туда нужно пихать урл картинки. (лол?) Хотя в dle есть вроде как аплоадер (например для миниатюр в новости, xfields ). Захотел привязать этот аплоадер к иконке категории, показалось что не должно быть ничего сложного. Но когда начал ковырять этот аплоадер - оказалось везде где только можно идет привязка к "новостям"

/engine/ajax/upload.php

$row = $db->super_query( "SELECT id, onserver FROM " . PREFIX . "_files WHERE author = '$author' AND news_id = '$news_id' AND id='$file'" );

/engine/classes/upload.class

$row = $db->super_query( "SELECT COUNT(*) as count  FROM " . PREFIX . "_files WHERE author = '{$this->author}' AND news_id = '{$this->news_id}'" );

итд итп и вообще нет возможности юзать подсистему аплоада для каких-то других нужд... И вообще это сбор такого матерого говнокода... что проще было просто взять и привязать к полю левый аплоадер

В этой системе кстати кроме этого еще куча разных забавных вещей.
Да там даже орфографические ошибки присутствуют. $row['autor']

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

А я это поле использую как доп поле типа текст, бывает нужно что то выводить разное в каждой категории )

 

[Горбушка]

Нашёл очередное забавное место...
Файл templates.class.php в районе 360 строки:

		if( $tpl_name == '' || !file_exists( $templatefile ) ) {

			$templatefile = str_replace(ROOT_DIR,'',$templatefile);
			return "Template not found: " . $templatefile ;
			return false;

		}

return false; не выполнится никогда в жизни

Пояснение с php.net:
Если вызывается из функции, выражение return немедленно прекращает выполнение текущей функции и возвращает свой аргумент как значение данной функции.

Надо сказать правду - почти в любой CMS таких кусков полно (где добавили новый код, не удалив старый)... Хотя они находятся довольно легко любой IDE.