silverstripe / silverstripe-externallinks

src/Tasks/CurlLinkChecker.php

Indentation +63 added lines, -63 removed lines

@@ -11,74 +11,74 @@
  */
 class CurlLinkChecker implements LinkChecker
 {
-    use Configurable;
+	use Configurable;
 
-    /**
-     * If we want to follow redirects a 301 http code for example
-     * Set via YAML file
-     *
-     * @config
-     * @var boolean
-     */
-    private static $follow_location = false;
+	/**
+	 * If we want to follow redirects a 301 http code for example
+	 * Set via YAML file
+	 *
+	 * @config
+	 * @var boolean
+	 */
+	private static $follow_location = false;
 
-    /**
-     * If we want to bypass the cache
-     * Set via YAML file
-     *
-     * @config
-     * @var boolean
-     */
-    private static $bypass_cache = false;
+	/**
+	 * If we want to bypass the cache
+	 * Set via YAML file
+	 *
+	 * @config
+	 * @var boolean
+	 */
+	private static $bypass_cache = false;
 
-    /**
-     * Return cache
-     *
-     * @return CacheInterface
-     */
-    protected function getCache()
-    {
-        return Injector::inst()->get(CacheInterface::class . '.CurlLinkChecker');
-    }
+	/**
+	 * Return cache
+	 *
+	 * @return CacheInterface
+	 */
+	protected function getCache()
+	{
+		return Injector::inst()->get(CacheInterface::class . '.CurlLinkChecker');
+	}
 
-    /**
-     * Determine the http status code for a given link
-     *
-     * @param string $href URL to check
-     * @return int HTTP status code, or null if not checkable (not a link)
-     */
-    public function checkLink($href)
-    {
-        // Skip non-external links
-        if (!preg_match('/^https?[^:]*:\/\//', $href)) {
-            return null;
-        }
+	/**
+	 * Determine the http status code for a given link
+	 *
+	 * @param string $href URL to check
+	 * @return int HTTP status code, or null if not checkable (not a link)
+	 */
+	public function checkLink($href)
+	{
+		// Skip non-external links
+		if (!preg_match('/^https?[^:]*:\/\//', $href)) {
+			return null;
+		}
 
-        $cacheKey = md5($href);
-        if (!$this->config()->get('bypass_cache')) {
-            // Check if we have a cached result
-            $result = $this->getCache()->get($cacheKey, false);
-            if ($result !== false) {
-                return $result;
-            }
-        }
+		$cacheKey = md5($href);
+		if (!$this->config()->get('bypass_cache')) {
+			// Check if we have a cached result
+			$result = $this->getCache()->get($cacheKey, false);
+			if ($result !== false) {
+				return $result;
+			}
+		}
 
-        // No cached result so just request
-        $handle = curl_init($href);
-        curl_setopt($handle, CURLOPT_RETURNTRANSFER, true);
-        if ($this->config()->get('follow_location')) {
-            curl_setopt($handle, CURLOPT_FOLLOWLOCATION, true);
-        }
-        curl_setopt($handle, CURLOPT_CONNECTTIMEOUT, 5);
-        curl_setopt($handle, CURLOPT_TIMEOUT, 10);
-        curl_exec($handle);
-        $httpCode = curl_getinfo($handle, CURLINFO_HTTP_CODE);
-        curl_close($handle);
+		// No cached result so just request
+		$handle = curl_init($href);
+		curl_setopt($handle, CURLOPT_RETURNTRANSFER, true);
+		if ($this->config()->get('follow_location')) {
+			curl_setopt($handle, CURLOPT_FOLLOWLOCATION, true);
+		}
+		curl_setopt($handle, CURLOPT_CONNECTTIMEOUT, 5);
+		curl_setopt($handle, CURLOPT_TIMEOUT, 10);
+		curl_exec($handle);
+		$httpCode = curl_getinfo($handle, CURLINFO_HTTP_CODE);
+		curl_close($handle);
 
-        if (!$this->config()->get('bypass_cache')) {
-            // Cache result
-            $this->getCache()->set($cacheKey, $httpCode);
-        }
-        return $httpCode;
-    }
+		if (!$this->config()->get('bypass_cache')) {
+			// Cache result
+			$this->getCache()->set($cacheKey, $httpCode);
+		}
+		return $httpCode;
+	}
 }

src/Model/BrokenExternalPageTrackStatus.php

Indentation +149 added lines, -149 removed lines

@@ -17,153 +17,153 @@
  */
 class BrokenExternalPageTrackStatus extends DataObject implements i18nEntityProvider
 {
-    private static $table_name = 'BrokenExternalPageTrackStatus';
-
-    private static $db = array(
-        'Status' => 'Enum("Completed, Running", "Running")',
-        'JobInfo' => 'Varchar(255)'
-    );
-
-    private static $has_many = array(
-        'TrackedPages' => BrokenExternalPageTrack::class,
-        'BrokenLinks' => BrokenExternalLink::class
-    );
-
-    /**
-     * Get the latest track status
-     *
-     * @return BrokenExternalPageTrackStatus
-     */
-    public static function get_latest()
-    {
-        return self::get()
-            ->sort('ID', 'DESC')
-            ->first();
-    }
-
-    /**
-     * Returns the list of provided translations for this object
-     *
-     * @return array
-     */
-    public function provideI18nEntities()
-    {
-        return [
-            __CLASS__ . '.SINGULARNAME' => 'Broken External Page Track Status',
-            __CLASS__ . '.PLURALNAME' => 'Broken External Page Track Statuses',
-            __CLASS__ . '.PLURALS' => [
-              'one' => 'A Broken External Page Track Status',
-              'other' => '{count} Broken External Page Track Statuses',
-            ],
-        ];
-    }
-
-    /**
-     * Gets the list of Pages yet to be checked
-     *
-     * @return DataList
-     */
-    public function getIncompletePageList()
-    {
-        $pageIDs = $this
-            ->getIncompleteTracks()
-            ->column('PageID');
-        if ($pageIDs) {
-            return Versioned::get_by_stage(SiteTree::class, 'Stage')
-            ->byIDs($pageIDs);
-        }
-    }
-
-    /**
-     * Get the list of incomplete BrokenExternalPageTrack
-     *
-     * @return DataList
-     */
-    public function getIncompleteTracks()
-    {
-        return $this
-            ->TrackedPages()
-            ->filter('Processed', 0);
-    }
-
-    /**
-     * Get total pages count
-     *
-     * @return int
-     */
-    public function getTotalPages()
-    {
-        return $this->TrackedPages()->count();
-    }
-
-    /**
-     * Get completed pages count
-     *
-     * @return int
-     */
-    public function getCompletedPages()
-    {
-        return $this
-            ->TrackedPages()
-            ->filter('Processed', 1)
-            ->count();
-    }
-
-    /**
-     * Returns the latest run, or otherwise creates a new one
-     *
-     * @return BrokenExternalPageTrackStatus
-     */
-    public static function get_or_create()
-    {
-        // Check the current status
-        $status = self::get_latest();
-        if ($status && $status->Status == 'Running') {
-            $status->updateStatus();
-            return $status;
-        }
-
-        return self::create_status();
-    }
-
-    /**
-     * Create and prepare a new status
-     *
-     * @return BrokenExternalPageTrackStatus
-     */
-    public static function create_status()
-    {
-        // If the script is to be started create a new status
-        $status = self::create();
-        $status->updateJobInfo('Creating new tracking object');
-
-        // Setup all pages to test
-        $pageIDs = Versioned::get_by_stage(SiteTree::class, 'Stage')
-            ->column('ID');
-        foreach ($pageIDs as $pageID) {
-            $trackPage = BrokenExternalPageTrack::create();
-            $trackPage->PageID = $pageID;
-            $trackPage->StatusID = $status->ID;
-            $trackPage->write();
-        }
-
-        return $status;
-    }
-
-    public function updateJobInfo($message)
-    {
-        $this->JobInfo = $message;
-        $this->write();
-    }
-
-    /**
-     * Self check status
-     */
-    public function updateStatus()
-    {
-        if ($this->CompletedPages == $this->TotalPages) {
-            $this->Status = 'Completed';
-            $this->updateJobInfo('Setting to completed');
-        }
-    }
+	private static $table_name = 'BrokenExternalPageTrackStatus';
+
+	private static $db = array(
+		'Status' => 'Enum("Completed, Running", "Running")',
+		'JobInfo' => 'Varchar(255)'
+	);
+
+	private static $has_many = array(
+		'TrackedPages' => BrokenExternalPageTrack::class,
+		'BrokenLinks' => BrokenExternalLink::class
+	);
+
+	/**
+	 * Get the latest track status
+	 *
+	 * @return BrokenExternalPageTrackStatus
+	 */
+	public static function get_latest()
+	{
+		return self::get()
+			->sort('ID', 'DESC')
+			->first();
+	}
+
+	/**
+	 * Returns the list of provided translations for this object
+	 *
+	 * @return array
+	 */
+	public function provideI18nEntities()
+	{
+		return [
+			__CLASS__ . '.SINGULARNAME' => 'Broken External Page Track Status',
+			__CLASS__ . '.PLURALNAME' => 'Broken External Page Track Statuses',
+			__CLASS__ . '.PLURALS' => [
+			  'one' => 'A Broken External Page Track Status',
+			  'other' => '{count} Broken External Page Track Statuses',
+			],
+		];
+	}
+
+	/**
+	 * Gets the list of Pages yet to be checked
+	 *
+	 * @return DataList
+	 */
+	public function getIncompletePageList()
+	{
+		$pageIDs = $this
+			->getIncompleteTracks()
+			->column('PageID');
+		if ($pageIDs) {
+			return Versioned::get_by_stage(SiteTree::class, 'Stage')
+			->byIDs($pageIDs);
+		}
+	}
+
+	/**
+	 * Get the list of incomplete BrokenExternalPageTrack
+	 *
+	 * @return DataList
+	 */
+	public function getIncompleteTracks()
+	{
+		return $this
+			->TrackedPages()
+			->filter('Processed', 0);
+	}
+
+	/**
+	 * Get total pages count
+	 *
+	 * @return int
+	 */
+	public function getTotalPages()
+	{
+		return $this->TrackedPages()->count();
+	}
+
+	/**
+	 * Get completed pages count
+	 *
+	 * @return int
+	 */
+	public function getCompletedPages()
+	{
+		return $this
+			->TrackedPages()
+			->filter('Processed', 1)
+			->count();
+	}
+
+	/**
+	 * Returns the latest run, or otherwise creates a new one
+	 *
+	 * @return BrokenExternalPageTrackStatus
+	 */
+	public static function get_or_create()
+	{
+		// Check the current status
+		$status = self::get_latest();
+		if ($status && $status->Status == 'Running') {
+			$status->updateStatus();
+			return $status;
+		}
+
+		return self::create_status();
+	}
+
+	/**
+	 * Create and prepare a new status
+	 *
+	 * @return BrokenExternalPageTrackStatus
+	 */
+	public static function create_status()
+	{
+		// If the script is to be started create a new status
+		$status = self::create();
+		$status->updateJobInfo('Creating new tracking object');
+
+		// Setup all pages to test
+		$pageIDs = Versioned::get_by_stage(SiteTree::class, 'Stage')
+			->column('ID');
+		foreach ($pageIDs as $pageID) {
+			$trackPage = BrokenExternalPageTrack::create();
+			$trackPage->PageID = $pageID;
+			$trackPage->StatusID = $status->ID;
+			$trackPage->write();
+		}
+
+		return $status;
+	}
+
+	public function updateJobInfo($message)
+	{
+		$this->JobInfo = $message;
+		$this->write();
+	}
+
+	/**
+	 * Self check status
+	 */
+	public function updateStatus()
+	{
+		if ($this->CompletedPages == $this->TotalPages) {
+			$this->Status = 'Completed';
+			$this->updateJobInfo('Setting to completed');
+		}
+	}
 }

