crocodicstudio\crudbooster\helpers\FCM::send()

Complexity

Conditions	3
Paths	2

Size

Total Lines	21
Code Lines	16

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function send() {
        $url = 'https://fcm.googleapis.com/fcm/send';
        $headers = [
            'Authorization:key='.cbConfig('GOOGLE_FCM_SERVER_KEY'),
            'Content-Type:application/json',
        ];

        if(static::$title && $this->message) {

Since $title is declared private, accessing it with static will lead to errors in possible sub-classes; you can either use self, or increase the visibility of $title to at least protected.

            $ch = curl_init($url);
            curl_setopt($ch, CURLOPT_HTTPHEADER, $headers);

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

            curl_setopt($ch, CURLOPT_SSL_VERIFYHOST, 0);
            curl_setopt($ch, CURLOPT_SSL_VERIFYPEER, 0);
            curl_setopt($ch, CURLOPT_POST, 1);
            curl_setopt($ch, CURLOPT_POSTFIELDS, json_encode($this->fields));
            curl_setopt($ch, CURLOPT_RETURNTRANSFER, true);
            $result = curl_exec($ch);

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

            curl_close($ch);

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

            return $result;
        }
        
        return null;
    }
}