0006350: NPZ mounting and dismounting script error

View Issue Details [ Jump to Notes ]

[ Issue History ] [ Print ]

Project

Category

View Status

Date Submitted

Last Update

0006350

AFRF

[All Projects] General

public

2021-05-13 12:01

2021-05-17 11:53

Reporter

johnb43

Assigned To

reyhard

Priority

none

Severity

minor

Reproducibility

always

Status

assigned

Resolution

open

Platform

OS Version

Product Version

Target Version

Fixed in Version

Summary

0006350: NPZ mounting and dismounting script error

Description

NPZ dismounting:

11:51:19 Error in expression <wf;
removeAllPrimaryWeaponItems _p;
_p addMagazine [(primaryWeaponMagazine _p) >
11:51:19 Error position: <addMagazine [(primaryWeaponMagazine _p) >
11:51:19 Error Type Any, expected String

NPZ mounting:

11:54:20 Error in expression <n1;
removeAllPrimaryWeaponItems _p;
_p addMagazine [(primaryWeaponMagazine _p) >
11:54:20 Error position: <addMagazine [(primaryWeaponMagazine _p) >
11:54:20 Error Type Any, expected String
11:54:20 File rhsafrf\addons\rhs_c_weapons\scripts\rhs_npz.sqf..., line 24

Steps To Reproduce

1. Go into the virtual arsenal, give yourself any weapon that can dismount the NPZ.

2. Remove all compatible magazines on your person (the one in the gun can be either present or gone, same error is thrown).

3. Remove NPZ with keybind.

4. Error pops up.

Additional Information

From what I can gather, it never checks whether "primaryWeaponMagazine player" is not [].

On a further not, it does not take underslung grenades into account. As far as I'm aware, there are no weapons with working folding stocks AND UGLs, but it's an issue with RHS GREF (I will make a seperate bug report for that).

I have made a fix for all of these issues listed, feel free to ask for it and I will share it. I would need to slightly tweak though, as at the moment it uses 1 CBA function.

Issue History
Date Modified	Username	Field	Change
2021-05-13 12:01	johnb43	New Issue
2021-05-17 11:53	reyhard	Assigned To	=> reyhard
2021-05-17 11:53	reyhard	Status	new => assigned

Relationships

Notes
There are no notes attached to this issue.