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

Spacing +6 added lines, -6 removed lines

@@ -9,19 +9,19 @@
  * all the automated links and automated link global settings
  * for this site
  */
-class SEOToolboxAdmin extends ModelAdmin{
+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_priority   = 10;
+	private static $managed_models  = array('AutomatedLink');
+    private static $menu_priority = 10;
 
 
 	public function getEditForm($id = null, $fields = null) {
-		$form = parent::getEditForm( $id, $fields );
+		$form = parent::getEditForm($id, $fields);
 		$class = $this->sanitiseClassName($this->modelClass);
-		$grid   = $form->Fields()->fieldByName($class);
-		$grid->getConfig()->addComponent( new GridFieldSortableRows( 'Priority' ) );
+		$grid = $form->Fields()->fieldByName($class);
+		$grid->getConfig()->addComponent(new GridFieldSortableRows('Priority'));
 		
 		return $form;
 	}

code/logic/GridFieldExportAllAutomatedLinksButton.php

Spacing +10 added lines, -10 removed lines

@@ -3,29 +3,29 @@
     class GridFieldExportAllAutomatedLinksButton extends GridFieldExportButton {
         public function generateExportFileData($gridField) {
             $separator = $this->csvSeparator;
-            $csvColumns =$gridField->getColumns();
+            $csvColumns = $gridField->getColumns();
             $fileData = '';
-            if($this->csvHasHeader) {
+            if ($this->csvHasHeader) {
                 $headers = array();
-                foreach($csvColumns as $columnSource => $columnHeader) {
+                foreach ($csvColumns as $columnSource => $columnHeader) {
                     $headers[] = (!is_string($columnHeader) && is_callable($columnHeader)) ? $columnSource : $columnHeader;
                 }
-                $fileData .= "\"" . implode("\"{$separator}\"", array_values($headers)) . "\"";
+                $fileData .= "\"".implode("\"{$separator}\"", array_values($headers))."\"";
                 $fileData .= "\n";
             }
             $items = $gridField->getList();
 
-            foreach($gridField->getConfig()->getComponents() as $component){
-                if($component instanceof GridFieldFilterHeader || $component instanceof GridFieldSortableHeader) {
+            foreach ($gridField->getConfig()->getComponents() as $component) {
+                if ($component instanceof GridFieldFilterHeader || $component instanceof GridFieldSortableHeader) {
                     $items = $component->getManipulatedData($gridField, $items);
                 }
             }
-            foreach($items->limit(null) as $item) {
+            foreach ($items->limit(null) as $item) {
                 $columnData = array();
-                foreach($csvColumns as $columnSource => $columnHeader) {
-                    $value = ( $item->hasMethod( $columnHeader ) ) ? $item->$columnHeader() : $item->$columnHeader;
+                foreach ($csvColumns as $columnSource => $columnHeader) {
+                    $value = ($item->hasMethod($columnHeader)) ? $item->$columnHeader() : $item->$columnHeader;
                     $value = str_replace(array("\r", "\n"), "\n", $value);
-                    $columnData[] = '"' . str_replace('"', '\"', $value) . '"';
+                    $columnData[] = '"'.str_replace('"', '\"', $value).'"';
                 }
                 $fileData .= implode($separator, $columnData);
                 $fileData .= "\n";

code/logic/GridFieldPrintAllAutomatedLinksButton.php

Spacing +4 added lines, -4 removed lines

@@ -4,9 +4,9 @@ 
         public function generatePrintData(GridField $gridField) {
             $printColumns = $this->getPrintColumnsForGridField($gridField);
             $header = null;
-            if($this->printHasHeader) {
+            if ($this->printHasHeader) {
                 $header = new ArrayList();
-                foreach($printColumns as $field => $label){
+                foreach ($printColumns as $field => $label) {
                     $header->push(new ArrayData(array(
                         "CellString" => $label,
                         )));
@@ -15,9 +15,9 @@ 
 
             $items = $gridField->getList();
             $itemRows = new ArrayList();
-            foreach($items as $item) {
+            foreach ($items as $item) {
                 $itemRow = new ArrayList();
-                foreach($printColumns as $field => $label) {
+                foreach ($printColumns as $field => $label) {
                     $value = $gridField->getDataFieldValue($item, $field);
                     $itemRow->push(new ArrayData(array(
                         "CellString" => $value,

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;

Indentation +120 added lines, -120 removed lines

@@ -93,133 +93,133 @@
     }
 
     public 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

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

Spacing +18 added lines, -18 removed lines

@@ -14,9 +14,9 @@ 
  * @property string AddTo
  * @property string CrawlID
  */
-class GlobalAutoLinkSettings extends DataObject{
+class GlobalAutoLinkSettings extends DataObject {
 
-	public static $enabled  = true;
+	public static $enabled = true;
     public static $encoding = 'UTF-8';
 
 	private static $db = array(
@@ -42,8 +42,8 @@ 
      *
      * @return array
      */
-	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'));
 	}
 
     /**
@@ -51,19 +51,19 @@ 
      *
      * @return array
      */
-	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();
-        if(!$hasData) {
+        if (!$hasData) {
             $obj = self::create(self::$default_create_config);
             $obj->CrawlID = $this->createCrawlID();
             $obj->write();
-            DB::alteration_message("Added default Global Auto Link Settings","created");
-        } else{
-            if(!$hasData->CrawlID){
+            DB::alteration_message("Added default Global Auto Link Settings", "created");
+        }else {
+            if (!$hasData->CrawlID) {
                 $hasData->CrawlID = $this->createCrawlID();
                 $hasData->write();
             }
@@ -96,25 +96,25 @@ 
      *
      * @return array
      */
-        public function AllowedIn(){
-        $classes = array_values( ClassInfo::subclassesFor( 'SiteTree' ) );
-        if( !$this->AddTo ) return $classes;
+        public function AllowedIn() {
+        $classes = array_values(ClassInfo::subclassesFor('SiteTree'));
+        if (!$this->AddTo) return $classes;
 
-        $sanitized = explode( ',', str_replace( ' ', '', strtolower( $this->AddTo ) ) );
+        $sanitized = explode(',', str_replace(' ', '', strtolower($this->AddTo)));
         $len       = count($sanitized);
 
-        for( $x = 0; $x < $len; $x++ ){
+        for ($x = 0; $x < $len; $x++) {
             $found = false;
 
-            foreach( $classes as $class ){
-                if( strtolower( $class ) === $sanitized[$x] ){
+            foreach ($classes as $class) {
+                if (strtolower($class) === $sanitized[$x]) {
                     $sanitized[$x] = $class;
                     $found = true;
                     break 1;
                 }
             }
 
-            if( !$found ) unset( $sanitized[$x] );
+            if (!$found) unset($sanitized[$x]);
         }
 
         return (array) $sanitized;

Indentation +16 added lines, -16 removed lines

@@ -16,20 +16,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,
@@ -42,18 +42,18 @@ 
      *
      * @return array
      */
-	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' ) );
+    }
 
     /**
      * Return an array of where automated links can be created
      *
      * @return array
      */
-	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();

code/extenstions/SEOToolboxControllerExtension.php

Spacing +54 added lines, -54 removed lines

@@ -18,12 +18,12 @@ 
     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());
         }
@@ -31,15 +31,15 @@ 
         return array();
     }
 
-    private function crawl_response(){
+    private function crawl_response() {
         // Encoded version to detect which fields are being used
         $customize = array();
         $dbFields  = Config::inst()->get($this->owner->ClassName, 'db');
-        if(is_array($dbFields)) {
-            foreach ( $dbFields as $field => $type) {
+        if (is_array($dbFields)) {
+            foreach ($dbFields as $field => $type) {
                 if (strtolower($type) == 'htmltext') {
                     $data = ($this->owner->hasMethod($field)) ? $this->owner->$field() : $this->owner->$field;
-                    if($data){
+                    if ($data) {
                         $tmp = new HTMLText('tmp');
                         $tmp->setValue($data);
                         $data = base64_encode($tmp->forTemplate());
@@ -90,31 +90,31 @@ 
      *
      * @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 ) {
+            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);
                     $dom = AutomatedLink::constructDOMDocument($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 ) {
+                    $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;
                 }
@@ -130,19 +130,19 @@ 
      * @param array $hash_tags
      * @return array
      */
-    private function hashTags( DOMDocument $html, $hash_tags ){
+    private function hashTags(DOMDocument $html, $hash_tags) {
         $excluded = array();
-        foreach( $hash_tags 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 ($hash_tags 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);
             }
         }
 
@@ -157,38 +157,38 @@ 
      * @param string $content
      * @return array
      */
-    public function addLinkToContent(AutomatedLink $link, &$content){
+    public function addLinkToContent(AutomatedLink $link, &$content) {
         $links  = array();
-        $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 ) $links;
+        preg_match_all($regex, $content, $count);
+        $count = (is_array($count) && isset($count[0])) ? count($count[0]) : 0;
+        if ($count < 1) $links;
 
-        if( isset( $this->maxLinksPerPage[ $link->ID ] ) ) {
+        if (isset($this->maxLinksPerPage[$link->ID])) {
             $max -= $this->maxLinksPerPage[$link->ID];
-        } else {
+        }else {
             $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 );
-            $links[ $key ]  = (string) $html;
+            if (!$html = (string) $link->getHTML($match[0])) continue;
+            $key            = (string) crc32($html);
+            $links[$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]++;
         }
 
         return $links;
@@ -201,24 +201,24 @@ 
      * @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   = $this->hashTags($html, $this->excludeTags);
-        $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);
 
-        foreach( AutomatedLink::get()->sort('Priority') as $link){
-            if( $this->linkCount < $this->maxLinks && $link->canBeAdded( $this->owner, $field ) ) {
+        foreach (AutomatedLink::get()->sort('Priority') as $link) {
+            if ($this->linkCount < $this->maxLinks && $link->canBeAdded($this->owner, $field)) {
                 $links = $this->addLinkToContent($link, $content);
-                if( is_array($links) && count($links) > 0 ) {
-                    $excluded = $excluded + $links;
+                if (is_array($links) && count($links) > 0) {
+                    $excluded = $excluded+$links;
                 }
             }
         }
 
         // Re-add the excluded Tags
-        return str_replace( array_keys( $excluded ), array_values( $excluded ), $content );
+        return str_replace(array_keys($excluded), array_values($excluded), $content);
     }
 }

_config.php

Spacing +4 added lines, -4 removed lines

@@ -6,15 +6,15 @@
  *
  */
 
-define( 'SEOTOOLBOX_DIR', str_replace( Director::baseFolder().'/', '', __DIR__ ) );
+define('SEOTOOLBOX_DIR', str_replace(Director::baseFolder().'/', '', __DIR__));
 
 Config::inst()->update('SEOToolboxAdmin', 'menu_icon', SEOTOOLBOX_DIR.'/code/admin/icon/seotoolbox-icon.png');
-Config::inst()->update('LeftAndMain','extra_requirements_javascript', array(SEOTOOLBOX_DIR.'/js/seo_page_analyzer.js'));
-Config::inst()->update('LeftAndMain','extra_requirements_css', array(SEOTOOLBOX_DIR.'/css/seo_page_analyzer.css'));
+Config::inst()->update('LeftAndMain', 'extra_requirements_javascript', array(SEOTOOLBOX_DIR.'/js/seo_page_analyzer.js'));
+Config::inst()->update('LeftAndMain', 'extra_requirements_css', array(SEOTOOLBOX_DIR.'/css/seo_page_analyzer.css'));
 
 CMSMenu::add_link('seotest', 'Run Crawler', '/seotest', 9, array('target' => '_blank'));
 Requirements::customCSS('.icon.icon-16.icon-seotest{background:url('.SEOTOOLBOX_DIR.'/code/admin/icon/crawl-icon.png)}');
 
-if(Director::is_cli()){
+if (Director::is_cli()) {
     Config::inst()->update('Director', 'environment_type', 'dev');
 }

tests/AutomatedLinkTest.php

Spacing +25 added lines, -25 removed lines

@@ -13,8 +13,8 @@ 
     /**
      * Test that links are included in the content field
      */
-    public function testInsertion(){
-        $this->objFromFixture('AutomatedLink','link');
+    public function testInsertion() {
+        $this->objFromFixture('AutomatedLink', 'link');
         $page   = $this->createPage('<p>Checking if phrase is replaced</p>');
         $link   = $this->getLinkFromPage($page);
         $this->assertTrue($link->nodeValue === 'phrase', 'Link wasn\'t created');
@@ -23,41 +23,41 @@ 
     /**
      * Test the individual properties of the automated link
      */
-    public function testProperties(){
-        $this->objFromFixture('AutomatedLink','link2');
+    public function testProperties() {
+        $this->objFromFixture('AutomatedLink', 'link2');
 
         $page   = self::createPage('<p>Checking if properties is with correct props</p>');
         $link   = self::getLinkFromPage($page);
 
-        $this->assertTrue( $link->nodeValue === 'properties', 'Phrase did not match' );
-        $this->assertTrue( $link->getAttribute('title') === 'test title', 'Title did not match' );
-        $this->assertTrue( $link->getAttribute('href') === '#test-anchor', 'Anchor did not match' );
-        $this->assertTrue( $link->getAttribute('target') === '_blank', 'Target did not match' );
-        $this->assertTrue( $link->getAttribute('rel') === 'nofollow', 'Rel did not match' );
+        $this->assertTrue($link->nodeValue === 'properties', 'Phrase did not match');
+        $this->assertTrue($link->getAttribute('title') === 'test title', 'Title did not match');
+        $this->assertTrue($link->getAttribute('href') === '#test-anchor', 'Anchor did not match');
+        $this->assertTrue($link->getAttribute('target') === '_blank', 'Target did not match');
+        $this->assertTrue($link->getAttribute('rel') === 'nofollow', 'Rel did not match');
     }
 
     /**
      * Test that the default properties match what we expect
      */
-    public function testDefaultProperties(){
-        $this->objFromFixture('AutomatedLink','link3');
+    public function testDefaultProperties() {
+        $this->objFromFixture('AutomatedLink', 'link3');
 
         $page   = self::createPage('<p>Checking if default is correct</p>');
         $link   = self::getLinkFromPage($page);
         $auto   = AutomatedLink::get()->find('Phrase', 'default');
 
-        $this->assertTrue( $link->nodeValue === 'default', 'Phrase did not match' );
-        $this->assertTrue( $link->getAttribute('title') === '', 'Title did not match' );
-        $this->assertTrue( $link->getAttribute('href') === '#', 'href did not match' );
-        $this->assertTrue( $link->getAttribute('target') === '', 'Target did not match' );
-        $this->assertTrue( $link->getAttribute('rel') === '', 'Rel did not match' );
-        $this->assertTrue( (int) $link->getAttribute('data-id') === $auto->ID, 'data-id did not match' );
+        $this->assertTrue($link->nodeValue === 'default', 'Phrase did not match');
+        $this->assertTrue($link->getAttribute('title') === '', 'Title did not match');
+        $this->assertTrue($link->getAttribute('href') === '#', 'href did not match');
+        $this->assertTrue($link->getAttribute('target') === '', 'Target did not match');
+        $this->assertTrue($link->getAttribute('rel') === '', 'Rel did not match');
+        $this->assertTrue((int) $link->getAttribute('data-id') === $auto->ID, 'data-id did not match');
     }
 
     /**
      * Test that the default properties match what we expect
      */
-    public function testLimitFilter(){
+    public function testLimitFilter() {
         $page   = self::createPage('<p>Checking if default is created only twice. default default</p>');
         $dom    = self::getPageDOM($page);
         $links  = $dom->getElementsByTagName('a');
@@ -67,7 +67,7 @@ 
     /**
      * Test that the default properties match what we expect
      */
-    public function testDisableFilter(){
+    public function testDisableFilter() {
         GlobalAutoLinkSettings::$enabled = false;
         $page   = self::createPage('<p>Checking if default is created at all when automated links are disabled.</p>');
         $links  = self::getLinksFromPage($page);
@@ -81,7 +81,7 @@ 
      * @param SiteTree $page
      * @return DOMNameList
      */
-    public static function getLinksFromPage(SiteTree $page){
+    public static function getLinksFromPage(SiteTree $page) {
         $dom = self::getPageDOM($page);
         return $dom->getElementsByTagName('a');
     }
@@ -94,9 +94,9 @@ 
      * @param SiteTree $page
      * @return DOMDocument
      */
-    public static function getLinkFromPage(SiteTree $page){
+    public static function getLinkFromPage(SiteTree $page) {
         $links = self::getLinksFromPage($page);
-        return ( $links->length != 1 ) ? new DOMElement() : $links->item(0);
+        return ($links->length != 1) ? new DOMElement() : $links->item(0);
     }
 
     /**
@@ -105,8 +105,8 @@ 
      * @param string|null $content
      * @return Page
      */
-    public static function createPage($content=null){
-        $page = Page::create(array( 'Content' => $content ));
+    public static function createPage($content = null) {
+        $page = Page::create(array('Content' => $content));
         $page->write();
         return $page;
     }
@@ -117,7 +117,7 @@ 
      * @param Page $page
      * @return DOMDocument
      */
-    public static function getPageDOM(Page $page){
+    public static function getPageDOM(Page $page) {
         $controller = Page_Controller::create($page);
         $controller->invokeWithExtensions('addAutomatedLinks');
         return AutomatedLink::constructDOMDocument($controller->Content);