dylangrech92 / seotoolbox

code/admin/SEOToolboxAdmin.php

Indentation +10 added lines, -10 removed lines

@@ -11,18 +11,18 @@
  */
 class SEOToolboxAdmin extends ModelAdmin{
 	
-	private static $menu_title      = 'SEO Tool box';
-	private static $url_segment     = 'seo-tool-box';
-	private static $managed_models  = array( 'AutomatedLink' );
+    private static $menu_title      = 'SEO Tool box';
+    private static $url_segment     = 'seo-tool-box';
+    private static $managed_models  = array( 'AutomatedLink' );
     private static $menu_priority   = 10;
 
 
-	public function getEditForm($id = null, $fields = null) {
-		$form = parent::getEditForm( $id, $fields );
-		$class = $this->sanitiseClassName($this->modelClass);
-		$grid   = $form->Fields()->fieldByName($class);
-		$grid->getConfig()->addComponent( new GridFieldSortableRows( 'Priority' ) );
+    public function getEditForm($id = null, $fields = null) {
+        $form = parent::getEditForm( $id, $fields );
+        $class = $this->sanitiseClassName($this->modelClass);
+        $grid   = $form->Fields()->fieldByName($class);
+        $grid->getConfig()->addComponent( new GridFieldSortableRows( 'Priority' ) );
 		
-		return $form;
-	}
+        return $form;
+    }
 }

code/dataobjects/AutomatedLink.php

Indentation +120 added lines, -120 removed lines

@@ -66,133 +66,133 @@
     }
 
     function providePermissions() {
-       return array(
-         'AUTOMATEDLINK_VIEW'   => 'View Automated Links',
-         'AUTOMATEDLINK_EDIT'   => 'Edit Automated Links',
-         'AUTOMATEDLINK_DELETE' => 'Delete Automated Links',
-         'AUTOMATEDLINK_CREATE' => 'Create Automated Links',
-       );
-     }
-
-	public function requireDefaultRecords(){
-		parent::requireDefaultRecords();
-
-		// Update all links to redirector pages during dev/build
-		foreach( self::get() as $link ) {
-		    $link->CheckAndUpdateDestination( true );
-		}
-	}
-
-	/**
-	 * Returns the HTML Representation of this object
-	 *
+        return array(
+            'AUTOMATEDLINK_VIEW'   => 'View Automated Links',
+            'AUTOMATEDLINK_EDIT'   => 'Edit Automated Links',
+            'AUTOMATEDLINK_DELETE' => 'Delete Automated Links',
+            'AUTOMATEDLINK_CREATE' => 'Create Automated Links',
+        );
+        }
+
+    public function requireDefaultRecords(){
+        parent::requireDefaultRecords();
+
+        // Update all links to redirector pages during dev/build
+        foreach( self::get() as $link ) {
+            $link->CheckAndUpdateDestination( true );
+        }
+    }
+
+    /**
+     * Returns the HTML Representation of this object
+     *
      * @param  String $originalPhrase
-	 * @return String
-	 */
-	public function getHTML($originalPhrase = NULL) {
-		$link     = ($this->PageID) ? $this->Page()->Link() : '#';
-		$title    = ($this->TitleTag) ? "title='{$this->TitleTag}'" : '';
-		$nofollow = ($this->NoFollow) ? 'rel="nofollow"' : '';
-		$newtab   = ($this->NewWindow) ? 'target="_blank"' : '';
+     * @return String
+     */
+    public function getHTML($originalPhrase = NULL) {
+        $link     = ($this->PageID) ? $this->Page()->Link() : '#';
+        $title    = ($this->TitleTag) ? "title='{$this->TitleTag}'" : '';
+        $nofollow = ($this->NoFollow) ? 'rel="nofollow"' : '';
+        $newtab   = ($this->NewWindow) ? 'target="_blank"' : '';
         $anchor = ($originalPhrase) ? $originalPhrase : $this->Phrase;
-		$link     = ($this->AnchorTag) ? rtrim($link, '#').'#'.$this->AnchorTag : $link;
-		return "<a href=\"$link\" $title $nofollow $newtab data-id=\"{$this->ID}\">{$anchor}</a>";
-	}
-
-	public function getCMSFields() {
-		$fields = FieldList::create(TabSet::create('Root'));
-
-		$fields->addFieldsToTab('Root.LinkSettings', array(
-			TextField::create('Phrase', 'Phrase to search for', $this->Phrase, 255),
-			TextField::create('TitleTag', 'Title Tag', $this->TitleTag, 255),
-			TextField::create('AnchorTag', 'Anchor Tag(#)', $this->AnchorTag, 255),
-			FieldGroup::create(
-				CheckboxField::create('NoFollow'),
-				CheckboxField::create('NewWindow'),
-				CheckboxField::create('SelfLinking', 'Allow page to link to itself'),
-				CheckboxField::create('CaseSensitive', 'Match the case of the phrase')
-			),
-			NumericField::create('MaxLinksPerPage', 'Maximum amount of this link to be created on a single page( 0 = unlimited )'),
-			TreeDropdownField::create('PageID', 'Page to link to', 'SiteTree')
-		));
-
-		$settings = GlobalAutoLinkSettings::get_current();
-		if ($settings) {
-			$fields->addFieldsToTab('Root.Global', array(
-				NumericField::create(
-					'Global_MaxLinksPerPage',
-					'Maximum amount of links a single page can have ( 0 = unlimited )',
-					$settings->MaxLinksPerPage
-				),
-				TextField::create(
-					'Global_ExcludeTags',
-					'Do not include links into these HTML Tags ( comma seperated )',
-					$settings->ExcludeTags
-				),
-				TextField::create(
-				    'Global_AddTo',
-				    'Page types where links should be created in ( leave blank for all page types )',
-				    $settings->AddTo ),
-				TextField::create(
-					'Global_IncludeIn',
-					'Include Links into these fields ( comma seperated & field must support html injection )',
-					$settings->IncludeIn
-				)
-			));
-		}
-
-		return $fields;
-	}
+        $link     = ($this->AnchorTag) ? rtrim($link, '#').'#'.$this->AnchorTag : $link;
+        return "<a href=\"$link\" $title $nofollow $newtab data-id=\"{$this->ID}\">{$anchor}</a>";
+    }
+
+    public function getCMSFields() {
+        $fields = FieldList::create(TabSet::create('Root'));
+
+        $fields->addFieldsToTab('Root.LinkSettings', array(
+            TextField::create('Phrase', 'Phrase to search for', $this->Phrase, 255),
+            TextField::create('TitleTag', 'Title Tag', $this->TitleTag, 255),
+            TextField::create('AnchorTag', 'Anchor Tag(#)', $this->AnchorTag, 255),
+            FieldGroup::create(
+                CheckboxField::create('NoFollow'),
+                CheckboxField::create('NewWindow'),
+                CheckboxField::create('SelfLinking', 'Allow page to link to itself'),
+                CheckboxField::create('CaseSensitive', 'Match the case of the phrase')
+            ),
+            NumericField::create('MaxLinksPerPage', 'Maximum amount of this link to be created on a single page( 0 = unlimited )'),
+            TreeDropdownField::create('PageID', 'Page to link to', 'SiteTree')
+        ));
+
+        $settings = GlobalAutoLinkSettings::get_current();
+        if ($settings) {
+            $fields->addFieldsToTab('Root.Global', array(
+                NumericField::create(
+                    'Global_MaxLinksPerPage',
+                    'Maximum amount of links a single page can have ( 0 = unlimited )',
+                    $settings->MaxLinksPerPage
+                ),
+                TextField::create(
+                    'Global_ExcludeTags',
+                    'Do not include links into these HTML Tags ( comma seperated )',
+                    $settings->ExcludeTags
+                ),
+                TextField::create(
+                    'Global_AddTo',
+                    'Page types where links should be created in ( leave blank for all page types )',
+                    $settings->AddTo ),
+                TextField::create(
+                    'Global_IncludeIn',
+                    'Include Links into these fields ( comma seperated & field must support html injection )',
+                    $settings->IncludeIn
+                )
+            ));
+        }
+
+        return $fields;
+    }
 
     public function getCMSValidator() {
         return new RequiredFields(array('Phrase', 'PageID'));
     }
 
