GitHub
/
pbot
mirror of https://github.com/pragma-/pbot.git
3
0
Fork 0
Code Issues Projects Releases Wiki Activity

Make fact log of factset more concise

This commit is contained in:
Pragmatic Software 2017-08-27 19:02:48 -07:00
parent ca827a1499
commit 80bceb2c75
1 changed files with 1 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
PBot/FactoidCommands.pm
View File

@ -341,8 +341,6 @@ sub factset {
} }
} }
my $oldvalue;
if(defined $owner_channel) { if(defined $owner_channel) {
my $factoid = $self->{pbot}->{factoids}->{factoids}->hash->{$owner_channel}->{$owner_trigger}; my $factoid = $self->{pbot}->{factoids}->{factoids}->hash->{$owner_channel}->{$owner_trigger};
@ -351,18 +349,12 @@ sub factset {
if(lc $nick ne lc $owner and $level == 0) { if(lc $nick ne lc $owner and $level == 0) {
return "You are not the owner of $trigger."; return "You are not the owner of $trigger.";
} }
$oldvalue = $self->{pbot}->{factoids}->{factoids}->hash->{$channel}->{$trigger}->{$key};
} }
my $result = $self->{pbot}->{factoids}->{factoids}->set($channel, $trigger, $key, $value); my $result = $self->{pbot}->{factoids}->{factoids}->set($channel, $trigger, $key, $value);
if (defined $value and $result =~ m/set to/) { if (defined $value and $result =~ m/set to/) {
if (defined $oldvalue and $oldvalue ne $value) { $self->log_factoid($channel, $trigger, "$nick!$user\@$host", "set $key to $value");
$self->log_factoid($channel, $trigger, "$nick!$user\@$host", "set $key from $oldvalue to $value");
} else {
$self->log_factoid($channel, $trigger, "$nick!$user\@$host", "set $key to $value");
}
} }
return $result; return $result;