PRO-7997: Fix conditional fields BC break (hidden when it depends on a hidden) (#4996)

* Fix BC break - hidden when depends on a hidden

Author: myovchev

CHANGELOG.md

@@ -15,6 +15,7 @@
 
 * Add missing Pages manager shortcuts list helper.
 * Improve the `isEmpty` method of the rich text widget to take into account the HTML blocks (`<figure>` and `<table>`) that are not empty but do not contain any plain text.
+* (Backward compatibility break) Conditional field that depends on already hidden field is also hidden, again.
 
 ## 4.18.0 (2025-06-11)
 

modules/@apostrophecms/schema/ui/apos/lib/conditionalFields.js

@@ -124,7 +124,7 @@ export function getConditionalFields(
           values,
           field[conditionType],
           (propName, condition, docValue) =>
-            evaluateExternal(propName, condition, conditionType)
+            evaluateExternalAndHidden(propName, condition, conditionType)
         );
       }
     }
@@ -134,10 +134,14 @@ export function getConditionalFields(
 
   // Handle external conditions as a voter function.
   // Non-boolean returns are ignored by the `checkIfConditions` function.
-  function evaluateExternal(propName, conditionValue, conditionType) {
+  function evaluateExternalAndHidden(propName, conditionValue, conditionType) {
     if (isExternalCondition(propName, conditionType)) {
       return externalConditionsResults[conditionType]?.[propName] === conditionValue;
     }
+
+    if (result[conditionType]?.[propName] === false) {
+      return false;
+    }
   }
 }