sunnysideup / silverstripe-forsale

code/ForSaleOnePage.php

<?php
/**
 *@author nicolaas[at] sunnysideup.co.nz
 *@description: individual ForSale page. Usually, these are not viewed as they can be read completely from the parent (ForSale HOLDER) page.
 */
class ForSaleOnePage extends Page

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public static $icon = "mysite/images/treeicons/ForSaleOnePage";

    public static $default_parent = 'ForSaleHolderPage';

    public static $allowed_children = "none";

    public static $db = array(
        "IntroText" => "Varchar(255)",
        "PriceDescription" => "Varchar(255)",
        "ContactDetails" => "HTMLText"
    );

    public static $has_one = array(
        "ProductImage" => "Image"
    );


    //static $has_many = array();

60% of this comment could be valid code. Did you maybe forget this after debugging?

    public function getCMSFields()

The return type could not be reliably inferred; please add a @return annotation.

    {
        $fields = parent::getCMSFields();
        $fields->replaceField("Title", new TextField("Title", "Product Name"));
        $fields->replaceField("MenuTitle", new TextField("MenuTitle", "Product Name - short version for menus"));
        $fields->replaceField("Content", new HtmlEditorField("Content", "Product Description", $rows = 7, $cols = 7));
        $fields->addFieldToTab("Root.Content.ProductInfo", new TextField("IntroText", "Intro Text"));
        $fields->addFieldToTab("Root.Content.ProductInfo", new TextField("PriceDescription", "Price Description"));
        $fields->addFieldToTab("Root.Content.ProductInfo", new HtmlEditorField("ContactDetails", "Contact Details", $rows = 7, $cols = 7));
        $fields->addFieldToTab("Root.Content.ProductInfo", new ImageField("ProductImage", "Product Image"));
        return $fields;
    }
}

class ForSaleOnePage_Controller extends Page_Controller

PSR1 recommends that each class should be in its own file to aid autoloaders.

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    public function init()
    {
        parent::init();
    }
}