Project

General

Profile

Actions

Defect #8628

closed

"Related to" reference may yield circular dependency error message

Added by Christian Zagrodnick over 13 years ago. Updated over 8 years ago.

Status:
Closed
Priority:
Normal
Category:
Issues
Target version:
Start date:
2011-06-16
Due date:
% Done:

0%

Estimated time:
Resolution:
Fixed
Affected version:

Description

The "related to" reference is neither directed nor declaring a dependency. Thus the error "This relation would create a circular dependency" seems wrong. "Related to" references should be possible in any case, should they not?


Files

fix_issue_relations.diff (853 Bytes) fix_issue_relations.diff Egbert van der Wal, 2014-05-12 18:10
fix_issue_relations.diff (655 Bytes) fix_issue_relations.diff Egbert van der Wal, 2014-05-12 18:13
issue_relation_test.patch (1.82 KB) issue_relation_test.patch Uwe Koloska, 2014-09-05 16:42

Related issues

Related to Redmine - Feature #16818: Allow multiple relationships between two issuesNew

Actions
Related to Redmine - Defect #13654: Can't set parent issue when issue relations among child issues are presentClosedJean-Philippe Lang

Actions
Has duplicate Redmine - Feature #14660: Excluding "related to" relation from circular dependency checkClosed

Actions
Has duplicate Redmine - Defect #13914: Circular dependency issue when issue dependency is setClosed

Actions
Actions #1

Updated by Jean-Baptiste Barth over 13 years ago

  • Category set to Issues
  • Status changed from New to 7
  • Assignee set to Jean-Baptiste Barth
  • Target version set to Candidate for next major release

Agreed, definitely. I'll have a look at it since I already worked on issue relations today.

Actions #2

Updated by Victor Engmark almost 13 years ago

"Related to" is reciprocal to itself. In other words, when such a relation is created, there should be a link from both issues to the other one of the relation.

Actions #3

Updated by Christian Zagrodnick almost 13 years ago

Victor Engmark wrote:

"Related to" is reciprocal to itself. In other words, when such a relation is created, there should be a link from both issues to the other one of the relation.

That is the case but you could still want to build larger circles A -> B -> C -> A.

Actions #4

Updated by Fares Abdullah almost 13 years ago

I had the same problem -

In my case I had an issue, A. I created issue B and set it as related to A. Then I found that B was a duplicate of an existing issue, C, that someone else had created.

I marked B as a duplicate of C, and wanted to relate A to C, and got the "circular dependency" error. I had to manually go and delete the relation between A and B to be able to relate A to C, even though it should not be necessary.

"related to" should allow many different issues related together, as it is not a "dependency".

Cheers and thanks for the great application!

Actions #5

Updated by Jean-Baptiste Barth over 12 years ago

  • Assignee deleted (Jean-Baptiste Barth)
Actions #6

Updated by Jean-Philippe Lang almost 12 years ago

  • Status changed from 7 to New

Assigned issue with no assignee back to New status.

Actions #7

Updated by Leandro Guida over 11 years ago

Here we are having the same problem.

I have an issue A, which is "father" of the issues B and C. When I try to set that C follows B, it gives me the "circular dependency" message.

Actions #8

Updated by Robin McKenzie over 11 years ago

+1 "related to" isn't a dependency.

Having the same problem, since we upgraded to the latest version of Redmine.

I've had to advise all our devs to stop using the Related Issues functionality, as this is preventing us setting up tickets with parent/child relationships.

e.g. Create a parent ticket, and two child tickets of this. Attempt to relate the two child tickets - "This relation would create a circular dependency".

Alternatively, create two tickets and relate them to each other. Create another ticket, and then set child A to have this as its parent. Attempt to set child B to have the same parent - "Parent task is invalid".

Would it be possible to get a fix for this prioritised?

Actions #9

Updated by Ralf S. over 11 years ago

+1 for this ticket from our side to (using V2.3.1)

It would be very nice, if this fix would get high priority for the next release.

Actions #10

Updated by Benjamin Roberts over 11 years ago

+1 for this issue

Environment:
Redmine version 2.3.0.stable
Ruby version 1.9.3 (i386-mingw32)
Rails version 3.2.13
Environment production
Database adapter Mysql2
Redmine plugins:
no plugin installed

Actions #11

Updated by Luke Purdy over 11 years ago

