8000 Be consistent about how we use relative paths and Git paths by jcansdale · Pull Request #2386 · github/VisualStudio · GitHub
[go: up one dir, main page]
Skip to content
This repository was archived by the owner on Jun 21, 2023. It is now read-only.

Be consistent about how we use relative paths and Git paths #2386

Merged
merged 11 commits into from
Aug 9, 2019
Merged

Be consistent about how we use relative paths and Git paths #2386

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
4f6b57e
Pass Git path to PullRequestTextBufferInfo
jcansdale Jun 10, 2019
8ae29c8
FindGitRelativePath should always return a path or error
StanleyGoldman Jun 12, 2019
5fe36d6
Use relativePath and gitPath in a consistent way
jcansdale Jun 13, 2019
f8c52cd
Factor out ToGitPath and ToRelativePath
jcansdale Jun 13, 2019
eec41c3
Fix GetOldFileName
jcansdale Jun 13, 2019
ff60575
Use relativeOrGitPath when it doesn't matter
jcansdale Jun 13, 2019
e46f894
Add xmldoc comments to Paths
jcansdale Jun 13, 2019
4727302
AddBufferTag can accept relativePath
jcansdale Jun 13, 2019
cd67be2
Remove unnecessary GetAbsolutePath method
jcansdale Jun 13, 2019
286d287
Add Path_Can_Use_Windows_Directory_Separator test
jcansdale Jun 13, 2019
52f2bf0
Rename to Paths.ToWindowsPath from ToRelativePath
jcansdale Aug 9, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix GetOldFileName
  • Loading branch information
@jcansdale
jcansdale committed Jun 13, 2019
commit eec41c3a3a7df47a96af8f0f2f9cc236cb48c466
5 changes: 3 additions & 2 deletions src/GitHub.App/ViewModels/GitHubPane/PullRequestFilesViewModel.cs
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,7 @@
using System.Windows.Input;
using GitHub.Extensions;
using GitHub.Models;
using GitHub.Primitives;
using GitHub.Services;
using LibGit2Sharp;
using ReactiveUI;
Expand Down Expand Up @@ -224,8 +225,8 @@ static string GetOldFileName(PullRequestFileModel file, TreeChanges changes)
{
if (file.Status == PullRequestFileStatus.Renamed)
{
var fileName = file.FileName.Replace("/", "\\");
return changes?.Renamed.FirstOrDefault(x => x.Path == fileName)?.OldPath;
var gitPath = Paths.ToGitPath(file.FileName);
return changes?.Renamed.FirstOrDefault(x => x.Path == gitPath)?.OldPath;
}

return null;
Expand Down
0