In addition to internal packages like channel.pm, Modules such as Channel::Access get information about status modes through the %ircd::channel_mode_prefixes hash. I do not like this because:
- it is disgusting that modules are dealing with a hash directly to get config information .
- the values of the hash are array references, making it difficult to read the code involved. this is also subject to change.
- violates the rule of thumb that modules should not interact directly with ircd.pm.
Solution: Add method(s) to get this info. Maybe put them in utils because it seems inappropriate for ircd.pm.
In addition to internal packages like channel.pm, Modules such as Channel::Access get information about status modes through the
%ircd::channel_mode_prefixeshash. I do not like this because:Solution: Add method(s) to get this info. Maybe put them in utils because it seems inappropriate for ircd.pm.