salted-herring / silverstripe-block

code/SHBlock.php

Indentation +182 added lines, -182 removed lines

@@ -1,204 +1,204 @@
 <?php
 
 class SHBlock extends DataObject {
-	protected static $db = array (
-		'SortOrder'			=>	'Int',
-		'Title'				=>	'Varchar(64)',
-		'TitleWrapper'		=>	'Enum("h2,h3,h4,h5,h6")',
-		'hideTitle'			=>	'Boolean',
-		'showBlockbyClass'	=>	'Boolean',
-		'Description'		=>	'Varchar(128)',
-		'MemberVisibility'	=>	'Varchar(255)',
-		'shownInClass'		=>	'Text',
-		'addMarginTop'		=>	'Boolean',
-		'addMarginBottom'	=>	'Boolean'
-	);
+    protected static $db = array (
+        'SortOrder'			=>	'Int',
+        'Title'				=>	'Varchar(64)',
+        'TitleWrapper'		=>	'Enum("h2,h3,h4,h5,h6")',
+        'hideTitle'			=>	'Boolean',
+        'showBlockbyClass'	=>	'Boolean',
+        'Description'		=>	'Varchar(128)',
+        'MemberVisibility'	=>	'Varchar(255)',
+        'shownInClass'		=>	'Text',
+        'addMarginTop'		=>	'Boolean',
+        'addMarginBottom'	=>	'Boolean'
+    );
 	
-	protected static $many_many = array (
-		'Pages'				=>	'Page'
-	);
+    protected static $many_many = array (
+        'Pages'				=>	'Page'
+    );
 		
-	protected static $create_table_options = array(
-		'MySQLDatabase'		=> 'ENGINE=MyISAM'
+    protected static $create_table_options = array(
+        'MySQLDatabase'		=> 'ENGINE=MyISAM'
     );
 	
-	protected static $extensions = array (
-		'StandardPermissions'
-	);
+    protected static $extensions = array (
+        'StandardPermissions'
+    );
 	
-	protected static $summary_fields = array(
-		'BlockType',
-		'Title', 
-		'Description',
-		'shownOn',
-		'VisibleTo'
-	);
+    protected static $summary_fields = array(
+        'BlockType',
+        'Title', 
+        'Description',
+        'shownOn',
+        'VisibleTo'
+    );
 	
-	protected static $field_labels = array(
-		'BlockType'			=>	'Block type',
-		'shownOn'			=>	'is shown on',
-		'VisibleTo'			=>	'Visible to'
-	);
+    protected static $field_labels = array(
+        'BlockType'			=>	'Block type',
+        'shownOn'			=>	'is shown on',
+        'VisibleTo'			=>	'Visible to'
+    );
 	
-	public function VisibleTo() {
-		if (strlen(trim($this->MemberVisibility)) > 0) {
-			$lists = 'Group: ' . str_replace(',','<br />Group: ', $this->MemberVisibility);
-		}else{
-			$lists = '<em>&lt;All&gt;</em>';
-		}
-		
-		return new LiteralField('VisibleTo',$lists);
-	}
+    public function VisibleTo() {
+        if (strlen(trim($this->MemberVisibility)) > 0) {
+            $lists = 'Group: ' . str_replace(',','<br />Group: ', $this->MemberVisibility);
+        }else{
+            $lists = '<em>&lt;All&gt;</em>';
+        }
+		
+        return new LiteralField('VisibleTo',$lists);
+    }
 	
-	public function BlockType() {
-		return $this->singular_name();
-	}
+    public function BlockType() {
+        return $this->singular_name();
+    }
 	
-	public function shownOn() {
-		if ($this->showBlockbyClass) {
-			if (strlen(trim($this->shownInClass)) > 0) {
-				$lists = 'Type: ' . str_replace(',','<br />Type: ', $this->shownInClass);
-			}else{
-				$lists = '<em>&lt;not assigned&gt;</em>';
-			}
-		}else{
-			if ($this->Pages()->count() > 0) {
-				$lists = 'Page: ' . implode('<br />Page: ', $this->Pages()->column('Title'));
-			}else{
-				$lists = '<em>&lt;not assigned&gt;</em>';
-			}
-		}
-		return new LiteralField('shownOn',$lists);
-	}
+    public function shownOn() {
+        if ($this->showBlockbyClass) {
+            if (strlen(trim($this->shownInClass)) > 0) {
+                $lists = 'Type: ' . str_replace(',','<br />Type: ', $this->shownInClass);
+            }else{
+                $lists = '<em>&lt;not assigned&gt;</em>';
+            }
+        }else{
+            if ($this->Pages()->count() > 0) {
+                $lists = 'Page: ' . implode('<br />Page: ', $this->Pages()->column('Title'));
+            }else{
+                $lists = '<em>&lt;not assigned&gt;</em>';
+            }
+        }
+        return new LiteralField('shownOn',$lists);
+    }
 	
-	public function getCMSFields() {
-		$fields = parent::getCMSFields();
-		$fields->removeFieldFromTab('Root', 'Pages');
-		$fields->removeFieldsFromTab('Root.Main', array(
-			'SortOrder',
-			'showBlockbyClass',
-			'shownInClass',
-			'MemberVisibility'
-		));
-		
-		$memberGroups = Group::get();
-		$sourcemap = $memberGroups->map('Code', 'Title');
-		$source = array(
-			'anonymous'		=>	'Anonymous visitors'
-		);
-		foreach ($sourcemap as $mapping => $key) {
-			$source[$mapping] = $key;
-		}
-		
-		$memberVisibility = new CheckboxSetField(
-			$name = "MemberVisibility",
-			$title = "Show block for specific groups",
-			$source
-		);
-		
-		$memberVisibility->setDescription('Show this block only for the selected group(s). If you select no groups, the block will be visible to all members.');
-		
-		$availabelClasses = $this->availableClasses();
-		$inClass = new CheckboxSetField(
-			$name = "shownInClass",
-			$title = "Show block for specific content types",
-			$availabelClasses
-		);
-		
-		$filterSelector = OptionsetField::create(
-			'showBlockbyClass',
-			'Choose filter set',
-			array(
-				'0'			=>	'by page',
-				'1'			=>	'by page/data type'
-			)
-		)->setDescription('<p><br /><strong>by page</strong>: block will be displayed in the selected page(s)<br /><strong>by page/data type</strong>: block will be displayed on the pages created with the particular page/data type. e.g. is <strong>"InternalPage"</strong> is picked, the block will be displayed, and will ONLY be displayed on all <strong>Internal Pages</strong></p>');
-		
-		$availablePages = Page::get()->exclude('ClassName', array(
-			'ErrorPage',
-			'RedirectorPage',
-			'VirtualPage'
-		));
-		$pageSelector = new CheckboxSetField(
-			$name = "Pages",
-			$title = "Show on Page(s)",
-			$availablePages->map('ID','Title')
-		);
-		
-		
-		if ($this->canConfigPageAndType(Member::currentUser())) {
-			$fields->addFieldsToTab('Root.VisibilitySettings', array(
-				$filterSelector,
-				$pageSelector,
-				$inClass
-			));
-		}
-		
-		if ($this->canConfigMemberVisibility(Member::currentUser())) {
-			$fields->addFieldToTab('Root.VisibilitySettings', $memberVisibility);
-		}
-		
-		if (!$fields->fieldByName('Options')) {
-			$fields->insertBefore($right = RightSidebar::create('Options'), 'Root');
-	    }
+    public function getCMSFields() {
+        $fields = parent::getCMSFields();
+        $fields->removeFieldFromTab('Root', 'Pages');
+        $fields->removeFieldsFromTab('Root.Main', array(
+            'SortOrder',
+            'showBlockbyClass',
+            'shownInClass',
+            'MemberVisibility'
+        ));
+		
+        $memberGroups = Group::get();
+        $sourcemap = $memberGroups->map('Code', 'Title');
+        $source = array(
+            'anonymous'		=>	'Anonymous visitors'
+        );
+        foreach ($sourcemap as $mapping => $key) {
+            $source[$mapping] = $key;
+        }
+		
+        $memberVisibility = new CheckboxSetField(
+            $name = "MemberVisibility",
+            $title = "Show block for specific groups",
+            $source
+        );
+		
+        $memberVisibility->setDescription('Show this block only for the selected group(s). If you select no groups, the block will be visible to all members.');
+		
+        $availabelClasses = $this->availableClasses();
+        $inClass = new CheckboxSetField(
+            $name = "shownInClass",
+            $title = "Show block for specific content types",
+            $availabelClasses
+        );
+		
+        $filterSelector = OptionsetField::create(
+            'showBlockbyClass',
+            'Choose filter set',
+            array(
+                '0'			=>	'by page',
+                '1'			=>	'by page/data type'
+            )
+        )->setDescription('<p><br /><strong>by page</strong>: block will be displayed in the selected page(s)<br /><strong>by page/data type</strong>: block will be displayed on the pages created with the particular page/data type. e.g. is <strong>"InternalPage"</strong> is picked, the block will be displayed, and will ONLY be displayed on all <strong>Internal Pages</strong></p>');
+		
+        $availablePages = Page::get()->exclude('ClassName', array(
+            'ErrorPage',
+            'RedirectorPage',
+            'VirtualPage'
+        ));
+        $pageSelector = new CheckboxSetField(
+            $name = "Pages",
+            $title = "Show on Page(s)",
+            $availablePages->map('ID','Title')
+        );
+		
+		
+        if ($this->canConfigPageAndType(Member::currentUser())) {
+            $fields->addFieldsToTab('Root.VisibilitySettings', array(
+                $filterSelector,
+                $pageSelector,
+                $inClass
+            ));
+        }
+		
+        if ($this->canConfigMemberVisibility(Member::currentUser())) {
+            $fields->addFieldToTab('Root.VisibilitySettings', $memberVisibility);
+        }
+		
+        if (!$fields->fieldByName('Options')) {
+            $fields->insertBefore($right = RightSidebar::create('Options'), 'Root');
+        }
 	
-	    $fields->addFieldsToTab('Options', array(
-			CheckboxField::create('addMarginTop', 'add "margin-top" class to block wrapper'),
-			CheckboxField::create('addMarginBottom', 'add "margin-bottom" class to block wrapper')
-	    ));
+        $fields->addFieldsToTab('Options', array(
+            CheckboxField::create('addMarginTop', 'add "margin-top" class to block wrapper'),
+            CheckboxField::create('addMarginBottom', 'add "margin-bottom" class to block wrapper')
+        ));
 
-		return $fields;
-	}
+        return $fields;
+    }
 	
-	public function availableClasses() {
-		$Classes = array_diff(
-			ClassInfo::subclassesFor('Page'),
-			ClassInfo::subclassesFor('RedirectorPage'),
-			ClassInfo::subclassesFor('VirtualPage')
-		);
-		return $Classes;
-	}
+    public function availableClasses() {
+        $Classes = array_diff(
+            ClassInfo::subclassesFor('Page'),
+            ClassInfo::subclassesFor('RedirectorPage'),
+            ClassInfo::subclassesFor('VirtualPage')
+        );
+        return $Classes;
+    }
 	
-	public function forTemplate() {
-		if ($this->canDisplayMemberCheck()) {
-			return $this->renderWith(array($this->getClassName(), 'BaseBlock'));
-		}
+    public function forTemplate() {
+        if ($this->canDisplayMemberCheck()) {
+            return $this->renderWith(array($this->getClassName(), 'BaseBlock'));
+        }
 		
-		return false;
-	}
+        return false;
+    }
 	
-	public function canDisplayMemberCheck() {
-		$rawVisibility = $this->MemberVisibility;
-		
-		if (empty($rawVisibility)) {
-			return true;
-		}
-		
-		$visibility = explode(',', $rawVisibility);
-		$member = Member::currentUser();
-		
-		if (!$member && in_array('anonymous', $visibility)) {
-			return true;
-		}
-		
-		if ($member) {
-			$memberGroups = $member->Groups()->column('Code');
-			foreach ($memberGroups as $memberGroup) {
-				if (in_array($memberGroup, $visibility)) {
-					return true;
-				}
-			}
-		}
-		
-		return false;
-	}
+    public function canDisplayMemberCheck() {
+        $rawVisibility = $this->MemberVisibility;
+		
+        if (empty($rawVisibility)) {
+            return true;
+        }
+		
+        $visibility = explode(',', $rawVisibility);
+        $member = Member::currentUser();
+		
+        if (!$member && in_array('anonymous', $visibility)) {
+            return true;
+        }
+		
+        if ($member) {
+            $memberGroups = $member->Groups()->column('Code');
+            foreach ($memberGroups as $memberGroup) {
+                if (in_array($memberGroup, $visibility)) {
+                    return true;
+                }
+            }
+        }
+		
+        return false;
+    }
 	
-	public function frontendEditable() {
-		$member = Member::currentUser();		
-		return $this->canEdit($member);
-	}
+    public function frontendEditable() {
+        $member = Member::currentUser();		
+        return $this->canEdit($member);
+    }
 	
-	public function Type2Class() {
-		return strtolower(str_replace(' ', '-', $this->singular_name()));
-	}
+    public function Type2Class() {
+        return strtolower(str_replace(' ', '-', $this->singular_name()));
+    }
 }
\ No newline at end of file

Spacing +13 added lines, -13 removed lines

@@ -1,7 +1,7 @@ 
 <?php
 
 class Block extends DataObject {
-	protected static $db = array (
+	protected static $db = array(
 		'SortOrder'			=>	'Int',
 		'Title'				=>	'Varchar(64)',
 		'TitleWrapper'		=>	'Enum("h2,h3,h4,h5,h6")',
@@ -14,7 +14,7 @@ 
 	
 	protected static $use_own_template = false;
 	
-	protected static $many_many = array (
+	protected static $many_many = array(
 		'Pages'				=>	'Page'
 	);
 		
@@ -22,7 +22,7 @@ 
 		'MySQLDatabase'		=> 'ENGINE=MyISAM'
     );
 	
-	protected static $extensions = array (
+	protected static $extensions = array(
 		'StandardPermissions'
 	);
 	
@@ -42,12 +42,12 @@ 
 	
 	public function VisibleTo() {
 		if (strlen(trim($this->MemberVisibility)) > 0) {
-			$lists = 'Group: ' . str_replace(',','<br />Group: ', $this->MemberVisibility);
-		}else{
+			$lists = 'Group: '.str_replace(',', '<br />Group: ', $this->MemberVisibility);
+		} else {
 			$lists = '<em>&lt;All&gt;</em>';
 		}
 		
-		return new LiteralField('VisibleTo',$lists);
+		return new LiteralField('VisibleTo', $lists);
 	}
 	
 	public function BlockType() {
@@ -57,18 +57,18 @@ 
 	public function shownOn() {
 		if ($this->showBlockbyClass) {
 			if (strlen(trim($this->shownInClass)) > 0) {
-				$lists = 'Type: ' . str_replace(',','<br />Type: ', $this->shownInClass);
-			}else{
+				$lists = 'Type: '.str_replace(',', '<br />Type: ', $this->shownInClass);
+			} else {
 				$lists = '<em>&lt;not assigned&gt;</em>';
 			}
-		}else{
+		} else {
 			if ($this->Pages()->count() > 0) {
-				$lists = 'Page: ' . implode('<br />Page: ', $this->Pages()->column('Title'));
-			}else{
+				$lists = 'Page: '.implode('<br />Page: ', $this->Pages()->column('Title'));
+			} else {
 				$lists = '<em>&lt;not assigned&gt;</em>';
 			}
 		}
-		return new LiteralField('shownOn',$lists);
+		return new LiteralField('shownOn', $lists);
 	}
 	
 	public function getCMSFields() {
@@ -122,7 +122,7 @@ 
 		$pageSelector = new CheckboxSetField(
 			$name = "Pages",
 			$title = "Show on Page(s)",
-			$availablePages->map('ID','Title')
+			$availablePages->map('ID', 'Title')
 		);
 		
 		

Braces +4 added lines, -4 removed lines

@@ -41,7 +41,7 @@ 
 	public function VisibleTo() {
 		if (strlen(trim($this->MemberVisibility)) > 0) {
 			$lists = 'Group: ' . str_replace(',','<br />Group: ', $this->MemberVisibility);
-		}else{
+		} else{
 			$lists = '<em>&lt;All&gt;</em>';
 		}
 		
@@ -56,13 +56,13 @@ 
 		if ($this->showBlockbyClass) {
 			if (strlen(trim($this->shownInClass)) > 0) {
 				$lists = 'Type: ' . str_replace(',','<br />Type: ', $this->shownInClass);
-			}else{
+			} else{
 				$lists = '<em>&lt;not assigned&gt;</em>';
 			}
-		}else{
+		} else{
 			if ($this->Pages()->count() > 0) {
 				$lists = 'Page: ' . implode('<br />Page: ', $this->Pages()->column('Title'));
-			}else{
+			} else{
 				$lists = '<em>&lt;not assigned&gt;</em>';
 			}
 		}

code/SHBlocksAdmin.php

Indentation +30 added lines, -30 removed lines

@@ -4,38 +4,38 @@
  *
  * Left-hand-side tab : Admin Blocks
  * */
-	class SHBlocksAdmin extends ModelAdmin {
-		private static $managed_models = array('SHBlock');
-		private static $url_segment = 'sh-blocks';
-		private static $menu_title = 'SH Blocks';
-		private static $menu_priority = 10;
-		private static $menu_icon = 'silverstripe-block/images/icon-block.png';
+    class SHBlocksAdmin extends ModelAdmin {
+        private static $managed_models = array('SHBlock');
+        private static $url_segment = 'sh-blocks';
+        private static $menu_title = 'SH Blocks';
+        private static $menu_priority = 10;
+        private static $menu_icon = 'silverstripe-block/images/icon-block.png';
 			
-		public function getEditForm($id = null, $fields = null) {
-			$form = parent::getEditForm($id, $fields);
+        public function getEditForm($id = null, $fields = null) {
+            $form = parent::getEditForm($id, $fields);
 			
-			$grid = $form->Fields()->fieldByName($this->sanitiseClassName($this->modelClass));
+            $grid = $form->Fields()->fieldByName($this->sanitiseClassName($this->modelClass));
 			
-			$grid->getConfig()
-				->removeComponentsByType('GridFieldPaginator')
-				->removeComponentsByType('GridFieldAddNewButton')
-				->removeComponentsByType('GridFieldPrintButton')
-				->removeComponentsByType('GridFieldExportButton')
-				->addComponents(
-					new GridFieldPaginatorWithShowAll(30),
-					$multiClass = new MultiClassSelector(),
-					$sortable = new GridFieldOrderableRows('SortOrder')
-				);
+            $grid->getConfig()
+                ->removeComponentsByType('GridFieldPaginator')
+                ->removeComponentsByType('GridFieldAddNewButton')
+                ->removeComponentsByType('GridFieldPrintButton')
+                ->removeComponentsByType('GridFieldExportButton')
+                ->addComponents(
+                    new GridFieldPaginatorWithShowAll(30),
+                    $multiClass = new MultiClassSelector(),
+                    $sortable = new GridFieldOrderableRows('SortOrder')
+                );
 			
-			$subBlocks = ClassInfo::subclassesFor('SHBlock');
-			if (is_null($subBlocks)) {
-				$subBlocks = array('SHBlock');
-			}else{
-				unset($subBlocks['SHBlock']);
-			}
+            $subBlocks = ClassInfo::subclassesFor('SHBlock');
+            if (is_null($subBlocks)) {
+                $subBlocks = array('SHBlock');
+            }else{
+                unset($subBlocks['SHBlock']);
+            }
 			
-			$multiClass->setClasses($subBlocks);
-			$grid->setTitle('All Blcoks');
-			return $form;
-		}
-	}
\ No newline at end of file
+            $multiClass->setClasses($subBlocks);
+            $grid->setTitle('All Blcoks');
+            return $form;
+        }
+    }
\ No newline at end of file

code/Extensions/PrintSHBlocks.php

Indentation +20 added lines, -20 removed lines

@@ -1,27 +1,27 @@
 <?php
 class PrintSHBlocks extends Extension {
 	
-	public function getAllBlocks() {
-		$blocks = new ArrayList();
-		$blocks->merge($this->getMyBlocks());
-		$blocks->merge($this->getDockedBlocks());
-		return $blocks;
-	}
+    public function getAllBlocks() {
+        $blocks = new ArrayList();
+        $blocks->merge($this->getMyBlocks());
+        $blocks->merge($this->getDockedBlocks());
+        return $blocks;
+    }
 		
-	public function getMyBlocks() {
-		return $this->owner->SHBlocks()->sort(array('SortOrder' => 'ASC', 'ID' => 'DESC'));
-	}
+    public function getMyBlocks() {
+        return $this->owner->SHBlocks()->sort(array('SortOrder' => 'ASC', 'ID' => 'DESC'));
+    }
 	
-	public function getDockedBlocks() {
-		$blocks = SHBlock::get()->filter('showBlockbyClass', true);
-		$blocks_map = $blocks->map('ID', 'shownInClass');
-		foreach ($blocks_map as $blockID => $Classes) {
-			$Classes = explode(',', $Classes);
-			if (!in_array($this->owner->ClassName, $Classes)) {
-				$blocks = $blocks->exclude('ID', $blockID);
-			}
-		}
+    public function getDockedBlocks() {
+        $blocks = SHBlock::get()->filter('showBlockbyClass', true);
+        $blocks_map = $blocks->map('ID', 'shownInClass');
+        foreach ($blocks_map as $blockID => $Classes) {
+            $Classes = explode(',', $Classes);
+            if (!in_array($this->owner->ClassName, $Classes)) {
+                $blocks = $blocks->exclude('ID', $blockID);
+            }
+        }
 		
-		return $blocks->sort('SortOrder', 'ASC');
-	}
+        return $blocks->sort('SortOrder', 'ASC');
+    }
 }
\ No newline at end of file

code/Extensions/SHBlockinPage.php

Indentation +52 added lines, -52 removed lines

@@ -1,65 +1,65 @@
 <?php
 class SHBlockinPage extends Extension {
-	protected static $belongs_many_many = array (
-		'SHBlocks'			=>	'SHBlock'
-	);
+    protected static $belongs_many_many = array (
+        'SHBlocks'			=>	'SHBlock'
+    );
 	
-	public function updateCMSFields( FieldList $fields ) {
-		$ancestry = ClassInfo::ancestry($this->owner->ClassName);
-		if (!in_array('RedirectorPage', $ancestry) && !in_array('VirtualPage', $ancestry)) {
-			$blocks = $this->owner->SHBlocks();
-			$blocks_grid = $this->gridBuilder('SHBlocks', $blocks, '', true,'GridFieldConfig_RelationEditor');
-			$docked_grid = $this->gridBuilder('DockedBlocks', $this->dockedBlocks(), '');
+    public function updateCMSFields( FieldList $fields ) {
+        $ancestry = ClassInfo::ancestry($this->owner->ClassName);
+        if (!in_array('RedirectorPage', $ancestry) && !in_array('VirtualPage', $ancestry)) {
+            $blocks = $this->owner->SHBlocks();
+            $blocks_grid = $this->gridBuilder('SHBlocks', $blocks, '', true,'GridFieldConfig_RelationEditor');
+            $docked_grid = $this->gridBuilder('DockedBlocks', $this->dockedBlocks(), '');
 			
-			$fields->addFieldToTab('Root.MyBlocks', $blocks_grid);
-			$fields->addFieldToTab('Root.DockedBlocks', $docked_grid);
-		}
-	}
+            $fields->addFieldToTab('Root.MyBlocks', $blocks_grid);
+            $fields->addFieldToTab('Root.DockedBlocks', $docked_grid);
+        }
+    }
 	
-	private function gridBuilder($name, $source, $label = '', $canAdd = false, $gridHeaderType = 'GridFieldConfig_RecordEditor') {
-		/*
+    private function gridBuilder($name, $source, $label = '', $canAdd = false, $gridHeaderType = 'GridFieldConfig_RecordEditor') {
+        /*
 		GridFieldConfig_Base
 		GridFieldConfig_RecordViewer
 		GridFieldConfig_RecordEditor
 		GridFieldConfig_RelationEditor
 		*/
-		if ($label == '') { $label = $name; }
-		$grid = new GridField($name, $label, $source);
-		$config = $gridHeaderType::create();
-		$config->removeComponentsByType('GridFieldAddNewButton');
-		if ( $canAdd ) {
-			$config->addComponents(
-				$multiClass = new MultiClassSelector(),
-				$sortable = new GridFieldOrderableRows('SortOrder')
-			);
-			$subBlocks = ClassInfo::subclassesFor('SHBlock');
-			if (is_null($subBlocks)) {
-				$subBlocks = array('SHBlock');
-			}else{
-				unset($subBlocks['SHBlock']);
+        if ($label == '') { $label = $name; }
+        $grid = new GridField($name, $label, $source);
+        $config = $gridHeaderType::create();
+        $config->removeComponentsByType('GridFieldAddNewButton');
+        if ( $canAdd ) {
+            $config->addComponents(
+                $multiClass = new MultiClassSelector(),
+                $sortable = new GridFieldOrderableRows('SortOrder')
+            );
+            $subBlocks = ClassInfo::subclassesFor('SHBlock');
+            if (is_null($subBlocks)) {
+                $subBlocks = array('SHBlock');
+            }else{
+                unset($subBlocks['SHBlock']);
 				
-				foreach ($subBlocks as $key => &$value) {
-					$value = empty($key::$singular_name) ? ucwords(trim(strtolower(preg_replace('/_?([A-Z])/', ' $1', $value)))) : $key::$singular_name;
-				}
-			}
-			$multiClass->setClasses($subBlocks);
-		}
-		$grid->setConfig($config);
-		return $grid;
-	}
+                foreach ($subBlocks as $key => &$value) {
+                    $value = empty($key::$singular_name) ? ucwords(trim(strtolower(preg_replace('/_?([A-Z])/', ' $1', $value)))) : $key::$singular_name;
+                }
+            }
+            $multiClass->setClasses($subBlocks);
+        }
+        $grid->setConfig($config);
+        return $grid;
+    }
 		
-	private function dockedBlocks() {
-		$blocks = SHBlock::get();
-		$IDs = array();
-		$ClassName = $this->owner->ClassName;
-		$Classes = $blocks->map('ID', 'shownInClass');
-		foreach ($Classes as $BlockID => $Class) {
-			$listedClasses = explode(',', $Class);
-			if (in_array($ClassName, $listedClasses)) {
-				$IDs[] = $BlockID;
-			}
-		}
-		$blocks = SHBlock::get()->filter('ID', $IDs)->sort('SortOrder','ASC');
-		return $blocks;
-	}
+    private function dockedBlocks() {
+        $blocks = SHBlock::get();
+        $IDs = array();
+        $ClassName = $this->owner->ClassName;
+        $Classes = $blocks->map('ID', 'shownInClass');
+        foreach ($Classes as $BlockID => $Class) {
+            $listedClasses = explode(',', $Class);
+            if (in_array($ClassName, $listedClasses)) {
+                $IDs[] = $BlockID;
+            }
+        }
+        $blocks = SHBlock::get()->filter('ID', $IDs)->sort('SortOrder','ASC');
+        return $blocks;
+    }
 }
\ No newline at end of file

Spacing +6 added lines, -6 removed lines

@@ -1,14 +1,14 @@ 
 <?php
 class SHBlockinPage extends Extension {
-	protected static $belongs_many_many = array (
+	protected static $belongs_many_many = array(
 		'SHBlocks'			=>	'SHBlock'
 	);
 	
-	public function updateCMSFields( FieldList $fields ) {
+	public function updateCMSFields(FieldList $fields) {
 		$ancestry = ClassInfo::ancestry($this->owner->ClassName);
 		if (!in_array('RedirectorPage', $ancestry) && !in_array('VirtualPage', $ancestry)) {
 			$blocks = $this->owner->SHBlocks();
-			$blocks_grid = $this->gridBuilder('SHBlocks', $blocks, '', true,'GridFieldConfig_RelationEditor');
+			$blocks_grid = $this->gridBuilder('SHBlocks', $blocks, '', true, 'GridFieldConfig_RelationEditor');
 			$docked_grid = $this->gridBuilder('DockedBlocks', $this->dockedBlocks(), '');
 			
 			$fields->addFieldToTab('Root.MyBlocks', $blocks_grid);
@@ -27,7 +27,7 @@ 
 		$grid = new GridField($name, $label, $source);
 		$config = $gridHeaderType::create();
 		$config->removeComponentsByType('GridFieldAddNewButton');
-		if ( $canAdd ) {
+		if ($canAdd) {
 			$config->addComponents(
 				$multiClass = new MultiClassSelector(),
 				$sortable = new GridFieldOrderableRows('SortOrder')
@@ -35,7 +35,7 @@ 
 			$subBlocks = ClassInfo::subclassesFor('SHBlock');
 			if (is_null($subBlocks)) {
 				$subBlocks = array('SHBlock');
-			}else{
+			} else {
 				unset($subBlocks['SHBlock']);
 				
 				foreach ($subBlocks as $key => &$value) {
@@ -59,7 +59,7 @@ 
 				$IDs[] = $BlockID;
 			}
 		}
-		$blocks = SHBlock::get()->filter('ID', $IDs)->sort('SortOrder','ASC');
+		$blocks = SHBlock::get()->filter('ID', $IDs)->sort('SortOrder', 'ASC');
 		return $blocks;
 	}
 }
\ No newline at end of file