View GitHub Repository XoopsCore25

Schedule Inspection
Subscribe
Passed
Push — master ( 3c6756...8b3b37 )
by Michael
20:43 queued 12:59
created

Demo_resizeCanvas::execute()   A

↳ Parent: Demo_resizeCanvas

Complexity

Conditions 2
Paths 1

Size

Total Lines 11
Code Lines 8

Duplication

Lines 0
Ratio 0 %

Importance

Changes 2
Bugs 0 Features 0
Metric Value
cc 2
eloc 8
c 2
b 0
f 0
nc 1
nop 2
dl 0
loc 11
rs 10
1 
<?php
2 
	/**
3 
	 * @package Demos
4 
	 */
5 
	class Demo_resizeCanvas extends Demo
6 
	{
7 
		public $order = 910;
8
9 
		function init()
10 
		{
11 
			$this->addField(new CoordinateField('width', '100%+30'));
12 
			$this->addField(new CoordinateField('height', 200));
13 
			$this->addField(new CoordinateField('left', '2'));
14 
			$this->addField(new CoordinateField('top', 'bottom-10'));
15 
			$this->addField(new ColorField('color', 'ffffff'));
16 
			$this->addField(new SelectField('scale', array('any', 'down', 'up'), 'any'));
17 
			$this->addField(new CheckboxField('merge', false, "Merge or copy over"));
18 
		}
19
20 
		function execute($image, $request)
0 ignored issues
– show
Unused Code introduced by
The parameter $request is not used and could be removed. ( Ignorable by Annotation )

If this is a false-positive, you can also ignore this issue in your code via the ignore-unused  annotation

20 
		function execute($image, /** @scrutinizer ignore-unused */ $request)

This check looks for parameters that have been defined for a function or method, but which are not used in the method body.

Loading history...
Best Practice introduced by
It is generally recommended to explicitly declare the visibility for methods.

Adding explicit visibility (private, protected, or public) is generally recommend to communicate to other developers how, and from where this method is intended to be used.

Loading history...
21 
		{
22 
			$width = $this->fields['width']->value;
23 
			$height = $this->fields['height']->value;
24 
			$left = $this->fields['left']->value;
25 
			$top = $this->fields['top']->value;
26 
			$color = $this->fields['color']->value;
27 
			$scale = $this->fields['scale']->value;
28 
			$merge = $this->fields['merge']->value;
29
30 
			return $image->resizeCanvas($width, $height, $left, $top, $color ? hexdec($color) : null, $scale, $merge);
31 
		}
32 
	}
33