Jasig/BlackboardVCPortlet — pull request #3 by apetro (2014-01-08 23:48:45+00:00)
Title: only send cancelation email if user actually removed

Condition sending the cancelation notice email on `removeSession[Non]Chairs()` actually having had an effect. This prevents sending unnecessary cancelation emails where the controller calls `removeSession[Non]Chairs()` blindly without knowing whether the given user is a chair or non-chair (the current, and it seems perfectly reasonable, behavior of `ManageParticipantsController`).

In practice this change de-duplicates cancellation emails: whereas currently the implementation will send an invitee two cancelation emails when he or she is removed from the invitees list (one for the attempt to remove from the set of chairs and one for the attempt to remove from the set of non-chairs), with this change the portlet will only send one such email (for whichever attempt had an actual effect).