zydronium/google-nomulus
1
0
Fork 0
mirror of https://github.com/google/nomulus.git synced 2025-05-14 16:37:13 +02:00
Code Issues Projects Releases Packages Wiki Activity

Rename loadByForeignKeys to loadByForeignKeysCached

Browse source 
This is to remind the user that the function actually uses cache, and also
for naming consistency with EppResourceUtils.loadByForeignKeyCached().

-------------
Created by MOE: https://github.com/google/moe
MOE_MIGRATED_REVID=232870846
This commit is contained in:
shicong 2019-02-07 07:28:14 -08:00 • committed by Gus Brodman
parent 93ff541006
commit c2445c3797
1 changed files with 10 additions and 9 deletions
Download patch file Download diff file Expand all files Collapse all files

13
java/google/registry/model/domain/DomainCommand.java
View file

@ -388,10 +388,11 @@ public class DomainCommand {
/** Creates a copy of this {@link Change} with hard links to hosts and contacts. */ /** Creates a copy of this {@link Change} with hard links to hosts and contacts. */
Change cloneAndLinkReferences(DateTime now) throws InvalidReferencesException { Change cloneAndLinkReferences(DateTime now) throws InvalidReferencesException {
Change clone = clone(this); Change clone = clone(this);
clone.registrant = Strings.isNullOrEmpty(clone.registrantContactId) clone.registrant =
Strings.isNullOrEmpty(clone.registrantContactId)
? null ? null
: getOnlyElement( : getOnlyElement(
loadByForeignKeys( loadByForeignKeysCached(
ImmutableSet.of(clone.registrantContactId), ContactResource.class, now) ImmutableSet.of(clone.registrantContactId), ContactResource.class, now)
.values()); .values());
return clone; return clone;
@ -420,7 +421,7 @@ public class DomainCommand {
return null; return null;
} }
return ImmutableSet.copyOf( return ImmutableSet.copyOf(
loadByForeignKeys(fullyQualifiedHostNames, HostResource.class, now).values()); loadByForeignKeysCached(fullyQualifiedHostNames, HostResource.class, now).values());
} }
private static Set<DesignatedContact> linkContacts( private static Set<DesignatedContact> linkContacts(
@ -433,7 +434,7 @@ public class DomainCommand {
foreignKeys.add(contact.contactId); foreignKeys.add(contact.contactId);
} }
ImmutableMap<String, Key<ContactResource>> loadedContacts = ImmutableMap<String, Key<ContactResource>> loadedContacts =
loadByForeignKeys(foreignKeys.build(), ContactResource.class, now); loadByForeignKeysCached(foreignKeys.build(), ContactResource.class, now);
ImmutableSet.Builder<DesignatedContact> linkedContacts = new ImmutableSet.Builder<>(); ImmutableSet.Builder<DesignatedContact> linkedContacts = new ImmutableSet.Builder<>();
for (ForeignKeyedDesignatedContact contact : contacts) { for (ForeignKeyedDesignatedContact contact : contacts) {
linkedContacts.add(DesignatedContact.create( linkedContacts.add(DesignatedContact.create(
@ -442,8 +443,8 @@ public class DomainCommand {
return linkedContacts.build(); return linkedContacts.build();
} }
/** Loads keys to EPP resources by their foreign keys. */ /** Loads keys to cached EPP resources by their foreign keys. */
private static <T extends EppResource> ImmutableMap<String, Key<T>> loadByForeignKeys( private static <T extends EppResource> ImmutableMap<String, Key<T>> loadByForeignKeysCached(
final Set<String> foreignKeys, final Class<T> clazz, final DateTime now) final Set<String> foreignKeys, final Class<T> clazz, final DateTime now)
throws InvalidReferencesException { throws InvalidReferencesException {
Map<String, ForeignKeyIndex<T>> fkis = ForeignKeyIndex.loadCached(clazz, foreignKeys, now); Map<String, ForeignKeyIndex<T>> fkis = ForeignKeyIndex.loadCached(clazz, foreignKeys, now);