(REF) CRM_Api4_Services - Cleanup extra comment/try/catch

diff --git a/CRM/Api4/Services.php b/CRM/Api4/Services.php
index fba89627dd9b027d97754ac9712e0437a0907d6d..35c24449e04a5a6bd17ed1ed7f27cb796d4deefc 100644 (file)
--- a/CRM/Api4/Services.php
+++ b/CRM/Api4/Services.php
@@ -84,10 +84,8 @@ class CRM_Api4_Services {
       if (!file_exists($path) || !is_dir($path)) {
         $resource = new \Symfony\Component\Config\Resource\FileExistenceResource($path);
         $container->addResource($resource);
-        continue;
       }
-
-      try {
+      else {
         $resource = new \Symfony\Component\Config\Resource\DirectoryResource($path, ';\.php$;');
         foreach (glob("$path*.php") as $file) {
           $matches = [];
@@ -101,15 +99,6 @@ class CRM_Api4_Services {
         }
         $container->addResource($resource);
       }
-      catch (\InvalidArgumentException $e) {
-        //Directory is not found so lets not do anything i suppose.
-
-        // FIXME: The above comment implies that the try/catch is specifically
-        // about the directory's existence, which would make it redundant with the
-        // newer "file_exists()" guard. However, the actual "catch()" seems broader,
-        // and I don't see anything in DirectoryResource that throws the exception.
-        // So... maybe it's not needed, or maybe there's some additional purpose.
-      }
     }
   }