Hotfix for reverse related.

django_object_manager/field_converters.py

@@ -34,22 +34,39 @@ def create_foreign_key(object_manager, field, value):
             **object_manager._data[name][value])
     return FieldConverterResult(value, post_actions, pass_field_value)
 
+
 # TODO M2M are similar, probably they can be combined
 def create_m2m_reverse(object_manager, field, values):
+
     def cb(field, field_val, instance):
         args = \
             {instance._meta.model.__name__.lower(): instance,
              field.related_model.__name__.lower(): field_val}
         # Create dependency after main object, using
         # M2M "through" model
         field.through(**args).save()
+
+    foreing_model = field.related_model
+    name = foreing_model.__name__.lower()
+
+    def gen():
+        for value in values:
+            if isinstance(value, foreing_model):
+                yield value
+            else:
+                assert isinstance(value, str), \
+                    'Related values must be either instances or str ids'
+                yield object_manager._get_or_create(name,
+                                          value,
+                                          **object_manager._data[name][value])
+    res = list(gen())
     return FieldConverterResult(
         [],
-        [partial(cb, field, related_val) for related_val in values],
+        [partial(cb, field, related_val) for related_val in res],
         False)
 
 
-def create_m2m_forward(self, field, values):
+def create_m2m_forward(object_manager, field, values):
 
     def cb(field, field_val, instance):
         field_val.save()
@@ -67,12 +84,12 @@ def gen():
             else:
                 assert isinstance(value, str), \
                     'Related values must be either instances or str ids'
-                yield self._get_or_create(name,
-                                          value,
-                                          **self._data[name][value])
+                yield object_manager._get_or_create(name,
+                                                    value,
+                                                    **object_manager._data[name][value])
     res = list(gen())
     return FieldConverterResult(
-        res,
+        [],
         [partial(cb, field, related_val) for related_val in res],
         False)
 

setup.py

@@ -11,7 +11,7 @@
 
 setup(
     name='django-object-manager',
-    version='0.0.6',
+    version='0.0.7',
     description='Django object manager for tests',
     license='BSD',
     long_description=long_description,

tests/app/tests.py

@@ -35,6 +35,16 @@
 ObjectManager.register(
     Film,
     {
+        'memento': {
+            'name': 'Memento',
+            'year': 2000,
+            'uploaded_by': 'bob',
+        },
+        'godfather': {
+            'name': 'The Godfather',
+            'year': 1974,
+            'uploaded_by': 'bob',
+        },
     })
 ObjectManager.register(
     UserExtraInfo,

tests/functional/test_object_manager.py

@@ -67,7 +67,7 @@ def test_foreing_key_as_instance(self):
         self.assertEqual(film.uploaded_by.email, 'test@test.org')
         self.assertEqual(models.User.objects.count(), 1)
 
-    def test_many_to_many_reversed_predefined(self):
+    def test_many_to_many_reversed(self):
         """Ensure that object with M2M relation can be created."""
         film1 = self.object_manager.get_film(name='Memento',
                                              year=2000,
@@ -81,6 +81,13 @@ def test_many_to_many_reversed_predefined(self):
         assert film1 in category.films.all()
         assert film2 in category.films.all()
 
+    def test_many_to_many_reversed_predefined(self):
+        """Ensure that object with M2M relation can be created."""
+        category = self.object_manager.get_filmcategory(
+            'crime', films=['memento', 'godfather'])
+        self.assertEqual(models.FilmCategory.objects.count(), 1)
+        assert len(category.films.all()) == 2
+
     def test_many_to_many_forward_predefined(self):
         """Ensure that object with M2M relation can be created."""
         self.object_manager.get_film(name='Memento',