Skip to:

  1. Skip to Jira Navigation
  2. Skip to Side Navigation
  3. Skip to Main Content

PermissionException should extend SakaiException

Description

In KNL-912 a nice refactor of the Exception hierarchy was done, but it seems Permission Exception was overlooked.

Attachments

1

Activity

Show:

Matthew Jones February 18, 2014 at 5:09 PM

Merged both of the changes to 10.x.

Hudson CI Server February 18, 2014 at 4:29 PM

Integrated in kernel-trunk #783 (See http://builds.sakaiproject.org:8080/job/kernel-trunk/783/)
Fix so that we still return a good message from getMessage() on PermissionException. (Revision 306153)

Result = SUCCESS

Matthew Jones February 18, 2014 at 3:51 PM

Ok, yea, I was wondering about that UNSTABLE result.

@SLRunning org.sakaiproject.exception.PermissionExceptionTest

Matthew Buckett February 18, 2014 at 3:37 PM

And it broke my test I've fixed it up and will put the fix to the test on this ticket as well. Maybe this was why it wasn't included in the first place.

Hudson CI Server February 18, 2014 at 12:44 PM

Integrated in kernel-trunk #782 (See http://builds.sakaiproject.org:8080/job/kernel-trunk/782/)
Make PermissionException extend SakaiException.

This should probably have been in the original commit that introduced SakaiException but got missed. (Revision 306142)

Result = UNSTABLE

Fixed

Details

Priority

Affects versions

Fix versions

Components

Assignee

Reporter

Labels

Created February 18, 2014 at 10:30 AM
Updated March 23, 2022 at 3:02 PM
Resolved February 18, 2014 at 11:19 AM