Update of /var/cvs/src/org/mmbase/framework/basic
In directory james.mmbase.org:/tmp/cvs-serv26256/basic

Modified Files:
        BasicFramework.java 
Log Message:
It does not make sense that for rendering a block you need the parameters of 
the framework. Actually, none of the renderer implemetnations indeed _used_ the 
framework parameters. Dropped the argument from the interfaces render method, 
hence makeing the API simpler and better understandable


See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/framework/basic


Index: BasicFramework.java
===================================================================
RCS file: /var/cvs/src/org/mmbase/framework/basic/BasicFramework.java,v
retrieving revision 1.29
retrieving revision 1.30
diff -u -b -r1.29 -r1.30
--- BasicFramework.java 26 Aug 2008 06:45:36 -0000      1.29
+++ BasicFramework.java 26 Aug 2008 07:48:38 -0000      1.30
@@ -34,7 +34,7 @@
  * are configured is the order in which they are processed.
  *
  * @author Michiel Meeuwissen
- * @version $Id: BasicFramework.java,v 1.29 2008/08/26 06:45:36 michiel Exp $
+ * @version $Id: BasicFramework.java,v 1.30 2008/08/26 07:48:38 michiel Exp $
  * @since MMBase-1.9
  */
 public class BasicFramework extends Framework {
@@ -277,19 +277,19 @@
                 state.process(processor);
                 log.service("Processing " + actualRenderer.getBlock() + " " + 
processor);
                 setBlockParametersForProcess(state, blockParameters);
-                processor.process(blockParameters, frameworkParameters);
+                processor.process(blockParameters);
             }
 
             state.render(actualRenderer);
             setBlockParametersForRender(state, blockParameters);
             RenderHints hints = new RenderHints(actualRenderer, windowState, 
state.getId(), getComponentClass());
-            actualRenderer.render(blockParameters, frameworkParameters, w, 
hints);
+            actualRenderer.render(blockParameters, w, hints);
         } catch (FrameworkException fe) {
             log.debug(fe);
             URI uri = renderer.getUri();
             Renderer error = new ErrorRenderer(renderer.getType(), 
renderer.getBlock(), (uri != null) ? uri.toString() : null, 500, fe);
             RenderHints hints = new RenderHints(error, windowState, 
state.getId(), getComponentClass());
-            error.render(blockParameters, frameworkParameters, w, hints);
+            error.render(blockParameters, w, hints);
         } finally {
             state.endBlock();
         }
@@ -304,7 +304,7 @@
         State state = State.getState(request);
         state.startBlock(frameworkParameters, null);
         setBlockParametersForProcess(state, blockParameters);
-        processor.process(blockParameters, frameworkParameters);
+        processor.process(blockParameters);
     }
 
     public Node getUserNode(Parameters frameworkParameters) {
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to