Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

T2 XL Torp/Cruise variants not displaying explosion radius and explosion velocity #1325

Closed
jacobmatthewjansen opened this issue Oct 27, 2017 · 1 comment
Labels
bug Confirmed to be a bug fixed This issue has been fixed! Oh joy!

Comments

@jacobmatthewjansen
Copy link

Bug Report

Expected behavior:

Explosion radius and explosion velocity displays in column on fitting window when Precision/Fury/Rage/Javelin ammo variants are loaded into XL Cruise Missile/Torpedo Launcher II's

Actual behavior:

The area in the fitting window that usually displays this info is blank

Detailed steps to reproduce:

Load T2 Ammo into a T2 XL launcher on a Phoenix

Fits involved in EFT format (Edit > To Clipboard > EFT):

[Phoenix, Blap]

[Empty Low slot]
[Empty Low slot]
[Empty Low slot]
[Empty Low slot]
[Empty Low slot]

[Empty Med slot]
[Empty Med slot]
[Empty Med slot]
[Empty Med slot]
[Empty Med slot]
[Empty Med slot]
[Empty Med slot]

XL Cruise Missile Launcher II, Scourge Precision XL Cruise Missile
XL Torpedo Launcher II, Nova Rage XL Torpedo
XL Torpedo Launcher II, Guristas Nova XL Torpedo
Siege Module II
[Empty High slot]

[Empty Rig slot]
[Empty Rig slot]
[Empty Rig slot]

Release or development git branch? Please note the release version or commit hash:

https://i.imgur.com/VCmZHUV.png

Operating system and version (eg: Windows 10, OS X 10.9, OS X 10.11, Ubuntu 16.10):

Windows 10

Other relevant information:

https://i.imgur.com/aJfdFTh.png

@blitzmann
Copy link
Collaborator

Thanks for this, looks like we weren't handling Advanced groups for XL stuff. Fixed with efea460

@blitzmann blitzmann added bug Confirmed to be a bug fixed This issue has been fixed! Oh joy! labels Oct 29, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Confirmed to be a bug fixed This issue has been fixed! Oh joy!
Projects
None yet
Development

No branches or pull requests

2 participants