Merged
Conversation
nagios check
Spaces were incorrectly used in the concatination of the metadata standards information. This removes them on creation. It does not address the issue that there is already data saved incorrectly in production. WVU is not currently using the metadata information in production. To mitigate this issue the best course of action will be to open all the forms that have metadata standards defined and save them. This should save the data correctly and not require aditional modifications.
…tandards Remove spaces around metadata standards
Retrieve an object with its IDNO, instead of the system ObjectID
This is provided so that import scripts for other systems ca update MFCS with regards to where the object is viewable by the public.
forms class file should be broken up. It is currently handling two distinct functions 1. building and taking web form submissions 1. handling everything about a "form" A place holder file has been created, called web_forms.php to support the refactoring
This is essential when getting all objects on systems with limited memory. Cache drastically speeds up many operations, but in some cases is not required.
Accepts parameter to exclude metadata objects
1. Add ability to add a public release select box to object forms. 1. Add ability to set the default value of yes/no to public release select 1. add public release to the object create/update forms Note: when this gets merged into master someone will have to update all of the forms and the export forms. Any form with "publicRelease" will need that removed from the form. Any exports that use public release will need to be updated.
This is required because some fields may be tagged for public release, to a local repository, but should not be included in an OAI release. this could be the case, especially, if there are multiple digital object fields. The public release can be utilized to export all digital objects, but then be hidden from OAI and a single URL can be provided for the item.
Contributor
|
Lets talk about this on Thursday. |
Contributor
|
I'm going to merge this in, but we need to answer the order of some of these items. I'm assuming that this will also be tested on MFCS dev before going to the production area. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
As soon as this is deployed into production several things need to take place.