Remove unnecessary condition

If fieldName is null or undefined, there is no reason to search for it in dataFields. Looking through the git history this looks to be dead code as a result of refactoring, as opposed to a legitimate bugfix, so I'm removing it.
elastic · Apr 3, 2021 · 8143acd · 8143acd
1 parent 168612b
commit 8143acd
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...k/plugins/security_solution/server/search_strategy/timeline/factory/events/all/helpers.ts b/...k/plugins/security_solution/server/search_strategy/timeline/factory/events/all/helpers.ts
@@ -134,7 +134,7 @@ const mergeTimelineFieldsWithHit = async <T>(
   dataFields: readonly string[],
   ecsFields: readonly string[]
 ) => {
-  if (fieldName != null || dataFields.includes(fieldName)) {
+  if (fieldName != null) {
     const nestedParentPath = getNestedParentPath(fieldName, hit.fields);
     if (
       nestedParentPath != null ||