diff options
author | Bjørn Christian Seime <bjorncs@yahoo-inc.com> | 2017-02-16 13:39:45 +0100 |
---|---|---|
committer | Bjørn Christian Seime <bjorncs@yahoo-inc.com> | 2017-02-16 13:39:45 +0100 |
commit | b0434818ef350079746b20d6f75b9f39ba96582d (patch) | |
tree | 9dcec87d6e2ac18a36d6d4d53d563662f2dcd1ed /config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java | |
parent | aa1c38bfb7bd0eb26d014abe9345a7cdc5ff3446 (diff) |
Disallow reference field to refer to an index field
Diffstat (limited to 'config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java')
-rw-r--r-- | config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java b/config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java index bc8d1eccbbc..7ff17329609 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/processing/ImportedFieldsResolver.java @@ -59,11 +59,13 @@ public class ImportedFieldsResolver extends Processor { fail(importedField, targetFieldAsString(targetFieldName, reference) + ": Not found"); } else if (!targetField.doesAttributing()) { fail(importedField, targetFieldAsString(targetFieldName, reference) + ": Is not an attribute"); + } else if (targetField.doesIndexing()) { + fail(importedField, targetFieldAsString(targetFieldName, reference) + ": Index not allowed"); } return targetField; } - private String targetFieldAsString(String targetFieldName, DocumentReference reference) { + private static String targetFieldAsString(String targetFieldName, DocumentReference reference) { return "Field '" + targetFieldName + "' via reference field '" + reference.referenceField().getName() + "'"; } |