[Bug]: wrong scope detected for wix installer

[a-mnich]

Is there an existing issue for this?

• I have searched the existing issues

What happened?

When creating an update for Pandoc the scopes aren't correct

komac update JohnMacFarlane.Pandoc --version 3.6 --urls https://github.com/jgm/pandoc/releases/download/3.6/pandoc-3.6-windows-x86_64.msi

Komac correctly detects the ALLUSERS installer switch but sets the manifest wide scope to user even though ALLUSERS=1 forces a per-machine install (AllUSERS Property)
I've only seen this behavior for Pandoc, therefore I'm not sure whether it's a specific problem with its installer. (microsoft/winget-pkgs#197015, microsoft/winget-pkgs#165049)
Maybe a detection of ALLUSERS=1 could be used as fallback to set the installer scope to machine even when komac would usually detect it otherwise?

[russellbanks]

Thanks for reporting! This is because the ALLUSERS value is missing in the MSI property table which usually indicates a per-user install scope. However, Pandoc lets you modify this in the installer:

Therefore, we can check the MSI control table for anything that controls ALLUSERS:

As this is present in Pandoc, we can consider the scope to be agnostic.