Патч-обновление FFCMS 2.0.4

[devlev]

Вы бы ссылки в профиле починили бы http://ffcms.ru/ru/user/id1/bookmarks
А то ваша обертка с редеректом че то не работает

http://ffcms.ru/ru/api.php?iface=front&object=redirect&url=http://ffcms.ru/static/doc.html
http://ffcms.ru/ru/api.php?iface=front&object=redirect&url=http://ffcms.ru/
И тем более если уж делать обертку через редерект ссылок, то на свой же сайт можно ссылки и не оборачивать

[zenn]

Да, раздел закладок будет существенно переработан, спасибо за напоминание.

[Khaperets]

Вид стал приятнее, молодцы.

Попробуйте для singleton'a (сразу бросилось в глаза) такой вид:

protected static $instance; // по умолчанию имеет значение «null»

public static function getInstance() {
    return self::$instance ? : new self; // так вроде изысканней)
}

Кстати, когда будете поддерживать php >= 5.4 — можно перенести реализацию одиночки в типаж (trait).
Code-style можно было бы поправить, советую придерживаться PSR-стандартов

[Fesor]

ваше «изысканней» не работает, так как getInstance будет всегда возвращать new self (даже не new statc). Вообще стоит отказываться от идеи использования сингелтонов в пользу нормального DI.

[Khaperets]

Ах, ну да… Могли бы подсказать, что забыл написать «self::$instance = new self».
codepad.viper-7.com/n3FkPf

[Fesor]

Ну тогда еще расстрою — ваш вариант все-равно не будет работать при наследовании. И да, вариант с if-ом намного более элегантен.

[Khaperets]

Для того, чтобы унаследовать синглтон просто используется «get_called_class»
codepad.viper-7.com/PWUYyz

[Fesor]

А теперь сравните:

$className = get_called_class();

return self::$instance ? : self::$instance = new $className;

и

if (!static::$instance) {
    static::$instance = new static;
}

return static::$instance;

[Khaperets]

Если через «static», то сравните:

return self::$instance ? : self::$instance = new static;

и ваш:

if (!static::$instance) {
    static::$instance = new static;
}

return static::$instance;

Пример: codepad.viper-7.com/RvU34D

[DeusModus]

У вас любовь с тернарным оператором?

[dobriykot]

Отличная работа над ошибками! Продолжайте в том же духе.

[AlexPTS]

Singletone можно вынести в trait вместо наследования, правда php 5.4 надо будет минимальную версию иметь в системных требованиях.

[Fesor]

А еще можно вообще отказаться от сингелтона и перестроить архитектуру CMS. Но тогда придется ее выкинуть и писать все с нуля.

[AlexPTS]

Да, DI. Или модули более независимыми делать, чтобы итеративно все же можно было поменять частями все.