dylangrech92 / seotoolbox

code/extenstions/SEOToolboxControllerExtension.php

Indentation +61 added lines, -61 removed lines

@@ -25,29 +25,29 @@ 
         // Do something here
     }
 
-	private $maxLinksPerPage;
+    private $maxLinksPerPage;
     private $settings       = null;
-	private $linkCount      = 0;
+    private $linkCount      = 0;
     private $addLinks       = true;
     private $excludeTags    = array();
     private $maxLinks       = 0;
 
-	public function index(){
+    public function index(){
         $this->addAutomatedLinks();
 
-	    // If we have a crawl request check the CrawlID so we're sure we didn't hit another SS site running our module
-	    if( $crawl_id = $this->owner->request->getHeader('X-Crawl-Id') ){
-	        return( $crawl_id == GlobalAutoLinkSettings::get_current()->CrawlID )
+        // If we have a crawl request check the CrawlID so we're sure we didn't hit another SS site running our module
+        if( $crawl_id = $this->owner->request->getHeader('X-Crawl-Id') ){
+            return( $crawl_id == GlobalAutoLinkSettings::get_current()->CrawlID )
                 ? $this->crawl_response()
                 : $this->owner->redirect(SEOTestSiteTreeController::getPermissionDeniedPage()->Link());
         }
 
-		return array();
-	}
+        return array();
+    }
 
-	private function crawl_response(){
+    private function crawl_response(){
         // Encoded version to detect which fields are being used
-	    $customize = array();
+        $customize = array();
         foreach( Config::inst()->get($this->owner->ClassName, 'db') as $field => $type ){
             if( strtolower( $type ) == 'htmltext' ){
                 $data = ($this->owner->hasMethod($field)) ? $this->owner->$field() : $this->owner->$field;
@@ -72,7 +72,7 @@ 
                 '/<!--(.*?)-->/im',
             ), '', $this->owner->customise($customize)->render())
         );
-	}
+    }
 
     /**
      * Get the global settings and check if we should be adding
@@ -94,13 +94,13 @@ 
         return $this->settings;
     }
 
-	/**
-	 * Goes through all the automated link settings and adds
-	 * the links where necessary
-	 *
-	 * @return void
-	 */
-	public function addAutomatedLinks(){
+    /**
+     * Goes through all the automated link settings and adds
+     * the links where necessary
+     *
+     * @return void
+     */
+    public function addAutomatedLinks(){
         if( GlobalAutoLinkSettings::$enabled && $this->owner->class != 'RedirectorPage' ) {
             $this->getSettings();
             if( !$this->addLinks ) return;
@@ -132,60 +132,60 @@ 
                 }
             }
         }
-	}
+    }
 
-	/**
-	 * Parse the provided field and add the necessary links
-	 *
-	 * @param DOMDocument $html
+    /**
+     * Parse the provided field and add the necessary links
+     *
+     * @param DOMDocument $html
      * @param String $field
-	 * @return string
-	 */
-	private function parseField( DOMDocument $html, $field ){
-		$this->owner->extend( 'beforeParseField', $html, $field );
-
-		// Remove Tags from Content we wown't be using
-		$excluded = array();
-		foreach( $this->excludeTags as $eTag ){
-		    while( $tags = $html->getElementsByTagName( $eTag ) ){
-		        if( !$tags->length ) break 1;
-		        $tag	= $tags->item(0);
+     * @return string
+     */
+    private function parseField( DOMDocument $html, $field ){
+        $this->owner->extend( 'beforeParseField', $html, $field );
+
+        // Remove Tags from Content we wown't be using
+        $excluded = array();
+        foreach( $this->excludeTags as $eTag ){
+            while( $tags = $html->getElementsByTagName( $eTag ) ){
+                if( !$tags->length ) break 1;
+                $tag	= $tags->item(0);
                 $value  = $html->saveHTML( $tag );
                 $key    = (string) crc32( $value );
 
-				// Convert back children nodes of this node if they were already hashed
-				$excluded[$key] = str_replace( array_keys( $excluded ), array_values( $excluded ), $value );
+                // Convert back children nodes of this node if they were already hashed
+                $excluded[$key] = str_replace( array_keys( $excluded ), array_values( $excluded ), $value );
 
                 $tag->parentNode->replaceChild( $html->createTextNode( $key ), $tag );
-		    }
-		}
+            }
+        }
 
         $body    = (string)$html->saveHTML( $html->getElementsByTagName('body')->item(0) );
         $content = preg_replace( array( '/\<body\>/is', '/\<\/body\>/is' ), '', $body, 1 );
 
-		// Create the links
-		$links = AutomatedLink::get()->sort('Priority');
-		foreach( $links as $link ){
-			// Check if self-linking is allowed and if current pagetype is allowed
-			if( !$link->canBeAdded( $this->owner, $field ) ) continue;
+        // Create the links
+        $links = AutomatedLink::get()->sort('Priority');
+        foreach( $links as $link ){
+            // Check if self-linking is allowed and if current pagetype is allowed
+            if( !$link->canBeAdded( $this->owner, $field ) ) continue;
 
-			$max    = (int) ( $link->MaxLinksPerPage > 0 ) ? $link->MaxLinksPerPage : PHP_INT_MAX;
+            $max    = (int) ( $link->MaxLinksPerPage > 0 ) ? $link->MaxLinksPerPage : PHP_INT_MAX;
             $escape = (string) preg_quote( $link->Phrase, '/' );
-			$regex  = (string) ( $link->CaseSensitive ) ? "/(\b{$escape}\b)/" : "/(\b{$escape}\b)/i";
+            $regex  = (string) ( $link->CaseSensitive ) ? "/(\b{$escape}\b)/" : "/(\b{$escape}\b)/i";
 
             // Count the matches
             preg_match_all( $regex, $content, $count );
             $count = ( is_array( $count ) && isset( $count[0] ) ) ? count( $count[0] ) : 0;
             if( $count < 1 ) continue;
 
-			if( isset( $this->maxLinksPerPage[ $link->ID ] ) )
-				$max -= $this->maxLinksPerPage[ $link->ID ];
-			else
-				$this->maxLinksPerPage[ $link->ID ] = 0;
+            if( isset( $this->maxLinksPerPage[ $link->ID ] ) )
+                $max -= $this->maxLinksPerPage[ $link->ID ];
+            else
+                $this->maxLinksPerPage[ $link->ID ] = 0;
 
-			for( $x = 0; $x < $count; $x++ ){
-				// Stop adding links if we reached the link or page limit
-				if( $x >= $max || $this->linkCount >= $this->maxLinks ) break;
+            for( $x = 0; $x < $count; $x++ ){
+                // Stop adding links if we reached the link or page limit
+                if( $x >= $max || $this->linkCount >= $this->maxLinks ) break;
 
                 // Check if there is anything else to replace else stop
                 preg_match( $regex, $content, $match );
@@ -195,18 +195,18 @@ 
                 $key              = (string) crc32( $html );
                 $excluded[ $key ] = (string) $html;
 
-				$content = preg_replace( $regex, $key, $content, 1 );
-				$this->linkCount++;
+                $content = preg_replace( $regex, $key, $content, 1 );
+                $this->linkCount++;
                 $this->maxLinksPerPage[ $link->ID ]++;
             }
 
-			// Stop Adding links if we reached the page limit
-			if( $this->linkCount >= $this->maxLinks ) break;
-		}
+            // Stop Adding links if we reached the page limit
+            if( $this->linkCount >= $this->maxLinks ) break;
+        }
 
-		// Re-add the excluded Tags
-		$content = str_replace( array_keys( $excluded ), array_values( $excluded ), $content );
+        // Re-add the excluded Tags
+        $content = str_replace( array_keys( $excluded ), array_values( $excluded ), $content );
 
-		return $content;
-	}
+        return $content;
+    }
 }

Spacing +64 added lines, -64 removed lines

@@ -10,7 +10,7 @@ 
  *
  * @see AutomatedLink
  */
