SpaceLocationPrototype::draw_orbit - API documentation doesn't match

plexpt
Long Handed Inserter
Long Handed Inserter
Posts: 96
Joined: Mon Feb 07, 2022 2:32 am
Contact:

SpaceLocationPrototype::draw_orbit - API documentation doesn't match

Post by plexpt »

https://lua-api.factorio.com/latest/pro ... draw_orbit

please update this
People would think it for a ring of satellite orbits around a planet.
but in fact it is---
"It does not draw the orbit around the Sun if false."
Xorimuth
Filter Inserter
Filter Inserter
Posts: 731
Joined: Sat Mar 02, 2019 9:39 pm
Contact:

Re: API documentation doesn't match

Post by Xorimuth »

Agreed, when I first used it I expected it to remove the ring around the planet, rather than around the sun.
My mods
Content: Lunar Landings | Freight Forwarding | Spidertron Patrols | Spidertron Enhancements | Power Overload
QoL: Factory Search | Module Inserter Simplified | Wire Shortcuts X | Ghost Warnings
User avatar
BraveCaperCat
Filter Inserter
Filter Inserter
Posts: 443
Joined: Mon Jan 15, 2024 10:10 pm
Contact:

Re: SpaceLocationPrototype::draw_orbit - API documentation doesn't match

Post by BraveCaperCat »

Luckily I had my friend (who made this topic) to help describe the property properly. Unfortunately, this issue wouldn't matter for me even if my friend didn't describe it as draw_orbit being a boolean and there being no property to change the parent object would be enough to push me in the right direction towards it's correct use.
Creator of multiple mods, including Quality Assurance - My most popular one.
Go check them out with the first and second links!
I'll probably be wanting or giving help with modding most of the time I spend here on the forum.
Bilka
Factorio Staff
Factorio Staff
Posts: 3511
Joined: Sat Aug 13, 2016 9:20 am
Contact:

Re: SpaceLocationPrototype::draw_orbit - API documentation doesn't match

Post by Bilka »

Thanks for the note, fixed for 2.0.64.
I'm an admin over at https://wiki.factorio.com. Feel free to contact me if there's anything wrong (or right) with it.
Post Reply

Return to “Resolved Requests”