Skip to content

Update display name retrieval in index.php#3222

Closed
Caelion wants to merge 1 commit intojeedom:masterfrom
Caelion:patch-10
Closed

Update display name retrieval in index.php#3222
Caelion wants to merge 1 commit intojeedom:masterfrom
Caelion:patch-10

Conversation

@Caelion
Copy link
Contributor

@Caelion Caelion commented Mar 23, 2026

Complément en lien avec ce commit :
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
@Mips2648
Copy link
Collaborator

Bonjour,

  1. Il sera impossible de valider ce commit, les PR ne peuvent pas être fait sur master
    Pour l'instant il faudrait faire la PR sur la branche 4.5.3 mais cela va probablement être clarifié bientot (et donc cela va changer)

  2. je n'ai pas testé mais en lisant la modif je me demande pourquoi avoir utilisé $object_li->getName() comme valeur par défaut si display_name n'existe pas au lieu de $object_li->getHumanName(true) comme précédemment?

@Mips2648 Mips2648 self-requested a review March 24, 2026 08:56
@Mips2648 Mips2648 closed this Mar 24, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants