Refs #9475 -- Fixed typo, used unpacking generalization, and made through_defaults kwarg-only.

1 files changed, 6 insertions, 6 deletions

diff --git a/django/db/models/fields/related_descriptors.py b/django/db/models/fields/related_descriptors.py
index 9f6c6858ec..ffeaf09987 100644
--- a/django/db/models/fields/related_descriptors.py
+++ b/django/db/models/fields/related_descriptors.py
@@ -967,7 +967,7 @@ def create_forward_many_to_many_manager(superclass, rel, reverse):
                 # If this is a symmetrical m2m relation to self, add the mirror
                 # entry in the m2m table. `through_defaults` aren't used here
                 # because of the system check error fields.E332: Many-to-many
-                # fields with intermediate tables mus not be symmetrical.
+                # fields with intermediate tables must not be symmetrical.
                 if self.symmetrical:
                     self._add_items(self.target_field_name, self.source_field_name, *objs)
         add.alters_data = True
@@ -1024,14 +1024,14 @@ def create_forward_many_to_many_manager(superclass, rel, reverse):
                     self.add(*new_objs, through_defaults=through_defaults)
         set.alters_data = True
 
-        def create(self, through_defaults=None, **kwargs):
+        def create(self, *, through_defaults=None, **kwargs):
             db = router.db_for_write(self.instance.__class__, instance=self.instance)
             new_obj = super(ManyRelatedManager, self.db_manager(db)).create(**kwargs)
             self.add(new_obj, through_defaults=through_defaults)
             return new_obj
         create.alters_data = True
 
-        def get_or_create(self, through_defaults=None, **kwargs):
+        def get_or_create(self, *, through_defaults=None, **kwargs):
             db = router.db_for_write(self.instance.__class__, instance=self.instance)
             obj, created = super(ManyRelatedManager, self.db_manager(db)).get_or_create(**kwargs)
             # We only need to add() if created because if we got an object back
@@ -1041,7 +1041,7 @@ def create_forward_many_to_many_manager(superclass, rel, reverse):
             return obj, created
         get_or_create.alters_data = True
 
-        def update_or_create(self, through_defaults=None, **kwargs):
+        def update_or_create(self, *, through_defaults=None, **kwargs):
             db = router.db_for_write(self.instance.__class__, instance=self.instance)
             obj, created = super(ManyRelatedManager, self.db_manager(db)).update_or_create(**kwargs)
             # We only need to add() if created because if we got an object back
@@ -1105,10 +1105,10 @@ def create_forward_many_to_many_manager(superclass, rel, reverse):
 
                     # Add the ones that aren't there already
                     self.through._default_manager.using(db).bulk_create([
-                        self.through(**dict(through_defaults, **{
+                        self.through(**{**through_defaults, **{
                             '%s_id' % source_field_name: self.related_val[0],
                             '%s_id' % target_field_name: obj_id,
-                        }))
+                        }})
                         for obj_id in new_ids
                     ])