Changeset 500 in code

Timestamp:

Apr 13, 2021, 5:33:06 PM (4 years ago)

Author:

contact

Message:

Error out on network name conflict

Closes: ​https://todo.sr.ht/~emersion/soju/29

File:

• trunk/user.go (modified) (2 diffs)

trunk/user.go

@@ -672 +672 @@
 }
 
+func (u *user) checkNetwork(record *Network) error {
+        for _, net := range u.networks {
+                if net.GetName() == record.GetName() && net.ID != record.ID {
+                        return fmt.Errorf("a network with the name %q already exists", record.GetName())
+                }
+        }
+        return nil
+}
+
 func (u *user) createNetwork(record *Network) (*network, error) {
         if record.ID != 0 {
                 panic("tried creating an already-existing network")
+        }
+
+        if err := u.checkNetwork(record); err != nil {
+                return nil, err
         }
 
@@ -691 +704 @@
         if record.ID == 0 {
                 panic("tried updating a new network")
+        }
+
+        if err := u.checkNetwork(record); err != nil {
+                return nil, err
         }