Small refactorings

Doctrine/Connection.php

@@ -204,6 +204,9 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
         return count($this->tables);
     }
     /**
+     * addTable
+     * adds a Doctrine_Table object into connection registry
+     *
      * @param $objTable             a Doctrine_Table object to be added into registry
      * @return boolean
      */
@@ -217,6 +220,7 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
         return true;
     }
     /**
+     * create
      * creates a record
      *
      * create                       creates a record
@@ -240,7 +244,7 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
     }
     /**
      * saveAll                      
-     * saves all the records from all tables
+     * persists all the records from all tables
      *
      * @return void
      */
@@ -274,6 +278,9 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
         }
     }
     /**
+     * evictTables
+     * evicts all tables
+     *
      * @return void
      */
     public function evictTables() {
@@ -424,16 +431,10 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
                         $foreign = $fk->getForeign();
 
                         if($record->getTable()->hasPrimaryKey($fk->getLocal())) {
-                            switch($record->getState()):
-                                case Doctrine_Record::STATE_TDIRTY:
-                                case Doctrine_Record::STATE_TCLEAN:
+                            if( ! $record->exists())
                                 $saveLater[$k] = $fk;
-                                break;
-                                case Doctrine_Record::STATE_CLEAN:
-                                case Doctrine_Record::STATE_DIRTY:
+                            else
                                 $v->save();
-                                break;
-                            endswitch;
                         } else {
                             // ONE-TO-ONE relationship
                             $obj = $record->get($fk->getTable()->getComponentName());
@@ -552,10 +553,9 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
      * @return boolean      true on success, false on failure
      */
     final public function delete(Doctrine_Record $record) {
-        switch($record->getState()):
-            case Doctrine_Record::STATE_PROXY:
-            case Doctrine_Record::STATE_CLEAN:
-            case Doctrine_Record::STATE_DIRTY:
+        if( ! $record->exists())
+            return false;
+            
         $this->beginTransaction();
 
         $this->deleteComposites($record);
@@ -563,10 +563,6 @@ abstract class Doctrine_Connection extends Doctrine_Configurable implements Coun
 
         $this->commit();
         return true;
-            break;
-            default:
-                return false;
-        endswitch;    
     }
     /**
      * returns a string representation of this object

Doctrine/Connection/UnitOfWork.php

@@ -20,7 +20,7 @@
  */
 
 /**
- * Doctrine_Transaction
+ * Doctrine_Connection_UnitOfWork
  *
  * @package     Doctrine ORM
  * @url         www.phpdoctrine.com