Menu

#4667 41.11, problem with assigning bombs to ASFs in the lobby

stable 0.42
closed
None
fixed
1
2015-12-25
2015-12-21
No

In the lobby, add a 100 ton ASF which can carry up to 20 bombs. Then click the drop down list for cluster bomb, and select "2". After you do this, the drop down list will only allow you to select up to 18 cluster bombs. The problem is that if you select 18 cluster bombs, this only adds 18 cluster bombs to the ASF in total, instead of adding 18 more cluster bombs.

Basically, if you want to add 20 bombs to a 100 ton ASF, you can only do it when the ASF has 0 bombs, or else the drop down list bugs out and prevents you from selecting 20 cluster bombs.

1 Attachments

Discussion

  • Magnus Kerensky

    Magnus Kerensky - 2015-12-21

    I tried to add 18 Torpedo Bombs each to a pair of 90 ton fighters in order to test them out, and after deployment neither fighter was equipped with any bombs. I'm not sure whether that is related to this topic or not, but it seems like it.

     
    • Nicholas Walczak

      Looks like torpedo bombs aren't implemented. If you look in the system tab, you'll see that the bombs get added, but the attacks don't (bombs consist of the bomb ammo, and the BombAttack, it's kind of confusing if you're not familiar with the code, and then maybe it's still confusing)

       
  • Nicholas Walczak

    • assigned_to: Nicholas Walczak
    • Resolution: none --> fixed
    • Milestone: undetermined --> stable 0.42
     
  • Nicholas Walczak

    Fixed in [17ab7b]. Basically, the maximum number of bombs that can be selected in any drop-down is the maximum number of bombs remaining. This is problematic for exactly the reason mentioned. What it should be is maximum number of bombs available + number currently selected. That's what I did to fix this.

    This is purely a UI issue, with how many bombs of a particular type the UI will allow you to select. Anything else is a separate issue.

     

    Related

    Commit: [17ab7b]

  • Hammer

    Hammer - 2015-12-25
    • Status: open --> closed
     

Log in to post a comment.

MongoDB Logo MongoDB