Some Eclipse Foundation services are deprecated, or will be soon. Please ensure you've read this important communication.

Bug 360073

Summary: History view must not filter changed files of a commit
Product: [Technology] EGit Reporter: Dani Megert <daniel_megert>
Component: UIAssignee: Kevin Sawicki <kevin>
Status: VERIFIED FIXED QA Contact:
Severity: major    
Priority: P3 CC: christian.halstrick, markus.kell.r, robin
Version: 1.2   
Target Milestone: ---   
Hardware: All   
OS: All   
Whiteboard:
Attachments:
Description Flags
Picture showing bug none

Description Dani Megert CLA 2011-10-06 04:40:09 EDT
Created attachment 204655 [details]
Picture showing bug

1.2.0.201110021920

The History view also applies the filters which filter commits to the changed files section of an individual commit (see attached picture).

This is very bad! I cherry-picked such a commit which resulted in tons of other changes that I didn't want.
Comment 1 Markus Keller CLA 2011-10-13 10:56:39 EDT
Wow, this is really unexpected!

At least, I can copy the commit id and use Open Git Commit... to see all files, but this makes the file list in the history view useless when the view is in file mode.

If you want to keep the filter, then please just render the hidden files a bit differently and maybe put them to the end of the list, but don't hide them completely.
Comment 2 Remy Suen CLA 2011-10-21 19:17:20 EDT
*** Bug 345437 has been marked as a duplicate of this bug. ***
Comment 3 Kevin Sawicki CLA 2011-11-07 19:34:27 EST
Proposed fix pushed to: http://egit.eclipse.org/r/4565
Comment 4 Stefan Lay CLA 2011-11-09 05:29:32 EST
Which diff would you expect for the files in the list if you compare it with the ancestor? 

- The diff compared to the real parent of the commit
- The diff to the parent commit in the filtered tree

For the file(s) used for filtering the diffs are the same, but not for the others. I think this was the reason that the filtering was also used for the file list.
Comment 5 Markus Keller CLA 2011-11-09 06:44:16 EST
I would always expect the diff to the real parent of the current commit (same as if the History view shows the whole repository).

Multi-selection and "Compare with ..." can be used to compare two commits that are farther apart.
Comment 6 Stefan Lay CLA 2011-11-11 03:05:06 EST
Fixed with 82939b12c643826bae97bfc60d65ba7147cc6f4f
Comment 7 Dani Megert CLA 2011-11-15 10:27:32 EST
Verified in 1.2.0.201111150513.
Comment 8 Robin Stocker CLA 2012-11-05 16:30:45 EST
(In reply to comment #1)
> If you want to keep the filter, then please just render the hidden files a
> bit differently and maybe put them to the end of the list, but don't hide
> them completely.

See bug 393610 for this.