-	/**
-	 * Save the Global Settings into the
-	 * Global Auto Link Settings Object
-	 *
-	 * @return void
-	 */
-	public function onBeforeWrite() {
-		parent::onBeforeWrite();
-
-		$settings = GlobalAutoLinkSettings::get_current();
-		if ($settings) {
-
-			foreach ($this->getChangedFields() as $field => $value) {
-				if (strpos($field, 'Global_') === 0 && isset($value['after'])) {
-					$field = str_replace('Global_', '', $field);
-					$settings->$field = $value['after'];
-				}
-			}
-
-			$settings->write();
-		}
-
-		$this->CheckAndUpdateDestination();
-	}
-
-	/**
-	 * Checks if the destination is a redirector page if so
-	 * it updates it to the destination of the redirector page
-	 *
-	 * @Boolean $write - Write the changes if any
-	 * @return void
-	 */
-	public function CheckAndUpdateDestination( $write = false ){
-		$this->extend('beforeCheckAndUpdateDestination', $write);
-
-		if( $this->PageID && $this->Page() &&
-			$this->Page()->ClassName == 'RedirectorPage' &&
-			$this->Page()->LinkToID && $this->Page()->RedirectionType == 'Internal' )
-		{
-			$this->PageID = $this->Page()->LinkToID;
-			if( $write ) {
-			    $this->write();
-			}
-		}
-	}
+    /**
+     * Save the Global Settings into the
+     * Global Auto Link Settings Object
+     *
+     * @return void
+     */
+    public function onBeforeWrite() {
+        parent::onBeforeWrite();
+
+        $settings = GlobalAutoLinkSettings::get_current();
+        if ($settings) {
+
+            foreach ($this->getChangedFields() as $field => $value) {
+                if (strpos($field, 'Global_') === 0 && isset($value['after'])) {
+                    $field = str_replace('Global_', '', $field);
+                    $settings->$field = $value['after'];
+                }
+            }
+
+            $settings->write();
+        }
+
+        $this->CheckAndUpdateDestination();
+    }
+
+    /**
+     * Checks if the destination is a redirector page if so
+     * it updates it to the destination of the redirector page
+     *
+     * @Boolean $write - Write the changes if any
+     * @return void
+     */
+    public function CheckAndUpdateDestination( $write = false ){
+        $this->extend('beforeCheckAndUpdateDestination', $write);
+
+        if( $this->PageID && $this->Page() &&
+            $this->Page()->ClassName == 'RedirectorPage' &&
+            $this->Page()->LinkToID && $this->Page()->RedirectionType == 'Internal' )
+        {
+            $this->PageID = $this->Page()->LinkToID;
+            if( $write ) {
+                $this->write();
+            }
+        }
+    }
 
     /**
      * Checks if the field is parable

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/dataobjects/AutomatedLinkPageResult.php

Braces +6 added lines, -2 removed lines

@@ -52,7 +52,9 @@ 
      */
     public static function add_or_update(SiteTree $page) {
         $obj = self::get()->find('PageID', $page->ID);
-        if (!$obj) $obj = self::create(array('PageID' => $page->ID));
+        if (!$obj) {
+            $obj = self::create(array('PageID' => $page->ID));
+        }
 
         $obj->OriginalLinkCount = $page->OriginalLinkCount;
         $obj->LinksCreatedCount = $page->LinkCount;
@@ -68,6 +70,8 @@ 
      * @return void
      */
     public static function remove_old_data() {
-        foreach (self::get() as $obj) if (!SiteTree::get()->byID($obj->PageID)) $obj->delete();
+        foreach (self::get() as $obj) {
+            if (!SiteTree::get()->byID($obj->PageID)) $obj->delete();
+        }
     }
 }

code/dataobjects/GlobalAutoLinkSettings.php

Indentation +16 added lines, -16 removed lines

