[#SAK-5465] discussion revise.own issues

Sakai

discussion revise.own issues

Created: 07-Jul-2006 10:15 Updated: 11-Jun-2008 12:16

Component/s:

Discussion (Retired as of 2.5)

Affects Version/s:

2.2.0, 2.2.1, 2.2.2, 2.2.3

Fix Version/s:

2.4.0

Time Tracking:

Not Specified

File Attachments:

None

Image Attachments:

1. ReviseOwnProblem.jpg
(32 kb)

2. RevisePermissions.jpg
(19 kb)

2.4.x Status:	None
2.5.x Status:	None

Description

« Hide

http://qa2-us.sakaiproject.org:9090/portal/
Firefox 1.5.0.4

I'm seeing some unusual behavior when I have deselected "revise.own" for one of my test users (eg, turned off this ability)

To replicate:
-login as instructor
-discussion
-change properties for one of the users to deselect revise.own (eg, kim_ta) [see screen capture]
-login as that user (eg, kim_ta)
-You will still have the ability to "Save as Draft" - which does not seem right.
-Additionally, if you try to edit the draft just created you will receive an error "Alert: You are not allowed to view this message" (see screen capture) when you click either "Save Draft" or "Add Reply". So basically now you have this draft that you can't do anything with.

All

Comments

Work Log

Change History

Subversion Commits

Sort Order:

Ascending order - Click to sort in descending order

[ Permlink | « Hide ]

John Leasia - 10-Jul-2006 15:45

Originally, I believe the revise permissions were meant to control the ability to edit a post that had already been made. But the UI to edit posts was never implemented. It seems revise is hooked up to drafts somehow, and probably shouldn't be. If a user makes a draft they should always have the ability to edit it. So maybe the revise.* permissions aren't really needed.

[ Permlink | « Hide ]

Zhen Qian - 31-Oct-2006 15:49

Fixed in r11717 and r11718.

Added a new allowAddDraftMessage() function into MessageChannel API, to check against "revise.any" and "revise.own" permission.

In Discussion tool, the "Save Draft" button will only be shown when the user has at aleast one of "revise.any" or "revise.own" turned on.

- Zhen

[ Permlink | « Hide ]

Megan May - 08-Feb-2007 08:43

Updating fix version from nightly2/trunk to 2.4.0.001 in prep for first QA tag (Prelim testing pre-code freeze)

[ Permlink | « Hide ]

Filter Maintainer - 10-Sep-2007 07:32

[Bulk Edit] This issue is Resolved for 2.4.0, however, there were not sufficient resources available for it to be QA'ed. As no further reports have caused it to be reopened we are assuming it was completely addressed, and are Closing it. Please feel free to reopen this issue if you are indeed experiencing problems with this specific issue's fix, otherwise, please open a new issue.