Забыли пароль?
Запросите новый здесь.

Автор темы: Chief
ID темы: 971
Информация:
Тема содержит 6 сообщения, была просмотрена 2963 раз.
Просмотр темы
PHP-Fusion Russia » Поддержка 7 версии » Пожелания и предложения
 Распечатать тему
Недочеты в коде PHP-Fusion
Chief
Наболело...

Давно возникло впечатление последние версии PHP-Fusion пишутся наспех. Из-за количества мелких ошибок и странностей в коде, многие файлы слегка похожи на творения школьника впервые научившегося выводить с помощью <? php ?> какие-то символы в браузер, а не на официальный релиз движка с десятилетней историей.

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

Из только что замеченного, файл 7.02.05/administration/submissions.php странные копипасты в коде указанного фрагмента просмотра загрузки

Скачать исходники  Код
$editlist = ""; $sel = "";
$result2 = dbquery("SELECT download_cat_id, download_cat_name FROM ".DB_DOWNLOAD_CATS." ORDER BY download_cat_name");
if (dbrows($result2) != 0) {
   while ($data2 = dbarray($result2)) {
      
if (isset($_GET['action']) && $_GET['action'] == "edit") { $sel = ($data['download_cat'] == $data2['download_cat_id'] ? " selected='selected'" : ""); }
      $editlist .= "<option value='".$data2['download_cat_id']."'$sel>".$data2['download_cat_name']."</option>\n";
   }
}

$photo_albums = ""; /* Зачем это здесь? */ $sel = "";
$editlist = ""; $sel = "";
$result2 = dbquery("SELECT download_cat_id, download_cat_name FROM ".DB_DOWNLOAD_CATS." ORDER BY download_cat_name");
if (dbrows($result2) != 0) {
   while ($data2 = dbarray($result2)) {
      $sel = ($data2['download_cat_id'] == $submit_criteria['download_cat'] ? " selected='selected'" : "");
      $editlist .= "<option value='".$data2['download_cat_id']."'$sel>".$data2['download_cat_name']."</option>\n";
   }

}



Ну и конечно с возможностями фьюжена говорить на нормальном русском языке тоже все совсем не гладко, большинство локалей и исходников приходится от версии к версии править вручную, иначе такие словесные перлы местами получаются - что людям такие сайты сдавать просто совесть не позволяет. Это хоть и проблема "буржуйского" происхождения, но все равно другие CMS (за некоторыми исключениями) с этим как-то более успешно борются.

Кто что думает по этому поводу?
У богатых людей — большая библиотека. У бедных людей — большой телевизор.
 
Polarfox
Выцепляешь, оформляешь, сдаешь в как баг (вот тут если что http://www.8.php-fusion.net/infusions/roadmap/submit.php ), все фиксится в след. релизах. Но щас уже поздновато хотя и может будет еще версия (7.03 или 7.02.06 хз).
Лениво?

А тут писать разве что по локалям следует.

Не знаю что еще тут ответить, вопросы иные будут?
Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле.
Ежели кто забанен за спам, но не считает себя ботом: можете сообщить мне об этом, все будет хорошо.

PolarLab - вход для подопытных
 
Web
Chief
Выцепляешь, оформляешь...

Это и так понятно, суть в том, что если использовать офрелизный сабж в качестве основной CMS для своих проектов - на выходе получаются результаты которые не каждого заказчика удовлетворят, нужно пилить, и пилить много, а обновлять потом еще то удовольствие...
У богатых людей — большая библиотека. У бедных людей — большой телевизор.
 
Polarfox
Ну что не удовлетворяет например? Опять скажешь чпу, кэш аяксы, и прочее?
Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле.
Ежели кто забанен за спам, но не считает себя ботом: можете сообщить мне об этом, все будет хорошо.

PolarLab - вход для подопытных
 
Web
Chief
ЧПУ конечно больная тема, многим пользователям весьма грустно в эпоху 16 ядерных CPU и десятков гигабайт RAM наблюдать в адресной строке этот зоопарк переменных. Современный сайт и современная CMS просто обязаны быть с дружелюбной таксономией, для этого всего-то нужно стать чуть более архитектурными, даже оставаясь не ООП.

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

Кэш небольшим проектам ни к чему, фьюжин и так неплохо летает, а больших проектов я на движке не видел. У rusbody конечно сравнительно неплохо с хостами, но там шестерка из которой выпилили весь хлам и запилили почти все толковое что было года 4 назад на netckе + куча своих доработок. Весьма сложно уже назвать это фьюженом.

Теоретически несложно сделать кэш статики, выкинуть встроенные системы комментирования и рейтинга, повесить Discus и прочие лайки, но необходимости в этом в реальных проектах пока не просто нет. А при 25000 хостов весь структурный и наАяксеный DLE с 5 запросами на морде и полноценным кэшированием - впадает в коллапс.

Вот если кто-нибудь предприимчивый, на основе (например) чего нибудь такого запилит на фьюжене очередную социалку с миллионом хостов, тогда...
У богатых людей — большая библиотека. У бедных людей — большой телевизор.
 
Polarfox
Отлично, не буду больше наводящие вопросы задавать,
я вообще о другом спрашивал просто.
Всегда делайте backup перед изменениями | Указывайте свою версию в подписи/профиле.
Ежели кто забанен за спам, но не считает себя ботом: можете сообщить мне об этом, все будет хорошо.

PolarLab - вход для подопытных
x1 x1
 
Web

Поделиться этой темой
Социальные закладки: Vkontakte Odnoklassniki Mail.ru Facebook Google Tweet This
URL:
BBcode:
HTML:

Перейти на форум: