dylangrech92 / seotoolbox

code/dataobjects/AutomatedLink.php

Spacing +29 added lines, -29 removed lines

@@ -43,9 +43,9 @@ 
         'Page' => 'SiteTree'
     );
 
-    private static $summary_fields    = array( 'Phrase', 'PointsTo' );
-    private static $searchable_fields = array( 'Phrase' );
-    private static $singular_name	  = 'Automated Link';
+    private static $summary_fields    = array('Phrase', 'PointsTo');
+    private static $searchable_fields = array('Phrase');
+    private static $singular_name = 'Automated Link';
     private static $plural_name	      = 'Automated Links';
     private static $parsableFields    = array();
 
@@ -54,7 +54,7 @@ 
      *
      * @return string
      */
-    public function PointsTo(){
+    public function PointsTo() {
         return $this->Page()->Link();
     }
 
@@ -63,7 +63,7 @@ 
      *
      * @return string
      */
-    public function Title(){
+    public function Title() {
         return $this->Phrase;
     }
 
@@ -72,23 +72,23 @@ 
      *
      * @return String
      */
-    public function forTemplate(){
+    public function forTemplate() {
         return $this->getHTML();
     }
 
-    public function canView( $member = false ){
+    public function canView($member = false) {
         return Permission::check('AUTOMATEDLINK_VIEW');
     }
 
-    public function canEdit( $member = false ){
+    public function canEdit($member = false) {
         return Permission::check('AUTOMATEDLINK_EDIT');
     }
 
-    public function canDelete( $member = false ){
+    public function canDelete($member = false) {
         return Permission::check('AUTOMATEDLINK_DELETE');
     }
 
-    public function canCreate( $member = false ){
+    public function canCreate($member = false) {
         return Permission::check('AUTOMATEDLINK_CREATE');
     }
 
@@ -101,12 +101,12 @@ 
        );
      }
 
-	public function requireDefaultRecords(){
+	public function requireDefaultRecords() {
 		parent::requireDefaultRecords();
 
 		// Update all links to redirector pages during dev/build
-		foreach( self::get() as $link ) {
-		    $link->CheckAndUpdateDestination( true );
+		foreach (self::get() as $link) {
+		    $link->CheckAndUpdateDestination(true);
 		}
 	}
 
@@ -207,15 +207,15 @@ 
 	 * @Boolean $write - Write the changes if any
 	 * @return void
 	 */
-	public function CheckAndUpdateDestination( $write = false ){
+	public function CheckAndUpdateDestination($write = false) {
 		$this->extend('beforeCheckAndUpdateDestination', $write);
 
-		if( $this->PageID && $this->Page() &&
+		if ($this->PageID && $this->Page() &&
 			$this->Page()->ClassName == 'RedirectorPage' &&
-			$this->Page()->LinkToID && $this->Page()->RedirectionType == 'Internal' )
+			$this->Page()->LinkToID && $this->Page()->RedirectionType == 'Internal')
 		{
 			$this->PageID = $this->Page()->LinkToID;
-			if( $write ) {
+			if ($write) {
 			    $this->write();
 			}
 		}
@@ -245,8 +245,8 @@ 
      * @param ContentController $controller
      * @return Boolean
      */
-    public function canBeAdded( ContentController $controller ){
-        return ( $this->SelfLinking || $controller->ID != $this->PageID );
+    public function canBeAdded(ContentController $controller) {
+        return ($this->SelfLinking || $controller->ID != $this->PageID);
     }
 
     /**
@@ -255,20 +255,20 @@ 
      * @param string $html
      * @return DOMDocument
      */
-    public static function constructDOMDocument($html){
-        if( class_exists( 'HTML5_Parser' ) ){
-            $html5 = HTML5_Parser::parse( $html );
-            if($html5 instanceof DOMNodeList){
+    public static function constructDOMDocument($html) {
+        if (class_exists('HTML5_Parser')) {
+            $html5 = HTML5_Parser::parse($html);
+            if ($html5 instanceof DOMNodeList) {
                 $dom = new DOMDocument();
-                while($html5->length > 0) {
+                while ($html5->length > 0) {
                     $dom->appendChild($html5->item(0));
                 }
-            }else{
+            }else {
                 $dom = $html5;
             }
-        } else{
+        }else {
             $dom = new DOMDocument();
-            $dom->loadHTML( $html );
+            $dom->loadHTML($html);
         }
 
         return $dom;
@@ -280,9 +280,9 @@ 
      * @param string $class
      * @return array
      */
-    public static function getAllDatabaseFields($class){
+    public static function getAllDatabaseFields($class) {
         $fields = array();
-        foreach (ClassInfo::ancestry($class, true) as $ancestor){
+        foreach (ClassInfo::ancestry($class, true) as $ancestor) {
             $fields = array_merge($fields, (array) DataObject::database_fields($ancestor));
         }
         return $fields;

Braces +1 added lines, -1 removed lines

@@ -263,7 +263,7 @@
                 while($html5->length > 0) {
                     $dom->appendChild($html5->item(0));
                 }
-            }else{
+            } else{
                 $dom = $html5;
             }
         } else{

code/reports/AutomatedLinkReport.php

Spacing +3 added lines, -3 removed lines

@@ -7,7 +7,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';
@@ -33,8 +33,8 @@ 
         );
     }
 
-    public function sourceRecords(){
-        if( !Config::inst()->get( $this->class, 'run_in_realtime' ) ) {
+    public function sourceRecords() {
+        if (!Config::inst()->get($this->class, 'run_in_realtime')) {
             return AutomatedLinkPageResult::get();
         }
 

code/controllers/AutomatedLinkReportTask.php

Braces +36 added lines, -15 removed lines

@@ -16,7 +16,9 @@ 
     private static $exclude_classes = array('RedirectorPage', 'VirtualPage');
 
     public function index() {
-        if (!Director::is_cli()) return 'Please run this controller in CLI';
+        if (!Director::is_cli()) {
+            return 'Please run this controller in CLI';
+        }
 
         libxml_use_internal_errors(true);
         set_time_limit(600);
@@ -52,15 +54,23 @@ 
         $exclude = Config::inst()->get($this->class, 'exclude_classes');
         $exclude = ($exclude) ? "'".implode("','", $exclude)."'" : '';
         foreach (SiteTree::get()->where("ClassName NOT IN($exclude)") as $page) {
-            if (!$this->checkForPossibleLinks($page, $includeInFields)) continue;
+            if (!$this->checkForPossibleLinks($page, $includeInFields)) {
+                continue;
+            }
             $page = $this->getLinkData($page, $includeInFields);
-            if (!$page) continue;
+            if (!$page) {
+                continue;
+            }
 
-            if (!$run_in_realtime) AutomatedLinkPageResult::add_or_update($page);
+            if (!$run_in_realtime) {
+                AutomatedLinkPageResult::add_or_update($page);
+            }
             $data->push($page);
         }
 
-        if (!$run_in_realtime) AutomatedLinkPageResult::remove_old_data();
+        if (!$run_in_realtime) {
+            AutomatedLinkPageResult::remove_old_data();
+        }
 
         return $data;
     }
@@ -78,15 +88,18 @@ 
         // Set a list of all fields that can have autolinks created in them
         $page->AutomateableFields = ArrayList::create();
 
-        foreach (AutomatedLink::getAllDatabaseFields($page->class) as $field => $type)
-            if (in_array($field, $includeIn) &&
+        foreach (AutomatedLink::getAllDatabaseFields($page->class) as $field => $type) {
+                    if (in_array($field, $includeIn) &&
                 !$page->AutomateableFields->find('DataField', $field) &&
                 AutomatedLink::isFieldParsable($page, $field)
             ) $page->AutomateableFields->push(DataObject::create(array('DataField' => $field)));
+        }
 
         // Get data Pre-Automated Links creation
         $withLinks = $this->getPageDOM($page);
-        if (!$withLinks) return false;
+        if (!$withLinks) {
+            return false;
+        }
 
         $links = $withLinks->getElementsByTagName('a');
 
@@ -96,10 +109,11 @@ 
 
         // List all automated links that were created in this $page
         $linksUsed = array();
-        foreach ($this->Links as $autolink)
-            foreach ($links as $link) {
+        foreach ($this->Links as $autolink) {
+                    foreach ($links as $link) {
                 if ($link->getAttribute('data-id') == $autolink->ID) {
                     $linksUsed[$autolink->ID] = $autolink->Phrase;
+        }
                     $page->OriginalLinkCount--;
                     $page->LinkCount++;
                 }
@@ -107,7 +121,9 @@ 
 
         $page->Links = implode(', ', $linksUsed);
 
-        if ($page->LinkCount < 1) return false;
+        if ($page->LinkCount < 1) {
+            return false;
+        }
 
         return $page;
     }
@@ -122,8 +138,12 @@ 
      */
     private function getPageDOM(SiteTree $page) {
         $controllerClass = $page->class.'_Controller';
-        if (!class_exists($controllerClass))  $controllerClass = $page->class.'Controller';
-        if (!class_exists($controllerClass)) return false;
+        if (!class_exists($controllerClass)) {
+            $controllerClass = $page->class.'Controller';
+        }
+        if (!class_exists($controllerClass)) {
+            return false;
+        }
 
         $controller = $controllerClass::create($page);
         $controller->invokeWithExtensions('addAutomatedLinks');
@@ -154,9 +174,10 @@ 
      * @return Boolean
      */
     private function checkForPossibleLinks(SiteTree $page, array $includeIn) {
-        foreach ($this->Links as $link)
-            foreach ($includeIn as $possibleField)
+        foreach ($this->Links as $link) {
+                    foreach ($includeIn as $possibleField)
                 if (isset($page->$possibleField) && preg_match("/\b{$link->Phrase}\b/i", $page->$possibleField)) return true;
+        }
 
         return false;
     }