URL:
<http://gna.org/bugs/?19885>
Summary: Sort out 'check' behaviour for playercolor/mapimg
Project: Freeciv
Submitted by: jtn
Submitted on: Sun Jul 1 11:35:03 2012
Category: None
Severity: 2 - Minor
Priority: 5 - Normal
Status: In Progress
Assigned to: jtn
Originator Email:
Open/Closed: Open
Release:
Discussion Lock: Any
Operating System: Any
Planned Release: 2.4.0,2.5.0
_______________________________________________________
Details:
Somehow I got it into my head that server commands should be silent when the
'check' argument is specified. Now I remind myself what it's for -- it's used
when deciding whether a vote is worth starting -- that's actually
counterproductive; it's useful to tell the person who submitted the command
why it didn't get even as far as a vote.
So far I've perpetrated this misapprehension on 'playercolor'. While reviewing
for other cases I found 'mapimg' had some bugs in the 'check' case; I dunno
whether I introduced them, but I may as well fix them.
In these cases it's academic, as these are currently ALLOW_ADMIN commands and
voting is only triggered on ALLOW_CTRL commands. So this fix is more about
future-proofing and not leaving incorrect patterns to propagate.
_______________________________________________________
Reply to this item at:
<http://gna.org/bugs/?19885>
_______________________________________________
Message sent via/by Gna!
http://gna.org/
_______________________________________________
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev
