Improperly Configured Exception 'Module "djstripe.forms" does not define a "StripeSubscriptionSignupForm" class' about dj-stripe

[pydanny]

Hi @dollydagr, I'm moving today so my ability to quickly help is curtained.

If it helps, my development with this has only been with PostgreSQL. Maybe try it with that or MySQL?

[dollydagr]

Thanks @pydanny for your quick reply. I am digging into this (but I don't believe it's a DBMS issue).
Good luck with your move! Had so many of those.

[dollydagr]

The exception seems to happen more exactly when djstripe/forms.py fails to import SetPasswordField and PasswordField from allauth.account.forms.

In fact, there is nothing that I can import from allauth.account.forms from djstripe's forms.py (or even my app). However, the method from allauth.account.utils could be imported fine. Right now, just to keep going, I redeclared SetPasswordField and PasswordField (and imported them) from myapp/forms.py (which works).

I know this is not an issue particular to djstripe, but if you have an idea why the classes of allauth.account.forms are inaccessible to the other apps, I'd really appreciate your insight (and remove my dirty fix).

My structure is the following:

my_project:
   |_ my_app
   |_ djstripe
   |_ allauth
         |_ account

Thank you!
dollydagr

[pydanny]

@dollydagr Are you using pip or actually copying the code of djstripe and allauth directly into your project?

[dollydagr]

@pydanny I ended up copying the code of djstripe and allauth into my project.
I actually started by pip when I first had this error. As it was hard to trace, I copied the source codes to get a better handle on things and be able to test assumptions (and workarounds). Should I try posting this (what seems to be an) import issue on stackoverflow instead?

[pydanny]

Admittedly, the test coverage on the project isn't where it needs to be. And since I delivered the project powered by dj-stripe, I haven't had a chance to work on it since.

In any case, including it the way you did is probably the cause of the import errors. If you want to use a development version, I recommend cloning it outside and then bringing it into your development environment via python setup.py develop.

[dollydagr]

Thanks @pydanny for the recommendation. For some reason, even with the development environment setup, I am still getting the same error.

In any case, I wanted to change the signup interaction flow. The current signup form, when filled with both credentials and payment errors, will first and only display stripe errors. When these are fixed, it erases all the payment information to display the credentials (username, password) errors.

So I ended up separating both out. I am making sure my user signs up and verifies their identify. And then I collect their payment. I now integrated all dj-stripe templates into my website, and it's great!

[pydanny]

Hey @dollydagr, what should I do with this ticket? Close it? Put a note in the docs and then close it?

[dollydagr]

Yes @pydanny, this left me perplex, just close it (until/if someone else encounters the issue).

[dustinfarris]

I ran into this as well. It looks like it is being caused by what would end up being a circular import

in allauth/account/forms.py (truncated):

def _base_signup_form_class():
    ...
    try:
        fc_class = getattr(mod, fc_classname)
    except AttributeError:
        raise exceptions.ImproperlyConfigured('Module "%s" does not define a'
                                              ' "%s" class' % (fc_module,
                                                               fc_classname))

class BaseSignupForm(_base_signup_form_class()):
    ...

djstripe tests for the presence of allauth with the following:

try:
    from allauth.account.utils import setup_user_email
    from allauth.account.forms import SetPasswordField  # circular import occurs here
    from allauth.account.forms import PasswordField
except ImportError:

When reading allauth.account.forms allauth attempts to import the signup form indicated in settings.py, in this case djstripe.forms.StripeSubscriptionSignupForm which hasn't been defined yet. The only reason this isn't actually causing a circular import error is because allauth is testing for the existence of the class as an attribute of the parent module and raising an exception when it doesn't find it.

[dollydagr]

Makes sense. Thanks @dustinfarris!