add success_message in ClassViews

[wilsonpjunior@…]

sample:

class CreatePersonView(CreateView):
    model = Person
    success_url = "/"
    success_message = "%(name)s was created successfully"

after object created calls messages.sucess_message

:-)

[Aymeric Augustin]

Just reformatted code excerpt.

[Łukasz Rekucki]

It should be quite easy to create a Mixin that does that (and put it in contrib.messages). I wouldn't incorporate that into the @CreateView@, as CBVs should be independent from contrib apps.

[martinogden]

Add SuccessMessageMixin to django.contrib.messages.views

[Preston Holmes]

Not sure if every corner of django needs to provide mixins. This is a good snippet, but needs docs and tests.

[Jacob]

Milestone 1.4 deleted

[Roman Gladkov]

this patch was improved, added tests and documentation (need to improve, English language non native for me.)

[Paul Collins]

Grabbing this one and updating for the current master.

[Paul Collins]

Pull request created ​https://github.com/django/django/pull/922

[Simon Charette]

#17818 was a duplicate.

[Carny Cheng]

Pull request looks good. Tests run.

[Claude Paroz]

Unfortunately, the .format() does not work well with the gettext-based translation system. So for all translatable strings in Django, we should stick with the %(name)s placeholder style.
​http://savannah.gnu.org/bugs/?30854

[Paul Collins]

Replying to claudep:

Unfortunately, the .format() does not work well with the gettext-based translation system.

Oh? Didn't know that. Okay I'll update the patch and resubmit

[Paul Collins]

PR updated. (​https://github.com/django/django/pull/922)
Since the change was pretty straight forward and all tests are still passing I've bounced this back to Ready, but if that's not kosher please let me know.

[Preston Holmes <preston@…>]

In 9a85ad89c20e88340cdba85b8e9f5806c95835db:

Fixed #16319 -- added SuccessMessageMixin to contrib.messages

Thanks martinogden for the initial patch and d1ffuz0r for tests.

[Preston Holmes <preston@…>]

In b1bfd9630ef049070b0cd6ae215470d3d1facd40:

Remove unnecessary and problematic parent class from SuccessMessageMixin

refs #16319, thanks to bmispelon for the catch

[anonymous]

SuccessMessageMixin is useful for django.views.generic.edit.DeleteView but it should be implemented as mixin.