Entities relations with invalid associations shown

[paulo-graca]

Describe the bug
My model has two relation types, with same relation name, but involving different entities:

	<type>
		<leftType>Publication</leftType>
		<rightType>Person</rightType>
		<leftwardType>isAuthorOfPublication</leftwardType>
		<rightwardType>isPublicationOfAuthor</rightwardType>
		<leftCardinality>
			<min>0</min>
		</leftCardinality>
		<rightCardinality>
			<min>0</min>
		</rightCardinality>
	</type>
	<type>
		<leftType>Publication</leftType>
		<rightType>OrgUnit</rightType>
		<leftwardType>isAuthorOfPublication</leftwardType>
		<rightwardType>isPublicationOfAuthor</rightwardType>
		<leftCardinality>
			<min>0</min>
		</leftCardinality>
		<rightCardinality>
			<min>0</min>
		</rightCardinality>
	</type>

But when I try to associate a Person as an author, that Person, will be shown simultaneously as Authors (persons) relation and Authors (organizational units)

To Reproduce
Steps to reproduce the behavior:

1. You can use /dspace/config/entities/openaire4-relationships.xml as your model
2. And set dspace.​entity.​type = Publication to any existing Item
3. At edit item Relationships separator, use lookup to associate an author
4. After you added the relations to that entity, strange associations will be shown
   

And if you refresh the page, you will get 2 associations:

It seems an issue with Relationships separator, because item page is shown as expected.

Expected behavior
I was expecting to have only one association, or to be shown only one Authors (persons) relation.

[tdonohue]

@paulo-graca : I think the issue here is that we have two relationship types of the same name (i.e. two isAuthorOfPublication and two isPublicationOfAuthor). I don't believe that's currently supported in our Entities model. I think the relationship type name is expected to be unique (in the same way that Entity names are expected to be unique).

So, I think this would work correctly if the second relationship was renamed to be isOrgAuthorOfPublication like this...

        <type>
		<leftType>Publication</leftType>
		<rightType>Person</rightType>
		<leftwardType>isAuthorOfPublication</leftwardType>
		<rightwardType>isPublicationOfAuthor</rightwardType>
		<leftCardinality>
			<min>0</min>
		</leftCardinality>
		<rightCardinality>
			<min>0</min>
		</rightCardinality>
	</type>
	<type>
		<leftType>Publication</leftType>
		<rightType>OrgUnit</rightType>
		<leftwardType>isOrgAuthorOfPublication</leftwardType>
		<rightwardType>isPublicationOfOrgAuthor</rightwardType>
		<leftCardinality>
			<min>0</min>
		</leftCardinality>
		<rightCardinality>
			<min>0</min>
		</rightCardinality>
	</type>

See also our Configurable Entities docs about "Configuring the object model" which says "Determine a clear an unambiguous name for the relation in both directions" (I believe the "unambiguous" there means it also needs to be unique): https://wiki.lyrasis.org/display/DSDOC7x/Configurable+Entities#ConfigurableEntities-Configuringtheobjectmodel

So, to me, it looks like we may need to fix the openaire4-relationships.xml file to have unique names for relationship types.

[saschaszott]

relationship-types.xml contains two different relationship types with identical names (isAuthorOfPublication and isPublicationOfAuthor). The names of left and right relationship types between Publication and OrgUnit should be adapted (same as in openaire4-relationships.xml).