Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Oct 7, 2025

Backport of PR #55316

joshtrichards and others added 2 commits October 7, 2025 14:26
Fixes #55276 and tidies things up a bit more for future maintenance.

Signed-off-by: Josh <[email protected]>
Signed-off-by: Joas Schilling <[email protected]>
@github-actions github-actions bot changed the title [stable30] [stable32] fix(workflowenigne): stricter length header handling [stable30] fix(workflowenigne): stricter length header handling Oct 7, 2025
$this->size = Util::numericToNumber($size);
} else {
// No valid size header found
$this->size = false;

Check notice

Code scanning / Psalm

PossiblyFalsePropertyAssignmentValue Note

$this->size with non-falsable declared type 'float|int|null' cannot be assigned possibly false type 'false'
protected $size;

/** @var int|float|null */
protected int|float|null $size;

Check notice

Code scanning / Psalm

PropertyNotSetInConstructor Note

Property OCA\WorkflowEngine\Check\FileSize::$size is not defined in constructor of OCA\WorkflowEngine\Check\FileSize or in any methods called in the constructor
@nickvergessen nickvergessen deleted the backport/55316/stable30 branch October 8, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants