Escaped elements in notices/warnings from accountmanager
The same issue while has been fixed in [15371] still exists in other locations.
I think Markup(tag_(...))
and Markup(tag(...))
have this issue.
$ grep -rF 'Markup(tag_(' .
./acct_mgr/admin.py: add_notice(req, Markup(tag_(
./acct_mgr/admin.py: add_warning(req, Markup(tag_(
./acct_mgr/admin.py: add_warning(req, Markup(tag_(
./acct_mgr/admin.py: add_warning(req, Markup(tag_(
./acct_mgr/web_ui.py: add_notice(req, Markup(tag_(
$ grep -rF 'Markup(tag(' .
./acct_mgr/admin.py: add_notice(req, Markup(tag(_("Successfully deleted:"),
Change History (9)
Description: |
modified (diff)
|
Owner: |
changed from Steffen Hoffmann to Ryan J Ollos
|
Status: |
new →
accepted
|
Resolution: |
→ fixed
|
Status: |
accepted →
closed
|
The attached patch is lightly tested.
I'm thinking to just target the next version to Trac 1.0+. Steffen isn't maintaining the plugin any longer and I hardly have much time to spend it. It will be bad for the community if Trac 1.2 is released and AcccountManagerPlugin doesn't work with it. Let me know what you think.
Feel free to commit any changes you like, and hopefully we can get AccountManagerPlugin 0.5 released soon.