Line length :)
On Feb 25, 2014 9:29 PM, "Biyeun" <notifications@github.com> wrote:

> In corehq/apps/accounting/models.py:
>
> > @@ -636,12 +636,20 @@ def get_subscribed_plan_by_domain(cls, domain):
> >          """
> >          Returns SoftwarePlanVersion, Subscription for the given domain.
> >          """
> > -        domain = assure_domain_instance(domain)
> > +        domain_obj = assure_domain_instance(domain)
> > +        if domain_obj is None:
> > +            # need more info to troubleshoot this further
> > +            logging.error("Tried to fetch a subscription for a domain that was not properly located. "
> > +                          "Domain name is %s." % domain)
> > +            plan_version = DefaultProductPlan.objects.get(edition=SoftwarePlanEdition.COMMUNITY,
> > +                                                          product_type=SoftwareProductType.COMMCARE).plan.get_version()
>
> what about this is not pep8? pycharm's pep8 was ok with this...
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/dimagi/commcare-hq/pull/2620/files#r10067285>
> .
>