HelicopterRevival


Adds a fully animated attack helicopter to the game. Ideal for getting from A to B quickly, exploring the map, building islands and nuking biters from the sky. Originally forked from Kumpu with fixes for Factorio 0.18.2+, but adding some enhancements of my own as well.

Content
10 hours ago
0.18 - 2.0
92.9K
Transportation

i [enhancement] Factoriopedia

a day ago
(updated a day ago)

Hi!
I have a request: the Factoriopedia shows three helicopters and two landing pads. Please fix this...

This happens because the entity/item/recipe has different names, and they aren't combined into a single entry in Factoriopedia. To create a single entry, you need to specify the same name for each entity/item/recipe.

If you don't want to do it yourself, I can send you the corrected mod :)

This is odd, because most entities have the flag hide_from_factoriopedia set to true.

But you can toss in a pull request c:

a day ago
(updated a day ago)

Yes, I noticed, but these entities aren't linked to the main entity/item/recipe, that is, the core of the prototype, and since they have different names, they appear as different cards.
Hmm... to my shame, I admit, I don't know how to use pull request, and even when I do something there, I just click buttons at random :(

To be more precise, we are talking about:
1. "heli-item" / "heli-recipe" / "heli-entity-_-"
2. "heli-pad-item" / "heli-pad-recipe"

a day ago

And one more small request: please add compatibility not only with "Krastorio2", but also with "Krastorio2-spaced-out" :)

And one more small request: please add compatibility not only with "Krastorio2", but also with "Krastorio2-spaced-out" :)

well, the "compatibility" with K2 is basically just fuel recalculation... Does spaced out do things differently?

a day ago
(updated a day ago)

No, but the regular "Krastorio2" only works without the Space Age expansion, but to play with the Space Age expansion, "Krastorio2-spaced-out" works.
I just noticed that the mod is compatible with the regular "Krastorio2," but those playing with the Space Age expansion won't receive the changes in the current mod.
So, I'm just asking for a correction:

if mods["Krastorio2"] or mods["Krastorio2-spaced-out"] then
a day ago
(updated a day ago)

No, but the regular "Krastorio2" only works without the Space Age expansion, but to play with the Space Age expansion, "Krastorio2-spaced-out" works.
I just noticed that the mod is compatible with the regular "Krastorio2," but those playing with the Space Age expansion won't receive the changes in the current mod.
So, I'm just asking for a correction:
if mods["Krastorio2"] or mods["Krastorio2-spaced-out"] then

Yeah, I kinda figured that'd be the case... Will be fixed shortly...

a day ago

Yes, I noticed, but these entities aren't linked to the main entity/item/recipe, that is, the core of the prototype, and since they have different names, they appear as different cards.
Hmm... to my shame, I admit, I don't know how to use pull request, and even when I do something there, I just click buttons at random :(

To be more precise, we are talking about:
1. "heli-item" / "heli-recipe" / "heli-entity-_-"
2. "heli-pad-item" / "heli-pad-recipe"

What exactly would be your proposal to change in the mod to make it work?

Yes, I noticed, but these entities aren't linked to the main entity/item/recipe, that is, the core of the prototype, and since they have different names, they appear as different cards.
Hmm... to my shame, I admit, I don't know how to use pull request, and even when I do something there, I just click buttons at random :(

To be more precise, we are talking about:
1. "heli-item" / "heli-recipe" / "heli-entity-_-"
2. "heli-pad-item" / "heli-pad-recipe"

What exactly would be your proposal to change in the mod to make it work?

I already have most of this working in my local copy... Namely, the -recepie items should be replaced with the corresponding item... like heli-recipe should be replaced heli-item, heli-remote-recipe should be renamed heli-remote-equipment and so on...

heli-item and heli-entity are most annoying things to squish together.

a day ago
(updated 23 hours ago)

What exactly would be your proposal to change in the mod to make it work?

You need to combine these names into a single name:
1. "heli-item" / "heli-recipe" / "heli-entity-_-" == "heli"
2. "heli-pad-item" / "heli-pad-recipe" == "heli-pad"
Or whatever you think is appropriate to call them.
I just did this by searching for all matches, since some of them were in different files.
I tried the pull request, but it redirects me to the owner's repository, the very first one. I don't understand how to make a request to your repo, so I can just tell you where I found the name matches, if that's appropriate.


line number
control.lua : 234, 404, 422
en \ locale.cfg : 30, 33, 39, 42, 46, 49, 50, 54, 60, 63
logic \ heliAttack.lua : 15, 37, 49
prototypes \ items.lua : 4, 150, 155
prototypes \ prototypes.lua : 57, 67, 84, 90, 162, 166
prototypes \ entities \ heli_entity.lua : 21, 97, 104, 193, 262, 332, 400, 472, 544, 634
prototypes \ entities \ heli_pad.lua : 17, 59
prototypes \ entities \ rotor_entity.lua : 10, 81

It seems like these are all coincidences.

23 hours ago
(updated 23 hours ago)

I already have most of this working in my local copy... Namely, the -recepie items should be replaced with the corresponding item... like heli-recipe should be replaced heli-item, heli-remote-recipe should be renamed heli-remote-equipment and so on...

heli-item and heli-entity are most annoying things to squish together.

It's enough to rename only these main ones; I didn't change the others, since it seemed like nothing would depend on them.
Essentially, you need to combine them into a single name:
1. "heli-item" / "heli-recipe" / "heli-entity-_-" == "heli"
2. "heli-pad-item" / "heli-pad-recipe" == "heli-pad"

10 hours ago

I already have most of this working in my local copy... Namely, the -recepie items should be replaced with the corresponding item... like heli-recipe should be replaced heli-item, heli-remote-recipe should be renamed heli-remote-equipment and so on...

heli-item and heli-entity are most annoying things to squish together.

It's enough to rename only these main ones; I didn't change the others, since it seemed like nothing would depend on them.
Essentially, you need to combine them into a single name:
1. "heli-item" / "heli-recipe" / "heli-entity-_-" == "heli"
2. "heli-pad-item" / "heli-pad-recipe" == "heli-pad"

All done in the latest release c:

10 hours ago

Thank you!

New response