Cannot add new relationships between Entities about dspace-angular HOT 5 CLOSED

I retested this issue in the sandbox and found, that if you hit save on the itempage as well, it works for me. I did the following:

1. Edit a publication, going to the "Relationships" tab
2. Click "+Add" next to Authors and attempt to add a new Author
3. Select a local author who is already in the system.
4. Click "Save".
5. Now, the popup hangs for me as well
6. So, click elsewhere on the page (the popup will close)
7. Click save on the top of the relationship page
8. Reload the page -> The related entities are still there.

Still, I see the issue with the hanging popup window, which is not closing itself.

from dspace-angular.

Thanks @Leano1998 for the additional test! I think I realize where I went wrong. I forgot your step 7. I didn't go back and click "Save" at the top of the relationship page. I had assumed the "Save" button on the popup modal was the only button I had to click.

So, this is working. But the behavior of saving a new relationship is slightly confusing & the popup window never closes itself.

from dspace-angular.

I looked a bit into this and think it might be related to this older issue #1371
If I add the following line (which has been removed in this Commit), the window closes for me:

        setTimeout( () => {
          this.submit.emit();
          this.modalRef.close()
        },1000);

edit-relationship-list.component.ts (L352)

But I don't fully understand, why it has been removed.

from dspace-angular.

Thanks @Leano1998 for working to track that down. I don't know the answer to your question, but it seems like the code you referenced was removed in #1329 by @atarix83 . This seems to have been over 3 years ago now, so it may be difficult to track down why that change was made.

If you want to simply create a PR to add the fix that works for you, that may be the best approach. That way we can get some additional testers to ensure it also works for them.

from dspace-angular.

Okay, I can create a PR.

from dspace-angular.