You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cayenne.apache.org by "Nikita Timofeev (JIRA)" <ji...@apache.org> on 2019/03/21 12:15:01 UTC

[jira] [Closed] (CAY-2555) Use explicit ArcId in GraphChangeHandler methods

     [ https://issues.apache.org/jira/browse/CAY-2555?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Nikita Timofeev closed CAY-2555.
--------------------------------
    Resolution: Fixed

> Use explicit ArcId in GraphChangeHandler methods
> ------------------------------------------------
>
>                 Key: CAY-2555
>                 URL: https://issues.apache.org/jira/browse/CAY-2555
>             Project: Cayenne
>          Issue Type: Improvement
>          Components: Core Library
>            Reporter: Nikita Timofeev
>            Assignee: Nikita Timofeev
>            Priority: Major
>             Fix For: 4.2.M1
>
>
> {{GraphChangeHandler}} uses plain {{Object}} as arc identifier:
> {code:java}
> ..
> void arcCreated(Object nodeId, Object targetNodeId, Object arcId);
> void arcDeleted(Object nodeId, Object targetNodeId, Object arcId);
> ..
> {code}
> We can introduce {{ArcId}} object instead that can hold forward and reverse arc names:
> {code:java}
> public class ArcId {
>     private final String forwardArc;
>     private final String reverseArc;
>     // ...
> }
> {code}
> This change should simplify code maintenance and refactoring.



--
This message was sent by Atlassian JIRA
(v7.6.3#76005)