Fix: uniformize code

This commit is contained in:
Regis Houssin 2011-01-15 09:31:09 +00:00
parent 07046ce3b4
commit 65f75f309e
3 changed files with 26 additions and 21 deletions

View File

@ -3,7 +3,7 @@
* Copyright (C) 2004 Eric Seigne <eric.seigne@ryxeo.com> * Copyright (C) 2004 Eric Seigne <eric.seigne@ryxeo.com>
* Copyright (C) 2004-2009 Laurent Destailleur <eldy@users.sourceforge.net> * Copyright (C) 2004-2009 Laurent Destailleur <eldy@users.sourceforge.net>
* Copyright (C) 2005 Marc Barilley / Ocebo <marc@ocebo.com> * Copyright (C) 2005 Marc Barilley / Ocebo <marc@ocebo.com>
* Copyright (C) 2005-2010 Regis Houssin <regis@dolibarr.fr> * Copyright (C) 2005-2011 Regis Houssin <regis@dolibarr.fr>
* Copyright (C) 2006 Andre Cianfarani <acianfa@free.fr> * Copyright (C) 2006 Andre Cianfarani <acianfa@free.fr>
* Copyright (C) 2008 Raphael Bertrand (Resultic) <raphael.bertrand@resultic.fr> * Copyright (C) 2008 Raphael Bertrand (Resultic) <raphael.bertrand@resultic.fr>
* Copyright (C) 2010 Juanjo Menent <jmenent@2byte.es> * Copyright (C) 2010 Juanjo Menent <jmenent@2byte.es>
@ -91,7 +91,9 @@ class Propal extends CommonObject
var $remise_absolue; var $remise_absolue;
var $note; var $note;
var $note_public; var $note_public;
var $fk_delivery_address; var $fk_delivery_address; // deprecated (for compatibility)
var $fk_address;
var $address_type;
var $adresse; var $adresse;
var $products=array(); var $products=array();
@ -952,7 +954,8 @@ class Propal extends CommonObject
$this->datep = $this->db->jdate($obj->dp); $this->datep = $this->db->jdate($obj->dp);
$this->fin_validite = $this->db->jdate($obj->dfv); $this->fin_validite = $this->db->jdate($obj->dfv);
$this->date_livraison = $this->db->jdate($obj->date_livraison); $this->date_livraison = $this->db->jdate($obj->date_livraison);
$this->fk_delivery_address = $obj->fk_adresse_livraison; $this->fk_delivery_address = $obj->fk_adresse_livraison; // TODO obsolete
$this->fk_address = $obj->fk_adresse_livraison;
$this->mode_reglement_id = $obj->fk_mode_reglement; $this->mode_reglement_id = $obj->fk_mode_reglement;
$this->mode_reglement_code = $obj->mode_reglement_code; $this->mode_reglement_code = $obj->mode_reglement_code;
@ -1201,16 +1204,16 @@ class Propal extends CommonObject
* \param adresse_livraison Adresse de livraison * \param adresse_livraison Adresse de livraison
* \return int <0 si ko, >0 si ok * \return int <0 si ko, >0 si ok
*/ */
function set_adresse_livraison($user, $address_id) function set_adresse_livraison($user, $fk_address)
{ {
if ($user->rights->propale->creer) if ($user->rights->propale->creer)
{ {
$sql = "UPDATE ".MAIN_DB_PREFIX."propal SET fk_adresse_livraison = '".$address_id."'"; $sql = "UPDATE ".MAIN_DB_PREFIX."propal SET fk_adresse_livraison = '".$fk_address."'";
$sql.= " WHERE rowid = ".$this->id." AND fk_statut = 0"; $sql.= " WHERE rowid = ".$this->id." AND fk_statut = 0";
if ($this->db->query($sql) ) if ($this->db->query($sql) )
{ {
$this->fk_delivery_address = $address_id; $this->fk_delivery_address = $fk_address;
return 1; return 1;
} }
else else

View File

@ -412,7 +412,7 @@ class CommonObject
$sql.= " ".MAIN_DB_PREFIX."c_type_contact as tc"; $sql.= " ".MAIN_DB_PREFIX."c_type_contact as tc";
$sql.= " WHERE ec.element_id = ".$this->id; $sql.= " WHERE ec.element_id = ".$this->id;
$sql.= " AND ec.fk_socpeople = c.rowid"; $sql.= " AND ec.fk_socpeople = c.rowid";
$sql.= " AND c.entity = ".$conf->entity; $sql.= " AND c.entity IN (0,".$conf->entity.")";
$sql.= " AND ec.fk_c_type_contact = tc.rowid"; $sql.= " AND ec.fk_c_type_contact = tc.rowid";
$sql.= " AND tc.element = '".$this->element."'"; $sql.= " AND tc.element = '".$this->element."'";
$sql.= " AND tc.source = '".$source."'"; $sql.= " AND tc.source = '".$source."'";
@ -507,17 +507,17 @@ class CommonObject
} }
/** /**
* \brief Charge l'adresse de livraison d'id $this->fk_delivery_address dans this->deliveryaddress * Charge l'adresse d'id $this->fk_address dans this->address
* \param userid Id du contact * @param fk_address Id de l'adresse
* \return int <0 if KO, >0 if OK * @return int <0 if KO, >0 if OK
*/ */
function fetch_adresse_livraison($deliveryaddressid) function fetch_address($fk_address)
{ {
$address = new Societe($this->db); $object = new Societe($this->db);
$result=$address->fetch_adresse_livraison($deliveryaddressid); $result=$object->fetch_address($fk_address);
$this->deliveryaddress = $address; $this->deliveryaddress = $object; // TODO obsolete
$this->adresse = $address; // TODO obsolete $this->adresse = $object; // TODO obsolete
$this->address = $address; $this->address = $object;
return $result; return $result;
} }

View File

@ -167,8 +167,10 @@ class User extends CommonObject
$this->ref = $obj->rowid; $this->ref = $obj->rowid;
$this->ldap_sid = $obj->ldap_sid; $this->ldap_sid = $obj->ldap_sid;
$this->nom = $obj->name; $this->nom = $obj->name; // TODO deprecated
$this->prenom = $obj->firstname; $this->lastname = $obj->name;
$this->prenom = $obj->firstname; // TODO deprecated
$this->firstname = $obj->firstname;
$this->login = $obj->login; $this->login = $obj->login;
$this->pass_indatabase = $obj->pass; $this->pass_indatabase = $obj->pass;