From ba93028b01a4bbf98c2ab81d726c8ba1178c3325 Mon Sep 17 00:00:00 2001
From: Sam Minnee <sam@silverstripe.com>
Date: Thu, 8 Mar 2012 09:54:31 +1300
Subject: [PATCH 1/2] API CHANGE: Added Form::VisibleFields() and
 FieldList::VisibleFields(), which list everything except hidden fields, to
 assist with the creation of custom form layouts.

---
 forms/FieldList.php           | 14 ++++++++++++++
 forms/Form.php                |  8 ++++++++
 tests/forms/FieldListTest.php | 35 +++++++++++++++++++++++++++++++++++
 3 files changed, 57 insertions(+)

diff --git a/forms/FieldList.php b/forms/FieldList.php
index 8cba6bd2003..fd8fcca905e 100644
--- a/forms/FieldList.php
+++ b/forms/FieldList.php
@@ -434,6 +434,20 @@ function HiddenFields() {
 		
 		return $hiddenFields;
 	}
+	
+	/**
+	 * Return all fields except for the hidden fields.
+	 * Useful when making your own simplified form layouts.
+	 */
+	function VisibleFields() {
+		$visibleFields = new FieldList();
+		
+		foreach($this as $field) {
+			if(!($field instanceof HiddenField)) $visibleFields->push($field);
+		}
+		
+		return $visibleFields;
+	}
 
 	/**
 	 * Transform this FieldList with a given tranform method,
diff --git a/forms/Form.php b/forms/Form.php
index df1ee2d7f87..e44ec9e2e2e 100644
--- a/forms/Form.php
+++ b/forms/Form.php
@@ -543,6 +543,14 @@ function Fields() {
 	function HiddenFields() {
 		return $this->fields->HiddenFields();
 	}
+
+	/**
+	 * Return all fields except for the hidden fields.
+	 * Useful when making your own simplified form layouts.
+	 */
+	function VisibleFields() {
+		return $this->fields->VisibleFields();
+	}
 	
 	/**
 	 * Setter for the form fields.
diff --git a/tests/forms/FieldListTest.php b/tests/forms/FieldListTest.php
index 497e6f48afc..ef001228a5b 100644
--- a/tests/forms/FieldListTest.php
+++ b/tests/forms/FieldListTest.php
@@ -729,4 +729,39 @@ function testMakeFieldReadonly() {
 			'Field nested inside a TabSet and FieldList can be marked readonly by FieldList->makeFieldReadonly()'
 		);
 	}
+
+	/**
+	 * Test VisibleFields and HiddenFields
+	 */
+	function testVisibleAndHiddenFields() {
+		$fields = new FieldList(
+			new TextField("A"),
+			new TextField("B"),
+			new HiddenField("C"),
+			new Tabset("Root",
+				new Tab("D", 
+					new TextField("D1"),
+					new HiddenField("D2")
+				)
+			)
+		);
+
+		$hidden = $fields->HiddenFields();
+		// Inside hidden fields, all HiddenField objects are included, even nested ones
+		$this->assertNotNull($hidden->dataFieldByName('C'));
+		$this->assertNotNull($hidden->dataFieldByName('D2'));
+		// Visible fields are not
+		$this->assertNull($hidden->dataFieldByName('B'));
+		$this->assertNull($hidden->dataFieldByName('D1'));
+		
+		$visible = $fields->VisibleFields();
+		// Visible fields exclude top level HiddenField objects
+		$this->assertNotNull($visible->dataFieldByName('A'));
+		$this->assertNull($visible->dataFieldByName('C'));
+		// But they don't exclude nested HiddenField objects.  This is a limitation; you should
+		// put all your HiddenFields at the top level.
+		$this->assertNotNull($visible->dataFieldByName('D2'));
+	}
+	
+	
 }

From 3d54668896dab25dccfc54dfbd0b56030a2be9dc Mon Sep 17 00:00:00 2001
From: Sam Minnee <sam@silverstripe.com>
Date: Fri, 9 Mar 2012 15:41:01 +1300
Subject: [PATCH 2/2] MINOR: Added explicit 'public' keyword on functions.

