Improvement: more commands need to abort before Markov #3

Closed
opened 1 year ago by bss · 1 comments
bss commented 1 year ago
Owner

I think this is a problem in a lot of places, but I should remember how to stop processing to avoid situations like

09:27:39 <bss> !load markov.ircplugin
09:27:42 <mr_devzo> Plugin 'markov.ircplugin' is already loaded.
09:27:42 <mr_devzo> !load markov.ircplugin? !load markov.ircplugin? !load markov.ircplugin. !load markov.ircplugin? !load markov.ircplugin?
I think this is a problem in a lot of places, but I should remember how to stop processing to avoid situations like ``` 09:27:39 <bss> !load markov.ircplugin 09:27:42 <mr_devzo> Plugin 'markov.ircplugin' is already loaded. 09:27:42 <mr_devzo> !load markov.ircplugin? !load markov.ircplugin? !load markov.ircplugin. !load markov.ircplugin? !load markov.ircplugin? ```
bss commented 1 year ago
Poster
Owner

Not a bug. I introduced this behavior in the switch to DB-based server configs; Markov was matching '|nick' which means it was matching everything. Fixed in 43f2b09057.

Not a bug. I introduced this behavior in the switch to DB-based server configs; Markov was matching '|nick' which means it was matching everything. Fixed in 43f2b090572f006c8b7674f80a10d0cd1e38bff9.
bss closed this issue 1 year ago
Sign in to join this conversation.
No Label
No Milestone
No Assignees
1 Participants
Notifications
Due Date

No due date set.

Dependencies

This issue currently doesn't have any dependencies.

Loading…
There is no content yet.