Show file when staged to add but deleted in working tree

[westbury]

This is the fix for eclipse-theia/theia#7789 .

If a file is created, then staged, then deleted, it is still staged for add (being in the index) and will be included in the commit. Changes that have been made to the working tree but not staged will not be included in the commit, this includes the deletion of the file.

In this screenshot Untitled2.txt is now shown added in the staged area and deleted in the unstaged area. Before it was not shown in either area, even though the add was being included in the commit.

[westbury]

I've pushed a second commit to also fix the problem where a file is deleted, staged, then re-created in the working tree. It now shows as D in the staged area and U in the unstaged area. See CONTRIBUTING.md

[kittaakos]

This is the fix for eclipse-theia/theia#7789 .

Can you add a test for it?

Also, aligning the implementation with GitHub Desktop would make sense: https://github.com/desktop/desktop/blob/3551d3742753b85084a165c46ac582a4be5fd090/app/src/lib/git/status.ts#L255-L297

[westbury]

I was wondering where the offending code came from. It looked like it was fixing someone's bug but it certainly was not fixing any bug in Theia.

aligning the implementation with GitHub Desktop would make sense

Not really. GitHub Desktop presents a simpler interface to the user. It does not show staged changes and unstaged changes. It just presents all the changes in a single list, each file listed once only, and has check-boxes to select which files will be committed. My 14 year old son says he won't use GitHub Desktop because it is too simple for his needs. Theia, however, is designed to show staged and unstaged changes correctly, like vs-code, git-gui and git-kraken.

Definitely best to return the status information as git does. Consumers can do this funky merging of staged and unstaged states if they want, but dugite-extra should not do this because then the information is lost and consumers can't present staged and unstaged states correctly

I'll get the test done tomorrow.

[westbury]

I said I would get the tests done last week but the tests have shown up more work. I'll get back to this as time permits, hopefully this week.

[kittaakos]

Excellent, please name me here when it's ready for the review, and we roll out a new release. Thank you!

[RDIL]

Hey @westbury any updates on this? :)

[westbury]

Hey @westbury any updates on this? :)

@RDIL sorry, it has been a while. I'll put something in on Wednesday, in a couple of days.

[westbury]

See eclipse-theia/theia#8221 . That PR may obsolete this one.