From 06a007f394da74b605e8702b300faed7168f8ef8 Mon Sep 17 00:00:00 2001
From: Sean Harvey <sean@silverstripe.com>
Date: Tue, 3 Feb 2009 20:34:13 +0000
Subject: [PATCH] MINOR Code formatting tidy up in ModelAdmin

git-svn-id: svn://svn.silverstripe.com/silverstripe/open/modules/cms/trunk@71302 467b73ca-7a2a-4603-9d3b-597d59a354a9
---
 code/ModelAdmin.php | 29 +++++++++++++++++------------
 1 file changed, 17 insertions(+), 12 deletions(-)

diff --git a/code/ModelAdmin.php b/code/ModelAdmin.php
index 39cbcc29..85be6540 100644
--- a/code/ModelAdmin.php
+++ b/code/ModelAdmin.php
@@ -210,12 +210,17 @@ abstract class ModelAdmin extends LeftAndMain {
 	 */
 	function getManagedModels() {
 		$models = $this->stat('managed_models');
-		if(is_string($models)) $models = array($models);
-		if(!count($models)) user_error('ModelAdmin::getManagedModels(): 
-			You need to specify at least one DataObject subclass in protected static $managed_models.
-			Make sure the visibility of your property is set to "protected"', 
-			E_USER_ERROR
-		);
+		if(is_string($models)) {
+			$models = array($models);
+		}
+		if(!count($models)) {
+			user_error(
+				'ModelAdmin::getManagedModels(): 
+				You need to specify at least one DataObject subclass in protected static $managed_models.
+				Make sure the visibility of your property is set to "protected"', 
+				E_USER_ERROR
+			);
+		}
 		
 		return $models;
 	}
@@ -342,11 +347,10 @@ class ModelAdmin_CollectionController extends Controller {
 	public function CreateForm() {
 		$modelName = $this->modelClass;
 
-		if ($this->hasMethod('alternatePermissionCheck')) {
-			if (!$this->alternatePermissionCheck()) return false;
-		}
-		else {
-			if (!singleton($modelName)->canCreate(Member::currentUser())) return false;
+		if($this->hasMethod('alternatePermissionCheck')) {
+			if(!$this->alternatePermissionCheck()) return false;
+		} else {
+			if(!singleton($modelName)->canCreate(Member::currentUser())) return false;
 		}
 		
 		$buttonLabel = sprintf(_t('ModelAdmin.CREATEBUTTON', "Create '%s'", PR_MEDIUM, "Create a new instance from a model class"), singleton($modelName)->i18n_singular_name());
@@ -354,6 +358,7 @@ class ModelAdmin_CollectionController extends Controller {
 		$actions = new FieldSet(
 			$createButton = new FormAction('add', $buttonLabel)
 		);
+		
 		$createButton->dontEscape = true;
 		
 		return new Form($this, "CreateForm", new FieldSet(), $actions);	
@@ -837,7 +842,7 @@ class ModelAdmin_RecordController extends Controller {
 	 * @return mixed
 	 */
 	function view($request) {
-		if ($this->currentRecord) {
+		if($this->currentRecord) {
 			$form = $this->ViewForm();
 			return $form->forAjaxTemplate();
 		} else {