EcommerceVoteDataDecorator

Complexity

Total Complexity

9

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	HasEcommerceVote()	0	4	1
A	EcommerceVotes()	0	8	2
B	updateCMSFields()	0	21	6

<?php

/**
 *@author nicolaas[at]sunnysideup.co.nz
 *
 *
 *
 **/

class EcommerceVoteDataDecorator extends DataObjectDecorator

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

{
    public function HasEcommerceVote()
    {
        return DataObject::get_one("EcommerceVote", "SessionID = '".Session_ID()."' AND PageID = ".$this->owner->ID);
    }

    public function EcommerceVotes()

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

    {
        $count = DB::query("Select COUNT(ID) c FROM EcommerceVote WHERE PageID = ".$this->owner->ID);
        foreach ($count as $item) {
            return $item["c"];
        }
        return 0;
    }

    public function updateCMSFields(FieldSet &$fields)
    {
        $array = EcommerceVote::get_array_of_classes_used();
        $show = false;
        if (!is_array($array) || !count($array)) {
            $show = true;
        } else {
            foreach ($array as $className) {
                if ($this->owner instanceof $className) {
                    $show = true;
                }
            }
        }
        if ($show) {
            $fields->addFieldsToTab(
                "Root.Content.Votes",
                new LiteralField("Votes", "Number of votes: ".$this->EcommerceVotes())

new \LiteralField('Votes', 'Number of votes: ' . $this->EcommerceVotes()) is of type object<LiteralField>, but the function expects a array.

            );
        }
        return $fields;
    }
}