Hello,
rien de très grave docteur, mais j’ai constaté lors d’un redémarrage qu’il manquait un ptit mot
en base je n’ai aucune ligne pour la clé product_name
pour autant côté page config ‹ Jeedom › ressort bien à certains endroit :
core version 4.4.6 & 4.4.7
nebz
Juin 12, 2024, 7:38
2
oui souvent le « nom de votre jeedom » est vide sur les systèmes que je visite… faudrait peut-etre indiquer « Jeedom » quand c’est vide…
d où ma 2nd remarque : sur la page de config c est egalement product_name qui est appelé, sans passer de valeur par defaut, et pourtant « Jeedom » s affiche bien (rouge) a ces différents endroits
Mais je n ai pas + creusé que ca, je dois passer a cote de qlq chose…
Edit : je t ai lu trop vite ce n est pas la saisie que l utilisateur fait qui est pris en compte/manquant dans ce que je remonte. C est vraiment en theorie le product_name qui pour nous tous « community » devrait etre « Jeedom » (versus la vente en marque blanche j imagine)
nebz
Juin 12, 2024, 7:52
4
c’est sans doute appelé avec une valeur par défaut genre config::byKey('product_name', 'core', 'Jeedom')
Mips
Juin 12, 2024, 8:20
6
le product_name est défini dans le core\config\default.config.ini, c’est le défaut si pas en db
il y a tout une série de config dans le même cas
yes j’avais vu le fichier, sans creuser de quand il était appelé
mais du coup ca change pas le point initial : on a bien ce default sur la page de config, mais pas lors du reboot
Hello,
Il y a bien un soucis, c’est le php dans le js qui fonctionne pas.
J’ai PR en utilisant une variable js :
jeedom:alpha ← Phpvarious:patch-26
opened 03:47PM - 12 Jun 24 UTC
https://community.jeedom.com/t/product-name-manquant/127053
## Description
D… elete php into js, and use a variable js
## Types of changes
- [X] Bug fix _(non-breaking change which fixes)_
- [ ] New feature _(non-breaking change which adds functionality)_
- [ ] Breaking change _(fix or feature that would cause existing functionality to change)_
- [ ] This change is only breaking for integrators, not for external standards or end-users.
- [ ] Documentation improvement
## PR checklist
- [ ] I have checked there is no other PR open for the same change.
- [ ] I have read the [[La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project](.github/CONTRIBUTING.md)).
- [ ] I grant the project the right to include and distribute the code under the GNU.
- [ ] I have added tests to cover my changes.
- [ ] I have verified that the code complies with the projects coding standards.
- [ ] [Required for new sniffs] I have added MD documentation for the sniff.
<!--
============================================================================================
Please make sure your pull request passes all continuous integration checks!
PRs which are failing their CI checks will likely be ignored by the maintainers.
PRs using atomic, descriptive commits are hugely appreciated as it will make
reviewing your changes easier for the maintainers.
============================================================================================
-->
6 « J'aime »
Tu vas finir par etre embauché toi !
Merci
6 « J'aime »
testé et approuvé
merci @Phpvarious
1 « J'aime »
system
A fermé ce sujet ()
Juin 14, 2024, 9:03
11
Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.