These are chat archives for PHPixie/Hotline

10th
Nov 2015
Vadim Meling
@Linfuby
Nov 10 2015 10:59
@dracony Добрый день. Закинул PullRequest для Slice с PHPDocs. Проверь, пожалуйста все ли там правильно...
Roman Tsiupa
@dracony
Nov 10 2015 13:23
@return array|string|null
mozhno prosto @return mixed
так как там в принципе все что угодно может быть
также
Vadim Meling
@Linfuby
Nov 10 2015 13:25
Ну да, как-то упустил
Roman Tsiupa
@dracony
Nov 10 2015 13:25
"@throws \PHPixie\Slice\Exception" в местах где он реально не бросаться
    /**
     * @throws \PHPixie\Slice\Exception
     */
    public function remove($path = null)
    {
        if ($path === null) {
            $this->data = array();
            return;
        }

        list($path, $key) = $this->splitPath($path);
        $parent = &$this->findGroup($path);

        if($parent !== null) {
            unset($parent[$key]);
        }
    }
в большинстве документаторов
достаточно указать что findGroup или splitPath может бросиь исключение
и они автоматически поймут что все методы которые их вызывают (типа remove() ) могут его бросить
Vadim Meling
@Linfuby
Nov 10 2015 13:28
У меня то только array или string. О других не подумал :)
Согласен, исключение по факту лишнее.
Roman Tsiupa
@dracony
Nov 10 2015 13:28
в другом случае в докблоке например контроллера которые трогает сразу куча библиотек был бы целый список @throws наверзу
принял =)
вечером пройдусь подправлю эти места чуть чуть) но это детали уже =)
Спасибо =)
доделаю нестед сетс займусь докблоками к всем компонентам)