Re: Issue 1672 in reviewboard: ReviewBoard hide existing revisions
Updates: Status: UserError Comment #5 on issue 1672 by trowb...@gmail.com: ReviewBoard hide existing revisions http://code.google.com/p/reviewboard/issues/detail?id=1672 After trying to figure this out for quite some time, the only thing I can think of is if parts of the diff are empty (for example, in an interdiff when some files don't change, or when posting things like perforce changesets that include non-modified files). If you can figure out better what's going on, we can reopen. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-issues@googlegroups.com. To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
Re: Issue 1672 in reviewboard: ReviewBoard hide existing revisions
Updates: Status: New Comment #4 on issue 1672 by trowbrds: ReviewBoard hide existing revisions http://code.google.com/p/reviewboard/issues/detail?id=1672 (No comment was entered for this change.) -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-iss...@googlegroups.com. To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
Re: Issue 1672 in reviewboard: ReviewBoard hide existing revisions
Comment #3 on issue 1672 by Jan.Koprowski: ReviewBoard hide existing revisions http://code.google.com/p/reviewboard/issues/detail?id=1672 I still don't know what is happen but I make a little investigation. There is reviewboard/diffviewer/views.py:view_diff_fragment():line 218 'standalone': chunkindex is not None and for some reason for this two files chunkindex is None. But I don't know why. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-iss...@googlegroups.com. To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
Re: Issue 1672 in reviewboard: ReviewBoard hide existing revisions
Comment #2 on issue 1672 by Jan.Koprowski: ReviewBoard hide existing revisions http://code.google.com/p/reviewboard/issues/detail?id=1672 I refresh page with firebug turned on. I get something like that (in attachment). Attachments: issue1672_firebug_output.gif 18.6 KB -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-iss...@googlegroups.com. To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.
Re: Issue 1672 in reviewboard: ReviewBoard hide existing revisions
Updates: Status: NeedInfo Labels: Component-DiffViewer Comment #1 on issue 1672 by chipx86: ReviewBoard hide existing revisions http://code.google.com/p/reviewboard/issues/detail?id=1672 Nothing has changed in the logic anywhere regarding the diff file display. It always shows the first file it will display (generated during the initial page render) and then asynchronously downloads the other files. What do you mean when you say turning off JavaScript works? You see all three files? Can you check your JavaScript debug output to see what errors are being displayed? It's most likely that something is erroring out and preventing further operations for the file display. -- You received this message because you are subscribed to the Google Groups reviewboard-issues group. To post to this group, send email to reviewboard-iss...@googlegroups.com. To unsubscribe from this group, send email to reviewboard-issues+unsubscr...@googlegroups.com. For more options, visit this group at http://groups.google.com/group/reviewboard-issues?hl=en.