fix(RHINENG-15212): Make rule expand work SystemDetails (#2329)

RedHatInsights · Jan 9, 2025 · 3096b55 · 3096b55
1 parent 8c1a625
commit 3096b55
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/PresentationalComponents/RulesTable/RulesTableRest.js b/src/PresentationalComponents/RulesTable/RulesTableRest.js
@@ -27,6 +27,7 @@ const RulesTable = ({
   setRuleValues,
   ruleValues,
   valueDefinitions,
+  skipValueDefinitions,
   onRuleValueReset,
   DedicatedAction,
   onValueOverrideSave,
@@ -81,9 +82,11 @@ const RulesTable = ({
         ruleValues: ruleRuleValues,
       };
 
-      return valueDefinitions === undefined ||
-        valueDefinitions.loading === true ||
-        valueDefinitions.data === undefined ? (
+      return (skipValueDefinitions === undefined ||
+        skipValueDefinitions === false) &&
+        (valueDefinitions === undefined ||
+          valueDefinitions.loading === true ||
+          valueDefinitions.data === undefined) ? (
         <List />
       ) : (
         <RuleDetailsRow
@@ -181,6 +184,7 @@ RulesTable.propTypes = {
     data: propTypes.any,
     loading: propTypes.bool.isRequired,
   }),
+  skipValueDefinitions: propTypes.bool,
   onValueOverrideSave: propTypes.func,
   onSelect: propTypes.oneOf([propTypes.func, propTypes.bool]),
   total: propTypes.number,

diff --git a/src/SmartComponents/SystemDetails/RuleResults.js b/src/SmartComponents/SystemDetails/RuleResults.js
@@ -59,6 +59,7 @@ const RuleResults = ({ reportTestResult }) => {
       onSelect={true}
       remediationsEnabled
       reportTestResult={reportTestResult}
+      skipValueDefinitions={true}
       // TODO: provide ruleTree
       // TODO: hide passed rules by default
     />