Skip to content

Fix NameError with vanilla special deal prices #2534

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.

Sign up for GitHub

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

Merged
cjohnson57 merged 2 commits into from
Apr 4, 2026
Merged

Fix NameError with vanilla special deal prices #2534

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
5f1fd52
Fix NameError with vanilla special deal prices
fenhl Mar 11, 2026
e435c15
Fix vanilla price for business scrubs
fenhl Mar 11, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 7 additions & 4 deletions World.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -704,11 +704,14 @@ def random_shop_prices(self) -> None:
for location in region.locations:
if location.type == 'Shop':
if location.name[-1:] in shop_item_indexes[:shop_item_count]:
self.shop_prices[location.name] = self.new_shop_price()
self.shop_prices[location.name] = self.new_shop_price(location)

def new_shop_price(self) -> int:
def new_shop_price(self, location: Location) -> int:
if self.settings.special_deal_price_distribution == 'vanilla':
return ItemInfo.items[location.vanilla_item].price
price = location.price
if price is None:
price = ItemInfo.items[location.vanilla_item].price
Comment thread
cjohnson57 marked this conversation as resolved.
return price
elif self.settings.special_deal_price_max < self.settings.special_deal_price_min:
raise ValueError('Maximum special deal price is lower than minimum, perhaps you meant to swap them?')
elif self.settings.special_deal_price_max == self.settings.special_deal_price_min:
Expand Down Expand Up @@ -1226,7 +1229,7 @@ def push_item(self, location: str | Location, item: Item, manual: bool = False)
# Reduce the frequency of obvious scams by rerolling the price once if it's too high, and taking the lower value.
# This affects logic so it should only be applied to refills that are logically irrelevant.
# Otherwise there could be seeds with e.g. a wallet that's hinted as logically required for a purchase even though the price was rerolled to no longer require the wallet.
price = min(location.price, self.new_shop_price())
price = min(location.price, self.new_shop_price(location))
else:
price = item.price

Expand Down
Loading