Anax\Model\GeoTag::getWeatherMultiCurl()

Complexity

Conditions	8
Paths	32

Size

Total Lines	56
Code Lines	32

Duplication

Lines	0
Ratio	0 %

Importance

Changes	1
Bugs	0	Features	0

<?php
    public function getWeatherMultiCurl($when, $lat, $long) : array
    {
        $dates = [];
        $now = time();

        if ($when == "past") {
            for ($i = 0; $i < 30; $i++) {
                // 24h = 86400 unix time
                $now -= 86400;
                $dates[] = $now;
            }
        } else {
            for ($i = 0; $i < 7; $i++) {
                // 24h = 86400 unix time
                $now += 86400;
                $dates[] = $now;
            }
        }

        $url = "https://api.darksky.net/forecast/{$this->config->config}/{$lat},{$long}";

        $options = [
            CURLOPT_RETURNTRANSFER => true,
        ];

        $mh = curl_multi_init();
        $chAll = [];
        foreach ($dates as $day) {
            $ch = curl_init("$url,{$day}?lang=sv&units=si");
            curl_setopt_array($ch, $options);

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

            curl_multi_add_handle($mh, $ch);

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

            $chAll[] = $ch;
        }

        // execute all queries simultaneously and countinue when all are complete
        $running = null;
        do {
            curl_multi_exec($mh, $running);
        } while ($running);

        // Close handle
        foreach ($chAll as $ch) {
            curl_multi_remove_handle($mh, $ch);
        }
        curl_multi_close($mh);

        //req are done, access results
        $response = [];
        foreach ($chAll as $ch) {
            $data = curl_multi_getcontent($ch);
            $response[] = json_decode($data, true);

            // $weather[] = $response->currently->summary;
            // $temp[] = $response->currently->temperature;
        }
        return $response;
    }