Icone Corbeille masque mais ne supprime pas l'historique

cddu33 · Juin 5, 2023, 7:29

BugReport
La corbeille ne supprime pas l’historique

Page concernée : Historique
Version Core : 4.4 beta 05 06 2023

Etape pour reproduire le Bug :
Activer une commande dans l’historique, cliquer sur la corbeille pour le supprimer: il se masque mais n’est pas supprimé

Loic · Juin 5, 2023, 8:32

Bonjour,
Effectivement je reproduis, je me note de corriger, merci pour le retour.

Loic · Juin 6, 2023, 9:17

Bonjour,
C’est bon j’ai trouvé ca semble mieux maintenant, peux tu retester ?

cddu33 · Juin 6, 2023, 10:02

en beta mis à jours à 12h, pas d’amélioration

Loic · Juin 6, 2023, 10:18

C’est en alpha la correction pour le moment.

Phpvarious · Juin 8, 2023, 9:50

Bonsoir,

Toujours aucun effet, en faite lorsqu’on ne choisi aucune date (Laissez vide pour tout supprimer) dans le prompt, le callback se comporte comme si on cliquer sur Annuler, et dans les 2 cas result === null, donc jeeP.emptyHistory n’est pas exécuté.

github.com

jeedom/core/blob/alpha/desktop/js/history.js#L615

      
        
            }
            
            
if (_target = event.target.closest('.li_history .remove')) {
              jeedomUtils.hideAlert()
              jeeDialog.prompt({
                inputType: 'date',
                pattern: '[0-9]{4}-[0-9]{2}-[0-9]{2} [0-9]{2}:[0-9]{2}:[0-9]{2}',
                placeholder: 'yyyy-mm-dd hh:mm:ss',
                message: '{{Veuillez indiquer la date (Y-m-d H:m:s) avant laquelle il faut supprimer l\'historique de}} <span style="font-weight: bold ;"> ' + _target.closest('.li_history').querySelector('.history').textContent + ' ?</span><br>({{Laissez vide pour tout supprimer}})'
                }, function(result) {
                if (result !== null) {
                  jeeP.emptyHistory(_target.closest('.li_history').attr('data-cmd_id'), result)
                }
              })
              return
            }
            
            
if (_target = event.target.closest('.li_history .export')) {
              window.open('core/php/export.php?type=cmdHistory&id=' + _target.closest('.li_history').getAttribute('data-cmd_id'), "_blank", null)
              return
            }

Loic · Juin 9, 2023, 9:11

Ok j’ai repris le truc dans l’autre sens, maintenant pour tout supprimer faut taper -1 (a voir si je pré remplis le champs avec, c’est pas plus mal de le taper volontairement).

cddu33 · Juillet 6, 2023, 9:30

Bonsoir, apaprement -1 n’est pas autorisé

Mips · Juillet 6, 2023, 9:58

essayes 3-4 ou 11-10-2

Loic · Juillet 7, 2023, 12:55

Bizarre que j’ai pas eu le soucis, bon j’ai supprimé le pattern tu peux retester ?

cddu33 · Juillet 8, 2023, 5:35

http://jeedom.home/core/php/getResource.php?file=desktop/js/history.js&md5=de132694e2ac391d6350bedfd00729e8&lang=fr_FR 616 Uncaught TypeError: _target.closest(...).attr is not a function

a chaque appuie sur le bouton ok

je suis en beta 4.4

Loic · Juillet 10, 2023, 7:40

Bonjour,
Il faut tester en alpha car c’est possible que j’ai deja corrigé.

Phpvarious · Juillet 10, 2023, 4:45

Bonjour,

Le dernier PR, résous bien l’erreur js en alpha

system · Juillet 13, 2023, 6:11

Ce sujet a été automatiquement fermé après 24 heures suivant le dernier commentaire. Aucune réponse n’est permise dorénavant.