Re microsoft#209158. Output replacement might use legacy output id. (microsoft#218237)

rebornix · web-flow · commit e53968c415d2 · 2024-06-25T16:41:10.000-07:00
diff --git a/src/vs/workbench/contrib/notebook/browser/controller/cellOutputActions.ts b/src/vs/workbench/contrib/notebook/browser/controller/cellOutputActions.ts
@@ -96,7 +96,7 @@ function getOutputViewModelFromId(outputId: string, notebookEditor: INotebookEdi
 	if (notebookViewModel) {
 		const codeCells = notebookViewModel.viewCells.filter(cell => cell.cellKind === CellKind.Code) as CodeCellViewModel[];
 		for (const cell of codeCells) {
-			const output = cell.outputsViewModels.find(output => output.model.outputId === outputId);
+			const output = cell.outputsViewModels.find(output => output.model.outputId === outputId || output.model.alternativeOutputId === outputId);
 			if (output) {
 				return output;
 			}

Original file line number	Diff line number	Diff line change
`@@ -96,7 +96,7 @@ function getOutputViewModelFromId(outputId: string, notebookEditor: INotebookEdi`
`96`	`96`	`if (notebookViewModel) {`
`97`	`97`	`const codeCells = notebookViewModel.viewCells.filter(cell => cell.cellKind === CellKind.Code) as CodeCellViewModel[];`
`98`	`98`	`for (const cell of codeCells) {`
`99`		`- const output = cell.outputsViewModels.find(output => output.model.outputId === outputId);`
	`99`	`+ const output = cell.outputsViewModels.find(output => output.model.outputId === outputId \|\| output.model.alternativeOutputId === outputId);`
`100`	`100`	`if (output) {`
`101`	`101`	`return output;`
`102`	`102`	`}`