dylangrech92 / seotoolbox

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')
         );

code/dataobjects/AutomatedLink.php

Spacing +55 added lines, -55 removed lines

@@ -7,7 +7,7 @@ 
  * Automated Link is a dataobject that contains all the data
  * about a link that should be created automatically to a page
  */
-class AutomatedLink extends DataObject implements PermissionProvider{
+class AutomatedLink extends DataObject implements PermissionProvider {
 
 	private static $db = array(
 		'Phrase'     	  => 'VARCHAR(255)',
@@ -31,37 +31,37 @@ 
 		'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();
+    private static $parsableFields = array();
 
-    public function PointsTo(){
+    public function PointsTo() {
         return $this->Page()->Link();
     }
 
-	public function Title(){
+	public function Title() {
 		return $this->Phrase;
 	}
 
-	public function forTemplate(){
+	public function forTemplate() {
 		return $this->getHTML();
 	}
 
-    function canView( $member = false ){
+    function canView($member = false) {
         return Permission::check('AUTOMATEDLINK_VIEW');
     }
 
-    function canEdit( $member = false ){
+    function canEdit($member = false) {
         return Permission::check('AUTOMATEDLINK_EDIT');
     }
 
-    function canDelete( $member = false ){
+    function canDelete($member = false) {
         return Permission::check('AUTOMATEDLINK_DELETE');
     }
 
-    function canCreate( $member = false ){
+    function canCreate($member = false) {
         return Permission::check('AUTOMATEDLINK_CREATE');
     }
 
@@ -74,11 +74,11 @@ 
        );
      }
 
-	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);
 	}
 
 	/**
@@ -87,36 +87,36 @@ 
      * @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"' 		    : '';
-        $anchor   = ( $originalPhrase )   ? $originalPhrase             : $this->Phrase;
-		$link     = ( $this->AnchorTag )  ? rtrim( $link, '#' ).'#'.$this->AnchorTag : $link;
+	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' ) );
+	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 ),
+		$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' )
+				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' )
-		) );
+			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(
+		if ($settings) {
+			$fields->addFieldsToTab('Root.Global', array(
 				NumericField::create(
 					'Global_MaxLinksPerPage',
 					'Maximum amount of links a single page can have ( 0 = unlimited )',
@@ -136,14 +136,14 @@ 
 					'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' ) );
+    public function getCMSValidator() {
+        return new RequiredFields(array('Phrase', 'PageID'));
     }
 
 	/**
@@ -152,15 +152,15 @@ 
 	 *
 	 * @return void
 	 */
-	public function onBeforeWrite(){
+	public function onBeforeWrite() {
 		parent::onBeforeWrite();
 
 		$settings = GlobalAutoLinkSettings::get_current();
-		if( $settings ){
+		if ($settings) {
 
-			foreach( $this->getChangedFields() as $field => $value ){
-				if( strpos( $field, 'Global_' ) === 0 && isset( $value['after'] ) ){
-					$field = str_replace( 'Global_', '', $field );
+			foreach ($this->getChangedFields() as $field => $value) {
+				if (strpos($field, 'Global_') === 0 && isset($value['after'])) {
+					$field = str_replace('Global_', '', $field);
 					$settings->$field = $value['after'];
 				}
 			}
@@ -178,15 +178,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 ) $this->write();
+			if ($write) $this->write();
 		}
 	}
 
@@ -198,15 +198,15 @@ 
      *
      * @return Boolean
      */
-    public static function isFieldParsable( SiteTree $page, $field ){
-        if( !isset( self::$parsableFields[$page->ID] ) || !isset( self::$parsableFields[$page->ID][$field] ) ){
+    public static function isFieldParsable(SiteTree $page, $field) {
+        if (!isset(self::$parsableFields[$page->ID]) || !isset(self::$parsableFields[$page->ID][$field])) {
             $fields = array();
 
-            foreach( ClassInfo::ancestry( $page->ClassName, true ) as $class )
-                $fields = array_merge( $fields, (array) DataObject::database_fields( $class ) );
+            foreach (ClassInfo::ancestry($page->ClassName, true) as $class)
+                $fields = array_merge($fields, (array) DataObject::database_fields($class));
 
             self::$parsableFields[$page->ID][$field] =
-                (Boolean) array_key_exists( $field, $fields ) && strtolower( $fields[$field] ) === 'htmltext' && $page->$field;
+                (Boolean) array_key_exists($field, $fields) && strtolower($fields[$field]) === 'htmltext' && $page->$field;
         }
 
         return self::$parsableFields[$page->ID][$field];
@@ -220,7 +220,7 @@ 
      *
      * @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);
     }
 }