Support "someone kicked ____" factoids

[dgw]

In addition to the currently supported $nick kicked $user and $nick kicked someone triggers, users on my instance have proposed use cases for someone kicked $user and someone kicked someone.

As an extension to the existing "kicked" triggers, the resolution hierarchy would go from most to least specific in this order:

1. $nick kicked $user (most specific)
2. $nick kicked someone
3. someone kicked $user
4. someone kicked someone (least specific)

Doing it this way essentially adds the new functionality in such a way that the existing kick factoid types are preferred over the new types.

Per @zigdon below, the implementation must also look up the new kick triggers only if a related config flag is enabled.

[zigdon]

We used to have support for that, and decided it was too spammy (especially that last one). If we going to restore it, it should be guarded in a config flag.

…

On Sun, Mar 19, 2017, 20:44 dgw ***@***.***> wrote: In addition to the currently supported $nick kicked $user and $nick kicked someone triggers, users on my instance have proposed use cases for someone kicked $nick and someone kicked someone. As an extension to the existing "kicked" triggers, the resolution hierarchy would go from most to least specific in this order: 1. $nick kicked $nick (most specific) 2. $nick kicked someone 3. someone kicked $nick 4. someone kicked someone (least specific) Doing it this way essentially adds the new functionality in such a way that the existing kick factoid types are preferred over the new types. — You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub <#98>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AACtVHUBrW4xEe18lzq6L3SD0ohbHWahks5rnfYRgaJpZM4MiATu> .

[dgw]

If we going to restore it, it should be guarded in a config flag.

That's fair.

[dgw]

@zigdon Would you prefer the default to be on or off? I'd lean toward making the default the current behavior, but only because I wouldn't want to suddenly switch it on for existing instances. Ideally I'd like the default to be enabled for new installs, and disabled for existing…not sure if that's possible with Bucket's current config (IIRC it has no awareness of whether it's been upgraded or created fresh).

[zigdon]

I'd say if it's a plugin, we can have it off by default, and have it added to the sample config, so that new installs will have it on.

[dgw]

This just came up in one of my Bucket-serviced channels, so… I guess it's time to put it officially on my plate.