-
-
Notifications
You must be signed in to change notification settings - Fork 2.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
NEW : hide external user in combobox #25740
NEW : hide external user in combobox #25740
Conversation
…EW_hide_external_user_in_combobox
if (!empty($conf->global->USER_HIDE_INACTIVE_IN_COMBOBOX)) { | ||
$sql_usr .= " AND u.statut <> 0"; | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
adding this in the same time because it was missing
In Dolibarr, an external user is attached to a third-party (fk_soc > 0). So there will be a confusion between external and non employee users |
@@ -171,6 +171,9 @@ public function multiselectselectSalesRepresentatives($htmlname, $selected_array | |||
if (!empty($conf->global->USER_HIDE_INACTIVE_IN_COMBOBOX)) { | |||
$sql_usr .= " AND u2.statut<>0 "; | |||
} | |||
if (!empty($conf->global->USER_HIDE_EXTERNAL_IN_COMBOBOX)) { | |||
$sql_usr .= " AND u2.employee<>0 "; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
May be you mean USER_HIDE_NONEMPLOYEE_IN_COMBOBOX ? (external term is used for external users so fk_soc > 0).
If you really mean hide external user, condition should be AND u2.fk_soc IS NULL
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indeed ! In the end, I have added the two options for non employee and external user if it's fine for you @eldy
…hibaultf/dolibarr into NEW_hide_external_user_in_combobox
NEW|New hide external user in combobox
Adding configuration to hide external user in combobox and select