[richfaces-svn-commits] JBoss Rich Faces SVN: r5415 - trunk/sandbox/ui/progressBAR/src/main/java/org/richfaces/renderkit.

[richfaces-svn-commits at lists.jboss.org]

Author: andrei_exadel
Date: 2008-01-16 08:56:31 -0500 (Wed, 16 Jan 2008)
New Revision: 5415

Modified:
   trunk/sandbox/ui/progressBAR/src/main/java/org/richfaces/renderkit/AbstractProgressBarRenderer.java
Log:
Refactoring

Modified: trunk/sandbox/ui/progressBAR/src/main/java/org/richfaces/renderkit/AbstractProgressBarRenderer.java
===================================================================
--- trunk/sandbox/ui/progressBAR/src/main/java/org/richfaces/renderkit/AbstractProgressBarRenderer.java	2008-01-16 13:50:02 UTC (rev 5414)
+++ trunk/sandbox/ui/progressBAR/src/main/java/org/richfaces/renderkit/AbstractProgressBarRenderer.java	2008-01-16 13:56:31 UTC (rev 5415)
@@ -131,24 +131,32 @@
 
     /**
      * Sets up the progressVar page attribute
-     * @param context   - facesContext
-     * @param component - component instance
-     * @param value     - percent value 
+     * 
+     * @param context -
+     *                facesContext
+     * @param component -
+     *                component instance
+     * @param value -
+     *                percent value
      */
     public void setProgressVar(FacesContext context, UIComponent component,
-	    Object value) {
-	Number persent = getNumber(value);
+	    Number persent) {
 	String progresssVar = (String) component.getAttributes().get(
 		"progressVar");
-	Map attrs = context.getExternalContext().getRequestMap();
-	attrs.put(progresssVar, persent);
+	if (progresssVar != null) {
+	    Map attrs = context.getExternalContext().getRequestMap();
+	    attrs.put(progresssVar, persent);
+	}
     }
 
     /**
      * Encodes progress state of component
-     * @param context      - faces context
-     * @param component    - component instance
-     * @throws IOException 
+     * 
+     * @param context -
+     *                faces context
+     * @param component -
+     *                component instance
+     * @throws IOException
      */
     public void encodeProgressState(FacesContext context, UIComponent component)
 	    throws IOException {
@@ -213,9 +221,10 @@
 	    writer.endElement("div");
 	}
     }
-    
-    private void encodeChildrenComponents (FacesContext context, UIComponent component) {
-	
+
+    private void encodeChildrenComponents(FacesContext context,
+	    UIComponent component) {
+
     }
 
     /**
@@ -266,8 +275,10 @@
     }
 
     /**
-     * Converts value attr to number value 
-     * @param v - value attr 
+     * Converts value attr to number value
+     * 
+     * @param v -
+     *                value attr
      * @return result
      */
     public Number getNumber(Object v) {
@@ -315,10 +326,10 @@
 	poll.setSubmitted(submitted);
 	return submitted;
     }
-    
-    
 
-    /* (non-Javadoc)
+    /*
+     * (non-Javadoc)
+     * 
      * @see javax.faces.render.Renderer#getRendersChildren()
      */
     @Override
@@ -326,12 +337,15 @@
 	return true;
     }
 
-    /* (non-Javadoc)
-     * @see org.ajax4jsf.renderkit.AjaxCommandRendererBase#encodeChildren(javax.faces.context.FacesContext, javax.faces.component.UIComponent)
+    /*
+     * (non-Javadoc)
+     * 
+     * @see org.ajax4jsf.renderkit.AjaxCommandRendererBase#encodeChildren(javax.faces.context.FacesContext,
+     *      javax.faces.component.UIComponent)
      */
     @Override
     public void encodeChildren(FacesContext context, UIComponent component)
 	    throws IOException {
-	return;  // We don't need to render children ! 
+	return; // We don't need to render children !
     }
 }