SCIPIO_TRUST_URL raises AttributeError

[Sergei Lebedev]

Despite the description in scipio/settings.py SCIPIO_TRUST_URL raises AttributeError when it's not defined in project's settings. A tiny patch to fix this is attached.

[Ivan Sagalaev (isagalaev)]

The thing is, it's kind of "by design" :-). General idea is that writing "get(settings, SETTING)" everywhere is not beautiful. So Scipio expects that a user should import all its default settings into the main settings file ("from scipio.settings import *") and then alter some of them if needed. Though currently it doesn't say this anywhere...

Are you happy with this approach?

[Sergei Lebedev (superbobry)]

Updated patch: another bug spotted, the check for HttpResponse in views.scipio.complete will never succeed.

[Sergei Lebedev (superbobry)]

Oh, i saw some apps, doing the same thing by introducing the local_settings module, which did some dirty monkeypatching on django.conf.settings, adding default values, when needed. Anyway, thanks for the reply and for scipio. ;)

[Ivan Sagalaev (isagalaev)]

I saw these too, didn't like 'em :-). What I'm doing here is the same thing that Django recommends for user urlconfs that all start with "from django.conf.urls.defaults import *".