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

Deep copy/paste does not retain ACE Traits #645

Open
YetheSamartaka opened this issue Oct 18, 2021 · 2 comments
Open

Deep copy/paste does not retain ACE Traits #645

YetheSamartaka opened this issue Oct 18, 2021 · 2 comments
Labels
bug Something isn't working

Comments

@YetheSamartaka
Copy link

Arma 3 Version: 2.06.148221
CBA Version: 3.15.6.211004
ZEN Version: 1.12.0

Mods:

- CBA_A3
- ace
- Zeus Enhanced
- Zeus Enhanced - ACE3 Compatibility

Description:
Deep copy/paste does not retain ACE Traits (Such as Medical Training, Engineering Skill and Explosive Specialist).

Expected behavior:
ACE Traits are also copied and applied on new unit.

Steps to reproduce:

  1. Have an unit with some ACE Traits
  2. Deep copy and paste said unit
  3. Inspect copied unit and see that ACE Traits are set to None or No.

Where did the issue occur?

  • Happens in all environments

Additional information:

  • None

RPT log file:

  • None

Screenshots:
Original unit:
Original Unit
Deep pasted unit(Which does not have the same ACE Traits as it should):
Deep pasted unit

@YetheSamartaka YetheSamartaka added the bug Something isn't working label Oct 18, 2021
@Kexanone
Copy link
Member

The traits are stored as variables in the unit's namespace. Question is whether we want to serialize them manually or if we go for a more general approach and just serialize all variable in the unit's namespace.

@YetheSamartaka
Copy link
Author

The traits are stored as variables in the unit's namespace. Question is whether we want to serialize them manually or if we go for a more general approach and just serialize all variable in the unit's namespace.

All variables could be dangerous. I'd go for a more focused approach.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants