#1647 [Control] add: labels for sheet and other object types #1809
Conversation
class/sheet.class.php
Outdated
There was a problem hiding this comment.
Pourquoi ce fix est dans cette PR ca n'a rien à faire la
class/actions_digiquali.class.php
Outdated
There was a problem hiding this comment.
Le but est de rajouter le label à la fin de getNomUrl, pour quelle raison tu modifie complètement la fonction ?
|
|
||
| unset($object->fields['fk_sheet']); | ||
|
|
There was a problem hiding this comment.
Si tu unset le field, on peut plus le manager par la suite
There was a problem hiding this comment.
fk_sheet est utilisé dans commonfields_view.tpl.php, ce qui crée un double du champ Modèle. Je ne vois pas comment empêche le doublon sans unset.
view/control/control_card.php
Outdated
| print '<tr class="field_fk_sheet"><td class="titlefield fieldname_fk_sheet">'; | ||
| print $langs->trans('Sheet'); | ||
| print '</td><td class="valuefield fieldname_fk_sheet">'; | ||
| print $object->showOutputField($object->fields['fk_sheet'], 'fk_sheet', $object->fk_sheet, '', '', '', 0) . '<span class="opacitymedium">' . ' - ' . dol_trunc($sheet->label, 0) . '</span>'; |
There was a problem hiding this comment.
Dol_trunc a 0 ne sert a rien autant laisser le paramètre par défaut
view/control/control_card.php
Outdated
| $linkedObject->add_label = true; | ||
|
|
There was a problem hiding this comment.
Faire ça sous-entend que 100 % des objets défini possède cette attribut définie et exploitable, c'est rarement une bonne façon de faire de la programmation orienté objet que de rajouter une propriété dynamique
evarisk-francois
force-pushed
the
add_labels_sheet_others
branch
from
73bf22b
to
71adb99
Compare
No description provided.