search

tewtal / alttp_sm_combo_randomizer

ALttP VT Randomizer and API - Super Metroid & ALTTP Combo Version (V10)
MIT License
43 stars 12 forks source link

Standardize internal Beam names, minor maridia logic fix (PlasmaBeam -> Plasma) #57

Closed MP2E closed 6 years ago

MP2E commented 6 years ago

pointed out by HalfARebel#8048 on Discord, PlasmaBeam does not exist in Item.php, but Plasma does

MP2E commented 6 years ago

as the discussion has continued on discord, it might be better instead to standardize the Beams such that they either all have the Beam suffix, or remove it from all of them, just to avoid further confusion. I'm impartial! Just wanted to make sure this doesn't slip through the cracks

MP2E commented 6 years ago

I have added a second commit that removes the Beam suffix from all of the beam items in Super Metroid for consistency, as it seemed like that was the better option from the discussion I saw about it