Multi cooker #8339

HuXTUS · Jul 26, 2014

(just image from internet, not advertising)

Expensive multi cooker with plenty of cooking functions. Main feature: he cooks asynchronously.

Just put in it the ingredients, select the desired program (recipe), and go about your business, he will do everything himself and signal when the dish is ready.

Now you do not have to watch your barbecue or potatoes, you can trust the robot, and do other things. But first you have to find it somewhere ...

Feature: using your electronics and fabrication skills you can upgrade device, so it will cooks faster.

This device will increase grace of your home.

kevingranade · Jul 26, 2014

Neat concept. I think this is a good way to introduce a prototype of this, and maybe we can convert some regular recipes to use something similar, like have a setup stage, then an unattended stage.

BevapDin · Jul 26, 2014

BevapDin reviewed Jul 26, 2014

View changes

BevapDin · Jul 26, 2014

src/iuse.cpp

+                p->add_msg_if_player(m_good, _("You pull out the %s."), dish.tname().c_str());
+            }
+
+            p->inv.assign_empty_invlet(dish, true);


You don't need this, it's not wrong, but the inventory will automatically assign an invlet when the item is added (by player::i_add).

KA101 · Jul 27, 2014

KA101 reviewed Jul 27, 2014

View changes

KA101 · Jul 27, 2014

data/json/items/tools.json

+    "symbol": "M",
+    "color": "red",
+    "name": "multi cooker",
+    "description": "An expensive multi cooker professional level.  Has a lot of buttons with different settings and programs.  The dream of every housewife and inveterate bachelor.  With its help you can prepare anything from the boiled potatoes, to exquisite dishes.  Unfortunately, user manual and a examples list of recipes lost, so the recipes you must know himself.",


A professional-grade multi-cooker, with a battery slot for camping trips or tailgating. Its multitude of settings and features promise to handle any sort of cooking, from parboiling potatoes to cooking curry to popping popcorn. There's no manual, but you're sure you can work it out.

KA101 · Jul 27, 2014

KA101 reviewed Jul 27, 2014

View changes

KA101 · Jul 27, 2014

src/iuse.cpp

+            it->item_vars["DISH"] = "";
+
+            point pos = g->find_item(it);
+            g->sound(pos.x, pos.y, 8, "Ring of bells");


KA101 · Jul 27, 2014

KA101 reviewed Jul 27, 2014

View changes

KA101 · Jul 27, 2014

src/iuse.cpp

+            //Smart or good cook or careful
+            if (p->int_cur + p->skillLevel("cooking") + p->skillLevel("survival") > 16) {
+                item dummy(it->item_vars["DISH"], 0);
+                add_msg(m_info, _("Do you feel that your %s will soon be ready."), dummy.display_name(1).c_str());


"The multi-cooker should be finishing shortly..."

KA101 · Jul 27, 2014

KA101 reviewed Jul 27, 2014

View changes

KA101 · Jul 27, 2014

src/iuse.cpp

+
+        uimenu menu;
+        menu.selected = 0;
+        menu.text = _("Welcome to the RobotWife3000. What would you like to do?");


Nope. RobotChef3000 or something similarly neutral.

KA101 · Jul 27, 2014

KA101 reviewed Jul 27, 2014

View changes

KA101 · Jul 27, 2014

data/json/items/tools.json

+  {
+    "type": "TOOL",
+    "id": "multi_cooker",
+    "symbol": "M",


As a tool, it should probably be ";" rather than "M".

How about a special occasion? Professional multi cooker it's almost furniture. Big device, not tool like scissors or a soldering iron.

Thus the monster hallu-multicooker it will look also adequately.

Although it is not essential.

Here I thought that seeing semicolons attacking would make the experience that much more freaky. Anyone else have an opinion?

No, i agree. Just I liked red symbols "M". )

KA101 · Jul 27, 2014

KA101 reviewed Jul 27, 2014

View changes

KA101 · Jul 27, 2014

data/json/monsters.json

+    "id" : "mon_hallu_multicooker",
+    "name": "multi cooker",
+    "species":"HALLUCINATION",
+    "symbol":"M",


Likewise here, even though it's the dreaded Evil Multi-Cooker.

KA101 · Jul 27, 2014

Not yet ready for merge testing. I like the hallucination idea, though.

HuXTUS · Jul 28, 2014

HuXTUS reviewed Jul 28, 2014

View changes

HuXTUS · Jul 28, 2014

src/iuse.cpp

+
+        uimenu menu;
+        menu.selected = 0;
+        menu.text = _("Welcome to the RobotChef KA101-model. Choose option:");


I hope you enjoy this easter egg, which will perpetuate your name in the draft.

Unexpected, unnecessary, and I imagine it'll make the forum unhappy. Thanks for the thought, though.

HuXTUS · Jul 28, 2014

Ok, no easter eggs now.

BevapDin · Jul 28, 2014

The other sounds are usually translated, this one should be, too.

Uh, yes... Now all ok, i hope.

HuXTUS · Jul 28, 2014

GitHub application for Windows scared me ...

KA101 · Jul 29, 2014

KA101 reviewed Jul 29, 2014

View changes

KA101 · Jul 29, 2014

src/iuse.cpp

@@ -8406,3 +8406,387 @@ int iuse::radiocontrol(player *p, item *it, bool t)

    return it->type->charges_to_use();
 }
+
+bool multicooker_hallu(player *p)


I'd not seen this call before and didn't expect it to work. I'm happy to be proven wrong, though.

Gonna give it another proofread now that everything's more or less working (doesn't keep going if you take the reality bubble away, but that's true of pretty much everything: not a dealbreaker IMO), and then merge it.

