diff options
author | Arne H Juul <arnej@yahooinc.com> | 2022-03-20 17:33:17 +0000 |
---|---|---|
committer | Arne H Juul <arnej@yahooinc.com> | 2022-03-20 17:33:17 +0000 |
commit | 4c47039526ff7ced1ddb5ee180935ce41f5ed84e (patch) | |
tree | 38fc6f2336de1a5173c2ff15ad5d507fa07a5497 /config-model | |
parent | d83910510077966d3ff26c0c8f98c7b8d12743cc (diff) |
lookup document types by name, not id
Diffstat (limited to 'config-model')
-rw-r--r-- | config-model/src/main/java/com/yahoo/searchdefinition/DocumentModelBuilder.java | 12 |
1 files changed, 6 insertions, 6 deletions
diff --git a/config-model/src/main/java/com/yahoo/searchdefinition/DocumentModelBuilder.java b/config-model/src/main/java/com/yahoo/searchdefinition/DocumentModelBuilder.java index 5acaf353faf..4077afe8c9a 100644 --- a/config-model/src/main/java/com/yahoo/searchdefinition/DocumentModelBuilder.java +++ b/config-model/src/main/java/com/yahoo/searchdefinition/DocumentModelBuilder.java @@ -233,13 +233,13 @@ public class DocumentModelBuilder { if (type instanceof TemporaryStructuredDataType) { DataType other = repo.getDataType(type.getId()); if (other == null || other == type) { - other = getDocumentType(docs, type.getId()); + other = getDocumentType(docs, type.getName()); } if (other != null) { type = other; } } else if (type instanceof DocumentType) { - DataType other = getDocumentType(docs, type.getId()); + DataType other = getDocumentType(docs, type.getName()); if (other != null) { type = other; } else if (! type.getName().equals("document")) { @@ -247,7 +247,7 @@ public class DocumentModelBuilder { ("Can not handle nested document definitions. Undefined document type: " + type.toString()); } } else if (type instanceof NewDocumentType) { - DataType other = getDocumentType(docs, type.getId()); + DataType other = getDocumentType(docs, type.getName()); if (other != null) { type = other; } @@ -297,7 +297,7 @@ public class DocumentModelBuilder { } else if (type instanceof NewDocumentReferenceDataType) { var t = (NewDocumentReferenceDataType) type; - var doc = getDocumentType(docs, t.getTargetTypeId()); + var doc = getDocumentType(docs, t.getTargetTypeName()); type = doc.getReferenceDataType(); } if (type != original) { @@ -306,9 +306,9 @@ public class DocumentModelBuilder { return type; } - private static NewDocumentType getDocumentType(Collection<NewDocumentType> docs, int id) { + private static NewDocumentType getDocumentType(Collection<NewDocumentType> docs, String name) { for (NewDocumentType doc : docs) { - if (doc.getId() == id) { + if (doc.getName().equals(name)) { return doc; } } |