Fix duplicate code
This commit is contained in:
Laurent Destailleur
parent
79c9745442
commit
a71541e6de
@ -318,178 +318,175 @@ if (empty($reshook)) {
|
|||||||
$error ++;
|
$error ++;
|
||||||
}
|
}
|
||||||
|
|
||||||
if (!$error) {
|
if (!$error)
|
||||||
|
{
|
||||||
$object->fetch($id);
|
$object->fetch($id);
|
||||||
|
|
||||||
$object->oldcopy = clone $object;
|
$object->oldcopy = clone $object;
|
||||||
|
|
||||||
if (!$error) {
|
$db->begin();
|
||||||
$db->begin();
|
|
||||||
|
|
||||||
$object->oldcopy = clone $object;
|
$object->lastname = GETPOST("lastname", 'alpha');
|
||||||
|
$object->firstname = GETPOST("firstname", 'alpha');
|
||||||
|
$object->login = GETPOST("login", 'alpha');
|
||||||
|
$object->gender = GETPOST("gender", 'alpha');
|
||||||
|
$object->pass = GETPOST("password");
|
||||||
|
$object->api_key = (GETPOST("api_key", 'alpha')) ? GETPOST("api_key", 'alpha') : $object->api_key;
|
||||||
|
if (! empty($user->admin)) $object->admin = GETPOST("admin"); // admin flag can only be set/unset by an admin user. A test is also done later when forging sql request
|
||||||
|
$object->address = GETPOST('address', 'alpha');
|
||||||
|
$object->zip = GETPOST('zipcode', 'alpha');
|
||||||
|
$object->town = GETPOST('town', 'alpha');
|
||||||
|
$object->country_id = GETPOST('country_id', 'int');
|
||||||
|
$object->state_id = GETPOST('state_id', 'int');
|
||||||
|
$object->office_phone = GETPOST("office_phone", 'alpha');
|
||||||
|
$object->office_fax = GETPOST("office_fax", 'alpha');
|
||||||
|
$object->user_mobile = GETPOST("user_mobile");
|
||||||
|
$object->skype = GETPOST("skype", 'alpha');
|
||||||
|
$object->email = GETPOST("email", 'alpha');
|
||||||
|
$object->job = GETPOST("job", 'alpha');
|
||||||
|
$object->signature = GETPOST("signature");
|
||||||
|
$object->accountancy_code = GETPOST("accountancy_code");
|
||||||
|
$object->openid = GETPOST("openid");
|
||||||
|
$object->fk_user = GETPOST("fk_user") > 0 ? GETPOST("fk_user") : 0;
|
||||||
|
$object->employee = GETPOST('employee');
|
||||||
|
|
||||||
$object->lastname = GETPOST("lastname", 'alpha');
|
$object->thm = GETPOST("thm") != '' ? GETPOST("thm") : '';
|
||||||
$object->firstname = GETPOST("firstname", 'alpha');
|
$object->tjm = GETPOST("tjm") != '' ? GETPOST("tjm") : '';
|
||||||
$object->login = GETPOST("login", 'alpha');
|
$object->salary = GETPOST("salary") != '' ? GETPOST("salary") : '';
|
||||||
$object->gender = GETPOST("gender", 'alpha');
|
$object->salaryextra = GETPOST("salaryextra") != '' ? GETPOST("salaryextra") : '';
|
||||||
$object->pass = GETPOST("password");
|
$object->weeklyhours = GETPOST("weeklyhours") != '' ? GETPOST("weeklyhours") : '';
|
||||||
$object->api_key = (GETPOST("api_key", 'alpha')) ? GETPOST("api_key", 'alpha') : $object->api_key;
|
|
||||||
if (! empty($user->admin)) $object->admin = GETPOST("admin"); // admin flag can only be set/unset by an admin user. A test is also done later when forging sql request
|
|
||||||
$object->address = GETPOST('address', 'alpha');
|
|
||||||
$object->zip = GETPOST('zipcode', 'alpha');
|
|
||||||
$object->town = GETPOST('town', 'alpha');
|
|
||||||
$object->country_id = GETPOST('country_id', 'int');
|
|
||||||
$object->state_id = GETPOST('state_id', 'int');
|
|
||||||
$object->office_phone = GETPOST("office_phone", 'alpha');
|
|
||||||
$object->office_fax = GETPOST("office_fax", 'alpha');
|
|
||||||
$object->user_mobile = GETPOST("user_mobile");
|
|
||||||
$object->skype = GETPOST("skype", 'alpha');
|
|
||||||
$object->email = GETPOST("email", 'alpha');
|
|
||||||
$object->job = GETPOST("job", 'alpha');
|
|
||||||
$object->signature = GETPOST("signature");
|
|
||||||
$object->accountancy_code = GETPOST("accountancy_code");
|
|
||||||
$object->openid = GETPOST("openid");
|
|
||||||
$object->fk_user = GETPOST("fk_user") > 0 ? GETPOST("fk_user") : 0;
|
|
||||||
$object->employee = GETPOST('employee');
|
|
||||||
|
|
||||||
$object->thm = GETPOST("thm") != '' ? GETPOST("thm") : '';
|
$object->color = GETPOST("color") != '' ? GETPOST("color") : '';
|
||||||
$object->tjm = GETPOST("tjm") != '' ? GETPOST("tjm") : '';
|
$dateemployment = dol_mktime(0, 0, 0, GETPOST('dateemploymentmonth'), GETPOST('dateemploymentday'), GETPOST('dateemploymentyear'));
|
||||||
$object->salary = GETPOST("salary") != '' ? GETPOST("salary") : '';
|
$object->dateemployment = $dateemployment;
|
||||||
$object->salaryextra = GETPOST("salaryextra") != '' ? GETPOST("salaryextra") : '';
|
|
||||||
$object->weeklyhours = GETPOST("weeklyhours") != '' ? GETPOST("weeklyhours") : '';
|
|
||||||
|
|
||||||
$object->color = GETPOST("color") != '' ? GETPOST("color") : '';
|
if (! empty($conf->multicompany->enabled))
|
||||||
$dateemployment = dol_mktime(0, 0, 0, GETPOST('dateemploymentmonth'), GETPOST('dateemploymentday'), GETPOST('dateemploymentyear'));
|
{
|
||||||
$object->dateemployment = $dateemployment;
|
if (! empty($_POST["superadmin"]))
|
||||||
|
|
||||||
if (! empty($conf->multicompany->enabled))
|
|
||||||
{
|
{
|
||||||
if (! empty($_POST["superadmin"]))
|
$object->entity = 0;
|
||||||
{
|
}
|
||||||
$object->entity = 0;
|
else if ($conf->multicompany->transverse_mode)
|
||||||
}
|
{
|
||||||
else if ($conf->multicompany->transverse_mode)
|
$object->entity = 1; // all users in master entity
|
||||||
{
|
|
||||||
$object->entity = 1; // all users in master entity
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
$object->entity = (! GETPOST('entity', 'int') ? 0 : GETPOST('entity', 'int'));
|
|
||||||
}
|
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
{
|
{
|
||||||
$object->entity = (! GETPOST('entity', 'int') ? 0 : GETPOST('entity', 'int'));
|
$object->entity = (! GETPOST('entity', 'int') ? 0 : GETPOST('entity', 'int'));
|
||||||
}
|
}
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
$object->entity = (! GETPOST('entity', 'int') ? 0 : GETPOST('entity', 'int'));
|
||||||
|
}
|
||||||
|
|
||||||
// Fill array 'array_options' with data from add form
|
// Fill array 'array_options' with data from add form
|
||||||
$ret = $extrafields->setOptionalsFromPost($extralabels, $object);
|
$ret = $extrafields->setOptionalsFromPost($extralabels, $object);
|
||||||
|
if ($ret < 0) {
|
||||||
|
$error ++;
|
||||||
|
}
|
||||||
|
|
||||||
|
if (GETPOST('deletephoto')) {
|
||||||
|
$object->photo = '';
|
||||||
|
}
|
||||||
|
if (!empty($_FILES['photo']['name'])) {
|
||||||
|
$object->photo = dol_sanitizeFileName($_FILES['photo']['name']);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!$error) {
|
||||||
|
$ret = $object->update($user);
|
||||||
if ($ret < 0) {
|
if ($ret < 0) {
|
||||||
$error ++;
|
$error++;
|
||||||
}
|
if ($db->errno() == 'DB_ERROR_RECORD_ALREADY_EXISTS') {
|
||||||
|
|
||||||
if (GETPOST('deletephoto')) {
|
|
||||||
$object->photo = '';
|
|
||||||
}
|
|
||||||
if (!empty($_FILES['photo']['name'])) {
|
|
||||||
$object->photo = dol_sanitizeFileName($_FILES['photo']['name']);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$error) {
|
|
||||||
$ret = $object->update($user);
|
|
||||||
if ($ret < 0) {
|
|
||||||
$error++;
|
|
||||||
if ($db->errno() == 'DB_ERROR_RECORD_ALREADY_EXISTS') {
|
|
||||||
$langs->load("errors");
|
|
||||||
setEventMessages($langs->trans("ErrorLoginAlreadyExists", $object->login), null, 'errors');
|
|
||||||
}
|
|
||||||
else
|
|
||||||
{
|
|
||||||
setEventMessages($object->error, $object->errors, 'errors');
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$error && isset($_POST['contactid'])) {
|
|
||||||
$contactid = GETPOST('contactid', 'int');
|
|
||||||
|
|
||||||
if ($contactid > 0) {
|
|
||||||
$contact = new Contact($db);
|
|
||||||
$contact->fetch($contactid);
|
|
||||||
|
|
||||||
$sql = "UPDATE ".MAIN_DB_PREFIX."user";
|
|
||||||
$sql .= " SET fk_socpeople=".$db->escape($contactid);
|
|
||||||
if (!empty($contact->socid)) {
|
|
||||||
$sql .= ", fk_soc=".$db->escape($contact->socid);
|
|
||||||
}
|
|
||||||
$sql .= " WHERE rowid=".$object->id;
|
|
||||||
} else {
|
|
||||||
$sql = "UPDATE ".MAIN_DB_PREFIX."user";
|
|
||||||
$sql .= " SET fk_socpeople=NULL, fk_soc=NULL";
|
|
||||||
$sql .= " WHERE rowid=".$object->id;
|
|
||||||
}
|
|
||||||
dol_syslog("usercard::update", LOG_DEBUG);
|
|
||||||
$resql = $db->query($sql);
|
|
||||||
if (!$resql) {
|
|
||||||
$error ++;
|
|
||||||
setEventMessages($db->lasterror(), null, 'errors');
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$error && !count($object->errors)) {
|
|
||||||
if (GETPOST('deletephoto') && $object->photo) {
|
|
||||||
$fileimg = $conf->user->dir_output.'/'.get_exdir($object->id, 2, 0, 1, $object, 'user').'/logos/'.$object->photo;
|
|
||||||
$dirthumbs = $conf->user->dir_output.'/'.get_exdir($object->id, 2, 0, 1, $object, 'user').'/logos/thumbs';
|
|
||||||
dol_delete_file($fileimg);
|
|
||||||
dol_delete_dir_recursive($dirthumbs);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (isset($_FILES['photo']['tmp_name']) && trim($_FILES['photo']['tmp_name'])) {
|
|
||||||
$dir = $conf->user->dir_output.'/'.get_exdir($object->id, 2, 0, 1, $object, 'user');
|
|
||||||
|
|
||||||
dol_mkdir($dir);
|
|
||||||
|
|
||||||
if (@is_dir($dir)) {
|
|
||||||
$newfile = $dir.'/'.dol_sanitizeFileName($_FILES['photo']['name']);
|
|
||||||
$result = dol_move_uploaded_file($_FILES['photo']['tmp_name'], $newfile, 1, 0, $_FILES['photo']['error']);
|
|
||||||
|
|
||||||
if (!$result > 0) {
|
|
||||||
setEventMessages($langs->trans("ErrorFailedToSaveFile"), null, 'errors');
|
|
||||||
} else {
|
|
||||||
// Create thumbs
|
|
||||||
$object->addThumbs($newfile);
|
|
||||||
}
|
|
||||||
} else {
|
|
||||||
$error ++;
|
|
||||||
$langs->load("errors");
|
|
||||||
setEventMessages($langs->trans("ErrorFailedToCreateDir", $dir), $mesgs, 'errors');
|
|
||||||
}
|
|
||||||
}
|
|
||||||
}
|
|
||||||
|
|
||||||
if (! $error && ! count($object->errors))
|
|
||||||
{
|
|
||||||
// Then we add the associated categories
|
|
||||||
$categories = GETPOST( 'usercats', 'array' );
|
|
||||||
$object->setCategories($categories);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$error && !count($object->errors)) {
|
|
||||||
setEventMessages($langs->trans("UserModified"), null, 'mesgs');
|
|
||||||
$db->commit();
|
|
||||||
|
|
||||||
$login = $_SESSION["dol_login"];
|
|
||||||
if ($login && $login == $object->oldcopy->login && $object->oldcopy->login != $object->login) // Current user has changed its login
|
|
||||||
{
|
|
||||||
$error++;
|
|
||||||
$langs->load("errors");
|
$langs->load("errors");
|
||||||
setEventMessages($langs->transnoentitiesnoconv("WarningYourLoginWasModifiedPleaseLogin"), null, 'warnings');
|
setEventMessages($langs->trans("ErrorLoginAlreadyExists", $object->login), null, 'errors');
|
||||||
|
}
|
||||||
|
else
|
||||||
|
{
|
||||||
|
setEventMessages($object->error, $object->errors, 'errors');
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
else {
|
}
|
||||||
$db->rollback();
|
|
||||||
|
if (!$error && isset($_POST['contactid'])) {
|
||||||
|
$contactid = GETPOST('contactid', 'int');
|
||||||
|
|
||||||
|
if ($contactid > 0) {
|
||||||
|
$contact = new Contact($db);
|
||||||
|
$contact->fetch($contactid);
|
||||||
|
|
||||||
|
$sql = "UPDATE ".MAIN_DB_PREFIX."user";
|
||||||
|
$sql .= " SET fk_socpeople=".$db->escape($contactid);
|
||||||
|
if (!empty($contact->socid)) {
|
||||||
|
$sql .= ", fk_soc=".$db->escape($contact->socid);
|
||||||
|
}
|
||||||
|
$sql .= " WHERE rowid=".$object->id;
|
||||||
|
} else {
|
||||||
|
$sql = "UPDATE ".MAIN_DB_PREFIX."user";
|
||||||
|
$sql .= " SET fk_socpeople=NULL, fk_soc=NULL";
|
||||||
|
$sql .= " WHERE rowid=".$object->id;
|
||||||
}
|
}
|
||||||
}
|
dol_syslog("usercard::update", LOG_DEBUG);
|
||||||
|
$resql = $db->query($sql);
|
||||||
|
if (!$resql) {
|
||||||
|
$error ++;
|
||||||
|
setEventMessages($db->lasterror(), null, 'errors');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!$error && !count($object->errors)) {
|
||||||
|
if (GETPOST('deletephoto') && $object->photo) {
|
||||||
|
$fileimg = $conf->user->dir_output.'/'.get_exdir($object->id, 2, 0, 1, $object, 'user').'/logos/'.$object->photo;
|
||||||
|
$dirthumbs = $conf->user->dir_output.'/'.get_exdir($object->id, 2, 0, 1, $object, 'user').'/logos/thumbs';
|
||||||
|
dol_delete_file($fileimg);
|
||||||
|
dol_delete_dir_recursive($dirthumbs);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (isset($_FILES['photo']['tmp_name']) && trim($_FILES['photo']['tmp_name'])) {
|
||||||
|
$dir = $conf->user->dir_output.'/'.get_exdir($object->id, 2, 0, 1, $object, 'user');
|
||||||
|
|
||||||
|
dol_mkdir($dir);
|
||||||
|
|
||||||
|
if (@is_dir($dir)) {
|
||||||
|
$newfile = $dir.'/'.dol_sanitizeFileName($_FILES['photo']['name']);
|
||||||
|
$result = dol_move_uploaded_file($_FILES['photo']['tmp_name'], $newfile, 1, 0, $_FILES['photo']['error']);
|
||||||
|
|
||||||
|
if (!$result > 0) {
|
||||||
|
setEventMessages($langs->trans("ErrorFailedToSaveFile"), null, 'errors');
|
||||||
|
} else {
|
||||||
|
// Create thumbs
|
||||||
|
$object->addThumbs($newfile);
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
$error ++;
|
||||||
|
$langs->load("errors");
|
||||||
|
setEventMessages($langs->trans("ErrorFailedToCreateDir", $dir), $mesgs, 'errors');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
if (! $error && ! count($object->errors))
|
||||||
|
{
|
||||||
|
// Then we add the associated categories
|
||||||
|
$categories = GETPOST( 'usercats', 'array' );
|
||||||
|
$object->setCategories($categories);
|
||||||
|
}
|
||||||
|
|
||||||
|
if (!$error && !count($object->errors)) {
|
||||||
|
setEventMessages($langs->trans("UserModified"), null, 'mesgs');
|
||||||
|
$db->commit();
|
||||||
|
|
||||||
|
$login = $_SESSION["dol_login"];
|
||||||
|
if ($login && $login == $object->oldcopy->login && $object->oldcopy->login != $object->login) // Current user has changed its login
|
||||||
|
{
|
||||||
|
$error++;
|
||||||
|
$langs->load("errors");
|
||||||
|
setEventMessages($langs->transnoentitiesnoconv("WarningYourLoginWasModifiedPleaseLogin"), null, 'warnings');
|
||||||
|
}
|
||||||
|
}
|
||||||
|
else {
|
||||||
|
$db->rollback();
|
||||||
|
}
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
else
|
else
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user