tests/Model/BrokenExternalLinkTest.php

Indentation +24 added lines, -24 removed lines

@@ -7,29 +7,29 @@
 
 class BrokenExternalLinkTest extends SapphireTest
 {
-    /**
-     * @param int $httpCode
-     * @param string $expected
-     * @dataProvider httpCodeProvider
-     */
-    public function testGetHTTPCodeDescription($httpCode, $expected)
-    {
-        $link = new BrokenExternalLink();
-        $link->HTTPCode = $httpCode;
-        $this->assertSame($expected, $link->getHTTPCodeDescription());
-    }
+	/**
+	 * @param int $httpCode
+	 * @param string $expected
+	 * @dataProvider httpCodeProvider
+	 */
+	public function testGetHTTPCodeDescription($httpCode, $expected)
+	{
+		$link = new BrokenExternalLink();
+		$link->HTTPCode = $httpCode;
+		$this->assertSame($expected, $link->getHTTPCodeDescription());
+	}
 
-    /**
-     * @return array[]
-     */
-    public function httpCodeProvider()
-    {
-        return [
-            [200, '200 (OK)'],
-            [302, '302 (Found)'],
-            [404, '404 (Not Found)'],
-            [500, '500 (Internal Server Error)'],
-            [789, '789 (Unknown Response Code)'],
-        ];
-    }
+	/**
+	 * @return array[]
+	 */
+	public function httpCodeProvider()
+	{
+		return [
+			[200, '200 (OK)'],
+			[302, '302 (Found)'],
+			[404, '404 (Not Found)'],
+			[500, '500 (Internal Server Error)'],
+			[789, '789 (Unknown Response Code)'],
+		];
+	}
 }