Les logs du plugin Freebos_OS ne fonctionne plus depuis le changement du fonctionnement dans le core V4.5

Salon des Développeurs Core V4 (dev)
,
1

BugReport

Depuis le changement du fonctionnement des logs, je n’ai plus de log du tout avec le plugin-freebox_os.
je pense que c’est le fait que l’ID du plugin a un « _ » dans celui-ci.

j’ai mis le plugin en début et je n’ai vraiment aucun log qui apparait

voici un exemple de code pour les log que j’ai

 log::add('Freebox_OS', 'debug', '───▶︎ La version API est nulle mise en place version provisoire : ' . $API_version);

Page concernée : log
Version Core : V 4.5 update de ce jour

Que dois-je faire pour pouvoir faire fonctionner ceux-ci ?

merci par avance

2

Bonjour,
Pas cool un plugin id avec un _… Je vais voir si je peux le prendre en compte mais ca risque de poser d’autre soucis c’est vraiment un cas pas prévu et la j’en ai pour des mois a le prendre en compte.

3

Oui je sais mais il est très vieux ce plugin
Au lieu de faire une modification dans le code du core
Le core ne pourrait pas lors d’une mise à jour changer l’ID du plugin ? Et je fais le nécessaire dans le code du plugin
Ou alors on fait une version V2 du plugin sans _

Je crois que c’est le seul plugin malheureusement avec ce problème

4

Moi aussi mon Fully KiosK fraîchement passé en stable et avec un _ .
Id fully_kiosk.:sweat_smile:

5

Donc tu n as plus de log en v4.5 ?

6

J’ai pas tester je regarde😂

7

J’ai fait une correction pour les logs.

Et non un changement d’id est presque impossible ca a jamais été prévu un id ne doit jamais changer (c’est le but d’ailleurs).

8

Bon moi ça fonctionne en version debian 11 jeedom 4.4.5.


0059|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create setOverlayMessage(setOverlayMessage) message
0060|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create playSound(Jouer musique/son) message
0061|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create stopSound(Arrêter musique) subtypedefault
0062|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create textToSpeech(Envoyer TTS) message
0063|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create textToSpeech2(Envoyer TTS +) message
0064|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create stopTextToSpeech(Arrêter TTS) subtypedefault
0065|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create TTS_javascript(TTS javascript) message
0066|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create setAudioVolume(Changer volume tablette) message
0067|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create setBooleanSetting(Paramètre true/false) message
0068|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create setStringSetting(Paramètre valeur) message
0069|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create Refresh(Refresh) subtypedefault
0070|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create shutdownDevice(Arrêter équipement (root)) subtypedefault
0071|[2024-09-12 11:24:59] DEBUG  fully_kiosk::postSave 1414 cmdAction create rebootDevice(Redémarrer équipement (root)) subtypedefault
0072|[2024-09-12 11:26:02] DEBUG  fully_kiosk::getInformations 1252 requesting http://192.168.60.11:2323/?type=json&cmd=deviceInfo&password=
0073|[2024-09-12 11:26:02] DEBUG  fully_kiosk::getInformations 1264 $jsondata
0074|[2024-09-12 11:26:02] INFO  Connexion KO for Ddddddd (192.168.60.11)
0075|[2024-09-12 11:27:02] DEBUG  fully_kiosk::getInformations 1252 requesting http://192.168.60.9:2323/?type=json&cmd=deviceInfo&password=
0076|[2024-09-12 11:27:02] DEBUG  fully_kiosk::getInformations 1264 $jsondata {"deviceName":"HUAWEI MediaPad T5","packageName":"de.ozerov.fully","batteryLevel":82,"isPlugged":true,"SSID":"\"Wifi-Object\"","BSSID":"d2:21:f9:b0:a0:b5","Mac":"34:46:EC:B2:ED:68","ip4":"192.168.60.9","ip6":"FE80::3646:ECFF:FEB2:ED68,FE80::3446:ECFF:FEB2:ED68","hostname4":"HUAWEI_MediaPad_T5-44f488","hostname6":"fe80::3646:ecff:feb2:ed68%wlan0","wifiSignalLevel":9,"isMobileDataEnabled":true,"isHotspotEnabled":false,"screenOrientation":90,"screenBrightness":185,"screenLocked":false,"screenOn":true,"displayState":2,"batteryTemperature":24,"plugged":true,"keyguardLocked":false,"locale":"fr_FR","serial":"U8BBB20727101631","build":"AGS2-W09 8.0.0.317(OCEC431)","androidVersion":"8.0.0","webviewUA":"Mozilla/5.0 (Linux; Android 8.0.0; AGS2-W09 Build/HUAWEIAGS2-W09; wv) AppleWebKit/537.36 (KHTML, like Gecko) Version/4.0 Chrome/128.0.6613.127 Safari/537.36","isDeviceAdmin":true,"isDeviceOwner":false,"internalStorageFreeSpace":18003550208,"internalStorageTotalSpace":25205653504,"ramUsedMemory":1454891008,"ramFreeMemory":1550282752,"ramTotalMemory":3005173760,"appUsedMemory":16693984,"appFreeMemory":385959200,"appTotalMemory":402653184,"audioVolumes":[{"4":0},{"8":0},{"5":0},{"3":0},{"1":0},{"0":26},{"2":0}],"installerSource":"com.android.vending","timestamp":1726133220916,"scopedStorage":false,"displayHeightPixels":1200,"displayWidthPixels":1920,"isMenuOpen":false,"topFragmentTag":"","systemSigned":false,"isInDaydream":false,"isRooted":false,"isLicensed":true,"isInScreensaver":false,"kioskLocked":false,"isInForcedSleep":false,"maintenanceMode":false,"kioskMode":false,"startUrl":"http://192.168.70.4/index.php?v=d&p=plan&plan_id=4&fullscreen=1","currentTabIndex":0,"motionDetectorStarted":false,"deviceID":"5a667050-fd1ea121","appVersionCode":1302,"appVersionName":"1.56.1-play","androidSdk":26,"deviceModel":"AGS2-W09","deviceManufacturer":"HUAWEI","foregroundApp":"de.ozerov.fully","currentPage":"http://192.168.70.4/index.php?v=d&p=plan&plan_id=4&fullscreen=1","lastAppStart":"2024-09-07 20:35:17+0200"}
0077|[2024-09-12 11:27:03] DEBUG  isScreenOn is an array isScreenOn,screenOn
9

Pour moi cela fonctionne

cela ne vaudrait-il pas le coup pour tous nouveaux plugins interdire « _ » dans l’ID ?

merci pour la correction

10

Si il faudrait mais j’ai pas le temps pour mettre ca en place malheureusement pour le moment (mais ya bien une ligne sur ma todo depuis plusieurs qui est de gerer les caracteres interdit sur les id de plugin)

1 « J'aime »
11

Par contre Loic,
j’ai bien les logs mais si maintenant je veux faire « un sous log » cela ne fonctionne pas

si je met

log::add('Freebox_OS_TV', 'debug', '───▶︎ La version API est nulle mise en place version provisoire : ' . $API_version);

cela se met automatiquement dans le log du plugin, c’est à dire Freebox_OS

comment peut-on faire alors des sous logs si cela est faisable
merci

12

A oui ca c’est pas possible désolé, la seule solution ca serait que je récupère a chaque demain de log la liste des plugins donc une requête db a chaque fois ca serait terriblement lent

1 « J'aime »
13

Et une méthode static dans la class des plugins qui renvoi un array des libellés de logs ? Sinon default

14

C’est pareil ca m’oblige a charger tous les plugins (une requête par plugin) pour trouver a qui appartient le log puis a charger la classe a valider que ca appartient bien au plugin puis a faire une requête ensuite pour connaitre le niveau de log voulu.

Clairement la on est en dehors des limites (id plugin avec un _ et multiple log sur les plugins, ce qui je le rappels n’est pas prévu) et j’ai pas trop envie de tordre le core dans tous les sens de le ralentir pour ca.

15

J’en ai pas mais première fois que j’entends ça… s’il y a d’autres règles non-dites, ça m’intéresse de les connaître, ne fussent que pour mettre à jour la doc…

16

C’est pas des règles non dite c’est juste des cas que je n’avais jamais imaginé donc le code n’est pas prévu et tout refaire est vraiment très lourd.

Le cas d’un _ dans les id de plugin clairement j’ai jamais pensé que quelqu’un ferait ça un jour. Tout comme avoir plusieurs logs pour un plugin.

Après c’est entièrement ma faute je le reconnais c’est des cas auquel j’ai pas pensé et donc c’est pas dit que faut pas le faire. Malheureusement je suis un être humain j’ai donc mes limites et je peux pas penser à tout.

De plus dans le cas ici je n’ai pas de solution pour le prendre en compte sans que l’impact sur les jeedom ma soit en énorme. Je m’excuse de mon incompétence et manque d’intelligence mais là je vois pas comment faire.

17

Pfff ça commence à me saouler ces réponses…

image
image1200×1200 137 KB

Déso mais à un moment faut arrêter de s’apitoyer et trouver juste une solution ou trancher.

1 « J'aime »
18

Quand je tranche je m’en prend plein la tête.

Tu veux que je tranche : ben le cas dans ce sujet n’est pas possible il n’y aura donc aucune évolution la dessus c’est à vous à de vous adapter.

19

Ok mais que faire de l’existant depuis des années ?

20

Comme dit à vous de vous adapter. J’ai tranché comme tu as voulus tu es dans un cas qui n’est pas prévu donc faut vous débrouiller