Fix: Adjusted BPN validations on different operators

[danielkenji01]

WHAT

• Fixed IS_ALL_OF operator validation. It had the same behavior of EQ operator
• Fixed IS_NONE_OF operator validation. It had the same behavior of the NEQ operator
• Fixed NEQ operator validation. It was considering the opposite result of the IN operator, and it should be the opposite of the EQ operator
• Removed a validation that returned false when the list of assigned groups is empty. This should be evaluated by the operators.

WHY

In order to keep the correct behavior on BPNL validations

FURTHER NOTES

Some minor adjusts reported by Sonar, including:

• Adjusted String format and casting

Closes ##1411
Closes ##775

[paullatzelsperger]

TL;DR: don't just change unrelated code, don't use final except for field declarations

[danielkenji01]

TL;DR: don't just change unrelated code, don't use final except for field declarations

Thank you for the comments, I followed them and adjusted the code.

[paullatzelsperger]

please resolve conversations that you fixed, and then re-request the review

[sonarcloud]

Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarCloud