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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

martian hotfix #77562

Merged
merged 6 commits into from
Aug 13, 2023
Merged

martian hotfix #77562

merged 6 commits into from
Aug 13, 2023

Conversation

EOBGames
Copy link
Contributor

@EOBGames EOBGames commented Aug 12, 2023

About The Pull Request

Fixes a few issues with Martian food. Makes stuff that's cooked on the grill or oven not burn, and adds guide recipes for the non-crafting-menu stuff.

Why It's Good For The Game

Fixes some problems, removes some confusion

Changelog

馃啈
fix: a few issues with Martian food
/:cl:

@tgstation-server tgstation-server added the Fix Rewrites a bug so it appears in different circumstances label Aug 12, 2023
@EOBGames EOBGames added the GBP: No Update Prevent changes to the user's GBP balance from this PR label Aug 12, 2023
@EOBGames
Copy link
Contributor Author

Adding no update since these are my own mistakes

@AdipemDragon
Copy link
Contributor

AdipemDragon commented Aug 12, 2023

image
comin in with a couple things too,
firstly an incorrect ingredient/description
and secondly, less an actual problem but moreso asking, should one cookie eat a whole bar of butter? there's butter slices now
(also thirdly i should get in and like, make more sprites for this martian food pr, cause that's straight up just the rootbreadsprite with no twist whatsoever for rice bread and its children recipes, should've known bout it before it was merged honestly)

@EOBGames
Copy link
Contributor Author

EOBGames commented Aug 13, 2023

image comin in with a couple things too, firstly an incorrect ingredient/description and secondly, less an actual problem but moreso asking, should one cookie eat a whole bar of butter? there's butter slices now (also thirdly i should get in and like, make more sprites for this martian food pr, cause that's straight up just the rootbreadsprite with no twist whatsoever for rice bread and its children recipes, should've known bout it before it was merged honestly)

Looks to me like I buggered up the pathing on the Prospect Sunrise- it's meant to be an omelette and egg sandwich (with pickles, yes). As for the boh loh yah, I made that long before butter slices were a thing and hadn't retrofitted it for that purpose because, frankly, I didn't know butter slices existed before this comment (keep in mind that this PR had been a rough WIP for about... 18 months? shit got real bad last year)

@AdipemDragon
Copy link
Contributor

(keep in mind that this PR had been a rough WIP for about... 18 months? shit got real bad last year)

well it's glad to see that it's gotten at least good enough to work through the pr, good luck in the future

@san7890 san7890 merged commit b981014 into tgstation:master Aug 13, 2023
19 checks passed
comfyorange added a commit that referenced this pull request Aug 13, 2023
Jolly-66 pushed a commit to TaleStation/TaleStation that referenced this pull request Aug 13, 2023
Original PR: tgstation/tgstation#77562
-----

## About The Pull Request
Fixes a few issues with Martian food. Makes stuff that's cooked on the
grill or oven not burn, and adds guide recipes for the non-crafting-menu
stuff.
## Why It's Good For The Game
Fixes some problems, removes some confusion
## Changelog
:cl:
fix: a few issues with Martian food
/:cl:

---------

Co-authored-by: EOBGames <58124831+EOBGames@users.noreply.github.com>
AlbertNanotracen pushed a commit to AlbertNanotracen/Monkestation2.0 that referenced this pull request Apr 3, 2024
## About The Pull Request
Fixes a few issues with Martian food. Makes stuff that's cooked on the
grill or oven not burn, and adds guide recipes for the non-crafting-menu
stuff.
## Why It's Good For The Game
Fixes some problems, removes some confusion
## Changelog
:cl:
fix: a few issues with Martian food
/:cl:
dwasint pushed a commit to Monkestation/Monkestation2.0 that referenced this pull request Apr 13, 2024
* Split drinks into different files (tgstation#75898)

Splits our nearly 4000 and 1600 line long files. Mostly just moves the
glass appearances into their own files.

* undoes removals of some bypass restrictions & metabolized_traits related things

* removed changes to on_hydroponics_apply()

* we had more arguments, missed that

* sigh

* Martian Food: A Taste of the Red Planet (tgstation#75988)

Adds a selection of new foods and drinks based around Mars.
More information on Mars can be found here:
https://github.com/tgstation/common_core/blob/master/Interesting%20Planets/Human%20Space/The%20Sol%20System.md
To summarise for the general audience, Mars is a vital colony of the
Terran Federation, having been primarily settled (at least originally)
by Cybersun Industries to harvest its lucrative supplies of plasma, the
second largest in human space behind Lavaland. This has given Mars a
diverse culture evolving from the mostly East Asian colonists, and their
food reflects this.

Thanks to Melbert for their work on the soup portion of this PR.

The food:
Martian cuisine draws upon the culinary traditions of East Asia, and
adds in fusion cuisine from the later colonists. Expect classics such as
ramen, curry, noodles and donburi, as well as new takes on the formula
like the Croque-Martienne, Peanut Butter Ice Cream Mochi, and the
Kitzushi- chilli cheese and rice inside a fried tofu casing. Oh, and
lots of pineapple. The Martians love pineapple:

![image](https://github.com/tgstation/tgstation/assets/58124831/c9ae33a1-e03a-4f94-8ce0-8ad124e88e8d)
Also included are some foods for Ethereals, which may or may not be
hinting at something I've got planned...

The drinks:
Four new base drinks make their way to the game, bringing with them a
host of new cocktails: enjoy new ventures in bartending with Coconut
Rum, Shochu/Soju, Yuyake (our favourite legally-distinct melon liqueur),
and Mars' favourite alcoholic beverage, rice beer. Each is available in
the dispenser, as well as bottles in the booze-o-mat:

![image](https://github.com/tgstation/tgstation/assets/58124831/914a6e2a-7ef5-4791-ae31-d08fa9211083)

The recipes:
To make your (and the wiki editors) lives easier, please find below the
recipes for both foods and drinks:
Food: https://hackmd.io/@EOBGames/BkVFU0w9Y
Drinks: https://hackmd.io/@EOBGames/rJ1OhnsJ2

Another lot of variety for the chef and bartender, as well as continuing
the work started with lizard and moth food in getting Common Core into
the game in a tangible and fun way.

:cl: EOBGames, MrMelbert
add: Mars celebrates the 250th anniversary of the Martian Concession
this year, and this has brought Martian cuisine to new heights of
popularity. Find a new selection of Martian foods and drinks available
in your crafting menu today!
/:cl:

---------

Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>

* updates martian can sprites, fixes misc text and bugs in wellcheers (tgstation#77757)

this is kind of a miscellaneous pile of fixes for soda.dmi, rolled up
with a larger resprite of the new cans that were added with tgstation#75988. you
can see everything visually changed (other than inhands) in the image
below

![image](https://github.com/tgstation/tgstation/assets/110322848/4c28f0c7-3975-4510-9958-9eef7f913051)
and then the inhands here:

![image](https://github.com/tgstation/tgstation/assets/110322848/03967e6e-6551-40d5-b2ec-7f27fabbc175)
also changes wellcheers mood event text, and the mood switch to be based
on sanity instead. apparently the mood test allowed for foolproof
traitor testing, which is not ideal. this hopefully fixes that.

the new martian drinks used the previous style of cans, which was
updated in tgstation#77424. this makes them consistent with that style.
everything else is minor things i missed or forgot to do in that PR.
also things that enable traitor testing are not great, even if they're
niche and require amputation to pull off.

:cl:
fix: wellcheers is now based on sanity rather than mood
image: resprites martian cans
image: adds inhands for martian cans and wellcheers
image: minor tweaks to other cans
spellcheck: updates wellcheers mood text
/:cl:

* Removing enriched liquid electricity references

* look, icons are hard

* boiled rice now needs to be boiled and recipe guide for boiled spaghetti/rice

* martian hotfix (tgstation#77562)

## About The Pull Request
Fixes a few issues with Martian food. Makes stuff that's cooked on the
grill or oven not burn, and adds guide recipes for the non-crafting-menu
stuff.
## Why It's Good For The Game
Fixes some problems, removes some confusion
## Changelog
:cl:
fix: a few issues with Martian food
/:cl:

* removed references to butter slices for now

* Change from improvised medicine

* better block comment formating

---------

Co-authored-by: tattle <66640614+dragomagol@users.noreply.github.com>
Co-authored-by: EOBGames <58124831+EOBGames@users.noreply.github.com>
Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: Lamb <110322848+CoiledLamb@users.noreply.github.com>
dwasint added a commit to Monkestation/Monkestation2.0 that referenced this pull request Jun 13, 2024
* Split drinks into different files (tgstation#75898)

Splits our nearly 4000 and 1600 line long files. Mostly just moves the
glass appearances into their own files.

* undoes removals of some bypass restrictions & metabolized_traits related things

* removed changes to on_hydroponics_apply()

* we had more arguments, missed that

* sigh

* Martian Food: A Taste of the Red Planet (tgstation#75988)

Adds a selection of new foods and drinks based around Mars.
More information on Mars can be found here:
https://github.com/tgstation/common_core/blob/master/Interesting%20Planets/Human%20Space/The%20Sol%20System.md
To summarise for the general audience, Mars is a vital colony of the
Terran Federation, having been primarily settled (at least originally)
by Cybersun Industries to harvest its lucrative supplies of plasma, the
second largest in human space behind Lavaland. This has given Mars a
diverse culture evolving from the mostly East Asian colonists, and their
food reflects this.

Thanks to Melbert for their work on the soup portion of this PR.

The food:
Martian cuisine draws upon the culinary traditions of East Asia, and
adds in fusion cuisine from the later colonists. Expect classics such as
ramen, curry, noodles and donburi, as well as new takes on the formula
like the Croque-Martienne, Peanut Butter Ice Cream Mochi, and the
Kitzushi- chilli cheese and rice inside a fried tofu casing. Oh, and
lots of pineapple. The Martians love pineapple:

![image](https://github.com/tgstation/tgstation/assets/58124831/c9ae33a1-e03a-4f94-8ce0-8ad124e88e8d)
Also included are some foods for Ethereals, which may or may not be
hinting at something I've got planned...

The drinks:
Four new base drinks make their way to the game, bringing with them a
host of new cocktails: enjoy new ventures in bartending with Coconut
Rum, Shochu/Soju, Yuyake (our favourite legally-distinct melon liqueur),
and Mars' favourite alcoholic beverage, rice beer. Each is available in
the dispenser, as well as bottles in the booze-o-mat:

![image](https://github.com/tgstation/tgstation/assets/58124831/914a6e2a-7ef5-4791-ae31-d08fa9211083)

The recipes:
To make your (and the wiki editors) lives easier, please find below the
recipes for both foods and drinks:
Food: https://hackmd.io/@EOBGames/BkVFU0w9Y
Drinks: https://hackmd.io/@EOBGames/rJ1OhnsJ2

Another lot of variety for the chef and bartender, as well as continuing
the work started with lizard and moth food in getting Common Core into
the game in a tangible and fun way.

:cl: EOBGames, MrMelbert
add: Mars celebrates the 250th anniversary of the Martian Concession
this year, and this has brought Martian cuisine to new heights of
popularity. Find a new selection of Martian foods and drinks available
in your crafting menu today!
/:cl:

---------

Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>

* updates martian can sprites, fixes misc text and bugs in wellcheers (tgstation#77757)

this is kind of a miscellaneous pile of fixes for soda.dmi, rolled up
with a larger resprite of the new cans that were added with tgstation#75988. you
can see everything visually changed (other than inhands) in the image
below

![image](https://github.com/tgstation/tgstation/assets/110322848/4c28f0c7-3975-4510-9958-9eef7f913051)
and then the inhands here:

![image](https://github.com/tgstation/tgstation/assets/110322848/03967e6e-6551-40d5-b2ec-7f27fabbc175)
also changes wellcheers mood event text, and the mood switch to be based
on sanity instead. apparently the mood test allowed for foolproof
traitor testing, which is not ideal. this hopefully fixes that.

the new martian drinks used the previous style of cans, which was
updated in tgstation#77424. this makes them consistent with that style.
everything else is minor things i missed or forgot to do in that PR.
also things that enable traitor testing are not great, even if they're
niche and require amputation to pull off.

:cl:
fix: wellcheers is now based on sanity rather than mood
image: resprites martian cans
image: adds inhands for martian cans and wellcheers
image: minor tweaks to other cans
spellcheck: updates wellcheers mood text
/:cl:

* Removing enriched liquid electricity references

* look, icons are hard

* boiled rice now needs to be boiled and recipe guide for boiled spaghetti/rice

* martian hotfix (tgstation#77562)

## About The Pull Request
Fixes a few issues with Martian food. Makes stuff that's cooked on the
grill or oven not burn, and adds guide recipes for the non-crafting-menu
stuff.
## Why It's Good For The Game
Fixes some problems, removes some confusion
## Changelog
:cl:
fix: a few issues with Martian food
/:cl:

* removed references to butter slices for now

* Change from improvised medicine

* better block comment formating

* T (tgstation#77442)

## About The Pull Request
Adds T, basically a boosted tea for mimes.
Can be made by mixing Nothing with Tea, trading off the brute healing
for toxin & drowsiness healing.


![dinky-export2](https://github.com/tgstation/tgstation/assets/66052067/8a81f719-8fd8-4dd2-be74-58ff2442ce94)
## Why It's Good For The Game


https://github.com/tgstation/tgstation/assets/66052067/9df3ee77-ff45-49ce-a56d-59004e4b308c
## Changelog
:cl: Wallem
add: T
/:cl:

* Fixes silliness involving soup pots clearing ingredients list  (tgstation#74882)

Fixes tgstation#74866

The issue is misleading, it never deleted the disk. The disk remained in
the pot's contents but was no longer tracked as an ingredient, leaving
it stuck

Foolishly I had a case for indestructible things when soup reactions
finished, where it wouldn't delete stuff like the disk.
But I never removed it from the pot. I just nulled the ingredients list.
Oops.

So pots will drop ingredients that fail to cook into anything when the
reaction finishes.

---

Semi-unrelated but stuff like stoves, ovens, and chem heater should
probably `allowdrop() return FALSE`. I cannot think of many (any)
situations where you would want to use `drop_location` from something in
an oven, stove, or heater and actually want those returned rather than
their own loc.

Whoops

:cl: Melbert
fix: Ingredients which fail to react in soup reactions drop when the
reaction finishes as intended
/:cl:

* Adds the Death Sandwich to the game (tgstation#75013)

Adds the Death Sandwich to the game, the ultimate form of
bread-conveyed-meat-based consumables.

![deathsandwich](https://user-images.githubusercontent.com/66052067/235041733-287be1fd-1eed-4d6d-840b-96f95494f093.png)

And remember;
Eat it right, or you die!

I'm genuinely surprised we don't already have a meatball sub in the game
also I love humor food, and I doubt my edition of the Eggcellent
Challenge would ever be merged if I tried to do so, so this is the next
best thing.

:cl: Wallem
add: The ancient recipe for the Death Sandwich has been rediscovered
buried in the deepest depths of an erupting volcano.
/:cl:

* Wedding Cakes (tgstation#75076)

## About The Pull Request

Added wedding cakes, which are sliceable into wedding cake slices.
Mechanically, they are just worse pound cakes - they require more
ingredients and provide less nutrients - but they are here for the
flavor. Sometimes a pound cake just doesn't suffice when you want to
roleplay a wedding.

Fair warning, I've not done a pull request here before, any advice
welcome.

<img
src='https://user-images.githubusercontent.com/102194057/235379312-2eaecffa-c48f-45b6-a3c6-09bf4bbff14a.png'>

## Why It's Good For The Game

Roleplayers like weddings, and what is a wedding without a cake? (This
PR was originally made for Skyrat but they recommended I take it
upstream).

## Changelog
:cl:
add: Added wedding cakes
/:cl:

---------

Co-authored-by: tattle <66640614+dragomagol@users.noreply.github.com>

* Adds a var to chemical reactions that allows them to take place in subtypes of their required container (tgstation#75527)

## About The Pull Request

As the title may imply, a variable has been added to chemical reactions
that'll make them check istype on the required container, rather than
looking for the exact type path.
## Why It's Good For The Game

In my specific example, lets say I want to have sybtypes of soup pots.
The current behavior doesn't allow for these subtypes of soup pots to
actually make soup, which is sub-optimal.
## Changelog
:cl:
code: A var has been added to chemical reactions (defaulted to FALSE to
keep current behavior) that allows subtypes of that reactions required
container to be used as well, rather than the exact type.
/:cl:

* adds toast sandwich (tgstation#75545)

## About The Pull Request

Adds toast sandwich to the game.

![image](https://github.com/tgstation/tgstation/assets/32695675/6860aff1-7dde-4e0b-9ed8-a4343cf3f115)

## Why It's Good For The Game

Gives our UK players a little taste of home :)
Also it's funny

## Changelog

:cl: Epoc
add: Toast sandwich
/:cl:

---------

Co-authored-by: alec-b-williams <alec.b.williams@wsu.edu>
Co-authored-by: alec-b-williams <78933704+alec-b-williams@users.noreply.github.com>

* Fixes a little inconsistency with the peanut butter banana sandwich and makes sandwiches subtypes of the generic sandwhich. (tgstation#75750)

The peanut butter banana sandwich is presented ingame as grilled when it
doesn't require a griddle to make.
Also, I'm taking the opportunity to repath the sandwich types, since
they all share the same weight class, food flags and icons file.

This will fix tgstation#74585.

:cl:
fix: The peanut butter banana sandwich is no longer presented as grilled
when it doesn't require a griddle to make.
/:cl:

* Map changes needed for sandwich refactor

* i really need to find a better way to do this

* Modify death sandwich disease to work with pathology

* Blood brothers can now chose the death sandwich recipe as their gear

* the cure should be shown somewhere in game, that was dumb of me

* changed our boiling recipes that output items to match 24-volt fist

* wedding cake can feed a small wedding party

---------

Co-authored-by: tattle <66640614+dragomagol@users.noreply.github.com>
Co-authored-by: EOBGames <58124831+EOBGames@users.noreply.github.com>
Co-authored-by: MrMelbert <51863163+MrMelbert@users.noreply.github.com>
Co-authored-by: Lamb <110322848+CoiledLamb@users.noreply.github.com>
Co-authored-by: Wallem <66052067+Wallemations@users.noreply.github.com>
Co-authored-by: Thlumyn <102194057+Thlumyn@users.noreply.github.com>
Co-authored-by: Paxilmaniac <82386923+Paxilmaniac@users.noreply.github.com>
Co-authored-by: Epoc <32695675+epochayur@users.noreply.github.com>
Co-authored-by: alec-b-williams <alec.b.williams@wsu.edu>
Co-authored-by: alec-b-williams <78933704+alec-b-williams@users.noreply.github.com>
Co-authored-by: Ghom <42542238+Ghommie@users.noreply.github.com>
Co-authored-by: dwasint <82520990+dwasint@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fix Rewrites a bug so it appears in different circumstances GBP: No Update Prevent changes to the user's GBP balance from this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants