Skip to main content

This site requires you to update your browser. Your browsing experience maybe affected by not having the most up to date version.

We've moved the forum!

Please use forum.silverstripe.org for any new questions (announcement).
The forum archive will stick around, but will be read only.

You can also use our Slack channel or StackOverflow to ask for help.
Check out our community overview for more options to contribute.

General Questions /

General questions about getting started with SilverStripe that don't fit in any of the categories above.

Moderators: martimiz, Sean, Ed, biapar, Willr, Ingo, swaiba

CustomSiteConfig should be compatible with DataExtension::extraStatics()


Go to End


3 Posts   3221 Views

Avatar
priithansen

Community Member, 25 Posts

21 June 2012 at 8:29am

I'm getting this line in my php log on every page load

PHP Strict Standards:  Declaration of CustomSiteConfig::extraStatics() should be compatible with that of DataExtension::extraStatics() in /ss3/mysite/code/CustomSiteConfig.php on line 17

CustomSiteConfig.php files content

<?php

class CustomSiteConfig extends DataExtension {

	public function extraStatics(){
      	return array(
        	'db' => array(
        		'FacebookLink' => 'Varchar(255)'
			)
		); 
	}

	public function updateCMSFields(FieldList $fields) {
		$fields->addFieldToTab('Root.Main', new TextField('FacebookLink', 'Facebook Link'));
		$fields->removeByName('Theme');
	}
}

Using SS3 RC1 wiht MAMP and php version 5.3.6

Any ideas what could be wrong with my code?

Avatar
(deleted)

Community Member, 473 Posts

21 June 2012 at 10:31am

Edited: 21/06/2012 10:32am

The upgrading documentation explains that extraStatics() is deprecated and you should just use

public static $db = array(
	'FacebookLink' => 'Varchar(255)',
);

The error itself is because 3.0 now includes E_STRICT errors, so method declarations must match the declaration of the parent method it is overriding.

Avatar
priithansen

Community Member, 25 Posts

21 June 2012 at 10:54am

Thank You for the help, don't know how I missed that.