Skip to content
This repository was archived by the owner on Aug 26, 2025. It is now read-only.

Error with "OR" conditions on Workflows#14703

Open
MacarenaReina wants to merge 1 commit intoYetiForceCompany:developerfrom
MacarenaReina:patch-1
Open

Error with "OR" conditions on Workflows#14703
MacarenaReina wants to merge 1 commit intoYetiForceCompany:developerfrom
MacarenaReina:patch-1

Conversation

@MacarenaReina
Copy link
Contributor

@MacarenaReina MacarenaReina commented May 21, 2021

Description

Both conditions must be true, because if you only put "OR" conditions, the variable $finalResult will always be true since it is not modified anywhere else.

Testing

Creating a Workflow only with OR conditions and other with AND and OR conditions

Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

Both conditions must be true, because if you only put "OR" conditions, the variable $finalResult will always be true since it is not modified anywhere else.
@ghost
Copy link

ghost commented May 21, 2021

Black Duck Security Report

Merging #14703 into developer will increase security risk!

Added Components

Medium Risk: 3
Clean: 647

Removed Components

Clean: 161

Click here to see full report

@mariuszkrzaczkowski mariuszkrzaczkowski added the 👮‍♂️ VerificationRequired We have to verify if this issue exists in the developer version of YetiForce. label May 24, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

👮‍♂️ VerificationRequired We have to verify if this issue exists in the developer version of YetiForce.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants