Fix merge lost
This commit is contained in:
parent
d0fd00c4e9
commit
024e1673e3
@ -1495,15 +1495,17 @@ class Ticket extends CommonObject
|
|||||||
* 1- create entry into database for message storage
|
* 1- create entry into database for message storage
|
||||||
* 2- if trigger, send an email to ticket contacts
|
* 2- if trigger, send an email to ticket contacts
|
||||||
*
|
*
|
||||||
* @param User $user User that create
|
* @param User $user User that create
|
||||||
* @param string $message Log message
|
* @param string $message Log message
|
||||||
* @param int $noemail 0=send email after, 1=disable emails
|
* @param int $noemail 0=send email after, 1=disable emails
|
||||||
* @return int <0 if KO, >0 if OK
|
* @return int <0 if KO, >0 if OK
|
||||||
*/
|
*/
|
||||||
public function createTicketLog(User $user, $message, $noemail = 0)
|
public function createTicketLog(User $user, $message, $noemail = 0)
|
||||||
{
|
{
|
||||||
global $conf, $langs;
|
global $conf, $langs;
|
||||||
|
|
||||||
|
$error = 0;
|
||||||
|
|
||||||
$this->db->begin();
|
$this->db->begin();
|
||||||
|
|
||||||
// Clean parameters
|
// Clean parameters
|
||||||
@ -1515,37 +1517,23 @@ class Ticket extends CommonObject
|
|||||||
return -1;
|
return -1;
|
||||||
}
|
}
|
||||||
|
|
||||||
// Insert request
|
// TODO Should call the trigger TICKET_MODIFY with $this->context with all data to record event
|
||||||
$sql = "INSERT INTO " . MAIN_DB_PREFIX . "ticket_logs(";
|
// so the event is stored by the agenda/event trigger
|
||||||
$sql .= "entity,";
|
|
||||||
$sql .= "datec,";
|
|
||||||
$sql .= "fk_track_id,";
|
|
||||||
$sql .= "fk_user_create,";
|
|
||||||
$sql .= "message";
|
|
||||||
$sql .= ") VALUES (";
|
|
||||||
$sql .= " " . $conf->entity . ",";
|
|
||||||
$sql .= " '" . $this->db->idate(dol_now()) . "',";
|
|
||||||
$sql .= " '" . $this->db->escape($this->track_id) . "',";
|
|
||||||
$sql .= " " . ($user->id > 0 ? $user->id : 'NULL') . ",";
|
|
||||||
$sql .= " '" . $this->db->escape($message) . "'";
|
|
||||||
$sql .= ")";
|
|
||||||
|
|
||||||
dol_syslog(get_class($this) . "::create_ticket_log sql=" . $sql, LOG_DEBUG);
|
if (!$error) {
|
||||||
$resql = $this->db->query($sql);
|
$this->db->commit();
|
||||||
if ($resql) {
|
|
||||||
if ($conf->global->TICKET_ACTIVATE_LOG_BY_EMAIL && !$noemail) {
|
|
||||||
$this->sendLogByEmail($user, $message);
|
|
||||||
}
|
|
||||||
|
|
||||||
if (!$error) {
|
if ($conf->global->TICKET_ACTIVATE_LOG_BY_EMAIL && !$noemail) {
|
||||||
$this->db->commit();
|
$this->sendLogByEmail($user, $message);
|
||||||
return 1;
|
}
|
||||||
}
|
|
||||||
} else {
|
return 1;
|
||||||
$this->db->rollback();
|
}
|
||||||
$this->error = "Error " . $this->db->lasterror();
|
else
|
||||||
dol_syslog(get_class($this) . "::create_ticket_log " . $this->error, LOG_ERR);
|
{
|
||||||
return -1;
|
$this->db->rollback();
|
||||||
|
|
||||||
|
return -1;
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user