Skip to content

Fixed bad ArgumentNullException constructors

Mateusz Brawański requested to merge github/fork/Lachee/master into master

Created by: Lachee

Summary

Fixed a issue with the ArgumentNullExceptions within the CommandExtensions that resulted in weird messages such as the one below. This pull request only changes the CommandExtensions and further investigations will have to be made for the rest of the codebase.

example

Details

Literally just changed the ArgumentNullException constructors in DSharpPlus.CommandsNext/CommandsNextExtension.cs to match the prototype ArgumentNullException(string paramName, string message) as they were reversed, resulting in weird errors such as "ArgumentNullException: t".

Notes

I have only been through the CommandsNextExtension, so there could be more hiding.

Merge request reports