Brownie\HttpClient\Cookie\CookieList::initList()

Complexity

Conditions	12
Paths	18

Size

Total Lines	44
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	32
CRAP Score	12

Importance

Changes	1
Bugs	0	Features	0

<?php
    protected function initList()
    {
        $cookieList = array();

        foreach (explode("\r\n", trim($this->getInitData())) as $headerLine) {
            if (empty($headerLine)) {
                $cookieList = array();
                continue;
            }
            if ('Set-Cookie:' != substr($headerLine, 0, 11)) {
                continue;
            }
            $cookieParams = explode(':', trim($headerLine), 2);
            $params = $cookieParams[1];

            $cookieParams = array();
            foreach (explode(';', trim($params)) as $index => $param) {
                $param = trim($param);
                $params = explode('=', $param, 2);
                $name = trim($params[0]);
                if (!empty($params[1])) {
                    $value = trim($params[1]);
                }
                if (0 == $index) {
                    $cookieParams['name'] = $name;
                    $cookieParams['value'] = $value;

The variable $value does not seem to be defined for all execution paths leading up to this point.

                    continue;
                }
                switch ($name) {
                    case 'expires':
                    case 'path':
                    case 'domain':
                        $cookieParams[$name] = $value;
                        break;
                    case 'secure':
                    case 'httponly':
                        $cookieParams[$name] = true;
                        break;
                }
            }
            $cookieList[$cookieParams['name']] = new Cookie($cookieParams);
        }

        $this->setList($cookieList);
    }