Browse Source

Merge pull request #177 from jnyrup/ArgumentException

Use `paramName` overload of `ArgumentException`
Marcin Ziąbek 3 years ago
parent
commit
e4d55e0298
1 changed files with 2 additions and 2 deletions
  1. 2 2
      QuestPDF/Fluent/TextExtensions.cs

+ 2 - 2
QuestPDF/Fluent/TextExtensions.cs

@@ -185,7 +185,7 @@ namespace QuestPDF.Fluent
         public TextSpanDescriptor SectionLink(string? text, string sectionName)
         public TextSpanDescriptor SectionLink(string? text, string sectionName)
         {
         {
             if (IsNullOrEmpty(sectionName))
             if (IsNullOrEmpty(sectionName))
-                throw new ArgumentException(nameof(sectionName));
+                throw new ArgumentException("Section name cannot be null or empty", nameof(sectionName));
 
 
             var style = DefaultStyle.Clone();
             var style = DefaultStyle.Clone();
             var descriptor = new TextSpanDescriptor(style);
             var descriptor = new TextSpanDescriptor(style);
@@ -212,7 +212,7 @@ namespace QuestPDF.Fluent
         public TextSpanDescriptor Hyperlink(string? text, string url)
         public TextSpanDescriptor Hyperlink(string? text, string url)
         {
         {
             if (IsNullOrEmpty(url))
             if (IsNullOrEmpty(url))
-                throw new ArgumentException(nameof(url));
+                throw new ArgumentException("Url cannot be null or empty", nameof(url));
 
 
             var style = DefaultStyle.Clone();
             var style = DefaultStyle.Clone();
             var descriptor = new TextSpanDescriptor(style);
             var descriptor = new TextSpanDescriptor(style);