Re: [Dolibarr-dev] Question sur showrefnav / Question on showrefnav

[Destailleur Laurent]

La question en amène une autre.

Pourquoi aurions-nous des éléments qui n'ont pas de ->ref

En effet, tout élément est identifié par

->id   id technique non visible de l'utilisateur (rowid en base de type int et autoincrément)

mais aussi par

->ref   id fonctionnel visible de l'utilisateur (champ parfois en string et généré par module)

souvent ->id et ->ref sont différent mais parfois ils sont identiques pour certains éléments qui n'ont pas de ref. 

Donc au niveau de la base, il n'y a q'un champ (rowid) mais au niveau du code par contre, cela doit rester séparé. La technique utilisée et de positionner la proprité ->ref en meme tant qu'on positionne ->id quand on fait un fetch sur tel élément.

Ainsi, on garde l'homogénétié de fonctionnement.

La methode show_ref_nav est consort servent à faire suivant/précédent pour l'utilisateur, donc cela doit se faire sur le champ ->ref (champ visible de l'utilisateur). Il faut selon moi conserver cela. Il vaut mieux uniformiser le code pour nommer touts les champs ref en ref (il y a encore les facture qui utiliser ->facnum) plutot que de modifier pour pouvoir utiliser des libelles de champs différents.

2012/5/2 Florian Henry <address@hidden>

Bonjour a tous,

  Dans la branche develop, J'ai une question sur la méthode (html.form.class)showrefnav et son acolyte (CommonObject)load_previous_next_ref

  Pour déterminer le next id et le prev id, on utilise systématiquement, dans (CommonObject)load_previous_next_ref : $this->ref  (au niveau de la requete SQL < ou >) alors que dans la méthode (html.form.class)showrefnav, il y a bien un paramètre $fieldref qui n'est jamais utiliser. Voyez vous un inconvénient a ce que l'on transmette ce paramètre à (CommonObject)load_previous_next_ref pour ne plus être obliger de déclarer la propriété ref sur des objects qui n'en on pas ?
 Par exemple en remplacant dans (CommonObject)load_previous_next_ref
       $sql.= " WHERE te.".$fieldid." < '".$this->db->escape($this->ref)."'";
par
       $sql.= " WHERE te.".$fieldid." < '".$this->db->escape($this->$fieldref)."'";

Cordialement

-----------------------------------------------

Dear all,

  In the develop branch, I've got a question/remark on  method (html.form.class)showrefnav and her friend (CommonObject)load_previous_next_ref

   To determine the next id and prev id, we use, in (CommonObject)load_previous_next_ref : $this->ref  (in SQL query < ou >).  Beside in the method (html.form.class)showrefnav, there is a parameters $fieldref that is use nowhere after. What do you think about send this parameters to  (CommonObject)load_previous_next_ref to avoid to declare ref property on object that don't have one ?
for exemple by change in  (CommonObject)load_previous_next_ref
       $sql.= " WHERE te.".$fieldid." < '".$this->db->escape($this->ref)."'";
by
       $sql.= " WHERE te.".$fieldid." < '".$this->db->escape($this->$fieldref)."'";


Kind regards

--
Florian HENRY
address@hidden
+33 6 03 76 48 07
http://www.open-concept.pro


_______________________________________________
Dolibarr-dev mailing list
address@hidden
https://lists.nongnu.org/mailman/listinfo/dolibarr-dev