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

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

2019-03-11T13:43:24Z

admin пишет:

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

IDE ругается

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

2019-03-11T13:11:50Z

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

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

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

2019-03-11T10:08:37Z

admin пишет:

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

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

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

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

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

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

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

2019-03-10T12:15:39Z

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

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

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

2019-03-07T22:10:17Z

Вот типичный пример наследования \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 .