Skip to content

Fix DwCA exporting#7857

Merged
grantfitzsimmons merged 1 commit intov7.12.0-prereleasefrom
issue-7856
Mar 30, 2026
Merged

Fix DwCA exporting#7857
grantfitzsimmons merged 1 commit intov7.12.0-prereleasefrom
issue-7856

Conversation

@alesan99
Copy link
Copy Markdown
Contributor

Fixes #7856

Adds an extra check to field formatters to prevent crashing. I haven't found the root cause of the regression, however.

Checklist

  • Self-review the PR after opening it to make sure the changes look good and
    self-explanatory (or properly documented)
  • Add relevant issue to release milestone
  • Add pr to documentation list
  • Add automated tests

Testing instructions

  • Open User Tools
  • Click 'Create DwC Archive' and create an archive
  • There is no error when creating a DwC archive on your database.
  • Log into the database from the original issue DwCA exporting fails #7856
  • There is no error when creating a DwC archive in the database from the issue.

@alesan99 alesan99 added this to the 7.12.0 milestone Mar 27, 2026
@github-project-automation github-project-automation bot moved this to 📋Back Log in General Tester Board Mar 27, 2026
@alesan99 alesan99 changed the base branch from main to v7.12.0-prerelease March 27, 2026 14:17
@alesan99
Copy link
Copy Markdown
Contributor Author

NOTE: I set the PR to be merged into v7.12.0-prerelease. I can change it to main if that's better.

@alesan99 alesan99 requested review from a team March 27, 2026 20:25
Copy link
Copy Markdown
Collaborator

@emenslin emenslin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • There is no error when creating a DwC archive on your database.
  • There is no error when creating a DwC archive in the database from the issue.

Looks good, DwcA is exported without error!

@emenslin emenslin requested a review from a team March 30, 2026 14:26
Copy link
Copy Markdown
Collaborator

@bhumikaguptaa bhumikaguptaa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • There is no error when creating a DwC archive on your database.
  • There is no error when creating a DwC archive in the database from the issue.

--
Works as expected, I didn't come across any error.

Copy link
Copy Markdown
Contributor

@Iwantexpresso Iwantexpresso left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I get the following error:
3ckH2Lef.txt

This was tested on CalvertMarine museum Db and I'm currently attempting to test this on the saiab database

@github-project-automation github-project-automation bot moved this from 📋Back Log to Dev Attention Needed in General Tester Board Mar 30, 2026
Copy link
Copy Markdown
Member

@grantfitzsimmons grantfitzsimmons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • There is no error when creating a DwC archive on your database.
  • Log into the database from the original issue #7856
  • There is no error when creating a DwC archive in the database from the issue.

These testing instructions do not seem very robust, but I've verified on 3 databases, including the original, that exporting is working 😄

@grantfitzsimmons grantfitzsimmons merged commit d8a23b0 into v7.12.0-prerelease Mar 30, 2026
14 checks passed
@grantfitzsimmons grantfitzsimmons deleted the issue-7856 branch March 30, 2026 18:37
@github-project-automation github-project-automation bot moved this from Dev Attention Needed to ✅Done in General Tester Board Mar 30, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: ✅Done

Development

Successfully merging this pull request may close these issues.

DwCA exporting fails

6 participants