Author: husted
Date: Mon Dec 3 03:30:02 2007
New Revision: 600489
URL: http://svn.apache.org/viewvc?rev=600489&view=rev
Log:
WW-1399 Minor JavaDoc update.
Modified:
struts/sandbox/trunk/jpa-mailreader/src/main/java/action/Index.java
Modified: struts/sandbox/trunk/jpa-mailreader/src/main/java/action/Index.java
URL:
http://svn.apache.org/viewvc/struts/sandbox/trunk/jpa-mailreader/src/main/java/action/Index.java?rev=600489&r1=600488&r2=600489&view=diff
==============================================================================
--- struts/sandbox/trunk/jpa-mailreader/src/main/java/action/Index.java
(original)
+++ struts/sandbox/trunk/jpa-mailreader/src/main/java/action/Index.java Mon Dec
3 03:30:02 2007
@@ -44,6 +44,14 @@
* logic is delegated to a service class for each entity. The Action classes
* interact only with the entity helpers, and not the underlying JPA.
* </p>
+ * <p>
+ * Any helper method that is not physically bound the the UI API should be
+ * moved to the service layer! In most cases, the remaining Action methods
+ * will either call ActionErrors, return a result value meaningful only to
+ * the UI, or retain state used by other Action helper methods. (A generic
+ * ServiceError class with an ActionError adapter might allow us to move
+ * 3/4 of the Action helper methods to the service layer.)
+ * </p>
*/
@Conversion(conversions = {
@TypeConversion(type = ConversionType.APPLICATION, key =
"entity.protocol.Protocol", converter =
"entity.protocol.ProtocolTypeConverter"),
