lainwir3d/dolibarr
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Merge pull request #10424 from EuskalMoneta/7.0

Browse Source 
FIX 'statut' is ignored when updating a user with the REST API.
This commit is contained in:
Laurent Destailleur 2019-01-27 17:56:14 +01:00 committed by GitHub
commit 3e78807cbc
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 14 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
htdocs/user/class/api_users.class.php
View File

@ -212,11 +212,22 @@ class Users extends DolibarrApi
foreach ($request_data as $field => $value) foreach ($request_data as $field => $value)
{ {
if ($field == 'id') continue; if ($field == 'id') continue;
$this->useraccount->$field = $value; // The status must be updated using setstatus() because it
// is not handled by the update() method.
if ($field == 'statut') {
$result = $this->useraccount->setstatus($value);
if ($result < 0) {
throw new RestException(500, 'Error when updating status of user: '.$this->useraccount->error);
}
} else {
$this->useraccount->$field = $value;
}
} }
if ($this->useraccount->update(DolibarrApiAccess::$user) > 0) // If there is no error, update() returns the number of affected
// rows so if the update is a no op, the return value is zezo.
if ($this->useraccount->update(DolibarrApiAccess::$user) >= 0)
{ {
return $this->get($id); return $this->get($id);
} }