Fix warnings
This commit is contained in:
parent
86b6f9c742
commit
bb436e556d
@ -796,7 +796,7 @@ class BlockedLog
|
|||||||
{
|
{
|
||||||
|
|
||||||
$res = $this->db->query("UPDATE ".MAIN_DB_PREFIX."blockedlog SET certified=1 WHERE rowid=".((int) $this->id));
|
$res = $this->db->query("UPDATE ".MAIN_DB_PREFIX."blockedlog SET certified=1 WHERE rowid=".((int) $this->id));
|
||||||
if ($res === false) {
|
if (!$res) {
|
||||||
return false;
|
return false;
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
@ -1977,7 +1977,7 @@ class FactureLigneRec extends CommonInvoiceLine
|
|||||||
$sql = 'DELETE FROM '.MAIN_DB_PREFIX.$this->table_element.' WHERE rowid='.((int) $this->id);
|
$sql = 'DELETE FROM '.MAIN_DB_PREFIX.$this->table_element.' WHERE rowid='.((int) $this->id);
|
||||||
|
|
||||||
$res = $this->db->query($sql);
|
$res = $this->db->query($sql);
|
||||||
if ($res === false) {
|
if (!$res) {
|
||||||
$error++;
|
$error++;
|
||||||
$this->errors[] = $this->db->lasterror();
|
$this->errors[] = $this->db->lasterror();
|
||||||
}
|
}
|
||||||
|
|||||||
@ -8740,7 +8740,7 @@ abstract class CommonObject
|
|||||||
$sql .= " VALUES (".implode(", ", $values).")"; // $values can contains 'abc' or 123
|
$sql .= " VALUES (".implode(", ", $values).")"; // $values can contains 'abc' or 123
|
||||||
|
|
||||||
$res = $this->db->query($sql);
|
$res = $this->db->query($sql);
|
||||||
if ($res === false) {
|
if (!$res) {
|
||||||
$error++;
|
$error++;
|
||||||
if ($this->db->lasterrno() == 'DB_ERROR_RECORD_ALREADY_EXISTS') {
|
if ($this->db->lasterrno() == 'DB_ERROR_RECORD_ALREADY_EXISTS') {
|
||||||
$this->errors[] = "ErrorRefAlreadyExists";
|
$this->errors[] = "ErrorRefAlreadyExists";
|
||||||
@ -8992,7 +8992,7 @@ abstract class CommonObject
|
|||||||
$this->db->begin();
|
$this->db->begin();
|
||||||
if (!$error) {
|
if (!$error) {
|
||||||
$res = $this->db->query($sql);
|
$res = $this->db->query($sql);
|
||||||
if ($res === false) {
|
if (!$res) {
|
||||||
$error++;
|
$error++;
|
||||||
$this->errors[] = $this->db->lasterror();
|
$this->errors[] = $this->db->lasterror();
|
||||||
}
|
}
|
||||||
|
|||||||
@ -227,7 +227,7 @@ interface Database
|
|||||||
* Note that with Mysql, this parameter is not used as Myssql can already commit a transaction even if one request is in error, without using savepoints.
|
* Note that with Mysql, this parameter is not used as Myssql can already commit a transaction even if one request is in error, without using savepoints.
|
||||||
* @param string $type Type of SQL order ('ddl' for insert, update, select, delete or 'dml' for create, alter...)
|
* @param string $type Type of SQL order ('ddl' for insert, update, select, delete or 'dml' for create, alter...)
|
||||||
* @param int $result_mode Result mode
|
* @param int $result_mode Result mode
|
||||||
* @return resource Resultset of answer
|
* @return bool|resource Resultset of answer or false
|
||||||
*/
|
*/
|
||||||
public function query($query, $usesavepoint = 0, $type = 'auto', $result_mode = 0);
|
public function query($query, $usesavepoint = 0, $type = 'auto', $result_mode = 0);
|
||||||
|
|
||||||
|
|||||||
@ -495,7 +495,7 @@ class DoliDBPgsql extends DoliDB
|
|||||||
* @param int $usesavepoint 0=Default mode, 1=Run a savepoint before and a rollback to savepoint if error (this allow to have some request with errors inside global transactions).
|
* @param int $usesavepoint 0=Default mode, 1=Run a savepoint before and a rollback to savepoint if error (this allow to have some request with errors inside global transactions).
|
||||||
* @param string $type Type of SQL order ('ddl' for insert, update, select, delete or 'dml' for create, alter...)
|
* @param string $type Type of SQL order ('ddl' for insert, update, select, delete or 'dml' for create, alter...)
|
||||||
* @param int $result_mode Result mode (not used with pgsql)
|
* @param int $result_mode Result mode (not used with pgsql)
|
||||||
* @return false|resource Resultset of answer
|
* @return bool|resource Resultset of answer
|
||||||
*/
|
*/
|
||||||
public function query($query, $usesavepoint = 0, $type = 'auto', $result_mode = 0)
|
public function query($query, $usesavepoint = 0, $type = 'auto', $result_mode = 0)
|
||||||
{
|
{
|
||||||
|
|||||||
@ -394,7 +394,7 @@ class DoliDBSqlite3 extends DoliDB
|
|||||||
* Note that with Mysql, this parameter is not used as Myssql can already commit a transaction even if one request is in error, without using savepoints.
|
* Note that with Mysql, this parameter is not used as Myssql can already commit a transaction even if one request is in error, without using savepoints.
|
||||||
* @param string $type Type of SQL order ('ddl' for insert, update, select, delete or 'dml' for create, alter...)
|
* @param string $type Type of SQL order ('ddl' for insert, update, select, delete or 'dml' for create, alter...)
|
||||||
* @param int $result_mode Result mode (not used with sqlite)
|
* @param int $result_mode Result mode (not used with sqlite)
|
||||||
* @return SQLite3Result Resultset of answer
|
* @return bool|SQLite3Result Resultset of answer
|
||||||
*/
|
*/
|
||||||
public function query($query, $usesavepoint = 0, $type = 'auto', $result_mode = 0)
|
public function query($query, $usesavepoint = 0, $type = 'auto', $result_mode = 0)
|
||||||
{
|
{
|
||||||
@ -407,6 +407,7 @@ class DoliDBSqlite3 extends DoliDB
|
|||||||
$this->error = '';
|
$this->error = '';
|
||||||
|
|
||||||
// Convert MySQL syntax to SQLite syntax
|
// Convert MySQL syntax to SQLite syntax
|
||||||
|
$reg = array();
|
||||||
if (preg_match('/ALTER\s+TABLE\s*(.*)\s*ADD\s+CONSTRAINT\s+(.*)\s*FOREIGN\s+KEY\s*\(([\w,\s]+)\)\s*REFERENCES\s+(\w+)\s*\(([\w,\s]+)\)/i', $query, $reg)) {
|
if (preg_match('/ALTER\s+TABLE\s*(.*)\s*ADD\s+CONSTRAINT\s+(.*)\s*FOREIGN\s+KEY\s*\(([\w,\s]+)\)\s*REFERENCES\s+(\w+)\s*\(([\w,\s]+)\)/i', $query, $reg)) {
|
||||||
// Ajout d'une clef étrangère à la table
|
// Ajout d'une clef étrangère à la table
|
||||||
// procédure de remplacement de la table pour ajouter la contrainte
|
// procédure de remplacement de la table pour ajouter la contrainte
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user