Complete.Org: Mailing Lists: Archives: freeciv-dev: March 2004:
[Freeciv-Dev] Re: (PR#7895) voting doesn't work with commands with argum
Home

[Freeciv-Dev] Re: (PR#7895) voting doesn't work with commands with argum

[Top] [All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index] [Thread Index]
To: use_less@xxxxxxxxxxx
Subject: [Freeciv-Dev] Re: (PR#7895) voting doesn't work with commands with arguments
From: "Per I. Mathisen" <per@xxxxxxxxxxx>
Date: Tue, 16 Mar 2004 06:47:21 -0800
Reply-to: rt@xxxxxxxxxxx

<URL: http://rt.freeciv.org/Ticket/Display.html?id=7895 >

On Sat, 13 Mar 2004, James Canete wrote:
> Seems "A little work here and there" consisted of changing caller to
> NULL. :)

Actually, the proper fix is a little bit more complicated:

--- server/stdinhand.c  16 Mar 2004 14:42:40 -0000      1.309
+++ server/stdinhand.c  16 Mar 2004 14:45:42 -0000
@@ -3319,7 +3319,7 @@
              _("Ambiguous option name."));
     return FALSE;
   }
-  if (!may_set_option(caller,cmd)) {
+  if (!may_set_option(caller,cmd) && !check) {
      cmd_reply(CMD_SET, caller, C_FAIL,
               _("You are not allowed to set this option."));
     return FALSE;

;-)

The problem was only with the /set command, AFAICT.

  - Per




[Prev in Thread] Current Thread [Next in Thread]