Se mezclan los archivos generados

[germankay]

fixes #17

[Copilot]

Pull Request Overview

This PR ensures that updating the iati_base_resource_id extra preserves any other existing extras and refines how datasets are queried for that extra.

• Merge or replace the iati_base_resource_id extra without discarding other extras when patching a package.
• Switch the dataset search to use CKAN’s fq filter and add debug logging for each found dataset.

Reviewed Changes

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

File	Description
ckanext/iati_generator/blueprint/iati.py	Merge new iati_base_resource_id into existing extras instead of overwriting them.
ckanext/iati_generator/actions/iati.py	Change package_search to use fq filter and log each dataset found.

Comments suppressed due to low confidence (1)

ckanext/iati_generator/blueprint/iati.py:68

• [nitpick] This comment is in Spanish and includes an emoji; consider translating it to English and removing informal markers for consistency with the codebase’s style.

# 🛠 Reemplazar o agregar el extra sin perder los anteriores

[Copilot]

CKAN indexes extras under extras_<key>, so the filter should likely be extras_iati_base_resource_id:* instead of iati_base_resource_id:* to return the correct datasets.

Suggested change

	"fq": "iati_base_resource_id:*",
	"fq": "extras_iati_base_resource_id:*",

[avdata99]

Some notes:

• Search for examples of how to create resources by uploading the file. It should not be necessary to store in the storage_path (resource_id[:3], resource_id[3:6] should not be used). CKAN should take care of saving the XML file where it belongs.
• Analyze whether to use extra resources or extra packages. It will be better to use resource extras because it is more straightforward and less confusing. There could be a dataset with multiple resources to transform into IATI and if the extras are at the beginning of the dataset, it will fail.