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

Fixes list for the vehicle computer. [NO GAMEPLAY EFFECT] #4902

Closed

Conversation

Waseemq1235
Copy link
Contributor

@Waseemq1235 Waseemq1235 commented Nov 11, 2023

Apparentally the vehicle ordering computer wouldn't work even if one attempted to set spent = TRUE. This was due to the datums in the vehicles list not properly set up.

Explain why it's good for the game

Helps staff spawn vehicles through the consoles, the OG way, instead of spawning stuff individually.

Testing Photographs and Procedure

Screenshots & Videos

Put screenshots and videos here with an empty line between the screenshots and the <details> tags.

Changelog

🆑
admin: Fixes vehicle list for the vehicle ASRS computer.
/:cl:

@github-actions github-actions bot added the Admin Make things harder for admins label Nov 11, 2023
@fira
Copy link
Member

fira commented Nov 12, 2023

I think you're missing the parenthesis ()

@Waseemq1235
Copy link
Contributor Author

I'm stupid. That's fixed now.

Copy link
Contributor

@Birdtalon Birdtalon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can't new them twice.

Comment on lines +1334 to +1336
new /datum/vehicle_order/apc(),
new /datum/vehicle_order/apc/med(),
new /datum/vehicle_order/apc/cmd(),
Copy link
Contributor

@Birdtalon Birdtalon Nov 17, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

If you instantiate these here it can't also be done again below as this causes runtimes on mapload.

for(var/order as anything in vehicles)
new order

@harryob
Copy link
Member

harryob commented Nov 17, 2023

exactly what birdtalon said, but also, fira's pr (which was made after this one, my bad) also fixes this and was already merged

@harryob harryob closed this Nov 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Admin Make things harder for admins
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants