silverstripe / silverstripe-externallinks

tests/ExternalLinksTestPage.php

Indentation +4 added lines, -4 removed lines

@@ -7,9 +7,9 @@
 
 class ExternalLinksTestPage extends Page implements TestOnly
 {
-    private static $table_name = 'ExternalLinksTestPage';
+	private static $table_name = 'ExternalLinksTestPage';
 
-    private static $db = array(
-        'ExpectedContent' => 'HTMLText'
-    );
+	private static $db = array(
+		'ExpectedContent' => 'HTMLText'
+	);
 }

src/Model/BrokenExternalPageTrack.php

Indentation +19 added lines, -19 removed lines

@@ -13,27 +13,27 @@
  */
 class BrokenExternalPageTrack extends DataObject
 {
-    private static $table_name = 'BrokenExternalPageTrack';
+	private static $table_name = 'BrokenExternalPageTrack';
 
-    private static $db = array(
-        'Processed' => 'Boolean'
-    );
+	private static $db = array(
+		'Processed' => 'Boolean'
+	);
 
-    private static $has_one = array(
-        'Page' => SiteTree::class,
-        'Status' => BrokenExternalPageTrackStatus::class
-    );
+	private static $has_one = array(
+		'Page' => SiteTree::class,
+		'Status' => BrokenExternalPageTrackStatus::class
+	);
 
-    private static $has_many = array(
-        'BrokenLinks' => BrokenExternalLink::class
-    );
+	private static $has_many = array(
+		'BrokenLinks' => BrokenExternalLink::class
+	);
 
-    /**
-     * @return SiteTree
-     */
-    public function Page()
-    {
-        return Versioned::get_by_stage(SiteTree::class, 'Stage')
-            ->byID($this->PageID);
-    }
+	/**
+	 * @return SiteTree
+	 */
+	public function Page()
+	{
+		return Versioned::get_by_stage(SiteTree::class, 'Stage')
+			->byID($this->PageID);
+	}
 }

src/Tasks/LinkChecker.php

Indentation +7 added lines, -7 removed lines

@@ -8,11 +8,11 @@
 interface LinkChecker
 {
     
-    /**
-     * 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);
+	/**
+	 * 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);
 }

src/Tasks/CurlLinkChecker.php

Indentation +43 added lines, -43 removed lines

@@ -10,47 +10,47 @@
 class CurlLinkChecker implements LinkChecker
 {
 
-    /**
-     * Return cache
-     *
-     * @return Zend_Cache_Frontend
-     */
-    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;
-        }
-
-        // Check if we have a cached result
-        $cacheKey = md5($href);
-        $result = $this->getCache()->get($cacheKey);
-        if ($result !== false) {
-            return $result;
-        }
-
-        // No cached result so just request
-        $handle = curl_init($href);
-        curl_setopt($handle, CURLOPT_RETURNTRANSFER, 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);
-
-        // Cache result
-        $this->getCache()->set($httpCode, $cacheKey);
-        return $httpCode;
-    }
+	/**
+	 * Return cache
+	 *
+	 * @return Zend_Cache_Frontend
+	 */
+	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;
+		}
+
+		// Check if we have a cached result
+		$cacheKey = md5($href);
+		$result = $this->getCache()->get($cacheKey);
+		if ($result !== false) {
+			return $result;
+		}
+
+		// No cached result so just request
+		$handle = curl_init($href);
+		curl_setopt($handle, CURLOPT_RETURNTRANSFER, 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);
+
+		// Cache result
+		$this->getCache()->set($httpCode, $cacheKey);
+		return $httpCode;
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
      */
     protected function getCache()
     {
-        return Injector::inst()->get(CacheInterface::class . '.CurlLinkChecker');
+        return Injector::inst()->get(CacheInterface::class.'.CurlLinkChecker');
     }
 
     /**

src/Jobs/CheckExternalLinksJob.php

Indentation +27 added lines, -27 removed lines

@@ -7,7 +7,7 @@ 
 use SilverStripe\ExternalLinks\Tasks\CheckExternalLinksTask;
 
 if (!class_exists(AbstractQueuedJob::class)) {
-    return;
+	return;
 }
 
 /**
@@ -17,30 +17,30 @@ 
 class CheckExternalLinksJob extends AbstractQueuedJob implements QueuedJob
 {
 
-    public function getTitle()
-    {
-        return _t(__CLASS__ . '.TITLE', 'Checking for external broken links');
-    }
-
-    public function getJobType()
-    {
-        return QueuedJob::QUEUED;
-    }
-
-    public function getSignature()
-    {
-        return md5(get_class($this));
-    }
-
-    /**
-     * Check an individual page
-     */
-    public function process()
-    {
-        $task = CheckExternalLinksTask::create();
-        $track = $task->runLinksCheck(1);
-        $this->currentStep = $track->CompletedPages;
-        $this->totalSteps = $track->TotalPages;
-        $this->isComplete = $track->Status === 'Completed';
-    }
+	public function getTitle()
+	{
+		return _t(__CLASS__ . '.TITLE', 'Checking for external broken links');
+	}
+
+	public function getJobType()
+	{
+		return QueuedJob::QUEUED;
+	}
+
+	public function getSignature()
+	{
+		return md5(get_class($this));
+	}
+
+	/**
+	 * Check an individual page
+	 */
+	public function process()
+	{
+		$task = CheckExternalLinksTask::create();
+		$track = $task->runLinksCheck(1);
+		$this->currentStep = $track->CompletedPages;
+		$this->totalSteps = $track->TotalPages;
+		$this->isComplete = $track->Status === 'Completed';
+	}
 }

Spacing +1 added lines, -1 removed lines

@@ -19,7 +19,7 @@
 
     public function getTitle()
     {
-        return _t(__CLASS__ . '.TITLE', 'Checking for external broken links');
+        return _t(__CLASS__.'.TITLE', 'Checking for external broken links');
     }
 
     public function getJobType()

tests/Stubs/PretendLinkChecker.php

Indentation +20 added lines, -20 removed lines

@@ -5,24 +5,24 @@
 
 class PretendLinkChecker implements LinkChecker
 {
-    public function checkLink($href)
-    {
-        switch ($href) {
-            case 'http://www.working.com':
-                return 200;
-            case 'http://www.broken.com':
-                return 403;
-            case 'http://www.nodomain.com':
-                return 0;
-            case '/internal/link':
-            case '[sitetree_link,id=9999]':
-            case 'home':
-            case 'broken-internal':
-            case '[sitetree_link,id=1]':
-                return null;
-            case 'http://www.broken.com/url/thing':
-            default:
-                return 404;
-        }
-    }
+	public function checkLink($href)
+	{
+		switch ($href) {
+			case 'http://www.working.com':
+				return 200;
+			case 'http://www.broken.com':
+				return 403;
+			case 'http://www.nodomain.com':
+				return 0;
+			case '/internal/link':
+			case '[sitetree_link,id=9999]':
+			case 'home':
+			case 'broken-internal':
+			case '[sitetree_link,id=1]':
+				return null;
+			case 'http://www.broken.com/url/thing':
+			default:
+				return 404;
+		}
+	}
 }

tests/ExternalLinksTest.php

Indentation +80 added lines, -80 removed lines

@@ -16,95 +16,95 @@
 class ExternalLinksTest extends SapphireTest
 {
 
-    protected static $fixture_file = 'ExternalLinksTest.yml';
+	protected static $fixture_file = 'ExternalLinksTest.yml';
 
-    protected static $extra_dataobjects = array(
-        ExternalLinksTestPage::class
-    );
+	protected static $extra_dataobjects = array(
+		ExternalLinksTestPage::class
+	);
 
-    protected function setUp()
-    {
-        parent::setUp();
+	protected function setUp()
+	{
+		parent::setUp();
 
-        // Stub link checker
-        $checker = new PretendLinkChecker;
-        Injector::inst()->registerService($checker, LinkChecker::class);
-    }
+		// Stub link checker
+		$checker = new PretendLinkChecker;
+		Injector::inst()->registerService($checker, LinkChecker::class);
+	}
 
-    public function testLinks()
-    {
-        // Run link checker
-        $task = CheckExternalLinksTask::create();
-        $task->setSilent(true); // Be quiet during the test!
-        $task->runLinksCheck();
+	public function testLinks()
+	{
+		// Run link checker
+		$task = CheckExternalLinksTask::create();
+		$task->setSilent(true); // Be quiet during the test!
+		$task->runLinksCheck();
 
-        // Get all links checked
-        $status = BrokenExternalPageTrackStatus::get_latest();
-        $this->assertEquals('Completed', $status->Status);
-        $this->assertEquals(5, $status->TotalPages);
-        $this->assertEquals(5, $status->CompletedPages);
+		// Get all links checked
+		$status = BrokenExternalPageTrackStatus::get_latest();
+		$this->assertEquals('Completed', $status->Status);
+		$this->assertEquals(5, $status->TotalPages);
+		$this->assertEquals(5, $status->CompletedPages);
 
-        // Check all pages have had the correct HTML adjusted
-        for ($i = 1; $i <= 5; $i++) {
-            $page = $this->objFromFixture(ExternalLinksTestPage::class, 'page'.$i);
-            $this->assertNotEmpty($page->Content);
-            $this->assertEquals(
-                $page->ExpectedContent,
-                $page->Content,
-                "Assert that the content of page{$i} has been updated"
-            );
-        }
+		// Check all pages have had the correct HTML adjusted
+		for ($i = 1; $i <= 5; $i++) {
+			$page = $this->objFromFixture(ExternalLinksTestPage::class, 'page'.$i);
+			$this->assertNotEmpty($page->Content);
+			$this->assertEquals(
+				$page->ExpectedContent,
+				$page->Content,
+				"Assert that the content of page{$i} has been updated"
+			);
+		}
 
-        // Check that the correct report of broken links is generated
-        $links = $status
-            ->BrokenLinks()
-            ->sort('Link');
+		// Check that the correct report of broken links is generated
+		$links = $status
+			->BrokenLinks()
+			->sort('Link');
 
-        $this->assertEquals(4, $links->count());
-        $this->assertEquals(
-            array(
-                'http://www.broken.com',
-                'http://www.broken.com/url/thing',
-                'http://www.broken.com/url/thing',
-                'http://www.nodomain.com'
-            ),
-            array_values($links->map('ID', 'Link')->toArray())
-        );
+		$this->assertEquals(4, $links->count());
+		$this->assertEquals(
+			array(
+				'http://www.broken.com',
+				'http://www.broken.com/url/thing',
+				'http://www.broken.com/url/thing',
+				'http://www.nodomain.com'
+			),
+			array_values($links->map('ID', 'Link')->toArray())
+		);
 
-        // Check response codes are correct
-        $expected = array(
-            'http://www.broken.com' => 403,
-            'http://www.broken.com/url/thing' => 404,
-            'http://www.nodomain.com' => 0
-        );
-        $actual = $links->map('Link', 'HTTPCode')->toArray();
-        $this->assertEquals($expected, $actual);
+		// Check response codes are correct
+		$expected = array(
+			'http://www.broken.com' => 403,
+			'http://www.broken.com/url/thing' => 404,
+			'http://www.nodomain.com' => 0
+		);
+		$actual = $links->map('Link', 'HTTPCode')->toArray();
+		$this->assertEquals($expected, $actual);
 
-        // Check response descriptions are correct
-        i18n::set_locale('en_NZ');
-        $expected = array(
-            'http://www.broken.com' => '403 (Forbidden)',
-            'http://www.broken.com/url/thing' => '404 (Not Found)',
-            'http://www.nodomain.com' => '0 (Server Not Available)'
-        );
-        $actual = $links->map('Link', 'HTTPCodeDescription')->toArray();
-        $this->assertEquals($expected, $actual);
-    }
+		// Check response descriptions are correct
+		i18n::set_locale('en_NZ');
+		$expected = array(
+			'http://www.broken.com' => '403 (Forbidden)',
+			'http://www.broken.com/url/thing' => '404 (Not Found)',
+			'http://www.nodomain.com' => '0 (Server Not Available)'
+		);
+		$actual = $links->map('Link', 'HTTPCodeDescription')->toArray();
+		$this->assertEquals($expected, $actual);
+	}
 
-    /**
-     * Test that broken links appears in the reports list
-     */
-    public function testReportExists()
-    {
-        $reports = Report::get_reports();
-        $reportNames = array();
-        foreach ($reports as $report) {
-            $reportNames[] = get_class($report);
-        }
-        $this->assertContains(
-            BrokenExternalLinksReport::class,
-            $reportNames,
-            'BrokenExternalLinksReport is in reports list'
-        );
-    }
+	/**
+	 * Test that broken links appears in the reports list
+	 */
+	public function testReportExists()
+	{
+		$reports = Report::get_reports();
+		$reportNames = array();
+		foreach ($reports as $report) {
+			$reportNames[] = get_class($report);
+		}
+		$this->assertContains(
+			BrokenExternalLinksReport::class,
+			$reportNames,
+			'BrokenExternalLinksReport is in reports list'
+		);
+	}
 }

src/Tasks/CheckExternalLinksTask.php

Spacing +1 added lines, -1 removed lines

@@ -17,7 +17,7 @@
 {
 
     private static $dependencies = [
-        'LinkChecker' => '%$' . LinkChecker::class
+        'LinkChecker' => '%$'.LinkChecker::class
     ];
 
     /**

Indentation +209 added lines, -209 removed lines

@@ -18,213 +18,213 @@
 class CheckExternalLinksTask extends BuildTask
 {
 
-    private static $dependencies = [
-        'LinkChecker' => '%$' . LinkChecker::class
-    ];
-
-    /**
-     * @var bool
-     */
-    protected $silent = false;
-
-    /**
-     * @var LinkChecker
-     */
-    protected $linkChecker;
-
-    protected $title = 'Checking broken External links in the SiteTree';
-
-    protected $description = 'A task that records external broken links in the SiteTree';
-
-    protected $enabled = true;
-
-    /**
-     * Log a message
-     *
-     * @param string $message
-     */
-    protected function log($message)
-    {
-        if (!$this->silent) {
-            Debug::message($message);
-        }
-    }
-
-    public function run($request)
-    {
-        $this->runLinksCheck();
-    }
-    /**
-     * Turn on or off message output
-     *
-     * @param bool $silent
-     */
-    public function setSilent($silent)
-    {
-        $this->silent = $silent;
-    }
-
-    /**
-     * @param LinkChecker $linkChecker
-     */
-    public function setLinkChecker(LinkChecker $linkChecker)
-    {
-        $this->linkChecker = $linkChecker;
-    }
-
-    /**
-     * @return LinkChecker
-     */
-    public function getLinkChecker()
-    {
-        return $this->linkChecker;
-    }
-
-    /**
-     * Check the status of a single link on a page
-     *
-     * @param BrokenExternalPageTrack $pageTrack
-     * @param DOMNode $link
-     */
-    protected function checkPageLink(BrokenExternalPageTrack $pageTrack, DOMNode $link)
-    {
-        $class = $link->getAttribute('class');
-        $href = $link->getAttribute('href');
-        $markedBroken = preg_match('/\b(ss-broken)\b/', $class);
-
-        // Check link
-        $httpCode = $this->linkChecker->checkLink($href);
-        if ($httpCode === null) {
-            return; // Null link means uncheckable, such as an internal link
-        }
-
-        // If this code is broken then mark as such
-        if ($foundBroken = $this->isCodeBroken($httpCode)) {
-            // Create broken record
-            $brokenLink = new BrokenExternalLink();
-            $brokenLink->Link = $href;
-            $brokenLink->HTTPCode = $httpCode;
-            $brokenLink->TrackID = $pageTrack->ID;
-            $brokenLink->StatusID = $pageTrack->StatusID; // Slight denormalisation here for performance reasons
-            $brokenLink->write();
-        }
-
-        // Check if we need to update CSS class, otherwise return
-        if ($markedBroken == $foundBroken) {
-            return;
-        }
-        if ($foundBroken) {
-            $class .= ' ss-broken';
-        } else {
-            $class = preg_replace('/\s*\b(ss-broken)\b\s*/', ' ', $class);
-        }
-        $link->setAttribute('class', trim($class));
-    }
-
-    /**
-     * Determine if the given HTTP code is "broken"
-     *
-     * @param int $httpCode
-     * @return bool True if this is a broken code
-     */
-    protected function isCodeBroken($httpCode)
-    {
-        // Null represents no request attempted
-        if ($httpCode === null) {
-            return false;
-        }
-
-        // do we have any whitelisted codes
-        $ignoreCodes = $this->config()->get('IgnoreCodes');
-        if (is_array($ignoreCodes) && in_array($httpCode, $ignoreCodes)) {
-            return false;
-        }
-
-        // Check if code is outside valid range
-        return $httpCode < 200 || $httpCode > 302;
-    }
-
-    /**
-     * Runs the links checker and returns the track used
-     *
-     * @param int $limit Limit to number of pages to run, or null to run all
-     * @return BrokenExternalPageTrackStatus
-     */
-    public function runLinksCheck($limit = null)
-    {
-        // Check the current status
-        $status = BrokenExternalPageTrackStatus::get_or_create();
-
-        // Calculate pages to run
-        $pageTracks = $status->getIncompleteTracks();
-        if ($limit) {
-            $pageTracks = $pageTracks->limit($limit);
-        }
-
-        // Check each page
-        foreach ($pageTracks as $pageTrack) {
-            // Flag as complete
-            $pageTrack->Processed = 1;
-            $pageTrack->write();
-
-            // Check value of html area
-            $page = $pageTrack->Page();
-            $this->log("Checking {$page->Title}");
-            $htmlValue = Injector::inst()->create('HTMLValue', $page->Content);
-            if (!$htmlValue->isValid()) {
-                continue;
-            }
-
-            // Check each link
-            $links = $htmlValue->getElementsByTagName('a');
-            foreach ($links as $link) {
-                $this->checkPageLink($pageTrack, $link);
-            }
-
-            // Update content of page based on link fixes / breakages
-            $htmlValue->saveHTML();
-            $page->Content = $htmlValue->getContent();
-            $page->write();
-
-            // Once all links have been created for this page update HasBrokenLinks
-            $count = $pageTrack->BrokenLinks()->count();
-            $this->log("Found {$count} broken links");
-            if ($count) {
-                $siteTreeTable = DataObject::getSchema()->tableName(SiteTree::class);
-                // Bypass the ORM as syncLinkTracking does not allow you to update HasBrokenLink to true
-                DB::query(sprintf(
-                    'UPDATE "%s" SET "HasBrokenLink" = 1 WHERE "ID" = \'%d\'',
-                    $siteTreeTable,
-                    intval($pageTrack->ID)
-                ));
-            }
-        }
-
-        $status->updateJobInfo('Updating completed pages');
-        $status->updateStatus();
-        return $status;
-    }
-
-    private function updateCompletedPages($trackID = 0)
-    {
-        $noPages = BrokenExternalPageTrack::get()
-            ->filter(array(
-                'TrackID' => $trackID,
-                'Processed' => 1
-            ))
-            ->count();
-        $track = BrokenExternalPageTrackStatus::get_latest();
-        $track->CompletedPages = $noPages;
-        $track->write();
-        return $noPages;
-    }
-
-    private function updateJobInfo($message)
-    {
-        $track = BrokenExternalPageTrackStatus::get_latest();
-        if ($track) {
-            $track->JobInfo = $message;
-            $track->write();
-        }
-    }
+	private static $dependencies = [
+		'LinkChecker' => '%$' . LinkChecker::class
+	];
+
+	/**
+	 * @var bool
+	 */
+	protected $silent = false;
+
+	/**
+	 * @var LinkChecker
+	 */
+	protected $linkChecker;
+
+	protected $title = 'Checking broken External links in the SiteTree';
+
+	protected $description = 'A task that records external broken links in the SiteTree';
+
+	protected $enabled = true;
+
+	/**
+	 * Log a message
+	 *
+	 * @param string $message
+	 */
+	protected function log($message)
+	{
+		if (!$this->silent) {
+			Debug::message($message);
+		}
+	}
+
+	public function run($request)
+	{
+		$this->runLinksCheck();
+	}
+	/**
+	 * Turn on or off message output
+	 *
+	 * @param bool $silent
+	 */
+	public function setSilent($silent)
+	{
+		$this->silent = $silent;
+	}
+
+	/**
+	 * @param LinkChecker $linkChecker
+	 */
+	public function setLinkChecker(LinkChecker $linkChecker)
+	{
+		$this->linkChecker = $linkChecker;
+	}
+
+	/**
+	 * @return LinkChecker
+	 */
+	public function getLinkChecker()
+	{
+		return $this->linkChecker;
+	}
+
+	/**
+	 * Check the status of a single link on a page
+	 *
+	 * @param BrokenExternalPageTrack $pageTrack
+	 * @param DOMNode $link
+	 */
+	protected function checkPageLink(BrokenExternalPageTrack $pageTrack, DOMNode $link)
+	{
+		$class = $link->getAttribute('class');
+		$href = $link->getAttribute('href');
+		$markedBroken = preg_match('/\b(ss-broken)\b/', $class);
+
+		// Check link
+		$httpCode = $this->linkChecker->checkLink($href);
+		if ($httpCode === null) {
+			return; // Null link means uncheckable, such as an internal link
+		}
+
+		// If this code is broken then mark as such
+		if ($foundBroken = $this->isCodeBroken($httpCode)) {
+			// Create broken record
+			$brokenLink = new BrokenExternalLink();
+			$brokenLink->Link = $href;
+			$brokenLink->HTTPCode = $httpCode;
+			$brokenLink->TrackID = $pageTrack->ID;
+			$brokenLink->StatusID = $pageTrack->StatusID; // Slight denormalisation here for performance reasons
+			$brokenLink->write();
+		}
+
+		// Check if we need to update CSS class, otherwise return
+		if ($markedBroken == $foundBroken) {
+			return;
+		}
+		if ($foundBroken) {
+			$class .= ' ss-broken';
+		} else {
+			$class = preg_replace('/\s*\b(ss-broken)\b\s*/', ' ', $class);
+		}
+		$link->setAttribute('class', trim($class));
+	}
+
+	/**
+	 * Determine if the given HTTP code is "broken"
+	 *
+	 * @param int $httpCode
+	 * @return bool True if this is a broken code
+	 */
+	protected function isCodeBroken($httpCode)
+	{
+		// Null represents no request attempted
+		if ($httpCode === null) {
+			return false;
+		}
+
+		// do we have any whitelisted codes
+		$ignoreCodes = $this->config()->get('IgnoreCodes');
+		if (is_array($ignoreCodes) && in_array($httpCode, $ignoreCodes)) {
+			return false;
+		}
+
+		// Check if code is outside valid range
+		return $httpCode < 200 || $httpCode > 302;
+	}
+
+	/**
+	 * Runs the links checker and returns the track used
+	 *
+	 * @param int $limit Limit to number of pages to run, or null to run all
+	 * @return BrokenExternalPageTrackStatus
+	 */
+	public function runLinksCheck($limit = null)
+	{
+		// Check the current status
+		$status = BrokenExternalPageTrackStatus::get_or_create();
+
+		// Calculate pages to run
+		$pageTracks = $status->getIncompleteTracks();
+		if ($limit) {
+			$pageTracks = $pageTracks->limit($limit);
+		}
+
+		// Check each page
+		foreach ($pageTracks as $pageTrack) {
+			// Flag as complete
+			$pageTrack->Processed = 1;
+			$pageTrack->write();
+
+			// Check value of html area
+			$page = $pageTrack->Page();
+			$this->log("Checking {$page->Title}");
+			$htmlValue = Injector::inst()->create('HTMLValue', $page->Content);
+			if (!$htmlValue->isValid()) {
+				continue;
+			}
+
+			// Check each link
+			$links = $htmlValue->getElementsByTagName('a');
+			foreach ($links as $link) {
+				$this->checkPageLink($pageTrack, $link);
+			}
+
+			// Update content of page based on link fixes / breakages
+			$htmlValue->saveHTML();
+			$page->Content = $htmlValue->getContent();
+			$page->write();
+
+			// Once all links have been created for this page update HasBrokenLinks
+			$count = $pageTrack->BrokenLinks()->count();
+			$this->log("Found {$count} broken links");
+			if ($count) {
+				$siteTreeTable = DataObject::getSchema()->tableName(SiteTree::class);
+				// Bypass the ORM as syncLinkTracking does not allow you to update HasBrokenLink to true
+				DB::query(sprintf(
+					'UPDATE "%s" SET "HasBrokenLink" = 1 WHERE "ID" = \'%d\'',
+					$siteTreeTable,
+					intval($pageTrack->ID)
+				));
+			}
+		}
+
+		$status->updateJobInfo('Updating completed pages');
+		$status->updateStatus();
+		return $status;
+	}
+
+	private function updateCompletedPages($trackID = 0)
+	{
+		$noPages = BrokenExternalPageTrack::get()
+			->filter(array(
+				'TrackID' => $trackID,
+				'Processed' => 1
+			))
+			->count();
+		$track = BrokenExternalPageTrackStatus::get_latest();
+		$track->CompletedPages = $noPages;
+		$track->write();
+		return $noPages;
+	}
+
+	private function updateJobInfo($message)
+	{
+		$track = BrokenExternalPageTrackStatus::get_latest();
+		if ($track) {
+			$track->JobInfo = $message;
+			$track->write();
+		}
+	}
 }

