Skip to content

Fixes issue where database export was failing#13909

Open
roldy wants to merge 1 commit intodevelopmentfrom
bugfix-13851-database-export
Open

Fixes issue where database export was failing#13909
roldy wants to merge 1 commit intodevelopmentfrom
bugfix-13851-database-export

Conversation

@roldy
Copy link
Copy Markdown
Contributor

@roldy roldy commented Apr 14, 2026

Fixes #13851

Summary by CodeRabbit

Release Notes

  • New Features
    • Database exports now include adverse immunization following event data, encompassing adverse events, immunization reaction investigations, and associated incident information for more comprehensive health data reporting.

@roldy roldy requested review from obinna-h-n and raulbob April 14, 2026 11:50
@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Apr 14, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro

Run ID: 185f28d5-7b94-4d2c-be3a-0a168f067a21

📥 Commits

Reviewing files that changed from the base of the PR and between ac0e072 and 3ee183b.

📒 Files selected for processing (2)
  • sormas-backend/src/main/java/de/symeda/sormas/backend/importexport/DatabaseExportService.java
  • sormas-backend/src/test/java/de/symeda/sormas/backend/importexport/DatabaseExportServiceTest.java
💤 Files with no reviewable changes (1)
  • sormas-backend/src/test/java/de/symeda/sormas/backend/importexport/DatabaseExportServiceTest.java

📝 Walkthrough

Walkthrough

The pull request enables database export for three previously unsupported adverse-immunization entities (Aefi, AdverseEvents, and AefiInvestigation) by registering them in the export configuration. A SQL query is also updated for schema compatibility, and test exclusions for these entities are removed.

Changes

Cohort / File(s) Summary
Export Configuration Registration
sormas-backend/src/main/java/.../DatabaseExportService.java
Adds imports and registers three new adverse-immunization entities (Aefi, AdverseEvents, AefiInvestigation) in EXPORT_CONFIGS with their table mappings. Updates COUNT_TABLE_COLUMNS query to cast tableName parameter to text for schema compatibility.
Test Exclusion Removal
sormas-backend/src/test/java/.../DatabaseExportServiceTest.java
Removes Aefi, AdverseEvents, and AefiInvestigation from NOT_EXPORTED_ENTITIES list, allowing these entities to be validated against EXPORT_CONFIGS during testing.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐰 Three entities hopped into view,
Adverse events now export too!
With configs set and tests aligned,
No data left behind,
Database exports are finally true! 📊

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Title check ❓ Inconclusive The title is vague and generic, describing that 'database export was failing' without specifying what the actual issue or fix is. Improve the title to be more specific about the fix, such as 'Add missing entities to database export configuration' or 'Fix adverse event export in database export functionality'.
✅ Passed checks (4 passed)
Check name Status Explanation
Description check ✅ Passed The PR description is minimal but includes the required issue number. However, it provides no details about what was changed or why, only linking to issue #13851.
Linked Issues check ✅ Passed The PR successfully adds three missing entities (Aefi, AdverseEvents, AefiInvestigation) to the database export configuration, directly addressing the export failures reported in issue #13851.
Out of Scope Changes check ✅ Passed All changes are narrowly scoped to adding missing entities to export configuration and updating test exclusions; no unrelated modifications detected.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch bugfix-13851-database-export

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Error message while using Statistics - Database Export

1 participant