tonyair/silverstripe-framework
1
0
Fork 0
mirror of https://github.com/silverstripe/silverstripe-framework synced 2024-10-22 14:05:37 +02:00
Code Issues Projects Releases Wiki Activity

FIX: Strict error warnings on DataExtension

Browse Source 
PHP is throwing strict error warnings when overriding the
updateCMSFields and other functions in custom DataExtensions due to
the fact that the abstract class doesn't declare the variables should
be passed by reference
This commit is contained in:
Michael Andrewartha 2013-02-04 12:37:53 +00:00
parent b381e9933d
commit 1960df8bc3
1 changed files with 4 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
model/DataExtension.php
View File

@ -58,7 +58,7 @@ abstract class DataExtension extends Extension {
* @param $validationResult Local validation result * @param $validationResult Local validation result
* @throws ValidationException * @throws ValidationException
*/ */
public function validate(ValidationResult $validationResult) { public function validate(ValidationResult &$validationResult) {
} }
/** /**
@ -145,7 +145,7 @@ abstract class DataExtension extends Extension {
* *
* @param FieldList $fields FieldList with a contained TabSet * @param FieldList $fields FieldList with a contained TabSet
*/ */
public function updateCMSFields(FieldList $fields) { public function updateCMSFields(FieldList &$fields) {
} }
/** /**
@ -156,7 +156,7 @@ abstract class DataExtension extends Extension {
* *
* @param FieldList $fields FieldList without TabSet nesting * @param FieldList $fields FieldList without TabSet nesting
*/ */
public function updateFrontEndFields(FieldList $fields) { public function updateFrontEndFields(FieldList &$fields) {
} }
/** /**
@ -165,7 +165,7 @@ abstract class DataExtension extends Extension {
* *
* @param FieldList $actions FieldList * @param FieldList $actions FieldList
*/ */
public function updateCMSActions(FieldList $actions) { public function updateCMSActions(FieldList &$actions) {
} }
/** /**