Coco\SourceWatcher\Watcher\Handler\WebPageHandler

Complexity

Total Complexity

9

Size/Duplication

Total Lines	63
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	read()	0	24	4
A	getDom()	0	3	1
A	__construct()	0	5	1
A	getHtml()	0	3	1
A	setUrl()	0	3	1
A	getUrl()	0	3	1

<?php

namespace Coco\SourceWatcher\Watcher\Handler;

use Coco\SourceWatcher\Core\SourceWatcherException;
use DOMDocument;

/**
 * Class WebPageHandler
 * @package Coco\SourceWatcher\Watcher\Handler
 */
class WebPageHandler implements Handler
{
    protected string $url;

    protected string $html;

    protected DOMDocument $dom;

    public function __construct ( string $url )
    {
        $this->url = $url;
        $this->html = "";
        $this->dom = new DOMDocument();
    }

    public function getUrl () : string
    {
        return $this->url;
    }

    public function setUrl ( string $url ) : void
    {
        $this->url = $url;
    }

    /**
     * @throws SourceWatcherException
     */
    public function read () : void
    {
        if ( empty( $this->url ) ) {
            throw new SourceWatcherException( "A URL must be set before reading" );
        }

        $ch = curl_init();

        $timeout = 5;

        curl_setopt( $ch, CURLOPT_URL, $this->url );
        curl_setopt( $ch, CURLOPT_RETURNTRANSFER, 1 );
        curl_setopt( $ch, CURLOPT_CONNECTTIMEOUT, $timeout );

        $this->html = curl_exec( $ch );

        curl_close( $ch );

        # Create a DOM parser object
        $this->dom = new DOMDocument();

        if ( isset( $this->html ) && $this->html !== "" ) {
            # The @ before the method call suppresses any warnings that loadHTML might throw because of invalid HTML in the page.
            @$this->dom->loadHTML( $this->html );

It seems like you do not handle an error condition for loadHTML(). This can introduce security issues, and is generally not recommended.

        }
    }

    public function getHtml () : string
    {
        return $this->html;
    }

    public function getDom () : DOMDocument
    {
        return $this->dom;
    }
}