This project does not seem to handle request data directly as such no vulnerable execution paths were found.
include
, or for example
via PHP's auto-loading mechanism.
These results are based on our legacy PHP analysis, consider migrating to our new PHP analysis engine instead. Learn more
1 | <?php |
||
2 | |||
3 | class Ajde_Component_Image extends Ajde_Component |
||
4 | { |
||
5 | public static function processStatic(Ajde_Template_Parser $parser, $attributes) |
||
6 | { |
||
7 | $instance = new self($parser, $attributes); |
||
8 | |||
9 | return $instance->process(); |
||
10 | } |
||
11 | |||
12 | protected function _init() |
||
13 | { |
||
14 | return [ |
||
15 | 'base64' => 'base64', |
||
16 | 'filename' => 'html', |
||
17 | 'output' => 'image', |
||
18 | ]; |
||
19 | } |
||
20 | |||
21 | public function process() |
||
22 | { |
||
23 | switch ($this->_attributeParse()) { |
||
24 | case 'base64': |
||
25 | $image = new Ajde_Resource_Image($this->attributes['filename']); |
||
26 | $image->setWidth(issetor($this->attributes['width'])); |
||
0 ignored issues
–
show
|
|||
27 | $image->setHeight(issetor($this->attributes['height'])); |
||
0 ignored issues
–
show
The method
setHeight does not exist on object<Ajde_Resource_Image> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
28 | $image->setCrop(Ajde_Component_String::toBoolean(issetor($this->attributes['crop'], true))); |
||
0 ignored issues
–
show
The method
setCrop does not exist on object<Ajde_Resource_Image> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
29 | |||
30 | $controller = Ajde_Controller::fromRoute(new Ajde_Core_Route('_core/component:imageBase64')); |
||
31 | $controller->setImage($image); |
||
0 ignored issues
–
show
The method
setImage does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
32 | $controller->setWidth(issetor($this->attributes['width'], null)); |
||
0 ignored issues
–
show
The method
setWidth does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
33 | $controller->setHeight(issetor($this->attributes['height'], null)); |
||
0 ignored issues
–
show
The method
setHeight does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
34 | $controller->setExtraClass(issetor($this->attributes['class'], '')); |
||
0 ignored issues
–
show
The method
setExtraClass does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
35 | |||
36 | return $controller->invoke(); |
||
37 | break; |
||
0 ignored issues
–
show
break is not strictly necessary here and could be removed.
The break statement is not necessary if it is preceded for example by a return statement: switch ($x) {
case 1:
return 'foo';
break; // This break is not necessary and can be left off.
}
If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive. ![]() |
|||
38 | case 'html': |
||
39 | return self::getImageTag( |
||
40 | $this->attributes['filename'], |
||
41 | issetor($this->attributes['width']), |
||
42 | issetor($this->attributes['height']), |
||
43 | Ajde_Component_String::toBoolean(issetor($this->attributes['crop'], true)), |
||
44 | issetor($this->attributes['class'], ''), |
||
45 | issetor($this->attributes['lazy'], false), |
||
0 ignored issues
–
show
issetor($this->attributes['lazy'], false) is of type null , but the function expects a boolean .
It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { }
$x = '123'; // string "123"
// Instead of
acceptsInteger($x);
// we recommend to use
acceptsInteger((integer) $x);
![]() |
|||
46 | issetor($this->attributes['absoluteUrl'], false) |
||
0 ignored issues
–
show
issetor($this->attributes['absoluteUrl'], false) is of type null , but the function expects a boolean .
It seems like the type of the argument is not accepted by the function/method which you are calling. In some cases, in particular if PHP’s automatic type-juggling kicks in this might be fine. In other cases, however this might be a bug. We suggest to add an explicit type cast like in the following example: function acceptsInteger($int) { }
$x = '123'; // string "123"
// Instead of
acceptsInteger($x);
// we recommend to use
acceptsInteger((integer) $x);
![]() |
|||
47 | ); |
||
48 | break; |
||
0 ignored issues
–
show
break is not strictly necessary here and could be removed.
The break statement is not necessary if it is preceded for example by a return statement: switch ($x) {
case 1:
return 'foo';
break; // This break is not necessary and can be left off.
}
If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive. ![]() |
|||
49 | case 'image': |
||
50 | return false; |
||
51 | break; |
||
0 ignored issues
–
show
break is not strictly necessary here and could be removed.
The break statement is not necessary if it is preceded for example by a return statement: switch ($x) {
case 1:
return 'foo';
break; // This break is not necessary and can be left off.
}
If you would like to keep this construct to be consistent with other case statements, you can safely mark this issue as a false-positive. ![]() |
|||
52 | } |
||
53 | // TODO: |
||
54 | throw new Ajde_Component_Exception('Missing required attributes for component call'); |
||
55 | } |
||
56 | |||
57 | public static function getImageTag( |
||
58 | $filename, |
||
59 | $width = null, |
||
60 | $height = null, |
||
61 | $crop = true, |
||
62 | $class = '', |
||
63 | $lazy = false, |
||
64 | $absoluteUrl = false |
||
65 | ) { |
||
66 | $image = new Ajde_Resource_Image($filename); |
||
67 | $image->setWidth($width); |
||
0 ignored issues
–
show
The method
setWidth does not exist on object<Ajde_Resource_Image> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
68 | $image->setHeight($height); |
||
0 ignored issues
–
show
The method
setHeight does not exist on object<Ajde_Resource_Image> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
69 | $image->setCrop($crop); |
||
0 ignored issues
–
show
The method
setCrop does not exist on object<Ajde_Resource_Image> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
70 | |||
71 | $controller = Ajde_Controller::fromRoute(new Ajde_Core_Route('_core/component:image')); |
||
72 | $controller->setImage($image); |
||
0 ignored issues
–
show
The method
setImage does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
73 | $controller->setExtraClass($class); |
||
0 ignored issues
–
show
The method
setExtraClass does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
74 | $controller->setLazy($lazy); |
||
0 ignored issues
–
show
The method
setLazy does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
75 | $controller->setAbsoluteUrl($absoluteUrl); |
||
0 ignored issues
–
show
The method
setAbsoluteUrl does not exist on object<Ajde_Controller> ? Since you implemented __call , maybe consider adding a @method annotation.
If you implement This is often the case, when class ParentClass {
private $data = array();
public function __call($method, array $args) {
if (0 === strpos($method, 'get')) {
return $this->data[strtolower(substr($method, 3))];
}
throw new \LogicException(sprintf('Unsupported method: %s', $method));
}
}
/**
* If this class knows which fields exist, you can specify the methods here:
*
* @method string getName()
*/
class SomeClass extends ParentClass { }
![]() |
|||
76 | |||
77 | return $controller->invoke(); |
||
78 | } |
||
79 | } |
||
80 |
If you implement
__call
and you know which methods are available, you can improve IDE auto-completion and static analysis by adding a @method annotation to the class.This is often the case, when
__call
is implemented by a parent class and only the child class knows which methods exist: