You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by th...@apache.org on 2020/09/12 21:58:43 UTC

[tapestry-5] branch java9modules updated: TAP5-2641: fixing JavaDoc errors

This is an automated email from the ASF dual-hosted git repository.

thiagohp pushed a commit to branch java9modules
in repository https://gitbox.apache.org/repos/asf/tapestry-5.git


The following commit(s) were added to refs/heads/java9modules by this push:
     new 8ca50f5  TAP5-2641: fixing JavaDoc errors
8ca50f5 is described below

commit 8ca50f5b48ce5a0a079e0eef0ebcf5befadaab14
Author: Thiago H. de Paula Figueiredo <th...@arsmachina.com.br>
AuthorDate: Sat Sep 12 18:58:22 2020 -0300

    TAP5-2641: fixing JavaDoc errors
---
 .../java/org/apache/tapestry5/beanmodel/PropertyConduit2.java  |  2 +-
 .../java/org/apache/tapestry5/commons/MessageFormatter.java    |  2 +-
 .../org/apache/tapestry5/commons/OrderedConfiguration.java     |  2 +-
 .../commons/internal/services/AnnotationProviderChain.java     |  2 +-
 .../tapestry5/commons/internal/services/StringInterner.java    |  4 ++--
 .../apache/tapestry5/commons/internal/util/MessagesImpl.java   |  2 +-
 .../tapestry5/commons/services/ClassPropertyAdapter.java       |  2 +-
 .../org/apache/tapestry5/commons/services/CoercionTuple.java   |  4 ++--
 .../apache/tapestry5/commons/services/DataTypeAnalyzer.java    |  8 ++++----
 .../tapestry5/commons/services/InvalidationEventHub.java       |  6 +++---
 .../tapestry5/commons/services/InvalidationListener.java       |  2 +-
 .../org/apache/tapestry5/commons/services/PropertyAdapter.java |  5 ++---
 .../org/apache/tapestry5/commons/services/TypeCoercer.java     |  2 +-
 .../apache/tapestry5/commons/util/StringToEnumCoercion.java    |  2 +-
 .../main/java/org/apache/tapestry5/internal/plastic/Cache.java |  4 ++--
 .../apache/tapestry5/plastic/PlasticClassTransformation.java   |  2 +-
 .../src/main/java/org/apache/tapestry5/FieldValidator.java     |  2 +-
 .../src/main/java/org/apache/tapestry5/Translator.java         |  2 +-
 .../java/org/apache/tapestry5/annotations/PublishEvent.java    |  2 +-
 .../tapestry5/internal/services/EnumValueEncoderFactory.java   |  2 +-
 .../internal/services/GenericValueEncoderFactory.java          |  2 +-
 .../main/java/org/apache/tapestry5/internal/util/Holder.java   |  2 +-
 .../org/apache/tapestry5/services/ApplicationStateCreator.java |  2 +-
 .../tapestry5/services/ComponentEventResultProcessor.java      |  2 +-
 .../org/apache/tapestry5/tree/DefaultTreeExpansionModel.java   |  2 +-
 .../main/java/org/apache/tapestry5/tree/DefaultTreeModel.java  |  2 +-
 .../src/main/java/org/apache/tapestry5/func/Tuple.java         |  4 ++--
 .../src/main/java/org/apache/tapestry5/func/ZippedFlow.java    |  4 ++--
 .../src/main/java/org/apache/tapestry5/ioc/Orderable.java      |  2 +-
 .../ioc/internal/ValidatingMappedConfigurationWrapper.java     |  4 ++--
 .../main/java/org/apache/tapestry5/beaneditor/NonVisual.java   |  5 +++--
 .../java/org/apache/tapestry5/beaneditor/RelativePosition.java |  3 ++-
 .../java/org/apache/tapestry5/ioc/annotations/Autobuild.java   |  6 +++---
 .../org/apache/tapestry5/ioc/annotations/ComponentClasses.java |  2 +-
 .../org/apache/tapestry5/ioc/annotations/ComponentLayer.java   |  3 ++-
 .../main/java/org/apache/tapestry5/ioc/annotations/Inject.java |  4 ++--
 .../org/apache/tapestry5/ioc/annotations/InjectService.java    |  2 +-
 .../org/apache/tapestry5/ioc/annotations/IntermediateType.java |  8 ++++++--
 .../apache/tapestry5/ioc/annotations/UsesConfiguration.java    |  4 ++--
 .../tapestry5/ioc/annotations/UsesMappedConfiguration.java     | 10 +++++++---
 .../tapestry5/ioc/annotations/UsesOrderedConfiguration.java    |  4 ++--
 .../main/java/org/apache/tapestry5/ioc/annotations/Value.java  |  2 +-
 42 files changed, 74 insertions(+), 64 deletions(-)

diff --git a/beanmodel/src/main/java/org/apache/tapestry5/beanmodel/PropertyConduit2.java b/beanmodel/src/main/java/org/apache/tapestry5/beanmodel/PropertyConduit2.java
index 36fbf8d..cdb017f 100644
--- a/beanmodel/src/main/java/org/apache/tapestry5/beanmodel/PropertyConduit2.java
+++ b/beanmodel/src/main/java/org/apache/tapestry5/beanmodel/PropertyConduit2.java
@@ -23,7 +23,7 @@ import org.apache.tapestry5.beanmodel.services.PropertyConduitSource;
 /**
  * Extension to {@link PropertyConduit} that adds a method to access the generic property type.
  * {@link PropertyConduitSource} instances should ideally return PropertyConduit2 objects, not PropertyConduit.
- * This is only primarily of interest to {@link Binding2}.
+ * This is only primarily of interest to <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/Binding2.html">Binding2</a>.
  * 
  * @since 5.4
  */
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/MessageFormatter.java b/commons/src/main/java/org/apache/tapestry5/commons/MessageFormatter.java
index dc7a120..871dd2a 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/MessageFormatter.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/MessageFormatter.java
@@ -15,7 +15,7 @@
 package org.apache.tapestry5.commons;
 
 /**
- * Obtained from a {@link org.apache.tapestry5.ioc.Messages}, used to format messages for a specific localized message
+ * Obtained from a <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/ioc/MessageFormatter.html">org.apache.tapestry5.ioc.Messages</a>, used to format messages for a specific localized message
  * key.
  */
 public interface MessageFormatter
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/OrderedConfiguration.java b/commons/src/main/java/org/apache/tapestry5/commons/OrderedConfiguration.java
index eb420e2..dd2abaf 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/OrderedConfiguration.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/OrderedConfiguration.java
@@ -24,7 +24,7 @@ package org.apache.tapestry5.commons;
  * </ul>
  *
  * The service defines the <em>type</em> of contribution, in terms of a base class or service interface. Contributions
