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

Bug Report: equipment_by_ship.oxz #370

Closed
MosayIC opened this issue Oct 9, 2020 · 7 comments
Closed

Bug Report: equipment_by_ship.oxz #370

MosayIC opened this issue Oct 9, 2020 · 7 comments

Comments

@MosayIC
Copy link

MosayIC commented Oct 9, 2020

This oxp was causing my game to crash.

When I filtered between folders, this one crashed my game every time.

System: windows 10

@AnotherCommander
Copy link
Member

Can you please post the Latest.log following a crash? Thanks.

@MosayIC
Copy link
Author

MosayIC commented Oct 9, 2020

I don't know how do do that, but if you'll tell me I can recreate the problem.

@AnotherCommander
Copy link
Member

AnotherCommander commented Oct 9, 2020

Get the game to crash, then look for the file Latest.log inside [Your Oolite Install Folder]\oolite,app\Logs, and then post its contents here.

@MosayIC
Copy link
Author

MosayIC commented Oct 9, 2020

Latest.log

13:53:28.019 [startup.exception]: ***** Unhandled exception during startup: NSInvalidArgumentException (Tried to add nil value for key 'EQ_WEAPON_TWIN_PLASMA_CANNON' to dictionary).

@AnotherCommander
Copy link
Member

Confirmed. 1.88 does not throw an exception, but 1.90 does. Until this is fixed properly, the quick fix is to open the OXP's Config/equipment.plist, find this:

/* next item was a test item */
(
2, 2000, "Twin Plasma Cannon",
"EQ_WEAPON_TWIN_PLASMA_CANNON",
"Basic cannon delivering charged plasma bursts at sublight speed."
),

and change it to this:

/* next item was a test item */
(
2, 2000, "Twin Plasma Cannon",
"EQ_WEAPON_TWIN_PLASMA_CANNON",
"Basic cannon delivering charged plasma bursts at sublight speed.",{} // this is ths changed line
),

@MosayIC
Copy link
Author

MosayIC commented Oct 10, 2020

Done and done. It works now. Thanks.

@MosayIC MosayIC closed this as completed Oct 10, 2020
@AnotherCommander
Copy link
Member

Just a note to say that the bug should now be properly fixed in commit 6b45ec2. The equipment_by_ship_class OXP and any other OXPs that contain equipment definitions without an extra info dictionary should no longer cause an exception.

A Windows 64 bit fixed game executable can be downloaded from here.
(VirusTotal report here).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants