silverstripe / silverstripe-externallinks

src/Reports/BrokenExternalLinksReport.php

Indentation +68 added lines, -68 removed lines

@@ -18,80 +18,80 @@
 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::css('silverstripe/externallinks: css/BrokenExternalLinksReport.css');
-        Requirements::javascript('silverstripe/externallinks: javascript/BrokenExternalLinksReport.js');
+	public function getCMSFields()
+	{
+		Requirements::css('silverstripe/externallinks: css/BrokenExternalLinksReport.css');
+		Requirements::javascript('silverstripe/externallinks: javascript/BrokenExternalLinksReport.js');
 
-        $fields = parent::getCMSFields();
+		$fields = parent::getCMSFields();
 
-        $runReportButton = FormAction::create('createReport', _t(__CLASS__ . '.RUNREPORT', 'Create new report'))
-            ->addExtraClass('btn-primary external-links-report__create-report')
-            ->setUseButtonTag(true);
-        $fields->push($runReportButton);
+		$runReportButton = FormAction::create('createReport', _t(__CLASS__ . '.RUNREPORT', 'Create new report'))
+			->addExtraClass('btn-primary external-links-report__create-report')
+			->setUseButtonTag(true);
+		$fields->push($runReportButton);
 
-        $reportResultSpan = '<p class="external-links-report__report-progress"></p>';
-        $reportResult = LiteralField::create('ResultTitle', $reportResultSpan);
-        $fields->push($reportResult);
+		$reportResultSpan = '<p class="external-links-report__report-progress"></p>';
+		$reportResult = LiteralField::create('ResultTitle', $reportResultSpan);
+		$fields->push($reportResult);
 
-        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>',
@@ -83,7 +83,7 @@ 
 
         $fields = parent::getCMSFields();
 
-        $runReportButton = FormAction::create('createReport', _t(__CLASS__ . '.RUNREPORT', 'Create new report'))
+        $runReportButton = FormAction::create('createReport', _t(__CLASS__.'.RUNREPORT', 'Create new report'))
             ->addExtraClass('btn-primary external-links-report__create-report')
             ->setUseButtonTag(true);
         $fields->push($runReportButton);

src/Tasks/CurlLinkChecker.php

Indentation +43 added lines, -43 removed lines

@@ -11,47 +11,47 @@
 class CurlLinkChecker implements LinkChecker
 {
 
-    /**
-     * 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;
-        }
-
-        // Check if we have a cached result
-        $cacheKey = md5($href);
-        $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);
-        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($cacheKey, $httpCode);
-        return $httpCode;
-    }
+	/**
+	 * 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;
+		}
+
+		// Check if we have a cached result
+		$cacheKey = md5($href);
+		$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);
+		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($cacheKey, $httpCode);
+		return $httpCode;
+	}
 }

src/Controllers/CMSExternalLinksController.php

Indentation +53 added lines, -53 removed lines

@@ -13,62 +13,62 @@
 class CMSExternalLinksController extends Controller
 {
 
-    private static $allowed_actions = [
-        'getJobStatus',
-        'start'
-    ];
+	private static $allowed_actions = [
+		'getJobStatus',
+		'start'
+	];
 
-    /**
-     * Respond to Ajax requests for info on a running job
-     *
-     * @return string JSON string detailing status of the job
-     */
-    public function getJobStatus()
-    {
-        // Set headers
-        HTTP::set_cache_age(0);
-        HTTP::add_cache_headers($this->response);
-        $this->response
-            ->addHeader('Content-Type', 'application/json')
-            ->addHeader('Content-Encoding', 'UTF-8')
-            ->addHeader('X-Content-Type-Options', 'nosniff');
+	/**
+	 * Respond to Ajax requests for info on a running job
+	 *
+	 * @return string JSON string detailing status of the job
+	 */
+	public function getJobStatus()
+	{
+		// Set headers
+		HTTP::set_cache_age(0);
+		HTTP::add_cache_headers($this->response);
+		$this->response
+			->addHeader('Content-Type', 'application/json')
+			->addHeader('Content-Encoding', 'UTF-8')
+			->addHeader('X-Content-Type-Options', 'nosniff');
 
-        // Format status
-        $track = BrokenExternalPageTrackStatus::get_latest();
-        if ($track) {
-            return Convert::array2json([
-                'TrackID' => $track->ID,
-                'Status' => $track->Status,
-                'Completed' => $track->getCompletedPages(),
-                'Total' => $track->getTotalPages()
-            ]);
-        }
-    }
+		// Format status
+		$track = BrokenExternalPageTrackStatus::get_latest();
+		if ($track) {
+			return Convert::array2json([
+				'TrackID' => $track->ID,
+				'Status' => $track->Status,
+				'Completed' => $track->getCompletedPages(),
+				'Total' => $track->getTotalPages()
+			]);
+		}
+	}
 
 
-    /**
-     * Starts a broken external link check
-     */
-    public function start()
-    {
-        // return if the a job is already running
-        $status = BrokenExternalPageTrackStatus::get_latest();
-        if ($status && $status->Status == 'Running') {
-            return;
-        }
+	/**
+	 * Starts a broken external link check
+	 */
+	public function start()
+	{
+		// return if the a job is already running
+		$status = BrokenExternalPageTrackStatus::get_latest();
+		if ($status && $status->Status == 'Running') {
+			return;
+		}
 
-        // Create a new job
-        if (class_exists(QueuedJobService::class)) {
-            // Force the creation of a new run
-            BrokenExternalPageTrackStatus::create_status();
-            $checkLinks = new CheckExternalLinksJob();
-            singleton(QueuedJobService::class)->queueJob($checkLinks);
-        } else {
-            //TODO this hangs as it waits for the connection to be released
-            // should return back and continue processing
-            // http://us3.php.net/manual/en/features.connection-handling.php
-            $task = CheckExternalLinksTask::create();
-            $task->runLinksCheck();
-        }
-    }
+		// Create a new job
+		if (class_exists(QueuedJobService::class)) {
+			// Force the creation of a new run
+			BrokenExternalPageTrackStatus::create_status();
+			$checkLinks = new CheckExternalLinksJob();
+			singleton(QueuedJobService::class)->queueJob($checkLinks);
+		} else {
+			//TODO this hangs as it waits for the connection to be released
+			// should return back and continue processing
+			// http://us3.php.net/manual/en/features.connection-handling.php
+			$task = CheckExternalLinksTask::create();
+			$task->runLinksCheck();
+		}
+	}
 }