Modify

Opened 4 years ago

Closed 4 years ago

#7811 closed defect (fixed)

Failed to add download when no DOWNLOADS_ADMIN permission

Reported by: ak@… Owned by: Blackhex
Priority: normal Component: DownloadsPlugin
Severity: normal Keywords:
Cc: Trac Release: 0.12

Description

When a user has DOWNLOADS_ADD but not DOWNLOADS_ADMIN, the upload fails with "Insufficient privileges to perform this operation". The problem is that download_created is called and check_permission is set to require DOWNLOADS_ADMIN for modification. The result is that the upload fails and even worse, the file is already uploaded but not probably "processed". When someone tries to upload a file, the plug-in reports that the directory for that ID already exists (well, actually the error message is a bit more cryptic). Apparently the ID is not probably assigned or something like that before the bug causes the exception...

Attachments (0)

Change History (2)

comment:1 Changed 4 years ago by ak@…

P.S.: Changing the check_permission in tags.py to the following line seems to fix this.

permissions = {'view' : 'TAGS_VIEW', 'modify' : 'TAGS_MODIFY'}

comment:2 Changed 4 years ago by Blackhex

  • Resolution set to fixed
  • Status changed from new to closed

Fixed in r9282.

Add Comment

Modify Ticket

Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
Author


E-mail address and user name can be saved in the Preferences.

 
Note: See TracTickets for help on using tickets.