messages from django messages framework are shown in wiki page

Bug #1659018 reported by kaputtnik
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Widelands Website
Fix Released
Medium
kaputtnik

Bug Description

This was introduced with http://bazaar.launchpad.net/~widelands-dev/widelands-website/trunk/revision/447

Steps to reproduce:

1. Go to https://wl.widelands.org/messages/inbox/
2. Delete a message
3. Go to https://wl.widelands.org/wiki/Main%20Page/

Result: The message "Message successfully deleted." is shown then.

We have to check all other apps, if they use the django messages framework.

kaputtnik (franku)
Changed in widelands-website:
assignee: nobody → kaputtnik (franku)
kaputtnik (franku)
summary: - django messages are shown in wiki page
+ messages from django messages framework are shown in wiki page
description: updated
kaputtnik (franku)
description: updated
Revision history for this message
kaputtnik (franku) wrote :

The bug is fixed in the linked branch and i couldn't find any other app that triggers this behavior.

While i am on it, is there a need for more such messages that appear if something went ok?
In wiki there were some messages, but they weren't shown because some code in the template was missing:

"The article was created successfully"
"The article was edited successfully"
"The article was reverted successfully"

I removed them in the last branch, because they are not shown and nobody complained. Should i implement those again?

I think in other places such messages aren't needed? Any other suggestions?

Revision history for this message
GunChleoc (gunchleoc) wrote :

I haven't done anything int he wiki for a bit. I think anything that clearly changes the current page doesn't need a message, unless there's an error.

Revision history for this message
kaputtnik (franku) wrote :

I second that

Will merge this small fix directly then.

Revision history for this message
GunChleoc (gunchleoc) wrote :

+1 - code LGTM too.

kaputtnik (franku)
Changed in widelands-website:
status: Confirmed → Fix Committed
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.