Modified Blacklist

[Links]

  Links
     Browse SVN
     SVN Commit log
     Documentation (Wiki)
  Developers
     Taskmanager

Advanced search

[jjones666]

based on the standard !blacklist plugin (coded by Foxferal).

improved over original (imo):

- removed code to check if user exists via .xml download and replaced with aoc -> get_uid (burn in hell crappy FC servers).
- if blacklist is empty, it will tell you that with no blob.
- shows blacklist blob when adding and deleting people (if applicable).
- make it impossible to add duplicate entries.
- output also shows !whois clicky for each name (combined with !postit feature, this is very useful).
- changed rem to del, in line with most other plugins.
- colour scheme matches most other plugins.
- updated helpfiles.

http://www.jjones.co.uk/files/blacklist.php
http://www.jjones.co.uk/files/blacklist.txt

-jj-

[deathentry]

this is perfect only thing i'd like to do is add a reason for the ban too

[Naturalistic]

I was fooling around with another copy, and figured I might as well post it. Found it to be pretty useful.

Could mod mine to implement the few extra changes from Jack's.

PS: This one includes reasons, but not modified from Jack's (modded from the original)

[Dracutza]

Love both of these, so i am trying to combine the two.  Hoping someone here can give me a hand.

Using jjones as the base, and just adding the "added by" and "reason" from Naturalistic.

Everything is set up and converted correctly for the most part. I'm to a point where if i do !blacklist add player1 is properly entered into the db.  !blacklist shows player1 [whois] added by Dracutza Reason:

if i type !blacklist add player1 test, where test would be the reason, I get Player player1 test does not exist.

however... i can edit the db directly, entering a reason, and the reason displays as expected.

Unfortunately, the Sets new name in blacklist section (roughly starting on line 95) in both mods is completely different.  Copying the INSERT INTO line from Naturalistics works fine, except for the reason value.  It's obviously related to how each reads the add command and how the name to be blacklisted ($msg???) is created, but i know zero php, and several things I tried have had no positive effect.

Can maybe one of you tell me what i need to be looking at to get this to accept the reason?

[Naturalistic]

Issue with mine was the ' need to change to HTML before inserting into the DB.

Forgot to add that

[jjones666]

- added "blacklisted by" record.
- added (optional) "reason" - if you omit a reason, it will default to "see player notes" - we use !postit feature to record information usually in org.
- works in tell now also.
- alts lookup clicky added to !blacklist output.
- updated helpfiles.

http://www.jjones.co.uk/files/blacklist.php
http://www.jjones.co.uk/files/blacklist.txt

[jjones666]

Oops, had some DynDNS problems earlier - resolved now.

-jj-

[Dracutza]

thank you for this. I'll have to take a look at it later tonight.  I did manage to edit yours to the point where it will record the submitter and a reason. It works well up to the point when someone does not include a reason.  We require a reason so the enduser can determine the value of the blacklist entry.  We've had members blacklist people because "So and So didn't tip for a wrangle."  Requiring a reason at least allows other to determine the value of the blacklist entry.  Anyway, if a reason is not entered, the bot does nothing.  So now I am trying to figure out how to make the bot reply back that a reason is needed.

Thanks again.

[jjones666]

Hmm, my update isn't restrictive over the reason, simply because I find the !postit option better as this is more accessable from !whois.  If you want it restrictive, I can do this later and post the code I suppose...?

-jj-

[Dracutza]

maybe i should clarify that by "editing yours", i was refering to the one you had posted on 9-9.  I have not yet tried the one posted over this weekend.

If you are so compelled to resolve my little problem, I've attached my version of blacklist.  What would probably take someone with experience a few minutes will take me a few hours of trial and error, but I'm not afraid of getting this done on my own.  Even if i should be afraid of Wolfbiter flaming me for unethical coding   

In any event, that you all for the help already, and really making a great bot.

[jjones666]

Wolfbiter rocks, at least Bebot wise.

Natu, well, apart from being superadmin-of-some-bot-or-other, I'm still waiting for his updated depot module and he is persistant troller on Zodsnet forum :-)

(And, I'll have a look at the code later for you).

-jj-

[jjones666]

maybe i should clarify that by "editing yours", i was refering to the one you had posted on 9-9.  I have not yet tried the one posted over this weekend.

If you are so compelled to resolve my little problem, I've attached my version of blacklist.  What would probably take someone with experience a few minutes will take me a few hours of trial and error, but I'm not afraid of getting this done on my own.  Even if i should be afraid of Wolfbiter flaming me for unethical coding   

In any event, that you all for the help already, and really making a great bot.

Ok try replacing the code to handle messages in the module I posted yesterday with the below.  Should be fine for what you need I hope.

-jj-

Code:

/*
This gets called on a msg in the privgroup with the command
*/
function pgmsg($name, $msg)
{
if (preg_match("/^" . $this -> bot -> commpre . "blacklist add ([^\s]+) (.+)$/i", $msg, $info))
$this -> bot -> send_pgroup($this -> set_blacklist($name, $info[1], $info[2]));
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist add (.+)$/i", $msg, $info))
{
$msg = "You must enter a reason!";
$this -> bot -> send_pgroup($msg);
}
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist del (.+)$/i", $msg, $info))
$this -> bot -> send_pgroup($this -> del_blacklist($name, $info[1]));
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist{0,1}$/i", $msg))
$this -> bot -> send_pgroup($this -> get_blacklist());
else
$this -> bot -> send_pgroup($this -> get_blacklist());
}


/*
This gets called on a msg in the guildchat with the command
*/
function tell($name, $msg)
{

if (preg_match("/^" . $this -> bot -> commpre . "blacklist add ([^\s]+) (.+)$/i", $msg, $info))
$this -> bot -> send_tell($name, $this -> set_blacklist($name, $info[1], $info[2]));
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist add (.+)$/i", $msg, $info))
{
$msg = "You must enter a reason!";
$this -> bot -> send_tell($name, $msg);
}
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist del (.+)$/i", $msg, $info))
$this -> bot -> send_tell($name, $this -> del_blacklist($name, $info[1]));
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist{0,1}$/i", $msg))
$this -> bot -> send_tell($name, $this -> get_blacklist());
else
$this -> bot -> send_tell($name, $this -> get_blacklist());
}


/*
This gets called on a msg in the guildchat with the command
*/
function gc($name, $msg)
{
if (preg_match("/^" . $this -> bot -> commpre . "blacklist add ([^\s]+) (.+)$/i", $msg, $info))
$this -> bot -> send_gc($this -> set_blacklist($name, $info[1], $info[2]));
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist add (.+)$/i", $msg, $info))
{
$msg = "You must enter a reason!";
$this -> bot -> send_gc($msg);
}
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist del (.+)$/i", $msg, $info))
$this -> bot -> send_gc($this -> del_blacklist($name, $info[1]));
else if (preg_match("/^" . $this -> bot -> commpre . "blacklist{0,1}$/i", $msg))
$this -> bot -> send_gc($this -> get_blacklist());
else
$this -> bot ->send_gc($this -> get_blacklist());
}

[Khalem]

Just a suggestion JJ.

You might want to look at how we handle commands in newer modules in SVN. Imho it's a cleaner way, although i still need to clean up more.
See http://svn.shadow-realm.org/index.py/BeBot/trunk/modules/Roster.php?revision=247&view=markup as an example.

Basically, common handler for all messages (avoids typos and code repetition), common handler is told what type of command it is so it can handle tells differenly from pgroup etc if wanted. But most importantly, the way commands are parsed.
First we split the incoming string into variables ($vars) using explode(), then we strip the command prefix, assign the actual command passed to $command for easy parsing and reference, and then use switch() to handle what we today use regexp for.

What i still need to do with this is more as much code as possible out of the main command handler, for example, the error checking done there now on each command should be done inside send_out instead. But non the less, it should serve as a decent example of what i think is a better way to do command handling.

[jjones666]

Khalem,

Would that type of command processing work in Bebot 0.2.x?

I agree, it does look a lot simpler...

-jj-

[Naturalistic]

I would have to disagree about adding it... 0.2.x is going out of date