Improved /revert command

Request new features or present your ideas.
Post Reply
User avatar
cu-kai
Web Crew
Posts: 63
Joined: Fri Jan 02, 2015 1:29 am UTC
Clan: CU

Improved /revert command

Post 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

Web Raccoon

User avatar
Ishq
Project Head
Posts: 1145
Joined: Tue Mar 06, 2012 8:32 pm UTC

Re: Improved /revert command

Post 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.

Post Reply