Opened 11 years ago

Last modified 5 days ago

#25656 assigned Bug

Recent Actions admin section contains link to edit form even when user does not have edit permission

Reported by: Anton Baklanov Owned by: Artyom Kotovskiy
Component: contrib.admin Version: dev
Severity: Normal Keywords:
Cc: Triage Stage: Accepted
Has patch: yes Needs documentation: no
Needs tests: no Patch needs improvement: yes
Easy pickings: no UI/UX: no

Description

Steps to reproduce:

  • Login to admin with user that has add-only permission to certain model
  • Create an instance of this model
  • See that there is new logline within "Recent Actions" which contains link to edit form and results in 403 Forbidden.

This is something similar to what has been spotted by Tim Graham during https://github.com/django/django/pull/5244 review so perhaps it will be handy to fix it after PR 5244 is merged so one can extend test_no_forbidden_links_visible test with checks for this ticket.

Change History (12)

comment:1 by Anton Baklanov, 11 years ago

Owner: changed from nobody to Anton Baklanov
Status: newassigned

comment:2 by Tim Graham, 11 years ago

Triage Stage: UnreviewedAccepted

A bit similar to #2856 in case discussion on that ticket helps.

comment:3 by Mariusz Felisiak, 3 years ago

Owner: Anton Baklanov removed
Status: assignednew

comment:4 by AP Jama, 3 years ago

Owner: set to AP Jama
Status: newassigned

comment:7 by Sarah Boyce, 3 years ago

Has patch: set
Version 0, edited 3 years ago by Sarah Boyce (next)

comment:8 by Mariusz Felisiak, 3 years ago

Patch needs improvement: set

comment:9 by Anil Rai, 10 months ago

Hiya, I wanted to check if anyone is working right now on this ticket. If it available and no one is working on it then I would like to take this one.

comment:10 by Artyom Kotovskiy, 5 weeks ago

Owner: changed from AP Jama to Artyom Kotovskiy

comment:11 by Artyom Kotovskiy, 5 weeks ago

Patch needs improvement: unset

comment:12 by H. White, 4 weeks ago

Patch needs improvement: set

comment:13 by H. White, 4 weeks ago

Patch needs improvement: unset
Triage Stage: AcceptedReady for checkin

Working as expected and tests fail when run on main.

comment:14 by Sarah Boyce, 5 days ago

Patch needs improvement: set
Triage Stage: Ready for checkinAccepted
Note: See TracTickets for help on using tickets.
Back to Top