- * Additionally provides encapsulation so that clients of BigqueryConnection don't need to take + * + *
Additionally provides encapsulation so that clients of BigqueryConnection don't need to take * any direct dependencies on Bigquery API classes and can instead use DestinationTable. */ public static class DestinationTable { @@ -365,8 +365,8 @@ public class BigqueryConnection implements AutoCloseable { * A function that updates the specified Bigquery table to reflect the metadata from the input * DestinationTable, passing the same DestinationTable through as the output. If the specified * table does not already exist, it will be inserted into the dataset. - *
- * Clients can call this function directly to update a table on demand, or can pass it to + * + *
Clients can call this function directly to update a table on demand, or can pass it to * Futures.transform() to update a table produced as the asynchronous result of a load or query * job (e.g. to add a description to it). */ @@ -441,8 +441,8 @@ public class BigqueryConnection implements AutoCloseable { * ImmutableTable object, row-keyed by the row number (indexed from 1), column-keyed by the * TableFieldSchema for that column, and with the value object as the cell value. Note that null * values will not actually be null, but they can be checked for using Data.isNull(). - *
- * Returns a ListenableFuture that holds the ImmutableTable on success. + * + *
Returns a ListenableFuture that holds the ImmutableTable on success.
*/
public ListenableFuture
- * This table is fully materialized in memory (not lazily loaded), so it should not be used with
- * queries expected to return large results.
+ *
+ * This table is fully materialized in memory (not lazily loaded), so it should not be used
+ * with queries expected to return large results.
*/
private ImmutableTable
- * Returns a ListenableFuture that holds the destination GCS URI on success.
+ *
+ * Returns a ListenableFuture that holds the destination GCS URI on success.
*/
public ListenableFuture
- * The general format definition is "YYYY-MM-DD HH:MM:SS.SSS[ ZZ]", where the fractional seconds
- * portion can have 0-6 decimal places (although we restrict it to 0-3 here since Joda DateTime
- * only supports up to millisecond precision) and the zone if not specified defaults to UTC.
- *
- * Although we expect a zone specification of "UTC" when parsing, we don't emit it when printing
- * because in some cases BigQuery does not allow any time zone specification (instead it assumes
- * UTC for whatever input you provide) for input timestamp strings (see b/16380363).
+ *
+ * The general format definition is "YYYY-MM-DD HH:MM:SS.SSS[ ZZ]", where the fractional
+ * seconds portion can have 0-6 decimal places (although we restrict it to 0-3 here since Joda
+ * DateTime only supports up to millisecond precision) and the zone if not specified defaults to
+ * UTC.
+ *
+ * Although we expect a zone specification of "UTC" when parsing, we don't emit it when
+ * printing because in some cases BigQuery does not allow any time zone specification (instead it
+ * assumes UTC for whatever input you provide) for input timestamp strings (see b/16380363).
*
* @see "https://developers.google.com/bigquery/timestamp"
*/
diff --git a/java/google/registry/flows/EppTlsServlet.java b/java/google/registry/flows/EppTlsServlet.java
index 5e7b2c7c5..565eea836 100644
--- a/java/google/registry/flows/EppTlsServlet.java
+++ b/java/google/registry/flows/EppTlsServlet.java
@@ -28,8 +28,8 @@ import javax.servlet.http.HttpServletResponse;
/**
* The {@link EppTlsServlet} class establishes a transport for EPP+TLS over* HTTP. All commands and
* responses are EPP XML according to RFC 5730. Commands must must requested via POST.
- *
- * There are a number of expected headers to this endpoint:
+ *
+ * There are a number of expected headers to this endpoint:
*
- * This class also contains static methods for loading an appropriate flow based on model classes.
+ *
+ * This class also contains static methods for loading an appropriate flow based on model
+ * classes.
*/
public abstract class Flow {
diff --git a/java/google/registry/flows/FlowRegistry.java b/java/google/registry/flows/FlowRegistry.java
index b837cfe5a..350abd301 100644
--- a/java/google/registry/flows/FlowRegistry.java
+++ b/java/google/registry/flows/FlowRegistry.java
@@ -151,9 +151,9 @@ public class FlowRegistry {
/**
* The domain restore command is technically a domain {@literal
- * This provider must be tried before {@link #RESOURCE_CRUD_FLOW_PROVIDER}. Otherwise, the regular
- * domain update flow will match first.
+ *
+ * This provider must be tried before {@link #RESOURCE_CRUD_FLOW_PROVIDER}. Otherwise, the
+ * regular domain update flow will match first.
*/
private static final FlowProvider DOMAIN_RESTORE_FLOW_PROVIDER = new FlowProvider() {
@Override
diff --git a/java/google/registry/flows/FlowRunner.java b/java/google/registry/flows/FlowRunner.java
index d3303fbf2..344c0ae7a 100644
--- a/java/google/registry/flows/FlowRunner.java
+++ b/java/google/registry/flows/FlowRunner.java
@@ -144,18 +144,18 @@ public class FlowRunner {
/**
* Helper for logging in json format.
- *
- * This is needed because the usual json outputters perform normalizations that we don't want or
- * need, since we know that our values never need to be escaped - there are only strings and
+ *
+ * This is needed because the usual json outputters perform normalizations that we don't want
+ * or need, since we know that our values never need to be escaped - there are only strings and
* numbers, and the strings are not allowed to contain quote characters.
- *
- * An example output for an EPP_Mutation:
- * {"trid":"abc-123", "client":"some_registrar", "tld":"com", "xmlBytes":"abc123DEF"}
- *
- * An example output for an EPP_Mutation_Committed that doesn't create a new resource:
+ *
+ * An example output for an EPP_Mutation: {"trid":"abc-123", "client":"some_registrar",
+ * "tld":"com", "xmlBytes":"abc123DEF"}
+ *
+ * An example output for an EPP_Mutation_Committed that doesn't create a new resource:
* {"trid":"abc-123", "executionTime":123456789}
- *
- * An example output for an EPP_Mutation_Committed that creates a new resource:
+ *
+ * An example output for an EPP_Mutation_Committed that creates a new resource:
* {"trid":"abc-123", "executionRepoId":123, "executionTime":123456789}
*/
private static class JsonLogStatement {
diff --git a/java/google/registry/flows/ResourceFlowUtils.java b/java/google/registry/flows/ResourceFlowUtils.java
index 5a47cd771..c165bc267 100644
--- a/java/google/registry/flows/ResourceFlowUtils.java
+++ b/java/google/registry/flows/ResourceFlowUtils.java
@@ -86,9 +86,9 @@ public class ResourceFlowUtils {
/**
* Create a pending action notification response indicating the resolution of a transfer.
- *
- * The returned object will use the id and type of this resource, the trid of the resource's last
- * transfer request, and the specified status and date.
+ *
+ * The returned object will use the id and type of this resource, the trid of the resource's
+ * last transfer request, and the specified status and date.
*/
public static PendingActionNotificationResponse createPendingTransferNotificationResponse(
EppResource eppResource,
diff --git a/java/google/registry/flows/TransactionalFlow.java b/java/google/registry/flows/TransactionalFlow.java
index a2ea7f85c..cbf9846c7 100644
--- a/java/google/registry/flows/TransactionalFlow.java
+++ b/java/google/registry/flows/TransactionalFlow.java
@@ -16,8 +16,8 @@ package google.registry.flows;
/**
* Marker interface indicating that a {@link Flow} needs to be run transactionally.
- *
- * Any flow that mutates the datastore should be tagged with this so that {@link FlowRunner} will
+ *
+ * Any flow that mutates the datastore should be tagged with this so that {@link FlowRunner} will
* know how to run it.
*/
public interface TransactionalFlow {}
diff --git a/java/google/registry/flows/domain/BaseDomainCreateFlow.java b/java/google/registry/flows/domain/BaseDomainCreateFlow.java
index 2eac62cc8..2880a6be1 100644
--- a/java/google/registry/flows/domain/BaseDomainCreateFlow.java
+++ b/java/google/registry/flows/domain/BaseDomainCreateFlow.java
@@ -142,10 +142,10 @@ public abstract class BaseDomainCreateFlow Try to load the domain non-transactionally, since this can hit memcache. If we succeed, and
+ * the domain is not in the ADD grace period (the only state that allows instantaneous transition
+ * to being deleted), we can assume that the domain will not be deleted (and therefore won't be
* creatable) until its deletion time. For repeated failed creates this means we can avoid the
* datastore lookup, which is very expensive (and first-seen failed creates are no worse than they
* otherwise would be). This comes at the cost of the extra lookup for successful creates (or
diff --git a/java/google/registry/flows/domain/DomainTransferApproveFlow.java b/java/google/registry/flows/domain/DomainTransferApproveFlow.java
index 1204115f0..255f4b6a3 100644
--- a/java/google/registry/flows/domain/DomainTransferApproveFlow.java
+++ b/java/google/registry/flows/domain/DomainTransferApproveFlow.java
@@ -47,8 +47,8 @@ import org.joda.time.DateTime;
/**
* An EPP flow that approves a pending transfer on a {@link DomainResource}.
- *
- * The logic in this flow, which handles client approvals, very closely parallels the logic in
+ *
+ * The logic in this flow, which handles client approvals, very closely parallels the logic in
* {@link DomainResource#cloneProjectedAtTime} which handles implicit server approvals.
*
* @error {@link google.registry.flows.domain.DomainFlowUtils.NotAuthorizedForTldException}
diff --git a/java/google/registry/flows/domain/DomainTransferRequestFlow.java b/java/google/registry/flows/domain/DomainTransferRequestFlow.java
index ccd8967c2..0c2dad803 100644
--- a/java/google/registry/flows/domain/DomainTransferRequestFlow.java
+++ b/java/google/registry/flows/domain/DomainTransferRequestFlow.java
@@ -184,10 +184,10 @@ public class DomainTransferRequestFlow
/**
* When a transfer is requested, schedule a billing event and poll message for the automatic
* approval case.
- *
- * Note that the action time is AUTOMATIC_TRANSFER_DAYS in the future, matching the server policy
- * on automated approval of transfers. There is no equivalent grace period added; if the transfer
- * is implicitly approved, the resource will project a grace period on itself.
+ *
+ * Note that the action time is AUTOMATIC_TRANSFER_DAYS in the future, matching the server
+ * policy on automated approval of transfers. There is no equivalent grace period added; if the
+ * transfer is implicitly approved, the resource will project a grace period on itself.
*/
@Override
protected Set
- * By adding a subclass of this to every package in the model, we can write generic code that can
+ *
+ * By adding a subclass of this to every package in the model, we can write generic code that can
* access fields with package private access. The other alternative is to call
* {@link Field#setAccessible} with {@code true} on any such Field objects, but that does not work
* reliably in Google App Engine cross-package because of its custom security manager
diff --git a/java/google/registry/model/Buildable.java b/java/google/registry/model/Buildable.java
index 8d6d7e572..90c814e50 100644
--- a/java/google/registry/model/Buildable.java
+++ b/java/google/registry/model/Buildable.java
@@ -32,8 +32,8 @@ public interface Buildable {
/**
* Boilerplate for immutable builders.
- *
- * This can be used without implementing {@link Buildable}.
+ *
+ * This can be used without implementing {@link Buildable}.
*/
public abstract static class Builder
- * Warning: Do not use {@code emptyToNull} methods in the getters of an {@link Overlayable}! We
+ *
+ * Warning: Do not use {@code emptyToNull} methods in the getters of an {@link Overlayable}! We
* use null to mean "skip this field" whereas empty means "set this field to empty", so they are
* semantically different.
*
diff --git a/java/google/registry/model/EppResourceUtils.java b/java/google/registry/model/EppResourceUtils.java
index 5f7b8774e..daf8b70a2 100644
--- a/java/google/registry/model/EppResourceUtils.java
+++ b/java/google/registry/model/EppResourceUtils.java
@@ -139,9 +139,9 @@ public final class EppResourceUtils {
/**
* Checks multiple {@link EppResource} objects from the datastore by unique ids.
- *
- * There are currently no resources that support checks and do not use foreign keys. If we need to
- * support that case in the future, we can loosen the type to allow any {@link EppResource} and
+ *
+ * There are currently no resources that support checks and do not use foreign keys. If we need
+ * to support that case in the future, we can loosen the type to allow any {@link EppResource} and
* add code to do the lookup by id directly.
*
* @param clazz the resource type to load
diff --git a/java/google/registry/model/ModelUtils.java b/java/google/registry/model/ModelUtils.java
index 2b724cec3..3926eb570 100644
--- a/java/google/registry/model/ModelUtils.java
+++ b/java/google/registry/model/ModelUtils.java
@@ -213,10 +213,10 @@ public class ModelUtils {
/**
* Returns a map from field names (including non-public and inherited fields) to values.
- *
- * This turns arrays into {@link List} objects so that ImmutableObject can more easily use the
- * returned map in its implementation of {@link ImmutableObject#toString} and
- * {@link ImmutableObject#equals}, which work by comparing and printing these maps.
+ *
+ * This turns arrays into {@link List} objects so that ImmutableObject can more easily use the
+ * returned map in its implementation of {@link ImmutableObject#toString} and {@link
+ * ImmutableObject#equals}, which work by comparing and printing these maps.
*/
static Map
- * Unlike {@link OneTime} events, these do not store an explicit cost, since the cost of the
+ *
+ * Unlike {@link OneTime} events, these do not store an explicit cost, since the cost of the
* recurring event might change and each time we bill for it we need to bill at the current cost,
* not the value that was in use at the time the recurrence was created.
*/
@@ -346,15 +346,15 @@ public abstract class BillingEvent extends ImmutableObject
/**
* The eventTime recurs every year on this [month, day, time] between {@link #eventTime} and
* {@link #recurrenceEndTime}, inclusive of the start but not of the end.
- *
- * This field is denormalized from {@link #eventTime} to allow for an efficient index, but it
+ *
+ * This field is denormalized from {@link #eventTime} to allow for an efficient index, but it
* always has the same data as that field.
- *
- * Note that this is a recurrence of the event time, not the billing time. The billing time can
- * be calculated by adding the relevant grace period length to this date. The reason for this
- * requirement is that the event time recurs on a {@link org.joda.time.Period} schedule (same
- * day of year, which can be 365 or 366 days later) which is what {@link TimeOfYear} can model,
- * whereas the billing time is a fixed {@link org.joda.time.Duration} later.
+ *
+ * Note that this is a recurrence of the event time, not the billing time. The billing time
+ * can be calculated by adding the relevant grace period length to this date. The reason for
+ * this requirement is that the event time recurs on a {@link org.joda.time.Period} schedule
+ * (same day of year, which can be 365 or 366 days later) which is what {@link TimeOfYear} can
+ * model, whereas the billing time is a fixed {@link org.joda.time.Duration} later.
*/
@Index
TimeOfYear recurrenceTimeOfYear;
@@ -401,9 +401,9 @@ public abstract class BillingEvent extends ImmutableObject
/**
* An event representing a cancellation of one of the other two billable event types.
- *
- * This is implemented as a separate event rather than a bit on BillingEvent in order to preserve
- * the immutability of billing events.
+ *
+ * This is implemented as a separate event rather than a bit on BillingEvent in order to
+ * preserve the immutability of billing events.
*/
@Entity
public static class Cancellation extends BillingEvent {
diff --git a/java/google/registry/model/common/TimeOfYear.java b/java/google/registry/model/common/TimeOfYear.java
index 9d747c9f9..e4fe12abd 100644
--- a/java/google/registry/model/common/TimeOfYear.java
+++ b/java/google/registry/model/common/TimeOfYear.java
@@ -32,14 +32,14 @@ import java.util.List;
/**
* A time of year (month, day, millis of day) that can be stored in a sort-friendly format.
- *
- * This is conceptually similar to {@code MonthDay} in Joda or more generally to Joda's
+ *
+ * This is conceptually similar to {@code MonthDay} in Joda or more generally to Joda's
* {@code Partial}, but the parts we need are too simple to justify a full implementation of
* {@code Partial}.
- *
- * For simplicity, the native representation of this class's data is its stored format. This allows
- * it to be embeddable with no translation needed and also delays parsing of the string on load
- * until it's actually needed.
+ *
+ * For simplicity, the native representation of this class's data is its stored format. This
+ * allows it to be embeddable with no translation needed and also delays parsing of the string on
+ * load until it's actually needed.
*/
@Embed
public class TimeOfYear extends ImmutableObject {
@@ -53,8 +53,8 @@ public class TimeOfYear extends ImmutableObject {
/**
* Constructs a {@link TimeOfYear} from a {@link DateTime}.
- *
- * This handles leap years in an intentionally peculiar way by always treating February 29 as
+ *
+ * This handles leap years in an intentionally peculiar way by always treating February 29 as
* February 28. It is impossible to construct a {@link TimeOfYear} for February 29th.
*/
public static TimeOfYear fromDateTime(DateTime dateTime) {
diff --git a/java/google/registry/model/common/TimedTransitionProperty.java b/java/google/registry/model/common/TimedTransitionProperty.java
index 578c66bf3..b1eef1dfc 100644
--- a/java/google/registry/model/common/TimedTransitionProperty.java
+++ b/java/google/registry/model/common/TimedTransitionProperty.java
@@ -40,17 +40,17 @@ import java.util.TreeMap;
* at a corresponding instant, and remains active until the next transition occurs. At least one
* "start of time" value (corresponding to START_OF_TIME, i.e. the Unix epoch) must be provided
* so that the property will have a value for all possible times.
- *
- * This concept is naturally represented by a sorted map of {@code DateTime} to {@code V}, but
- * the AppEngine datastore cannot natively represent a map keyed on non-strings. Instead, we
- * store an ordered list of transitions and use Objectify's @Mapify annotation to automatically
- * recreate the sorted map on load from the datastore, which is used as a backing map for this
- * property; the property itself also implements Map by way of extending ForwardingMap, so that
- * this property can stored directly as the @Mapify field in the entity.
- *
- * The type parameter {@code T} specifies a user-defined subclass of {@code TimedTransition This concept is naturally represented by a sorted map of {@code DateTime} to {@code V}, but
+ * the AppEngine datastore cannot natively represent a map keyed on non-strings. Instead, we store
+ * an ordered list of transitions and use Objectify's @Mapify annotation to automatically recreate
+ * the sorted map on load from the datastore, which is used as a backing map for this property; the
+ * property itself also implements Map by way of extending ForwardingMap, so that this property can
+ * stored directly as the @Mapify field in the entity.
+ *
+ * The type parameter {@code T} specifies a user-defined subclass of {@code TimedTransition The public visibility of this class exists only so that it can be subclassed; clients should
+ * never call any methods on this class or attempt to access its members, but should instead treat
+ * it as a customizable implementation detail of {@code TimedTransitionProperty}. However, note
+ * that subclasses must also have public visibility so that they can be instantiated via
* reflection in a call to {@code fromValueMap}.
*/
public abstract static class TimedTransition This method should be the normal method for constructing a {@link TimedTransitionProperty}.
*/
public static This method should only be used for initializing fields that are declared with the
+ * @Mapify annotation. The map for those fields must be mutable so that Objectify can load values
+ * from the datastore into the map, but clients should still never mutate the field's map
+ * directly.
*/
public static This method should only be used for initializing fields that are declared with the
+ * @Mapify annotation. The map for those fields must be mutable so that Objectify can load values
+ * from the datastore into the map, but clients should still never mutate the field's map
+ * directly.
*/
public static
- * This class is embedded inside the {@link PostalInfo} of an EPP contact to hold its address. The
- * fields are all defined in parent class {@link Address}, but the subclass is still necessary to
- * pick up the contact namespace.
- *
- * This does not implement {@code Overlayable} because it is intended to be bulk replaced on update.
+ *
+ * This class is embedded inside the {@link PostalInfo} of an EPP contact to hold its
+ * address. The fields are all defined in parent class {@link Address}, but the subclass is still
+ * necessary to pick up the contact namespace.
+ *
+ * This does not implement {@code Overlayable} because it is intended to be bulk replaced on
+ * update.
*
* @see PostalInfo
*/
diff --git a/java/google/registry/model/contact/ContactCommand.java b/java/google/registry/model/contact/ContactCommand.java
index a52f2cc92..b33c5a51e 100644
--- a/java/google/registry/model/contact/ContactCommand.java
+++ b/java/google/registry/model/contact/ContactCommand.java
@@ -120,10 +120,10 @@ public class ContactCommand {
implements SingleResourceCommand, ResourceCreateOrChange
- * This is only unique in the sense that for any given lifetime specified as the time range from
- * (creationTime, deletionTime) there can only be one contact in the datastore with this id.
- * However, there can be many contacts with the same id and non-overlapping lifetimes.
+ *
+ * This is only unique in the sense that for any given lifetime specified as the time range
+ * from (creationTime, deletionTime) there can only be one contact in the datastore with this
+ * id. However, there can be many contacts with the same id and non-overlapping lifetimes.
*/
@XmlElement(name = "id")
String contactId;
diff --git a/java/google/registry/model/contact/ContactPhoneNumber.java b/java/google/registry/model/contact/ContactPhoneNumber.java
index 8a7dfbc89..abee7564d 100644
--- a/java/google/registry/model/contact/ContactPhoneNumber.java
+++ b/java/google/registry/model/contact/ContactPhoneNumber.java
@@ -20,10 +20,10 @@ import google.registry.model.eppcommon.PhoneNumber;
/**
* EPP Contact Phone Number
- *
- * This class is embedded inside a {@link ContactResource} hold the phone number of an EPP contact.
- * The fields are all defined in the parent class {@link PhoneNumber}, but the subclass is still
- * necessary to pick up the contact namespace.
+ *
+ * This class is embedded inside a {@link ContactResource} hold the phone number of an EPP
+ * contact. The fields are all defined in the parent class {@link PhoneNumber}, but the subclass is
+ * still necessary to pick up the contact namespace.
*
* @see ContactResource
*/
diff --git a/java/google/registry/model/contact/ContactResource.java b/java/google/registry/model/contact/ContactResource.java
index 9d385819d..94653ebe1 100644
--- a/java/google/registry/model/contact/ContactResource.java
+++ b/java/google/registry/model/contact/ContactResource.java
@@ -69,9 +69,9 @@ public class ContactResource extends EppResource implements ForeignKeyedEppResou
/**
* Unique identifier for this contact.
- *
- * This is only unique in the sense that for any given lifetime specified as the time range from
- * (creationTime, deletionTime) there can only be one contact in the datastore with this id.
+ *
+ * This is only unique in the sense that for any given lifetime specified as the time range
+ * from (creationTime, deletionTime) there can only be one contact in the datastore with this id.
* However, there can be many contacts with the same id and non-overlapping lifetimes.
*/
@XmlTransient
@@ -153,8 +153,8 @@ public class ContactResource extends EppResource implements ForeignKeyedEppResou
/**
* Postal info for the contact.
- *
- * The XML marshalling expects the {@link PostalInfo} objects in a list, but we can't actually
+ *
+ * The XML marshalling expects the {@link PostalInfo} objects in a list, but we can't actually
* persist them to datastore that way because Objectify can't handle collections of embedded
* objects that themselves contain collections, and there's a list of streets inside. This method
* transforms the persisted format to the XML format for marshalling.
diff --git a/java/google/registry/model/domain/DomainBase.java b/java/google/registry/model/domain/DomainBase.java
index 6cb05b519..0ceea074f 100644
--- a/java/google/registry/model/domain/DomainBase.java
+++ b/java/google/registry/model/domain/DomainBase.java
@@ -59,9 +59,9 @@ public abstract class DomainBase extends EppResource {
/**
* Fully qualified domain name (puny-coded), which serves as the foreign key for this domain.
- *
- * This is only unique in the sense that for any given lifetime specified as the time range from
- * (creationTime, deletionTime) there can only be one domain in the datastore with this name.
+ *
+ * This is only unique in the sense that for any given lifetime specified as the time range
+ * from (creationTime, deletionTime) there can only be one domain in the datastore with this name.
* However, there can be many domains with the same name and non-overlapping lifetimes.
*
* @invariant fullyQualifiedDomainName == fullyQualifiedDomainName.toLowerCase()
@@ -83,8 +83,8 @@ public abstract class DomainBase extends EppResource {
/**
* Associated contacts for the domain (other than registrant).
- *
- * This field is marked with {@literal @}Ignore so that {@link DomainBase} subclasses won't
+ *
+ * This field is marked with {@literal @}Ignore so that {@link DomainBase} subclasses won't
* persist it. Instead, the data in this field and in the {@link #registrant} are both stored in
* {@link DomainBase#allContacts} to allow for more efficient queries.
*/
@@ -101,10 +101,10 @@ public abstract class DomainBase extends EppResource {
/**
* A reference to the registrant who registered this domain.
- *
- * This field is marked with {@literal @}Ignore so that {@link DomainBase} subclasses won't
- * persist it. Instead, the data in this field and in the {@link DomainBase#contacts} are
- * both stored in {@link DomainBase#allContacts} to allow for more efficient queries.
+ *
+ * This field is marked with {@literal @}Ignore so that {@link DomainBase} subclasses won't
+ * persist it. Instead, the data in this field and in the {@link DomainBase#contacts} are both
+ * stored in {@link DomainBase#allContacts} to allow for more efficient queries.
*/
@Ignore
//TODO(b/28713909): Make this a Ref
- * This is {@literal @}XmlTransient because it needs to be returned under the "extension" tag
+ *
+ * This is {@literal @}XmlTransient because it needs to be returned under the "extension" tag
* of an info response rather than inside the "infData" tag.
*/
@XmlTransient
diff --git a/java/google/registry/model/domain/DomainCommand.java b/java/google/registry/model/domain/DomainCommand.java
index f778527c5..2c0811f67 100644
--- a/java/google/registry/model/domain/DomainCommand.java
+++ b/java/google/registry/model/domain/DomainCommand.java
@@ -445,9 +445,9 @@ public class DomainCommand {
/**
* Creates a copy of this {@link Update} with hard links to hosts and contacts.
- *
- * As a side effect, this will turn null innerAdd/innerRemove/innerChange into empty versions of
- * those classes, which is harmless because the getters do that anyways.
+ *
+ * As a side effect, this will turn null innerAdd/innerRemove/innerChange into empty versions
+ * of those classes, which is harmless because the getters do that anyways.
*/
@Override
public Update cloneAndLinkReferences(DateTime now) throws InvalidReferencesException {
diff --git a/java/google/registry/model/domain/DomainResource.java b/java/google/registry/model/domain/DomainResource.java
index a6de03c43..6c9d2b2c1 100644
--- a/java/google/registry/model/domain/DomainResource.java
+++ b/java/google/registry/model/domain/DomainResource.java
@@ -100,8 +100,8 @@ public class DomainResource extends DomainBase implements ForeignKeyedEppResourc
/**
* The poll message associated with this domain being deleted.
- *
- * This field should be null if the domain is not in pending delete. If it is, the field should
+ *
+ * This field should be null if the domain is not in pending delete. If it is, the field should
* refer to a {@link PollMessage} timed to when the domain is fully deleted. If the domain is
* restored, the message should be deleted.
*/
@@ -110,9 +110,9 @@ public class DomainResource extends DomainBase implements ForeignKeyedEppResourc
/**
* The recurring billing event associated with this domain's autorenewals.
- *
- * The recurrence should be open ended unless the domain is in pending delete or fully deleted, in
- * which case it should be closed at the time the delete was requested. Whenever the domain's
+ *
+ * The recurrence should be open ended unless the domain is in pending delete or fully deleted,
+ * in which case it should be closed at the time the delete was requested. Whenever the domain's
* {@link #registrationExpirationTime} is changed the recurrence should be closed, a new one
* should be created, and this field should be updated to point to the new one.
*/
@@ -121,9 +121,9 @@ public class DomainResource extends DomainBase implements ForeignKeyedEppResourc
/**
* The recurring poll message associated with this domain's autorenewals.
- *
- * The recurrence should be open ended unless the domain is in pending delete or fully deleted, in
- * which case it should be closed at the time the delete was requested. Whenever the domain's
+ *
+ * The recurrence should be open ended unless the domain is in pending delete or fully deleted,
+ * in which case it should be closed at the time the delete was requested. Whenever the domain's
* {@link #registrationExpirationTime} is changed the recurrence should be closed, a new one
* should be created, and this field should be updated to point to the new one.
*/
@@ -206,8 +206,8 @@ public class DomainResource extends DomainBase implements ForeignKeyedEppResourc
/**
* Returns the Registry Grace Period Statuses for this domain.
- *
- * This collects all statuses from the domain's {@link GracePeriod}s and also adds the
+ *
+ * This collects all statuses from the domain's {@link GracePeriod} entries and also adds the
* PENDING_DELETE status if needed.
*/
public ImmutableSet
- * When a grace period expires, it is lazily removed from the {@link DomainResource} the next time
- * the resource is loaded from the datastore.
+ *
+ * When a grace period expires, it is lazily removed from the {@link DomainResource} the next
+ * time the resource is loaded from the datastore.
*/
@Embed
public class GracePeriod extends ImmutableObject {
diff --git a/java/google/registry/model/domain/ReferenceUnion.java b/java/google/registry/model/domain/ReferenceUnion.java
index 336489d1d..1add0fd25 100644
--- a/java/google/registry/model/domain/ReferenceUnion.java
+++ b/java/google/registry/model/domain/ReferenceUnion.java
@@ -28,8 +28,8 @@ import javax.xml.bind.annotation.adapters.XmlAdapter;
/**
* Legacy shell of a "union" type to represent referenced objects as either a foreign key or as a
* link to another object in the datastore. In its current form it merely wraps a {@link Ref}.
- *
- * This type always marshals as the "foreign key". We no longer use this type for unmarshalling.
+ *
+ * This type always marshals as the "foreign key". We no longer use this type for unmarshalling.
*
* @param
- * This object holds XML data which JAXB will unmarshal from an EPP domain create command extension.
- * The XML will have the following enclosing structure:
+ *
+ * This object holds XML data which JAXB will unmarshal from an EPP domain create command
+ * extension. The XML will have the following enclosing structure:
*
*
- * This is a list because a single operation can involve multiple fees.
+ *
+ * This is a list because a single operation can involve multiple fees.
*/
@XmlElement(name = "fee")
List
- * This is a list because a single operation can involve multiple fees.
+ *
+ * This is a list because a single operation can involve multiple fees.
*/
List
- * This is a list because a single operation can involve multiple fees.
+ *
+ * This is a list because a single operation can involve multiple fees.
*/
List
- * We will use "premium" for fees on premium names, and omit the field otherwise.
+ *
+ * We will use "premium" for fees on premium names, and omit the field otherwise.
*/
@XmlElement(name = "class")
String feeClass;
diff --git a/java/google/registry/model/domain/fee/FeeDeleteResponseExtension.java b/java/google/registry/model/domain/fee/FeeDeleteResponseExtension.java
index 072972532..84059a0e2 100644
--- a/java/google/registry/model/domain/fee/FeeDeleteResponseExtension.java
+++ b/java/google/registry/model/domain/fee/FeeDeleteResponseExtension.java
@@ -39,8 +39,8 @@ public class FeeDeleteResponseExtension extends ImmutableObject implements Respo
/**
* The magnitude of the credit(s), in the specified units, with an optional description.
- *
- * This is a list because a single delete can receive multiple credits.
+ *
+ * This is a list because a single delete can receive multiple credits.
*/
@XmlElement(name = "credit")
List
- * These values are never read from a command and only used in responses, so, we don't need to model
- * anything we don't output. We don't model the CUSTOM status because we don't use it. This allows
- * us to also avoid modeling the "name" attribute which is only used with CUSTOM. We don't model the
- * "lang" attribute because we only support English and that's the default.
- *
- * Given all of this, we can use {@link EnumToAttributeAdapter} to make this code very simple.
+ *
+ * These values are never read from a command and only used in responses, so, we don't need to
+ * model anything we don't output. We don't model the CUSTOM status because we don't use it. This
+ * allows us to also avoid modeling the "name" attribute which is only used with CUSTOM. We don't
+ * model the "lang" attribute because we only support English and that's the default.
+ *
+ * Given all of this, we can use {@link EnumToAttributeAdapter} to make this code very simple.
*
* @see "http://tools.ietf.org/html/draft-tan-epp-launchphase-11#section-2.3"
*/
diff --git a/java/google/registry/model/domain/launch/LaunchCheckExtension.java b/java/google/registry/model/domain/launch/LaunchCheckExtension.java
index 6ffe74abf..9e0289ee6 100644
--- a/java/google/registry/model/domain/launch/LaunchCheckExtension.java
+++ b/java/google/registry/model/domain/launch/LaunchCheckExtension.java
@@ -26,9 +26,9 @@ import javax.xml.bind.annotation.XmlRootElement;
/**
* An XML data object that represents a launch extension that may be present on EPP domain check
* commands.
- *
- * This object holds XML data which JAXB will unmarshal from an EPP domain check command extension.
- * The XML will have the following enclosing structure:
+ *
+ * This object holds XML data which JAXB will unmarshal from an EPP domain check command
+ * extension. The XML will have the following enclosing structure:
*
*
- * This object holds XML data which JAXB will unmarshal from an EPP domain create command extension.
- * The XML will have the following enclosing structure:
+ *
+ * This object holds XML data which JAXB will unmarshal from an EPP domain create command
+ * extension. The XML will have the following enclosing structure:
*
*
- * The launch phase refers to the various stages that a TLD goes through before entering general
+ *
+ * The launch phase refers to the various stages that a TLD goes through before entering general
* availability. The various phases are described below (in order that they usually occur).
*/
@Embed
diff --git a/java/google/registry/model/domain/secdns/SecDnsCreateExtension.java b/java/google/registry/model/domain/secdns/SecDnsCreateExtension.java
index 441943b85..e6ca812e5 100644
--- a/java/google/registry/model/domain/secdns/SecDnsCreateExtension.java
+++ b/java/google/registry/model/domain/secdns/SecDnsCreateExtension.java
@@ -32,8 +32,9 @@ import javax.xml.bind.annotation.XmlType;
public class SecDnsCreateExtension extends ImmutableObject implements CommandExtension {
/**
* Time in seconds until the signature should expire.
- *
- * We do not support expirations, but we need this field to be able to return appropriate errors.
+ *
+ * We do not support expirations, but we need this field to be able to return appropriate
+ * errors.
*/
Long maxSigLife;
diff --git a/java/google/registry/model/domain/secdns/SecDnsUpdateExtension.java b/java/google/registry/model/domain/secdns/SecDnsUpdateExtension.java
index a72e15c3a..cac8b66e2 100644
--- a/java/google/registry/model/domain/secdns/SecDnsUpdateExtension.java
+++ b/java/google/registry/model/domain/secdns/SecDnsUpdateExtension.java
@@ -36,8 +36,8 @@ public class SecDnsUpdateExtension extends ImmutableObject implements CommandExt
/**
* Specifies whether this update is urgent.
- *
- * We don't support urgent updates but we need this to be present to provide appropriate error
+ *
+ * We don't support urgent updates but we need this to be present to provide appropriate error
* messages if a client requests it.
*/
@XmlAttribute
@@ -98,8 +98,8 @@ public class SecDnsUpdateExtension extends ImmutableObject implements CommandExt
public static class Change extends ImmutableObject {
/**
* Time in seconds until the signature should expire.
- *
- * We do not support expirations, but we need this field to be able to return appropriate
+ *
+ * We do not support expirations, but we need this field to be able to return appropriate
* errors.
*/
Long maxSigLife;
diff --git a/java/google/registry/model/eppcommon/Address.java b/java/google/registry/model/eppcommon/Address.java
index 618c756e8..d924dfe28 100644
--- a/java/google/registry/model/eppcommon/Address.java
+++ b/java/google/registry/model/eppcommon/Address.java
@@ -36,8 +36,8 @@ import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
/**
* Container for generic street address.
- *
- * This is the "addrType" type from {@link "http://tools.ietf.org/html/rfc5733"}. It also matches
+ *
+ * This is the "addrType" type from {@link "http://tools.ietf.org/html/rfc5733"}. It also matches
* the "addrType" type from {@link "http://tools.ietf.org/html/draft-lozano-tmch-smd"}.
*
* @see google.registry.model.contact.ContactAddress
diff --git a/java/google/registry/model/eppcommon/AuthInfo.java b/java/google/registry/model/eppcommon/AuthInfo.java
index f8fd3a253..aefacaab4 100644
--- a/java/google/registry/model/eppcommon/AuthInfo.java
+++ b/java/google/registry/model/eppcommon/AuthInfo.java
@@ -29,8 +29,8 @@ import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
/**
* The "authInfoType" complex type.
- *
- * RFCs 5731 and 5732 define this almost identically up to the namespace.
+ *
+ * RFCs 5731 and 5732 define this almost identically up to the namespace.
*/
@XmlTransient
public abstract class AuthInfo extends ImmutableObject {
diff --git a/java/google/registry/model/eppcommon/PhoneNumber.java b/java/google/registry/model/eppcommon/PhoneNumber.java
index 5b75d9c2d..db36779ac 100644
--- a/java/google/registry/model/eppcommon/PhoneNumber.java
+++ b/java/google/registry/model/eppcommon/PhoneNumber.java
@@ -29,12 +29,12 @@ import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
/**
* Container for generic E164 phone number.
- *
- * This is the "e164" type from {@link "http://tools.ietf.org/html/rfc5733"}. It also matches the
+ *
+ * This is the "e164" type from {@link "http://tools.ietf.org/html/rfc5733"}. It also matches the
* "e164Type" type from {@link "http://tools.ietf.org/html/draft-lozano-tmch-smd"}.
*
- *
- * "Contact telephone number structure is derived from structures defined in [ITU.E164.2005].
+ * "Contact telephone number structure is derived from structures defined in [ITU.E164.2005].
* Telephone numbers described in this mapping are character strings that MUST begin with a plus
* sign ("+", ASCII value 0x002B), followed by a country code defined in [ITU.E164.2005], followed
* by a dot (".", ASCII value 0x002E), followed by a sequence of digits representing the telephone
diff --git a/java/google/registry/model/eppcommon/PresenceMarker.java b/java/google/registry/model/eppcommon/PresenceMarker.java
index a4a949fb1..00ba8ac28 100644
--- a/java/google/registry/model/eppcommon/PresenceMarker.java
+++ b/java/google/registry/model/eppcommon/PresenceMarker.java
@@ -24,8 +24,8 @@ import javax.xml.bind.annotation.XmlTransient;
/**
* Used as the value of a tag that is present in the XML but has no children or value.
- *
- * When placed in a field "foo", this will correctly unmarshal from both {@code When placed in a field "foo", this will correctly unmarshal from both {@code
- * This package is intended to hold classes which are shared across multiple XML namespaces. As
+ *
+ * This package is intended to hold classes which are shared across multiple XML namespaces. As
* such, no default namespace is declared in this package, and all objects in this package should be
* declared XmlTransient.
*/
diff --git a/java/google/registry/model/eppinput/ResourceCommand.java b/java/google/registry/model/eppinput/ResourceCommand.java
index 63fbf840a..c76c1e3ac 100644
--- a/java/google/registry/model/eppinput/ResourceCommand.java
+++ b/java/google/registry/model/eppinput/ResourceCommand.java
@@ -40,11 +40,11 @@ public interface ResourceCommand {
/**
* A command for a single {@link EppResource}.
- *
- * In general commands should extend {@link AbstractSingleResourceCommand} instead of
- * implementing this directly, but "Create" commands can't do that since they need to inherit
- * from a base class that gives them all of the resource's fields. The domain "Info" command
- * also can't do that since it's "name" field is overloaded with a "hosts" attribute.
+ *
+ * In general commands should extend {@link AbstractSingleResourceCommand} instead of
+ * implementing this directly, but "Create" commands can't do that since they need to inherit from
+ * a base class that gives them all of the resource's fields. The domain "Info" command also can't
+ * do that since it's "name" field is overloaded with a "hosts" attribute.
*/
public interface SingleResourceCommand extends ResourceCommand {
String getTargetId();
diff --git a/java/google/registry/model/eppoutput/CheckData.java b/java/google/registry/model/eppoutput/CheckData.java
index ba6debf0d..18069b62e 100644
--- a/java/google/registry/model/eppoutput/CheckData.java
+++ b/java/google/registry/model/eppoutput/CheckData.java
@@ -71,9 +71,9 @@ public abstract class CheckData extends ImmutableObject implements ResponseData
/**
* The "checkNameType" and "checkIDType" types.
- *
- * Although these are specified in the Epp extension RFCs and not in RFC 5730, which implies that
- * they should be implemented per-extension, all of RFCs 5731, 5732 and 5733 define them
+ *
+ * Although these are specified in the Epp extension RFCs and not in RFC 5730, which implies
+ * that they should be implemented per-extension, all of RFCs 5731, 5732 and 5733 define them
* identically except for the namespace and some slightly renamed fields, allowing us to share
* some code between the different extensions.
*/
diff --git a/java/google/registry/model/eppoutput/Greeting.java b/java/google/registry/model/eppoutput/Greeting.java
index fbc6fa18e..d32d43ccf 100644
--- a/java/google/registry/model/eppoutput/Greeting.java
+++ b/java/google/registry/model/eppoutput/Greeting.java
@@ -30,8 +30,9 @@ import javax.xml.bind.annotation.XmlElementWrapper;
/**
* A greeting, defined in {@link "http://tools.ietf.org/html/rfc5730"}.
- *
- * It would be nice to make this a singleton, but we need the {@link #svDate} field to stay current.
+ *
+ * It would be nice to make this a singleton, but we need the {@link #svDate} field to stay
+ * current.
*/
public class Greeting extends ImmutableObject implements ResponseOrGreeting {
diff --git a/java/google/registry/model/eppoutput/Response.java b/java/google/registry/model/eppoutput/Response.java
index 069727e6e..85ea8a4e8 100644
--- a/java/google/registry/model/eppoutput/Response.java
+++ b/java/google/registry/model/eppoutput/Response.java
@@ -79,16 +79,16 @@ public class Response extends ImmutableObject implements ResponseOrGreeting {
/**
* The time the command that created this response was executed.
- *
- * This is for logging purposes only and is not returned to the user.
+ *
+ * This is for logging purposes only and is not returned to the user.
*/
@XmlTransient
DateTime executionTime;
/**
* The repository id of a new object if this is a create response, or null.
- *
- * This is for logging purposes only and is not returned to the user.
+ *
+ * This is for logging purposes only and is not returned to the user.
*/
@XmlTransient
String createdRepoId;
diff --git a/java/google/registry/model/host/HostResource.java b/java/google/registry/model/host/HostResource.java
index 1e2052bf9..865cd6d06 100644
--- a/java/google/registry/model/host/HostResource.java
+++ b/java/google/registry/model/host/HostResource.java
@@ -74,9 +74,9 @@ public class HostResource extends EppResource implements ForeignKeyedEppResource
/**
* Fully qualified hostname, which is a unique identifier for this host.
- *
- * This is only unique in the sense that for any given lifetime specified as the time range from
- * (creationTime, deletionTime) there can only be one host in the datastore with this name.
+ *
+ * This is only unique in the sense that for any given lifetime specified as the time range
+ * from (creationTime, deletionTime) there can only be one host in the datastore with this name.
* However, there can be many hosts with the same name and non-overlapping lifetimes.
*/
// TODO(b/25644770): Backfill this index. Until that's done, don't rely on it!
diff --git a/java/google/registry/model/mark/Mark.java b/java/google/registry/model/mark/Mark.java
index 77f0269f4..c2aff2091 100644
--- a/java/google/registry/model/mark/Mark.java
+++ b/java/google/registry/model/mark/Mark.java
@@ -27,8 +27,8 @@ import javax.xml.bind.annotation.XmlRootElement;
/**
* Information about one or more marks.
- *
- * A mark is a term for a label with some sort of legally protected status. The most well known
+ *
+ * A mark is a term for a label with some sort of legally protected status. The most well known
* version is a registered trademark, but marks can also be derived from court opinions, treaties,
* or statutes.
*/
diff --git a/java/google/registry/model/mark/MarkAddress.java b/java/google/registry/model/mark/MarkAddress.java
index 2900bea58..252887ac7 100644
--- a/java/google/registry/model/mark/MarkAddress.java
+++ b/java/google/registry/model/mark/MarkAddress.java
@@ -18,10 +18,10 @@ import google.registry.model.eppcommon.Address;
/**
* Mark Holder/Owner Address
- *
- * This class is embedded inside {@link CommonMarkContactFields} hold the address of a mark contact
- * or holder. The fields are all defined in parent class {@link Address}, but the subclass is still
- * necessary to pick up the mark namespace.
+ *
+ * This class is embedded inside {@link CommonMarkContactFields} hold the address of a mark
+ * contact or holder. The fields are all defined in parent class {@link Address}, but the subclass
+ * is still necessary to pick up the mark namespace.
*
* @see CommonMarkContactFields
*/
diff --git a/java/google/registry/model/mark/MarkPhoneNumber.java b/java/google/registry/model/mark/MarkPhoneNumber.java
index e78e008b1..0973891d7 100644
--- a/java/google/registry/model/mark/MarkPhoneNumber.java
+++ b/java/google/registry/model/mark/MarkPhoneNumber.java
@@ -18,8 +18,8 @@ import google.registry.model.eppcommon.PhoneNumber;
/**
* Mark Holder/Owner Phone Number
- *
- * This class is embedded inside {@link CommonMarkContactFields} hold the phone number of a mark
+ *
+ * This class is embedded inside {@link CommonMarkContactFields} hold the phone number of a mark
* contact or holder. The fields are all defined in parent class {@link PhoneNumber}, but the
* subclass is still necessary to pick up the mark namespace.
*
diff --git a/java/google/registry/model/ofy/Ofy.java b/java/google/registry/model/ofy/Ofy.java
index cc573352f..8b0230dc8 100644
--- a/java/google/registry/model/ofy/Ofy.java
+++ b/java/google/registry/model/ofy/Ofy.java
@@ -60,10 +60,10 @@ import javax.inject.Inject;
/**
* A wrapper around ofy().
- *
- * The primary purpose of this class is to add functionality to support commit logs. It is simpler
- * to wrap {@link Objectify} rather than extend it because this way we can remove some methods that
- * we don't really want exposed and add some shortcuts.
+ *
+ * The primary purpose of this class is to add functionality to support commit logs. It is
+ * simpler to wrap {@link Objectify} rather than extend it because this way we can remove some
+ * methods that we don't really want exposed and add some shortcuts.
*/
public class Ofy {
@@ -71,10 +71,10 @@ public class Ofy {
/**
* Recommended memcache expiration time, which is one hour, specified in seconds.
- *
- * This value should used as a cache expiration time for any entities annotated with an Objectify
- * {@code @Cache} annotation, to put an upper bound on unlikely-but-possible divergence between
- * memcache and datastore when a memcache write fails.
+ *
+ * This value should used as a cache expiration time for any entities annotated with an
+ * Objectify {@code @Cache} annotation, to put an upper bound on unlikely-but-possible divergence
+ * between memcache and datastore when a memcache write fails.
*/
public static final int RECOMMENDED_MEMCACHE_EXPIRATION = 3600;
@@ -181,8 +181,8 @@ public class Ofy {
/**
* Save, without any augmentations except to check that we're not saving any virtual entities.
- *
- * No backups get written.
+ *
+ * No backups get written.
*/
public Saver saveWithoutBackup() {
return new AugmentedSaver() {
diff --git a/java/google/registry/model/package-info.java b/java/google/registry/model/package-info.java
index e61f490e6..166137455 100644
--- a/java/google/registry/model/package-info.java
+++ b/java/google/registry/model/package-info.java
@@ -32,15 +32,15 @@ import javax.xml.bind.annotation.adapters.XmlJavaTypeAdapter;
/**
* This package defines all entities which are managed via EPP XML and persisted to the Datastore
* via Objectify.
- *
- * All first class entities are represented as a "Resource" class - {@link DomainResource},
+ *
+ * All first class entities are represented as a "Resource" class - {@link DomainResource},
* {@link HostResource}, {@link ContactResource}, and {@link RegistrarResource}. Resource objects
* are written in a single shared entity group per TLD. All commands that operate on those entities
* are grouped in a "Command" class- {@link DomainCommand}, {@link HostCommand},
* {@link ContactCommand}. The Resource does double duty as both the persisted representation and as
* the XML-marshallable object returned in respond to Info commands.
- *
- * Command classes are never persisted, and the Objectify annotations on the Create and Update
+ *
+ * Command classes are never persisted, and the Objectify annotations on the Create and Update
* classes are purely for the benefit of the derived Resource classes that inherit from them.
* Whenever a command that mutates the model is executed, a HistoryEvent is stored with the affected
* Resource as its datastore parent. All history entries have an indexed modification time field so
diff --git a/java/google/registry/model/registrar/Registrar.java b/java/google/registry/model/registrar/Registrar.java
index d7beef51a..0113276cb 100644
--- a/java/google/registry/model/registrar/Registrar.java
+++ b/java/google/registry/model/registrar/Registrar.java
@@ -202,8 +202,8 @@ public class Registrar extends ImmutableObject implements Buildable, Jsonifiable
/**
* Registrar name. This is a distinct from the client identifier since there are no restrictions
* on its length.
- *
- * NB: We are assuming that this field is unique across all registrar entities. This is not
+ *
+ * NB: We are assuming that this field is unique across all registrar entities. This is not
* formally enforced in our datastore, but should be enforced by ICANN in that no two registrars
* will be accredited with the same name.
*
diff --git a/java/google/registry/model/registrar/RegistrarAddress.java b/java/google/registry/model/registrar/RegistrarAddress.java
index 9f533f6df..9ab88280a 100644
--- a/java/google/registry/model/registrar/RegistrarAddress.java
+++ b/java/google/registry/model/registrar/RegistrarAddress.java
@@ -23,9 +23,9 @@ import google.registry.model.eppcommon.Address;
/**
* Registrar Address
- *
- * This class is embedded inside a {@link Registrar} object to hold its address. The fields are all
- * defined in parent class {@link Address} so that it can share it with other similar address
+ *
+ * This class is embedded inside a {@link Registrar} object to hold its address. The fields are
+ * all defined in parent class {@link Address} so that it can share it with other similar address
* classes.
*/
@Embed
diff --git a/java/google/registry/model/registry/Registry.java b/java/google/registry/model/registry/Registry.java
index fb836a7d0..05f24733b 100644
--- a/java/google/registry/model/registry/Registry.java
+++ b/java/google/registry/model/registry/Registry.java
@@ -263,8 +263,8 @@ public class Registry extends ImmutableObject implements Buildable {
/**
* The unicode-aware representation of the TLD associated with this {@link Registry}.
- *
- * This will be equal to {@link #tldStr} for ASCII TLDs, but will be non-ASCII for IDN TLDs.
+ *
+ * This will be equal to {@link #tldStr} for ASCII TLDs, but will be non-ASCII for IDN TLDs.
* We store this in a field so that it will be retained upon import into BigQuery.
*/
String tldUnicode;
@@ -348,10 +348,10 @@ public class Registry extends ImmutableObject implements Buildable {
/**
* A property that transitions to different renew billing costs at different times. Stored as a
* list of BillingCostTransition embedded objects using the @Mapify annotation.
- *
- * A given value of this property represents the per-year billing cost for renewing a domain name.
- * This cost is also used to compute costs for transfers, since each transfer includes a renewal
- * to ensure transfers have a cost.
+ *
+ * A given value of this property represents the per-year billing cost for renewing a domain
+ * name. This cost is also used to compute costs for transfers, since each transfer includes a
+ * renewal to ensure transfers have a cost.
*/
@Mapify(TimedTransitionProperty.TimeMapper.class)
TimedTransitionProperty
- * Note that {@link TldState#PDT} TLDs pretend to be in {@link TldState#GENERAL_AVAILABILITY}.
+ *
+ * Note that {@link TldState#PDT} TLDs pretend to be in {@link TldState#GENERAL_AVAILABILITY}.
*/
public TldState getTldState(DateTime now) {
TldState state = tldStateTransitions.getValueAtTime(now);
@@ -703,8 +703,8 @@ public class Registry extends ImmutableObject implements Buildable {
/**
* Sets the renew billing cost to transition to the specified values at the specified times.
- *
- * Renew billing costs transitions should only be added at least 5 days (the length of an
+ *
+ * Renew billing costs transitions should only be added at least 5 days (the length of an
* automatic transfer) in advance, to avoid discrepancies between the cost stored with the
* billing event (created when the transfer is requested) and the cost at the time when the
* transfer actually occurs (5 days later).
diff --git a/java/google/registry/model/server/ServerSecret.java b/java/google/registry/model/server/ServerSecret.java
index d8f7861f4..7ad13a2b5 100644
--- a/java/google/registry/model/server/ServerSecret.java
+++ b/java/google/registry/model/server/ServerSecret.java
@@ -42,8 +42,8 @@ public class ServerSecret extends CrossTldSingleton {
/**
* Get the server secret, creating it if the datastore doesn't have one already.
- *
- * There's a tiny risk of a race here if two calls to this happen simultaneously and create
+ *
+ * There's a tiny risk of a race here if two calls to this happen simultaneously and create
* different keys, in which case one of the calls will end up with an incorrect key. However, this
* happens precisely once in the history of the system (after that it's always in datastore) so
* it's not worth worrying about.
diff --git a/java/google/registry/model/smd/SignedMarkRevocationList.java b/java/google/registry/model/smd/SignedMarkRevocationList.java
index 18335a50a..38641d636 100644
--- a/java/google/registry/model/smd/SignedMarkRevocationList.java
+++ b/java/google/registry/model/smd/SignedMarkRevocationList.java
@@ -53,14 +53,14 @@ import java.util.Map;
/**
* Signed Mark Data Revocation List (SMDRL).
- *
- * Represents a SMDRL file downloaded from the TMCH MarksDB each day. The list holds the ids of
- * all the {@link SignedMark SignedMarks} that have been revoked. A new list is created for each
- * new file that's created, depending on the timestamp.
- *
- * We'll be putting the entire table into a single entity for the sake of performance. But in order
- * to avoid exceeding the one megabyte max entity size limit, we'll also be sharding that entity
- * into multiple entities, each entity containing {@value #SHARD_SIZE} rows.
+ *
+ * Represents a SMDRL file downloaded from the TMCH MarksDB each day. The list holds the ids of
+ * all the {@link SignedMark SignedMarks} that have been revoked. A new list is created for each new
+ * file that's created, depending on the timestamp.
+ *
+ * We'll be putting the entire table into a single entity for the sake of performance. But in
+ * order to avoid exceeding the one megabyte max entity size limit, we'll also be sharding that
+ * entity into multiple entities, each entity containing {@value #SHARD_SIZE} rows.
*
* @see google.registry.tmch.SmdrlCsvParser
* @see "http://tools.ietf.org/html/draft-lozano-tmch-func-spec-08#section-6.2"
diff --git a/java/google/registry/model/tmch/TmchCrl.java b/java/google/registry/model/tmch/TmchCrl.java
index 126c5efa1..de9d9a634 100644
--- a/java/google/registry/model/tmch/TmchCrl.java
+++ b/java/google/registry/model/tmch/TmchCrl.java
@@ -49,8 +49,8 @@ public final class TmchCrl extends CrossTldSingleton {
/**
* Change the datastore singleton to a new ASCII-armored X.509 CRL.
- *
- * Please do not call this function unless your CRL is properly formatted, signed by the root,
+ *
+ * Please do not call this function unless your CRL is properly formatted, signed by the root,
* and actually newer than the one currently in the datastore.
*/
public static void set(final String crl) {
diff --git a/java/google/registry/model/transfer/TransferData.java b/java/google/registry/model/transfer/TransferData.java
index 1fed90b24..4918cdf39 100644
--- a/java/google/registry/model/transfer/TransferData.java
+++ b/java/google/registry/model/transfer/TransferData.java
@@ -45,42 +45,42 @@ public class TransferData extends BaseTransferObject implements Buildable {
/**
* The billing event and poll messages associated with a server-approved transfer.
- *
- * This field should be null if there is not currently a pending transfer or if the object being
- * transferred is not a domain. If there is a pending transfer for a domain there should be a
- * number of poll messages and billing events for both the gaining and losing registrars. If the
+ *
+ * This field should be null if there is not currently a pending transfer or if the object
+ * being transferred is not a domain. If there is a pending transfer for a domain there should be
+ * a number of poll messages and billing events for both the gaining and losing registrars. If the
* pending transfer is explicitly approved, rejected or cancelled, the referenced entities should
* be deleted.
- *
- * Keys are stored here instead of references to facilitate bulk deletion (the typical use case,
- * as described above), since Objectify allows bulk deletion by key but not by reference.
+ *
+ * Keys are stored here instead of references to facilitate bulk deletion (the typical use
+ * case, as described above), since Objectify allows bulk deletion by key but not by reference.
*/
@IgnoreSave(IfNull.class)
Set
- * This field should be null if there is not currently a pending transfer or if the object being
- * transferred is not a domain.
+ *
+ * This field should be null if there is not currently a pending transfer or if the object
+ * being transferred is not a domain.
*/
@IgnoreSave(IfNull.class)
Ref
- * This field should be null if there is not currently a pending transfer or if the object being
- * transferred is not a domain.
+ *
+ * This field should be null if there is not currently a pending transfer or if the object
+ * being transferred is not a domain.
*/
@IgnoreSave(IfNull.class)
Ref
- * This field should be null if there is not currently a pending transfer or if the object being
- * transferred is not a domain.
+ *
+ * This field should be null if there is not currently a pending transfer or if the object
+ * being transferred is not a domain.
*/
@IgnoreSave(IfNull.class)
Ref
- * It's meant to allow more information inside the element, but it's an annoyance when we want to
- * deal with pure enums.
+ *
+ * It's meant to allow more information inside the element, but it's an annoyance when we want
+ * to deal with pure enums.
*/
static class EnumShim {
@XmlAttribute
diff --git a/java/google/registry/model/translators/ReadableInstantUtcTranslatorFactory.java b/java/google/registry/model/translators/ReadableInstantUtcTranslatorFactory.java
index 06336e3fc..3a30a0c81 100644
--- a/java/google/registry/model/translators/ReadableInstantUtcTranslatorFactory.java
+++ b/java/google/registry/model/translators/ReadableInstantUtcTranslatorFactory.java
@@ -33,10 +33,10 @@ import java.util.Date;
/**
* Stores Joda {@link ReadableInstant} types ({@code DateTime}, etc) as a {@link java.util.Date}.
- *
- * This is a fork of the {@code ReadableInstantTranslatorFactory} that comes bundled with Objectify.
- * The original reifies a {@link ReadableInstant} using the machine's local time zone. This version
- * always uses UTC.
+ *
+ * This is a fork of the {@code ReadableInstantTranslatorFactory} that comes bundled with
+ * Objectify. The original reifies a {@link ReadableInstant} using the machine's local time
+ * zone. This version always uses UTC.
*/
public class ReadableInstantUtcTranslatorFactory
extends ValueTranslatorFactory
- * If there is no user, the entire xsrf check becomes basically a no-op, but that's ok because any
- * callback that doesn't have a user shouldn't be able to access any per-user resources anyways.
+ *
+ * If there is no user, the entire xsrf check becomes basically a no-op, but that's ok because
+ * any callback that doesn't have a user shouldn't be able to access any per-user resources
+ * anyways.
*/
public static String generateToken(String scope) {
return generateToken(scope, getLoggedInEmailOrEmpty());
diff --git a/java/google/registry/tmch/TmchCertificateAuthority.java b/java/google/registry/tmch/TmchCertificateAuthority.java
index da0b2f68b..0db1cc49a 100644
--- a/java/google/registry/tmch/TmchCertificateAuthority.java
+++ b/java/google/registry/tmch/TmchCertificateAuthority.java
@@ -50,9 +50,9 @@ public final class TmchCertificateAuthority {
/**
* A cached supplier that loads the crl from datastore or chooses a default value.
- *
- * We keep the cache here rather than caching TmchCrl in the model, because loading the crl string
- * into an X509CRL instance is expensive and should itself be cached.
+ *
+ * We keep the cache here rather than caching TmchCrl in the model, because loading the crl
+ * string into an X509CRL instance is expensive and should itself be cached.
*/
private static final Supplier
- * Just implementing this is sufficient to use the remote api; {@link RegistryTool} will install
- * it as needed.
+ *
+ * Just implementing this is sufficient to use the remote api; {@link RegistryTool} will
+ * install it as needed.
*/
public interface RemoteApiCommand extends Command {}
diff --git a/java/google/registry/tools/CreateAuctionCreditsCommand.java b/java/google/registry/tools/CreateAuctionCreditsCommand.java
index 86d5284ed..1bb0b26d9 100644
--- a/java/google/registry/tools/CreateAuctionCreditsCommand.java
+++ b/java/google/registry/tools/CreateAuctionCreditsCommand.java
@@ -54,15 +54,14 @@ import java.util.List;
/**
* Command for creating new auction credits based on a CSV file from Pool.
- *
- * The CSV file from the auction provider uses double-quotes around every field, so in order to
+ *
+ * The CSV file from the auction provider uses double-quotes around every field, so in order to
* extract the raw field value we strip off the quotes after splitting each line by commas. We are
* using a simple parsing strategy that does not support embedded quotation marks, commas, or
* newlines.
- *
- * TODO(b/16009815): Switch this file to using a real CSV parser.
- *
- * Example file format:
+ *
+ * Example file format:
+ *
* We only care about three fields: 1) the "Affiliate" field which corresponds to the registrar
+ * clientId stored in datastore, and which we use to determine which registrar gets the credit, 2)
+ * the "Commissions" field which contains the amount of the auction credit (as determined by logic
+ * on the auction provider's side, see the Finance Requirements Doc for more information), and 3)
+ * the "CurrencyCode" field, which we validate matches the TLD-wide currency for this TLD.
*/
+// TODO(b/16009815): Switch this file to using a real CSV parser.
@Parameters(separators = " =", commandDescription = "Create new auction credits based on CSV")
final class CreateAuctionCreditsCommand extends MutatingCommand {
diff --git a/java/google/registry/tools/MakeBillingTablesCommand.java b/java/google/registry/tools/MakeBillingTablesCommand.java
index 55d26a4ec..a57a99a5a 100644
--- a/java/google/registry/tools/MakeBillingTablesCommand.java
+++ b/java/google/registry/tools/MakeBillingTablesCommand.java
@@ -146,8 +146,8 @@ final class MakeBillingTablesCommand extends BigqueryCommand {
* Generates a view of registrar credit balance entities that collapses them down to the one
* 'true' credit balance for a given credit ID and effective time, eliminating any duplicates by
* choosing the most recently written balance entry of the set.
- *
- * The result is a list of the historical balances of each credit (according to the most recent
+ *
+ * The result is a list of the historical balances of each credit (according to the most recent
* data written) that can be used to find the active balance of a credit at any point in time.
*/
private void makeCreditBalanceView() throws Exception {
diff --git a/java/google/registry/tools/MutatingCommand.java b/java/google/registry/tools/MutatingCommand.java
index 8c8790826..b3eacb831 100644
--- a/java/google/registry/tools/MutatingCommand.java
+++ b/java/google/registry/tools/MutatingCommand.java
@@ -130,10 +130,10 @@ public abstract class MutatingCommand extends ConfirmingCommand implements Remot
/**
* Initializes the command.
- *
- * Subclasses override this method to populate {@link #changedEntitiesMap} with updated
- * entities. The old entity is the key and the new entity is the value; the key is null for
- * newly created entities and the value is null for deleted entities.
+ *
+ * Subclasses override this method to populate {@link #changedEntitiesMap} with updated
+ * entities. The old entity is the key and the new entity is the value; the key is null for newly
+ * created entities and the value is null for deleted entities.
*/
@Override
protected abstract void init() throws Exception;
diff --git a/java/google/registry/tools/UpdateApplicationStatusCommand.java b/java/google/registry/tools/UpdateApplicationStatusCommand.java
index 46da172aa..f3f99c894 100644
--- a/java/google/registry/tools/UpdateApplicationStatusCommand.java
+++ b/java/google/registry/tools/UpdateApplicationStatusCommand.java
@@ -85,8 +85,8 @@ final class UpdateApplicationStatusCommand extends MutatingCommand {
/**
* Stages changes to update the status of an application and also enqueue a poll message for the
* status change, which may contain a PendingActionNotificationResponse if this is a final status.
- *
- * This method must be called from within a transaction.
+ *
+ * This method must be called from within a transaction.
*/
private void updateApplicationStatus(String applicationId) {
ofy().assertInTransaction();
diff --git a/java/google/registry/tools/params/DateTimeParameter.java b/java/google/registry/tools/params/DateTimeParameter.java
index dc5296283..d1af3f7fb 100644
--- a/java/google/registry/tools/params/DateTimeParameter.java
+++ b/java/google/registry/tools/params/DateTimeParameter.java
@@ -35,8 +35,8 @@ public final class DateTimeParameter extends ParameterConverterValidator We use this instead of the default {@link ISODateTimeFormat#dateTimeParser()} because that
* parser is very flexible and accepts date times with missing dates, missing dates, and various
* other unspecified fields that can lead to confusion and ambiguity.
*/
diff --git a/java/google/registry/tools/params/LoggingLevelParameter.java b/java/google/registry/tools/params/LoggingLevelParameter.java
index 81e072db2..06312118c 100644
--- a/java/google/registry/tools/params/LoggingLevelParameter.java
+++ b/java/google/registry/tools/params/LoggingLevelParameter.java
@@ -18,8 +18,8 @@ import java.util.logging.Level;
/**
* Java logging level CLI parameter converter/validator.
- *
- * If Level were an enum then this wouldn't be needed since JCommander has built-in conversion
+ *
+ * If Level were an enum then this wouldn't be needed since JCommander has built-in conversion
* support for enums, but Level is actually just a regular class with some static instances.
*/
public final class LoggingLevelParameter extends ParameterConverterValidator
- * These strings have the form {@code These strings have the form {@code
- * Subclasses must implement parseValue() to define how to parse {@code Subclasses must implement parseValue() to define how to parse {@code
- * Users can select aliased fields for display using either the original name or the alias. By
+ *
+ * Users can select aliased fields for display using either the original name or the alias. By
* default, aliased fields will use the alias name as the header instead of the original name.
*/
ImmutableBiMap
- * This can be used to specify customized printing of certain fields (e.g. to print out a boolean
- * field as "active" or "-" instead of "true" or "false"). It can also be used to add fields to
- * the data, e.g. for computed fields that can be accessed from the object directly but aren't
- * stored as simple fields.
+ *
+ * This can be used to specify customized printing of certain fields (e.g. to print out a
+ * boolean field as "active" or "-" instead of "true" or "false"). It can also be used to add
+ * fields to the data, e.g. for computed fields that can be accessed from the object directly but
+ * aren't stored as simple fields.
*/
ImmutableMap
- * The resulting strings separate padded fields with two spaces and each end in a newline.
+ *
+ * The resulting strings separate padded fields with two spaces and each end in a newline.
*/
private static Function
- * This servlet returns plain JSON without a safety prefix, so it's vital that the output not be
+ *
+ * This servlet returns plain JSON without a safety prefix, so it's vital that the output not be
* user controlled, lest it open an XSS vector. Do not modify this to return the domain name in the
* response.
*/
diff --git a/java/google/registry/util/AppEngineTimeLimiter.java b/java/google/registry/util/AppEngineTimeLimiter.java
index aa539f5b3..a8a0e07ef 100644
--- a/java/google/registry/util/AppEngineTimeLimiter.java
+++ b/java/google/registry/util/AppEngineTimeLimiter.java
@@ -26,22 +26,22 @@ import java.util.concurrent.TimeUnit;
/**
* A factory for {@link TimeLimiter} instances that use request threads, which carry the namespace
* and live only as long as the request that spawned them.
- *
- * It is safe to reuse instances of this class, but there is no benefit in doing so over creating a
- * fresh instance each time.
+ *
+ * It is safe to reuse instances of this class, but there is no benefit in doing so over creating
+ * a fresh instance each time.
*/
public class AppEngineTimeLimiter {
/**
* An {@code ExecutorService} that uses a new thread for every task.
- *
- * We need to use fresh threads for each request so that we can use App Engine's request threads.
- * If we cached these threads in a thread pool (and if we were executing on a backend, where there
- * is no time limit on requests) the caching would cause the thread to keep the task that opened
- * it alive even after returning an http response, and would also cause the namespace that the
- * original thread was created in to leak out to later reuses of the thread.
- *
- * Since there are no cached resources, this class doesn't have to support being shutdown.
+ *
+ * We need to use fresh threads for each request so that we can use App Engine's request
+ * threads. If we cached these threads in a thread pool (and if we were executing on a backend,
+ * where there is no time limit on requests) the caching would cause the thread to keep the task
+ * that opened it alive even after returning an http response, and would also cause the namespace
+ * that the original thread was created in to leak out to later reuses of the thread.
+ *
+ * Since there are no cached resources, this class doesn't have to support being shutdown.
*/
private static class NewRequestThreadExecutorService extends AbstractExecutorService {
diff --git a/java/google/registry/util/CollectionUtils.java b/java/google/registry/util/CollectionUtils.java
index 43d2c3de5..62006f118 100644
--- a/java/google/registry/util/CollectionUtils.java
+++ b/java/google/registry/util/CollectionUtils.java
@@ -113,9 +113,9 @@ public class CollectionUtils {
/**
* Turns an empty collection into a null collection.
- *
- * This is unwise in the general case (nulls are bad; empties are good) but occasionally needed to
- * cause JAXB not to emit a field, or to avoid saving something to datastore. The method name
+ *
+ * This is unwise in the general case (nulls are bad; empties are good) but occasionally needed
+ * to cause JAXB not to emit a field, or to avoid saving something to datastore. The method name
* includes "force" to indicate that you should think twice before using it.
*/
@Nullable
diff --git a/java/google/registry/util/DateTimeUtils.java b/java/google/registry/util/DateTimeUtils.java
index 2bedeb091..6c9392c6d 100644
--- a/java/google/registry/util/DateTimeUtils.java
+++ b/java/google/registry/util/DateTimeUtils.java
@@ -31,10 +31,10 @@ public class DateTimeUtils {
/**
* A date in the far future that we can treat as infinity.
- *
- * This value is (2^63-1)/1000 rounded down. AppEngine stores dates as 64 bit microseconds, but
- * Java uses milliseconds, so this is the largest representable date that will survive
- * a round-trip through the datastore.
+ *
+ * This value is (2^63-1)/1000 rounded down. AppEngine stores dates as 64 bit microseconds, but
+ * Java uses milliseconds, so this is the largest representable date that will survive a
+ * round-trip through the datastore.
*/
public static final DateTime END_OF_TIME = new DateTime(Long.MAX_VALUE / 1000, DateTimeZone.UTC);
diff --git a/java/google/registry/util/X509Utils.java b/java/google/registry/util/X509Utils.java
index 96e4b4da1..de63db854 100644
--- a/java/google/registry/util/X509Utils.java
+++ b/java/google/registry/util/X509Utils.java
@@ -53,9 +53,9 @@ public final class X509Utils {
/**
* Parse the encoded certificate and return a base64 encoded string (without padding) of the
* SHA-256 digest of the certificate.
- *
- * Note that this must match the method used by the GFE to generate the client certificate hash so
- * that the two will match when we check against the whitelist.
+ *
+ * Note that this must match the method used by the GFE to generate the client certificate hash
+ * so that the two will match when we check against the whitelist.
*/
public static String getCertificateHash(X509Certificate cert) {
try {
diff --git a/java/google/registry/xml/XmlTransformer.java b/java/google/registry/xml/XmlTransformer.java
index 27d520275..7d8c27a0c 100644
--- a/java/google/registry/xml/XmlTransformer.java
+++ b/java/google/registry/xml/XmlTransformer.java
@@ -121,9 +121,9 @@ public class XmlTransformer {
/**
* Validates XML text against {@link #schema} without marshalling.
- *
- * You must specify the XML class you expect to receive as the root element.
- * Validation is performed in accordance with the hard-coded XML schemas.
+ *
+ * You must specify the XML class you expect to receive as the root element. Validation is
+ * performed in accordance with the hard-coded XML schemas.
*
* @throws XmlException if XML input was invalid or root element doesn't match {@code expect}.
*/
@@ -137,8 +137,8 @@ public class XmlTransformer {
/**
* Turns XML text into an object, validating against {@link #schema}.
- *
- * You must specify the XML class you expect to receive as the root element. Validation is
+ *
+ * You must specify the XML class you expect to receive as the root element. Validation is
* performed in accordance with the hard-coded XML schemas.
*
* @throws XmlException if failed to read from {@code bytes}, XML input is invalid, or root
@@ -181,11 +181,11 @@ public class XmlTransformer {
/**
* Streams {@code root} without XML declaration, optionally validating against the schema.
- *
- * The root object must be annotated with {@link javax.xml.bind.annotation.XmlRootElement}. If the
- * validation parameter is set to {@link ValidationMode#STRICT} this method will verify that your
- * object strictly conforms to {@link #schema}. Because the output is streamed,
- * {@link XmlException} will most likely be thrown after output has been written.
+ *
+ * The root object must be annotated with {@link javax.xml.bind.annotation.XmlRootElement}. If
+ * the validation parameter is set to {@link ValidationMode#STRICT} this method will verify that
+ * your object strictly conforms to {@link #schema}. Because the output is streamed, {@link
+ * XmlException} will most likely be thrown after output has been written.
*
* @param root the object to write
* @param writer to write the output to
@@ -206,10 +206,10 @@ public class XmlTransformer {
/**
* Validates and streams {@code root} as formatted XML bytes with XML declaration.
- *
- * The root object must be annotated with {@link javax.xml.bind.annotation.XmlRootElement}. If the
- * validation parameter is set to {@link ValidationMode#STRICT} this method will verify that your
- * object strictly conforms to {@link #schema}. Because the output is streamed,
+ *
+ * The root object must be annotated with {@link javax.xml.bind.annotation.XmlRootElement}. If
+ * the validation parameter is set to {@link ValidationMode#STRICT} this method will verify that
+ * your object strictly conforms to {@link #schema}. Because the output is streamed,
* {@link XmlException} will most likely be thrown after output has been written.
*
* @param root the object to write
@@ -233,11 +233,11 @@ public class XmlTransformer {
/**
* Validates and streams {@code root} as characters, always using strict validation.
- *
- * The root object must be annotated with {@link javax.xml.bind.annotation.XmlRootElement}. This
- * method will verify that your object strictly conforms to {@link #schema}. Because the output is
- * streamed, {@link XmlException} will most likely be thrown after output has been
- * written.
+ *
+ * The root object must be annotated with {@link javax.xml.bind.annotation.XmlRootElement}.
+ * This method will verify that your object strictly conforms to {@link #schema}. Because the
+ * output is streamed, {@link XmlException} will most likely be thrown after output has
+ * been written.
*
* @param root the object to write
* @param result to write the output to
*
{
@@ -95,8 +95,8 @@ public interface Buildable {
/**
* Interface for objects that can produce an "overlay", which means a copy where non-null fields
* from another object are copied over, but null fields on the source are not.
- * {@code
* {@code
* {@code
*
+ *
* "Affiliate","DomainName","Email","BidderId","BidderStatus","UpdatedAt",
* "SalePrice","Commissions","CurrencyCode"
@@ -71,12 +70,14 @@ import java.util.List;
* "reg2","foo.xn--q9jyb4c","email2@example.net","???_64","WIN","4/3/2014 7:13:09 PM",
* "1000.0000","40.0000","JPY"
*
- * We only care about three fields: 1) the "Affiliate" field which corresponds to the registrar
- * clientId stored in datastore, and which we use to determine which registrar gets the credit,
- * 2) the "Commissions" field which contains the amount of the auction credit (as determined by
- * logic on the auction provider's side, see the Finance Requirements Doc for more information), and
- * 3) the "CurrencyCode" field, which we validate matches the TLD-wide currency for this TLD.
+ *
+ *