Fixed Cache-WindowsPlayer path retrieval from VRChat config.json

[Luois45]

Fixes the path retrieval of the Cache-WindowsPlayer folder based on the VRChat config.json file.

Tested on Windows with all possible config cases
Couldn't test on Linux yet but should work.

[ShayBox]

I don't believe the code needed to be fixed, it should work as tested on Windows and Linux since none of the code has changed since I tested it, but if it's not working for you, based on your code, I'm taking a guess that your config is malformed (possibly an extra/missing comma/quote) because your code defaults on malformed config instead of showing the error.

Without the deserialize method, if the path does get parsed from the config instead of defaulted from the lazy_static, and contains environment variables or relative paths, they won't get parsed and the PathBuf will contain an invalid path.

Could you share your config.json so I can test input with it?

EDIT: the load method also shouldn't contain the .join("Cache-WindowsPlayer") part because the path should already be the full parsed path to the config directory prior to attempting to load the path.

[Luois45]

It didn't work before. It works fine in VRChat. Thats what my config looked like.

{
	"cache_directory": "E:\\Games\\VRChat",
	"cache_size": 100
}

[ShayBox]

I found that it was defaulting to the original path within the deserialize method because Deserialize::deserialize(deserializer) was trying to deserialize into a borrowed string, but because the path contains escaped backslashes it was failing, switching it to an explicit String and then borrowing that fixes the problem.

serde-rs/serde#1746

I must have either tested with forward slashes or had only copied my cache to another drive when testing, which means the default path still had files and appeared to work.

[ShayBox]

Try this release and see if it works https://github.com/ShayBox/VRC-LOG/releases/tag/0.5.8