KA101 · Jul 29, 2014

KA101 reviewed Jul 29, 2014

View changes

KA101 · Jul 29, 2014

data/json/monsters.json

+{
+    "type" : "MONSTER",
+    "id" : "mon_hallu_multicooker",
+    "name": "Evil Multi-Cooker",


For future reference, you need to add listings for this in a few other files. Works for Curses, but essential for tileset support, etc.

Now I understand why you add monsters in the source files.

It will be necessary to sort out this issue, or maybe someone will do it for me, since I do not know how to build tiles version and not be able to verify that the game does not crash, because of my changes.

Taken care of in the merge. np.

project started

Loading status checks…

a35000b

HuXTUS changed the title ~~[WIP] Multi cooker~~ [CR] Multi cooker Jul 27, 2014

HuXTUS changed the title ~~[CR] Multi cooker~~ [ready?] [CR] Multi cooker Jul 27, 2014

premonition that meal is ready soon

Loading status checks…

09200d7

HuXTUS changed the title ~~[ready?] [CR] Multi cooker~~ [WIP] Multi cooker Jul 28, 2014

Styling texts (KA101 suggestions )

Loading status checks…

8096b74

HuXTUS changed the title ~~[WIP] Multi cooker~~ [READY] Multi cooker Jul 28, 2014

protection forum from unhappy

Loading status checks…

2db1853

lost underscore for "ding"

Loading status checks…

53341ec

HuXTUS closed this Jul 28, 2014

HuXTUS deleted the HuXTUS:multi-cooker branch Jul 28, 2014

HuXTUS restored the HuXTUS:multi-cooker branch Jul 28, 2014

HuXTUS reopened this Jul 28, 2014

check for underwater

Loading status checks…

857ae97

KA101 self-assigned this Jul 28, 2014

KA101 merged commit 857ae97 into CleverRaven:master Jul 29, 2014
1 check failed

1 check failed

default
Details

HuXTUS deleted the HuXTUS:multi-cooker branch Jul 29, 2014

HuXTUS changed the title ~~[READY] Multi cooker~~ Multi cooker Jul 29, 2014

kevingranade changed the title ~~Multi cooker~~ Multi cooker Sep 20, 2014

CleverRaven/Cataclysm-DDA

Join GitHub today

Multi cooker #8339

Conversation

HuXTUS commented Jul 26, 2014

This comment has been minimized.

kevingranade commented Jul 26, 2014

BevapDin reviewed Jul 26, 2014 View changes

This comment has been minimized.

HuXTUS added some commits Jul 27, 2014

HuXTUS changed the title [WIP] Multi cooker [CR] Multi cooker Jul 27, 2014

HuXTUS added some commits Jul 27, 2014

HuXTUS changed the title [CR] Multi cooker [ready?] [CR] Multi cooker Jul 27, 2014

KA101 reviewed Jul 27, 2014 View changes

This comment has been minimized.

KA101 reviewed Jul 27, 2014 View changes

This comment has been minimized.

KA101 reviewed Jul 27, 2014 View changes

This comment has been minimized.

KA101 reviewed Jul 27, 2014 View changes

This comment has been minimized.

KA101 reviewed Jul 27, 2014 View changes

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

KA101 reviewed Jul 27, 2014 View changes

This comment has been minimized.

This comment has been minimized.

KA101 commented Jul 27, 2014

HuXTUS changed the title [ready?] [CR] Multi cooker [WIP] Multi cooker Jul 28, 2014

HuXTUS reviewed Jul 28, 2014 View changes

This comment has been minimized.

This comment has been minimized.

HuXTUS changed the title [WIP] Multi cooker [READY] Multi cooker Jul 28, 2014

This comment has been minimized.

HuXTUS commented Jul 28, 2014

This comment has been minimized.

BevapDin commented on src/iuse.cpp in 8096b74 Jul 28, 2014

This comment has been minimized.

HuXTUS replied Jul 28, 2014

HuXTUS closed this Jul 28, 2014

HuXTUS deleted the HuXTUS:multi-cooker branch Jul 28, 2014

HuXTUS restored the HuXTUS:multi-cooker branch Jul 28, 2014

This comment has been minimized.

HuXTUS commented Jul 28, 2014

HuXTUS reopened this Jul 28, 2014

KA101 self-assigned this Jul 28, 2014

KA101 reviewed Jul 29, 2014 View changes

This comment has been minimized.

KA101 reviewed Jul 29, 2014 View changes

This comment has been minimized.

This comment has been minimized.

This comment has been minimized.

Hide details View details KA101 merged commit 857ae97 into CleverRaven:master Jul 29, 2014 1 check failed

1 check failed

HuXTUS deleted the HuXTUS:multi-cooker branch Jul 29, 2014

HuXTUS changed the title [READY] Multi cooker Multi cooker Jul 29, 2014

kevingranade changed the title Multi cooker Multi cooker Sep 20, 2014

BevapDin reviewed Jul 26, 2014

View changes

KA101 reviewed Jul 27, 2014

View changes

KA101 reviewed Jul 27, 2014

View changes

KA101 reviewed Jul 27, 2014

View changes

KA101 reviewed Jul 27, 2014

View changes

KA101 reviewed Jul 27, 2014

View changes

KA101 reviewed Jul 27, 2014

View changes

HuXTUS reviewed Jul 28, 2014

View changes

BevapDin commented on `src/iuse.cpp` in `8096b74` Jul 28, 2014

KA101 reviewed Jul 29, 2014

View changes

KA101 reviewed Jul 29, 2014

View changes

KA101 merged commit `857ae97` into CleverRaven:master Jul 29, 2014
1 check failed