View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0005143 | unreal | module api | public | 2018-09-13 11:31 | 2018-09-20 20:03 |
| Reporter | Gottem | Assigned To | syzop | ||
| Priority | normal | Severity | feature | Reproducibility | N/A |
| Status | resolved | Resolution | fixed | ||
| Fixed in Version | 4.2.0 | ||||
| Summary | 0005143: Priorities for CmdOverrides | ||||
| Description | I currently have a few modules implementing command overrides, one of which needs to be changed to not call _other_ overrides anymore. So this one should always be processed last, but there's currently no way to assign priorities like you can with hooks. This is in relation to PeGaSuS's report about m_fixhop throwing duplicate errors. My override for INVITE also runs CallCmdOverride() (because I have other INVITE overrides too), but eventually the m_invite CMD_FUNC will be called which throws the "cannot invite" error. Due to my override the INVITE may already be sent to another server, which does the same checks and also returns an error. So this results in 2 errors from 2 different servers but the invite actually went through. :D So I was thinking to just have m_fixhop's override run as the very last thing and return 0 instead of CallCmdOverride. =] | ||||
| Tags | No tags attached. | ||||
| 3rd party modules | |||||
|
|
Thanks, added. I will actually need this for WHOX, hence the fast commit :D commit 30da2ae5530b5ec8fb66ea694d81a73fcb5cc8d1 (HEAD -> unreal40, origin/unreal40, origin/HEAD) Author: Bram Matthys <[email protected]> Date: Thu Sep 20 20:01:39 2018 +0200 Module devs: Add priorities for command overrides. Use: CmdoverrideAddEx(module, name, priority, function) Suggested by Gottem in https://bugs.unrealircd.org/view.php?id=5143 ..and needed by some upcoming stuff. |
