Edgewall Software
Modify

Opened 16 years ago

Closed 14 years ago

Last modified 14 years ago

#7507 closed defect (fixed)

No diff shown in changeset view even with FILE_VIEW permission

Reported by: nick@… Owned by: Remy Blank
Priority: normal Milestone: 0.12
Component: version control/changeset view Version: 0.12dev
Severity: minor Keywords:
Cc: Branch:
Release Notes:
API Changes:
Internal Changes:

Description

If a user is created with CHANGESET_VIEW and FILE_VIEW permissions I would expect them to be able to see diffs in the changeset view, view the list of any changed files, along with being able to follow the link to the files themselves. Currently you can only see these items if you also have the BROWSER_VIEW permission.

I've attached a patch, produced against trunk r7444, which produces the behaviour I expected.

If this isn't the desired behaviour, would it be possible to update the documentation to highlight the current behaviour? I spent a while trying to figure out why anonymous users couldn't see files in the changeset, even when the, apparently, appropriate permissions were set, and think it would prevent other people from encountering the same issue.

This behaviour was introduced in r3384 by a patch from issue #2268.

Attachments (1)

patch.txt (676 bytes ) - added by nick@… 16 years ago.
Fixes behaviour highlighted in ticket #7507

Download all attachments as: .zip

Change History (4)

by nick@…, 16 years ago

Attachment: patch.txt added

Fixes behaviour highlighted in ticket #7507

comment:1 by osimons, 16 years ago

Milestone: 0.11.2
Owner: set to osimons

Need to look into this and assess the impact of the problem and the proposed change.

Tentatively scheduling for 0.11-stable (as a regression-type bug), but if it constitues a change from established working behaviour it should be 0.12 at the earliest.

comment:2 by Remy Blank, 14 years ago

Milestone: next-minor-0.12.x0.12
Resolution: fixed
Status: newclosed

This should be fixed on trunk with the recent authz changes. Please reopen if you find any issues with current trunk.

comment:3 by Remy Blank, 14 years ago

Owner: changed from osimons to Remy Blank

Modify Ticket

Change Properties
Set your email in Preferences
Action
as closed The owner will remain Remy Blank.
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from Remy Blank to the specified user.

Add Comment


E-mail address and name can be saved in the Preferences .
 
Note: See TracTickets for help on using tickets.