Skip to content

Conversation

@backportbot
Copy link

@backportbot backportbot bot commented Dec 4, 2025

Backport of #56839

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

@backportbot backportbot bot added the 3. to review Waiting for reviews label Dec 4, 2025
@backportbot backportbot bot self-assigned this Dec 4, 2025
@CarlSchwan CarlSchwan changed the title [stable30] [stable31] refactor(workflowengine): Check if class is correct [stable30] refactor(workflowengine): Check if class is correct Dec 4, 2025
@CarlSchwan CarlSchwan force-pushed the backport/56839/stable30 branch from 1cb1dcf to 1eb4a8d Compare December 4, 2025 14:12
@CarlSchwan CarlSchwan marked this pull request as ready for review December 4, 2025 14:12
@CarlSchwan CarlSchwan force-pushed the backport/56839/stable30 branch from 1eb4a8d to 8c3968d Compare December 4, 2025 16:28
@blizzz blizzz requested a review from CarlSchwan December 8, 2025 11:52
@CarlSchwan CarlSchwan force-pushed the backport/56839/stable30 branch from 8c3968d to db20a53 Compare December 9, 2025 09:33

protected function validateEvents(string $entity, array $events, IOperation $operation) {
/** @psalm-suppress TaintedCallable newInstance is not called */
$reflection = new \ReflectionClass($entity);

Check notice

Code scanning / Psalm

ArgumentTypeCoercion Note

Argument 1 of ReflectionClass::__construct expects class-string|object|trait-string, but parent type string provided
@blizzz blizzz force-pushed the backport/56839/stable30 branch from db20a53 to cad12d7 Compare December 10, 2025 18:12
@blizzz blizzz merged commit 2938337 into stable30 Dec 10, 2025
189 of 204 checks passed
@blizzz blizzz deleted the backport/56839/stable30 branch December 10, 2025 22:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review Waiting for reviews

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants