Skip to content

Exclude current user from user options when sending PM#5561

Open
rica-carv wants to merge 2 commits intoe107inc:masterfrom
rica-carv:pm_userlist
Open

Exclude current user from user options when sending PM#5561
rica-carv wants to merge 2 commits intoe107inc:masterfrom
rica-carv:pm_userlist

Conversation

@rica-carv
Copy link
Copy Markdown
Contributor

@rica-carv rica-carv commented Dec 20, 2025

Motivation and Context

The current code for the form userlist DON'T excludes current user even if the trigger 'excludeSelf' is true.
Ie: this line userlist('pm_to',null,array('excludeSelf'=>true, 'default'=>'blank', 'classes'=>varset($this->pmPrefs['send_to_class'], e_UC_MEMBER))) would ALWAYS give the list INCLUNDING current user...
Issue is visible when trying to send a PM to another user

Description

Added code line to exclude current user if trigger is set

How Has This Been Tested?

Tested on latest e107 from github

Types of Changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Performance enhancement (non-breaking change which improves efficiency)
  • Code cleanup (non-breaking change which makes code smaller or more readable)
  • Breaking change (fix or feature that would cause existing functionality to change)
  • Documentation (a change to man pages or other documentation)

Checklist

@rica-carv rica-carv changed the title Exclude current user from user options Exclude current user from user options when sending PM Dec 20, 2025
@Deltik Deltik force-pushed the master branch 3 times, most recently from 241b734 to 4f50dac Compare February 20, 2026 22:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant