Filtration icon indicating copy to clipboard operation
Filtration copied to clipboard

Upgrading to a release when default POE user data directory does not exist results in prompt

Open ben-wallis opened this issue 5 years ago • 2 comments

When upgrading Filtration to a new version on a system that does not have the default POE user data directory present at C:\Users\<username>\Documents\My Games\Path of Exile, the user is incorrectly prompted to select their POE data directory.

This occurs because ItemFilterPersistenceService has the following code in its constructor:

ItemFilterScriptDirectory = Settings.Default.DefaultFilterDirectory;

As this is in the constructor, it runs during IoC container dependency resolution, which occurs before the settings migration for a new version has run. Therefore, the settings file it reads has the default settings instead of the migrated settings.

The fix for this is to make ItemFilterPersistenceService.ItemFilterScriptDirectory use a Lazy<string> backing field so that it is evaluated the first time it's called rather than during construction.

ben-wallis avatar Sep 11 '19 12:09 ben-wallis

Getting this as well on a fresh install. I'm guessing its the same issue.

Some info on this PC's state.. Had Path Of Exile installed Removed Path of Exile.
Noticed the Path of Exile folder in My Games.
Deleted the folder.

Filtration Gives Various errors trying to create a new filter or open an existing one...

Same Path Error... C:\Users<username>\Documents\My Games\Path of Exile

synicasm avatar Oct 02 '19 15:10 synicasm

Anyone else running into this issue.

Simply adding a folder for Path of Exile seems to satisfy it for now. C:\Users\Documents\My Games\Path of Exile

synicasm avatar Oct 02 '19 17:10 synicasm