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

Allows for sabre renaming. #20367

Merged
merged 5 commits into from Sep 12, 2016
Merged

Allows for sabre renaming. #20367

merged 5 commits into from Sep 12, 2016

Conversation

TechnoAlchemisto
Copy link
Contributor

@TechnoAlchemisto TechnoAlchemisto commented Sep 6, 2016

🆑 Papa Bones
tweak: You can now rename the captain's sabre by using a pen on it.
/:cl:

Requested by some schmuck on the forums.

Allows for any weapon to use the unique_rename var.

@AnturK AnturK added the Feature Exposes new bugs in interesting ways label Sep 6, 2016
@DfourC
Copy link

DfourC commented Sep 6, 2016

I'm that schmuck!

@Ohlos
Copy link

Ohlos commented Sep 6, 2016

#Best feature 2016

@optimumtact
Copy link
Member

This is already functionality from the item relabeller why are you snowflaking it?

@PKPenguin321
Copy link
Contributor

Actually this looks copypasted from gun code

@TechnoAlchemisto
Copy link
Contributor Author

TechnoAlchemisto commented Sep 6, 2016

that's because it is

The label gun does not rename the item it just adds () at the end of it.

This gives the same functionality as KA's and Revolvers

@zig-for
Copy link
Contributor

zig-for commented Sep 6, 2016

If the same code already exists in /weapon/gun, can you just promote it to /weapon/ ?

@TechnoAlchemisto
Copy link
Contributor Author

If the obj/item/weapon/ path wasn't used for half the things in this game I'd say that'd be the better option, I mean if a maintainer wants me to do that I will.

@jughu
Copy link
Contributor

jughu commented Sep 6, 2016

Papa bones is the best 👍

@zig-for
Copy link
Contributor

zig-for commented Sep 7, 2016

I don't see the harm in promoting it. It would only work for things that have unique_rename set, and let renaming work for a larger range of shit. If you're worried about extra memory pressure from the extra variable on a bunch of shit, combine some flags into a single bitfield.

@TehZombehz
Copy link
Contributor

I leave you this sword given to me by my precious queen and the word engraved upon it: Luck. But first, let me affix my own benediction: Pluck!

@PKPenguin321
Copy link
Contributor

you can probably remove these vars from guns since guns are childs of weapons, arent they? just move the whole system up

@TechnoAlchemisto
Copy link
Contributor Author

TechnoAlchemisto commented Sep 7, 2016

They are a child of weapons but not a child of /melee/ (which is what I put this on)

@optimumtact
Copy link
Member

optimumtact commented Sep 8, 2016

Fix the snowflake code, I suggest some helper procs for renaming

@optimumtact optimumtact added the Do Not Merge You must have really upset someone label Sep 8, 2016
@TechnoAlchemisto
Copy link
Contributor Author

Yeah, I tried to standardize it to all weapons but it breaks renaming for anything that's a gun, no idea what the exact cause is, RIP.

M << "You name the weapon [input]. Say hello to your new friend."
return
>>>>>>> 04bb24d27a666b5821c9c053014095c032095f9d

Copy link
Contributor

Choose a reason for hiding this comment

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

Leaving merge conflict markers in

@TechnoAlchemisto
Copy link
Contributor Author

fuck my ass 1 sec

@@ -21,6 +23,7 @@
name = input
M << "You name the weapon [input]. Say hello to your new friend."
return
>>>>>>> 04bb24d27a666b5821c9c053014095c032095f9d
Copy link
Contributor

Choose a reason for hiding this comment

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

Uh ohhh spaghettio

@TechnoAlchemisto
Copy link
Contributor Author

where is god in these troubling times?

@TechnoAlchemisto
Copy link
Contributor Author

This is ready for review by the way, @optimumtact

@optimumtact optimumtact added Ready for Review and removed Do Not Merge You must have really upset someone labels Sep 10, 2016
@zig-for
Copy link
Contributor

zig-for commented Sep 10, 2016

It amuses me that pen is a /weapon/. So you could concievably use another pen to rename your pen.

@optimumtact optimumtact merged commit eec3391 into tgstation:master Sep 12, 2016
@TechnoAlchemisto
Copy link
Contributor Author

Thanks oranges, you the real MVP.

@TechnoAlchemisto TechnoAlchemisto deleted the cant_believe_shadowlight_banned_me branch September 12, 2016 23:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Feature Exposes new bugs in interesting ways
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

9 participants