PiedWeb\SeoPocketCrawler\Recorder::record()

Complexity

Conditions	2
Paths	2

Size

Total Lines	12
Code Lines	6

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2.0438

Importance

Changes

0

<?php
    public function record(array $urls)
    {
        $fp = fopen($this->folder.'/index.csv', 'w');

        $header = array_keys(get_object_vars(array_values($urls)[0]));
        fputcsv($fp, $header);

It seems like $fp can also be of type false; however, parameter $handle of fputcsv() does only seem to accept resource, maybe add an additional type check?

        foreach ($urls as $url) {
            fputcsv($fp, get_object_vars($url));
        }

        fclose($fp);

It seems like $fp can also be of type false; however, parameter $handle of fclose() does only seem to accept resource, maybe add an additional type check?

    }