FlashObjectDOD

Complexity

Total Complexity

17

Size/Duplication

Total Lines	76
Duplicated Lines	13.16 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	extraStatics()	0	11	1
A	set_classes_with_flash()	0	4	1
A	get_classes_with_flash()	0	4	1
A	set_classes_without_flash()	0	4	1
A	get_classes_without_flash()	0	4	1
B	updateCMSFields()	10	21	9
A	CreateFlashObject()	0	10	2
A	FlashObjectData()	0	4	1

<?php
/*
    @ see http://code.google.com/p/swfobject/wiki/documentation
    @ see http://www.swffix.org/swfobject/generator/
*/

class FlashObjectDOD extends SiteTreeDecorator

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

{
    public function extraStatics()
    {
        return array(
            'db' =>   array(
                "Title" => "Varchar(255)"
            ),
            'has_one' => array(
                "FlashFile" => "File"
            ),
        );
    }

    protected static $classes_with_flash = array();
    public static function set_classes_with_flash($v)
    {
        self::$classes_with_flash = $v;
    }
    public static function get_classes_with_flash()
    {
        return self::$classes_with_flash;
    }

    protected static $classes_without_flash = array();
    public static function set_classes_without_flash($v)
    {
        self::$classes_without_flash = $v;
    }
    public static function get_classes_without_flash()
    {
        return self::$classes_without_flash;
    }

    public function updateCMSFields(FieldSet &$fields)
    {
        if (! FlashObject::has_external_flash_file()) {
            $show = true;
            if (is_array(self::get_classes_with_flash()) && count(self::get_classes_with_flash())) {

This code seems to be duplicated across your project.

                if (!in_array($this->owner->ClassName, self::get_classes_with_flash())) {
                    $show = false;
                }
            }
            if (is_array(self::get_classes_without_flash()) && count(self::get_classes_without_flash())) {

This code seems to be duplicated across your project.

                if (in_array($this->owner->ClassName, self::get_classes_without_flash())) {
                    $show = false;
                }
            }
            if ($show) {
                $fields->addFieldToTab("Root.Content.FlashObject", new FileIFrameField('FlashFile', 'File'));
                $fields->addFieldToTab("Root.Content.FlashObject", new TextField('Title', 'Title'));
            }
        }
        return $fields;
    }

    public function CreateFlashObject()
    {
        if ($this->owner->FlashFileID) {
            $obj = new FlashObject();
            $flashFile = $this->owner->FlashFile();
            return $obj->CreateFlashObject($this->owner->Title, null, $flashFile->Filename);
        } else {
            return new DataObjectSet();
        }
    }

    /*
    legacy function
    */

    public function FlashObjectData()
    {
        return $this->CreateFlashObject();
    }
}