+1 for this update.

At the moment I can't create any parent "summary" tasks, for any children who have following dependencies between them (as 4).

It would be helpful if this fix is included as a high priority within the next release.

Actions #12

Updated by Robin McKenzie about 11 years ago

Would it be possible to get some feedback as to whether this is being considered for the next release? It was originally set to "Candidate for next major release" over two years ago...

Actions #13

Updated by Akiko Takano over 10 years ago

+1 for this update.

Actions #14

Updated by Egbert van der Wal over 10 years ago

How can this still be open when it's such a simple fix?

I applied the attached patch to my Redmine SVN checkout to fix this. Feel free to merge it in. It basically takes the type of relations into account when traversing them and ignores 'copied_to', 'duplicates' and 'relates' relationships as these do not imply any dependency.

Actions #15

Updated by Egbert van der Wal over 10 years ago

Sorry, this was the incorrect patch. Attached is the correct version. Please remove the previous patch.

Actions #16

Updated by Egbert van der Wal over 10 years ago

Just a tiny update after reading the Patch page.

The patch I submitted is against revision 13127 of the stable-2.5 branch. I just tested it against trunk, revision 13138 and it also applies cleanly.

Actions #17

Updated by Toshi MARUYAMA over 10 years ago

  • Has duplicate Feature #14660: Excluding "related to" relation from circular dependency check added
Actions #18

Updated by Toshi MARUYAMA over 10 years ago

  • Related to deleted (Defect #13914: Circular dependency issue when issue dependency is set)
Actions #19

Updated by Toshi MARUYAMA over 10 years ago

  • Has duplicate Defect #13914: Circular dependency issue when issue dependency is set added
Actions #20

Updated by James H over 10 years ago

+1

Actions #21

Updated by Uwe Koloska about 10 years ago

What about the fix for #7320? Shouldn't the tests for r4723 and r5104 detect the problem described here?

Actions #22

Updated by Uwe Koloska about 10 years ago

It looks like related to has a direction!

  • create three tickets 1, 2, 3
  • create relation 1 -> 2
  • create relation 2 -> 3
  • create relation 3 -> 1 gives the circular dependency error
  • create relation 1 -> 3 succeeds
instead of related to you can also use in any combination:
  • duplicates (no error with duplicated by)
  • copied to (no error with copied from)

Since both are really meant to have a direction their's nothing wrong if there is a circle of all duplicates or copied to, but it should definitely not treat all relations the same.

Actions #23

Updated by Uwe Koloska about 10 years ago

attached a test for the current 2.5 branch (but has not changed in trunk) that tests for the following dependencies that should both not be circular:
  • 1->2, 2->3, 3->1 (error)
  • 1->2, 2->3, 1->3 (no error)
Actions #24

Updated by Etienne Rossignon about 10 years ago

+1 to get a fix for this one

Actions #25

Updated by Mischa The Evil almost 10 years ago

  • Related to Feature #16818: Allow multiple relationships between two issues added
Actions #26

Updated by Marco Shima almost 10 years ago

+1 to finally get this fixed. Redmine is only partially useful as long as this major bug exists.

Actions #27

Updated by Anton Statutov over 9 years ago

+1. This definitely needs to be fixed.

Actions #28

Updated by Nayuta Taga about 9 years ago

+1

Actions #29

Updated by Jānis Elmeris about 9 years ago

+1

Actions #30

Updated by Go MAEDA almost 9 years ago

r15056 also seems to have fixed this issue.
Now I can make A->B, B->C, C->A relations on Redmine 3.2.0.devel.15056.

Actions #31

Updated by Go MAEDA almost 9 years ago

  • Status changed from New to Resolved
  • Target version changed from Candidate for next major release to 3.3.0

Fixed by #13654.
I am setting target version to 3.3.0 because I think it would be better to list this important issue in Changelog.

Actions #32

Updated by Go MAEDA almost 9 years ago

  • Related to Defect #13654: Can't set parent issue when issue relations among child issues are present added
Actions #33

Updated by Jean-Philippe Lang almost 9 years ago

  • Status changed from Resolved to Closed
  • Assignee set to Jean-Philippe Lang
  • Resolution set to Fixed

Thanks.

Actions #34

Updated by Robin McKenzie over 8 years ago

Hi, any idea when 3.3.0 will be released?

Actions

Also available in: Atom PDF