Constructor overload better than passing a null to affect its behaviour. Also a fair few commands missing autocompletion, most of which had to do with enums.
I also refactored the client parsing for the kick/ban commands since I had a look at the file and it stood out. For sake of sanity I have to confirm that at the end of the refactored method it's supposed to check for client == null, right? Because before ban checked for not null. I think it's an error unless I'm missing something.
Constructor overload better than passing a null to affect its behaviour. Also a fair few commands missing autocompletion, most of which had to do with enums.
I also refactored the client parsing for the kick/ban commands since I had a look at the file and it stood out. For sake of sanity I have to confirm that at the end of the refactored method it's supposed to check for
client == null
, right? Because beforeban
checked for not null. I think it's an error unless I'm missing something.