Add ability to hide "Archival History" field from public view
|Assignee:||José Raddaoui Marín||% Done:|
|Category:||Usability||Estimated time:||2.00 hours|
|Target version:||Release 2.1.0|
|Google Code Legacy ID:||Tested version:|
We should implement this via the existing "Visible elements" module, I think.
The field in question:
ISAD: Context Area > Archival history.
RAD: Archival description area > Custodial history.
In DACS the field maps to: Acquisition and Appraisal elements > Custodial history. But we don't currently have a Visible elements module for the DACS template.
#1 Updated by Tim Hutchinson over 8 years ago
I found the thread where we scoped out the visible elements module:
This confirms my recollection that we specifically excluded archival/custodial history since it's considered mandatory at the highest level of description.
In RAD, material acquired directly from the donor is supposed to be recorded in immediate source of acquisition, although it is common to see these elements combined. In ISAD, that's an optional rule. Custodial history should really provide context about the history of the records as opposed to confidential information about a donor. Immediate source of acquisition is already available on the visible elements module.
I can certainly see why the request comes up, but I wanted to provide that background.
#2 Updated by Dan Gillean over 8 years ago
Your points are sound, however: this is the public-facing ticket for a feature request being developed for a client, which will become available to all users. The work is being done for the ISAD template, though we will make the option available to the RAD template as well - it will be up to users to determine whether and how they use it.
#4 Updated by Tim Hutchinson over 8 years ago
Radda, our developer just pulled down the latest version and we're getting a 500 error on the visible elements module. I'm guessing it's related to your current work on this issue? the stack trace says:
500 | Internal Server Error | InvalidArgumentException
Widget "isad_archival_history" does not exist.
We should be able to work around this for now, but FYI just in case.
#7 Updated by Tim Hutchinson over 8 years ago
That did the trick.
I'm probably jumping the gun since you haven't marked this for review, but I just wanted to suggest that the archival history field be listed first (to reflect the order of the elements in the templates); and as you noted above, in the RAD template the correct label is Custodial history.
#11 Updated by Tim Hutchinson over 7 years ago
I think this one just needs QA - it was added to the visible elements module some time ago.