quackduck/devzat

How to delete a room

gansheer opened this issue · 7 comments

Is there a command that allows to delete a room (we tried rm #room-name and other equivalent without any success) ?

that wouldn't be too hard to add to clean up extra rooms. i'd probably want it to be admin-use only. thoughts @Arkaeriit?

Naively I through the rm command would do something since it displays some help text, but with this code there is no chance 😄 :

devzat/commands.go

Lines 684 to 691 in 3d2d744

func rmCMD(line string, u *User) {
if line == "" {
u.room.broadcast("", `usage: rm [-f | -i] [-dPRrvW] file ...
unlink file`)
} else {
u.room.broadcast("", "rm: "+line+": Permission denied, sucker")
}
}

And having the feature only for admin would be ideal IMHO.

that wouldn't be too hard to add to clean up extra rooms. i'd probably want it to be admin-use only. thoughts @Arkaeriit?

A command to delete empty room would be a good idea. I don't think it would need need admin access as deleting empty rooms is harmless.

Should that command be rm or rmdir ?

A command to delete empty room would be a good idea. I don't think it would need need admin access as deleting empty rooms is harmless.

Out of curiousity, what is constitute a room as empty ? A room without any users using it at the time of the deletion ?

Yes, can't see what else it could be tbh

We can probably close this now that #226 has been merged.