[Twinsen] [0.18.3] [Modding] NestedTriggerEffectItem tooltip ignores repeat count in a tooltip

This subforum contains all the issues which we already resolved.
User avatar
lovely_santa
Filter Inserter
Filter Inserter
Posts: 502
Joined: Sat Feb 18, 2017 9:41 pm
Contact:

[Twinsen] [0.18.3] [Modding] NestedTriggerEffectItem tooltip ignores repeat count in a tooltip

Post by lovely_santa »

Hi,

In Angels Exploration I am creating an ammo that shoots 2 (identical) projectiles:

Code: Select all

      ammo_type = {
      category = "cannon-shell",
      target_type = "direction",
      action = {
        type = "direct",
        repeat_count = 2,
        action_delivery = {
          type = "projectile",
          projectile = "heavy-explosive-cannon-projectile",
          starting_speed = 1,
          direction_deviation = 0.2,
          range_deviation = 0.1,
          max_range = 40,
          source_effects = {
            type = "create-explosion",
            entity_name = "explosion-gunshot"
          }
        }
      }
    },
And the projectile itself has a nested result that has area of effect damage:

Code: Select all

    action = {
      type = "direct",
      action_delivery = {
        type = "instant",
        target_effects = {
          {
            type = "damage",
            damage = {amount = 150, type = "physical"}
          },
          {
            type = "create-entity",
            entity_name = "explosion"
          }
        }
      }
    },
    final_action = {
      type = "direct",
      action_delivery = {
        type = "instant",
        target_effects = {
          {
            type = "create-entity",
            entity_name = "big-explosion"
          },
          {
            type = "nested-result",
            action = {
              type = "area",
              radius = 4,
              action_delivery = {
                type = "instant",
                target_effects = {
                  {
                    type = "damage",
                    damage = {amount = 300, type = "explosion"}
                  },
                  {
                    type = "create-entity",
                    entity_name = "explosion"
                  }
                }
              }
            }
          }
        }
      }
    },
Bilka suggested to report this as it seems that the NestedTriggerEffectItem just ignores the repeat count, and this is visible ingame for example in the item info where the direct damage is showing '2x', but the (nested) area of effect isn't
Capture.PNG
Capture.PNG (937.18 KiB) Viewed 2987 times
To quickly test this, I've attached the mod and a save file.
The code for the projectile and ammo can be found in prototypes/entities/ammo-heavy-cannon-shells.lua line 99-208.

Kind regards
lovely_santa
Attachments
angelsexploration_bug.zip
(1.35 MiB) Downloaded 153 times
angelsexploration_0.3.0.zip
(12.1 MiB) Downloaded 128 times
You can find all my mods on the mod portal. Also helping on Arch666Angel's mods.
Image
User avatar
Klonan
Factorio Staff
Factorio Staff
Posts: 5267
Joined: Sun Jan 11, 2015 2:09 pm
Contact:

Re: [0.18.3][Modding] NestedTriggerEffectItem ignores repeat count

Post by Klonan »

I think this is more of a modding feature request, nothing is broken, just it was never added to that specific trigger effect
Bilka
Factorio Staff
Factorio Staff
Posts: 3309
Joined: Sat Aug 13, 2016 9:20 am
Contact:

Re: [0.18.3][Modding] NestedTriggerEffectItem ignores repeat count

Post by Bilka »

Klonan wrote: Tue Feb 04, 2020 6:10 pm I think this is more of a modding feature request, nothing is broken, just it was never added to that specific trigger effect
How is "the tooltip is wrong" not broken?
I'm an admin over at https://wiki.factorio.com. Feel free to contact me if there's anything wrong (or right) with it.
User avatar
lovely_santa
Filter Inserter
Filter Inserter
Posts: 502
Joined: Sat Feb 18, 2017 9:41 pm
Contact:

Re: [0.18.3][Modding] NestedTriggerEffectItem ignores repeat count

Post by lovely_santa »

Bilka wrote: Tue Feb 04, 2020 6:16 pm
Klonan wrote: Tue Feb 04, 2020 6:10 pm I think this is more of a modding feature request, nothing is broken, just it was never added to that specific trigger effect
How is "the tooltip is wrong" not broken?
What bilka said ^^ The effect is working correctly, the tooltip is wrong, it's saying the area of effect damage is only applied once (as the '2x' is missing).
You can find all my mods on the mod portal. Also helping on Arch666Angel's mods.
Image
Twinsen
Factorio Staff
Factorio Staff
Posts: 1348
Joined: Tue Sep 23, 2014 7:10 am
Contact:

Re: [Twinsen] [0.18.3] [Modding] NestedTriggerEffectItem tooltip ignores repeat count in a tooltip

Post by Twinsen »

Fixed in Version: 0.18.16, for Nested, Area and Cluster trigger effects.
Thanks for the mod and save.

Not sure if it was like that for a reason. So let me know if you notice some tooltips showing repeat count when it shouldn't.
User avatar
lovely_santa
Filter Inserter
Filter Inserter
Posts: 502
Joined: Sat Feb 18, 2017 9:41 pm
Contact:

Re: [Twinsen] [0.18.3] [Modding] NestedTriggerEffectItem tooltip ignores repeat count in a tooltip

Post by lovely_santa »

Thanks! I'll make a report if I find any mistakes!
You can find all my mods on the mod portal. Also helping on Arch666Angel's mods.
Image
Post Reply

Return to “Resolved Problems and Bugs”