I suppose that's better, previously, I had it all in one line and didn't
want to break it out to special case it. But I think I suppose this is
better.

-Rocky Meza
2014年6月4日 下午5:18于 "Antoine Catton" <notifications@github.com>写道：

> In betterforms/multiform.py:
>
> > +            fargs, fkwargs = self.get_form_args_kwargs(key, args, kwargs)
> > +            self.forms[key] = form_class(*fargs, **fkwargs)
> > +            # we don't need to do this on every iteration do we?
> > +            self.is_bound = self.forms[key].is_bound
> > +
> > +    def get_form_args_kwargs(self, key, args, kwargs):
> > +        """
> > +        Returns the args and kwargs for initializing one of our form children.
> > +        """
> > +        fkwargs = kwargs.copy()
> > +        prefix = kwargs.get('prefix')
> > +        if prefix is None:
> > +            prefix = ''
> > +        fkwargs.update(
> > +            initial=self.initials.get(key),
> > +            prefix='{0}__{1}'.format(key, prefix),
>
> bikeshed: what about:
>
> prefix_arg = kwargs.get('prefix')if prefix_arg is None:
>   prefix = keyelse:
>   prefix = '{}__{}'.format(key, prefix)
>
> Because, when you don't have a prefix, you get fields with the id
> id_formname__-fieldname. Is that expected?
>
> —
> Reply to this email directly or view it on GitHub
> <https://github.com/fusionbox/django-betterforms/pull/4/files#r13377561>.
>