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.

Sign up for GitHub

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

Jump to bottom

Containers variable return null object #601

Open
Jimz71 opened this issue Jan 7, 2020 · 4 comments · May be fixed by #652
Open

Containers variable return null object #601

Jimz71 opened this issue Jan 7, 2020 · 4 comments · May be fixed by #652

Comments

@Jimz71
Copy link

Jimz71 commented Jan 7, 2020 •
edited
Loading

Expected behaviour

Containers check should not count null object.

Actual behaviour

If player place container and relog containers var will return some null object until reboot, so the count of containers will be false .

Steps to reproduce the behaviour

  1. (Add,Delete) Container.
  2. Disconnect.
  3. Reconnect.
  4. Check the containers vars : return null object until reboot.

RPT and/or extDB3 logs (if applicable)

N/A

Mission version: 5.0

Launcher version: 1.5

Game version: 1.96

Branch: main / beta branch not specified
@BoGuu
Copy link
Member

BoGuu commented Jan 7, 2020

Seems you've omitted important parts of the issue template, could you please correct that?

@DomT602
Copy link
Member

DomT602 commented Apr 29, 2020

@Jimz71 Hi, sorry for the delay in responding, can I just clarify, are you saying that the variable "containers" which is set on the house is returning objNull, or is it an array containing objNull's? Also is this after adding then removing a container?

This was referenced Apr 30, 2020
Open
Merged
@DomT602
Copy link
Member

DomT602 commented Apr 30, 2020

Hi @Jimz71, above you can see 2 PR's to resolve this issue. #652 for the 5.x.x branch, and #653 for master branch. Please could you confirm if this resolves your issue?

@ghost
Copy link

ghost commented May 21, 2020

As already noted: #653 fixes the dup-bug for current master branch. I didn't checkecd #652 for 5.x.x branch.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Container bugfix DomT602/Framework
3 participants
@BoGuu @Jimz71 @DomT602