[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-13 Thread pepeto

Follow-up Comment #18, patch #1910 (project freeciv):

I tried to work on /mute. There is a deep problem with its concept. It is not
clear at all if it would affect a single user or all users based on the ip. As
long I don't have so much time, I give up. Only /kick will go into 2.3.0.


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-10 Thread pekka

Follow-up Comment #17, patch #1910 (project freeciv):

I think /kick will be best feature to have. 
Currently, if we want to allow a person to observe or not, it is necessary to
set allowtake. Having a /kick command will address both unwanted observing and
spamming. And it will save us for all this hassle changing allowtake for each
person that log on. I vote for a /kick command allowing us to kick based on
IP adress.

Pekka

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread anonymous

Follow-up Comment #7, patch #1910 (project freeciv):

Now that it seems ready to be deployed within next version, we've been
talking about a new command, that would protect every user from people who
insult, that would be a command issued only by the server dictator:

/mute conn

and all chat from conn would go to /dev/null

The purpose of that is to protect users which do not know /ignore command
from annoying people.
ks.

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread anonymous

Follow-up Comment #8, patch #1910 (project freeciv):

I vote for this. 
There is some more reasons for a /mute command too:

First, if players suspect an observer is giving away info to other players,
destroying the fun of the game, they can cast a vote to mute that observer
for all players.

Second, new players usually don't read messages all time. So when they
finally do, in the case of some players, they will be met by countless
messages of poor quality. 

For this reason, it would be nice if there was a /mute that silenced an
observer for both democratic (/cmdlevel basic) and games with a dictator
(/cmdlevel first).

Pekka

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread Alberto Bonsanto

Follow-up Comment #9, patch #1910 (project freeciv):

I agree totally, and maybe add too a filter or something like in warclient
where observers can't send coordinates. 


Thanks pepeto! :D
Master-Noobie

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread anonymous

Follow-up Comment #10, patch #1910 (project freeciv):

both /mute and /ignore features are just necessity when playing a game with
chat. I would gladly finally find those also in freeciv. Good work guys!

KG

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread pepeto

Follow-up Comment #11, patch #1910 (project freeciv):

Unfortunately, S2_3 is in a pre-freezing state. I doubt I will have time to
get it in. What do you think it is more urgent between /kick and /mute?


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread Alberto Bonsanto

Follow-up Comment #12, patch #1910 (project freeciv):

Personally i think /mute is better, but /mute can be for more than only 3
turns, because maybe the evil guy can wait until his mute ends and give the
info to the other player or spam like crazy again.

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread pekka

Follow-up Comment #13, patch #1910 (project freeciv):

What will be the difference between /cut and /kick. Will the /kick block user
from reconnect? The problem with them all, /mute /ignore /cut is that a
observer can just reconnect. And if the command is tied to login-name, they
will reconnect using a new name. 
If we really want to stop spamming and bad game-etiquette there should be a
/block that block login from a specified IP adress until the block is lifted
or game is over. 

Pekka


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread pepeto

Follow-up Comment #14, patch #1910 (project freeciv):

* /cut is based in the username.
* /ignore is based on username, hostname or ip (you choose).
* /mute is based on ip. Reconnecting doesn't help, even with a new name.
* /kick is based on username, hostname and ip. You cannot reconnect with the
same name, neither from this address.

Your /block command is just a variant of /kick.


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread Alberto Bonsanto

Follow-up Comment #15, patch #1910 (project freeciv):

please don't forget about the coordinates (links) filter, because observers
may cheat and give info to the enemy.

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-11-09 Thread pepeto

Follow-up Comment #16, patch #1910 (project freeciv):

Please, raise a new bug item for this.


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-12 Thread pepeto

Update of patch #1910 (project freeciv):

  Status: In Progress = Done   
 Open/Closed:Open = Closed 


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-10 Thread pepeto

Follow-up Comment #5, patch #1910 (project freeciv):

Taking in account your comment...


(file #10297)
___

Additional Item Attachment:

File name: trunk_ignore_command2.diff Size:23 KB


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-08 Thread pepeto

Follow-up Comment #3, patch #1910 (project freeciv):

Patch attached including the following commands (like in warclient):
* /ignore [type=]pattern
* /unignore range
* /list ignore

As I made important string changes and I am not a native-English (and some
string already appear strange to me), I wish someone will overview the
strings. Jacob?


(file #10249)
___

Additional Item Attachment:

File name: trunk_ignore_command.diff  Size:23 KB


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-08 Thread David Lowe

Follow-up Comment #4, patch #1910 (project freeciv):

The grammar looks good to me.  I just wish line # 10 were a little more
specific:

- list ignore\n
+ list ignored users\n

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-09-04 Thread pepeto

Update of patch #1910 (project freeciv):

  Depends on: = patch #1920


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-08-31 Thread Keyser Soze

URL:
  http://gna.org/patch/?1910

 Summary: Command /ignore for server
 Project: Freeciv
Submitted by: keysersoze
Submitted on: Tuesday 08/31/2010 at 13:58
Category: general
Priority: 5 - Normal
  Status: None
 Privacy: Public
 Assigned to: None
Originator Email: 
 Open/Closed: Open
 Discussion Lock: Any
 Planned Release: 

___

Details:

Some users on-line are annoying others while playing. That is bad for
freeciv, cause it keeps away new users. So I implemented a /ignore command to
be used by any player whenever he wants. The second time the command is
issued, it stops ignoring.
Syntax: 
/ignore connection
/ignore alone shows ignore list.





___

File Attachments:


---
Date: Tuesday 08/31/2010 at 13:58  Name: meu_patch  Size: 5kB   By:
keysersoze

http://gna.org/patch/download.php?file_id=10124

___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-08-31 Thread anonymous

Additional Item Attachment, patch #1910 (project freeciv):

File name: meu_patch_2Size:6 KB


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-08-31 Thread pepeto

Update of patch #1910 (project freeciv):

  Status:None = In Progress
 Assigned to:None = pepeto 
 Planned Release: = 2.3.0  

___

Follow-up Comment #1:

As I used to play regularly online, I know this is a useful feature. However,
the patch is not really good yet, I am thinking about something more like
warclient.

What is wrong:
* When connection is lost, behaviour is not granted.
* If you pass an invalid username to the command, it will print the list of
the connections you are ignoring, instead of print warning.
* You cannot modify *_gen.[ch] files because they are auto-generated.
* Indentation is not correct in regard to CodingStyle documentation.
* Using // C++ style comment is not allowed (see CodingStyle documentation).
Anyway, your username is not useful for the code. :)

What can be improved:
* Ignore list could use the connection patten like in warclient. Most tools
already exist in trunk/ to allow this.


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message posté via/par Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev


[Freeciv-Dev] [patch #1910] Command /ignore for server

2010-08-31 Thread anonymous

Follow-up Comment #2, patch #1910 (project freeciv):

I appreciate your comments and I'm glad you took charge of it.
By the time present, I'm not able to work these things out.
My patch was, in fact, only a prototype.
We're looking forward to see this command on next version of game.
Thanks.
KS


___

Reply to this item at:

  http://gna.org/patch/?1910

___
  Message sent via/by Gna!
  http://gna.org/


___
Freeciv-dev mailing list
Freeciv-dev@gna.org
https://mail.gna.org/listinfo/freeciv-dev