WIP Itemvalue misc fixes

[nicopop]

includes multiple small fixes to ItemValue functionality:
2586167 Fixed datavalidation to only check for requirements of used regions
63605b3 Made it so for sure that value name get lowercased (check items.py)
2b97f9d Renamed world.item_values_cache to world.itemvalue_rule_cache to stop potential confusion with world.item_values
5e64593 Stopped PreFill Datavalidation of ItemValue from reseting the world.item_values cache when running get_items_with_value so hook user can run it first then edit it to modify an item value for dev reasons

for some reason the diff of commit 63605b3 failed the only line changed in datavalidation is 249 -> 278

[nicopop]

since the first commit diff failed I might try to cherry pick the other commit in a new branch and then make a new pr idk

[FuzzyGamesOn]

Would love to get the item value lower fixes (63605b3) by themselves in a PR. That one's easy to digest and get through, and would fix a common issue people have with ItemValue in general.

I have to ask, though... why do they have to be lowercase?

[nicopop]

Would love to get the item value lower fixes (63605b3) by themselves in a PR. That one's easy to digest and get through, and would fix a common issue people have with ItemValue in general.

I have to ask, though... why do they have to be lowercase?

@FuzzyGamesOn
Since ItemValue has been a thing Id alway .lower() the value names in some situation (like the item groups) but I forgot to do it in some places and that lead to some issues (like data validation)
all this pr above does is to make it so the actually value get lower even if the dev write Star instead of star
I could go and remove that if you prefer
ill probably redo this pr anyway for the weird diff issue

[nicopop]

Redid this pr without the change that bugged the diff see #104