Fix: Fixed executable flag for shortcut items in drag operation

yaira2 · yaira2 · commit d0c4058d221b · 2025-09-07T11:30:56.000-04:00
Updates the drag-and-drop operation to correctly use the IsExecutable property from IShortcutItem when available, ensuring proper handling of shortcut items during file operations.
diff --git a/src/Files.App/Views/Layouts/BaseLayoutPage.cs b/src/Files.App/Views/Layouts/BaseLayoutPage.cs
@@ -1189,7 +1189,7 @@ protected virtual async void Item_Drop(object sender, DragEventArgs e)
 			dragOverItem = null;
 			var item = GetItemFromElement(sender);
 			if (item is not null)
-				await ParentShellPageInstance!.FilesystemHelpers.PerformOperationTypeAsync(e.AcceptedOperation, e.DataView, (item as IShortcutItem)?.TargetPath ?? item.ItemPath, false, true, item.IsExecutable, item.IsScriptFile);
+				await ParentShellPageInstance!.FilesystemHelpers.PerformOperationTypeAsync(e.AcceptedOperation, e.DataView, (item as IShortcutItem)?.TargetPath ?? item.ItemPath, false, true, (item as IShortcutItem)?.IsExecutable ?? item.IsExecutable, item.IsScriptFile);
 
 			deferral.Complete();
 		}

Original file line number	Diff line number	Diff line change
`@@ -1189,7 +1189,7 @@ protected virtual async void Item_Drop(object sender, DragEventArgs e)`
`1189`	`1189`	`dragOverItem = null;`
`1190`	`1190`	`var item = GetItemFromElement(sender);`
`1191`	`1191`	`if (item is not null)`
`1192`		`- await ParentShellPageInstance!.FilesystemHelpers.PerformOperationTypeAsync(e.AcceptedOperation, e.DataView, (item as IShortcutItem)?.TargetPath ?? item.ItemPath, false, true, item.IsExecutable, item.IsScriptFile);`
	`1192`	`+ await ParentShellPageInstance!.FilesystemHelpers.PerformOperationTypeAsync(e.AcceptedOperation, e.DataView, (item as IShortcutItem)?.TargetPath ?? item.ItemPath, false, true, (item as IShortcutItem)?.IsExecutable ?? item.IsExecutable, item.IsScriptFile);`
`1193`	`1193`
`1194`	`1194`	`deferral.Complete();`
`1195`	`1195`	`}`