summaryrefslogtreecommitdiffstats
path: root/config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java
diff options
context:
space:
mode:
authorTor Egge <Tor.Egge@broadpark.no>2018-11-27 15:07:13 +0100
committerTor Egge <Tor.Egge@broadpark.no>2018-11-27 15:07:13 +0100
commita92d0b2c764aa4d3e0acc99955ca1abc23a96f9d (patch)
tree344134b72259303cf9aac4aace2e238bba10d8b5 /config-model/src/main/java/com/yahoo/searchdefinition/derived/ImportedFields.java
parente18be113b03bb8138109ba1c183ab8e282c23ae3 (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.java8
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 {