WideImage\Operation\ApplyFilter

Complexity

Total Complexity

6

Size/Duplication

Total Lines	38
Duplicated Lines	0 %

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	execute()	0	19	6

<?php
	/**
##DOC-SIGNATURE##

    This file is part of WideImage.
		
    WideImage is free software; you can redistribute it and/or modify
    it under the terms of the GNU Lesser General Public License as published by
    the Free Software Foundation; either version 2.1 of the License, or
    (at your option) any later version.
		
    WideImage is distributed in the hope that it will be useful,
    but WITHOUT ANY WARRANTY; without even the implied warranty of
    MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
    GNU Lesser General Public License for more details.
		
    You should have received a copy of the GNU Lesser General Public License
    along with WideImage; if not, write to the Free Software
    Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA  02110-1301  USA

    * @package Internal/Operations
  **/

namespace WideImage\Operation;

use WideImage\Exception\GDFunctionResultException;

/**
 * ApplyFilter operation class
 * 
 * @package Internal/Operations
 */
class ApplyFilter
{
	/**
	 * A list of filters that only accept one arguments for imagefilter()
	 *
	 * @var array
	 */
	protected static $one_arg_filters = array(IMG_FILTER_SMOOTH, IMG_FILTER_CONTRAST, IMG_FILTER_BRIGHTNESS);
	
	/**
	 * Executes imagefilter
	 *
	 * @param \WideImage\Image $image
	 * @param int $filter 
	 * @param numeric $arg1

The type WideImage\Operation\numeric was not found. Maybe you did not declare it correctly or list all dependencies?

	 * @param numeric $arg2
	 * @param numeric $arg3
	 * @return \WideImage\TrueColorImage
	 */
	public function execute($image, $filter, $arg1 = null, $arg2 = null, $arg3 = null, $arg4 = null)
	{
		$new = $image->asTrueColor();
		
		if (in_array($filter, static::$one_arg_filters)) {
			$res = imagefilter($new->getHandle(), $filter, $arg1);

It seems like $arg1 can also be of type WideImage\Operation\numeric; however, parameter $arg1 of imagefilter() does only seem to accept integer, maybe add an additional type check?

		} elseif (defined('IMG_FILTER_PIXELATE') && $filter == IMG_FILTER_PIXELATE) {
			$res = imagefilter($new->getHandle(), $filter, $arg1, $arg2);

It seems like $arg2 can also be of type WideImage\Operation\numeric; however, parameter $arg2 of imagefilter() does only seem to accept integer, maybe add an additional type check?

		} elseif ($filter == IMG_FILTER_COLORIZE) {
			$res = imagefilter($new->getHandle(), $filter, $arg1, $arg2, $arg3, $arg4);

It seems like $arg3 can also be of type WideImage\Operation\numeric; however, parameter $arg3 of imagefilter() does only seem to accept integer, maybe add an additional type check?

		} else {
			$res = imagefilter($new->getHandle(), $filter);
		}
		
		if (!$res) {
			throw new GDFunctionResultException("imagefilter() returned false");
		}
		
		return $new;
	}
}