src/Reports/BrokenExternalLinksReport.php

Indentation +74 added lines, -74 removed lines

@@ -18,85 +18,85 @@
 class BrokenExternalLinksReport extends Report
 {
 
-    /**
-     * Returns the report title
-     *
-     * @return string
-     */
-    public function title()
-    {
-        return _t(__CLASS__ . '.EXTERNALBROKENLINKS', "External broken links report");
-    }
+	/**
+	 * Returns the report title
+	 *
+	 * @return string
+	 */
+	public function title()
+	{
+		return _t(__CLASS__ . '.EXTERNALBROKENLINKS', "External broken links report");
+	}
 
-    public function columns()
-    {
-        return array(
-            "Created" => "Checked",
-            'Link' => array(
-                'title' => 'External Link',
-                'formatting' => function ($value, $item) {
-                    return sprintf(
-                        '<a target="_blank" href="%s">%s</a>',
-                        Convert::raw2att($item->Link),
-                        Convert::raw2xml($item->Link)
-                    );
-                }
-            ),
-            'HTTPCodeDescription' => 'HTTP Error Code',
-            "Title" => array(
-                "title" => 'Page link is on',
-                'formatting' => function ($value, $item) {
-                    $page = $item->Page();
-                    return sprintf(
-                        '<a href="%s">%s</a>',
-                        Convert::raw2att($page->CMSEditLink()),
-                        Convert::raw2xml($page->Title)
-                    );
-                }
-            )
-        );
-    }
+	public function columns()
+	{
+		return array(
+			"Created" => "Checked",
+			'Link' => array(
+				'title' => 'External Link',
+				'formatting' => function ($value, $item) {
+					return sprintf(
+						'<a target="_blank" href="%s">%s</a>',
+						Convert::raw2att($item->Link),
+						Convert::raw2xml($item->Link)
+					);
+				}
+			),
+			'HTTPCodeDescription' => 'HTTP Error Code',
+			"Title" => array(
+				"title" => 'Page link is on',
+				'formatting' => function ($value, $item) {
+					$page = $item->Page();
+					return sprintf(
+						'<a href="%s">%s</a>',
+						Convert::raw2att($page->CMSEditLink()),
+						Convert::raw2xml($page->Title)
+					);
+				}
+			)
+		);
+	}
 
-    /**
-     * Alias of columns(), to support the export to csv action
-     * in {@link GridFieldExportButton} generateExportFileData method.
-     * @return array
-     */
-    public function getColumns()
-    {
-        return $this->columns();
-    }
+	/**
+	 * Alias of columns(), to support the export to csv action
+	 * in {@link GridFieldExportButton} generateExportFileData method.
+	 * @return array
+	 */
+	public function getColumns()
+	{
+		return $this->columns();
+	}
 
-    public function sourceRecords()
-    {
-        $track = BrokenExternalPageTrackStatus::get_latest();
-        if ($track) {
-            return $track->BrokenLinks();
-        }
-        return ArrayList::create();
-    }
+	public function sourceRecords()
+	{
+		$track = BrokenExternalPageTrackStatus::get_latest();
+		if ($track) {
+			return $track->BrokenLinks();
+		}
+		return ArrayList::create();
+	}
 
-    public function getCMSFields()
-    {
-        Requirements::javascript('silverstripe/externallinks: javascript/BrokenExternalLinksReport.js');
-        $fields = parent::getCMSFields();
+	public function getCMSFields()
+	{
+		Requirements::javascript('silverstripe/externallinks: javascript/BrokenExternalLinksReport.js');
+		$fields = parent::getCMSFields();
 
-        $reportResultSpan = '</ br></ br><h3 id="ReportHolder"></h3>';
-        $reportResult = LiteralField::create('ResultTitle', $reportResultSpan);
-        $fields->push($reportResult);
+		$reportResultSpan = '</ br></ br><h3 id="ReportHolder"></h3>';
+		$reportResult = LiteralField::create('ResultTitle', $reportResultSpan);
+		$fields->push($reportResult);
 
-        $button = HTML::createTag(
-            'button',
-            [
-                'id' => 'externalLinksReport',
-                'type' => 'button',
-                'class' => 'btn btn-primary'
-            ],
-            _t(__CLASS__ . '.RUNREPORT', 'Create new report')
-        );
-        $runReportButton = LiteralField::create('runReport', $button);
-        $fields->push($runReportButton);
+		$button = HTML::createTag(
+			'button',
+			[
+				'id' => 'externalLinksReport',
+				'type' => 'button',
+				'class' => 'btn btn-primary'
+			],
+			_t(__CLASS__ . '.RUNREPORT', 'Create new report')
+		);
+		$runReportButton = LiteralField::create('runReport', $button);
+		$fields->push($runReportButton);
 
-        return $fields;
-    }
+		return $fields;
+	}
 }