-class SEOToolboxControllerExtension extends Extension{
+class SEOToolboxControllerExtension extends Extension {
 
     /**
      * In here we can perform actions before the here mentioned $field
@@ -21,23 +21,23 @@ 
      * @param DOMDocument $html
      * @param             $field
      */
-    public function beforeParseField(DOMDocument $html, $field){
+    public function beforeParseField(DOMDocument $html, $field) {
         // Do something here
     }
 
 	private $maxLinksPerPage;
     private $settings       = null;
-	private $linkCount      = 0;
+	private $linkCount = 0;
     private $addLinks       = true;
     private $excludeTags    = array();
     private $maxLinks       = 0;
 
-	public function index(){
+	public function index() {
         $this->addAutomatedLinks();
 
 	    // If we have a crawl request check the CrawlID so we're sure we didn't hit another SS site running our module
-	    if( $crawl_id = $this->owner->request->getHeader('X-Crawl-Id') ){
-	        return( $crawl_id == GlobalAutoLinkSettings::get_current()->CrawlID )
+	    if ($crawl_id = $this->owner->request->getHeader('X-Crawl-Id')) {
+	        return($crawl_id == GlobalAutoLinkSettings::get_current()->CrawlID)
                 ? $this->crawl_response()
                 : $this->owner->redirect(SEOTestSiteTreeController::getPermissionDeniedPage()->Link());
         }
@@ -45,13 +45,13 @@ 
 		return array();
 	}
 
-	private function crawl_response(){
+	private function crawl_response() {
         // Encoded version to detect which fields are being used
 	    $customize = array();
-        foreach( Config::inst()->get($this->owner->ClassName, 'db') as $field => $type ){
-            if( strtolower( $type ) == 'htmltext' ){
+        foreach (Config::inst()->get($this->owner->ClassName, 'db') as $field => $type) {
+            if (strtolower($type) == 'htmltext') {
                 $data = ($this->owner->hasMethod($field)) ? $this->owner->$field() : $this->owner->$field;
-                if( !$data ) continue;
+                if (!$data) continue;
                 $tmp = new HTMLText('tmp');
                 $tmp->setValue($data);
                 $data = base64_encode($tmp->forTemplate());
@@ -59,7 +59,7 @@ 
             }
         }
 
-        if( in_array($this->owner->ClassName, ClassInfo::subclassesFor('ErrorPage')) ){
+        if (in_array($this->owner->ClassName, ClassInfo::subclassesFor('ErrorPage'))) {
             header("HTTP/1.0 405 Instance of ErrorPage");
             die();
         }
@@ -80,13 +80,13 @@ 
      *
      * @return GlobalAutoLinkSettings
      */
-    private function getSettings(){
-        if( $this->settings === null ) {
+    private function getSettings() {
+        if ($this->settings === null) {
             $this->settings = GlobalAutoLinkSettings::get_current();
             if (!$this->settings) return $this->addLinks = false;
 
-            $this->excludeTags = (array)$this->settings->ExcludeTags();
-            $this->maxLinks = (int)($this->settings->MaxLinksPerPage) ? $this->settings->MaxLinksPerPage : PHP_INT_MAX;
+            $this->excludeTags = (array) $this->settings->ExcludeTags();
+            $this->maxLinks = (int) ($this->settings->MaxLinksPerPage) ? $this->settings->MaxLinksPerPage : PHP_INT_MAX;
 
             if (!in_array($this->owner->ClassName, $this->settings->AllowedIn())) $this->addLinks = false;
         }
@@ -100,33 +100,33 @@ 
 	 *
 	 * @return void
 	 */
-	public function addAutomatedLinks(){
-        if( GlobalAutoLinkSettings::$enabled && $this->owner->class != 'RedirectorPage' ) {
+	public function addAutomatedLinks() {
+        if (GlobalAutoLinkSettings::$enabled && $this->owner->class != 'RedirectorPage') {
             $this->getSettings();
-            if( !$this->addLinks ) return;
+            if (!$this->addLinks) return;
 
-            foreach( $this->getSettings()->IncludeInFields() as $field ){
+            foreach ($this->getSettings()->IncludeInFields() as $field) {
                 // Check that the field provided by user exists in this object, is of type HTMLText and has content
-                if( AutomatedLink::isFieldParsable( $this->owner->data(), $field ) ){
+                if (AutomatedLink::isFieldParsable($this->owner->data(), $field)) {
 
                     // Create dummy object so we can parse the HTML
-                    $dummy = new HTMLText( $field );
-                    $dummy->setValue( $this->owner->$field );
+                    $dummy = new HTMLText($field);
+                    $dummy->setValue($this->owner->$field);
                     // Create DOMDocument Object
-                    $content = mb_convert_encoding( $dummy->forTemplate(), 'html-entities', GlobalAutoLinkSettings::$encoding );
+                    $content = mb_convert_encoding($dummy->forTemplate(), 'html-entities', GlobalAutoLinkSettings::$encoding);
 
-                    if( class_exists( 'HTML5_Parser' ) ){
-                        $dom = HTML5_Parser::parse( $content );
-                    }else{
+                    if (class_exists('HTML5_Parser')) {
+                        $dom = HTML5_Parser::parse($content);
+                    }else {
                         $dom = new DOMDocument();
-                        $dom->loadHTML( $content );
+                        $dom->loadHTML($content);
                     }
 
                     // Check current link count and if it's already exceeded do nothing
-                    $this->linkCount += (int) $dom->getElementsByTagName( 'a' )->length;
-                    if( $this->linkCount >= $this->maxLinks ) return;
+                    $this->linkCount += (int) $dom->getElementsByTagName('a')->length;
+                    if ($this->linkCount >= $this->maxLinks) return;
 
-                    $parsed = $this->parseField( $dom, $field );
+                    $parsed = $this->parseField($dom, $field);
                     $this->owner->data()->$field = $parsed;
                     $this->owner->$field         = $parsed;
                 }
@@ -141,71 +141,71 @@ 
      * @param String $field
 	 * @return string
 	 */
-	private function parseField( DOMDocument $html, $field ){
-		$this->owner->extend( 'beforeParseField', $html, $field );
+	private function parseField(DOMDocument $html, $field) {
+		$this->owner->extend('beforeParseField', $html, $field);
 
 		// Remove Tags from Content we wown't be using
 		$excluded = array();
-		foreach( $this->excludeTags as $eTag ){
-		    while( $tags = $html->getElementsByTagName( $eTag ) ){
-		        if( !$tags->length ) break 1;
-		        $tag	= $tags->item(0);
-                $value  = $html->saveHTML( $tag );
-                $key    = (string) crc32( $value );
+		foreach ($this->excludeTags as $eTag) {
+		    while ($tags = $html->getElementsByTagName($eTag)) {
+		        if (!$tags->length) break 1;
+		        $tag = $tags->item(0);
+                $value  = $html->saveHTML($tag);
+                $key    = (string) crc32($value);
 
 				// Convert back children nodes of this node if they were already hashed
-				$excluded[$key] = str_replace( array_keys( $excluded ), array_values( $excluded ), $value );
+				$excluded[$key] = str_replace(array_keys($excluded), array_values($excluded), $value);
 
-                $tag->parentNode->replaceChild( $html->createTextNode( $key ), $tag );
+                $tag->parentNode->replaceChild($html->createTextNode($key), $tag);
 		    }
 		}
 
-        $body    = (string)$html->saveHTML( $html->getElementsByTagName('body')->item(0) );
-        $content = preg_replace( array( '/\<body\>/is', '/\<\/body\>/is' ), '', $body, 1 );
+        $body    = (string) $html->saveHTML($html->getElementsByTagName('body')->item(0));
+        $content = preg_replace(array('/\<body\>/is', '/\<\/body\>/is'), '', $body, 1);
 
 		// Create the links
 		$links = AutomatedLink::get()->sort('Priority');
-		foreach( $links as $link ){
+		foreach ($links as $link) {
 			// Check if self-linking is allowed and if current pagetype is allowed
-			if( !$link->canBeAdded( $this->owner, $field ) ) continue;
+			if (!$link->canBeAdded($this->owner, $field)) continue;
 
-			$max    = (int) ( $link->MaxLinksPerPage > 0 ) ? $link->MaxLinksPerPage : PHP_INT_MAX;
-            $escape = (string) preg_quote( $link->Phrase, '/' );
-			$regex  = (string) ( $link->CaseSensitive ) ? "/(\b{$escape}\b)/" : "/(\b{$escape}\b)/i";
+			$max    = (int) ($link->MaxLinksPerPage > 0) ? $link->MaxLinksPerPage : PHP_INT_MAX;
+            $escape = (string) preg_quote($link->Phrase, '/');
+			$regex  = (string) ($link->CaseSensitive) ? "/(\b{$escape}\b)/" : "/(\b{$escape}\b)/i";
 
             // Count the matches
-            preg_match_all( $regex, $content, $count );
-            $count = ( is_array( $count ) && isset( $count[0] ) ) ? count( $count[0] ) : 0;
-            if( $count < 1 ) continue;
+            preg_match_all($regex, $content, $count);
+            $count = (is_array($count) && isset($count[0])) ? count($count[0]) : 0;
+            if ($count < 1) continue;
 
-			if( isset( $this->maxLinksPerPage[ $link->ID ] ) )
-				$max -= $this->maxLinksPerPage[ $link->ID ];
+			if (isset($this->maxLinksPerPage[$link->ID]))
+				$max -= $this->maxLinksPerPage[$link->ID];
 			else
-				$this->maxLinksPerPage[ $link->ID ] = 0;
+				$this->maxLinksPerPage[$link->ID] = 0;
 
-			for( $x = 0; $x < $count; $x++ ){
+			for ($x = 0; $x < $count; $x++) {
 				// Stop adding links if we reached the link or page limit
-				if( $x >= $max || $this->linkCount >= $this->maxLinks ) break;
+				if ($x >= $max || $this->linkCount >= $this->maxLinks) break;
 
                 // Check if there is anything else to replace else stop
-                preg_match( $regex, $content, $match );
-                if( !is_array( $match ) || !count( $match ) ) break;
+                preg_match($regex, $content, $match);
+                if (!is_array($match) || !count($match)) break;
 
-                if( !$html = (string) $link->getHTML( $match[0] ) ) continue;
-                $key              = (string) crc32( $html );
-                $excluded[ $key ] = (string) $html;
+                if (!$html = (string) $link->getHTML($match[0])) continue;
+                $key              = (string) crc32($html);
+                $excluded[$key] = (string) $html;
 
-				$content = preg_replace( $regex, $key, $content, 1 );
+				$content = preg_replace($regex, $key, $content, 1);
 				$this->linkCount++;
-                $this->maxLinksPerPage[ $link->ID ]++;
+                $this->maxLinksPerPage[$link->ID]++;
             }
 
 			// Stop Adding links if we reached the page limit
-			if( $this->linkCount >= $this->maxLinks ) break;
+			if ($this->linkCount >= $this->maxLinks) break;
 		}
 
 		// Re-add the excluded Tags
-		$content = str_replace( array_keys( $excluded ), array_values( $excluded ), $content );
+		$content = str_replace(array_keys($excluded), array_values($excluded), $content);
 
 		return $content;
 	}

Braces +42 added lines, -17 removed lines

@@ -51,7 +51,9 @@ 
         foreach( Config::inst()->get($this->owner->ClassName, 'db') as $field => $type ){
             if( strtolower( $type ) == 'htmltext' ){
                 $data = ($this->owner->hasMethod($field)) ? $this->owner->$field() : $this->owner->$field;
-                if( !$data ) continue;
+                if( !$data ) {
+                    continue;
+                }
                 $tmp = new HTMLText('tmp');
                 $tmp->setValue($data);
                 $data = base64_encode($tmp->forTemplate());
@@ -83,12 +85,16 @@ 
     private function getSettings(){
         if( $this->settings === null ) {
             $this->settings = GlobalAutoLinkSettings::get_current();
-            if (!$this->settings) return $this->addLinks = false;
+            if (!$this->settings) {
+                return $this->addLinks = false;
+            }
 
             $this->excludeTags = (array)$this->settings->ExcludeTags();
             $this->maxLinks = (int)($this->settings->MaxLinksPerPage) ? $this->settings->MaxLinksPerPage : PHP_INT_MAX;
 
-            if (!in_array($this->owner->ClassName, $this->settings->AllowedIn())) $this->addLinks = false;
+            if (!in_array($this->owner->ClassName, $this->settings->AllowedIn())) {
+                $this->addLinks = false;
+            }
         }
 
         return $this->settings;
@@ -103,7 +109,9 @@ 
 	public function addAutomatedLinks(){
         if( GlobalAutoLinkSettings::$enabled && $this->owner->class != 'RedirectorPage' ) {
             $this->getSettings();
-            if( !$this->addLinks ) return;
+            if( !$this->addLinks ) {
+                return;
+            }
 
             foreach( $this->getSettings()->IncludeInFields() as $field ){
                 // Check that the field provided by user exists in this object, is of type HTMLText and has content
@@ -117,14 +125,16 @@ 
 
                     if( class_exists( 'HTML5_Parser' ) ){
                         $dom = HTML5_Parser::parse( $content );
-                    }else{
+                    } else{
                         $dom = new DOMDocument();
                         $dom->loadHTML( $content );
                     }
 
                     // Check current link count and if it's already exceeded do nothing
                     $this->linkCount += (int) $dom->getElementsByTagName( 'a' )->length;
-                    if( $this->linkCount >= $this->maxLinks ) return;
+                    if( $this->linkCount >= $this->maxLinks ) {
+                        return;
+                    }
 
                     $parsed = $this->parseField( $dom, $field );
                     $this->owner->data()->$field = $parsed;
@@ -148,7 +158,9 @@ 
 		$excluded = array();
 		foreach( $this->excludeTags as $eTag ){
 		    while( $tags = $html->getElementsByTagName( $eTag ) ){
-		        if( !$tags->length ) break 1;
+		        if( !$tags->length ) {
+		            break 1;
+		        }
 		        $tag	= $tags->item(0);
                 $value  = $html->saveHTML( $tag );
                 $key    = (string) crc32( $value );
@@ -167,7 +179,9 @@ 
 		$links = AutomatedLink::get()->sort('Priority');
 		foreach( $links as $link ){
 			// Check if self-linking is allowed and if current pagetype is allowed
-			if( !$link->canBeAdded( $this->owner, $field ) ) continue;
+			if( !$link->canBeAdded( $this->owner, $field ) ) {
+			    continue;
+			}
 
 			$max    = (int) ( $link->MaxLinksPerPage > 0 ) ? $link->MaxLinksPerPage : PHP_INT_MAX;
             $escape = (string) preg_quote( $link->Phrase, '/' );
@@ -176,22 +190,31 @@ 
             // Count the matches
             preg_match_all( $regex, $content, $count );
             $count = ( is_array( $count ) && isset( $count[0] ) ) ? count( $count[0] ) : 0;
-            if( $count < 1 ) continue;
+            if( $count < 1 ) {
+                continue;
+            }
 
-			if( isset( $this->maxLinksPerPage[ $link->ID ] ) )
-				$max -= $this->maxLinksPerPage[ $link->ID ];
-			else
-				$this->maxLinksPerPage[ $link->ID ] = 0;
+			if( isset( $this->maxLinksPerPage[ $link->ID ] ) ) {
+							$max -= $this->maxLinksPerPage[ $link->ID ];
+			} else {
+							$this->maxLinksPerPage[ $link->ID ] = 0;
+			}
 
 			for( $x = 0; $x < $count; $x++ ){
 				// Stop adding links if we reached the link or page limit
-				if( $x >= $max || $this->linkCount >= $this->maxLinks ) break;
+				if( $x >= $max || $this->linkCount >= $this->maxLinks ) {
+				    break;
+				}
 
                 // Check if there is anything else to replace else stop
                 preg_match( $regex, $content, $match );
-                if( !is_array( $match ) || !count( $match ) ) break;
+                if( !is_array( $match ) || !count( $match ) ) {
+                    break;
+                }
 
-                if( !$html = (string) $link->getHTML( $match[0] ) ) continue;
+                if( !$html = (string) $link->getHTML( $match[0] ) ) {
+                    continue;
+                }
                 $key              = (string) crc32( $html );
                 $excluded[ $key ] = (string) $html;
 
@@ -201,7 +224,9 @@ 
             }
 
 			// Stop Adding links if we reached the page limit
-			if( $this->linkCount >= $this->maxLinks ) break;
+			if( $this->linkCount >= $this->maxLinks ) {
+			    break;
+			}
 		}
 
 		// Re-add the excluded Tags

code/reports/AutomatedLinkReport.php

Spacing +6 added lines, -6 removed lines

@@ -8,7 +8,7 @@ 
  * Automated Link Report is a report that lists all automated
  * links and how they affect the website
  */
-class AutomatedLinkReport extends SS_Report{
+class AutomatedLinkReport extends SS_Report {
         
     protected $title       = 'Automated Link Report';
     protected $description = 'Shows a list of all automated links and how they affect the site';
@@ -16,11 +16,11 @@ 
     // Configs
     private static $run_in_realtime = true;
 
-    public function Title(){
+    public function Title() {
         return (string) $this->title;
     }
 
-    public function columns(){
+    public function columns() {
         return array(
             'Title'              => array(
                 'title' => 'Page Title',
@@ -34,8 +34,8 @@ 
         );
     }
     
-    public function sourceRecords( $params, $sort, $limit ){
-        if( !Config::inst()->get( $this->class, 'run_in_realtime' ) ) {
+    public function sourceRecords($params, $sort, $limit) {
+        if (!Config::inst()->get($this->class, 'run_in_realtime')) {
             return AutomatedLinkPageResult::get();
         }
 
@@ -53,7 +53,7 @@ 
         $gridConfig->removeComponentsByType('GridFieldPrintButton');
         $gridConfig->removeComponentsByType('GridFieldExportButton');
 
-        $gridConfig->addComponents (
+        $gridConfig->addComponents(
             new GridFieldPrintAllAutomatedLinksButton('buttons-after-left'),
             new GridFieldExportAllAutomatedLinksButton('buttons-after-left')
         );