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.
Issue 1672 in reviewboard: ReviewBoard hide existing revisions
Status: New Owner: Labels: Type-Defect Priority-Medium New issue 1672 by Jan.Koprowski: ReviewBoard hide existing revisions http://code.google.com/p/reviewboard/issues/detail?id=1672 *NOTE: Do not post confidential information in this bug report.* What version are you running? 1.5 beta 3 What's the URL of the page containing the problem? http://SERVER/r/NUMBER/diff/#index_header What steps will reproduce the problem? 1. Turn on view diff 2. 3. What is the expected output? What do you see instead? Diffviewer should show all files. I see only last one. Strange because in HTML source of file I see all files and downloading raw diff show all three files. Also turning off JavaScript work. But when JS is turn on only last file is show. What operating system are you using? What browser? Linux Please provide any additional information below. This doesn't work on 1.5 beta 2 also. -- 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.