refactoring on scenarios list filter

[noodom]

Bonjour Loïc,

J'ai modifié la proposition en supprimant la fonction avec le code du listener directement dans la balise . J'ai aussi ajouté le filtre en balise HTML (A voir si la déclaration est à retoucher).

---

Add a scenarios list filter when adding a scenario in a scenario modification

Description

The PR adds a filter to the scenarios list when the action 'scenario' is added. Only matching scenarios to the filter (scenario name, scenario group or scenario object parent) are visible in the list.

Suggested changelog entry

add a scenarios list filter in a scenario

Types of changes

• [ ] Bug fix (non-breaking change which fixes)
• [x ] 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

• [x ] I have checked there is no other PR open for the same change.
• [ x] I have read the [La ligne directrice pour contribuer à ce projet / Contribution guidelines for this project).
• [x ] I grant the project the right to include and distribute the code under the GNU.
• [ ] I have added tests to cover my changes.
• [ x] I have verified that the code complies with the projects coding standards.
• [ ] [Required for new sniffs] I have added MD documentation for the sniff.

[zoic21]

Bonjour, Merci pour la modification par contre je pense il va y avoir un soucis avec le const filter qui sera different pour chaque bloc de selection de scénario mais avec une const ca va poser soucis peut etre plutot faire :

document.querySelector('.expressionAttr[data-uid="#uid#"][data-l1key="options"][data-l2key="filter"]').addEventListener('keyup', function(event) {

Pour les autres const dans la fonction je pense il faudrait plutot des let.

[noodom]

Pour moi, ça ne pose pas de problème. La portée de la déclaration du const est interne à la callback success.

J'avais d'ailleurs testé avec plusieurs actions 'scenario' et ça ne remonte aucun souci. Pour moi, c'est plus sûr (et sain) de laisser les const là où c'est suffisant (un let ne changera pas la portée et les variables déclarées n'ont pas besoin d'être modifiées).

[zoic21]

Effectivement j'ai rien dit const et let ont la meme portée. C'est donc tout bon pour moi