Skip to content
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

Crypt items release and boss drop changes #224

Closed
wants to merge 5 commits into from

Conversation

Cassy1
Copy link
Contributor

@Cassy1 Cassy1 commented Nov 26, 2014

Goes with themanaworld/tmwa-client-data#202 .

@wushin some explanations:
I made all three items be slightly less good than the usual head armor for every class (Warlord/Crusade Helmet, Cat Ears and Wizard Hat), but when they are equipped while being in the Crypt or Graveyard they are better than those. We can't make them better in general but making them better in the Crypt, the area designed for high level players, will make people be interested in them. However, as discussed on IRC the cryptitem.txt file isn't working so I need your help there.

I made the three Generals drop them at a rate of 0.25% and adjusted nearly all other equipment drops. E.g. 0.4% means that even if that boss is killed 100 times you don't even have a 50% chance to get that crypt item. 0.25% means about 400 kills. Both are very low drop rates for bosses, but that makes those items even more interesting without being ridiculously hard to get (like other equipment drops before :D ).

// Script to boost Dark Helm, Underworld Mask and Phylactery when those are equipped while being in Graveyard or Crypt.
function|script|CryptItem
{
if (isin("027-1.gat", 0, 0, 0, 0)) // Graveyard
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

instead of using lots of isin() I would put all the maps in an array and then check with getmap()

isin() is to detect if the player is in a square area within a map but in your case you want to know the map

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants