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

Added: Optional Name Fields & New Library Folder Format #4

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

Sewer56
Copy link
Contributor

@Sewer56 Sewer56 commented Jul 19, 2021

This PR adds code related to format changes of the Steam files over time.

Legacy Packages

When using the library, I found some users reported that certain very old packages, such as Source 2007 Dedicated Server (ID: 310), might not have a name in their acf file.

I'm not sure when the change is made, but it seemed to happen with people who mostly installed Steam a long time ago. As such, I added a fail-safe to ensure no exception is thrown and the whole operation doesn't fail when a single name is missing.

New Library Folder Format

Recent Steam installations will use a new format for storing library folders, which holds more metadata about each individual library folder.

Old:

New:

As the current version looks for keys rather than nodes, I added extra code to also check the nodes for any library folders.

@magnusjjj
Copy link

Please merge this, fixes #5

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.

None yet

2 participants