SPSS\Sav\Record\Info\VariableAttributes::read()

Complexity

Conditions	4
Paths	4

Size

Total Lines	13
Code Lines	10

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	0
CRAP Score	20

Importance

Changes

0

<?php
    public function read(Buffer $buffer)
    {
        parent::read($buffer);
        $data = $buffer->readString($this->dataSize * $this->dataCount);
        foreach (explode('/', $data) as $item) {

It seems like $data can also be of type false; however, parameter $string of explode() does only seem to accept string, maybe add an additional type check?

            list($var, $value) = explode(':', $item);
            if (preg_match_all('#(.+)\((.+)\)#Uis', $value, $matches)) {
                $this->data[$var] = [];
                foreach ($matches[1] as $key => $val) {
                    $this->data[$var][$val] = trim(trim($matches[2][$key]), '\'');
                }
            } else {
                $this->data[$var] = $value;
            }