fix(articlemeta): corrige memory leak por lru_cache em instância e robustez dos formatters journal/issue (#1424)

* refactor(journal/articlemeta_format): elimina lru_cache, melhora robustez e legibilidade

- Substitui @lru_cache por cache manual em _titles_in_database_medline_secs
  para evitar memory leak (lru_cache em método de instância retém self
  indefinidamente no cache do decorador)
- Renomeia _medline_titles para _titles_in_database_medline_secs para
  consistência com o nome da property
- Renomeia _former_dict_journal_history para _format_journal_history_entry
  (corrige typo e segue convenção _format_* da classe)
- Renomeia variável key_to_issn para key_to_value (nem todos os valores
  são ISSNs)
- Adiciona guard clause 'journal_acron.upper() if journal_acron else None'
  em v930 para evitar AttributeError quando journal_acron é None
- Refatora _format_publisher_info: substitui loop com break por .first()
  para obter apenas o primeiro owner, eliminando iteração desnecessária
- Refatora _format_contact_address_info: substitui try/except genérico
  por checagem explícita 'if address', removendo tratamento silencioso
  de exceções
- Refatora _format_journal_history: inverte condicional para early return,
  unifica blocos ADMITTED/INTERRUPTED duplicados em um único 'if in',
  e garante que subfield_b é determinado por evento (não acumulado)
- Refatora _format_indexing_info: substitui duas list comprehensions
  separadas por um único loop para classificar medline vs secs,
  evitando dupla iteração sobre titles_in_db
- Refatora _format_collection_info: remove checagem redundante
  'if collection' (já coberta pelo 'if self.scielo_journal.collection')
- Remove guard desnecessário 'if self.official' em _format_issn_list
  (método só é chamado dentro de bloco que já verifica self.official)
- Adiciona TODO/docstring em _format_issn_type sinalizando possível
  inversão na lógica de negócio (issn_print == issn_scielo retorna
  'ONLIN', o que parece invertido)
- Corrige docstring typo 'Title Journalal' -> 'Title Journal'
- Remove comentários óbvios/redundantes (e.g. 'tem que ser objeto
  datetime', 'Deixa preparado para tornar obsoleto')
- Normaliza trailing whitespace e vírgulas finais em dicts/listas

* refactor(issue/articlemeta_format): elimina lru_cache, corrige queryset e robustez

- Substitui @lru_cache por cache manual em medline_titles para evitar
  memory leak (mesmo padrão aplicado no journal formatter)
- Substitui atributo self.article (queryset avaliado no __init__) por
  property lazy self.article_qs que só executa a query quando acessado,
  evitando query desnecessária quando o dado não é utilizado
- Corrige _format_article_info: usa self.article_qs (filtrado por
  issue + journal) em vez de self.obj.article_set (sem filtro por
  journal), garantindo contagem consistente com o queryset da classe
- Adiciona guard clause em _format_field_use_system para checar
  journal_acron antes de chamar .upper(), e usa 'or empty string'
  para volume/number None, evitando concatenação com None
- Refatora _format_issn_info com early return (guard clause) para
  reduzir nível de indentação; adiciona placeholder v435 no
  add_multiple_to_result para documentar que será preenchido por
  _format_issn_with_type
- Adiciona guard 'if issns' antes de atribuir v435 em
  _format_issn_with_type para evitar lista vazia no resultado
- Melhora select_related em _format_code_sections: inclui
  'journal_toc' além de 'journal_toc__language'; adiciona guard
  'if not journal_toc: continue' para pular registros sem relação
- Remove comentários redundantes (e.g. 'Path to base issue',
  'Ordem de publicação', 'Só adiciona v/n se houver')
- Remove import não utilizado 'from functools import lru_cache'
- Corrige missing newline at end of file

Author: robertatakenaka

issue/formats/articlemeta_format.py

@@ -1,5 +1,4 @@
 from collections import defaultdict
-from functools import lru_cache
 
 from article.models import Article
 from core.utils.articlemeta_dict_utils import (
@@ -10,6 +9,7 @@
 from journal.formats.articlemeta_format import ArticlemetaJournalFormatter
 from journal.models import SciELOJournal, TitleInDatabase
 
+
 def get_issue_type(issue):
     if issue.supplement:
         return "supplement"
@@ -32,7 +32,16 @@ def __init__(self, obj, collection):
         self.journal = self.obj.journal
         self._scielo_journal = None
         self._medline_titles = None
-        self.article = Article.objects.filter(issue=self.obj, journal=self.journal)
+        self._article_qs = None
+
+    @property
+    def article_qs(self):
+        """Lazy queryset — só executa query quando necessário."""
+        if self._article_qs is None:
+            self._article_qs = Article.objects.filter(
+                issue=self.obj, journal=self.journal
+            )
+        return self._article_qs
 
     @property
     def scielo_journal(self):
@@ -49,13 +58,15 @@ def scielo_journal(self):
         return self._scielo_journal
 
     @property
-    @lru_cache(maxsize=1)
     def medline_titles(self):
-        return list(
-            TitleInDatabase.objects.filter(
-                journal=self.journal, indexed_at__acronym__iexact="medline"
+        """Cache manual em vez de lru_cache (evita memory leak em instância)."""
+        if self._medline_titles is None:
+            self._medline_titles = list(
+                TitleInDatabase.objects.filter(
+                    journal=self.journal, indexed_at__acronym__iexact="medline"
+                )
             )
-        )
+        return self._medline_titles
 
     def format(self):
         """Formata todos os dados do issue"""
@@ -85,8 +96,6 @@ def format(self):
 
     def _format_basic_info(self):
         """Informações básicas do issue"""
-        # Path to base issue
-
         add_multiple_to_result(
             {
                 "v31": self.obj.volume,
@@ -100,7 +109,6 @@ def _format_basic_info(self):
             },
             self.result["issue"],
         )
-        # "v6": Ordem de publicação dos fascículos para apresentação na interface
 
         if hasattr(self.obj, "issue_title"):
             items = [item.title for item in self.obj.issue_title.all() if item.title]
@@ -214,7 +222,6 @@ def _format_title_in_database(self):
 
     def _format_metadata(self):
         """Metadados e relacionamentos"""
-        # tem que ser objeto datetime
         processing_date = self.obj.updated.strftime("%Y-%m-%d")
 
         key_to_code = {
@@ -252,8 +259,12 @@ def _format_register_order_info(self):
 
     def _format_field_use_system(self):
         """Campo usado no sistema"""
-        if self.scielo_journal:
-            field_value = f"{self.scielo_journal.journal_acron.upper()}{self.obj.volume}{self.obj.number}"
+        if self.scielo_journal and self.scielo_journal.journal_acron:
+            field_value = (
+                f"{self.scielo_journal.journal_acron.upper()}"
+                f"{self.obj.volume or ''}"
+                f"{self.obj.number or ''}"
+            )
             add_to_result("v888", field_value, self.result["issue"])
 
     def _format_legend_bibliographic(self):
@@ -278,10 +289,8 @@ def _format_legend_bibliographic(self):
                 "a": self.obj.year,
                 "_": "",
             }
-            # Só adiciona 'v' se houver volume
             if self.obj.volume:
                 entry["v"] = f"vol.{self.obj.volume}"
-            # Só adiciona 'n' se houver number
             if self.obj.number:
                 entry["n"] = f"no.{self.obj.number}"
             if self.obj.season:
@@ -308,28 +317,37 @@ def _format_title_summary(self):
         ]
 
     def _format_article_info(self):
-        """Informações de artigo"""
-        if self.article.exists():
-            article_count = str(self.obj.article_set.count())
+        """Informações de artigo — usa o queryset consistente (filtrado por issue + journal)."""
+        if self.article_qs.exists():
+            article_count = str(self.article_qs.count())
             add_to_result("v122", article_count, self.result["issue"])
 
     def _format_issn_info(self):
-        """Informações de edição"""
-        if self.scielo_journal and self.scielo_journal.journal and self.scielo_journal.journal.official:
-            issn_print = self.scielo_journal.journal.official.issn_print
-            issn_electronic = self.scielo_journal.journal.official.issn_electronic
-            issn_scielo = self.scielo_journal.issn_scielo
-            add_multiple_to_result(
-                {
-                    "v35": issn_scielo,
-                    "v935": issn_electronic,
-                },
-                self.result["issue"],
-            )
+        """Informações de ISSN"""
+        if not (
+            self.scielo_journal
+            and self.scielo_journal.journal
+            and self.scielo_journal.journal.official
+        ):
+            return
+
+        official = self.scielo_journal.journal.official
+        issn_print = official.issn_print
+        issn_electronic = official.issn_electronic
+        issn_scielo = self.scielo_journal.issn_scielo
+
+        add_multiple_to_result(
+            {
+                "v35": issn_scielo,
+                "v435": None,  # preenchido abaixo por _format_issn_with_type
+                "v935": issn_electronic,
+            },
+            self.result["issue"],
+        )
 
-            self._format_issn_with_type(issn_print, issn_electronic)
-            self._format_issn_code_title(issn_print, issn_electronic)
-            self._format_code(issn_scielo)
+        self._format_issn_with_type(issn_print, issn_electronic)
+        self._format_issn_code_title(issn_print, issn_electronic)
+        self._format_code(issn_scielo)
 
     def _format_code(self, issn_scielo):
         """Informações de código"""
@@ -346,7 +364,8 @@ def _format_issn_with_type(self, issn_print, issn_electronic):
             issns.append({"_": issn_print, "t": "PRINT"})
         if issn_electronic:
             issns.append({"_": issn_electronic, "t": "ONLIN"})
-        self.result["issue"]["v435"] = issns
+        if issns:
+            self.result["issue"]["v435"] = issns
 
     def _format_issn_code_title(self, issn_print, issn_electronic):
         self.result["code_title"] = [
@@ -355,10 +374,18 @@ def _format_issn_code_title(self, issn_print, issn_electronic):
 
     def _format_code_sections(self):
         data = []
-        for toc in self.obj.table_of_contents.select_related("journal_toc__language").all():
+        for toc in self.obj.table_of_contents.select_related(
+            "journal_toc", "journal_toc__language"
+        ).all():
             journal_toc = toc.journal_toc
+            if not journal_toc:
+                continue
             code = getattr(journal_toc, "code", None)
-            lang = getattr(journal_toc.language, "code2", None) if journal_toc.language else None
+            lang = (
+                getattr(journal_toc.language, "code2", None)
+                if journal_toc.language
+                else None
+            )
             text = journal_toc.text
             if code:
                 item = {"c": code, "_": ""}
@@ -379,4 +406,4 @@ def get_articlemeta_format_issue(obj, collection):
     data["title"] = ArticlemetaJournalFormatter(obj.journal, collection).format()
     formatter_issue = ArticlemetaIssueFormatter(obj, collection).format()
     data.update(formatter_issue)
-    return data
+    return data