[DBAL-1692] Throw exception if table has no primary key instead of fa…

…tal error.
rdohms · Mar 14, 2012 · de5e4b0 · de5e4b0
1 parent 0e1eff1
commit de5e4b0
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/lib/Doctrine/ORM/Mapping/Driver/DatabaseDriver.php b/lib/Doctrine/ORM/Mapping/Driver/DatabaseDriver.php
@@ -130,6 +130,13 @@ private function reverseEngineerMappingFromDatabase()
                 $allForeignKeyColumns = array_merge($allForeignKeyColumns, $foreignKey->getLocalColumns());
             }
 
+            if ( ! $table->hasPrimaryKey()) {
+                throw new MappingException(
+                    "Table " . $table->getName() . " has no primary key. Doctrine does not ".
+                    "support reverse engineering from tables that don't have a primary key."
+                );
+            }
+
             $pkColumns = $table->getPrimaryKey()->getColumns();
             sort($pkColumns);
             sort($allForeignKeyColumns);