nabble /
ajde
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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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. Loading history...
|
|||
| 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);
Loading history...
|
|||
| 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);
Loading history...
|
|||
| 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. Loading history...
|
|||
| 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. Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 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 { }
Loading history...
|
|||
| 76 | |||
| 77 | return $controller->invoke(); |
||
| 78 | } |
||
| 79 | } |
||
| 80 |
If you implement
__calland 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
__callis implemented by a parent class and only the child class knows which methods exist: