Skip to content

Update for display_name#3225

Open
Caelion wants to merge 6 commits into
jeedom:developfrom
Caelion:4.5.3
Open

Update for display_name#3225
Caelion wants to merge 6 commits into
jeedom:developfrom
Caelion:4.5.3

Conversation

@Caelion
Copy link
Copy Markdown
Contributor

@Caelion Caelion commented Mar 28, 2026

Complément en lien avec ce commit :
efad4aa
Permet de prendre le nom à afficher dans le menu de jeedom

Concernant getHumaneName VS getName : voir ici : #3222 (comment)

Caelion added 2 commits March 28, 2026 01:49
Complément en lien avec ce commit :
jeedom@efad4aa
Permet de prendre le nom à afficher dans le menu de jeedom
Complément en lien avec ce commit :
jeedom@efad4aa
Permet de prendre le nom à afficher dans le menu de jeedom
Copy link
Copy Markdown
Collaborator

@Mips2648 Mips2648 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Il faut garder $object_li->getHumanName(true) dans desktop/php/index.php

edit: et en fait je ne sais pas si c'est une bonne idée d'utiliser display_name comme ca ici car du coup ca casse la couleur et l'icone qui ne seront plus affichées

Je me demande s'il ne faudrait pas adapter getHumanName() plutôt

Ca n'a pas été suffisamment testé.

Comment thread desktop/php/index.php Outdated
Caelion added 3 commits March 28, 2026 14:08
Annulation du changement pour displayname pour modification directement du getHumanName
Ajout d'une fonction getDisplayName et mise à jour de getHumanName (sauf dernière partie car probablement utilisé pour d'autres fonctionnalités par pas mal d'utilisateurs (ou alors il faut ajouter un troisième paramètre ?)
@Salvialf Salvialf deleted the branch jeedom:develop March 31, 2026 17:44
@Salvialf Salvialf closed this Mar 31, 2026
@Salvialf Salvialf reopened this Mar 31, 2026
@Caelion Caelion requested a review from Mips2648 March 31, 2026 18:07
Comment thread core/class/jeeObject.class.php Outdated
Comment thread desktop/php/index.php Outdated
@Mips2648
Copy link
Copy Markdown
Collaborator

Mips2648 commented Apr 2, 2026

ok pour moi, merci pour les modifs
plus qu'à avoir un retour des autres review

@Salvialf Salvialf requested a review from reineabs April 17, 2026 05:46
@Salvialf Salvialf requested a review from Mips2648 April 17, 2026 15:37
@Salvialf
Copy link
Copy Markdown
Contributor

Salvialf commented May 6, 2026

Bonjour,

Je n’ai pas toutes les informations relatives à la demande initiale à l’origine de ce champ « Nom d'objet à afficher », mais d’après ce que je comprends cela ne concerne que l’usage qui en est actuellement fait au niveau Dashboard / Synthèse.

Il ne semble pas destiné à être utilisé ailleurs dans Jeedom, et surtout pas à avoir d’impact sur le retour de getHumanName notamment. En effet, ce nom à afficher peut être identique pour plusieurs objets; l’utiliser dans d’autres contextes que ceux prévus initialement pourrait donc entraîner des incohérences.

@Salvialf Salvialf requested a review from Sekiro-kost May 6, 2026 11:28
@Caelion
Copy link
Copy Markdown
Contributor Author

Caelion commented May 6, 2026

Bonjour Salvialf
De mémoire, le champ n’est actuellement pas prit en compte dans les vues sans ces modifications

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants