Opened 12 years ago

Closed 11 years ago

#17818 closed New feature (duplicate)

Generic edit generic views don't create `fail_silently` success messages anymore.

Reported by: Simon Charette Owned by: Simon Charette
Component: Generic views Version: dev
Severity: Normal Keywords: views generic edit messages
Cc: marc.tamlyn@…, hirokiky@… Triage Stage: Accepted
Has patch: yes Needs documentation: yes
Needs tests: no Patch needs improvement: no
Easy pickings: no UI/UX: no

Description

It looks like the creation of success messages on valid form submission was oversight when the generic class based views were designed.

Here's a link to the django-developper discussion that motivated the creation of this ticket.

I'm also attaching a patch (with no tests and docs) that provide a simple yet overridable implementation that mimic the deprecated function based behaviour.

Attachments (2)

class-based-generic-edit-views-messages-v1.diff (4.6 KB ) - added by Simon Charette 12 years ago.
17818_test_v1.diff (10.2 KB ) - added by Hiroki Kiyohara 11 years ago.
Tests for sending messages of edie views.

Download all attachments as: .zip

Change History (6)

comment:1 by Aymeric Augustin, 12 years ago

Triage Stage: UnreviewedAccepted
Type: UncategorizedNew feature

comment:2 by Marc Tamlyn, 12 years ago

Cc: marc.tamlyn@… added
Needs documentation: set
Needs tests: set
Version: 1.3master

I'm +1 on this feature being reintroduced, but -1 on it being on by default, especially as this would constitute a backwards incompatibility to me. We have views around which create messages similar to this, and turning it back on by default would mean duplicate messages, or messages where none were expected.

That said it's probably a useful feature by default, but I think there should be an upgrade path.

by Hiroki Kiyohara, 11 years ago

Attachment: 17818_test_v1.diff added

Tests for sending messages of edie views.

comment:3 by Hiroki Kiyohara, 11 years ago

Cc: hirokiky@… added
Needs tests: unset

I added a patch for testing this feature.

comment:4 by Simon Charette, 11 years ago

Resolution: duplicate
Status: newclosed

Closing as duplicate of #16319.

Note: See TracTickets for help on using tickets.
Back to Top