- * must be compatible with the type, or be {@linkplain org.apache.tapestry5.ioc.services.TypeCoercer coercable} to the type.
+ * must be compatible with the type, or be {@linkplain org.apache.tapestry5.commons.services.TypeCoercer coercable} to the type.
  *
  * @see org.apache.tapestry5.ioc.annotations.Contribute
  * @see org.apache.tapestry5.ioc.annotations.UsesConfiguration
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/internal/services/AnnotationProviderChain.java b/commons/src/main/java/org/apache/tapestry5/commons/internal/services/AnnotationProviderChain.java
index d106aa2..7b4b07f 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/internal/services/AnnotationProviderChain.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/internal/services/AnnotationProviderChain.java
@@ -20,7 +20,7 @@ import java.lang.annotation.Annotation;
 import java.util.List;
 
 /**
- * Chain of command for {@link org.apache.tapestry5.ioc.AnnotationProvider}.
+ * Chain of command for {@link org.apache.tapestry5.commons.AnnotationProvider}.
  */
 public class AnnotationProviderChain implements AnnotationProvider
 {
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/internal/services/StringInterner.java b/commons/src/main/java/org/apache/tapestry5/commons/internal/services/StringInterner.java
index 80dbe5e..eb4db44 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/internal/services/StringInterner.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/internal/services/StringInterner.java
@@ -17,8 +17,8 @@ package org.apache.tapestry5.commons.internal.services;
 /**
  * Creates "interned" strings that are unique for the same content. This is used for common description strings,
  * particularly those used by {@link org.apache.tapestry5.Binding} instances.  The internal cache of interned strings id
- * cleared whenever the {@link org.apache.tapestry5.annotations.ComponentClasses} {@link
- * org.apache.tapestry5.services.InvalidationEventHub} is invalidated (i.e., when component class files change).
+ * cleared whenever the {@link org.apache.tapestry5.annotations.ComponentClasses} 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/services/InvalidationEventHub.html">InvalidationEventHub</a> is invalidated (i.e., when component class files change).
  *
  * @since 5.1.0.0
  */
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/internal/util/MessagesImpl.java b/commons/src/main/java/org/apache/tapestry5/commons/internal/util/MessagesImpl.java
index 2d42d58..0c7c231 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/internal/util/MessagesImpl.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/internal/util/MessagesImpl.java
@@ -25,7 +25,7 @@ import org.apache.tapestry5.commons.util.AbstractMessages;
 import org.apache.tapestry5.commons.util.CollectionFactory;
 
 /**
- * Implementation of {@link org.apache.tapestry5.ioc.Messages} based around a {@link java.util.ResourceBundle}.
+ * Implementation of {@link org.apache.tapestry5.commons.Messages} based around a {@link java.util.ResourceBundle}.
  */
 public class MessagesImpl extends AbstractMessages
 {
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/ClassPropertyAdapter.java b/commons/src/main/java/org/apache/tapestry5/commons/services/ClassPropertyAdapter.java
index 1232a1b..1523819 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/ClassPropertyAdapter.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/ClassPropertyAdapter.java
@@ -16,7 +16,7 @@ import java.lang.annotation.Annotation;
 import java.util.List;
 
 /**
- * Organizes all {@link org.apache.tapestry5.ioc.services.PropertyAdapter}s for a particular class.
+ * Organizes all {@link org.apache.tapestry5.commons.services.PropertyAdapter}s for a particular class.
  *
  * Only provides access to <em>simple</em> properties. Indexed properties are ignored.
  *
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/CoercionTuple.java b/commons/src/main/java/org/apache/tapestry5/commons/services/CoercionTuple.java
index af795b5..144306b 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/CoercionTuple.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/CoercionTuple.java
@@ -18,8 +18,8 @@ import org.apache.tapestry5.plastic.PlasticUtils;
 
 /**
  * An immutable object that represents a mapping from one type to another. This is also the contribution type when
- * building the {@link org.apache.tapestry5.ioc.services.TypeCoercer} service. Wraps a
- * {@link org.apache.tapestry5.ioc.services.Coercion} object that performs the work with additional properties that
+ * building the {@link org.apache.tapestry5.comons.services.TypeCoercer} service. Wraps a
+ * {@link org.apache.tapestry5.commons.services.Coercion} object that performs the work with additional properties that
  * describe
  * the input and output types of the coercion, needed when searching for an appropriate coercion (or sequence of
  * coercions).
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/DataTypeAnalyzer.java b/commons/src/main/java/org/apache/tapestry5/commons/services/DataTypeAnalyzer.java
index ab7165a..d5863be 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/DataTypeAnalyzer.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/DataTypeAnalyzer.java
@@ -16,7 +16,7 @@ import org.apache.tapestry5.ioc.annotations.UsesMappedConfiguration;
 import org.apache.tapestry5.ioc.annotations.UsesOrderedConfiguration;
 
 /**
- * Used by {@link BeanModelSource} to identify the type of data associated with a particular property (represented as a
+ * Used by {@link org.apache.tapestry5.beanmodel.services.BeanModelSource} to identify the type of data associated with a particular property (represented as a
  * {@link PropertyAdapter}). The data type is a string used to determine what kind of interface to use for displaying
  * the value of the property, or what kind of interface to use for editing the value of the property. Common property
  * types are "text", "enum", "checkbox", but the list is extensible.
@@ -27,12 +27,12 @@ import org.apache.tapestry5.ioc.annotations.UsesOrderedConfiguration;
  * The DefaultDataTypeAnalyzer service maps property types to data type names.
  *
  * The DataTypeAnalyzer service is an extensible {@linkplain org.apache.tapestry5.ioc.services.ChainBuilder chain of
- * command}), that (by default) includes {@link org.apache.tapestry5.internal.services.AnnotationDataTypeAnalyzer} and
- * the {@link org.apache.tapestry5.internal.services.DefaultDataTypeAnalyzer} service (ordered last).   It uses an ordered configuration.
+ * command}), that (by default) includes {@link org.apache.tapestry5.commons.internal.services.AnnotationDataTypeAnalyzer} and
+ * the {@link org.apache.tapestry5.commons.internal.services.DefaultDataTypeAnalyzer} service (ordered last).   It uses an ordered configuration.
  *
  * @see org.apache.tapestry5.corelib.components.Grid
  * @see org.apache.tapestry5.corelib.components.BeanEditForm
- * @see BeanBlockSource
+ * @see org.apache.tapestry5.BeanBlockSource
  */
 @UsesOrderedConfiguration(DataTypeAnalyzer.class)
 @UsesMappedConfiguration(key = Class.class, value = String.class)
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationEventHub.java b/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationEventHub.java
index 89ccd57..077cc5a 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationEventHub.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationEventHub.java
@@ -21,11 +21,11 @@ import java.util.Map;
  * for:
  * <dl>
  * <dt>message catalog resources
- * <dd>{@link org.apache.tapestry5.services.ComponentMessages} marker annotation
+ * <dd><a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/services/ComponentMessages.html">ComponentMessages</a> marker annotation
  * <dt>component templates
- * <dd>{@link org.apache.tapestry5.services.ComponentTemplates} marker annotation
+ * <dd><a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/services/ComponentTemplates.html">ComponentTemplates</a> marker annotation
  * <dt>component classes
- * <dd>{@link org.apache.tapestry5.annotations.ComponentClasses} marker annotation
+ * <dd><a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/services/ComponentClasses.html">ComponentClasses</a> marker annotation
  * </dl>
  *
  * Starting in Tapestry 5.3, these services are disabled in production (it does nothing).
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationListener.java b/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationListener.java
index 843dc6b..56b3b26 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationListener.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/InvalidationListener.java
@@ -20,7 +20,7 @@ package org.apache.tapestry5.commons.services;
  * file is found to have changed, the event is fired (with the expectation that the cleared cache will be repopulated as
  * necessary).
  *
- * @see org.apache.tapestry5.services.InvalidationEventHub
+ * @see org.apache.tapestry5.commons.services.InvalidationEventHub
  * @since 5.1.0.0
  * @deprecated In 5.4; use {@link InvalidationEventHub#addInvalidationCallback(Runnable)} instead
  */
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/PropertyAdapter.java b/commons/src/main/java/org/apache/tapestry5/commons/services/PropertyAdapter.java
index 4569359..e8b79ee 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/PropertyAdapter.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/PropertyAdapter.java
@@ -18,7 +18,7 @@ import java.lang.reflect.Method;
 import org.apache.tapestry5.commons.AnnotationProvider;
 
 /**
- * Provides access to a single property within a class. Acts as an {@link org.apache.tapestry5.ioc.AnnotationProvider};
+ * Provides access to a single property within a class. Acts as an {@link org.apache.tapestry5.commons.AnnotationProvider};
  * when searching for annotations, the read method (if present) is checked first, followed by the write method, followed
  * by the underlying field (when the property name matches the field name).
  *
@@ -26,7 +26,6 @@ import org.apache.tapestry5.commons.AnnotationProvider;
  *
  * @see org.apache.tapestry5.ioc.services.ClassPropertyAdapter
  */
-@SuppressWarnings("unchecked")
 public interface PropertyAdapter extends AnnotationProvider
 {
     /**
@@ -85,7 +84,7 @@ public interface PropertyAdapter extends AnnotationProvider
     boolean isCastRequired();
 
     /**
-     * Returns the {@link org.apache.tapestry5.ioc.services.ClassPropertyAdapter} that provides access to other
+     * Returns the {@link org.apache.tapestry5.commons.services.ClassPropertyAdapter} that provides access to other
      * properties defined by the same class.
      */
     ClassPropertyAdapter getClassAdapter();
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/services/TypeCoercer.java b/commons/src/main/java/org/apache/tapestry5/commons/services/TypeCoercer.java
index be77a98..bcb59cd 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/services/TypeCoercer.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/services/TypeCoercer.java
@@ -15,7 +15,7 @@ package org.apache.tapestry5.commons.services;
 import org.apache.tapestry5.ioc.annotations.UsesConfiguration;
 
 /**
- * Makes use of {@link org.apache.tapestry5.ioc.services.Coercion}s to convert between an input value (of some specific
+ * Makes use of {@link org.apache.tapestry5.commons.services.Coercion}s to convert between an input value (of some specific
  * type) and a desired output type. Smart about coercing, even if it requires multiple coercion steps (i.e., via an
  * intermediate type, such as String).
  */
diff --git a/commons/src/main/java/org/apache/tapestry5/commons/util/StringToEnumCoercion.java b/commons/src/main/java/org/apache/tapestry5/commons/util/StringToEnumCoercion.java
index db59464..b6ec295 100644
--- a/commons/src/main/java/org/apache/tapestry5/commons/util/StringToEnumCoercion.java
+++ b/commons/src/main/java/org/apache/tapestry5/commons/util/StringToEnumCoercion.java
@@ -18,7 +18,7 @@ import org.apache.tapestry5.commons.internal.util.InternalCommonsUtils;
 import org.apache.tapestry5.commons.services.Coercion;
 
 /**
- * A {@link org.apache.tapestry5.ioc.services.Coercion} for converting strings into an instance of a particular
+ * A {@link org.apache.tapestry5.commons.services.Coercion} for converting strings into an instance of a particular
  * enumerated type. The {@link Enum#name() name} is used as the key to identify the enum instance, in a case-insensitive
  * fashion.
  *
diff --git a/plastic/src/main/java/org/apache/tapestry5/internal/plastic/Cache.java b/plastic/src/main/java/org/apache/tapestry5/internal/plastic/Cache.java
index 611d626..d5f6d36 100644
--- a/plastic/src/main/java/org/apache/tapestry5/internal/plastic/Cache.java
+++ b/plastic/src/main/java/org/apache/tapestry5/internal/plastic/Cache.java
@@ -21,8 +21,8 @@ import java.util.Map;
  * Quick and dirty key/value cache that is subclassed to provide the logic that generates the value for
  * a missing key.
  * 
- * @param <S>
- * @param <T>
+ * @param <S> key type
+ * @param <T> value type
  */
 public abstract class Cache<S, T>
 {
diff --git a/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticClassTransformation.java b/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticClassTransformation.java
index 2d80ba4..09afa86 100644
--- a/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticClassTransformation.java
+++ b/plastic/src/main/java/org/apache/tapestry5/plastic/PlasticClassTransformation.java
@@ -16,7 +16,7 @@ package org.apache.tapestry5.plastic;
  * A wrapper around a {@link PlasticClass} that allows the class to be converted into a concrete class, once all
  * transformations are complete.
  * 
- * @param <T>
+ * @param <T> the type being generated.
  */
 public interface PlasticClassTransformation<T>
 {
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/FieldValidator.java b/tapestry-core/src/main/java/org/apache/tapestry5/FieldValidator.java
index 0f49b95..9f6bd2d 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/FieldValidator.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/FieldValidator.java
@@ -17,7 +17,7 @@ package org.apache.tapestry5;
 /**
  * Responsible for validation of a single field.
  *
- * @param <T>
+ * @param <T> the type of the values being validated
  * @see Validator
  * @see org.apache.tapestry5.services.FieldValidatorDefaultSource
  */
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/Translator.java b/tapestry-core/src/main/java/org/apache/tapestry5/Translator.java
index 8ca1c3f..7400ae2 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/Translator.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/Translator.java
@@ -19,7 +19,7 @@ import org.apache.tapestry5.services.FormSupport;
 /**
  * Translates between client-side and server-side values. Client-side values are always strings.
  * 
- * @param <T>
+ * @param <T> the type being translated
  * @see org.apache.tapestry5.services.TranslatorSource
  * @see org.apache.tapestry5.FieldValidationSupport
  * @see org.apache.tapestry5.FieldTranslator
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/annotations/PublishEvent.java b/tapestry-core/src/main/java/org/apache/tapestry5/annotations/PublishEvent.java
index 33e5cfd..acd5858 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/annotations/PublishEvent.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/annotations/PublishEvent.java
@@ -28,7 +28,7 @@ import org.apache.tapestry5.ioc.annotations.UseWith;
  * Marks an event handler method to be published as an event to be called in JavaScript
  * through the <code>t5/core/ajax</code> function when the <code>options</code>
  * parameter has an <code>element</code> attribute.
- * <p>
+ * </p>
  * 
  * <p>
  * The logic for obtaining the URL is actually located implemented in the
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/EnumValueEncoderFactory.java b/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/EnumValueEncoderFactory.java
index a563e45..90b9cf1 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/EnumValueEncoderFactory.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/EnumValueEncoderFactory.java
@@ -22,7 +22,7 @@ import org.apache.tapestry5.util.EnumValueEncoder;
 /**
  * Factory that provides a configured instance of {@link EnumValueEncoder}.
  *
- * @param <E>
+ * @param <E> the type of the value
  * @deprecated this class is superseded by {@link TypeCoercedValueEncoderFactory} and 
  * scheduled to be removed in Tapestry 5.5
  */
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/GenericValueEncoderFactory.java b/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/GenericValueEncoderFactory.java
index b903319..5cce40d 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/GenericValueEncoderFactory.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/internal/services/GenericValueEncoderFactory.java
@@ -22,7 +22,7 @@ import org.apache.tapestry5.services.ValueEncoderFactory;
  * odd for a factory, because it doesn't actually create the returned instance, just stores it until the encoder is
  * needed.
  *
- * @param <V>
+ * @param <V> the type of the value.
  */
 public class GenericValueEncoderFactory<V> implements ValueEncoderFactory<V>
 {
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/internal/util/Holder.java b/tapestry-core/src/main/java/org/apache/tapestry5/internal/util/Holder.java
index d457564..935cbd1 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/internal/util/Holder.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/internal/util/Holder.java
@@ -21,7 +21,7 @@ import java.util.concurrent.atomic.AtomicReference;
  * (used as a closure) to the containing method. This is similar to {@link AtomicReference}, except that it is simpler
  * but <strong>not</strong> thread safe.
  *
- * @param <T>
+ * @param <T> the type being holded.
  */
 public class Holder<T>
 {
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/services/ApplicationStateCreator.java b/tapestry-core/src/main/java/org/apache/tapestry5/services/ApplicationStateCreator.java
index 47ce8f8..516fec9 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/services/ApplicationStateCreator.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/services/ApplicationStateCreator.java
@@ -26,7 +26,7 @@ package org.apache.tapestry5.services;
  * (Application State Objects). This service would be better named "SessionStateCreator"
  * (but renaming it would cause backwards compatibility issues).</em>
  * 
- * @param <T>
+ * @param <T> the type of the created objects
  */
 public interface ApplicationStateCreator<T>
 {
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/services/ComponentEventResultProcessor.java b/tapestry-core/src/main/java/org/apache/tapestry5/services/ComponentEventResultProcessor.java
index 5505ff9..63ed7a3 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/services/ComponentEventResultProcessor.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/services/ComponentEventResultProcessor.java
@@ -26,7 +26,7 @@ import java.io.IOException;
  * for Ajax requests (which typically return a partially rendered page), distinguished by the @{@link
  * org.apache.tapestry5.services.Ajax} marker annotation.
  *
- * @param <T>
+ * @param <T> the type of the values being handled
  */
 @UsesMappedConfiguration(key = Class.class, value = ComponentEventResultProcessor.class)
 public interface ComponentEventResultProcessor<T>
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java b/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java
index 5097880..748fb27 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeExpansionModel.java
@@ -22,7 +22,7 @@ import java.util.Set;
 /**
  * Manages a Set of String {@link TreeNode} ids.
  * 
- * @param <T>
+ * @param <T> the type of the tree.
  * @since 5.3
  * @see TreeModel
  */
diff --git a/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeModel.java b/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeModel.java
index 27ddbf4..38780b0 100644
--- a/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeModel.java
+++ b/tapestry-core/src/main/java/org/apache/tapestry5/tree/DefaultTreeModel.java
@@ -28,7 +28,7 @@ import java.util.Map;
  *
  * This implementation is <em>not</em> thread safe.
  *
- * @param <T>
+ * @param <T> the type of data in the tree
  * @since 5.3
  */
 @SuppressWarnings({"UnusedDeclaration"})
diff --git a/tapestry-func/src/main/java/org/apache/tapestry5/func/Tuple.java b/tapestry-func/src/main/java/org/apache/tapestry5/func/Tuple.java
index 86625f3..40759a7 100644
--- a/tapestry-func/src/main/java/org/apache/tapestry5/func/Tuple.java
+++ b/tapestry-func/src/main/java/org/apache/tapestry5/func/Tuple.java
@@ -17,8 +17,8 @@ package org.apache.tapestry5.func;
 /**
  * A Tuple holds two values of two different types.
  * 
- * @param <A>
- * @param <B>
+ * @param <A> first type
+ * @param <B> second type
  * @since 5.3
  */
 public class Tuple<A, B>
diff --git a/tapestry-func/src/main/java/org/apache/tapestry5/func/ZippedFlow.java b/tapestry-func/src/main/java/org/apache/tapestry5/func/ZippedFlow.java
index a2caa87..ab7e85c 100644
--- a/tapestry-func/src/main/java/org/apache/tapestry5/func/ZippedFlow.java
+++ b/tapestry-func/src/main/java/org/apache/tapestry5/func/ZippedFlow.java
@@ -20,8 +20,8 @@ import java.util.Map;
  * The result of the {@link Flow#zipWith(Flow)} method (or created from a Map via {@link F#zippedFlow(Map)}), a Flow of
  * combined {@link Tuple} values (that can be deconstructed, eventually, using {@link #unzip()}).
  * 
- * @param <A>
- * @param <B>
+ * @param <A> first type
+ * @param <B> second type
  * @since 5.3
  */
 public interface ZippedFlow<A, B> extends FlowOperations<Tuple<A, B>, ZippedFlow<A, B>>
diff --git a/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/Orderable.java b/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/Orderable.java
index a7d084b..af3a542 100644
--- a/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/Orderable.java
+++ b/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/Orderable.java
@@ -21,7 +21,7 @@ import org.apache.tapestry5.ioc.internal.util.InternalUtils;
  * pre-requisites (objects which should be ordered earlier) and post-requisites (objects which should be ordered
  * later).
  *
- * @param <T>
+ * @param <T> the wrapped type
  */
 public class Orderable<T>
 {
diff --git a/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/ValidatingMappedConfigurationWrapper.java b/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/ValidatingMappedConfigurationWrapper.java
index 3b55bf1..b4409a7 100644
--- a/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/ValidatingMappedConfigurationWrapper.java
+++ b/tapestry-ioc/src/main/java/org/apache/tapestry5/ioc/internal/ValidatingMappedConfigurationWrapper.java
@@ -31,8 +31,8 @@ import java.util.Map;
  *
  * Handles instantiation of instances.
  * 
- * @param <K>
- * @param <V>
+ * @param <K> the key type
+ * @param <V> the value type
  */
 public class ValidatingMappedConfigurationWrapper<K, V> extends AbstractConfigurationImpl<V> implements
         MappedConfiguration<K, V>
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/NonVisual.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/NonVisual.java
index 442b452..360df36 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/NonVisual.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/NonVisual.java
@@ -24,8 +24,9 @@ import static java.lang.annotation.RetentionPolicy.RUNTIME;
 import java.lang.annotation.Target;
 
 /**
- * Marker annotation for properties which are non-visual, and so should not appear (by default) inside a {@link
- * BeanModel}. The annotation may be placed on either the getter or the setter method or on the field.
+ * Marker annotation for properties which are non-visual, and so should not appear (by default) inside a 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/beaneditor/BeanModel.html">BeanModel</a>. 
+ * The annotation may be placed on either the getter or the setter method or on the field.
  */
 @Target({ ElementType.FIELD, ElementType.METHOD })
 @Retention(RUNTIME)
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/RelativePosition.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/RelativePosition.java
index 71664f3..97942c0 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/RelativePosition.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/beaneditor/RelativePosition.java
@@ -15,7 +15,8 @@
 package org.apache.tapestry5.beaneditor;
 
 /**
- * Controls the position of newly added {@link PropertyModel}s inside a {@link BeanModel}.
+ * Controls the position of newly added <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/beanmodel/PropertyModel.html">PropertyModel</a> inside a 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/beanmodel/BeanModel.html">BeanModel</a>.
  */
 public enum RelativePosition
 {
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Autobuild.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Autobuild.java
index 8d681af..394f6fe 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Autobuild.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Autobuild.java
@@ -18,9 +18,9 @@ import java.lang.annotation.*;
 
 
 /**
- * Directs that the value to be built should be an autobuild instance of the type with injections performed, via {@link
- * org.apache.tapestry5.ioc.ObjectLocator#autobuild(Class)}. This should only be placed on a field or parameter of an
- * instantiable type (not an interface).
+ * Directs that the value to be built should be an autobuild instance of the type with injections performed, via 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/ObjectLocator.html#autobuild(Class)">ObjectLocator.html#autobuild(Class)</a>.
+ * This should only be placed on a field or parameter of an instantiable type (not an interface).
  *
  * @since 5.2
  */
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentClasses.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentClasses.java
index 4c8a10e..b4a09e5 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentClasses.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentClasses.java
@@ -21,7 +21,7 @@ import java.lang.annotation.RetentionPolicy;
 import java.lang.annotation.Target;
 
 /**
- * Marker annotation used to inject the correct {@link org.apache.tapestry5.services.InvalidationEventHub} service
+ * Marker annotation used to inject the correct <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/services/InvalidationEventHub.html">InvalidationEventHub</a> service
  * responsible for invalidations when underlying component class files are changed.
  * 
  * @since 5.1.0.0
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentLayer.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentLayer.java
index 76efacf..b58ca42 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentLayer.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/ComponentLayer.java
@@ -24,7 +24,8 @@ import static java.lang.annotation.RetentionPolicy.RUNTIME;
 
 /**
  * Marker annotation used to identify a service from the component layer that conflicts, in terms of service interface,
- * with a service from elsewhere. In particular, this is used to disambiguate {@link org.apache.tapestry5.ioc.services.PlasticProxyFactory} which has one implementation (marked with {@link
+ * with a service from elsewhere. In particular, this is used to disambiguate 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/PlasticProxyFactory.html">PlasticProxyFactory</a> which has one implementation (marked with {@link
  * org.apache.tapestry5.ioc.services.Builtin} and another with this annotation.
  */
 @Target(
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Inject.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Inject.java
index 45a5369..603cc23 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Inject.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Inject.java
@@ -30,11 +30,11 @@ import static org.apache.tapestry5.ioc.annotations.AnnotationUseContext.*;
  *
  * In terms of the IoC container; normally, resources take precedence over annotations when injecting. The Inject
  * annotation overrides this default, forcing the resolution of the parameters value via the master
- * {@link org.apache.tapestry5.ioc.ObjectProvider}, even when the parameter's type matches a type that is normally a
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/ObjectProvider.html">ObjectProvider</a>, even when the parameter's type matches a type that is normally a
  * resource.
  *
  * For service implementations, module classes, and other objects constructed via
- * {@link org.apache.tapestry5.ioc.ObjectLocator#autobuild(Class)}, this annotation indicates that an injection is
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/ObjectLocator.html#autobuild(Class)">ObjectLocator#autobuild(Class)</a>, this annotation indicates that an injection is
  * desired on the field, as with fields of a Tapestry component.
  *
  * In terms of the IoC container, the Inject annotation is only used on parameters to service builder methods (and
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/InjectService.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/InjectService.java
index 80be73c..959ec1d 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/InjectService.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/InjectService.java
@@ -27,7 +27,7 @@ import static java.lang.annotation.RetentionPolicy.RUNTIME;
  * more flexible or appropriate.
  *
  * This annotation may also be used with fields of service implementation classes, modules, or other objects constructed
- * via {@link org.apache.tapestry5.ioc.ObjectLocator#autobuild(Class)}.
+ * via <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/ObjectLocator.html#autobuild(Class)">ObjectLocator#autobuild(Class)</a>.
  */
 @Target({PARAMETER, FIELD})
 @Retention(RUNTIME)
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/IntermediateType.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/IntermediateType.java
index 86eeec2..37f1bbe 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/IntermediateType.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/IntermediateType.java
@@ -14,13 +14,17 @@
 
 package org.apache.tapestry5.ioc.annotations;
 
-import java.lang.annotation.*;
+import java.lang.annotation.Documented;
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
 
 
 /**
  * Used to guide Tapestry when coercing from a raw type to a field or parameter type, by forcing Tapestry to coerce to
  * the intermediate type.  This was introduced to allow coercion from string to a time period (in milliseconds) via
- * {@link org.apache.tapestry5.ioc.util.TimeInterval}.
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/util/TimeInterval.html">TimeInterval</a>
  *
  * @see org.apache.tapestry5.ioc.annotations.Value
  * @see org.apache.tapestry5.ioc.annotations.Symbol
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesConfiguration.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesConfiguration.java
index 2e03557..3ade6dd 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesConfiguration.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesConfiguration.java
@@ -18,8 +18,8 @@ import java.lang.annotation.*;
 
 
 /**
- * A documentation-only interface placed on service interfaces for services which have an {@linkplain
- * org.apache.tapestry5.ioc.Configuration unordered configuration}, to identify the type of contribution.
+ * A documentation-only interface placed on service interfaces for services which have an 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/Configuration.html">unordered configuration</a>, to identify the type of contribution.
  */
 @Target(ElementType.TYPE)
 @Retention(RetentionPolicy.CLASS)
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesMappedConfiguration.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesMappedConfiguration.java
index f2a1fc8..46918e5 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesMappedConfiguration.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesMappedConfiguration.java
@@ -12,11 +12,15 @@
 
 package org.apache.tapestry5.ioc.annotations;
 
-import java.lang.annotation.*;
+import java.lang.annotation.Documented;
+import java.lang.annotation.ElementType;
+import java.lang.annotation.Retention;
+import java.lang.annotation.RetentionPolicy;
+import java.lang.annotation.Target;
 
 /**
- * A documentation-only interface placed on service interfaces for services which have a {@linkplain
- * org.apache.tapestry5.ioc.MappedConfiguration mapped configuration}, to identify the type of key (often, a String),
+ * A documentation-only interface placed on service interfaces for services which have a 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/MappedConfiguration.html">mapped configuration</a>, to identify the type of key (often, a String),
  * and type of contribution.
  *
  * Remember that when the key type is String, the map will be case-insensitive.
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesOrderedConfiguration.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesOrderedConfiguration.java
index 7c608f6..724611d 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesOrderedConfiguration.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/UsesOrderedConfiguration.java
@@ -17,8 +17,8 @@ package org.apache.tapestry5.ioc.annotations;
 import java.lang.annotation.*;
 
 /**
- * A documentation-only interface placed on service interfaces for services which have an {@linkplain
- * org.apache.tapestry5.ioc.OrderedConfiguration ordered configuration}, to identify the type of contribution.
+ * A documentation-only interface placed on service interfaces for services which have an 
+ * <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/OrderedConfiguration.html">ordered configuration</a>, to identify the type of contribution.
  */
 @Target(ElementType.TYPE)
 @Retention(RetentionPolicy.CLASS)
diff --git a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Value.java b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Value.java
index 2fcee2d..d8ce50d 100644
--- a/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Value.java
+++ b/tapestry5-annotations/src/main/java/org/apache/tapestry5/ioc/annotations/Value.java
@@ -30,7 +30,7 @@ import static org.apache.tapestry5.ioc.annotations.AnnotationUseContext.*;
  * it may also be applied to field.
  *
  * @see org.apache.tapestry5.ioc.services.SymbolSource
- * @see org.apache.tapestry5.ioc.services.TypeCoercer
+ * @see <a href="https://tapestry.apache.org/current/apidocs/org/apache/tapestry5/commons/services/TypeCoecer.html">TypeCoercer</a>
  */
 @Target(
         { PARAMETER, FIELD })