Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix update function for 'Language and script notes'. (#13657) #1664

Merged
merged 1 commit into from
Oct 4, 2023

Conversation

melaniekung
Copy link
Contributor

@melaniekung melaniekung commented Sep 23, 2023

Bug was due to the 'Language and script notes' field being specific to the ISAD, DACS, and RAD templates only, so they were handled differently and was not checking to see if the string/field was updated.

Added a function to check the stored memory to see if notes were updated when duplicating an archival description. This was created specifically for the 'Language and script notes' field, defined by the QubitTerm::LANGUAGE_NOTE_ID.

@melaniekung melaniekung added Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result. work-in-progress labels Sep 23, 2023
@melaniekung melaniekung added this to the 2.8.0 milestone Sep 23, 2023
@melaniekung melaniekung self-assigned this Sep 23, 2023
@melaniekung melaniekung force-pushed the dev/issue-13657-edit-lang-script-notes branch 2 times, most recently from f26368b to 9f7b325 Compare September 24, 2023 04:29
@melaniekung melaniekung changed the title Fix save for Lang and script notes on duplication. (#13657) [WIP] Fix update function for 'Language and script notes'. (#13657) Sep 24, 2023
@melaniekung melaniekung requested a review from a team September 24, 2023 05:28
@melaniekung melaniekung force-pushed the dev/issue-13657-edit-lang-script-notes branch 4 times, most recently from f356053 to fec893f Compare September 28, 2023 19:55
lib/model/QubitObject.php Outdated Show resolved Hide resolved
lib/model/QubitObject.php Show resolved Hide resolved
@melaniekung melaniekung force-pushed the dev/issue-13657-edit-lang-script-notes branch from fec893f to c438925 Compare October 4, 2023 15:22
@melaniekung melaniekung merged commit 7e25350 into qa/2.x Oct 4, 2023
6 checks passed
@melaniekung melaniekung deleted the dev/issue-13657-edit-lang-script-notes branch October 4, 2023 16:41
@melaniekung melaniekung restored the dev/issue-13657-edit-lang-script-notes branch October 4, 2023 17:26
@melaniekung melaniekung deleted the dev/issue-13657-edit-lang-script-notes branch October 4, 2023 17:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Type: bug A flaw in the code that causes the software to produce an incorrect or unexpected result.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants