Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Implement ISteamUGC019 #83

Merged
merged 2 commits into from
Nov 7, 2024
Merged

Conversation

universal963
Copy link
Contributor

This interface is not public, so header is based on reversing and maybe more checks are needed.

@otavepto
Copy link
Contributor

otavepto commented Nov 6, 2024

I'd be skeptical about GetQueryUGCResult() and RequestUGCDetails(), we don't know if in this v19 they started supporting the new mod details struct, or they were still using the older one.
A safer approach would be using the xxx_old() variant like the other older versions.

btw there's a new undocumented ISteamUserStats013 (you can hijack Steam_IsKnownInterface in assembly debugger to get the complete list of supported interfaces) where they simply removed the first function RequestCurrentStats() if you also want to add that 👍

@universal963
Copy link
Contributor Author

Good point, I've tested again and found out that, for interface ISteamUGC019, GetQueryUGCResult() is now using newer function, just the same as ISteamUGC020, while RequestUGCDetails() is still using older one, which btw is now officially bugged. (for those who care, m_bCachedData of callresult SteamUGCRequestUGCDetailsResult_t is filled with garbage if you are calling it with any interfaces older than ISteamUGC020, and new member m_details.m_ulTotalFilesSize is most probably filled with garbage when calling ISteamUGC020)
And for ISteamUserStats013, I'll wait for next public SDK now, it's a lot easier to check it then.

@Detanup01 Detanup01 merged commit aa728a3 into Detanup01:dev Nov 7, 2024
62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants