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

Add check for empty default ammo type when spawning matching ammo for guns #7789

Merged
merged 1 commit into from May 31, 2014

Conversation

Projects
None yet
3 participants
@BevapDin
Copy link
Contributor

commented May 30, 2014

The ammo type "darts" does not come with a default ammo item type. Because the blowgun is in an item group ("homeguns") which should spawn the matching ammo for guns (guns_with_ammo:true), this would spawn an item with an empty string as ident ("darts" has this as default ammo), this shows up as "undefined-s".

This PR adds a check to prevent this behavior. One could additionally define a default ammo item type for darts, but this kind of error might come up for other guns, too.

@KA101 KA101 assigned KA101 and unassigned KA101 May 30, 2014

@kevingranade kevingranade merged commit 72196c8 into CleverRaven:master May 31, 2014

@BevapDin BevapDin deleted the BevapDin:fix-item-spanwammo branch May 31, 2014

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.