search

SEModCommunity / SE-Community-Mod-API

Space Engineers Community Modding API
GNU Lesser General Public License v3.0
60 stars 47 forks source link

Suggestion: ChatCommand "/delete" shall not delete attached grids #126

Open Spcemarine opened 10 years ago

Spcemarine commented 10 years ago

When I perform the command "/delete [all, station, ship] nobeacon" all grids without a beacon are deleted. So far not a bug but I would like to suggest to spare the grids that are attached on pistons and rotors if the "main grid" has a beacon. We would like to delete some trash but if we do it stuff like hangar gates will also disappear. BTW: You are doing great stuff!

mikeloeven commented 10 years ago

yes seconded i was cleaning my server and deleted a few unknown grids only to find out that they were the pistons on my drilling rig XD not fun to replace. though this is more keens fault for not giving sub-grids better identification and i am not sure if the script can even tell what is and is not attached

Spcemarine commented 10 years ago

Well, I browsed through the code and couldn't find a way to detect whether a grid is attached or not, otherwise I would do it on my own. Since the API is not finished yet it may be possible for the creator to find a way. I don't think it's anyone's fault because keen is not involved in the developement of this API and the API is still in developement.

chessmaster42 commented 10 years ago

Yeah this is doable but it's no easy task. I'm slowly getting the bits of the API built out to support rotors and pistons and once that's done those blocks will have a property that indicates any attached cube grids.

Until that point it's not possible to do what you're asking but it is something we're working on.

Spcemarine commented 10 years ago

Ok, very well. So it's like I thought. Will you change the "/delete" command when you finished it or am I supposed to implement my own commands?

chessmaster42 commented 10 years ago

Yes, the default command will be updated once implementation of all of MergeBlockEntity, PistonEntity, and RotorEntity is complete. Right now the first two are finished but RotorEntity is still being worked on. So changes are good that you will see the change in the next dev build or two.

Spcemarine commented 10 years ago

That's great! Thank you very much.

fvgoulet commented 9 years ago

@Spcemarine Is this issue fixed?