You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tapestry.apache.org by hl...@apache.org on 2008/03/02 18:21:32 UTC

svn commit: r632791 [3/8] - in /tapestry/tapestry5/trunk: tapestry-core/src/main/java/org/apache/tapestry/ tapestry-core/src/main/java/org/apache/tapestry/annotations/ tapestry-core/src/main/java/org/apache/tapestry/corelib/components/ tapestry-core/sr...

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditForm.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditForm.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditForm.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditForm.java Sun Mar  2 09:21:16 2008
@@ -15,10 +15,7 @@
 package org.apache.tapestry.corelib.components;
 
 import org.apache.tapestry.*;
-import org.apache.tapestry.annotations.Component;
-import org.apache.tapestry.annotations.InjectComponent;
-import org.apache.tapestry.annotations.Parameter;
-import org.apache.tapestry.annotations.SupportsInformalParameters;
+import org.apache.tapestry.annotations.*;
 import org.apache.tapestry.beaneditor.BeanModel;
 import org.apache.tapestry.ioc.annotations.Inject;
 import org.apache.tapestry.services.BeanModelSource;
@@ -47,6 +44,7 @@
      * The text label for the submit button of the form, by default "Create/Update".
      */
     @Parameter(value = "message:submit-label", defaultPrefix = "literal")
+    @GenerateAccessors
     private String _submitLabel;
 
     /**
@@ -57,6 +55,7 @@
      */
     @SuppressWarnings("unused")
     @Parameter(required = true)
+    @GenerateAccessors
     private Object _object;
 
     /**
@@ -95,6 +94,7 @@
      */
     @SuppressWarnings("unused")
     @Parameter
+    @GenerateAccessors
     private BeanModel _model;
 
     @Inject
@@ -114,25 +114,6 @@
         return _defaultProvider.defaultBinding("object", _resources);
     }
 
-    public void setObject(Object object)
-    {
-        _object = object;
-    }
-
-    public Object getObject()
-    {
-        return _object;
-    }
-
-    public BeanModel getModel()
-    {
-        return _model;
-    }
-
-    public void setModel(BeanModel model)
-    {
-        _model = model;
-    }
 
     void onPrepareFromForm()
     {
@@ -153,11 +134,6 @@
     public String getClientId()
     {
         return _form.getClientId();
-    }
-
-    public String getSubmitLabel()
-    {
-        return _submitLabel;
     }
 
     public void clearErrors()

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditor.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditor.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditor.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/BeanEditor.java Sun Mar  2 09:21:16 2008
@@ -18,6 +18,7 @@
 import org.apache.tapestry.ComponentAction;
 import org.apache.tapestry.ComponentResources;
 import org.apache.tapestry.annotations.Environmental;
+import org.apache.tapestry.annotations.GenerateAccessors;
 import org.apache.tapestry.annotations.Parameter;
 import org.apache.tapestry.annotations.SupportsInformalParameters;
 import org.apache.tapestry.beaneditor.BeanModel;
@@ -74,6 +75,7 @@
      * default bean model will be created from the type of the object bound to the object parameter.
      */
     @Parameter
+    @GenerateAccessors(write = false)
     private BeanModel _model;
 
     /**
@@ -83,6 +85,7 @@
      * be searched.
      */
     @Parameter(value = "componentResources")
+    @GenerateAccessors(write = false)
     private ComponentResources _overrides;
 
     @Inject
@@ -99,6 +102,7 @@
 
     // Value that change with each change to the current property:
 
+    @GenerateAccessors
     private String _propertyName;
 
     /**
@@ -109,32 +113,11 @@
         return _defaultProvider.defaultBinding("object", _resources);
     }
 
-    public String getPropertyName()
-    {
-        return _propertyName;
-    }
+    // Needed for testing as well
 
-    public void setPropertyName(String propertyName)
-    {
-        _propertyName = propertyName;
-    }
-
-    /**
-     * Returns the object being edited.
-     */
     public Object getObject()
     {
         return _object;
-    }
-
-    public ComponentResources getOverrides()
-    {
-        return _overrides;
-    }
-
-    public BeanModel getModel()
-    {
-        return _model;
     }
 
     void setupRender()

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormFragment.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormFragment.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormFragment.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormFragment.java Sun Mar  2 09:21:16 2008
@@ -34,11 +34,10 @@
 import java.util.List;
 
 /**
- * A SubForm is a portion of a Form that may be selectively displayed.  Form elements inside a FormFragment
- * will automatically bypass validation when the fragment is invisible.  The trick is to also bypass server-side
- * form processing for such fields when the form is submitted; the fragment uses a hidden field
- * to track its client-side visibility and will bypass field component submission logic for
- * the components it encloses.
+ * A SubForm is a portion of a Form that may be selectively displayed.  Form elements inside a FormFragment will
+ * automatically bypass validation when the fragment is invisible.  The trick is to also bypass server-side form
+ * processing for such fields when the form is submitted; the fragment uses a hidden field to track its client-side
+ * visibility and will bypass field component submission logic for the components it encloses.
  *
  * @see org.apache.tapestry.corelib.mixins.TriggerFragment
  */
@@ -46,26 +45,24 @@
 public class FormFragment implements ClientElement
 {
     /**
-     * Determines if the fragment is intially visible or initially invisible (the default). This is
-     * only used when rendering; when the form is submitted, the hidden field value
-     * is used to determine whether the elements within the fragment should be processed (or ignored
-     * if still invisible).
+     * Determines if the fragment is intially visible or initially invisible (the default). This is only used when
+     * rendering; when the form is submitted, the hidden field value is used to determine whether the elements within
+     * the fragment should be processed (or ignored if still invisible).
      */
     @Parameter
     private boolean _visible;
 
 
     /**
-     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked to
-     * make the fragment  visible.  If not specified, then
-     * the default "slidedown" function is used.
+     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked to make the fragment
+     * visible.  If not specified, then the default "slidedown" function is used.
      */
     @Parameter(defaultPrefix = TapestryConstants.LITERAL_BINDING_PREFIX)
     private String _show;
 
     /**
-     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked
-     * when the fragment is to be hidden. If not specified, the default "slideup" function is used.
+     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked when the fragment is to be
+     * hidden. If not specified, the default "slideup" function is used.
      */
     @Parameter(defaultPrefix = TapestryConstants.LITERAL_BINDING_PREFIX)
     private String _hide;
@@ -131,7 +128,8 @@
     }
 
     /**
-     * Renders a <div> tag and provides an override of the {@link org.apache.tapestry.services.FormSupport} environmental.
+     * Renders a <div> tag and provides an override of the {@link org.apache.tapestry.services.FormSupport}
+     * environmental.
      */
     void beginRender(MarkupWriter writer)
     {
@@ -198,7 +196,8 @@
     }
 
     /**
-     * Closes the <div> tag and pops off the {@link org.apache.tapestry.services.FormSupport} environmental override.
+     * Closes the <div> tag and pops off the {@link org.apache.tapestry.services.FormSupport} environmental
+     * override.
      *
      * @param writer
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormInjector.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormInjector.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormInjector.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/FormInjector.java Sun Mar  2 09:21:16 2008
@@ -57,8 +57,8 @@
     private InsertPosition _position;
 
     /**
-     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked to make added content visible.
-     * Leaving as null uses the default function, "highlight".
+     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked to make added content
+     * visible. Leaving as null uses the default function, "highlight".
      */
     @Parameter(defaultPrefix = TapestryConstants.LITERAL_BINDING_PREFIX)
     private String _show;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Grid.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Grid.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Grid.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Grid.java Sun Mar  2 09:21:16 2008
@@ -65,6 +65,7 @@
      * set.
      */
     @Parameter("25")
+    @GenerateAccessors
     private int _rowsPerPage;
 
     /**
@@ -80,6 +81,7 @@
      * use the property bound to the row parameter to know what they should render.
      */
     @Parameter
+    @GenerateAccessors
     private Object _row;
 
     /**
@@ -92,9 +94,9 @@
     private BeanModel _model;
 
     /**
-     * The model used to handle sorting of the Grid. This is generally not specified, and the built-in
-     * model supports only single column sorting. The sort constraints (the column that is sorted,
-     * and ascending vs. descending) is stored as persistent fields of the Grid component.
+     * The model used to handle sorting of the Grid. This is generally not specified, and the built-in model supports
+     * only single column sorting. The sort constraints (the column that is sorted, and ascending vs. descending) is
+     * stored as persistent fields of the Grid component.
      */
     @Parameter
     private GridSortModel _sortModel;
@@ -146,9 +148,11 @@
      * between CSS values (for the "zebra effect"). If null or not bound, then no particular CSS class value is used.
      */
     @Parameter(cache = false)
+    @GenerateAccessors(write = false)
     private String _rowClass;
 
     @Persist
+    @GenerateAccessors
     private int _currentPage = 1;
 
     @Persist
@@ -350,65 +354,6 @@
     public GridSortModel getSortModel()
     {
         return _sortModel;
-    }
-
-    public String getRowClass()
-    {
-        return _rowClass;
-    }
-
-    public int getCurrentPage()
-    {
-        return _currentPage;
-    }
-
-    public void setCurrentPage(int currentPage)
-    {
-        _currentPage = currentPage;
-    }
-
-    /**
-     * Returns the current row being rendered by the Grid. This property can be accessed as an alternative to binding
-     * the row parameter.
-     */
-    public Object getRow()
-    {
-        return _row;
-    }
-
-    public void setRow(Object row)
-    {
-        _row = row;
-    }
-
-    public int getRowsPerPage()
-    {
-        return _rowsPerPage;
-    }
-
-    public void setRowsPerPage(int rowsPerPage)
-    {
-        _rowsPerPage = rowsPerPage;
-    }
-
-    public boolean isSortAscending()
-    {
-        return _sortAscending;
-    }
-
-    public String getSortColumnId()
-    {
-        return _sortColumnId;
-    }
-
-    public void setSortAscending(boolean sortAscending)
-    {
-        _sortAscending = sortAscending;
-    }
-
-    public void setSortColumnId(String sortColumnId)
-    {
-        _sortColumnId = sortColumnId;
     }
 
     public Object getPagerTop()

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridCell.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridCell.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridCell.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridCell.java Sun Mar  2 09:21:16 2008
@@ -18,8 +18,8 @@
 import org.apache.tapestry.corelib.base.AbstractPropertyOutput;
 
 /**
- * Part of {@link Grid} that renders the markup inside a single data cell. GridCell is used inside a
- * pair of loops; the outer loop for each row, the inner loop for each property of the row.
+ * Part of {@link Grid} that renders the markup inside a single data cell. GridCell is used inside a pair of loops; the
+ * outer loop for each row, the inner loop for each property of the row.
  */
 public class GridCell extends AbstractPropertyOutput
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridColumns.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridColumns.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridColumns.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridColumns.java Sun Mar  2 09:21:16 2008
@@ -17,10 +17,7 @@
 import org.apache.tapestry.Asset;
 import org.apache.tapestry.Block;
 import org.apache.tapestry.ComponentResources;
-import org.apache.tapestry.annotations.Component;
-import org.apache.tapestry.annotations.Parameter;
-import org.apache.tapestry.annotations.Path;
-import org.apache.tapestry.annotations.SupportsInformalParameters;
+import org.apache.tapestry.annotations.*;
 import org.apache.tapestry.beaneditor.PropertyModel;
 import org.apache.tapestry.grid.ColumnSort;
 import org.apache.tapestry.grid.GridConstants;
@@ -83,10 +80,12 @@
     @Inject
     private Block _standardHeader;
 
+    @GenerateAccessors
     private int _columnIndex;
 
     private int _lastColumnIndex;
 
+    @GenerateAccessors(write = false)
     private PropertyModel _columnModel;
 
     void setupRender()
@@ -183,28 +182,12 @@
         return _gridModel.getDataModel().getPropertyNames();
     }
 
