diff --git a/code/controller/WigetContentControllerExtension.php b/code/controller/WigetContentControllerExtension.php index 78d729f..c9351c8 100644 --- a/code/controller/WigetContentControllerExtension.php +++ b/code/controller/WigetContentControllerExtension.php @@ -1,10 +1,10 @@ owner->getRequest()->param('ID'); if(!$SQL_id) return false; diff --git a/code/form/WidgetAreaEditor.php b/code/form/WidgetAreaEditor.php index 63f1290..e72e6f6 100644 --- a/code/form/WidgetAreaEditor.php +++ b/code/form/WidgetAreaEditor.php @@ -6,20 +6,20 @@ */ class WidgetAreaEditor extends FormField { - function __construct($name, $widgetClasses = array('Widget'), $maxWidgets = 0) { + public function __construct($name, $widgetClasses = array('Widget'), $maxWidgets = 0) { $this->MaxWidgets = $maxWidgets; $this->widgetClasses = $widgetClasses; parent::__construct($name); } - function FieldHolder($properties = array()) { + public function FieldHolder($properties = array()) { Requirements::css('widgets/css/WidgetAreaEditor.css'); Requirements::javascript('widgets/javascript/WidgetAreaEditor.js'); return $this->renderWith("WidgetAreaEditor"); } - function AvailableWidgets() { + public function AvailableWidgets() { $widgets= new ArrayList(); @@ -34,7 +34,7 @@ class WidgetAreaEditor extends FormField { return $widgets; } - function UsedWidgets() { + public function UsedWidgets() { // Call class_exists() to load Widget.php earlier and avoid a segfault class_exists('Widget'); @@ -43,16 +43,16 @@ class WidgetAreaEditor extends FormField { return $widgets; } - function IdxField() { + public function IdxField() { return $this->id() . 'ID'; } - function Value() { + public function Value() { $relationName = $this->name; return $this->form->getRecord()->getComponent($relationName)->ID; } - function saveInto(DataObjectInterface $record) { + public function saveInto(DataObjectInterface $record) { $name = $this->name; $idName = $name . "ID"; diff --git a/code/model/Widget.php b/code/model/Widget.php index e855cd4..0156b66 100644 --- a/code/model/Widget.php +++ b/code/model/Widget.php @@ -11,30 +11,31 @@ * @subpackage widgets */ class Widget extends DataObject { - static $db = array( + + public static $db = array( "Sort" => "Int", "Enabled" => "Boolean" ); - static $defaults = array( + public static $defaults = array( 'Enabled' => true ); - static $has_one = array( + public static $has_one = array( "Parent" => "WidgetArea", ); - static $has_many = array(); - static $many_many = array(); - static $belongs_many_many = array(); + public static $has_many = array(); + public static $many_many = array(); + public static $belongs_many_many = array(); - static $default_sort = "\"Sort\""; + public static $default_sort = "\"Sort\""; - static $title = "Widget Title"; - static $cmsTitle = "Name of this widget"; - static $description = "Description of what this widget does."; + public static $title = "Widget Title"; + public static $cmsTitle = "Name of this widget"; + public static $description = "Description of what this widget does."; - function getCMSFields() { + public function getCMSFields() { $fields = new FieldList(); $this->extend('updateCMSFields', $fields); return $fields; @@ -45,7 +46,7 @@ class Widget extends DataObject { * * @return string HTML */ - function WidgetHolder() { + public function WidgetHolder() { return $this->renderWith("WidgetHolder"); } @@ -60,34 +61,34 @@ class Widget extends DataObject { * * @return string HTML */ - function Content() { + public function Content() { return $this->renderWith(array_reverse(ClassInfo::ancestry($this->class))); } - function Title() { + public function Title() { return _t($this->class.'.TITLE', Object::get_static($this->class, 'title')); } - function CMSTitle() { + public function CMSTitle() { return _t($this->class.'.CMSTITLE', Object::get_static($this->class, 'cmsTitle')); } - function Description() { + public function Description() { return _t($this->class.'.DESCRIPTION', Object::get_static($this->class, 'description')); } - function DescriptionSegment() { + public function DescriptionSegment() { return $this->renderWith('WidgetDescription'); } /** * @see Widget_Controller->editablesegment() */ - function EditableSegment() { + public function EditableSegment() { return $this->renderWith('WidgetEditor'); } - function CMSEditor() { + public function CMSEditor() { $fields = $this->getCMSFields(); $outputFields = new FieldList(); foreach($fields as $field) { @@ -103,15 +104,15 @@ class Widget extends DataObject { return $outputFields; } - function ClassName() { + public function ClassName() { return $this->class; } - function Name() { + public function Name() { return "Widget[".$this->ID."]"; } - function populateFromPostData($data) { + public function populateFromPostData($data) { $fields = $this->getCMSFields(); foreach($data as $name => $value) { if($name != "Type") { @@ -159,11 +160,11 @@ class Widget_Controller extends Controller { */ protected $widget; - static $allowed_actions = array( + public static $allowed_actions = array( 'editablesegment' ); - function __construct($widget = null) { + public function __construct($widget = null) { // TODO This shouldn't be optional, is only necessary for editablesegment() if($widget) { $this->widget = $widget; @@ -186,7 +187,7 @@ class Widget_Controller extends Controller { /** * @return Widget */ - function getWidget() { + public function getWidget() { return $this->widget; } @@ -196,7 +197,7 @@ class Widget_Controller extends Controller { * * @return string HTML */ - function Content() { + public function Content() { return $this->renderWith(array_reverse(ClassInfo::ancestry($this->widget->class))); } @@ -206,7 +207,7 @@ class Widget_Controller extends Controller { * * @return string HTML */ - function WidgetHolder() { + public function WidgetHolder() { return $this->renderWith("WidgetHolder"); } @@ -218,7 +219,7 @@ class Widget_Controller extends Controller { * * @return string HTML */ - function editablesegment() { + public function editablesegment() { $className = $this->urlParams['ID']; if (class_exists('Translatable') && Member::currentUserID()) { // set current locale based on logged in user's locale @@ -241,7 +242,8 @@ class Widget_Controller extends Controller { * @subpackage widgets */ class Widget_TreeDropdownField extends TreeDropdownField { - function FieldHolder($properties = array()) {} - function Field($properties = array()) {} + + public function FieldHolder($properties = array()) {} + public function Field($properties = array()) {} } diff --git a/code/model/WidgetArea.php b/code/model/WidgetArea.php index a3f1d9e..aec1375 100644 --- a/code/model/WidgetArea.php +++ b/code/model/WidgetArea.php @@ -6,17 +6,17 @@ */ class WidgetArea extends DataObject { - static $db = array(); + public static $db = array(); - static $has_one = array(); + public static $has_one = array(); - static $has_many = array( + public static $has_many = array( "Widgets" => "Widget" ); - static $many_many = array(); + public static $many_many = array(); - static $belongs_many_many = array(); + public static $belongs_many_many = array(); public $template = __CLASS__; @@ -28,7 +28,7 @@ class WidgetArea extends DataObject { * * @return SS_List Collection of {@link Widget_Controller} */ - function WidgetControllers() { + public function WidgetControllers() { $controllers = new ArrayList(); foreach($this->ItemsToRender() as $widget) { @@ -46,23 +46,23 @@ class WidgetArea extends DataObject { return $controllers; } - function Items() { + public function Items() { return $this->getComponents('Widgets'); } - function ItemsToRender() { + public function ItemsToRender() { return $this->getComponents('Widgets', "\"Widget\".\"Enabled\" = 1"); } - function forTemplate() { + public function forTemplate() { return $this->renderWith($this->template); } - function setTemplate($template) { + public function setTemplate($template) { $this->template = $template; } - function onBeforeDelete() { + public function onBeforeDelete() { parent::onBeforeDelete(); foreach($this->Widgets() as $widget) { $widget->delete();