Chemical - Fix mortar issues

KAT-Advanced-Medical / KAM

The ACE Advanced Medical System is nice, but we can do it better, even on a more realistic way and bring it on another level with these addons.

https://katalam.github.io/KAM

GNU General Public License v3.0

66 stars 100 forks source link

Chemical - Fix mortar issues #458

Closed MiszczuZPolski closed 6 months ago

MiszczuZPolski commented 10 months ago

When merged this pull request will:

Fix wrong inheritance
Fix "No matching carry mag" that was blocking unloading 8rnd and 5rnd ammo from mortar

IMPORTANT

Development Guidelines are read, understood and applied.
Title of this PR uses our standard template Component - Add|Fix|Improve|Change|Make|Remove {changes}.

diaverso commented 9 months ago

This not fix https://i.imgur.com/ZuvJSei.jpeg

MiszczuZPolski commented 9 months ago

This not fix https://i.imgur.com/ZuvJSei.jpeg

Yes, i know there are still some issues but i dont have time to search for solutions.

nobodyshome commented 3 months ago

Somehow this change has broken/effected the L16 static weapon from 3CB weapons

Removed recently updated mods one by one till it worked as intended and it worked after disabling KAT.

When working, info is described as the "L16 Static Weapon" with "Charge 0" and "81mm HE/Smoke", the charges go up to "Charge 6"

with KAT v2.15.0, it reverts the mortar to just "82mm Mortar" and has "Semi (close)" "Semi (medium)" and "Semi (far)". Believe whatever changes were made regarding this issue may be the cause. Can test and help debug if required.

MiszczuZPolski commented 3 months ago

Please contact on discord and create bug thread on it, so its easier to discuss