Last chance to object to new features

[Russell Keith-Magee]

Hi all,

A quick reminder - if you have any objections or feedback on #10109,
the M2M refactor [1], I intend to land this patch in approximately 48
hours.

Also, following some mailing list and IRC discussion [2], I'll be
trying to land #10355 at the same time. This is the addition of the
email backend API.

If you have any objections to either of these commits - even if its
just "hold off until I've had a chance to take a look", now is the
time to speak.

[1] http://groups.google.com/group/django-developers/browse_thread/thread/b67956af9bee2ac1
[2] http://groups.google.com/group/django-developers/browse_thread/thread/33437e28e3b56e81

Yours,
Russ Magee %-)

[Jacob Kaplan-Moss]

On Mon, Nov 2, 2009 at 10:02 AM, Russell Keith-Magee
<freakb...@gmail.com> wrote:
> A quick reminder - if you have any objections or feedback on #10109,
> the M2M refactor [1], I intend to land this patch in approximately 48
> hours.

I've left a few comments in that thread, but they don't need to be
addressed pre-merge. Fire away!

> Also, following some mailing list and IRC discussion [2], I'll be
> trying to land #10355 at the same time. This is the addition of the
> email backend API.

I've left a few notes on the ticket. The only one that needs to be
addressed pre-merge is a simple design decision, and the final call's
yours.

Thanks!

Jacob

[Andi Albrecht]

Russel, thanks for finishing the mail backend patch! Jacob's notes on
the ticket sound very reasonable to me - please let me know which of
them you'll address before merging. I'd be happy to provide a patch
for anything remaining.

Andi

>
> Thanks!
>
> Jacob
>
> >
>

[Russell Keith-Magee]

I've already integrated Jacob's comments into the code in my local
checkout. Actually, Jacob's comments pointed out an interesting code
cleanup: the File email backend is really just the Console email
backend pointing at a file object instead of stdout. As a result, I've
been able to delete a bunch of code by modifying the console backend
to accept an arbitrary stream, and making the file backend a subclass
of the console backend.

I'll commit tonight (my time) once I have a chance to give the patch a
final review.

Thanks for your work on this patch Andi, and everyone else that has
contributed feedback and comments.

Yours,
Russ Magee %-)

[Waldemar Kornewald]

On Tue, Nov 3, 2009 at 6:37 AM, Russell Keith-Magee
<freakb...@gmail.com> wrote:
> I've already integrated Jacob's comments into the code in my local
> checkout. Actually, Jacob's comments pointed out an interesting code
> cleanup: the File email backend is really just the Console email
> backend pointing at a file object instead of stdout. As a result, I've
> been able to delete a bunch of code by modifying the console backend
> to accept an arbitrary stream, and making the file backend a subclass
> of the console backend.
>
> I'll commit tonight (my time) once I have a chance to give the patch a
> final review.

Thanks a lot, Russell!

Andi, could you please add your App Engine email backend to our test project?

Bye,
Waldemar Kornewald

[Andi Albrecht]

I'd prefer to keep the App Engine email backends separate until we're
able to bundle a complete package that provides App Engine support for
Django.
Two backends (a synchronous and an asynchronous one) are available on
bitbucket: http://bitbucket.org/andialbrecht/appengine_emailbackends/

Andi

>
> Bye,
> Waldemar Kornewald
>
> >
>

[Andi Albrecht]

This just came up on IRC: in django.test.utils
setup_test_environment() the lines 46/47 are mixed up.

Here's a diff that fixes this issue:

diff --git a/django/test/utils.py b/django/test/utils.py
index a30bb7e..9d39eee 100644
--- a/django/test/utils.py
+++ b/django/test/utils.py
@@ -43,8 +43,8 @@ def setup_test_environment():
mail.original_SMTPConnection = mail.SMTPConnection
mail.SMTPConnection = locmem.EmailBackend

- settings.EMAIL_BACKEND = 'django.core.mail.backends.locmem'
mail.original_email_backend = settings.EMAIL_BACKEND
+ settings.EMAIL_BACKEND = 'django.core.mail.backends.locmem'

mail.outbox = []

Sorry for pasting it here, but it's really just one line :)

Andi

[Russell Keith-Magee]

On Wed, Nov 4, 2009 at 6:23 PM, Andi Albrecht
<albrec...@googlemail.com> wrote:
>
> This just came up on IRC: in django.test.utils
> setup_test_environment() the lines 46/47 are mixed up.
>
> Here's a diff that fixes this issue:
>
> diff --git a/django/test/utils.py b/django/test/utils.py
> index a30bb7e..9d39eee 100644
> --- a/django/test/utils.py
> +++ b/django/test/utils.py
> @@ -43,8 +43,8 @@ def setup_test_environment():
>     mail.original_SMTPConnection = mail.SMTPConnection
>     mail.SMTPConnection = locmem.EmailBackend
>
> -    settings.EMAIL_BACKEND = 'django.core.mail.backends.locmem'
>     mail.original_email_backend = settings.EMAIL_BACKEND
> +    settings.EMAIL_BACKEND = 'django.core.mail.backends.locmem'
>
>     mail.outbox = []
>
> Sorry for pasting it here, but it's really just one line :)

No problems. Fixed in r11718.

Yours,
Russ Magee %-)