Remember sort order of the file table in git diff view

[Saljack]

I want to always sort files by path in Git Diff view. This stores the sort configuration of the table in the Git Diff view. It is inspired by the same implementation in the subversion module.

Closes #7742

---

^Add meaningful description above

Click to collapse/expand PR instructions

By opening a pull request you confirm that, unless explicitly stated otherwise, the changes -

• are all your own work, and you have the right to contribute them.
• are contributed solely under the terms and conditions of the Apache License 2.0 (see section 5 of the license for more information).

Please make sure (eg. git log) that all commits have a valid name and email address for you in the Author field.

If you're a first time contributor, see the Contributing guidelines for more information.

If you're a committer, please label the PR before pressing "Create pull request" so that the right test jobs can run.

PR approval and merge checklist:

1. Was this PR correctly labeled, did the right tests run? When did they run?
2. Is this PR squashed?
3. Are author name / email address correct? Are co-authors correctly listed? Do the commit messages need updates?
4. Does the PR title and description still fit after the Nth iteration? Is the description sufficient to appear in the release notes?

If this PR targets the delivery branch: don't merge. (full wiki article)

[matthiasblaesing]

I had a quick look and initially though that it did not work. I opened a testrun with this, changed sorting, closed IDE and reopened. The state was not persistet. Looking at the implementation i then saw that sorting is stored when the diff is closed. This seems counter intuitive. Consider this event order:

1. Open two diffs
2. Change the sort order in the first view
3. Change the sort order in the second view
4. Close the second view
5. Close the first view

I would expect that the last setting survives, but in this case it is the first one.

It is inspired by the same implementation in the subversion module.

Can you point where that can be found? I was not able to find it.

[Saljack]

@matthiasblaesing Thanks for your review. I did not realize that this table is used for general git diffs. I wanted mainly sorted files in Git Commit Diff view. Where you can have only one instance of the table.
image

Let me check it and fix it.

Can you point where that can be found? I was not able to find it.

https://github.com/apache/netbeans/blob/master/ide/subversion/src/org/netbeans/modules/subversion/ui/commit/CommitTable.java#L195

[Saljack]

@matthiasblaesing Updated. Now it stores on click on the table header, and there is a change in sort order. Storing on a close diff is removed.