From 5a9ea3ee10b6717707b38084f71fe70ab373b324 Mon Sep 17 00:00:00 2001 From: github-actions Date: Sat, 28 Oct 2023 15:43:34 +0000 Subject: [PATCH] Merge pull request #1408 from hydephp/code-cleanup Remove low priority todo comments https://github.com/hydephp/develop/commit/d5b24d891eec98c1a47a2c353bb0a914a9a0be1c --- src/Framework/Features/Blogging/Models/FeaturedImage.php | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/src/Framework/Features/Blogging/Models/FeaturedImage.php b/src/Framework/Features/Blogging/Models/FeaturedImage.php index 25f80a2b..71bf71ee 100644 --- a/src/Framework/Features/Blogging/Models/FeaturedImage.php +++ b/src/Framework/Features/Blogging/Models/FeaturedImage.php @@ -225,9 +225,7 @@ protected function getContentLengthForLocalImage(): int protected function getContentLengthForRemoteImage(): int { - // TODO: We may want to globalize this check in the config, but for now, - // we just check the server arguments and skip remote requests if - // the --no-api flag is present (in the build command call) + // Check if the --no-api flag is set when running the build command, and if so, skip the API call. if (! (isset($_SERVER['argv']) && in_array('--no-api', $_SERVER['argv'], true))) { $headers = Http::withHeaders([ 'User-Agent' => Config::getString('hyde.http_user_agent', 'RSS Request Client'),