[jira] [Commented] (SANDBOX-336) [Graph] Duplicate Edge

["Simone Tripodi (JIRA)" <ji...@apache.org>]

    [ https://issues.apache.org/jira/browse/SANDBOX-336?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13058819#comment-13058819 ] 

Simone Tripodi commented on SANDBOX-336:
----------------------------------------

Hi Marco,
according to the Vertices checks the method already does, if an Edge between head/tail already exists, it should throw an exception, I mean:

{code}
if ( getEdge( head, tail ) != null ) 
{
    throw new GraphException( "Vertices '%s' and '%s' already present in the Graph", head, tail );
}
getAllEdges().add( e );
{code}

or not?

> [Graph] Duplicate Edge  
> ------------------------
>
>                 Key: SANDBOX-336
>                 URL: https://issues.apache.org/jira/browse/SANDBOX-336
>             Project: Commons Sandbox
>          Issue Type: Bug
>          Components: Graph
>            Reporter: Marco Speranza
>         Attachments: missingControllAlreadyExistEdge.patch
>
>
> The method  BaseMutableGraph#addEdge does not check if the edge already exists.

--
This message is automatically generated by JIRA.
For more information on JIRA, see: http://www.atlassian.com/software/jira