From daff56151c80cdbd160bded19743cff422639637 Mon Sep 17 00:00:00 2001
From: GlazerMann <kurt@ioiowa.com>
Date: Wed, 11 Dec 2024 13:05:12 -0600
Subject: [PATCH] Remove outdated comments

---
 apiFunctions.php | 9 ++++-----
 1 file changed, 4 insertions(+), 5 deletions(-)

diff --git a/apiFunctions.php b/apiFunctions.php
index 89a94e639c..056709bc2e 100644
--- a/apiFunctions.php
+++ b/apiFunctions.php
@@ -484,7 +484,7 @@ function adsabs_api(array $ids, array &$templates, string $identifier): void {
 }
 
 /** @param array<string> $_ids
-        @param array<Template> $templates */
+    @param array<Template> $templates */
 function query_doi_api(array $_ids, array &$templates): void { // $id not used yet  // Pointer to save memory
     foreach ($templates as $template) {
         expand_by_doi($template);
@@ -494,10 +494,6 @@ function query_doi_api(array $_ids, array &$templates): void { // $id not used y
 
 function expand_by_doi(Template $template, bool $force = false): void {
     set_time_limit(120);
-    // Because it can recover rarely used parameters such as editors, series & isbn,
-    // there will be few instances where it could not in principle be profitable to
-    // run this function, so we don't check this first.
-
     if (!$template->verify_doi()) {
         return; // TODO - sometimes CrossRef has Data even when DOI is broken, perhaps skip this
     }
@@ -509,6 +505,9 @@ function expand_by_doi(Template $template, bool $force = false): void {
     if (preg_match(REGEXP_DOI_ISSN_ONLY, $doi)) {
         return;
     }
+    if (in_array($doi, BAD_DOI_ARRAY)) { // Really bad ones that do not really exist at all
+        return;
+    }
     if ($doi && preg_match('~^10\.2307/(\d+)$~', $doi)) {
         $template->add_if_new('jstor', substr($doi, 8));
     }