Coco\SourceWatcher\Watcher\Communicator\SlackCommunicator::send()

Complexity

Conditions	9
Paths	5

Size

Total Lines	25
Code Lines	14

Duplication

Lines	0
Ratio	0 %

Importance

Changes

0

<?php
    public function send ()
    {
        if ( $this->webHookUrl == null || $this->webHookUrl == "" ) {
            throw new Exception( "A web hook url must be defined first." );
        }

        if ( $this->method == null || $this->method == "" ) {
            throw new Exception( "A method must be defined first." );
        }

        if ( $this->contentType == null || $this->contentType == "" ) {
            throw new Exception( "A content type be defined first." );
        }

        if ( $this->data == null || $this->data == "" ) {
            throw new Exception( "Data must be defined first." );
        }

        $ch = curl_init( $this->webHookUrl );
        curl_setopt( $ch, CURLOPT_CUSTOMREQUEST, $this->method );

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_HTTPHEADER, array( $this->contentType, 'Content-Length: ' . strlen( $this->data ) ) );
        curl_setopt( $ch, CURLOPT_POSTFIELDS, $this->data );
        curl_setopt( $ch, CURLOPT_RETURNTRANSFER, true );

        return 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?

    }