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.
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
Multiword Potions pushed to Database as One Word #1227
Comments
Okay, I lied -__- guess I have only identified the offended code piece, not sure what it should be though. Its confusing because the drop system references items.eggs in the items.coffee file, which actually doesn't exist (in that file). And when the items.hatchingPotions is referenced, it uses a different syntax. Egg Drop [lines 116-120 of node_modules/habit-shared/script/algos.coffee]:
Hatching Potion Drop [lines 150-155 of node_modules/habit-shared/script/algos.coffee]:
Line 153 uses drop.name, which in items.coffee is the single word variation of the potion. I tried replacing with just drop (like used with eggs) and breaks all together. Tried with drops.text, and works with everything but Cotton Candies (which makes sense, but took me forever to find out... So. Much. Clicking. to finally land a cotton candy on test system >_<). To be honest, I'm barely able to comprehend how this all fits together, so I figured I'd throw this out there to the developers of the group. |
Hi! The site has been totally rewritten from the ground up, I'm closing the issue since it's probably fixed. Check if it's still happening and in case reopen the bug, thanks! |
Since Cotton Candy potions are the only ones (at this moment) that are multiword, they are the only ones affected. This could be a problem later on, with more users generating more potions, and any new potions added.
Isolated the code in habitshared/algos.coffee in the Drops System. Pushes drop.name when it should be drop.text (the human readable version). Will try to submit a pull request for this soon.
The text was updated successfully, but these errors were encountered: