We're updating the issue view to help you get more done. 

NPE when admin deletes logged in owner from members-only room

Description

When admin deletes a currently-logged in owner from a members-only room's owner list (via Admin console), the result is this NPE:

The owner is deleted, so the NPE is more of a nuisance.

Environment

None

Acceptance Test - Entry

None

Activity

Show:
csh
June 9, 2014, 8:43 PM

Daryl, would you test this again with the latest trunk version?
I can't reproduce this. Furthermore affiliation changes are now managed by IQAdminHandler.
I only see a NotAllowedException in the logs (), if the logged in user has also joined the room.

Daryl Herzmann
June 9, 2014, 9:03 PM

@csh , when I attempt this with trunk, nothing happens. The room owner remains in the ACL eventhough the console UI indicates "User removed successfully." No exceptions are logged.

csh
June 9, 2014, 9:24 PM

Yes, I noticed this too after doing https://github.com/igniterealtime/Openfire/commit/36d45788598e94b06c20c507352d3f6d9de2ee4d

I have solved this with https://github.com/igniterealtime/Openfire/commit/1651423c29c60add05d2738147a9b960f6de8129

You probably don't have this commit or you have do rebuild or something.

Daryl Herzmann
June 10, 2014, 11:48 AM

@csh, sorry, I didn't notice your commit prior to my building. Anyway, I built fresh this morning and removing an admin from the admin console works as expected. The user was not booted from the room, but that would be a different issue than this one.

Daryl Herzmann
October 31, 2015, 3:14 AM

this issue was fixed by CSH, closing. Can reopen if others disagree

Assignee

Daryl Herzmann

Reporter

Derek DeMoro

Labels

None

Expected Effort

None

Ignite Forum URL

None

Components

Fix versions

Affects versions

Priority

Major
Configure