RelatedObjectDoesNotExist: Organization has no owner

[Dani Hodovic]

I'm using the helper function to create a default organization when a user signs up.

django-organizations/organizations/utils.py

Lines 55 to 63 in c6ead20

 def create_organization( 
     user, 
     name, 
     slug=None, 
     is_active=None, 
     org_defaults=None, 
     org_user_defaults=None, 
     **kwargs 
 ): 

The docstring says:

Returns a new organization, also creating an initial organization user who is the owner.

However the first user doesn't in fact seem to be the organization owner:

>>> org.is_owner(user)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "/home/dani/repos/ninjads/.venv/lib/python3.7/site-packages/organizations/abstract.py", line 181, in is_owner
    return self.owner.organization_user.user == user
  File "/home/dani/repos/ninjads/.venv/lib/python3.7/site-packages/django/db/models/fields/related_descriptors.py", line 423, in __get__
    self.related.get_accessor_name()
ninjads.organizations.models.Organization.owner.RelatedObjectDoesNotExist: Organization has no owner.

Organization has no owner.

The member and admin checks succeed:

>>> org.is_admin(user)
True
>>> org.is_member(user)
True

[Ben Lopatin]

That definitely does not sound right.

django-organizations/tests/test_utils.py

Lines 22 to 38 in c6ead20

 def test_create_organization(self): 
     acme = create_organization( 
         self.user, "Acme", org_defaults={"slug": "acme-slug"} 
     ) 
     self.assertTrue(isinstance(acme, Organization)) 
     self.assertEqual(self.user, acme.owner.organization_user.user) 
     self.assertTrue(acme.owner.organization_user.is_admin) 
  
 def test_create_custom_org(self): 
     custom = create_organization(self.user, "Custom", model=Account) 
     self.assertTrue(isinstance(custom, Account)) 
     self.assertEqual(self.user, custom.owner.organization_user.user) 
  
 def test_create_custom_org_from_abstract(self): 
     custom = create_organization(self.user, "Custom", model=CustomOrganization) 
     self.assertTrue(isinstance(custom, CustomOrganization)) 
     self.assertEqual(self.user, custom.owner.organization_user.user) 

That function creates the owner right at the end.

django-organizations/organizations/utils.py

Lines 109 to 112 in c6ead20

 org_owner_model.objects.create( 
     organization=organization, organization_user=new_user 
 ) 
 return organization 

Before I dig further, could you share how you're calling that function and what models you're using?

[Dani Hodovic]

Organization models

from hashid_field import HashidAutoField
from organizations.abstract import (
    AbstractOrganization,
    AbstractOrganizationOwner,
    AbstractOrganizationUser,
)


class Organization(AbstractOrganization):
    id = HashidAutoField(primary_key=True)


class OrganizationUser(AbstractOrganizationUser):
    pass


class OrganizationOwner(AbstractOrganizationOwner):
    pass

Organization creation on user sign up

import logging

from allauth.account.signals import user_signed_up
from django.dispatch import receiver
from organizations.utils import create_organization

from ninjads.organizations.models import Organization

logger = logging.LoggerAdapter(logging.getLogger(), extra={})


@receiver(user_signed_up)
def create_default_organization(
    request, user, **kwargs
):  # pylint: disable=unused-argument
    logger.info("User %s signed up, creating a default organization", user)
    create_organization(user, user.email, is_active=True, model=Organization)

[Hari Mahadevan]

Please check my comment here