Skip to content
This repository has been archived by the owner on Dec 5, 2020. It is now read-only.

Kēki ❤️ #35

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

Kēki ❤️ #35

wants to merge 38 commits into from

Conversation

hitomishirichan
Copy link

cakes! cakes! cakes! cakes! cakes! cakes! cakes!

more information and DMIs located on the forum post :D

WARNING:
The item path for icing tubes has been modified, so maps will need to be updated. Icing tubes have been turned into bottles so they can be placed in drink dispensers (as opposed to chem dispensers) and players can drink out of them

Copy link
Member

@ZeWaka ZeWaka left a comment

Choose a reason for hiding this comment

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

Here's some comments.

code/datums/cookingrecipes.dm Outdated Show resolved Hide resolved
proc/put_out(var/mob/user as mob)
if (!src) return
if (src.litfam)
src.litfam = 0
Copy link
Member

Choose a reason for hiding this comment

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

Why not use initial() for this and below?

code/modules/food_and_drink/cakes.dm Outdated Show resolved Hide resolved
code/modules/food_and_drink/cakes.dm Outdated Show resolved Hide resolved
slicecolor.color = buffer.color
s.UpdateOverlays(slicecolor,"base") //setting the base overlay of the temporary slice object
src.ClearSpecificOverlays(thing)
staticiterator--
Copy link
Member

Choose a reason for hiding this comment

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

I'm unsure as to the purpose of this line, as well as similar usages below

code/modules/food_and_drink/cakes.dm Outdated Show resolved Hide resolved
else
src.clayer--
return
else if((!(src.litfam)) && (((istype(W,/obj/item/weldingtool) && W:welding)) || (istype(W, /obj/item/device/igniter))) || \
Copy link
Member

Choose a reason for hiding this comment

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

similar toolflag

boutput(user, "<span style=\"color:red\">The icing tube isn't full enough to frost the cake!</span>")
return
var/frostingtype
frostingtype = input("Which frosting style would you like?", "Frosting Style", null) as null|anything in frostingstyles
Copy link
Member

Choose a reason for hiding this comment

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

It would be nice and much more user-friendly for mass-making of cakes if this selection could be transferred over to the icing tube self attack.

var/blowout
var/staticiterator = src.overlays.len
for(var/i=1,i<=staticiterator,i++)
if(src.sliced && ("[src.overlay_refs[i]]" == "slice-candle_lit"))
Copy link
Member

Choose a reason for hiding this comment

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

could put this inside the other if statement to avoid comparing twice

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

Successfully merging this pull request may close these issues.

3 participants