Page 1 of 1

Improved /revert command

Posted: Sat Dec 12, 2015 12:50 am UTC
by cu-kai

New EDGE recently ran a 1.1 server which used cqvm (a serverside qvm with some enhanced cvars and admin system / commands) and I would like to make a request regarding it here.

The current revert command is great and it has worked for years, but for a first time user (which many prospective unvanquished server hosts will be) it would be rather confusing and slow to use. For example, they have to /buildlog, find the ID of where they want to buildlog, then /revert that specific ID. The /revert command also reverts both teams' buildings, which may not be what they would like to do.

The /revert command from 1.1 has the option for the current method of reverting (by buildlog id) but admins are also able to /revert the last x actions to happen to team y.

Here is the adminhelp entry for the command on our server:

!help: help for '!revert':
Function: revert one or more buildlog events, optionally of only one team
Syntax: !revert (xnum) (#ID) (-name|num) (a|h)
Example: '!revert x5 h' reverts the last 5 events affecting human buildables
Flag: 'revert'

Here is the code: https://github.com/paradox460/cqvm/blob ... in.c#L7606


Re: Improved /revert command

Posted: Sat Dec 19, 2015 10:58 pm UTC
by Ishq

The 1.1 revert command, while more flexible, had much more potential to shoot yourself in the foot. There were a ton of unhandled edge cases that could case what you wanted to revert not be reverted. The current command is meant to be simpler and be less buggy. While, we want more flexibility, the amount of effort required to implement revert with all those bugs fixed is hardly worth it. If someone were to make a match, we'd take it, but I don't think we'd focus our effort there yet.