Why is this an issue?
Redundant Boolean literals should be removed from expressions to improve readability.
Noncompliant code example
SET SERVEROUTPUT ON
DECLARE
foo BOOLEAN := TRUE;
BEGIN
IF foo = FALSE THEN -- Noncompliant
DBMS_OUTPUT.PUT_LINE('foo = false!');
ELSIF foo = TRUE THEN -- Noncompliant
DBMS_OUTPUT.PUT_LINE('foo = true!');
END IF;
END;
/
Compliant solution
DECLARE
foo BOOLEAN := TRUE;
BEGIN
IF NOT foo THEN -- Compliant
DBMS_OUTPUT.PUT_LINE('foo = false!');
ELSIF foo THEN -- Compliant
DBMS_OUTPUT.PUT_LINE('foo = true!');
END IF;
END;
/