Bug #9254

Public option breaks EAD export in some cases

Added by José Raddaoui Marín almost 5 years ago. Updated almost 5 years ago.

Status:VerifiedStart date:12/17/2015
Priority:HighDue date:
Assignee:Dan Gillean% Done:

0%

Category:Import/Export
Target version:Release 2.2.1
Google Code Legacy ID: Tested version:2.2
Sponsored:No Requires documentation:

Description

When a description is exported using the EAD XML export as a public user, or via cli export or generating finding aids with the public option, draft descriptions are removed from the hierarchy exported.

If the last child of a description is draft, the current method we use to create the nested <c> tags in the EAD file doesn't work, leaving tags openned and causing the export to fail.


Related issues

Related to Access to Memory (AtoM) - Feature #7743: Add option to export EAD like public user using CLI Verified 12/19/2014

History

#2 Updated by José Raddaoui Marín almost 5 years ago

  • Related to Feature #7743: Add option to export EAD like public user using CLI added

#3 Updated by José Raddaoui Marín almost 5 years ago

  • Status changed from New to Code Review
  • Assignee changed from José Raddaoui Marín to Nick Wilkinson

Ready for code review: PR 259

#4 Updated by Nick Wilkinson almost 5 years ago

  • Assignee changed from Nick Wilkinson to Jesús García Crespo

Hi Jesus, can you take a look at this for CR?

#5 Updated by Nick Wilkinson almost 5 years ago

  • Assignee changed from Jesús García Crespo to Mike Cantelon

#6 Updated by Mike Cantelon almost 5 years ago

  • Status changed from Code Review to Feedback
  • Assignee changed from Mike Cantelon to José Raddaoui Marín

The code looks good to me!

#7 Updated by José Raddaoui Marín almost 5 years ago

  • Status changed from Feedback to QA/Review
  • Assignee changed from José Raddaoui Marín to Dan Gillean

Merged in qa/2.3.x and stable/2.2.x

#8 Updated by Sara Allain almost 5 years ago

Tested successfully using a few different scenarios on http://qa-22x.test.artefactual.com/george-gale-fonds, looks good!

#9 Updated by Sara Allain almost 5 years ago

  • Status changed from QA/Review to Verified

Also available in: Atom PDF