From 924e5e3e3ad0dcee292feb6047dbd2e9c120646b Mon Sep 17 00:00:00 2001 From: nickfelt Date: Wed, 26 Apr 2017 16:26:15 -0700 Subject: [PATCH] Clean up one usage of ForeignKeyIndex.mapToFkiClass() Followup to [] ------------- Created by MOE: https://github.com/google/moe MOE_MIGRATED_REVID=154361061 --- .../registry/batch/DeleteProberDataAction.java | 2 +- .../google/registry/model/index/ForeignKeyIndex.java | 12 ++++++------ 2 files changed, 7 insertions(+), 7 deletions(-) diff --git a/java/google/registry/batch/DeleteProberDataAction.java b/java/google/registry/batch/DeleteProberDataAction.java index 026a56c0f..32626e01d 100644 --- a/java/google/registry/batch/DeleteProberDataAction.java +++ b/java/google/registry/batch/DeleteProberDataAction.java @@ -136,7 +136,7 @@ public class DeleteProberDataAction implements Runnable { } final Key eppIndex = Key.create(EppResourceIndex.create(domainKey)); - final Key> fki = ForeignKeyIndex.createKey(domain); + final Key> fki = ForeignKeyIndex.createKey(domain); int entitiesDeleted = ofy().transact(new Work() { @Override diff --git a/java/google/registry/model/index/ForeignKeyIndex.java b/java/google/registry/model/index/ForeignKeyIndex.java index 1b54ecce2..515ea284e 100644 --- a/java/google/registry/model/index/ForeignKeyIndex.java +++ b/java/google/registry/model/index/ForeignKeyIndex.java @@ -113,9 +113,8 @@ public abstract class ForeignKeyIndex extends BackupGroup public static ForeignKeyIndex create( E resource, DateTime deletionTime) { @SuppressWarnings("unchecked") - Class> fkiClass = - ForeignKeyIndex.mapToFkiClass((Class) resource.getClass()); - ForeignKeyIndex instance = instantiate(fkiClass); + Class resourceClass = (Class) resource.getClass(); + ForeignKeyIndex instance = instantiate(mapToFkiClass(resourceClass)); instance.topReference = Key.create(resource); instance.foreignKey = resource.getForeignKey(); instance.deletionTime = deletionTime; @@ -123,9 +122,10 @@ public abstract class ForeignKeyIndex extends BackupGroup } /** Create a {@link ForeignKeyIndex} key for a resource. */ - public static Key> createKey(EppResource resource) { - return Key.>create( - mapToFkiClass(resource.getClass()), resource.getForeignKey()); + public static Key> createKey(E resource) { + @SuppressWarnings("unchecked") + Class resourceClass = (Class) resource.getClass(); + return Key.>create(mapToFkiClass(resourceClass), resource.getForeignKey()); } /**