---
 forms/FieldList.php |  22 +++---
 forms/Form.php      | 158 ++++++++++++++++++++++----------------------
 2 files changed, 90 insertions(+), 90 deletions(-)

diff --git a/forms/FieldList.php b/forms/FieldList.php
index fd8fcca905e..cafd4b2f673 100644
--- a/forms/FieldList.php
+++ b/forms/FieldList.php
@@ -219,7 +219,7 @@ public function replaceField($fieldName, $newField) {
 	 * @param string $newFieldTitle New title of field
 	 * @return boolean
 	 */
-	function renameField($fieldName, $newFieldTitle) {
+	public function renameField($fieldName, $newFieldTitle) {
 		$field = $this->dataFieldByName($fieldName);
 		if(!$field) return false;
 		
@@ -424,7 +424,7 @@ public function setValues($data) {
 	 * 
 	 * @return FieldList
 	 */
-	function HiddenFields() {
+	public function HiddenFields() {
 		$hiddenFields = new FieldList();
 		$dataFields = $this->dataFields();
 		
@@ -439,7 +439,7 @@ function HiddenFields() {
 	 * Return all fields except for the hidden fields.
 	 * Useful when making your own simplified form layouts.
 	 */
-	function VisibleFields() {
+	public function VisibleFields() {
 		$visibleFields = new FieldList();
 		
 		foreach($this as $field) {
@@ -455,7 +455,7 @@ function VisibleFields() {
 	 * 
 	 * @return FieldList
 	 */
-	function transform($trans) {
+	public function transform($trans) {
 		$this->flushFieldsCache();
 		$newFields = new FieldList();
 		foreach($this as $field) {
@@ -467,12 +467,12 @@ function transform($trans) {
 	/**
 	 * Returns the root field set that this belongs to
 	 */
-	function rootFieldSet() {
+	public function rootFieldSet() {
 		if($this->containerField) return $this->containerField->rootFieldSet();
 		else return $this;
 	}
 	
-	function setContainerField($field) {
+	public function setContainerField($field) {
 		$this->containerField = $field;
 		return $this;
 	}
@@ -482,7 +482,7 @@ function setContainerField($field) {
 	 *
 	 * @return FieldList
 	 */
-	function makeReadonly() {
+	public function makeReadonly() {
 		return $this->transform(new ReadonlyTransformation());
 	}
 
@@ -491,7 +491,7 @@ function makeReadonly() {
 	 * 
 	 * @param string|FormField
 	 */
-	function makeFieldReadonly($field) {
+	public function makeFieldReadonly($field) {
 		$fieldName = ($field instanceof FormField) ? $field->getName() : $field;
 		$srcField = $this->dataFieldByName($fieldName);
 		$this->replaceField($fieldName, $srcField->performReadonlyTransformation());
@@ -506,7 +506,7 @@ function makeFieldReadonly($field) {
 	 *
 	 * @param array $fieldNames Field names can be given as an array, or just as a list of arguments.
 	 */
-	function changeFieldOrder($fieldNames) {
+	public function changeFieldOrder($fieldNames) {
 		// Field names can be given as an array, or just as a list of arguments.
 		if(!is_array($fieldNames)) $fieldNames = func_get_args();
 		
@@ -541,7 +541,7 @@ function changeFieldOrder($fieldNames) {
 	 * @param string|FormField
 	 * @return Position in children collection (first position starts with 0). Returns FALSE if the field can't be found.
 	 */
-	function fieldPosition($field) {
+	public function fieldPosition($field) {
 		if(is_object($field)) $field = $field->getName();
 		
 		$i = 0;
@@ -563,7 +563,7 @@ function fieldPosition($field) {
  * @subpackage fields-structural
  */
 class HiddenFieldList extends FieldList {
-	function forTemplate() {
+	public function forTemplate() {
 		$output = "";
 		foreach($this as $field) {
 			$output .= $field->Field();
diff --git a/forms/Form.php b/forms/Form.php
index e44ec9e2e2e..2ab13b8635d 100644
--- a/forms/Form.php
+++ b/forms/Form.php
@@ -152,7 +152,7 @@ class Form extends RequestHandler {
 	 * @param FieldList $actions All of the action buttons in the form - a {@link FieldLis} of {@link FormAction} objects
 	 * @param Validator $validator Override the default validator instance (Default: {@link RequiredFields})
 	 */
-	function __construct($controller, $name, FieldList $fields, FieldList $actions, $validator = null) {
+	public function __construct($controller, $name, FieldList $fields, FieldList $actions, $validator = null) {
 		parent::__construct();
 		
 		if(!$fields instanceof FieldList) throw new InvalidArgumentException('$fields must be a valid FieldList instance');
@@ -198,7 +198,7 @@ function __construct($controller, $name, FieldList $fields, FieldList $actions,
 	 * Set up current form errors in session to
 	 * the current form if appropriate.
 	 */
-	function setupFormErrors() {
+	public function setupFormErrors() {
 		$errorInfo = Session::get("FormInfo.{$this->FormName()}");
 
 		if(isset($errorInfo['errors']) && is_array($errorInfo['errors'])) {
@@ -228,7 +228,7 @@ function setupFormErrors() {
 	 * and only triggers the requested form action/method
 	 * if the form is valid.
 	 */
-	function httpSubmission($request) {
+	public function httpSubmission($request) {
 		$vars = $request->requestVars();
 		if(isset($funcName)) {
 			Form::set_current_action($funcName);
@@ -384,7 +384,7 @@ protected function checkFieldsForAction($fields, $funcName) {
 	 * @param SS_HTTPRequest $request
 	 * @return FormField
 	 */
-	function handleField($request) {
+	public function handleField($request) {
 		$field = $this->Fields()->dataFieldByName($request->param('FieldName'));
 		
 		if($field) {
@@ -398,7 +398,7 @@ function handleField($request) {
 	/**
 	 * Convert this form into a readonly form
 	 */
-	function makeReadonly() {
+	public function makeReadonly() {
 		$this->transform(new ReadonlyTransformation());
 	}
 	
@@ -428,7 +428,7 @@ public function getRedirectToFormOnValidationError() {
 	 * Add an error message to a field on this form.  It will be saved into the session
 	 * and used the next time this form is displayed.
 	 */
-	function addErrorMessage($fieldName, $message, $messageType) {
+	public function addErrorMessage($fieldName, $message, $messageType) {
 		Session::add_to_array("FormInfo.{$this->FormName()}.errors",  array(
 			'fieldName' => $fieldName,
 			'message' => $message,
@@ -436,7 +436,7 @@ function addErrorMessage($fieldName, $message, $messageType) {
 		));
 	}
 
-	function transform(FormTransformation $trans) {
+	public function transform(FormTransformation $trans) {
 		$newFields = new FieldList();
 		foreach($this->fields as $field) {
 			$newFields->push($field->transform($trans));
@@ -459,14 +459,14 @@ function transform(FormTransformation $trans) {
 	 * Get the {@link Validator} attached to this form.
 	 * @return Validator
 	 */
-	function getValidator() {
+	public function getValidator() {
 		return $this->validator;
 	}
 
 	/**
 	 * Set the {@link Validator} on this form.
 	 */
-	function setValidator( Validator $validator ) {
+	public function setValidator( Validator $validator ) {
 		if($validator) {
 			$this->validator = $validator;
 			$this->validator->setForm($this);
@@ -477,14 +477,14 @@ function setValidator( Validator $validator ) {
 	/**
 	 * Remove the {@link Validator} from this from.
 	 */
-	function unsetValidator(){
+	public function unsetValidator(){
 		$this->validator = null;
 	}
 
 	/**
 	 * Convert this form to another format.
 	 */
-	function transformTo(FormTransformation $format) {
+	public function transformTo(FormTransformation $format) {
 		$newFields = new FieldList();
 		foreach($this->fields as $field) {
 			$newFields->push($field->transformTo($format));
@@ -525,7 +525,7 @@ public function getExtraFields() {
 	 * 
 	 * @return FieldList The form fields
 	 */
-	function Fields() {
+	public function Fields() {
 		foreach($this->getExtraFields() as $field) {
 			if(!$this->fields->fieldByName($field->getName())) $this->fields->push($field);
 		}
@@ -540,7 +540,7 @@ function Fields() {
 	 * 
 	 * @return FieldList
 	 */
-	function HiddenFields() {
+	public function HiddenFields() {
 		return $this->fields->HiddenFields();
 	}
 
@@ -548,7 +548,7 @@ function HiddenFields() {
 	 * Return all fields except for the hidden fields.
 	 * Useful when making your own simplified form layouts.
 	 */
-	function VisibleFields() {
+	public function VisibleFields() {
 		return $this->fields->VisibleFields();
 	}
 	
@@ -557,7 +557,7 @@ function VisibleFields() {
 	 *
 	 * @param FieldList $fields
 	 */
-	function setFields($fields) {
+	public function setFields($fields) {
 		$this->fields = $fields;
 		return $this;
 	}
@@ -570,7 +570,7 @@ function setFields($fields) {
 	 * @deprecated 3.0 Use Fields() and FieldList API instead
 	 * @return FormField
 	 */
-	function dataFieldByName($name) {
+	public function dataFieldByName($name) {
 		Deprecation::notice('3.0', 'Use Fields() and FieldList API instead');
 
 		foreach($this->getExtraFields() as $field) {
@@ -585,7 +585,7 @@ function dataFieldByName($name) {
 	 * 
 	 * @return FieldList The action list
 	 */
-	function Actions() {
+	public function Actions() {
 		return $this->actions;
 	}
 
@@ -594,7 +594,7 @@ function Actions() {
 	 *
 	 * @param FieldList $actions
 	 */
-	function setActions($actions) {
+	public function setActions($actions) {
 		$this->actions = $actions;
 		return $this;
 	}
@@ -602,7 +602,7 @@ function setActions($actions) {
 	/**
 	 * Unset all form actions
 	 */
-	function unsetAllActions(){
+	public function unsetAllActions(){
 		$this->actions = new FieldList();
 		return $this;
 	}
@@ -613,7 +613,7 @@ function unsetAllActions(){
 	 * @deprecated 3.0 Use Actions() and FieldList API instead
 	 * @param string $name
 	 */
-	function unsetActionByName($name) {
+	public function unsetActionByName($name) {
 		Deprecation::notice('3.0', 'Use Actions() and FieldList API instead');
 
 		$this->actions->removeByName($name);
@@ -623,7 +623,7 @@ function unsetActionByName($name) {
 	 * @param String
 	 * @param String
 	 */
-	function setAttribute($name, $value) {
+	public function setAttribute($name, $value) {
 		$this->attributes[$name] = $value;
 		return $this;
 	}
@@ -631,11 +631,11 @@ function setAttribute($name, $value) {
 	/**
 	 * @return String
 	 */
-	function getAttribute($name) {
+	public function getAttribute($name) {
 		return @$this->attributes[$name];
 	}
 
-	function getAttributes() {
+	public function getAttributes() {
 		$attrs = array(
 			'id' => $this->FormName(),
 			'action' => $this->FormAction(),
@@ -659,7 +659,7 @@ function getAttributes() {
 	 *
 	 * @deprecated 3.0 Use Fields() and FieldList API instead
 	 */
-	function unsetDataFieldByName($fieldName){
+	public function unsetDataFieldByName($fieldName){
 		Deprecation::notice('3.0', 'Use Fields() and FieldList API instead');
 
 		foreach($this->Fields()->dataFields() as $child) {
@@ -685,7 +685,7 @@ public function unsetFieldFromTab($tabName, $fieldName) {
 	 * If at least one argument is passed as a string, all arguments act as excludes by name.
 	 * @return String HTML attributes, ready for insertion into an HTML tag
 	 */
-	function getAttributesHTML($attrs = null) {
+	public function getAttributesHTML($attrs = null) {
 		$exclude = (is_string($attrs)) ? func_get_args() : null;
 
 		if(!$attrs || is_string($attrs)) $attrs = $this->getAttributes();
@@ -710,7 +710,7 @@ function getAttributesHTML($attrs = null) {
 		return implode(' ', $parts);
 	}
 
-	function FormAttributes() {
+	public function FormAttributes() {
 		return $this->getAttributesHTML();
 	}
 
@@ -719,7 +719,7 @@ function FormAttributes() {
 	* 
 	* @param target The value of the target
 	*/
-	function setTarget($target) {
+	public function setTarget($target) {
 		$this->target = $target;
 		return $this;
 	}
@@ -728,7 +728,7 @@ function setTarget($target) {
 	 * Set the legend value to be inserted into
 	 * the <legend> element in the Form.ss template.
 	 */
-	function setLegend($legend) {
+	public function setLegend($legend) {
 		$this->legend = $legend;
 		return $this;
 	}
@@ -739,7 +739,7 @@ function setLegend($legend) {
 	 * 
 	 * @param string $template The name of the template (without the .ss extension)
 	 */
-	function setTemplate($template) {
+	public function setTemplate($template) {
 		$this->template = $template;
 		return $this;
 	}
@@ -751,7 +751,7 @@ function setTemplate($template) {
 	 * 
 	 * @return string
 	 */
-	function getTemplate() {
+	public function getTemplate() {
 		if($this->template) return $this->template;
 		else return $this->class;
 	}
@@ -808,7 +808,7 @@ public function FormEncType() {
 	 * 
 	 * @return string HTTP method
 	 */
-	function FormHttpMethod() {
+	public function FormHttpMethod() {
 		return $this->formMethod;
 	}
 	
@@ -818,7 +818,7 @@ function FormHttpMethod() {
 	 * 
 	 * @return string Form tag compatbile HTTP method: 'get' or 'post'
 	 */
-	function FormMethod() {
+	public function FormMethod() {
 		if(in_array($this->formMethod,array('get','post'))) {
 			return $this->formMethod;
 		} else {
@@ -831,7 +831,7 @@ function FormMethod() {
 	 * 
 	 * @param $method string
 	 */
-	function setFormMethod($method) {
+	public function setFormMethod($method) {
 		$this->formMethod = strtolower($method);
 		return $this;
 	}
@@ -842,7 +842,7 @@ function setFormMethod($method) {
 	 * 
 	 * @return string 
 	 */
-	function FormAction() {
+	public function FormAction() {
 		if ($this->formActionPath) {
 			return $this->formActionPath;
 		} elseif($this->controller->hasMethod("FormObjectLink")) {
@@ -861,7 +861,7 @@ function FormAction() {
 	 * Note: For "normal" forms, you shouldn't need to use this method.  It is recommended only for situations where you have
 	 * two relatively distinct parts of the system trying to communicate via a form post.
 	 */
-	function setFormAction($path) {
+	public function setFormAction($path) {
 		$this->formActionPath = $path;
 		return $this;
 	}
@@ -874,7 +874,7 @@ function setFormAction($path) {
 	/**
 	 * Returns the name of the form
 	 */
-	function FormName() {
+	public function FormName() {
 		if($this->htmlID) return $this->htmlID;
 		else return $this->class . '_' . str_replace(array('.','/'),'',$this->name);
 	}
@@ -882,21 +882,21 @@ function FormName() {
 	/**
 	 * Set the HTML ID attribute of the form
 	 */
-	function setHTMLID($id) {
+	public function setHTMLID($id) {
 		$this->htmlID = $id;
 	}
 	
 	/**
 	 * Returns this form's controller
 	 */
-	function Controller() {
+	public function Controller() {
 		return $this->controller;
 	}
 	
 	/**
 	 * @return string
 	 */
-	function Name() {
+	public function Name() {
 		return $this->name;
 	}
 	
@@ -905,7 +905,7 @@ function Name() {
 	 * That method will return the field itself.
 	 * It means that you can execute $firstNameField = $form->FieldMap()->FirstName(), which can be handy
 	 */
-	function FieldMap() {
+	public function FieldMap() {
 		return new Form_FieldMap($this);
 	}
 
@@ -916,7 +916,7 @@ function FieldMap() {
 	 * 
 	 * @return string
 	 */
-	function Message() {
+	public function Message() {
 		$this->getMessageFromSession();
 		$message = $this->message;
 		$this->clearMessage();
@@ -926,7 +926,7 @@ function Message() {
 	/**
 	 * @return string
 	 */
-	function MessageType() {
+	public function MessageType() {
 		$this->getMessageFromSession();
 		return $this->messageType;
 	}
@@ -948,7 +948,7 @@ protected function getMessageFromSession() {
 	 * @param message the text of the message
 	 * @param type Should be set to good, bad, or warning.
 	 */
-	function setMessage($message, $type) {
+	public function setMessage($message, $type) {
 		$this->message = $message;
 		$this->messageType = $type;
 		return $this;
@@ -960,22 +960,22 @@ function setMessage($message, $type) {
 	 * @param message the text of the message
 	 * @param type Should be set to good, bad, or warning.
 	 */
-	function sessionMessage($message, $type) {
+	public function sessionMessage($message, $type) {
 		Session::set("FormInfo.{$this->FormName()}.formError.message", $message);
 		Session::set("FormInfo.{$this->FormName()}.formError.type", $type);
 	}
 
-	static function messageForForm( $formName, $message, $type ) {
+	public static function messageForForm( $formName, $message, $type ) {
 		Session::set("FormInfo.{$formName}.formError.message", $message);
 		Session::set("FormInfo.{$formName}.formError.type", $type);
 	}
 
-	function clearMessage() {
+	public function clearMessage() {
 		$this->message  = null;
 		Session::clear("FormInfo.{$this->FormName()}.errors");
 		Session::clear("FormInfo.{$this->FormName()}.formError");
 	}
-	function resetValidation() {
+	public function resetValidation() {
 		Session::clear("FormInfo.{$this->FormName()}.errors");
 	}
 
@@ -985,7 +985,7 @@ function resetValidation() {
 	 * 
 	 * @return DataObject
 	 */
-	function getRecord() {
+	public function getRecord() {
 		return $this->record;
 	}
 	
@@ -995,7 +995,7 @@ function getRecord() {
 	 *
 	 * @return string
 	 */
-	function getLegend() {
+	public function getLegend() {
 		return $this->legend;
 	}
 
@@ -1062,7 +1062,7 @@ function validate(){
 	 * @param $fieldList An optional list of fields to process.  This can be useful when you have a 
 	 * form that has some fields that save to one object, and some that save to another.
 	 */
-	function loadDataFrom($data, $clearMissingFields = false, $fieldList = null) {
+	public function loadDataFrom($data, $clearMissingFields = false, $fieldList = null) {
 		if(!is_object($data) && !is_array($data)) {
 			user_error("Form::loadDataFrom() not passed an array or an object", E_USER_WARNING);
 			return false;
@@ -1125,7 +1125,7 @@ function loadDataFrom($data, $clearMissingFields = false, $fieldList = null) {
 	 * @param $fieldList An optional list of fields to process.  This can be useful when you have a 
 	 * form that has some fields that save to one object, and some that save to another.
 	 */
-	function saveInto(DataObjectInterface $dataObject, $fieldList = null) {
+	public function saveInto(DataObjectInterface $dataObject, $fieldList = null) {
 		$dataFields = $this->fields->saveableFields();
 		$lastField = null;
 		if($dataFields) foreach($dataFields as $field) {
@@ -1156,7 +1156,7 @@ function saveInto(DataObjectInterface $dataObject, $fieldList = null) {
 	 * 
 	 * @return array
 	 */
-	function getData() {
+	public function getData() {
 		$dataFields = $this->fields->dataFields();
 		$data = array();
 		
@@ -1178,7 +1178,7 @@ function getData() {
 	 * @param string $fieldName
 	 * @param mixed $fieldValue
 	 */
-	function resetField($fieldName, $fieldValue = null) {
+	public function resetField($fieldName, $fieldValue = null) {
 		Deprecation::notice('3.0', 'Use Fields() and FieldList API instead');
 
 		$dataFields = $this->fields->dataFields();
@@ -1196,7 +1196,7 @@ function resetField($fieldName, $fieldValue = null) {
 	 * @param fieldName The name of the field.  Can be overridden by $_REQUEST[fieldName]
 	 * @param methodName The name of the field.  Can be overridden by $_REQUEST[methodName]
 	 */
-	function callfieldmethod($data) {
+	public function callfieldmethod($data) {
 		$fieldName = $data['fieldName'];
 		$methodName = $data['methodName'];
 		$fields = $this->fields->dataFields();
@@ -1226,7 +1226,7 @@ function callfieldmethod($data) {
 	 * This is returned when you access a form as $FormObject rather
 	 * than <% control FormObject %>
 	 */
-	function forTemplate() {
+	public function forTemplate() {
 		return $this->renderWith(array_merge(
 			(array)$this->getTemplate(),
 			array('Form')
@@ -1237,7 +1237,7 @@ function forTemplate() {
 	 * Return a rendered version of this form, suitable for ajax post-back.
 	 * It triggers slightly different behaviour, such as disabling the rewriting of # links
 	 */
-	function forAjaxTemplate() {
+	public function forAjaxTemplate() {
 		$view = new SSViewer(array(
 			$this->getTemplate(),
 			'Form'
@@ -1251,7 +1251,7 @@ function forAjaxTemplate() {
 	 * Attaches 3 extra hidden files, _form_action, _form_name, _form_method, and _form_enctype.  These are
 	 * the attributes of the form.  These fields can be used to send the form to Ajax.
 	 */
-	function formHtmlContent() {
+	public function formHtmlContent() {
 		$this->IncludeFormTag = false;
 		$content = $this->forTemplate();
 		$this->IncludeFormTag = true;
@@ -1268,7 +1268,7 @@ function formHtmlContent() {
 	 * Render this form using the given template, and return the result as a string
 	 * You can pass either an SSViewer or a template name
 	 */
-	function renderWithoutActionButton($template) {
+	public function renderWithoutActionButton($template) {
 		$custom = $this->customise(array(
 			"Actions" => "",
 		));
@@ -1282,12 +1282,12 @@ function renderWithoutActionButton($template) {
 	 * Sets the button that was clicked.  This should only be called by the Controller.
 	 * @param funcName The name of the action method that will be called.
 	 */
-	function setButtonClicked($funcName) {
+	public function setButtonClicked($funcName) {
 		$this->buttonClickedFunc = $funcName;
 		return $this;
 	}
 
-	function buttonClicked() {
+	public function buttonClicked() {
 		foreach($this->actions as $action) {
 			if($this->buttonClickedFunc == $action->actionName()) return $action;
 		}
@@ -1296,7 +1296,7 @@ function buttonClicked() {
 	/**
 	 * Return the default button that should be clicked when another one isn't available
 	 */
-	function defaultAction() {
+	public function defaultAction() {
 		if($this->hasDefaultAction && $this->actions)
 			return $this->actions->First();
 	}
@@ -1306,7 +1306,7 @@ function defaultAction() {
 	 * Ordinarily, when a form is processed and no action_XXX button is available, then the first button in the actions list
 	 * will be pressed.  However, if this is "delete", for example, this isn't such a good idea.
 	 */
-	function disableDefaultAction() {
+	public function disableDefaultAction() {
 		$this->hasDefaultAction = false;
 		return $this;
 	}
@@ -1318,7 +1318,7 @@ function disableDefaultAction() {
 	 * 
 	 * Check for token state with {@link getSecurityToken()} and {@link SecurityToken->isEnabled()}.
 	 */
-	function disableSecurityToken() {
+	public function disableSecurityToken() {
 		$this->securityToken = new NullSecurityToken();
 		return $this;
 	}
@@ -1328,7 +1328,7 @@ function disableSecurityToken() {
 	 * 
 	 * Check for token state with {@link getSecurityToken()} and {@link SecurityToken->isEnabled()}.
 	 */
-	function enableSecurityToken() {
+	public function enableSecurityToken() {
 		$this->securityToken = new SecurityToken();
 		return $this;
 	}
@@ -1343,7 +1343,7 @@ function enableSecurityToken() {
 	 * 
 	 * @deprecated 2.5 Use SecurityToken::disable()
 	 */
-	static function disable_all_security_tokens() {
+	public static function disable_all_security_tokens() {
 		Deprecation::notice('2.5', 'Use SecurityToken::disable() instead.');
 		SecurityToken::disable();
 	}
@@ -1356,7 +1356,7 @@ static function disable_all_security_tokens() {
 	 *
 	 * @return bool
 	 */
-	function securityTokenEnabled() {
+	public function securityTokenEnabled() {
 		Deprecation::notice('2.5', 'Use Form->getSecurityToken()->isEnabled() instead.');
 		return $this->securityToken->isEnabled();
 	}
@@ -1368,7 +1368,7 @@ function securityTokenEnabled() {
 	 * 
 	 * @return SecurityToken|null
 	 */
-	function getSecurityToken() {
+	public function getSecurityToken() {
 		return $this->securityToken;
 	}
 		
@@ -1379,7 +1379,7 @@ function getSecurityToken() {
 	 * 
 	 * @return string
 	 */
-	static function single_field_required() {
+	public static function single_field_required() {
 		if(self::current_action() == 'callfieldmethod') return $_REQUEST['fieldName'];
 	}
 
@@ -1387,14 +1387,14 @@ static function single_field_required() {
 	 * Return the current form action being called, if available.
 	 * This is useful for optimising your forms
 	 */
-	static function current_action() {
+	public static function current_action() {
 		return self::$current_action;
 	}
 
 	/**
 	 * Set the current form action.  Should only be called by Controller.
 	 */
-	static function set_current_action($action) {
+	public static function set_current_action($action) {
 		self::$current_action = $action;
 	}
 	
@@ -1403,7 +1403,7 @@ static function set_current_action($action) {
 	 * 
 	 * @return string
 	 */
-	function extraClass() {		
+	public function extraClass() {		
 		return implode(array_unique($this->extraClasses), ' ');
 	}
 	
@@ -1414,7 +1414,7 @@ function extraClass() {
 	 * @param string $class A string containing a classname or several class
 	 *				names delimited by a single space.
 	 */
-	function addExtraClass($class) {
+	public function addExtraClass($class) {
 		$classes = explode(' ', $class);
 		
 		foreach($classes as $class) {
@@ -1432,13 +1432,13 @@ function addExtraClass($class) {
 	 *
 	 * @param string $class
 	 */
-	function removeExtraClass($class) {
+	public function removeExtraClass($class) {
 		$classes = explode(' ', $class);
 		$this->extraClasses = array_diff($this->extraClasses, $classes);
 		return $this;
 	}
 	
-	function debug() {
+	public function debug() {
 		$result = "<h3>$this->class</h3><ul>";
 		foreach($this->fields as $field) {
 			$result .= "<li>$field" . $field->debug() . "</li>";
@@ -1460,7 +1460,7 @@ function debug() {
 	 * Test a submission of this form.
 	 * @return SS_HTTPResponse the response object that the handling controller produces.  You can interrogate this in your unit test.
 	 */
-	function testSubmission($action, $data) {
+	public function testSubmission($action, $data) {
 		$data['action_' . $action] = true;
         
         return Director::test($this->FormAction(), $data, Controller::curr()->getSession());
@@ -1473,7 +1473,7 @@ function testSubmission($action, $data) {
 	 * Test an ajax submission of this form.
 	 * @return SS_HTTPResponse the response object that the handling controller produces.  You can interrogate this in your unit test.
 	 */
-	function testAjaxSubmission($action, $data) {
+	public function testAjaxSubmission($action, $data) {
 		$data['ajax'] = 1;
 		return $this->testSubmission($action, $data);
 	}
@@ -1486,7 +1486,7 @@ function testAjaxSubmission($action, $data) {
 class Form_FieldMap extends ViewableData {
 	protected $form;
 	
-	function __construct($form) {
+	public function __construct($form) {
 		$this->form = $form;
 		parent::__construct();
 	}
@@ -1494,11 +1494,11 @@ function __construct($form) {
 	/**
 	 * Ensure that all potential method calls get passed to __call(), therefore to dataFieldByName
 	 */
-	function hasMethod($method) {
+	public function hasMethod($method) {
 		return true;
 	}
 
-	function __call($method, $args = null) {
+	public function __call($method, $args = null) {
 		return $this->form->Fields()->fieldByName($method);
 	}
 }