REMOVEDEMOS removes FreeOnDemand packages which are not demos

[woctezuma]

It seems the !REMOVEDEMOS command removes FreeOnDemand packages which do not belong to the Demo category.

I believe the filter must be based on the package name, but I am not sure as I have not dived into the code.

See https://store.steampowered.com/app/1729830/Tactical_Monsters__Strategy_Edition/

Indeed, these two packages are actually both named Tactical Monsters - Strategy Edition Demo:

• sub/623611 Tactical Monsters - Strategy Edition Demo ➡️ app/1739920 App Type: Demo
• sub/619516 Tactical Monsters - Strategy Edition Demo ➡️ app/1729830 App Type: Game

And it seems that the !REMOVEDEMOS command has removed both packages.

If this is the case, it may be useful to:

• check that the App Type is Demo before removing the package,
• otherwise, mention in the README that the filter is based on package names, and thus imperfect.

---

Edit: Confirmed. There is a check for x.Name.EndsWith("Demo") in:

x.Type == LicenseType.Complimentary && x.Name.EndsWith("Demo")

Reference:

ASFEnhance/ASFEnhance/Account/Command.cs

Line 279 in 0876ba2

var oldSubs = licensesOld.Where(x => x.PackageId > 0 && x.Type == LicenseType.Complimentary && x.Name.EndsWith("Demo")).Select(x => x.PackageId).ToHashSet();

So the command catches FreeOnDemand packages:

• whose name ends with the word Demo,
• yet which do not necessarily belong to the Demo category ("App Type").

[chr233]

Got it

[LORDIF56]

update pls to NET 8.0 global version ASF now use 8.0 and creat conflict if we start ASF and if we have ASFEnchance

[woctezuma]

update pls to NET 8.0 global version ASF now use 8.0 and creat conflict if we start ASF and if we have ASFEnchance

You have posted about an issue which was fixed 3 weeks ago, and in the wrong thread on top of that:

• https://github.com/chr233/ASFEnhance/releases/tag/2.0.3.0