sunnysideup / silverstripe-vimeoembed

code/model/VimeoDOD.php

<?php

class VimeoDOD extends DataExtension

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

{
    private static $has_one = array(

The property $has_one is not used and could be removed.

        "VimeoDataObject" => "VimeoDataObject"
    );

    private static $exclude_vimeo_from_page_classes = array();

The property $exclude_vimeo_from_page_classes is not used and could be removed.

    private static $include_vimeo_in_page_classes = array();

The property $include_vimeo_in_page_classes is not used and could be removed.

    public function updateCMSFields(FieldList $fields)
    {
        if ($this->HasVimeo()) {
            $listObject = VimeoDataObject::get();
            if ($listObject->count()) {
                $tab = _t("VimeoDOD.TAB", "Root.Vimeo");
                $list = array( 0 => _t("VimeoDOD.EMPTYSTRING", "--- select vimeo video ---")) + $listObject->map($index = 'ID', $titleField = 'Title')->toArray();
                $fields->addFieldToTab($tab, new DropdownField("VimeoDataObjectID", _t("VimeoDOD.URLFIELD", "Video"), $list));
                $linkToModelAdmin = _t("VimeoDOD.LINKTOMODELADMIN", "To edit your videos, please go to <a href=\"/admin/vimeos\">Vimeo Editing Page</a>.");
                $fields->addFieldToTab($tab, new LiteralField("VimeoDataObjectIDEDIT", "<p>$linkToModelAdmin</p>"));
            }
        }
        return $fields;
    }

    public function HasVimeo()
    {
        $hasVimeo = true;
        $includeClasses = $this->owner->Config()->get("include_vimeo_in_page_classes");
        if (count($includeClasses)) {
            if (!in_array($this->owner->ClassName, Config::inst()->get("VimeoDOD", "include_vimeo_in_page_classes"))) {
                $hasVimeo = false;
            }
        }
        $excludeClasses = Config::inst()->get("VimeoDOD", "exclude_vimeo_from_page_classes");
        if (count($excludeClasses)) {
            if (in_array($this->owner->ClassName, $excludeClasses)) {
                $hasVimeo = false;
            }
        }
        return $hasVimeo;
    }

    public function VimeosInThisSection()

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

    {
        return Page::get()->filter(array(
            "ParentID" => array(intval($this->owner->ParentID), intval($this->owner->ID)),
            "VimeoDataObjectID:GreaterThan" => 0,
            "ShowInSearch" => 1
        ));
    }
}