Update of /var/cvs/src/org/mmbase/bridge/implementation
In directory james.mmbase.org:/tmp/cvs-serv8737

Modified Files:
        VirtualNodeManager.java 
Log Message:
since CoreField.finish only calls finish on datatype,   MMB-1738 can be solved 
cheaper, by both not cloning and not finishing.


See also: http://cvs.mmbase.org/viewcvs/src/org/mmbase/bridge/implementation
See also: http://www.mmbase.org/jira/browse/MMB-1738


Index: VirtualNodeManager.java
===================================================================
RCS file: 
/var/cvs/src/org/mmbase/bridge/implementation/VirtualNodeManager.java,v
retrieving revision 1.51
retrieving revision 1.52
diff -u -b -r1.51 -r1.52
--- VirtualNodeManager.java     6 Nov 2008 13:45:46 -0000       1.51
+++ VirtualNodeManager.java     6 Nov 2008 16:29:39 -0000       1.52
@@ -28,7 +28,7 @@
  * It's sole function is to provide a type definition for the results of a 
search.
  * @author Rob Vermeulen
  * @author Pierre van Rooden
- * @version $Id: VirtualNodeManager.java,v 1.51 2008/11/06 13:45:46 michiel 
Exp $
+ * @version $Id: VirtualNodeManager.java,v 1.52 2008/11/06 16:29:39 michiel 
Exp $
  */
 public class VirtualNodeManager extends AbstractNodeManager implements 
NodeManager {
     private static final  Logger log = 
Logging.getLoggerInstance(VirtualNodeManager.class);
@@ -97,12 +97,11 @@
             if (query != null) { // means not yet called (lazy loading of 
fields)
                 // code to solve the fields.
                 for (Step step : query.getSteps()) {
-                    DataType nodeType  = DataTypes.getDataType("node").clone();
+                    DataType nodeType  = DataTypes.getDataType("node");
 
                     String name = step.getAlias();
                     if (name == null) name = step.getTableName();
                     CoreField fd = Fields.createField(name, Field.TYPE_NODE, 
Field.TYPE_UNKNOWN, Field.STATE_VIRTUAL, nodeType);
-                    fd.finish();
 
                     Field ft = new VirtualNodeManagerField(fd, name);
                     fieldTypes.put(name, ft);
_______________________________________________
Cvs mailing list
[email protected]
http://lists.mmbase.org/mailman/listinfo/cvs

Reply via email to