-    public PropertyModel getColumnModel()
-    {
-        return _columnModel;
-    }
 
     public void setColumnName(String columnName)
     {
         _columnModel = _gridModel.getDataModel().get(columnName);
     }
 
-    /**
-     * Set by the Loop component.
-     */
-    public void setColumnIndex(int columnIndex)
-    {
-        _columnIndex = columnIndex;
-    }
-
-    public int getColumnIndex()
-    {
-        return _columnIndex;
-    }
 
     public Block getBlockForColumn()
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridPager.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridPager.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridPager.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridPager.java Sun Mar  2 09:21:16 2008
@@ -29,9 +29,8 @@
 public class GridPager
 {
     /**
-     * The source of the data displayed by the grid (this is used to determine
-     * {@link GridDataSource#getAvailableRows() how many rows are available}, which in turn
-     * determines the page count).
+     * The source of the data displayed by the grid (this is used to determine {@link GridDataSource#getAvailableRows()
+     * how many rows are available}, which in turn determines the page count).
      */
     @Parameter(required = true)
     private GridDataSource _source;
@@ -49,8 +48,8 @@
     private int _currentPage;
 
     /**
-     * Number of pages before and after the current page in the range. The pager always displays
-     * links for 2 * range + 1 pages, unless that's more than the total number of available pages.
+     * Number of pages before and after the current page in the range. The pager always displays links for 2 * range + 1
+     * pages, unless that's more than the total number of available pages.
      */
     @Parameter("5")
     private int _range;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridRows.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridRows.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridRows.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/GridRows.java Sun Mar  2 09:21:16 2008
@@ -17,6 +17,7 @@
 import org.apache.tapestry.ComponentAction;
 import org.apache.tapestry.annotations.Environmental;
 import org.apache.tapestry.annotations.Parameter;
+import org.apache.tapestry.annotations.GenerateAccessors;
 import org.apache.tapestry.beaneditor.PropertyModel;
 import org.apache.tapestry.grid.GridConstants;
 import org.apache.tapestry.grid.GridDataSource;
@@ -86,6 +87,7 @@
      * container, to know what object is being rendered.
      */
     @Parameter(required = true)
+    @GenerateAccessors(write=false)
     private Object _row;
 
     /**
@@ -115,6 +117,7 @@
 
     private String _propertyName;
 
+    @GenerateAccessors(write=false)
     private PropertyModel _columnModel;
 
     public String getRowClass()
@@ -153,7 +156,7 @@
                 case DESCENDING:
                     classes.add(GridConstants.SORT_DESCENDING_CLASS);
                     break;
-                
+
                 default:
             }
         }
@@ -224,15 +227,5 @@
         _propertyName = propertyName;
 
         _columnModel = _gridModel.getDataModel().get(propertyName);
-    }
-
-    public Object getRow()
-    {
-        return _row;
-    }
-
-    public PropertyModel getColumnModel()
-    {
-        return _columnModel;
     }
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/OutputRaw.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/OutputRaw.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/OutputRaw.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/OutputRaw.java Sun Mar  2 09:21:16 2008
@@ -22,17 +22,17 @@
 import org.apache.tapestry.services.ComponentDefaultProvider;
 
 /**
- * Used to output raw markup to the client. Unlike, say, an expansion, the output from OutputRaw is
- * unfiltered, with any special characters or entities left exactly as is. This is used in
- * situations where the markup is provided externally, rather than constructed within Tapestry.
+ * Used to output raw markup to the client. Unlike, say, an expansion, the output from OutputRaw is unfiltered, with any
+ * special characters or entities left exactly as is. This is used in situations where the markup is provided
+ * externally, rather than constructed within Tapestry.
  *
  * @see MarkupWriter#writeRaw(String)
  */
 public class OutputRaw
 {
     /**
-     * The value to to render. If unbound, and a property of the container matches the component's
-     * id, then that property will be the source of the value.
+     * The value to to render. If unbound, and a property of the container matches the component's id, then that
+     * property will be the source of the value.
      */
     @Parameter(required = true)
     private String _value;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Palette.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Palette.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Palette.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Palette.java Sun Mar  2 09:21:16 2008
@@ -18,6 +18,7 @@
 import org.apache.tapestry.annotations.Environmental;
 import org.apache.tapestry.annotations.IncludeJavaScriptLibrary;
 import org.apache.tapestry.annotations.Parameter;
+import org.apache.tapestry.annotations.GenerateAccessors;
 import org.apache.tapestry.corelib.base.AbstractField;
 import org.apache.tapestry.internal.util.SelectModelRenderer;
 import org.apache.tapestry.ioc.annotations.Inject;
@@ -31,26 +32,23 @@
 import java.util.Set;
 
 /**
- * Multiple selection component. Generates a UI consisting of two <select> elements configured
- * for multiple selection; the one on the left is the list of "available" elements, the one on the
- * right is "selected". Elements can be moved between the lists by clicking a button, or double
- * clicking an option (and eventually, via drag and drop).
+ * Multiple selection component. Generates a UI consisting of two <select> elements configured for multiple
+ * selection; the one on the left is the list of "available" elements, the one on the right is "selected". Elements can
+ * be moved between the lists by clicking a button, or double clicking an option (and eventually, via drag and drop).
  * <p/>
- * The items in the available list are kept ordered as per {@link SelectModel} order. When items are
- * moved from the selected list to the available list, they items are inserted back into their
- * proper positions.
+ * The items in the available list are kept ordered as per {@link SelectModel} order. When items are moved from the
+ * selected list to the available list, they items are inserted back into their proper positions.
  * <p/>
  * The Palette may operate in normal or re-orderable mode, controlled by the reorder parameter.
  * <p/>
- * In normal mode, the items in the selected list are kept in the same "natural" order as the items
- * in the available list.
+ * In normal mode, the items in the selected list are kept in the same "natural" order as the items in the available
+ * list.
  * <p/>
- * In re-order mode, items moved to the selected list are simply added to the bottom of the list. In
- * addition, two extra buttons appear to move items up and down within the selected list.
+ * In re-order mode, items moved to the selected list are simply added to the bottom of the list. In addition, two extra
+ * buttons appear to move items up and down within the selected list.
  * <p/>
- * Much of the look and feel is driven by CSS, the default Tapestry CSS is used to set up the
- * columns, etc. By default, the &lt;select&gt; element's widths are 200px, and  it is common to override this to a specific
- * value:
+ * Much of the look and feel is driven by CSS, the default Tapestry CSS is used to set up the columns, etc. By default,
+ * the &lt;select&gt; element's widths are 200px, and  it is common to override this to a specific value:
  * <p/>
  * <pre>
  * &lt;style&gt;
@@ -58,11 +56,11 @@
  * &lt;/style&gt;
  * </pre>
  * <p/>
- * You'll want to ensure that both &lt;select&gt; in each column is the same width, otherwise the display will update poorly
- * as options are moved from one column to the other.
+ * You'll want to ensure that both &lt;select&gt; in each column is the same width, otherwise the display will update
+ * poorly as options are moved from one column to the other.
  * <p/>
- * Option groups within the {@link SelectModel} will be rendered, but are not supported by many
- * browsers, and are not fully handled on the client side.
+ * Option groups within the {@link SelectModel} will be rendered, but are not supported by many browsers, and are not
+ * fully handled on the client side.
  */
 @IncludeJavaScriptLibrary("palette.js")
 public class Palette extends AbstractField
@@ -160,6 +158,7 @@
      * The image to use for the deselect button (the default is a left pointing arrow).
      */
     @Parameter(value = "asset:deselect.png")
+    @GenerateAccessors(write=false)
     private Asset _deselect;
 
     /**
@@ -178,12 +177,14 @@
      * The image to use for the move down button (the default is a downward pointing arrow).
      */
     @Parameter(value = "asset:move_down.png")
+    @GenerateAccessors(write=false)    
     private Asset _moveDown;
 
     /**
      * The image to use for the move up button (the default is an upward pointing arrow).
      */
     @Parameter(value = "asset:move_up.png")
+    @GenerateAccessors(write=false)
     private Asset _moveUp;
 
     /**
@@ -204,27 +205,27 @@
      * The image to use for the select button (the default is a right pointing arrow).
      */
     @Parameter(value = "asset:select.png")
+    @GenerateAccessors(write=false)
     private Asset _select;
 
     /**
-     * The list of selected values from the {@link SelectModel}. This will be updated when the form
-     * is submitted. If the value for the parameter is null, a new list will be created, otherwise
-     * the existing list will be cleared. If unbound, defaults to a property of the container
-     * matching this component's id.
+     * The list of selected values from the {@link SelectModel}. This will be updated when the form is submitted. If the
+     * value for the parameter is null, a new list will be created, otherwise the existing list will be cleared. If
+     * unbound, defaults to a property of the container matching this component's id.
      */
     @Parameter(required = true)
     private List<Object> _selected;
 
     /**
-     * If true, then additional buttons are provided on the client-side to allow for re-ordering of
-     * the values.
+     * If true, then additional buttons are provided on the client-side to allow for re-ordering of the values.
      */
     @Parameter("false")
+    @GenerateAccessors(write=false)    
     private boolean _reorder;
 
     /**
-     * Used during rendering to identify the options corresponding to selected values (from the
-     * selected parameter), in the order they should be displayed on the page.
+     * Used during rendering to identify the options corresponding to selected values (from the selected parameter), in
+     * the order they should be displayed on the page.
      */
     private List<OptionModel> _selectedOptions;
 
@@ -237,8 +238,7 @@
     private int _size;
 
     /**
-     * Defaults the selected parameter to a container property whose name matches this component's
-     * id.
+     * Defaults the selected parameter to a container property whose name matches this component's id.
      */
     final Binding defaultSelected()
     {
@@ -249,27 +249,7 @@
     {
         return new AvailableRenderer();
     }
-
-    public Asset getDeselect()
-    {
-        return _deselect;
-    }
-
-    public Asset getMoveDown()
-    {
-        return _moveDown;
-    }
-
-    public Asset getMoveUp()
-    {
-        return _moveUp;
-    }
-
-    public Asset getSelect()
-    {
-        return _select;
-    }
-
+ 
     public Renderable getSelectedRenderer()
     {
         return new SelectedRenderer();
@@ -415,10 +395,5 @@
         if (_selected == null) return Collections.emptyList();
 
         return _selected;
-    }
-
-    public boolean getReorder()
-    {
-        return _reorder;
     }
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/PropertyDisplay.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/PropertyDisplay.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/PropertyDisplay.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/PropertyDisplay.java Sun Mar  2 09:21:16 2008
@@ -19,9 +19,9 @@
 import org.apache.tapestry.corelib.base.AbstractPropertyOutput;
 
 /**
- * Outputs a single property value. Overrides for individual properties come from block parameters
- * whose name matches the {@linkplain PropertyModel#getId() property id}. This component is rarely
- * used on its own, but is a critical piece of the {@link BeanDisplay} component.
+ * Outputs a single property value. Overrides for individual properties come from block parameters whose name matches
+ * the {@linkplain PropertyModel#getId() property id}. This component is rarely used on its own, but is a critical piece
+ * of the {@link BeanDisplay} component.
  */
 public class PropertyDisplay extends AbstractPropertyOutput
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/RenderObject.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/RenderObject.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/RenderObject.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/RenderObject.java Sun Mar  2 09:21:16 2008
@@ -22,10 +22,9 @@
 import org.apache.tapestry.services.ObjectRenderer;
 
 /**
- * Renders out an object using the {@link ObjectRenderer} service. Used primarily on the
- * {@link ExceptionReport} page. This is focused on objects that have a specific
- * {@link ObjectRenderer} strategy. The {@link BeanDisplay} component is used for displaying the
- * contents of arbitrary objects in terms of a series of property names and values.
+ * Renders out an object using the {@link ObjectRenderer} service. Used primarily on the {@link ExceptionReport} page.
+ * This is focused on objects that have a specific {@link ObjectRenderer} strategy. The {@link BeanDisplay} component is
+ * used for displaying the contents of arbitrary objects in terms of a series of property names and values.
  */
 public class RenderObject
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Select.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Select.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Select.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Select.java Sun Mar  2 09:21:16 2008
@@ -118,7 +118,7 @@
         return TapestryInternalUtils.isEqual(clientValue, _selectedClientValue);
     }
 
-    @SuppressWarnings({"unchecked"})
+    @SuppressWarnings({ "unchecked" })
     @Override
     protected void processSubmission(String elementName)
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Zone.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Zone.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Zone.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/components/Zone.java Sun Mar  2 09:21:16 2008
@@ -25,17 +25,18 @@
 
 
 /**
- * A Zone is portion of the output page designed for easy dynamic updating via Ajax or other
- * client-side effects.  A Zone renders out as a &lt;div&gt; element and may have content initially,
- * or may only get its content as a result of client side activity.
+ * A Zone is portion of the output page designed for easy dynamic updating via Ajax or other client-side effects.  A
+ * Zone renders out as a &lt;div&gt; element and may have content initially, or may only get its content as a result of
+ * client side activity.
  * <p/>
- * Often, Zone's are initially invisible, in which case the visible parameter may be set to false (it defaults to false).
+ * Often, Zone's are initially invisible, in which case the visible parameter may be set to false (it defaults to
+ * false).
  * <p/>
  * <p/>
- * When a user clicks an {@link org.apache.tapestry.corelib.components.ActionLink} whose zone parameter is set,
- * the corresponding client-side Tapestry.Zone object is located. It will update the content of the Zone's &lt;div&gt; and
- * then invoke either a show method (if the div is not visible) or an update method (if the div is visible).  The show and update
- * parameters are the <em>names</em> of functions attached to the Tapestry.ElementEffect object.
+ * When a user clicks an {@link org.apache.tapestry.corelib.components.ActionLink} whose zone parameter is set, the
+ * corresponding client-side Tapestry.Zone object is located. It will update the content of the Zone's &lt;div&gt; and
+ * then invoke either a show method (if the div is not visible) or an update method (if the div is visible).  The show
+ * and update parameters are the <em>names</em> of functions attached to the Tapestry.ElementEffect object.
  * <p/>
  * Renders informal parameters, adding CSS class "t-zone" and possibly, "t-invisible".
  */
@@ -43,18 +44,16 @@
 public class Zone implements ClientElement
 {
     /**
-     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked to
-     * make the Zone's &lt;div&gt; visible before being updated.  If not specified, then
-     * the basic "show" method is used.
+     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked to make the Zone's
+     * &lt;div&gt; visible before being updated.  If not specified, then the basic "show" method is used.
      */
     @Parameter(defaultPrefix = TapestryConstants.LITERAL_BINDING_PREFIX)
     private String _show;
 
     /**
-     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked
-     * after the Zone's content has been updated. If not specified, then the basic "highlight"
-     * method is used, which performs a classic "yellow fade" to indicate to the user
-     * that and update has taken place.
+     * Name of a function on the client-side Tapestry.ElementEffect object that is invoked after the Zone's content has
+     * been updated. If not specified, then the basic "highlight" method is used, which performs a classic "yellow fade"
+     * to indicate to the user that and update has taken place.
      */
     @Parameter(defaultPrefix = TapestryConstants.LITERAL_BINDING_PREFIX)
     private String _update;
@@ -68,8 +67,8 @@
     private ClientBehaviorSupport _clientBehaviorSupport;
 
     /**
-     * If true (the default) then the zone will render normally.  If false, then the "t-invisible"
-     * CSS class is added, which will make the zone initially invisible.
+     * If true (the default) then the zone will render normally.  If false, then the "t-invisible" CSS class is added,
+     * which will make the zone initially invisible.
      */
     @Parameter
     private boolean _visible = true;

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/GridPagerPosition.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/GridPagerPosition.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/GridPagerPosition.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/GridPagerPosition.java Sun Mar  2 09:21:16 2008
@@ -17,8 +17,7 @@
 import org.apache.tapestry.corelib.components.Grid;
 
 /**
- * Used by the {@link Grid} component to control where the pager portion of the Grid should be
- * displayed.
+ * Used by the {@link Grid} component to control where the pager portion of the Grid should be displayed.
  */
 public enum GridPagerPosition
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/InsertPosition.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/InsertPosition.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/InsertPosition.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/data/InsertPosition.java Sun Mar  2 09:21:16 2008
@@ -17,10 +17,12 @@
 public enum InsertPosition
 {
     /**
-     * Insert the new content above (i.e., before) the insertion position. 
+     * Insert the new content above (i.e., before) the insertion position.
      */
     ABOVE,
 
-    /** Insert the new context below (i.e., after) the insertion position. */
+    /**
+     * Insert the new context below (i.e., after) the insertion position.
+     */
     BELOW;
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/internal/FormSupportAdapter.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/internal/FormSupportAdapter.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/internal/FormSupportAdapter.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/internal/FormSupportAdapter.java Sun Mar  2 09:21:16 2008
@@ -19,8 +19,8 @@
 import org.apache.tapestry.services.FormSupport;
 
 /**
- * An implementation of {@link org.apache.tapestry.services.FormSupport} that delegates all behavior
- * to another instance of FormSupport. This allows some of the behavior to be overridden easily.
+ * An implementation of {@link org.apache.tapestry.services.FormSupport} that delegates all behavior to another instance
+ * of FormSupport. This allows some of the behavior to be overridden easily.
  */
 public class FormSupportAdapter implements FormSupport
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/Autocomplete.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/Autocomplete.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/Autocomplete.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/Autocomplete.java Sun Mar  2 09:21:16 2008
@@ -33,17 +33,16 @@
  * A mixin for a text field that allows for autocompletion of text fields. This is based on Prototype's autocompleter
  * control.
  * <p/>
- * The mixin renders an (initially invisible) progress indicator after the field (it will also be after
- * the error icon most fields render). The progress indicator is made visible during the request to the server.
- * The mixin then renders a &lt;div&gt; that will be filled in on the client
- * side with dynamically obtained selections.
+ * The mixin renders an (initially invisible) progress indicator after the field (it will also be after the error icon
+ * most fields render). The progress indicator is made visible during the request to the server. The mixin then renders
+ * a &lt;div&gt; that will be filled in on the client side with dynamically obtained selections.
  * <p/>
- * Multiple selection on the client is enabled by binding the tokens parameter (however, the mixin doesn't
- * help split multiple selections up on the server, that is still your code's responsibility).
+ * Multiple selection on the client is enabled by binding the tokens parameter (however, the mixin doesn't help split
+ * multiple selections up on the server, that is still your code's responsibility).
  * <p/>
- * The container is responsible for providing an event handler for event "providecompletions".  The context
- * will be the partial input string sent from the client.  The return value should be an array or list of
- * completions, in presentation order.  I.e.
+ * The container is responsible for providing an event handler for event "providecompletions".  The context will be the
+ * partial input string sent from the client.  The return value should be an array or list of completions, in
+ * presentation order.  I.e.
  * <p/>
  * <pre>
  * String[] onProvideCompletionsFromMyField(String input)
@@ -95,22 +94,22 @@
 
 
     /**
-     * Overrides the default check frequency for determining whether to send a server request.
-     * The default is .4 seconds.
+     * Overrides the default check frequency for determining whether to send a server request. The default is .4
+     * seconds.
      */
     @Parameter(defaultPrefix = "literal")
     private double _frequency;
 
     /**
-     * If given, then the autocompleter will support multiple input values, seperated by
-     * any of the individual characters in the string.
+     * If given, then the autocompleter will support multiple input values, seperated by any of the individual
+     * characters in the string.
      */
     @Parameter(defaultPrefix = "literal")
     private String _tokens;
 
     /**
-     * Mixin afterRender phrase occurs after the component itself. This is where we write the &lt;div&gt;
-     * element and the JavaScript.
+     * Mixin afterRender phrase occurs after the component itself. This is where we write the &lt;div&gt; element and
+     * the JavaScript.
      *
      * @param writer
      */
@@ -189,7 +188,7 @@
             }
         };
 
-        _resources.triggerEvent("providecompletions", new Object[]{input}, callback);
+        _resources.triggerEvent("providecompletions", new Object[] { input }, callback);
 
         ContentType contentType = _responseRenderer.findContentType(this);
 
@@ -201,9 +200,9 @@
     }
 
     /**
-     * Invoked to allow subclasses to further configure the parameters passed to the JavaScript
-     * Ajax.Autocompleter options. The values minChars, frequency and tokens my be pre-configured.
-     * Subclasses may override this method to configure additional features of the Ajax.Autocompleter.
+     * Invoked to allow subclasses to further configure the parameters passed to the JavaScript Ajax.Autocompleter
+     * options. The values minChars, frequency and tokens my be pre-configured. Subclasses may override this method to
+     * configure additional features of the Ajax.Autocompleter.
      * <p/>
      * <p/>
      * This implementation does nothing.
@@ -215,9 +214,9 @@
     }
 
     /**
-     * Generates the markup response that will be returned to the client; this should be an
-     * &lt;ul&gt; element with nested &lt;li&gt; elements. Subclasses may override this to
-     * produce more involved markup (including images and CSS class attributes).
+     * Generates the markup response that will be returned to the client; this should be an &lt;ul&gt; element with
+     * nested &lt;li&gt; elements. Subclasses may override this to produce more involved markup (including images and
+     * CSS class attributes).
      *
      * @param writer  to write the list to
      * @param matches list of matching objects, each should be converted to a string

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/DiscardBody.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/DiscardBody.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/DiscardBody.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/DiscardBody.java Sun Mar  2 09:21:16 2008
@@ -18,10 +18,9 @@
 import org.apache.tapestry.annotations.MixinAfter;
 
 /**
- * Discards a component's body. Returns false from the {@link BeforeRenderBody} phase, which
- * prevents the rendering of the body. Set up as a "MixinAfter" so that components can render their
- * an alternative body if they so desire before this mixin cancels the normal body (from the
- * container's template).
+ * Discards a component's body. Returns false from the {@link BeforeRenderBody} phase, which prevents the rendering of
+ * the body. Set up as a "MixinAfter" so that components can render their an alternative body if they so desire before
+ * this mixin cancels the normal body (from the container's template).
  */
 @MixinAfter
 public class DiscardBody

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/RenderInformals.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/RenderInformals.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/RenderInformals.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/RenderInformals.java Sun Mar  2 09:21:16 2008
@@ -20,15 +20,16 @@
 import org.apache.tapestry.ioc.annotations.Inject;
 
 /**
- * Used to render out all informal parameters, at the end of the {@link org.apache.tapestry.annotations.BeginRender} phase.
+ * Used to render out all informal parameters, at the end of the {@link org.apache.tapestry.annotations.BeginRender}
+ * phase.
  * <p/>
- * This mixin can be used with components that render a single tag inside the {@link BeginRender}
- * phase. RenderInformals will activate during the PostBeginRender phase to write additional
- * attributes, from the informal parameters, into the active element.
+ * This mixin can be used with components that render a single tag inside the {@link BeginRender} phase. RenderInformals
+ * will activate during the PostBeginRender phase to write additional attributes, from the informal parameters, into the
+ * active element.
  * <p/>
- * If you want this behavior, but need to render more than a single tag, then implement render phase
- * methods for the {@link BeforeRenderTemplate} and {@link AfterRenderTemplate} phases. Use those
- * phases to write the additional elements and close them.
+ * If you want this behavior, but need to render more than a single tag, then implement render phase methods for the
+ * {@link BeforeRenderTemplate} and {@link AfterRenderTemplate} phases. Use those phases to write the additional
+ * elements and close them.
  * <p/>
  * This is often used as a base class, for cases where a component doesn't have other mixins.
  */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/TriggerFragment.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/TriggerFragment.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/TriggerFragment.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/mixins/TriggerFragment.java Sun Mar  2 09:21:16 2008
@@ -23,9 +23,8 @@
 import org.apache.tapestry.services.Heartbeat;
 
 /**
- * A mixin that can be applied to a Checkbox component that will link
- * the checkbox and a FormFragment, making the Checkbox control the client-side visibility
- * of the FormFragment.
+ * A mixin that can be applied to a Checkbox component that will link the checkbox and a FormFragment, making the
+ * Checkbox control the client-side visibility of the FormFragment.
  *
  * @see org.apache.tapestry.corelib.components.Checkbox
  * @see org.apache.tapestry.corelib.components.FormFragment

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ExceptionReport.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ExceptionReport.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ExceptionReport.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ExceptionReport.java Sun Mar  2 09:21:16 2008
@@ -16,6 +16,7 @@
 
 import org.apache.tapestry.TapestryConstants;
 import org.apache.tapestry.annotations.ContentType;
+import org.apache.tapestry.annotations.GenerateAccessors;
 import org.apache.tapestry.ioc.annotations.Inject;
 import org.apache.tapestry.ioc.annotations.Symbol;
 import org.apache.tapestry.services.ExceptionReporter;
@@ -32,15 +33,19 @@
 @ContentType("text/html")
 public class ExceptionReport implements ExceptionReporter
 {
+    @GenerateAccessors
     private String _attributeName;
 
     @Inject
+    @GenerateAccessors
     private Request _request;
 
     @Inject
     @Symbol(TapestryConstants.PRODUCTION_MODE_SYMBOL)
+    @GenerateAccessors
     private boolean _productionMode;
 
+    @GenerateAccessors
     private Throwable _rootException;
 
     public void reportException(Throwable exception)
@@ -53,38 +58,13 @@
         return _request.getSession(false) != null;
     }
 
-    public Request getRequest()
-    {
-        return _request;
-    }
-
     public Session getSession()
     {
         return _request.getSession(false);
     }
 
-    public String getAttributeName()
-    {
-        return _attributeName;
-    }
-
-    public void setAttributeName(String attributeName)
-    {
-        _attributeName = attributeName;
-    }
-
     public Object getAttributeValue()
     {
         return getSession().getAttribute(_attributeName);
-    }
-
-    public boolean isProductionMode()
-    {
-        return _productionMode;
-    }
-
-    public Throwable getRootException()
-    {
-        return _rootException;
     }
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ServiceStatus.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ServiceStatus.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ServiceStatus.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/corelib/pages/ServiceStatus.java Sun Mar  2 09:21:16 2008
@@ -15,6 +15,7 @@
 package org.apache.tapestry.corelib.pages;
 
 import org.apache.tapestry.ComponentResources;
+import org.apache.tapestry.annotations.GenerateAccessors;
 import org.apache.tapestry.annotations.Meta;
 import org.apache.tapestry.beaneditor.BeanModel;
 import org.apache.tapestry.ioc.Registry;
@@ -38,13 +39,16 @@
     @Inject
     private ServiceActivityScoreboard _scoreboard;
 
+    @GenerateAccessors
     private List<ServiceActivity> _activity;
 
+    @GenerateAccessors
     private ServiceActivity _row;
 
     @Inject
     private BeanModelSource _source;
 
+    @GenerateAccessors
     private final BeanModel _model;
 
     @Inject
@@ -62,29 +66,8 @@
         _model.reorder("serviceId", "serviceInterface", "scope", "status");
     }
 
-    public ServiceActivity getRow()
-    {
-        return _row;
-    }
-
-    public void setRow(ServiceActivity row)
-    {
-        _row = row;
-    }
-
     void setupRender()
     {
         _activity = _scoreboard.getServiceActivity();
     }
-
-    public List<ServiceActivity> getActivity()
-    {
-        return _activity;
-    }
-
-    public BeanModel getModel()
-    {
-        return _model;
-    }
-
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/CData.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/CData.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/CData.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/CData.java Sun Mar  2 09:21:16 2008
@@ -17,9 +17,8 @@
 import java.io.PrintWriter;
 
 /**
- * A node that stores parsed character content (CDATA).  For XML documents (as per {@link MarkupModel#isXML()},
- * this will be writtens as a CDATA section.
- * For non-XML documents, the content is filtered as it is written out.
+ * A node that stores parsed character content (CDATA).  For XML documents (as per {@link MarkupModel#isXML()}, this
+ * will be writtens as a CDATA section. For non-XML documents, the content is filtered as it is written out.
  */
 public class CData extends Node
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DTD.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DTD.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DTD.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DTD.java Sun Mar  2 09:21:16 2008
@@ -17,8 +17,8 @@
 import java.io.PrintWriter;
 
 /**
- * Representation of a document type. Note that technically, a Doctype isn't a node in an xml
- * document; hence this doesn't extend node.
+ * Representation of a document type. Note that technically, a Doctype isn't a node in an xml document; hence this
+ * doesn't extend node.
  */
 public class DTD
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DefaultMarkupModel.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DefaultMarkupModel.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DefaultMarkupModel.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/DefaultMarkupModel.java Sun Mar  2 09:21:16 2008
@@ -19,9 +19,9 @@
 import java.util.Set;
 
 /**
- * Default implementation of {@link org.apache.tapestry.dom.MarkupModel} that is appropriate for
- * traditional HTML markup. This conforms to the SGML HTML definition, including some things that
- * are not well formed XML-style markup. Assumes that all tags are lower-case.
+ * Default implementation of {@link org.apache.tapestry.dom.MarkupModel} that is appropriate for traditional HTML
+ * markup. This conforms to the SGML HTML definition, including some things that are not well formed XML-style markup.
+ * Assumes that all tags are lower-case.
  */
 public class DefaultMarkupModel implements MarkupModel
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Element.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Element.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Element.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Element.java Sun Mar  2 09:21:16 2008
@@ -27,8 +27,8 @@
 import java.util.Map;
 
 /**
- * An element that will render with a begin tag and attributes, a body, and an end tag. Also acts as
- * a factory for enclosed Element, Text and Comment nodes.
+ * An element that will render with a begin tag and attributes, a body, and an end tag. Also acts as a factory for
+ * enclosed Element, Text and Comment nodes.
  * <p/>
  * TODO: Support for CDATA nodes. Do we need Entity nodes?
  */
@@ -73,9 +73,9 @@
     private static final String CLASS_ATTRIBUTE = "class";
 
     /**
-     * URI of the namespace which contains the element.  A quirk in XML is that the element may be in a namespace
-     * it defines itself, so resolving the namespace to a prefix must wait until render time (since the Element is
-     * created before the namespaces for it are defined).
+     * URI of the namespace which contains the element.  A quirk in XML is that the element may be in a namespace it
+     * defines itself, so resolving the namespace to a prefix must wait until render time (since the Element is created
+     * before the namespaces for it are defined).
      */
     private final String _namespace;
 
@@ -113,8 +113,7 @@
     }
 
     /**
-     * Returns the containing element for this element. This will be null for the root element of a
-     * document.
+     * Returns the containing element for this element. This will be null for the root element of a document.
      */
     public Element getParent()
     {
@@ -125,8 +124,8 @@
      * Adds an attribute to the element, but only if the attribute name does not already exist.
      *
      * @param name  the name of the attribute to add
-     * @param value the value for the attribute. A value of null is allowed, and no attribute will be
-     *              added to the element.
+     * @param value the value for the attribute. A value of null is allowed, and no attribute will be added to the
+     *              element.
      */
     public void attribute(String name, String value)
     {
@@ -138,8 +137,8 @@
      *
      * @param namespace the namespace to contain the attribute, or null
      * @param name      the name of the attribute to add
-     * @param value     the value for the attribute. A value of null is allowed, and no attribute will be
-     *                  added to the element.
+     * @param value     the value for the attribute. A value of null is allowed, and no attribute will be added to the
+     *                  element.
      */
     public void attribute(String namespace, String name, String value)
     {
@@ -171,8 +170,7 @@
     }
 
     /**
-     * Forces changes to a number of attributes. The new attributes <em>overwrite</em> previous
-     * values.
+     * Forces changes to a number of attributes. The new attributes <em>overwrite</em> previous values.
      */
     public void forceAttributes(String... namesAndValues)
     {
@@ -259,8 +257,8 @@
     }
 
     /**
-     * Adds and returns a new text node (the text node is returned so that
-     * {@link Text#write(String)} or [@link {@link Text#writef(String, Object[])} may be invoked .
+     * Adds and returns a new text node (the text node is returned so that {@link Text#write(String)} or [@link {@link
+     * Text#writef(String, Object[])} may be invoked .
      *
      * @param text initial text for the node
      * @return the new Text node
@@ -371,8 +369,8 @@
     }
 
     /**
-     * Tries to find an element under this element (including itself) whose id is specified.
-     * Performs a width-first search of the document tree.
+     * Tries to find an element under this element (including itself) whose id is specified. Performs a width-first
+     * search of the document tree.
      *
      * @param id the value of the id attribute of the element being looked for
      * @return the element if found. null if not found.
@@ -407,8 +405,8 @@
     }
 
     /**
-     * Searchs for a child element with a particular name below this element. The path parameter is
-     * a slash separated series of element names.
+     * Searchs for a child element with a particular name below this element. The path parameter is a slash separated
+     * series of element names.
      *
      * @param path
      * @return
@@ -465,9 +463,8 @@
     }
 
     /**
-     * Adds one or more CSS class names to the "class" attribute. No check
-     * for duplicates is made. Note that CSS class names are case insensitive
-     * on the client.
+     * Adds one or more CSS class names to the "class" attribute. No check for duplicates is made. Note that CSS class
+     * names are case insensitive on the client.
      *
      * @param className one or more CSS class names
      * @return the element for further configuration
@@ -504,10 +501,9 @@
     }
 
     /**
-     * Defines a namespace for this element, mapping a URI to a prefix.   This will affect
-     * how namespaced elements and attributes nested within the element are rendered, and will
-     * also cause <code>xmlns:</code> attributes (to define the namespace and prefix) to
-     * be rendered.
+     * Defines a namespace for this element, mapping a URI to a prefix.   This will affect how namespaced elements and
+     * attributes nested within the element are rendered, and will also cause <code>xmlns:</code> attributes (to define
+     * the namespace and prefix) to be rendered.
      *
      * @param namespace       URI of the namespace
      * @param namespacePrefix prefix

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/EndTagStyle.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/EndTagStyle.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/EndTagStyle.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/EndTagStyle.java Sun Mar  2 09:21:16 2008
@@ -15,8 +15,8 @@
 package org.apache.tapestry.dom;
 
 /**
- * Part of a {@link MarkupModel}, used to define how end tags are handled when the {@link Document}
- * in rendered out as a text stream.
+ * Part of a {@link MarkupModel}, used to define how end tags are handled when the {@link Document} in rendered out as a
+ * text stream.
  */
 public enum EndTagStyle
 {
@@ -30,8 +30,8 @@
      */
     REQUIRE,
     /**
-     * Require an end tag, but abbreviate it if the element has no children. This is the only value
-     * used in XML documents.
+     * Require an end tag, but abbreviate it if the element has no children. This is the only value used in XML
+     * documents.
      */
     ABBREVIATE
 }

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/MarkupModel.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/MarkupModel.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/MarkupModel.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/MarkupModel.java Sun Mar  2 09:21:16 2008
@@ -15,17 +15,17 @@
 package org.apache.tapestry.dom;
 
 /**
- * Used by a the DOM to determine how to produce markup. Delegates details about converted entities
- * and some formatting details.  This exists to handle the differences between traditional HTML
- * output (which is SGML based, meaning there can be elements that are valid without a close tag) and
- * "modern" XML, such as XHTML.  Generally speaking, for XHTML it is vital that a !DOCTYPE be included
- * in the rendered response, or the browser will be unable to display the result properly.
+ * Used by a the DOM to determine how to produce markup. Delegates details about converted entities and some formatting
+ * details.  This exists to handle the differences between traditional HTML output (which is SGML based, meaning there
+ * can be elements that are valid without a close tag) and "modern" XML, such as XHTML.  Generally speaking, for XHTML
+ * it is vital that a !DOCTYPE be included in the rendered response, or the browser will be unable to display the result
+ * properly.
  */
 public interface MarkupModel
 {
     /**
-     * Encodes the characters into the buffer, converting control characters (such as '&lt;') into
-     * corresponding entities (such as &amp;lt;).
+     * Encodes the characters into the buffer, converting control characters (such as '&lt;') into corresponding
+     * entities (such as &amp;lt;).
      *
      * @param content to be filtered
      * @param buffer  to receive the filtered content
@@ -33,9 +33,9 @@
     void encode(String content, StringBuilder buffer);
 
     /**
-     * Encodes the characters into the buffer for use in a quoted value (that is, an attribute
-     * value), converting control characters (such as '&lt;') into corresponding entities (such as
-     * &amp;lt;). In addition, double quotes must be quoted or otherwise escaped.
+     * Encodes the characters into the buffer for use in a quoted value (that is, an attribute value), converting
+     * control characters (such as '&lt;') into corresponding entities (such as &amp;lt;). In addition, double quotes
+     * must be quoted or otherwise escaped.
      *
      * @param content to be filtered
      * @param buffer  to receive the filtered content
@@ -48,9 +48,8 @@
     EndTagStyle getEndTagStyle(String element);
 
     /**
-     * Returns true if the document markup is XML, which is used to determine
-     * the need for an XML declaration at the start of the document,
-     * and whether CDATA sections are supported.
+     * Returns true if the document markup is XML, which is used to determine the need for an XML declaration at the
+     * start of the document, and whether CDATA sections are supported.
      *
      * @return true for XML output, false for HTML (SGML) output
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Node.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Node.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Node.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Node.java Sun Mar  2 09:21:16 2008
@@ -31,8 +31,8 @@
     private List<Node> _children;
 
     /**
-     * Creates a new node, setting its container to the provided value. Container may also be null,
-     * but that is only used for Document nodes (the topmost node of a DOM).
+     * Creates a new node, setting its container to the provided value. Container may also be null, but that is only
+     * used for Document nodes (the topmost node of a DOM).
      *
      * @param container
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Raw.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Raw.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Raw.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/dom/Raw.java Sun Mar  2 09:21:16 2008
@@ -17,8 +17,7 @@
 import java.io.PrintWriter;
 
 /**
- * A specialized node in the document tree that contains raw markup to be printed to the client
- * exactly as-is.
+ * A specialized node in the document tree that contains raw markup to be printed to the client exactly as-is.
  */
 public final class Raw extends Node
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalComponentResourcesCommon.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalComponentResourcesCommon.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalComponentResourcesCommon.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalComponentResourcesCommon.java Sun Mar  2 09:21:16 2008
@@ -21,9 +21,8 @@
 import java.util.Map;
 
 /**
- * Operations shared by {@link InternalComponentResources} and {@link ComponentPageElement}.
- * Typically, these means methods of InternalComponentResources that are delegated to the component
- * page element.
+ * Operations shared by {@link InternalComponentResources} and {@link ComponentPageElement}. Typically, these means
+ * methods of InternalComponentResources that are delegated to the component page element.
  */
 public interface InternalComponentResourcesCommon
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalConstants.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalConstants.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalConstants.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/InternalConstants.java Sun Mar  2 09:21:16 2008
@@ -77,9 +77,8 @@
     public static final String IMMEDIATE_RESPONSE_PAGE_ATTRIBUTE = "tapestry.immediate-response-page";
 
     /**
-     * Required MIME type for JSON responses. If this MIME type is not used, the client-side
-     * Prototype code will not recognize the response as JSON, and the Ajax.Response.responseJSON property
-     * will be null.
+     * Required MIME type for JSON responses. If this MIME type is not used, the client-side Prototype code will not
+     * recognize the response as JSON, and the Ajax.Response.responseJSON property will be null.
      */
     public static final String JSON_MIME_TYPE = "application/json";
 

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/SyntheticModuleDef.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/SyntheticModuleDef.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/SyntheticModuleDef.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/SyntheticModuleDef.java Sun Mar  2 09:21:16 2008
@@ -24,8 +24,7 @@
 import java.util.Set;
 
 /**
- * A synthetic module definition, used to mix in some additional "pre-built" service configuration
- * contributions.
+ * A synthetic module definition, used to mix in some additional "pre-built" service configuration contributions.
  */
 public class SyntheticModuleDef implements ModuleDef
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/TapestryAppInitializer.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/TapestryAppInitializer.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/TapestryAppInitializer.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/TapestryAppInitializer.java Sun Mar  2 09:21:16 2008
@@ -25,14 +25,13 @@
 import org.apache.tapestry.services.TapestryModule;
 
 /**
- * This class is used to build the {@link Registry}. The Registry contains
- * {@link org.apache.tapestry.ioc.services.TapestryIOCModule} and {@link TapestryModule}, any
- * modules identified by {@link #addModules(Class[])} )}, plus the application module.
+ * This class is used to build the {@link Registry}. The Registry contains {@link org.apache.tapestry.ioc.services.TapestryIOCModule}
+ * and {@link TapestryModule}, any modules identified by {@link #addModules(Class[])} )}, plus the application module.
  * <p/>
  * The application module is optional.
  * <p/>
- * The application module is identified as <em>package</em>.services.<em>appName</em>Module,
- * where <em>package</em> and the <em>appName</em> are specified by the caller.
+ * The application module is identified as <em>package</em>.services.<em>appName</em>Module, where <em>package</em> and
+ * the <em>appName</em> are specified by the caller.
  */
 public class TapestryAppInitializer
 {
@@ -54,8 +53,7 @@
     }
 
     /**
-     * @param appProvider provides symbols for the application (normally, from the ServletContext init
-     *                    parameters)
+     * @param appProvider provides symbols for the application (normally, from the ServletContext init parameters)
      * @param appName     the name of the application (i.e., the name of the application servlet)
      * @param aliasMode   the mode, used by the {@link Alias} service, normally "servlet"
      */

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/URLEventContext.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/URLEventContext.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/URLEventContext.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/URLEventContext.java Sun Mar  2 09:21:16 2008
@@ -18,9 +18,8 @@
 import org.apache.tapestry.services.ContextValueEncoder;
 
 /**
- * Implementation based on values extracted from the URL (an event context, or a page activation context)
- * that uses a {@link org.apache.tapestry.services.ContextValueEncoder} to convert from string values
- * to the desired values.
+ * Implementation based on values extracted from the URL (an event context, or a page activation context) that uses a
+ * {@link org.apache.tapestry.services.ContextValueEncoder} to convert from string values to the desired values.
  */
 public class URLEventContext implements EventContext
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/beaneditor/PrimitiveFieldConstraintGenerator.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/beaneditor/PrimitiveFieldConstraintGenerator.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/beaneditor/PrimitiveFieldConstraintGenerator.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/beaneditor/PrimitiveFieldConstraintGenerator.java Sun Mar  2 09:21:16 2008
@@ -21,8 +21,7 @@
 import java.util.List;
 
 /**
- * Adds a "required" constraint for any property of whose type is a primitive (not a wrapper or
- * reference) type.
+ * Adds a "required" constraint for any property of whose type is a primitive (not a wrapper or reference) type.
  */
 public class PrimitiveFieldConstraintGenerator implements ValidationConstraintGenerator
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/AbstractBinding.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/AbstractBinding.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/AbstractBinding.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/AbstractBinding.java Sun Mar  2 09:21:16 2008
@@ -22,8 +22,8 @@
 import java.lang.annotation.Annotation;
 
 /**
- * Abstract base class for bindings. Assumes that the binding is read only and invariant. Subclasses
- * must provide an implementation of {@link Binding#get()}.
+ * Abstract base class for bindings. Assumes that the binding is read only and invariant. Subclasses must provide an
+ * implementation of {@link Binding#get()}.
  */
 public abstract class AbstractBinding extends BaseLocatable implements Binding
 {
@@ -46,8 +46,8 @@
     }
 
     /**
-     * Returns true. Subclasses that do not supply a fixed, read-only value should override this
-     * method to return false.
+     * Returns true. Subclasses that do not supply a fixed, read-only value should override this method to return
+     * false.
      */
     public boolean isInvariant()
     {
@@ -55,8 +55,8 @@
     }
 
     /**
-     * Returns the actual class, by invoking {@link Binding#get()}. Subclasses may override this
-     * method to work more efficiently (say, when the binding type is known statically).
+     * Returns the actual class, by invoking {@link Binding#get()}. Subclasses may override this method to work more
+     * efficiently (say, when the binding type is known statically).
      */
     public Class getBindingType()
     {
@@ -64,8 +64,8 @@
     }
 
     /**
-     * Always returns null. Bindings that provide access to a method or field will override this
-     * method to return the appropriate annotation.
+     * Always returns null. Bindings that provide access to a method or field will override this method to return the
+     * appropriate annotation.
      */
     public <T extends Annotation> T getAnnotation(Class<T> annotationClass)
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/MessageBindingFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/MessageBindingFactory.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/MessageBindingFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/MessageBindingFactory.java Sun Mar  2 09:21:16 2008
@@ -20,8 +20,8 @@
 import org.apache.tapestry.services.BindingFactory;
 
 /**
- * Implementation of the message: binding prefix -- we simply get the message key and store it
- * inside at {@link LiteralBinding}.
+ * Implementation of the message: binding prefix -- we simply get the message key and store it inside at {@link
+ * LiteralBinding}.
  */
 public class MessageBindingFactory implements BindingFactory
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBinding.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBinding.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBinding.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBinding.java Sun Mar  2 09:21:16 2008
@@ -21,9 +21,8 @@
 import java.lang.annotation.Annotation;
 
 /**
- * Base class for bindings created by the
- * {@link org.apache.tapestry.internal.bindings.PropBindingFactory}. A subclass of this is created
- * at runtime.
+ * Base class for bindings created by the {@link org.apache.tapestry.internal.bindings.PropBindingFactory}. A subclass
+ * of this is created at runtime.
  */
 public class PropBinding extends AbstractBinding
 {
@@ -44,8 +43,8 @@
     }
 
     /**
-     * The default implementation of get() will throw a TapestryException (binding is write only).
-     * The fabricated subclass <em>may</em> override this method (as well as set()).
+     * The default implementation of get() will throw a TapestryException (binding is write only). The fabricated
+     * subclass <em>may</em> override this method (as well as set()).
      */
     public Object get()
     {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBindingFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBindingFactory.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBindingFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/PropBindingFactory.java Sun Mar  2 09:21:16 2008
@@ -25,8 +25,8 @@
 /**
  * Binding factory for reading and updating JavaBean properties.
  * <p/>
- * Expression are evaluated via a {@link PropertyConduit}, which is generated by
- * {@link PropertyConduitSource} (which therefore defines the expression language).
+ * Expression are evaluated via a {@link PropertyConduit}, which is generated by {@link PropertyConduitSource} (which
+ * therefore defines the expression language).
  */
 public class PropBindingFactory implements BindingFactory
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/TranslateBindingFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/TranslateBindingFactory.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/TranslateBindingFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/TranslateBindingFactory.java Sun Mar  2 09:21:16 2008
@@ -22,8 +22,7 @@
 import org.apache.tapestry.services.TranslatorSource;
 
 /**
- * Interprets the binding expression as the name of a {@link Translator} provided by the
- * {@link TranslatorSource}.
+ * Interprets the binding expression as the name of a {@link Translator} provided by the {@link TranslatorSource}.
  */
 public class TranslateBindingFactory implements BindingFactory
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/ValidateBindingFactory.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/ValidateBindingFactory.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/ValidateBindingFactory.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/bindings/ValidateBindingFactory.java Sun Mar  2 09:21:16 2008
@@ -24,8 +24,8 @@
 import org.apache.tapestry.services.FieldValidatorSource;
 
 /**
- * Factory for bindings that provide a {@link FieldValidator} based on a validator specification.
- * This binding factory is only useable with components that implement the {@link Field} interface.
+ * Factory for bindings that provide a {@link FieldValidator} based on a validator specification. This binding factory
+ * is only useable with components that implement the {@link Field} interface.
  */
 public class ValidateBindingFactory implements BindingFactory
 {

Modified: tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/event/InvalidationEventHub.java
URL: http://svn.apache.org/viewvc/tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/event/InvalidationEventHub.java?rev=632791&r1=632790&r2=632791&view=diff
==============================================================================
--- tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/event/InvalidationEventHub.java (original)
+++ tapestry/tapestry5/trunk/tapestry-core/src/main/java/org/apache/tapestry/internal/event/InvalidationEventHub.java Sun Mar  2 09:21:16 2008
@@ -17,11 +17,10 @@
 import org.apache.tapestry.internal.events.InvalidationListener;
 
 /**
- * An object which manages a list of
- * {@link org.apache.tapestry.internal.events.InvalidationListener}s.
+ * An object which manages a list of {@link org.apache.tapestry.internal.events.InvalidationListener}s.
  * <p/>
- * TODO: This interface need to move to the public side (as it is extended by other public
- * interfaces), or we need to come up with an alternate mechanism for propogating invalidation data.
+ * TODO: This interface need to move to the public side (as it is extended by other public interfaces), or we need to
+ * come up with an alternate mechanism for propogating invalidation data.
  */
 public interface InvalidationEventHub
 {