search

Ultchad / snips-skill-spotifywebapi

SpotifyWebApi french skill for snips.ai
2 stars 4 forks source link

Bug pour monter ou descendre le volume #15

Closed bors-ltd closed 5 years ago

bors-ltd commented 5 years ago

Sur les intents de volume sans slot (juste « baisse la musique » par exemple), il y a une erreur parce que l'action cherche le slot de volume et sa valeur.

Ultchad commented 5 years ago

Je viens de le constaté ( merci ) sa viens du fait que j'ai oublié de vérifier si le slot n'est pas vide:

if intentMessage.slots:
    if intentMessage.slots.volume:

Faudrait faire une imbrication de sa mais la je ne voie pas comment le faire joliment

bors-ltd commented 5 years ago 
if intentMessage.slots and intentMessage.slots.volume:

mais c'est vrai que c'est un peu long.

slots = intentMessage.slots
if slots and slots.volume:

ou

try:
    volume = intentMessage.slots.volume.first().value
except AttributeError:
    volume = None

ou encore

volume = getattr(getattr(intentMessage, 'slots', None), 'volume', None)

mais c'est vraiment moche. :joy:

Ultchad commented 5 years ago

Ah j'etait persuader que 

if intentMessage.slots and intentMessage.slots.volume:

Allais retourner une erreur du genre "AttributeError: 'NoneType' object has no attribute 'volume'" Car il allais quand même essayer de checker intentMessage.slots.volume même si intentMessage.slots lui retournais None

On regardera ça une fois la migration python3 terminé Merci pour les conseil

Ultchad commented 5 years ago

J'ai fait une première correction en beta, de mon coté sa marche

Ultchad commented 5 years ago

Modification passer de beta vers master avec le reste #19

bors-ltd commented 5 years ago

On pourrait fermer ce ticket, non ?

Ultchad commented 5 years ago

Merci je l'avais oublié