From d9c7a0ca58dbfd5bf5f9e978ed24f6b8a2911ec5 Mon Sep 17 00:00:00 2001
From: Michael Bayne <mdb@threerings.net>
Date: Wed, 11 Oct 2006 02:46:41 +0000
Subject: [PATCH] Go through the fields in declaration order when doing the
 migration otherwise we might try to add a field after a field that we haven't
 yet added.

---
 src/java/com/samskivert/jdbc/depot/DepotMarshaller.java | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/java/com/samskivert/jdbc/depot/DepotMarshaller.java b/src/java/com/samskivert/jdbc/depot/DepotMarshaller.java
index 894c8a5..3f11e4d 100644
--- a/src/java/com/samskivert/jdbc/depot/DepotMarshaller.java
+++ b/src/java/com/samskivert/jdbc/depot/DepotMarshaller.java
@@ -317,7 +317,8 @@ public class DepotMarshaller<T>
             columns.add(rs.getString("COLUMN_NAME"));
         }
 
-        for (FieldMarshaller fmarsh : _fields.values()) {
+        for (String fname : _allFields) {
+            FieldMarshaller fmarsh = _fields.get(fname);
             if (columns.contains(fmarsh.getColumnName())) {
                 continue;
             }