You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@myfaces.apache.org by ma...@apache.org on 2009/08/06 11:21:11 UTC

svn commit: r801555 - /myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/context/AccessibilityProfile.java

Author: matzew
Date: Thu Aug  6 09:21:10 2009
New Revision: 801555

URL: http://svn.apache.org/viewvc?rev=801555&view=rev
Log:
TRINIDAD-1543 - AccessibilityProfile is not Serializable

Thanks to Dave Robinson  for the patch!

Modified:
    myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/context/AccessibilityProfile.java

Modified: myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/context/AccessibilityProfile.java
URL: http://svn.apache.org/viewvc/myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/context/AccessibilityProfile.java?rev=801555&r1=801554&r2=801555&view=diff
==============================================================================
--- myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/context/AccessibilityProfile.java (original)
+++ myfaces/trinidad/trunk/trinidad-api/src/main/java/org/apache/myfaces/trinidad/context/AccessibilityProfile.java Thu Aug  6 09:21:10 2009
@@ -18,16 +18,18 @@
  */
 package org.apache.myfaces.trinidad.context;
 
+import java.io.Serializable;
+
 /**
  * Specifies a set of accessibility-related properties that are applied
  * to the current request.
- * 
- * AccessibilityProfile instances are obtained by the getInstance() 
+ *
+ * AccessibilityProfile instances are obtained by the getInstance()
  * factory method.
- * 
+ *
  * AccessibilityProfile instances are immutable.
  */
-public class AccessibilityProfile
+public class AccessibilityProfile 
 {
   /**
    * Color contrast values
@@ -82,7 +84,7 @@
   {    
     // Note: we could cache and share AccessibilityProfile instances
     // here if that seems useful.
-    return new AccessibilityProfile(colorContrast, fontSize);
+    return new SerializableAccessibilityProfile(colorContrast, fontSize);
   }
 
   /**
@@ -132,15 +134,31 @@
     FontSize      fontSize
     )
   {
-
     _colorContrast = (colorContrast == null) ? ColorContrast.STANDARD : colorContrast;
     _fontSize = (fontSize == null) ? FontSize.MEDIUM : fontSize;
   }
 
   private final ColorContrast _colorContrast;
   private final FontSize      _fontSize;
-  
+
   // Default instance
   private static final AccessibilityProfile _sDefaultInstance =
     AccessibilityProfile.getInstance(ColorContrast.STANDARD, FontSize.MEDIUM);
+  
+  
+  /**
+   * We maintain a private internal serializable class for our singleton instance.
+   */
+  private static final class SerializableAccessibilityProfile extends AccessibilityProfile implements Serializable
+  {
+    public SerializableAccessibilityProfile(
+      ColorContrast colorContrast,
+      FontSize      fontSize
+      )
+    {
+      super(colorContrast, fontSize);
+    }
+    
+    private static final long serialVersionUID = 1L;
+  }
 }