You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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.
The text was updated successfully, but these errors were encountered:
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:Solution: Add method(s) to get this info. Maybe put them in utils because it seems inappropriate for ircd.pm.
The text was updated successfully, but these errors were encountered: