diff options
author | Tor Egge <Tor.Egge@broadpark.no> | 2018-11-27 15:07:13 +0100 |
---|---|---|
committer | Tor Egge <Tor.Egge@broadpark.no> | 2018-11-27 15:07:13 +0100 |
commit | a92d0b2c764aa4d3e0acc99955ca1abc23a96f9d (patch) | |
tree | 344134b72259303cf9aac4aace2e238bba10d8b5 /config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java | |
parent | e18be113b03bb8138109ba1c183ab8e282c23ae3 (diff) |
Fix typo.
Factor out logic to check for nested field name.
Use camel case instead of underscore for compound function name.
Simplify test.
Diffstat (limited to 'config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java')
-rw-r--r-- | config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java b/config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java index a69581559e5..82b56f9c961 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java @@ -39,11 +39,15 @@ public class ImportedFields extends Derived implements ImportedFieldsConfig.Prod } } + private static boolean isNestedFieldName(String fieldName) { + return fieldName.indexOf('.') != -1; + } + private static void considerField(ImportedFieldsConfig.Builder builder, ImportedField field) { ImmutableSDField targetField = field.targetField(); String targetFieldName = targetField.getName(); - if (targetFieldName.indexOf('.') == -1) { - if (field.targetField().doesAttributing()) { + if (!isNestedFieldName(targetFieldName)) { + if (targetField.doesAttributing()) { builder.attribute.add(createAttributeBuilder(field)); } } else { |