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

Making Glock accept silencer and autolathe emaggable to build silencers #968

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

Limeliz
Copy link
Contributor

@Limeliz Limeliz commented Aug 21, 2014

Made the silencer fit on the Glock (Glock has new typepath).
Made the Autolathe emaggable.
Emagged Autolathe can now build silencers (costs 200k metal, which means the Autolathe needs to be upgraded some).
New sprites for Glock so it can fit the silencer:
New Glock sprites

Fixes #855

Made autolathe emaggable.
Added silencer to autolathe (available for 200k metal when emagged).
Silencer now fits on the Glock.
Silencer costs 200k metal to produce from an emagged Autolathe.
Forgot to remove some commented out code that was never needed.
New sprites for Glock, including two with silencer attached.
var/datum/effect/effect/system/spark_spread/s = new /datum/effect/effect/system/spark_spread
s.set_up(5, 1, src)
s.start()
user << "You short out the [src]'s circuit board, enabling debug mode."
Copy link
Contributor

Choose a reason for hiding this comment

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

Add a span class please. Preferably 'warning' but 'notice' works.

@ike709
Copy link
Contributor

ike709 commented Aug 21, 2014

Also, you have an error. You change the type path for the glock but didn't change it every time it is used in the code. In dream maker use Ctrl + F to search for the old type path(s) and change them to the new type path. You could use Find and Replace but there shouldn't be more than one or two type paths to correct. Otherwise this PR is great.

Added silencer to be summoned together with the Glock during Summon Weapons.
Updated Glock typepath.
@Limeliz
Copy link
Contributor Author

Limeliz commented Aug 21, 2014

@ike709 Thanks, forgot about span classes (fixed by 29e3c0c). Also forgot to upload summon_weapons.dm to git (fixed by 26dcb62).

@Limeliz
Copy link
Contributor Author

Limeliz commented Aug 31, 2014

This is finished and is ready to be merged.

@Bawhoppen
Copy link

Bawhoppen commented Jan 1, 2020

I know I'm kinda late to the party... but I still like this PR and wanna see it get merged... It deserves it.

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.

The syndicate silencer does not fit the buildable glock.
3 participants