4482: 500 error when clicking 'Publish Changes' (or 'Discard Draft') button

mgarsvinskas 
What version are you running?


2.0.18


What's the URL of the page containing the problem?


http://reviewboard.exigengroup.com/r/32451/


What steps will reproduce the problem?


    

  1. Request was successfully posted to Reviewboard (hg postreview) (Request Number: 32451)
    2. 

  2. After code review. Executed hg rollback;
    3. 

  3. Made some code changes using Intelij (During that, also edited bugs created in Review Request - Added comments, selected Action - Fixed.);
    4. 

  4. Commited new changes with same commit message using Tortoise HG;
    5. 

  5. Executed command: hg postreview -e 32451 -> Changes can be seen in Review Board Request
    6. 

  6. Try to Publish Changes at the top of the page:
    7. 





What is the expected output? What do you see instead?


New Changes are published.


What operating system are you using? What browser?


Windows 10. Chrome (tried with firefox 45 as well)


Please provide any additional information below.


When trying to Discard Changes (500 Error popup appears)

When trying to Publish Changes - Nothing Happens;

When trying to Publish Changes with added comment: Loading loading loading...Nothing happens

Error provided in dev tools (F12).
#1 mgarsvinskas 
Attaching console.
brennie
#2 brennie 
Hi there,


We'll need more information to debug this. Can you post the relevant part from the the reviewboard logs?
  • -New
    +NeedInfo
#3 mgarsvinskas 
Hello. I've managed to solve my problem (hope so). The problem was: 

1st Request is created - published;

2nd Request is created - published;

Add Dependencies:

1st Request DEPENDS ON = 2nd request number;

2nd Request DEPENDS ON = 1st request number;


And then if user makes some changes in any of requests - Changes cannot be published, discarded and so on. 500 error is thrown in browser console or popup.


Issue can be closed (if needed, maybe other with dependencies have to be opened)


Have a nice day.
david
#4 david 
Thanks for the update. The issue with circular dependencies has already been fixed in newer versions.
  • -NeedInfo
    +Fixed