Only use the commit in share link when upstream (microsoft#156658)

alexr00 · web-flow · commit 12b08be500f8 · 2022-07-29T12:30:05.000+02:00
Fixes microsoft#156627
diff --git a/extensions/github/src/links.ts b/extensions/github/src/links.ts
@@ -125,11 +125,11 @@ export function getPermalink(gitAPI: GitAPI, useSelection: boolean, hostPrefix?:
 		return;
 	}
 
-	const commitHash = gitRepo.state.HEAD?.commit;
+	const commitHash = (gitRepo.state.HEAD?.ahead === 0) ? `/blob/${gitRepo.state.HEAD?.commit}` : '';
 	const fileSegments = fileAndPosition.type === LinkType.File
 		? (useSelection ? `${uri.path.substring(gitRepo.rootUri.path.length)}${rangeString(fileAndPosition.range)}` : '')
 		: (useSelection ? `${uri.path.substring(gitRepo.rootUri.path.length)}${notebookCellRangeString(fileAndPosition.cellIndex, fileAndPosition.range)}` : '');
 
-	return `${hostPrefix}/${repo.owner}/${repo.repo}/blob/${commitHash
+	return `${hostPrefix}/${repo.owner}/${repo.repo}${commitHash
 		}${fileSegments}`;
 }

Original file line number	Diff line number	Diff line change
`@@ -125,11 +125,11 @@ export function getPermalink(gitAPI: GitAPI, useSelection: boolean, hostPrefix?:`
`125`	`125`	`return;`
`126`	`126`	`}`
`127`	`127`
`128`		`- const commitHash = gitRepo.state.HEAD?.commit;`
	`128`	+ const commitHash = (gitRepo.state.HEAD?.ahead === 0) ? `/blob/${gitRepo.state.HEAD?.commit}` : '';
`129`	`129`	`const fileSegments = fileAndPosition.type === LinkType.File`
`130`	`130`	? (useSelection ? `${uri.path.substring(gitRepo.rootUri.path.length)}${rangeString(fileAndPosition.range)}` : '')
`131`	`131`	: (useSelection ? `${uri.path.substring(gitRepo.rootUri.path.length)}${notebookCellRangeString(fileAndPosition.cellIndex, fileAndPosition.range)}` : '');
`132`	`132`
`133`		- return `${hostPrefix}/${repo.owner}/${repo.repo}/blob/${commitHash
	`133`	+ return `${hostPrefix}/${repo.owner}/${repo.repo}${commitHash
`134`	`134`	}${fileSegments}`;
`135`	`135`	`}`