...came up here: #2128257-1: Adding related issues is extremely painful. :( by Dave Reid:

Also, you can add an issue related to itself. We should probably have some validation around that.

Comments

eliza411’s picture

Issue tags: +Drupal.org 7.1
dww’s picture

Yeah, probably wise. ;) I just made exactly this error at #2097925-4: Rename "Edit" to "Update" for consistency with the "Update this issue" button

There's also talk of displaying parent/child relationships as a tree, which would require even more fancy validation (to ensure you don't get cycles and you've got an actual tree, not a directed graph). See #1975806: Provide a way to represent the relationships between issues in a tree list format. for more on that.

m1r1k’s picture

I'm gona add "Allow self reference" settings disabled by default for Entity reference module and it will fix this issue.

dww’s picture

Yes, hurray for fixing it "upstream" in EntityReference itself.

Thanks!
-Derek

klonos’s picture

This seems to be the best solution indeed. Thanx.

m1r1k’s picture

I've posted the patch :) https://drupal.org/node/1364802#comment-8276681
Can someone review it?

tvn’s picture

Status: Active » Postponed

Postponing until we have a review in #1364802: Prevent reference self

drumm’s picture

Issue tags: -Drupal.org 7.1

Untagging since this isn't really a regression. We didn't have issue relations in D6.

tvn’s picture

Project: [Archive] Drupal.org D7 upgrade QA » Drupal.org customizations
Version: » 7.x-3.x-dev

Cleaning up d7qa queue.

YesCT’s picture