Форум ReadyScript — Конфиг модуля

Re: Конфиг модуля

null@example.com (EnsoStudio) — Mon, 11 Mar 2019 13:43:24 +0000

admin пишет:

Это не баг (ничего не ломает)

IDE ругается

Re: Конфиг модуля

null@example.com (admin) — Mon, 11 Mar 2019 13:11:50 +0000

Это не баг (ничего не ломает) и не фича (ничего не привносит).

Конечно, все должно быть по фен шую. )

Re: Конфиг модуля

null@example.com (EnsoStudio) — Mon, 11 Mar 2019 10:08:37 +0000

admin пишет:

1. Не смог найти информацию о том, что отсутствие public перед функцией дает notice.

Сейчас не могу найти, но точно помню что в одном из последнем релизов в strict modе попадалось.

2. Так исторически сложилось.

Это не баг, а фича?

Но в наследнике все равно всегда можно сделать так

а какой в этом смысл именно в таком варианте? и зачем рекомендовать некорретный код, многие разработчики копируют ваш код не задумываясь, например, модуль "Генератор модулей (codegen)" генерируют болванку с такой же ошибкой.

Re: Конфиг модуля

null@example.com (admin) — Sun, 10 Mar 2019 12:15:39 +0000

1. Не смог найти информацию о том, что отсутствие public перед функцией дает notice.
2. Так исторически сложилось. Это абсолютно ни на что не влияет, так как конфигурационные файлы никогда не наследуются. Но в наследнике все равно всегда можно сделать так:

function _init()
{
    parent::_init();
    $this->getPropertyIterator()->append(....);
}

Конфиг модуля

null@example.com (EnsoStudio) — Thu, 07 Mar 2019 22:10:17 +0000

Вот типичный пример наследования \RS\Orm\ConfigObject взятый из модуля shop:

class File extends \RS\Orm\ConfigObject
{
    function _init()
    {
        parent::_init()->append(array(...));
    }
}

1. Почему в публичных методах опущен public? Насколько я помню в последних версиях PHP это вызывает notice.
2. Почему _init возвращает void, а не \RS\Orm\PropertyIterator? Это же явная ошибка + усложняе отладку в IDE .