Author: dr
Date: Thu Aug 2 11:12:46 2007
New Revision: 5804
Log:
- Comment out deleteDataForNode as it does not seem to be necessary. If a node
is deleted it is likely that there are children. As we need to test for that
anyway there is no reason why not to use deleteDataForNodes().
Modified:
trunk/Tree/src/interfaces/data_store.php
trunk/Tree/src/stores/db_external.php
trunk/Tree/src/stores/memory.php
trunk/Tree/src/stores/xml_internal.php
Modified: trunk/Tree/src/interfaces/data_store.php
==============================================================================
--- trunk/Tree/src/interfaces/data_store.php [iso-8859-1] (original)
+++ trunk/Tree/src/interfaces/data_store.php [iso-8859-1] Thu Aug 2 11:12:46
2007
@@ -20,8 +20,8 @@
* Deletes the data for the node $node from the data store.
*
* @param ezcTreeNode $node
+ public function deleteDataForNode( ezcTreeNode $node );
*/
- public function deleteDataForNode( ezcTreeNode $node );
/**
* Deletes the data for all the nodes in the node list $nodeList.
Modified: trunk/Tree/src/stores/db_external.php
==============================================================================
--- trunk/Tree/src/stores/db_external.php [iso-8859-1] (original)
+++ trunk/Tree/src/stores/db_external.php [iso-8859-1] Thu Aug 2 11:12:46 2007
@@ -74,10 +74,10 @@
* Deletes the data for the node $node from the data store.
*
* @param ezcTreeNode $node
- */
public function deleteDataForNode( ezcTreeNode $node )
{
}
+ */
/**
* Deletes the data for all the nodes in the node list $nodeList.
Modified: trunk/Tree/src/stores/memory.php
==============================================================================
--- trunk/Tree/src/stores/memory.php [iso-8859-1] (original)
+++ trunk/Tree/src/stores/memory.php [iso-8859-1] Thu Aug 2 11:12:46 2007
@@ -20,11 +20,11 @@
* Deletes the data for the node $node from the data store.
*
* @param ezcTreeNode $node
- */
public function deleteDataForNode( ezcTreeNode $node )
{
- /* This is a no-op as the data is part of the node */
+ // This is a no-op as the data is part of the node
}
+ */
/**
* Deletes the data for all the nodes in the node list $nodeList.
@@ -33,7 +33,7 @@
*/
public function deleteDataForNodes( ezcTreeNodeList $nodeList )
{
- /* This is a no-op as the data is part of the node */
+ // This is a no-op as the data is part of the nodes
}
/**
@@ -41,7 +41,7 @@
*/
public function deleteDataForAllNodes()
{
- /* This is a no-op as the data is part of the node */
+ // This is a no-op as the data is part of the nodes
}
/**
@@ -52,7 +52,7 @@
*/
public function fetchDataForNode( ezcTreeNode $node )
{
- /* This is a no-op as the data is already in the $node */
+ // This is a no-op as the data is part of the node
}
/**
@@ -63,7 +63,7 @@
*/
public function fetchDataForNodes( ezcTreeNodeList $nodeList )
{
- /* This is a no-op as the data is already in the $node */
+ // This is a no-op as the data is part of the nodes
}
/**
@@ -73,7 +73,7 @@
*/
public function storeDataForNode( ezcTreeNode $node )
{
- /* This is a no-op as the data is already in the $node */
+ // This is a no-op as the data is part of the node
}
}
?>
Modified: trunk/Tree/src/stores/xml_internal.php
==============================================================================
--- trunk/Tree/src/stores/xml_internal.php [iso-8859-1] (original)
+++ trunk/Tree/src/stores/xml_internal.php [iso-8859-1] Thu Aug 2 11:12:46 2007
@@ -21,10 +21,10 @@
* Deletes the data for the node $node from the data store.
*
* @param ezcTreeNode $node
- */
public function deleteDataForNode( ezcTreeNode $node )
{
}
+ */
/**
* Deletes the data for all the nodes in the node list $nodeList.
@@ -33,7 +33,7 @@
*/
public function deleteDataForNodes( ezcTreeNodeList $nodeList )
{
- /* This is a no-op as the data is part of the node */
+ // This is a no-op as the data is part of the nodes
}
/**
@@ -41,7 +41,7 @@
*/
public function deleteDataForAllNodes()
{
- /* This is a no-op as the data is part of the node */
+ // This is a no-op as the data is part of the nodes
}
/**
--
svn-components mailing list
[email protected]
http://lists.ez.no/mailman/listinfo/svn-components
