Controller changes
The functionality for removal, was already in place.
There was an enforcement of two conditions:
- User must have Can Edit policy on the conpherence
- remove_person phid should be same as $user->getUserID (which I think is coming via the logged in session)
What I've done is just remove the second condition and let the first condition stay as is.
View updates
In the view, the previous logic was to display the a tag with fa-times icon only when userID from the participants array matched the userID of the logged in person.
I've changed this to -> Show that icon for all users.
I know this is wrong and would create problems, wherein, even users with View policy could see that icon, but since major validations have been taken care at the Controller level, this is just a cosmetic issue and with a little help, I could fix it too.
