3811: deleting a depends on request raise an issue because changedesc still reference it

nan****@gmai***** (Google Code) (Is this you? Claim this profile.)
What version are you running?
server 2.0.13

What's the URL of the page containing the problem?
any "view review"

What steps will reproduce the problem?
1. create a main review request X and publish
2. create a review request Y (test with draft only)
3. assign Y as a dependency of X and publish change
4. delete Y from admin database dashboard
5. display X (review) raise an internal error 500 (it looks displaying X review gets all changedescription including the one adding request Y as a dependency and since it does not exist anymore => not found)
Displaying diff does not raise the issue because changedesc are not looked within.

What is the expected output? What do you see instead?
at step 4, deleting Y should remove any reference to Y in X changedesc?

What operating system are you using? What browser?
firefox linux

Please provide any additional information below.
workaround is to remove changedesc containing this Y "depends on" change from review request linked object list
david
#1 david

Can you post the error traceback here (it should be found in the logs)?

We really don't intend anything to be deleted, which is why it's limited to admins only.

  • -New
    +NeedInfo
david
#2 david
  • -NeedInfo
    +Incomplete