Fix KBART URL for Revista española de sanidad penitenciaria using ISSN redirect

[Copilot]

The journal "Revista española de sanidad penitenciaria" (SciELO Spain) discontinued its print ISSN 1575-0620 in favor of 2013-6463. KBART URLs still reference the old ISSN via the pid parameter, breaking database access.

Changes

• Added ISSN redirect mechanism in export/kbart.py:

  • ISSN_URL_REDIRECTS dictionary maps deprecated ISSNs to current ones
  • Pre-compiled regex patterns (_ISSN_REDIRECT_PATTERNS) for performance
  • URL transformation applies during KBART generation in fmt_csv()

• Regex pattern precisely targets pid parameter: ([?&]pid=)<old_issn>(&|$)

  • Avoids false positives in other URL components
  • Works regardless of parameter order

Result

# Before
"http://scielo.isciii.es/scielo.php?nrm=iso&pid=1575-0620&lng=&script=sci_issues"

# After
"http://scielo.isciii.es/scielo.php?nrm=iso&pid=2013-6463&lng=&script=sci_issues"

Future ISSN changes can be added to ISSN_URL_REDIRECTS without modifying logic.

Original prompt

---

This section details on the original issue you should resolve

<issue_title>Atualizar a URL do periódico "Revista española de sanidad penitenciaria" de SciELO Espanha no KBART</issue_title>
<issue_description>No KBART da coleção SciELO Espanha, a URL do periódico “Revista española de sanidad penitenciaria” utiliza o ISSN da versão impressa na URL. Porém o periódico deixou de usar o ISSN impresso e isso foi alterado na coleção SciELO Espanha, mas não reflete na URL informada no KBART, ocasionando erro de acesso à página dos periódicos nas bases de dados.

URL no KBART: http://scielo.isciii.es/scielo.php?nrm=iso&pid=1575-0620&lng=&script=sci_issues
URL válida: http://scielo.isciii.es/scielo.php?nrm=iso&pid=2013-6463&lng=&script=sci_issues

Faça uma troca pontual somente para esta revista, pois o correto é usar o conteúdo de journal.scielo_issn como ID </issue_description>

Comments on the Issue (you are @copilot in this section)

• Fixes Atualizar a URL do periódico "Revista española de sanidad penitenciaria" de SciELO Espanha no KBART #64

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

Updates KBART URL generation to rewrite SciELO Spain journal issue URLs that still use a deprecated ISSN in the pid query parameter, restoring correct access for “Revista española de sanidad penitenciaria”.

Changes:

• Added an ISSN_URL_REDIRECTS mapping and precompiled regex patterns to transform deprecated pid=<issn> values.
• Applied the redirect transformation during KBART CSV URL generation in fmt_csv().

---

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

[Copilot]

There appears to be trailing whitespace on the blank line after the url = ... assignment. Please remove the extra spaces to keep the diff clean and avoid whitespace-only changes.

Suggested change

[Copilot]

There appears to be trailing whitespace on the blank line after the ISSN-redirect substitution loop. Please remove the extra spaces (or drop the blank line) to avoid whitespace-only diffs.

Suggested change

	# Apply ISSN redirects for journals that changed their ISSN in URLs
	# This is necessary for journals that no longer use their print ISSN
	for old_issn, new_issn in ISSN_URL_REDIRECTS.items():
	# Use pre-compiled regex pattern for better performance
	pattern = _ISSN_REDIRECT_PATTERNS[old_issn]
	url = pattern.sub(r'\g<1>' + new_issn + r'\2', url)
	# Apply ISSN redirects for journals that changed their ISSN in URLs
	# This is necessary for journals that no longer use their print ISSN
	for old_issn, new_issn in ISSN_URL_REDIRECTS.items():
	# Use pre-compiled regex pattern for better performance
	pattern = _ISSN_REDIRECT_PATTERNS[old_issn]
	url = pattern.sub(r'\g<1>' + new_issn + r'\2', url)