Spacing +4 added lines, -4 removed lines

@@ -25,7 +25,7 @@ 
      */
     public function title()
     {
-        return _t(__CLASS__ . '.EXTERNALBROKENLINKS', "External broken links report");
+        return _t(__CLASS__.'.EXTERNALBROKENLINKS', "External broken links report");
     }
 
     public function columns()
@@ -34,7 +34,7 @@ 
             "Created" => "Checked",
             'Link' => array(
                 'title' => 'External Link',
-                'formatting' => function ($value, $item) {
+                'formatting' => function($value, $item) {
                     return sprintf(
                         '<a target="_blank" href="%s">%s</a>',
                         Convert::raw2att($item->Link),
@@ -45,7 +45,7 @@ 
             'HTTPCodeDescription' => 'HTTP Error Code',
             "Title" => array(
                 "title" => 'Page link is on',
-                'formatting' => function ($value, $item) {
+                'formatting' => function($value, $item) {
                     $page = $item->Page();
                     return sprintf(
                         '<a href="%s">%s</a>',
@@ -92,7 +92,7 @@ 
                 'type' => 'button',
                 'class' => 'btn btn-primary'
             ],
-            _t(__CLASS__ . '.RUNREPORT', 'Create new report')
+            _t(__CLASS__.'.RUNREPORT', 'Create new report')
         );
         $runReportButton = LiteralField::create('runReport', $button);
         $fields->push($runReportButton);