Merge pull request Netflix#344 from DanielThomas/duplicate-ordinalget

Assign the initial shortcut ordinal value from the already populated field
secti6n · Nov 16, 2018 · 3784f9e · 3784f9e
2 parents c5f2938 + 4bb7287
commit 3784f9e
Showing 1 changed file with 1 addition and 1 deletion.
diff --git a/...java/com/netflix/hollow/api/codegen/delegate/HollowObjectDelegateCachedImplGenerator.java b/...java/com/netflix/hollow/api/codegen/delegate/HollowObjectDelegateCachedImplGenerator.java
@@ -120,7 +120,7 @@ public String generate() {
                 if(shortcut != null) {
                     String ordinalVariableName = fieldName + "TempOrdinal";
 
-                    builder.append("        int ").append(ordinalVariableName).append(" = typeAPI.get").append(uppercase(fieldName)).append("Ordinal(ordinal);\n");
+                    builder.append("        int ").append(ordinalVariableName).append(" = ").append(fieldName).append("Ordinal;\n");
 
                     for(int j=0;j<shortcut.getPath().length-1;j++) {
                         String typeAPIName = HollowCodeGenerationUtils.typeAPIClassname(shortcut.getPathTypes()[j]);