Damian Krzeminski wrote: > Arjun Nair wrote: >> David Saint wrote: >>> Hi, >>> >>> In main stream rev 16690 I've noticed that a newly added user can >>> inherit any voicemail messages left behind by a previous user with the >>> same User ID. >>> >>> For example: >>> >>> - Administrator adds User 207 for Employee#1. The employee receives >>> voicemail messages. - Employee#1 leaves the company and his user >>> information is deleted by the administrator. - Employee#2 arrives and >>> the administrator gives him User 207. - Employee#2 is surprised to >>> find messages for someone else in his voicemail box. >>> >>> Is there already a jira issue for this, or should I raise one? >>> >> Do you see any errors in your sipXconfig log? From the code, it looks we >> delete the mailbox directory when we save a new user >> (EditUser.commit()). >> > > Not every time when we save the user I hope... > > I think sipXconfig only deletes old voicemail when admin renames the user. > Which seems kind of strange: voicemail probably should be stored by userid > not by username which would solve both problems (keeping voicemail on > rename and not 'inheriting' someone's voicemail). >
Ah, you are right. It's NewUser.commit() that gets called when saving a new user (not EditUser.commit()). And, as you said the mailbox is supposed to get deleted when an admin renames the user. In EditUser.commit(), MailboxManager.deleteMailbox() is called with the argument user.getUserName() after the new userName has been saved, so it will only delete mailboxes under the new username, and any mailboxes under the old username remains untouched. I also agree, saving voicemail by userid would be better, but, we would need a to communicate the userid-username mapping to sipXivr (unless we already do this). Arjun _______________________________________________ sipx-dev mailing list [email protected] List Archive: http://list.sipfoundry.org/archive/sipx-dev Unsubscribe: http://list.sipfoundry.org/mailman/listinfo/sipx-dev sipXecs IP PBX -- http://www.sipfoundry.org/