@@ -10,20 +10,20 @@ 
  */
 class GlobalAutoLinkSettings extends DataObject{
 
-	public static $enabled  = true;
+    public static $enabled  = true;
     public static $encoding = 'UTF-8';
 
-	private static $db = array(
-		'MaxLinksPerPage' => 'INT',
-		'ExcludeTags'	  => 'VARCHAR(255)',
-		'IncludeIn'		  => 'Text',
-		'AddTo'           => 'Text',
+    private static $db = array(
+        'MaxLinksPerPage' => 'INT',
+        'ExcludeTags'	  => 'VARCHAR(255)',
+        'IncludeIn'		  => 'Text',
+        'AddTo'           => 'Text',
         'CrawlID'         => 'VARCHAR(15)'
-	);
+    );
 
-	private static $defaults = array(
-		'IncludeIn' => 'Content'
-	);
+    private static $defaults = array(
+        'IncludeIn' => 'Content'
+    );
 
     public static $default_create_config = array(
         'MaxLinksPerPage' => 0,
@@ -31,13 +31,13 @@ 
         'IncludeIn'		  => 'Content'
     );
 
-	public function ExcludeTags(){
-		return array_unique( explode( ',', str_replace( ' ', '', $this->ExcludeTags ).',a,img,iframe,video,object' ) );
-	}
+    public function ExcludeTags(){
+        return array_unique( explode( ',', str_replace( ' ', '', $this->ExcludeTags ).',a,img,iframe,video,object' ) );
+    }
 
-	public function IncludeInFields(){
-		return explode( ',', str_replace( ' ', '', $this->IncludeIn ) );
-	}
+    public function IncludeInFields(){
+        return explode( ',', str_replace( ' ', '', $this->IncludeIn ) );
+    }
 
     public function requireDefaultRecords() {
         $hasData = self::get()->first();

Braces +9 added lines, -4 removed lines

@@ -61,8 +61,9 @@ 
         $alpha = 'abcdefghijklm[)0123456789(]nopqrstuvwxyz';
         $alpha_len = strlen($alpha);
 
-        while (strlen($ret) < 14)
-            $ret .= (rand(0, 1) == 0) ? strtoupper($alpha[rand(0, $alpha_len-1)]) : $alpha[rand(0, $alpha_len-1)];
+        while (strlen($ret) < 14) {
+                    $ret .= (rand(0, 1) == 0) ? strtoupper($alpha[rand(0, $alpha_len-1)]) : $alpha[rand(0, $alpha_len-1)];
+        }
 
         return $ret;
     }
@@ -77,7 +78,9 @@ 
      */
         public function AllowedIn(){
         $classes = array_values( ClassInfo::subclassesFor( 'SiteTree' ) );
-        if( !$this->AddTo ) return $classes;
+        if( !$this->AddTo ) {
+            return $classes;
+        }
 
         $sanitized = explode( ',', str_replace( ' ', '', strtolower( $this->AddTo ) ) );
 
@@ -92,7 +95,9 @@ 
                 }
             }
 
-            if( !$found ) unset( $sanitized[$x] );
+            if( !$found ) {
+                unset( $sanitized[$x] );
+            }
         }
 
         return (array) $sanitized;

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;
     }

code/extenstions/SEOToolboxControllerExtension.php

Braces +21 added lines, -7 removed lines

@@ -73,12 +73,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;
@@ -134,7 +138,9 @@ 
         $excluded = array();
         foreach( $hash_tags 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 );
@@ -166,7 +172,9 @@ 
         // Count the matches
         preg_match_all( $regex, $content, $count );
         $count = ( is_array( $count ) && isset( $count[0] ) ) ? count( $count[0] ) : 0;
-        if( $count < 1 ) $links;
+        if( $count < 1 ) {
+            $links;
+        }
 
         if( isset( $this->maxLinksPerPage[ $link->ID ] ) ) {
             $max -= $this->maxLinksPerPage[$link->ID];
@@ -176,13 +184,19 @@ 
 
         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 );
             $links[ $key ]  = (string) $html;
 

code/controllers/SEOTestSiteTreeController.php

Braces +3 added lines, -2 removed lines

@@ -113,8 +113,9 @@
                 $phrases = array_merge($phrases, $matches[$commands['find_pos']]);
             }
 
-            if (isset($commands['replace']))
-                $html = preg_replace($commands['replace'], ' ', $html);
+            if (isset($commands['replace'])) {
+                            $html = preg_replace($commands['replace'], ' ', $html);
+            }
         }
 
         // Remove the empty elements