Перейти к содержанию
Официальный форум поддержки Simpla

Рекомендуемые сообщения

Как по мне то слабовато...
Бегло пробежался взглядом по исх коду...


Из увиденного вот замечания:


1) Инклудится все. Даже то что может не использоваться...

 

2) Не понимаю логики в наличии extends

class uCatalogInterface extends fcms {
    
private $fcms;
function uCatalogInterface($fcms = object) {
    $this->fcms = $fcms;
}


3) Нет класса для работы с БД

$sql = "select * from ajaxcontrollers where mode='admin' and level <= ".$this->user_level;
$res = mysql_query($sql);

все выполняется нативными функциями. Думаю тут разжевывать - нечего

 

4) Много что зависит от настроек прописанных в таблицах БД. Что если она повредится? 

Ссылка на сообщение
Поделиться на другие сайты

 

3) Нет класса для работы с БД

$sql = "select * from ajaxcontrollers where mode='admin' and level <= ".$this->user_level;
$res = mysql_query($sql);
все выполняется нативными функциями. Думаю тут разжевывать - нечего

 

5) Кроме уже отмеченного, используются функции mysql. Через 2-3 года используемые эти функции просто перестанут поддерживаться новыми версиями PHP, и владельцы сайтов на этой CMS получат сюрпризы. И как водится, это произойдет в тот момент, когда магазин начнет набирать настоящие обороты.

Чтобы работало, надо изменять в сотне файлов, что не слишком хорошо.

 

Так что использовать систему КРАЙНЕ НЕ РЕКОМЕНДУЕТСЯ...

Ссылка на сообщение
Поделиться на другие сайты
$sql = "select * from ajaxcontrollers where mode='admin' and level <= ".$this->user_level;
$res = mysql_query($sql);
 

Ну это принимается - исправим со временем.

 

Остальные хоть и минусы - но они притянутые за уши минусы.

 

Работает быстро и ресурсов не есть - для нас это самое главное.

Ссылка на сообщение
Поделиться на другие сайты
  • 1 год спустя...

А чем это лучше \ хуже Simpla CMS ? :rolleyes:

 

Рекомендую Active Record для работы с БД, очень крутая вещь.

http://www.internet-technologies.ru/articles/article_1505.html

Использую в своей CMSке.

Изменено пользователем a13x
Ссылка на сообщение
Поделиться на другие сайты

Присоединяйтесь к обсуждению

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

Гость
Ответить в этой теме...

×   Вставлено с форматированием.   Вставить как обычный текст

  Разрешено использовать не более 75 эмодзи.

×   Ваша ссылка была автоматически встроена.   Отображать как обычную ссылку

×   Ваш предыдущий контент был восстановлен.   Очистить редактор

×   Вы не можете вставлять изображения напрямую. Загружайте или вставляйте изображения по ссылке.

Загрузка...
×
×
  • Создать...