What version are you running? chipx86 says 1.0RC3. What's the URL of the page containing the problem? What steps will reproduce the problem? 1. Create a review that includes adds or edits to binary files. 2. View the diff. What is the expected output? What do you see instead? The binary files are listed among the "Files changed" momentarily with throbber icons, but after they're fetched, they're removed. Although we can't show diffs of binary files, listing them (and showing whether they're added, deleted, or edited) is important. (How else can you know if someone forgot to include a required binary file in their change?) What operating system are you using? What browser? Windows XP x64 SP2, Firefox 3.0.11 Please provide any additional information below.
Unicode encoded files don't diff and are treated as binary files. Not seeing them in the diff can be dangerous. It's good to know what's touched and what's not.
I looked at this briefly. The diff fragment URL is telling it to remove the entry.
From a URL like this: /r/10594/diff/1/fragment/113257/?index=1&1254766381
I got the following HTML code:
</table>
<script type="text/javascript">
$(document).ready(function() {
$("li.change_file_1").remove();
});
</script>
from the bottom of diff_file_fragment.html. Also, as a side issue, the </table>
probably shouldn't be there. It looks like it's trying to explicitly remove changes
that don't have change_chunks, presumably for the interfilediff case where you don't
want to show files that have no interdiff changes. If that is correct, maybe the
check for "if file.changed_chunks" should also be checking "if not file.interfilediff"?
That seems to be the problem. We should get this in for 1.0.5.
-
+ Confirmed -
+ Milestone-Release1.0.x + Component-DiffViewer
-
- Priority-Medium + Priority-High