Fix DOI ingestion bugs

[ssheikholeslami]

Summary

• Fix AttributeError when OpenAIRE API returns None for author name/surname fields
• Fix TypeError from double db injection when calling search_dois() from validate_dois()

Problem

When processing longer DOI lists, two bugs prevented successful ingestion:

1. parser.py: The OpenAIRE API can return {"name": None} for authors. Using .get("name", "") only provides a default when the key is missing, not when the value is None, causing AttributeError: 'NoneType' object has no attribute 'title'.

2. doi.py: Both validate_dois() and search_dois() are decorated with @connect_to_db. Calling self.search_dois(db) passed db explicitly while the decorator also injected it, causing TypeError: search_dois() takes 2 positional arguments but 3 were given.

Fix

1. Changed metadata.get("name", "") to (metadata.get("name") or "") to handle None values
2. Changed self.search_dois(db) to self.search_dois() since the decorator handles db injection

Test plan

• Tested with longer DOI list (list_of_doi.csv from CCG) that previously failed.

[Copilot]

Pull request overview

This PR fixes two runtime errors encountered during DOI ingestion: handling None author name fields from the OpenAIRE API and avoiding a double db injection when validating DOIs.

Changes:

• Updated parse_author in parser.py to safely handle None values for name and surname from OpenAIRE metadata without raising AttributeError.
• Updated validate_dois in doi.py to call the decorated search_dois without explicitly passing the db argument, preventing a TypeError from double-injecting the database driver.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File	Description
src/research_index_backend/parser.py	Adjusts author name parsing to coerce None name/surname values to empty strings before title-casing, preventing ingestion failures when OpenAIRE returns null for these fields.
src/research_index_backend/doi.py	Fixes the internal call from validate_dois to search_dois to rely solely on the @connect_to_db decorator, resolving the double-db injection error during DOI validation.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

Pull request overview

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[willu47]

I've tested that the code works locally and confirm that it does.