Demo_getChannels

Complexity

Total Complexity

4

Size/Duplication

Total Lines	22
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	execute()	0	8	3
A	init()	0	6	1

<?php
	/**
	 * @package Demos
	 */
	class Demo_getChannels extends Demo
	{
		public $order = 500;
		
		protected $channels = array('red', 'green', 'blue', 'alpha');
		
		function init()
		{
			$this->addField(new CheckboxField('red', true));
			$this->addField(new CheckboxField('green', false));
			$this->addField(new CheckboxField('blue', true));
			$this->addField(new CheckboxField('alpha', false));
		}
		
		function execute($img, $request)

It is generally recommended to explicitly declare the visibility for methods.

The parameter $request is not used and could be removed.

		{
			$on = array();
			foreach ($this->channels as $name)
				if ($this->fields[$name]->value)
					$on[] = $name;
			
			return $img->getChannels($on);
		}
	}