Issue #3114541 by wizonesolutions: Remove useless check.

src/Service/FillPdfLinkManipulator.php

@@ -61,10 +61,6 @@ class FillPdfLinkManipulator implements FillPdfLinkManipulatorInterface {
   public function parseLink(Url $link) {
     $query = $link->getOption('query');
 
-    if (!$query) {
-      throw new \InvalidArgumentException("This link doesn't specify a query string, so failing.");
-    }
-
     if (empty($query['fid'])) {
       throw new \InvalidArgumentException('No FillPDF Form was specified in the query string, so failing.');
     }

tests/src/Functional/LinkManipulatorTest.php

@@ -52,9 +52,6 @@ class LinkManipulatorTest extends BrowserTestBase {
     // Hit the generation route with no query string set.
     $fillpdf_route = Url::fromRoute('fillpdf.populate_pdf', [], []);
     $this->drupalGet($fillpdf_route);
-    // Ensure the exception is converted to an error and access is denied.
-    $this->assertSession()->statusCodeEquals(403);
-    $this->assertSession()->pageTextContains("This link doesn't specify a query string, so failing.");
 
     // Hit the generation route with no fid set.
     $fillpdf_route = Url::fromRoute('fillpdf.populate_pdf', [], [