[#CRUC-230] Crucible allows circular review linking and can't delete a review link

Crucible

Crucible allows circular review linking and can't delete a review link

Created: 24/Jan/08 06:30 PM Updated: 15/Apr/08 12:11 AM

Component/s:

User Interface

Affects Version/s:

None

Fix Version/s:

1.5

Time Tracking:

Not Specified

Participants:	Bryce Schober, Peter McNeil [Atlassian] and Tom Davies [Atlassian]
Since last comment:	26 weeks, 4 days ago
Resolution Date:	08/Apr/08 07:20 PM
Labels:

Description

« Hide

I've linked cr-10 to cr-9.
If I try to use cr-9's "link a review" input to link to cr-10, I get an error message "review cr-9 doesn't exist". I agree this is an error, but that's not the right text.

All

Comments

Work Log

Change History

Sort Order:

[ Permlink | « Hide ]

Peter McNeil [Atlassian] added a comment - 29/Jan/08 03:55 AM

Bryce the case of project keys maters when linking a review, can you please confirm that the "review cr-9 doesn't exist" isn't a problem to do with case sensitivity?

[ Permlink | « Hide ]

Bryce Schober added a comment - 29/Jan/08 10:14 AM

Good call. In fact, if I do use "CR-9", it does let me circularly link the review, so that I actually get 3 tabs. CR-10 was originally linked to CR-9. I then linked CR-9 back to CR-10, and now CR-9 shows three linked review tabs: CR-9, CR-10, and CR-9. Very strange. And using the "change parent review" input doesn't let me enter a blank, in order to remove the link. Is there some other way to remove an erroneous link?

[ Permlink | « Hide ]

Peter McNeil [Atlassian] added a comment - 01/Feb/08 11:09 PM

No there isn't a way to remove a link, that's a bug. We'll fix that with the circular review reference, problem.

[ Permlink | « Hide ]

Tom Davies [Atlassian] added a comment - 19/Mar/08 10:32 PM

Cycles fixed

[ Permlink | « Hide ]

Tom Davies [Atlassian] added a comment - 19/Mar/08 10:54 PM

Link removal fixed too.

[ Permlink | « Hide ]

Tom Davies [Atlassian] added a comment - 08/Apr/08 07:20 PM

Reviewed