Shortcut for AddLink

Author: wilr

src/ManyField.php

@@ -10,6 +10,8 @@
 use SilverStripe\Control\Director;
 use SilverStripe\Control\Controller;
 use SilverStripe\Forms\FormField;
+use SilverStripe\Security\SecurityToken;
+use SilverStripe\Core\Injector\Injector;
 use SilverStripe\Control\HTTPResponse;
 
 class ManyField extends CompositeField
@@ -258,6 +260,14 @@ public function createNewRecord()
         return $response;
     }
 
+    public function AddLink()
+    {
+        return Controller::join_links(
+            $this->AbsoluteLink('createNewRecord'),
+            '?SecurityID='. SecurityToken::inst()->getValue()
+        );
+    }
+
     /**
      * Return the list of fields. We'll create a row for each of the values if
      * they exist otherwise we'll only return
@@ -350,36 +360,42 @@ public function updateRelation(DataObjectInterface $record, $delete = true)
                     $existing->removeById($row->ID);
                 }
             }
+        }
 
+        if (isset($this->value['ID'])) {
             foreach ($this->value['ID'] as $key => $id) {
                 if ($id) {
                     $idKeyMap[$key] = $id;
                 }
             }
-            
-            foreach ($this->value as $col => $values) {
-                if ($col == 'ID') {
-                    continue;
-                }
-
-                foreach ($values as $key => $value) {
-                    if (!isset($updatedData[$key])) {
-                        $updatedData[$key] = [];
-                    }
+        }
+        
+        foreach ($this->value as $col => $values) {
+            if ($col == 'ID') {
+                continue;
+            }
 
-                    $updatedData[$key][$col] = $value;
+            foreach ($values as $key => $value) {
+                if (!isset($updatedData[$key])) {
+                    $updatedData[$key] = [];
                 }
+
+                $updatedData[$key][$col] = $value;
             }
+        }
 
-            foreach ($updatedData as $key => $data) {
-                if (isset($idKeyMap[$key])) {
-                    $existing->find('ID', $idKeyMap[$key])->update($row);
-                } else {
-                    $create = Injector::inst()->create($existing->dataClass());
-                    $create->update($data);
-                    $create->write();
-                    $existing->add($create);
-                }
+        foreach ($updatedData as $key => $data) {
+            if (isset($idKeyMap[$key])) {
+                $record = $existing->find('ID', $idKeyMap[$key]);
+                
+                $record->update($data);
+                $record->write();
+            } else {
+                $create = Injector::inst()->create($existing->dataClass());
+                $create->update($data);
+
+                $create->write();
+                $existing->add($create);
             }
         }
 

templates/ManyField.ss

@@ -17,7 +17,7 @@
 
 <% if canAdd %>
 <div class="manyfield__add">
-    <a class="btn btn-primary" href="$AbsoluteLink(createNewRecord)?SecurityID={$SecurityID}">$AddLabel +</a>
+    <a class="btn btn-primary" href="$AddLink">$AddLabel +</a>
 </div>
 <% end_if %>
 

templates/ManyField_holder.ss

@@ -17,7 +17,7 @@
 
 <% if canAdd %>
 <div class="manyfield__add">
-	<a class="btn btn-primary" href="$AbsoluteLink(createNewRecord)?SecurityID={$SecurityID}">$AddLabel +</a>
+	<a class="btn btn-primary" href="$AddLink">$AddLabel +</a>
 </div>
 <% end_if %>