From f869198108cd40042a72055da6dc005aaa9a33c7 Mon Sep 17 00:00:00 2001
From: Marnanel Thurman <marnanel@thurman.org.uk>
Date: Mon, 16 Sep 2019 23:14:29 +0100
Subject: [PATCH] create() stops policing whether new objects do or don't have
 an "id" field, based on their is_local status. The views should be doing
 that, anyway.

---
 django_kepi/create.py | 13 ++-----------
 1 file changed, 2 insertions(+), 11 deletions(-)

diff --git a/django_kepi/create.py b/django_kepi/create.py
index 3da5a1e..e6a9b43 100644
--- a/django_kepi/create.py
+++ b/django_kepi/create.py
@@ -86,15 +86,6 @@ def create(
                     str(k))
             del value[k]
 
-    if 'id' in value:
-        if is_local_user:
-            logger.warn('Removing "id" field at local Object creation')
-            del value['id']
-    else:
-        if not is_local_user:
-            logger.warn("Remote Objects must have an id; dropping message")
-            return None
-
     class_name = 'Ac'+value['type']
     try:
         import django_kepi.models as kepi_models
@@ -132,7 +123,7 @@ def create(
                     id = value['id'],
                     )
             result.save()
-            logger.info('  -- created local copy of remote object %s ',
+            logger.info('  -- created object %s',
                 result)
             del value['id']
         except django.db.utils.IntegrityError:
@@ -143,7 +134,7 @@ def create(
         result = cls(
                 )
         result.save()
-        logger.warn('  -- created local object %s',
+        logger.warn('  -- created object %s',
             result)
 
     for f,v in value.items():