962: Interdiffs are currently broken.

marc****@gmai***** (Google Code) (Is this you? Claim this profile.)
March 25, 2009
What's the URL of the page containing the problem?
http://demo.review-board.org/r/1555/diff/2-3/

What steps will reproduce the problem?
1. create a diff and submit it
2. update your diff by adding a lot of new files
3. submit it again
4. try to get the changes between the two diffs

What is the expected output? What do you see instead?
The excepted output should be the diff between the two diffs.
1st remark, when it works, it won't display the new files. For example,
http://demo.review-board.org/r/1555/diff/1-2/ should display the new file 
scmtools/core.py.
2dn remark, when you'll fix this issue by adding a test for instance in views.py:145, I think you will have another issue: the paginator will return 
4 pages, even if there is only less than 20 feils to be displayed. 

What operating system are you using? What browser?
Any OS / Any Browser

Please provide any additional information below.
david
#1 david
  • +Confirmed
  • +Interdiffs are currently broken.
chipx86
#2 chipx86
This just seems to be demo.review-board.org being outdated. My up-to-date tree with
the exact same diff works fine.
  • -Confirmed
    +Fixed
  • +Milestone-Release1.0
    +Component-DiffViewer
    +Interdiffs