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

Simplify cargo pods #13

Open
JensAyton opened this issue May 17, 2011 · 0 comments
Open

Simplify cargo pods #13

JensAyton opened this issue May 17, 2011 · 0 comments

Comments

@JensAyton
Copy link
Member

Cargo pods have an internal cargoType property, which can be RANDOM, SLAVES, ALLOY, MINERALS, THARGOID or SCRIPTED_ITEM. RANDOM actually contains a fixed item determined by commodityType and commodityAmount. The other values, except SCRIPTED_ITEM, each represent one unit of a fixed commodity.

This reflects the cargo information stored in shipdata.plist for ships, but representing it this way for cargo pods seems entirely pointless. Pods should either be scripted or contain some quantity of a commodity. This information should be exposed to scripts. Scripted pods should also be able to set nominal contents, either as a string or a commodity with the extra option of none/null. (A script that wishes to display pod contents should represent null as “Unknown” or similar.)

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

1 participant