Artillery on dropships needs "Ground Dropships Use Individual Weapons" option

Status: Beta

Brought to you by: arlith, beerockxs, bombaijin, hammer-gs

This project can now be found here.

#3766 Artillery on dropships needs "Ground Dropships Use Individual Weapons" option

Milestone: stable 0.42

Status: closed

Owner: Hammer

Labels: artillery (10) bay weapon (1)

Resolution: accepted

Priority: 1

Updated: 2016-11-26

Created: 2013-06-14

Creator: Dylan Myers

Private: No Discussion Disabled

When fixing another bug I discovered that ArtilleryBayWeapons wont work while on ground map. It detected as being always out of range.

Related

Bugs: ~~#3829~~
Bugs: ~~#4071~~
Bugs: ~~#4308~~
Bugs: ~~#4482~~

Discussion

Dylan Myers - 2013-06-14

WIP: I've partially fixed this by fixing the missing ranges. However, it now NPEs due to a bad getLinked() check. I've added a FIXME to the code since I'm not sure how to fix this just yet because I don't know if more than a single autocannon type can be in each bay.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dylan Myers - 2013-06-14

Ok, so I fixed that, but this continues to be a WIP because there are NPEs in the ArtilleryBay<direct indirect=""><homing normal="">Handlers.</homing></direct>

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Sebastian Brocks - 2013-06-24

Resolution: --> accepted
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Taharqa - 2013-07-22

The problem is that there are no rules for using artillery as a bay weapon. Does it work if the unofficial option to use dropship weapons individually when on the ground is checked? If so, then I think we should just disable any functionality for artillery bay weapons unless the user has that checked, for sanity. There are just too many rules holes to try to fill with the whole bay weapons on grounded dropships stuff.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
- Dylan Myers - 2013-07-22
  
  Works fine with individual weapons. I just left this open in case I (or anyone else) could think of a sane way to fix the bayweapon version and felt like doing it.
  
  If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
  - Nicholas Walczak - 2013-07-23
    
    Should we mark this as post stable?
    
    If you would like to refer to this comment somewhere else in this project, copy and paste the following link:
    - Dylan Myers - 2013-07-23
      
      Good idea, and done.
      
      If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dylan Myers - 2013-07-23

Milestone: stable 0.36 --> post-stable 0.36
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Sebastian Brocks - 2013-08-27

summary: ArtilleryBayWeapon not working on ground map (always out of range) --> artillerry on dropships needs "Ground Dropships Use Individual Weapons" option
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nicholas Walczak - 2014-07-06

Milestone: stable 0.38 --> stable 0.4
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dylan Myers - 2014-12-18

summary: artillerry on dropships needs "Ground Dropships Use Individual Weapons" option --> Artillery on dropships needs "Ground Dropships Use Individual Weapons" option

Milestone: stable 0.40 --> post-stable 0.40
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nicholas Walczak - 2015-01-04

Milestone: post-stable 0.40 --> stable 0.40
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nicholas Walczak - 2015-01-04

Milestone: stable 0.40 --> stable 0.42
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Dylan Myers - 2015-03-08

Duplicated by [#4482]

Related

Bugs: ~~#4482~~

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Nicholas Walczak - 2016-08-15

There's a duplicate issue for this reported on GitHub: https://github.com/MegaMek/megamek/issues/205

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Pheonix Rising - 2016-10-05

Since there is a newer ticket for this on GitHub this should be closed and the GH ticket reopened.

If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Hammer - 2016-11-26

status: open --> closed

assigned_to: Dylan Myers --> Hammer

discussion: enabled --> disabled
If you would like to refer to this comment somewhere else in this project, copy and paste the following link:

Hammer - 2016-11-26

This bug is also open on GitHub here. In light of some recent information closing this and using the ticket on the Github tracker.

https://github.com/MegaMek/megamek/issues/205

If you would like to refer to this comment somewhere else in this project, copy and paste the following link: