getConnectionsAny & getConnections

[armetiz]

Actuellement nous avons :

public function getConnectionsTo(NodeInterface $node, array $filters = array());
public function getConnectionsFrom(NodeInterface $node, array $filters = array());
public function getConnections(NodeInterface $node, array $filters = array());

Je propose de rajouter une méthode de récupération de connection entre deux NodeInterface et d'utiliser le mot clé "Any" pour spécifier la combination entre To & From.

public function getConnectionsTo(NodeInterface $node, array $filters = array());
public function getConnectionsFrom(NodeInterface $node, array $filters = array());
public function getConnectionsAny(NodeInterface $node, array $filters = array());
public function getConnections(NodeInterface $source, NodeInterface $destination, array $filters = array());

On améliore la cohésion avec les autres noms de fonction et on rajoute une fonction utile pour les internes du ConnectionManager et surement utile pour les utilisateurs.

[choomz]

👍 dans l'idée. J'aime pas Any par contre. Je dirais getConnectionsFor ou getAllConnectionsFor plutot ..

Par contre le getConnections() avec un sens, je trouve ça cool ! Ca fait un peu double emploi avec to & from mais c'est top pour l'aspect fonctionnel. Au moins avec ça on couvre tous les cas de requêtes.

[benjamindulau]

+1 pour le getConnectionsFor ;-)

J'ai même envie de proposer de remplacer getConnections par getConnectionsBetween :p

[choomz]

👍 pour getConnectionsBetween() c'est plus parlant.