Skip to content
This repository has been archived by the owner on Nov 6, 2021. It is now read-only.

Artifact Knowledge is wrongly applied to snapshotted items - see CF#4 #10

Open
rdw-software opened this issue Aug 19, 2017 · 0 comments

Comments

@rdw-software
Copy link
Collaborator

rdw-software commented Aug 19, 2017

Artifact Research notes being used while still snapshotted AP items are in inventory will mess up calculations.

Possible "solution" (which doesn't solve the problem): Show warning popup, which is reset when all items have been used. Actually solving this problem is tricky due to spellids being used and the API only returning the current values (with the latest Artifact Knowledge levels).

Better, but also tricky solution: Scan tooltips (different locales), using the Scanner algorithm, possibly both methods to find mismatches and use the scanned value when it's lower to detect snapshotted items?

Actual solution: Could cache all items as they are obtained and assign the current AK level to them. This will break if the user disables the addon (or it throws an error and is disabled as a result). It's also more difficult to implement and probably too elaborate for this rather obscure problem?

See CF#4 for more details.

@rdw-software rdw-software changed the title Artifact Knowledge is wrongly applied to snapshotted items (see CF#4) Artifact Knowledge is wrongly applied to snapshotted items - see CF#4 Aug 19, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant