Repository navigation
Modification de disconnect() du Manager #21
Description
Actuellement on ne peut pas détruire une connection entre deux NodeInterface à l'aide de disconnect().
Il faut d'abord récupérer toutes les connections d'une node, puis iterer sur la collection pour voir si notre deuxieme node est présente, puis enfin détruire la connection.
Je pense qu'on pourrait conserver la notion de destruction d'une connection dans une méthode destroyConnection(ConnectionInterface $connection); et modifier disconnect() vers
public function disconnect(NodeInterface $souce, NodeInterface $destination);
Activity
armetiz commentedon Mar 22, 2013
Pas faux,
Et puis avoir le prototype suivant pour disconnect
C'est plus cohérent vis-à-vis de la méthode connect.
Du coups, je suis pour cette modification 👍
Au niveau de destroyConnection, pourquoi ne pas directement faire destroy() ?
Si on modifie disconnect, l'ensemble des méthodes du Manager s'appliquera avec des NodeInterface en entrée.. Du coups, cela peut justifier d'appeler destroyConnection afin de contextualisé l'utilisation de cette méthode..
Maintenant, au niveau perso, je prefere destroy.. 🌴
@benjamindulau ?
benjamindulau commentedon Mar 22, 2013
👍 pour l'idée générale
et 👍 pour le
destroytout simple ;-)On pourrait aussi ajouter une option dans la méthode
disconnectpourdestroyles connections automatiquement suite au disconnect.choomz commentedon Mar 22, 2013
Ok vendu, cool ! Par contre pour le $destroyConnection, je ne sais pas si c'est utile, car l'idée derriere c'est effectivement de détruire la connection, avec ou sans parametre :)
armetiz commentedon Mar 22, 2013
@benjamindulau on avait décidé de ne plus conserver l'état d'une connexion.
Quand on déconnecter deux objets, on détruit la connexion.
benjamindulau commentedon Mar 22, 2013
Ok ! :)