WinUI targets require location permissions set in manifest when asking for location

[dotMorten]

Description

When requesting location access on windows, the application incorrectly validates that you have locations enabled in the application manifest. WinUI 3 apps do not require this to be defined to get a location. In fact the option isn't even in the manifest designer like it was with UWP apps.
The bug seems to be here, and caused by a raw import of the older UWP code:

maui/src/Essentials/src/Permissions/Permissions.uwp.cs

Lines 132 to 144 in 117436b

	internal static async Task<PermissionStatus> RequestLocationPermissionAsync()
	{
	if (!MainThread.IsMainThread)
	throw new PermissionException("Permission request must be invoked on main thread.");
	var accessStatus = await Geolocator.RequestAccessAsync();
	return accessStatus switch
	{
	GeolocationAccessStatus.Allowed => PermissionStatus.Granted,
	GeolocationAccessStatus.Unspecified => PermissionStatus.Unknown,
	_ => PermissionStatus.Denied,
	};
	}

Could likely just be changed to:
internal static Task<PermissionStatus> RequestLocationPermissionAsync() => Task.FromResult(PermissionStatus.Granted)

(looking over this entire file, it might be affecting most of those permissions)

Steps to Reproduce

1. Create a new maui app
2. Add this code: await Microsoft.Maui.ApplicationModel.Permissions.RequestAsync<Microsoft.Maui.ApplicationModel.Permissions.LocationWhenInUse>();
3. Run on windows.
   Observe the following exception thrown:

Version with bug

6.0.408 (current)

Last version that worked well

Unknown/Other

Affected platforms

Windows

Affected platform versions

Windows 10 18362+

Did you find any workaround?

Don't call this on Windows, or create custom permission:

private class LocationWhenInUse: Microsoft.Maui.ApplicationModel.Permissions.BasePlatformPermission
{
   internal static Task<PermissionStatus> RequestLocationPermissionAsync() => Task.FromResult(PermissionStatus.Granted);
}

Relevant log output

No response

[knocte]

@mattleibow what's the first non-preview MAUI version that will have this fix? 7.0?

[mattleibow]

I think this made it back to rc2. I am afk now but will check and make sure.

[knocte]

But I guess rc is preview, I asked non-preview :)

[mattleibow]

Ah, in that case yes. 7.0.

But the workaround will be to just define the permissions in the manifest?