Mirrors/dolibarr
1
0
Fork 0
mirror of https://github.com/Dolibarr/dolibarr.git synced 2025-02-20 13:46:52 +01:00
Code Issues Packages Projects Releases Wiki Activity

FIX set sales representatives on create company card

Browse Source
This commit is contained in:
lvessiller 2020-08-26 09:40:20 +02:00
parent f819509442
commit 9268955256
2 changed files with 14 additions and 11 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
htdocs/societe/card.php
View File

@ -539,7 +539,7 @@ if (empty($reshook))
// Links with users
$salesreps = GETPOST('commercial', 'array');
$result = $object->setSalesRep($salesreps);
$result = $object->setSalesRep($salesreps, true);
if ($result < 0)
{
$error++;

23
htdocs/societe/class/societe.class.php
View File

@ -4133,9 +4133,10 @@ class Societe extends CommonObject
* Sets sales representatives of the thirdparty
*
* @param int[]|int $salesrep User ID or array of user IDs
* @param bool $onlyAdd Only add (no delete before)
* @return int <0 if KO, >0 if OK
*/
public function setSalesRep($salesrep)
public function setSalesRep($salesrep, $onlyAdd = false)
{
global $user;
@ -4144,16 +4145,18 @@ class Societe extends CommonObject
$salesrep = array($salesrep);
}
// Get current users
$existing = $this->getSalesRepresentatives($user, 1);
// Diff
if (is_array($existing)) {
$to_del = array_diff($existing, $salesrep);
$to_add = array_diff($salesrep, $existing);
} else {
$to_del = array(); // Nothing to delete
$to_add = $salesrep;
$to_del = array(); // Nothing to delete
$to_add = $salesrep;
if ($onlyAdd === false) {
// Get current users
$existing = $this->getSalesRepresentatives($user, 1);
// Diff
if (is_array($existing)) {
$to_del = array_diff($existing, $salesrep);
$to_add = array_diff($salesrep, $existing);
}
}
$error = 0;