Refactor toot-toot

- Now can specify the "order" of fields

InvenTree/templates/js/forms.js

@@ -193,23 +193,39 @@ function constructCreateForm(url, fields, options={}) {
         ignored_fields.push('id');
     }
 
-    for (const key in fields) {
+    // Construct an ordered list of field names
+    var field_names = [];
 
-        // Skip over fields
+    if (allowed_fields) {
-        if (allowed_fields && !allowed_fields.includes(key)) {
+        allowed_fields.forEach(function(name) {
-            continue;
+
+            // Only push names which are actually in the set of fields
+            if (name in fields) {
+
+                if (!ignored_fields.includes(name) && !field_names.includes(name)) {
+                    field_names.push(name);
+                }
+            } else {
+                console.log(`WARNING: '${name}' does not match a valid field name.`);
+            }
+        });
+    } else {
+        for (const name in fields) {
+
+            if (!ignored_fields.includes(name) && !field_names.includes(name)) {
+                field_names.push(name);
+            }
+        }
     }
 
-        if (ignored_fields &&  ignored_fields.includes(key)) {
+    field_names.forEach(function(name) {
-            continue;
-        }
 
-        var field = fields[key];
+        var field = fields[name];
         
-        var f = constructField(key, field, options);
+        var f = constructField(name, field, options);
         
         html += f;
-    }
+    });
 
     var modal = '#modal-form';